Skip to content

feat: define OAuth connection validation contract - #219

Merged
vishu-bh merged 1 commit into
mainfrom
chore/5722-openapi-contract
Nov 4, 2026
Merged

vishu-bh merged 1 commit into
mainfrom
chore/5722-openapi-contract

Conversation

@vishu-bh

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

Copy link
Copy Markdown
Contributor

Define a small handwritten TypeScript compatibility contract for post-authorization OAuth connection validation from IBM/mcp-context-forge#7343. The request fixes the exact gateway ID, saved URL, and validation mode. The response reuses existing generated handshake metadata and adds stage outcomes, failure information, and reauthorization guidance.

This PR does not change openapi.json or commit generated files. It avoids overlapping the backend's automated schema refresh. Remove the compatibility contract once the generated schema includes these fields. Missing stage evidence remains representable so older backends can be rejected safely by the caller.

Related to IBM/mcp-context-forge#5722. Backend support for the new validation mode remains a runtime prerequisite.

Validation: production build against types regenerated from the unchanged main schema. The updated shared validation and complete UI stack pass their builds, lint, formatting, full frontend coverage suites, and OAuth browser tests.

Native GitHub stack: #223, rooted at main.

  1. feat: define OAuth connection validation contract #219 — Handwritten OAuth compatibility contract; base: main.
  2. feat: add shared OAuth connection validation #220 — Shared validation; base: feat: define OAuth connection validation contract #219's branch.
  3. feat: surface OAuth validation on MCP Servers #221 — MCP Servers UI; base: feat: add shared OAuth connection validation #220's branch.
  4. feat: surface OAuth validation on Catalog cards #222 — Catalog UI; base: feat: surface OAuth validation on MCP Servers #221's branch.

#218 contains the independent E2E typing fix. These PRs replace #216.

@marekdano marekdano left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@vishu-bh
vishu-bh force-pushed the chore/5722-openapi-contract branch from 070b9a6 to 7dd1e03 Compare November 3, 2026 14:34
@vishu-bh vishu-bh changed the title chore: refresh OAuth post-authorization API contract feat: define OAuth connection validation contract Nov 3, 2026
Signed-off-by: Vishu Bhatnagar <vishu.bhatnagar@ibm.com>
@gcgoncalves
gcgoncalves force-pushed the chore/5722-openapi-contract branch from 7dd1e03 to a2e9544 Compare November 4, 2026 09:09
@vishu-bh
vishu-bh merged commit fc36551 into main Nov 4, 2026
5 checks passed
@vishu-bh
vishu-bh deleted the chore/5722-openapi-contract branch November 4, 2026 13:15
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.

3 participants