[artickc]

Summary

Kiro (AWS CodeWhisperer) currently tops out at Claude Opus 4.7 in the provider registry, even though:

• Claude Opus 4.8 is already defined and served by the claude (Claude Code) provider, and
• the Kiro executor's openai-to-kiro translator passes the requested model id straight through to CodeWhisperer (userInputMessage.modelId), so no special model mapping is required for a new Claude model.

This PR exposes claude-opus-4.8 on the kiro provider so clients can select it via kiro/claude-opus-4.8 (and kr/claude-opus-4.8), consistent with how claude-opus-4.7 is already wired.

Changes

• open-sse/config/providerRegistry.ts — add claude-opus-4.8 to the kiro models list (1M context, 128k max output), placed above claude-opus-4.7.
• src/shared/constants/pricing.ts — add claude-opus-4.8 to the kiro pricing block at Kiro's standard Opus rate ($15 / $75 per 1M, matching the existing claude-opus-4.6 Kiro entry). Also adds the previously-missing claude-opus-4.7 entry to the same block so its usage cost is no longer $0.
• tests/unit/catalog-updates-v3x.test.ts — new assertion that the Kiro catalog exposes claude-opus-4.8 (with the expected context/output) and that the Kiro pricing block covers it.

Why the pricing rate

The new Kiro entries follow the rate already used by the existing kiro → claude-opus-4.6 entry ($15/$75), keeping the Kiro channel internally consistent. Adjust if Kiro should mirror the Claude Code (cc) Opus rate instead.

Testing

node --import tsx --test tests/unit/catalog-updates-v3x.test.ts

All tests pass, including the new Kiro catalog exposes Claude Opus 4.8 alongside 4.7 with matching pricing case.

[gemini-code-assist]

Code Review

This pull request adds the claude-opus-4.8 model to the Kiro provider registry, configures pricing for both claude-opus-4.8 and claude-opus-4.7, and adds a unit test to verify their presence and pricing. The review feedback suggests adding an alias mapping for claude-opus-4-8 to ensure backward compatibility and proper resolution, and expanding the unit test to assert the pricing configuration for claude-opus-4.7 to prevent future regressions.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

While adding claude-opus-4.8 to the Kiro model list is correct, please note that clients requesting the hyphenated version kiro/claude-opus-4-8 (consistent with the claude provider's claude-opus-4-8 ID) will fail to resolve to the dotted claude-opus-4.8 ID.\n\nTo ensure proper resolution and backward compatibility, you must also add the alias mapping to PROVIDER_MODEL_ALIASES.kiro in open-sse/services/model.ts:\n\ntypescript\n"claude-opus-4-8": "claude-opus-4.8",\n\n\nThis matches how claude-opus-4-7 and claude-opus-4-6 are currently mapped.

[gemini-code-assist]

Since this PR also adds the previously-missing claude-opus-4.7 pricing entry to prevent $0 usage costs, it would be highly beneficial to assert its presence in the test suite as well to prevent future regressions.

Suggested change

	// Pricing for the Kiro channel must cover the new model so usage cost is non-zero.
	const kiroPricing = (DEFAULT_PRICING as Record<string, Record<string, unknown>>).kiro;
	assert.ok(kiroPricing["claude-opus-4.8"], "kiro pricing must include claude-opus-4.8");
	// Pricing for the Kiro channel must cover the new models so usage cost is non-zero.\n const kiroPricing = (DEFAULT_PRICING as Record<string, Record<string, unknown>>).kiro;\n assert.ok(kiroPricing["claude-opus-4.8"], "kiro pricing must include claude-opus-4.8");\n assert.ok(kiroPricing["claude-opus-4.7"], "kiro pricing must include claude-opus-4.7");

[kilo-code-bot]

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
open-sse/services/model.ts	75-81	The kiro provider model aliases are missing the hyphen-notation variant claude-opus-4-8. Other models like claude-opus-4-7 and claude-opus-4-6 have hyphen-notation aliases for backward compatibility, but claude-opus-4-8 does not. This could cause requests using kiro/claude-opus-4-8 to fail validation.

Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File	Line	Issue
tests/unit/catalog-updates-v3x.test.ts	63	The test only checks that pricing exists for claude-opus-4.8 but does not validate the pricing values (input/output/cached/reasoning) or that claude-opus-4.7 pricing values are correct. The existing reviewer (gemini-code-assist) already noted this in their comment.
src/shared/constants/pricing.ts	1321-1326	The claude-opus-4.6 pricing in the kiro block uses $15/$75 (reasoning: $75) which differs from CLAUDE_OPUS_46_PRICING ($5/$25, reasoning: $37.5) and the cc provider ($5/$25, reasoning: $25). The PR author noted this may be intentional for Kiro's internal consistency, but this pricing delta should be verified against actual Kiro pricing.

Files Reviewed (3 files)

• open-sse/config/providerRegistry.ts - 1 change (model addition)
• src/shared/constants/pricing.ts - 2 changes (pricing entries)
• tests/unit/catalog-updates-v3x.test.ts - 1 change (new test)

Note: A previous review by gemini-code-assist already identified the missing claude-opus-4-8 alias in model.ts and suggested expanding the test to validate pricing values for claude-opus-4.7. These align with my findings.

Fix these issues in Kilo Cloud

---

_{Reviewed by laguna-m.1-20260312:free · 2,878,921 tokens}