Skip to content

fix: harden OpenAI and Anthropic model params - #4

Closed
guillaumegay13 wants to merge 1 commit into
mainfrom
codex/provider-param-smoke
Closed

fix: harden OpenAI and Anthropic model params#4
guillaumegay13 wants to merge 1 commit into
mainfrom
codex/provider-param-smoke

Conversation

@guillaumegay13

Copy link
Copy Markdown
Collaborator

Summary

  • add an explicit live provider smoke harness for MPS rows (npm run smoke:params)
  • correct OpenAI API-key rows against live Chat Completions behavior
  • correct Anthropic API-key rows to provider-native/current model IDs and live-tested applicability rules

Live smoke validation

  • npm run smoke:params -- --provider openai --auth-type api_key --keys-file <local test keys>: 141 passed, 0 failed
  • npm run smoke:params -- --provider anthropic --auth-type api_key --keys-file <local test keys>: 139 passed, 0 failed

Local validation

  • npm run validate
  • npm test
  • npm run typecheck
  • npm run lint
  • npm run build
  • npx prettier --check .
  • git diff --check

@vercel

vercel Bot commented May 21, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
modelparameters-dev Ready Ready Preview May 21, 2026 7:21am

Request Review

@guillaumegay13

Copy link
Copy Markdown
Collaborator Author

Closing this in favor of a smaller PR that contains only the OpenAI/Anthropic catalog fixes. The live smoke harness will be split out separately.

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