Skip to content

fix: refine Google Gemini parameter applicability#16

Merged
guillaumegay13 merged 1 commit into
mnfst:mainfrom
guillaumegay13:codex/google-applicability
May 21, 2026
Merged

fix: refine Google Gemini parameter applicability#16
guillaumegay13 merged 1 commit into
mnfst:mainfrom
guillaumegay13:codex/google-applicability

Conversation

@guillaumegay13
Copy link
Copy Markdown
Collaborator

Summary

  • remove Google Gemini presence/frequency penalty params because live generateContent rejects them for all four cataloged Gemini models
  • relax generationConfig.topK lower bound from 1 to 0, matching live validation (topK: 0 accepted, topK: -1 rejected as nonnegative)

Verification

  • Official docs checked today:
  • Live Google API matrix with key from .env.keys:
    • getModel confirmed gemini-3.5-flash exists today as 3.5-flash-05-2026
    • sampling params without penalties are accepted with thinking enabled on gemini-3.5-flash, gemini-2.5-pro, gemini-2.5-flash, and gemini-2.5-flash-lite
    • presencePenalty: 0.1 and frequencyPenalty: 0.1 are rejected on all four with Penalty is not enabled
    • all listed post-patch params were sent together per YAML row and returned HTTP 200 for all four models
  • npm run validate
  • npm test
  • npm run typecheck
  • npm run lint
  • npm run build
  • npm run guard:params
  • npx prettier --check .
  • git diff --check

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

@guillaumegay13 is attempting to deploy a commit to the Manifest Team on Vercel.

A member of the Team first needs to authorize it.

@guillaumegay13 guillaumegay13 added the allow-param-removal Maintainer override: allow a parameter removal label May 21, 2026
@guillaumegay13 guillaumegay13 merged commit ec03f07 into mnfst:main May 21, 2026
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

allow-param-removal Maintainer override: allow a parameter removal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant