Skip to content

refactor(tool): use key defaults in parameter schemas#23319

Open
kitlangton wants to merge 1 commit intokit/tools-schemafrom
kit/schema-native-cleanup-ts
Open

refactor(tool): use key defaults in parameter schemas#23319
kitlangton wants to merge 1 commit intokit/tools-schemafrom
kit/schema-native-cleanup-ts

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented Apr 18, 2026

Summary

  • switch codesearch, webfetch, and websearch defaults to key defaults and finite numbers where that better matches native Effect JSON Schema
  • teach the effect-zod bridge to emit defaults for key-defaulted struct fields so LLM-facing schemas stay aligned
  • add focused tests for native-schema invariants and key-default bridge behavior

Verification

  • bun run test test/util/effect-zod.test.ts test/tool/parameters.test.ts test/tool/tool-define.test.ts
  • bun typecheck

@kitlangton kitlangton changed the title refactor(tool): clean up native-friendly parameter schemas refactor(tool): use key defaults in parameter schemas Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant