Skip to content

fix(kiro): seed GPT-5.6 Sol/Terra/Luna from official Kiro catalog#141

Merged
lidge-jun merged 1 commit into
lidge-jun:mainfrom
ost527:fix/kiro-gpt-5.6-models
Jul 17, 2026
Merged

fix(kiro): seed GPT-5.6 Sol/Terra/Luna from official Kiro catalog#141
lidge-jun merged 1 commit into
lidge-jun:mainfrom
ost527:fix/kiro-gpt-5.6-models

Conversation

@ost527

@ost527 ost527 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Seed Kiro's official GPT-5.6 tiers (gpt-5.6-sol / terra / luna, 272K context) into the static KIRO_MODELS catalog so Codex shows kiro/gpt-5.6-*.
  • Set liveModels: false for the Kiro provider — CodeWhisperer runtime is not OpenAI-style GET /models, so a spurious live response must not drop seeded catalog ids.
  • Cover id normalization + registry context windows in tests.

Context

Kiro published GPT-5.6 Sol/Terra/Luna on 2026-07-13 (docs). opencodex still shipped the pre-GPT static list, so OAuth reconcile rewrote user configs and routed models never appeared as kiro/gpt-5.6-*.

Test plan

  • bun test tests/kiro-adapter.test.ts -t "normalizes versioned|per-model context|kiro catalog is static|registry includes"
  • Local install patch: after restart, /v1/models lists kiro/gpt-5.6-sol, kiro/gpt-5.6-terra, kiro/gpt-5.6-luna
  • CI full suite on this PR

Kiro added OpenAI GPT-5.6 tiers (Sol/Terra/Luna, 272K context) on
2026-07-13. The static KIRO_MODELS list lagged, so routed
kiro/gpt-5.6-* never appeared in Codex. Seed the three models with
documented context windows, pin liveModels=false (CodeWhisperer has no
OpenAI-style GET /models), and cover normalize + registry in tests.
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@lidge-jun
lidge-jun merged commit 535bed7 into lidge-jun:main Jul 17, 2026
lidge-jun added a commit that referenced this pull request Jul 17, 2026
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