Skip to content

Fix Den MCP tool input schemas#1634

Merged
src-opn merged 1 commit intodevfrom
check-mcp-post-commands
May 1, 2026
Merged

Fix Den MCP tool input schemas#1634
src-opn merged 1 commit intodevfrom
check-mcp-post-commands

Conversation

@OmarMcAdam
Copy link
Copy Markdown
Collaborator

Summary

  • Generate per-operation MCP input schemas from OpenAPI path/query parameters instead of exposing a generic input envelope.
  • Add tool-description guidance that tells agents to put URL params under path, query params under query, and JSON fields under body.
  • Preserve the existing MCP invoker contract while making path parameters like teamId explicit and required.

Validation

  • pnpm --filter @openwork-ee/den-api exec tsc -p tsconfig.json --noEmit
  • pnpm --filter @openwork-ee/den-api build
  • Validated real generated OpenAPI catalog for patchV1TeamsByTeamId: { path: { teamId }, body: { name } } accepted and { body: { parameters, requestBody } } rejected.
  • Validated invokeMcpOperation() builds /v1/teams/{teamId} from toolInput.path.teamId and returns Missing path parameter: teamId for wrapper-shaped input.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented May 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openwork-app Ready Ready Preview, Comment May 1, 2026 2:55pm
openwork-den Ready Ready Preview, Comment May 1, 2026 2:55pm
openwork-den-worker-proxy Ready Ready Preview, Comment May 1, 2026 2:55pm
openwork-landing Ready Ready Preview, Comment, Open in v0 May 1, 2026 2:55pm
openwork-share Ready Ready Preview, Comment May 1, 2026 2:55pm

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

The following comment was made by an LLM, it may be inaccurate:

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