Skip to content

Add You.com Inference Router to AI gateway benchmarks - #534

Open
zairahms wants to merge 3 commits into
computesdk:masterfrom
zairahms:add-you-com-inference-router
Open

zairahms wants to merge 3 commits into
computesdk:masterfrom
zairahms:add-you-com-inference-router

Conversation

@zairahms

@zairahms zairahms commented Oct 22, 2026 •

Copy link
Copy Markdown

Add You.com Inference Router to AI gateway benchmarks

Adds you-dot-com (https://you.com/docs/guides/inference-router/quickstart) as an AI gateway participant.

What's included

Benchmark Model Route
Anthropic family claude-sonnet-5 /v1/messages + anthropic-version
OpenAI family gpt-5.6-luna /v1/chat/completions (router has no Responses API)
Kimi family kimi-k3 /v1/chat/completions
Model index — GET /v1/models

Not added to the Gemini family: the router's catalog has no Gemini model.

Model change for the Anthropic and OpenAI families

The router doesn't serve older model: Claude Haiku 4.5 or gpt-5.4-mini, so to keep every participant on the same model:

  • Anthropic family: Claude Haiku 4.5 → Claude Sonnet 5
  • OpenAI family: gpt-5.4-mini → gpt-5.6-luna

Existing results in results/ai-gateway-latency/anthropic/ and .../openai/ are for the old models, so trend lines across this change won't be like-for-like.

Other changes

  • YOU_API_KEY added to .env.example, provider-vars.json, and the AI gateway workflow's vault-secret pattern
  • you-dot-com added to the workflow provider choices, PR-comment labels, SVG labels, and bench:*:you-dot-com scripts
  • The router's x-ydc-inference-timing-* response headers are captured in each iteration's receipts
  • AI_GATEWAYS.md updated for the model change, plus a You.com section

Testing

Ran you-dot-com alongside anthropic-direct and openai-direct locally with --iterations 2 --dry-run. All requests succeeded on the Anthropic, OpenAI and Kimi routes. No results are committed in this PR.

For maintainers

  • Add YOU_API_KEY to the vault
  • Run the AI Gateway Benchmark workflow (workflow_dispatch, family anthropic, openai, kimi) to produce results with the full roster. This also checks the other gateways' new Sonnet 5 / gpt-5.6-luna model ids, which haven't been run yet.

@zairahms
zairahms marked this pull request as ready for review October 22, 2026 22:22
@open-cla

open-cla Bot commented Oct 22, 2026

Copy link
Copy Markdown

Contributor License Agreement

The following contributors need CLA coverage:

Review and sign the CLA

@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 found 1 potential issue.

1 flag not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

Devin Review

requiredEnvVars: ['YOU_API_KEY'],
host: 'api.you.com',
modelsPath: '/v1/models',
modelListFormat: 'openai',

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.

🟡 You.com context limits disappear from index

When You.com lists models with max_input_tokens, modelListFormat: 'openai' produces entries without context limits. extractContextLength ignores that field, so the model index omits those limits.

Learn more

The model-index benchmark fetches each configured gateway's model list and normalizes entries into shared fields. You.com's model objects expose max_input_tokens for their input limit, but extractContextLength recognizes only four other field names. With modelListFormat: 'openai', normalizeModels calls normalizeModel and stores contextLength as undefined for these entries. The index omits a limit available in the response.

Example: A listed You.com model with { "id": "kimi-k3", "max_input_tokens": 128000 } becomes an index entry with no contextLength instead of 128000.

Recommended fix: Teach extractContextLength to read max_input_tokens, and validate the normalized You.com model list against an example response.

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

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