Skip to content

docs: document uppercase image_config.image_size values - #611

Open
LukasParke wants to merge 1 commit into
mainfrom
claude/image-size-casing-269
Open

LukasParke wants to merge 1 commit into
mainfrom
claude/image-size-casing-269

Conversation

@LukasParke

@LukasParke LukasParke commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Description

Related to #269. Please retest with the uppercase values below and tell us if image_size still has no effect.

I traced the request path for this issue. The SDK already forwards image_config unchanged, through either providerOptions.openrouter or extraBody. The OpenRouter Gemini adapter already uppercases image_size before it checks the value against 0.5K/1K/2K/4K. So lowercase '4k' reaches Google as '4K', and letter casing is not the cause. 0.5K only works on the Gemini 3.1 Flash Image models. Other models return a 400 error.

Because the backend already handles casing, this PR changes only tests and docs:

The e2e test was not run locally because no API key was available.

Checklist

  • I have run pnpm stylecheck and pnpm typecheck
  • I have run pnpm test and all tests pass
  • I have added tests for my changes (if applicable)
  • I have updated documentation (if applicable)

Changeset

  • I have run pnpm changeset to create a changeset file (--empty, docs/test only)

🤖 Generated with Claude Code


Devin Review

Use the documented uppercase tiers (1K/2K/4K) in the issue #269 e2e test
and add a README example for passing image_config via providerOptions.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

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