Skip to content

fix: migrate to nested _meta.ui.resourceUri format#604

Open
AngeloGiacco wants to merge 1 commit intomodelcontextprotocol:mainfrom
AngeloGiacco:fix/use-nested-meta-uri-format
Open

fix: migrate to nested _meta.ui.resourceUri format#604
AngeloGiacco wants to merge 1 commit intomodelcontextprotocol:mainfrom
AngeloGiacco:fix/use-nested-meta-uri-format

Conversation

@AngeloGiacco
Copy link
Copy Markdown

Summary

  • Migrate remaining examples (qr-server, transcript-server, map-server) from the deprecated flat _meta["ui/resourceUri"] format to the newer nested _meta.ui.resourceUri format
  • Update registerAppTool JSDoc example in src/server/index.ts to show the new format
  • Remove unused RESOURCE_URI_META_KEY imports from migrated examples

This ensures mobile client compatibility, which only supports the nested meta schema URI format.

Test plan

  • Verify examples still render correctly in the basic-host
  • Confirm npm run build:all succeeds

🤖 Generated with Claude Code

Copy link
Copy Markdown
Contributor

@ochafik ochafik left a comment

Choose a reason for hiding this comment

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

Thanks @AngeloGiacco ! Could you please resolve the conflicts before this can be merged?

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.

2 participants