Skip to content

fix: strip unsupported regex patterns from tool schemas#58

Open
ebbsanchez wants to merge 1 commit into
Dokploy:mainfrom
ebbsanchez:codex/strip-lookaround-schema-patterns
Open

fix: strip unsupported regex patterns from tool schemas#58
ebbsanchez wants to merge 1 commit into
Dokploy:mainfrom
ebbsanchez:codex/strip-lookaround-schema-patterns

Conversation

@ebbsanchez

Copy link
Copy Markdown

Summary

  • remove provider-incompatible regex lookaround pattern fields from MCP tools/list JSON schemas
  • keep the existing Zod runtime validators unchanged for actual tool calls
  • add a regression test that walks every exposed tool schema and fails on lookaround patterns

Fixes #45.

Verification

  • npm exec --yes pnpm@10 -- install --frozen-lockfile
  • npm exec --yes pnpm@10 -- test src/server.test.ts
  • npm exec --yes pnpm@10 -- format
  • npm exec --yes pnpm@10 -- lint
  • npm exec --yes pnpm@10 -- test
  • npm exec --yes pnpm@10 -- type-check
  • git diff --check

Note: biome check still reports the existing ResponseFormatter static-only class warning, but exits successfully after formatting.

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.

fix(schema): regex lookaround in bitbucketEmail/letsEncryptEmail/email patterns breaks OpenAI / Sonnet-strict validators

2 participants