Skip to content

Enable GPT-5.6 OpenRouter promotion - #4814

Merged
chrarnoldus merged 4 commits into
mainfrom
enable-openrouter-gpt56-promo
Jul 28, 2026
Merged

Enable GPT-5.6 OpenRouter promotion#4814
chrarnoldus merged 4 commits into
mainfrom
enable-openrouter-gpt56-promo

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • add an enabled AI gateway promotion flag for the four GPT-5.6 Terra/Luna variants
  • keep funded requests for those models on OpenRouter and preserve discounted catalog pricing
  • label promoted model names as 50% off

Verification

  • pnpm --filter web exec jest --runInBand src/lib/ai-gateway/providers/openrouter/index.test.ts src/lib/ai-gateway/providers/vercel/index.test.ts
  • pnpm --filter web typecheck
  • pnpm --filter web lint
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Jul 28, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The incremental change adds a dynamic promo-discount percentage to formatName, routes GPT-5.6 promo models to the OpenAI provider via OpenRouter, and formally documents pricing.discount in OpenRouterModelSchema; the schema update resolves the previously flagged discount-leak concern, and no new correctness, security, or breaking-change issues were found in the changed lines.

Files Reviewed (5 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts
  • apps/web/src/lib/organizations/organization-types.ts
Previous Review Summaries (3 snapshots, latest commit ab2f0a2)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit ab2f0a2)

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

Removing the explicit discount stripping in getModelDisplayPricing for GPT-5.6 promo models leaks the internal pricing.discount field into the catalog response, deviating from the documented OpenRouterModel.pricing schema.

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/lib/ai-gateway/providers/openrouter/index.ts 128 Promo pricing now returns pricing unmodified, leaking the internal discount field into the public catalog response instead of stripping it as before
Files Reviewed (4 files)
  • apps/web/src/lib/ai-gateway/providers/openai.ts - 0 issues
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts - 0 issues
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.ts - 1 issue
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts - 0 issues
  • apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts - 0 issues

Fix these issues in Kilo Cloud

Previous review (commit 8c65ea8)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Incremental update simplifies formatName so promoted GPT-5.6 models return only the (50% off) label (skipping $$$$/new/retires suffixes), adds a clarifying comment on the BYOK-disable flag, and updates the test to match the new early-return behavior; no correctness, security, or breaking-change issues found in the changed lines.

Files Reviewed (3 files)
  • apps/web/src/lib/ai-gateway/providers/openai.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts

Previous review (commit 8ba090c)

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the GPT-5.6 OpenRouter promo flag, pricing-preservation, and Vercel-routing bypass logic; all changed code is consistent, covered by new tests, and no correctness, security, or breaking-change issues were found.

Files Reviewed (5 files)
  • apps/web/src/lib/ai-gateway/providers/openai.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.ts
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.ts
  • apps/web/src/lib/ai-gateway/providers/vercel/index.test.ts

Reviewed by claude-sonnet-5 · Input: 30 · Output: 8.8K · Cached: 689.9K

Review guidance: REVIEW.md from base branch main

Comment thread apps/web/src/lib/ai-gateway/providers/openrouter/index.ts
@chrarnoldus
chrarnoldus merged commit fbdaa0a into main Jul 28, 2026
15 checks passed
@chrarnoldus
chrarnoldus deleted the enable-openrouter-gpt56-promo branch July 28, 2026 10:32
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.

2 participants