Skip to content

fix: keep injected MCP parameters optional - #78

Open
dajiaohuang wants to merge 1 commit into
agentcathq:mainfrom
dajiaohuang:fix/mcp-app-optional-injected-fields
Open

dajiaohuang wants to merge 1 commit into
agentcathq:mainfrom
dajiaohuang:fix/mcp-app-optional-injected-fields

Conversation

@dajiaohuang

@dajiaohuang dajiaohuang commented Oct 22, 2026 •

Copy link
Copy Markdown

Summary

Verification

  • pnpm run lint, pnpm run typecheck, and pnpm run build passed.
  • v1 tests: 806 passed, 2 skipped. The ESM consumer smoke test could not spawn pnpm on Windows (spawnSync pnpm ENOENT); all other v1 tests passed.
  • v2 tests: 105 passed.
  • Targeted local HTTP strict-schema checks: 5 passed. Full HTTP matrix attempted; 25 cases failed with local connection resets/bad port, so the full matrix is not green.
  • No CI queried or awaited. HTTP tests use only ephemeral localhost servers; no analytics/provider or real MCP service was contacted.

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.

1 participant