[RHIDP-12903] Remove Bearer From MCP HEADERS for Lightspeed

[Jdubrick]

Hey, I just made a Pull Request!

• Removes Bearer prefix from MCP headers on calls to Lightspeed Core as they do not want it there

https://redhat.atlassian.net/browse/RHIDP-12903

✔️ Checklist

• A changeset describing the change and affected packages. (more info)
• Added or Updated documentation
• Tests for new functionality and regression tests for bug fixes
• Screenshots attached (for UI changes)

[rhdh-gh-app]

Changed Packages

Package Name	Package Path	Changeset Bump	Current Version
@red-hat-developer-hub/backstage-plugin-lightspeed-backend	workspaces/lightspeed/plugins/lightspeed-backend	patch	v1.4.0

[rhdh-qodo-merge]

Review Summary by Qodo

Remove Bearer prefix from MCP authorization headers

🐞 Bug fix

Walkthroughs

Description

• Removes Bearer prefix from MCP authorization headers
• Updates header format to send raw token instead
• Aligns with Lightspeed Core API requirements
• Includes test updates to reflect new header format

Diagram

flowchart LR
  A["MCP Header Configuration"] -->|Previously| B["Authorization: Bearer token"]
  A -->|Now| C["Authorization: token"]
  C --> D["Lightspeed Core API"]

Loading

File Changes

1. workspaces/lightspeed/plugins/lightspeed-backend/src/service/router.ts 🐞 Bug fix +1/-1

Remove Bearer prefix from MCP headers

• Removes Bearer prefix from MCP authorization header construction
• Changes header format from Bearer ${token} to ${token}
• Affects MCP server configuration header generation

workspaces/lightspeed/plugins/lightspeed-backend/src/service/router.ts

---

2. workspaces/lightspeed/plugins/lightspeed-backend/src/service/router.test.ts 🧪 Tests +3/-3

Update MCP header tests for Bearer removal

• Updates test expectations to match new header format without Bearer
• Modifies three test cases to expect raw tokens instead of Bearer-prefixed tokens
• Ensures test coverage aligns with implementation changes

workspaces/lightspeed/plugins/lightspeed-backend/src/service/router.test.ts

---

3. workspaces/lightspeed/.changeset/rotten-papayas-add.md 📝 Documentation +5/-0

Add changeset for Bearer header removal

• Adds changeset documenting the Bearer removal change
• Marks as patch version bump for lightspeed-backend plugin
• Documents compliance with LCORE standard

workspaces/lightspeed/.changeset/rotten-papayas-add.md

---

[rhdh-qodo-merge]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0) 📐 Spec deviations (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

ⓘ The new review experience is currently in Beta. Learn more

[maysunfaisal]

lgtm

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[yangcao77]

lgtm as @maysunfaisal has approved this PR