Skip to content

feat: test OAuth MCP connections after authorization - #216

Closed
vishu-bh wants to merge 1 commit into
mainfrom
feat/5722-post-authorization-test
Closed

vishu-bh wants to merge 1 commit into
mainfrom
feat/5722-post-authorization-test

Conversation

@vishu-bh

@vishu-bh vishu-bh commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Superseded by smaller PRs for review:

Merge the feature stack in the order #219 → #220 → #221 → #222. The split stack plus #218 exactly reproduces this PR. Each feature checkpoint passes its production build and full coverage suite; browser flows pass on the Servers and Catalog checkpoints.

Original description

OAuth authorization currently reloads token status and imports components without proving that the stored token can access tools. Add an explicit Test connection action with shared token, MCP negotiation, and tools-access results on MCP Servers, server details, and catalog cards.

Tests use the exact saved gateway ID and URL with validationMode: "post_authorization". The UI supports cancellation, retesting, reauthorization, permission checks, and stale-result protection. Structured callback errors receive localized guidance in English, Spanish, and Portuguese. Missing stage evidence cannot report success.

Related to IBM/mcp-context-forge#5722. Depends on IBM/mcp-context-forge#7343, which is still unmerged. The OpenAPI schema is pinned to backend revision d2da38b56be5e8e4bdf51db9b2c43d4c96036b62. Keep this PR in draft until the backend dependency lands; then refresh the schema and repeat contract checks against the merged revision.

Also fixes two existing standalone E2E TypeScript errors and records the verified permission-expression false positive in the secret-scan baseline.

Validation:

  • Frontend and BFF production builds; frontend, BFF, and E2E TypeScript checks; ESLint; Prettier; whitespace checks: passed.
  • Frontend: 3,852 tests passed, 1 skipped; all coverage thresholds passed.
  • BFF: 300 tests passed.
  • Playwright: 266 tests passed, 2 real Keycloak tests skipped in mocked-API mode.
  • Backend PR smoke tests: 6 passed against an isolated running gateway.
  • Repository secret-detection check: passed.

Signed-off-by: Vishu Bhatnagar <vishu.bhatnagar@ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant