Skip to content

Use AI Catalog for Server Card discovery#42

Merged
SamMorrowDrums merged 3 commits into
mainfrom
sammorrowdrums-remove-mcp-catalog
Jul 20, 2026
Merged

Use AI Catalog for Server Card discovery#42
SamMorrowDrums merged 3 commits into
mainfrom
sammorrowdrums-remove-mcp-catalog

Conversation

@SamMorrowDrums

@SamMorrowDrums SamMorrowDrums commented Jul 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • use AI Catalog at /.well-known/ai-catalog.json for domain-level Server Card discovery
  • select application/mcp-server-card+json entries and support both URL-referenced and inline cards
  • preserve the existing Server Card location, security, caching, and client-flow guidance

Validation

  • npm run check
  • npm run validate
  • npm run format:check

Closes #26

Closes #26

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: bc66c0fe-15a9-493d-81c7-258a6948e4f0
@SamMorrowDrums
SamMorrowDrums marked this pull request as draft July 18, 2026 20:52
Restore the existing discovery structure and explicitly surface the AI Catalog well-known path.

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: bc66c0fe-15a9-493d-81c7-258a6948e4f0
@SamMorrowDrums
SamMorrowDrums marked this pull request as ready for review July 18, 2026 21:37
@SamMorrowDrums
SamMorrowDrums requested a review from tadasant July 18, 2026 22:03
tadasant
tadasant previously approved these changes Jul 20, 2026

@tadasant tadasant left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lgtm just one nit!

Comment thread README.md Outdated
Co-authored-by: Tadas Antanavicius <[email protected]>
@SamMorrowDrums
SamMorrowDrums merged commit 1491bd2 into main Jul 20, 2026
2 checks passed
tadasant pushed a commit that referenced this pull request Jul 24, 2026
Several paragraphs were left unwrapped while the rest of the file wraps
at ~95 columns. Reflow them so the file is internally consistent.

The only line still over 100 is an atomic markdown link that cannot be
broken. Deliberately not adding a repo-wide prettier proseWrap setting
here: it would reflow discovery.md, which PR #42 is actively rewriting.
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.

Remove the MCP Catalog

2 participants