Skip to content

Conversation

@jonathannorris
Copy link
Member

@jonathannorris jonathannorris commented Aug 11, 2025

Summary

  • Fix: Ensure all array params in MCP tool JSON Schemas include items, resolving Copilot Chat validation error.
  • Add: src/mcp/utils/schema.ts with toJsonSchema and processToolConfig (normalizes array schemas).
  • Wire-up: Use schema processing in mcp-worker/src/index.ts and src/mcp/server.ts.
  • Tests: Added src/mcp/utils/schema.test.ts with snapshot src/mcp/utils/__snapshots__/schema.test.ts.snap.

Why

  • Copilot Chat failed tool validation with “tool parameters array type must have items” for tools like create_feature.

Impact

  • Developer experience: Tools validate and run in Copilot Chat.
  • No breaking changes: Runtime logic unchanged; only JSON Schema generation/normalization.

@jonathannorris jonathannorris requested a review from a team as a code owner August 11, 2025 01:02
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 11, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
devcycle-mcp-server 082440b Aug 11 2025, 06:15 PM

@jonathannorris jonathannorris deleted the fix-schema-issue branch August 11, 2025 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants