feat: adopt claude-opus-5 as judge and known model - #161
Conversation
📝 WalkthroughWalkthroughClaude Opus 4.8 references are replaced with Claude Opus 5 across judge configuration, Bedrock mappings, shared model settings, CLI model selection, tests, and documentation. ChangesClaude Opus 5 model migration
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/evals-core/src/config/costs.ts`:
- Line 8: Add package-local Vitest tests covering the runtime migration
contracts: in packages/evals-core/src/config/costs.ts:8-8, verify estimateCost
uses the claude-opus-5 pricing; in
packages/evals-core/src/config/settings.ts:10-11, verify Opus 5 is eligible for
effort models; in apps/auth0-evals/eval.config.js:83-83, verify the configured
judge model; and in apps/auth0-evals/eval.config.js:118-128, verify both LiteLLM
passthrough and Bedrock alias mapping branches. Place the tests in each relevant
package-local tests/ directory.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: c92af611-fa6f-41c2-8eb0-e57d428f7caa
📒 Files selected for processing (10)
AGENTS.mdapps/auth0-evals/eval.config.jsdocs/ARCHITECTURE.mdpackages/evals-core/src/config/costs.tspackages/evals-core/src/config/settings.tspackages/evals-core/tests/graders/engine.test.tspackages/evals-core/tests/model-detect.test.tspackages/evals/src/cli/constants.tspackages/evals/tests/cli-config.test.tspackages/evals/tests/recommendations.test.ts
| 'gpt-5.6-terra': [2.5, 15.0], | ||
| 'claude-sonnet-5': [2.0, 10.0], | ||
| 'claude-opus-4-8': [5.0, 25.0], | ||
| 'claude-opus-5': [5.0, 25.0], |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Add package-local Vitest coverage for the runtime model migration.
The migration changes pricing lookup, effort-model eligibility, judge selection, and Bedrock alias routing, but the supplied tests do not cover these runtime configuration contracts.
packages/evals-core/src/config/costs.ts#L8-L8: testestimateCostwithclaude-opus-5.packages/evals-core/src/config/settings.ts#L10-L11: test Opus 5 effort-model membership.apps/auth0-evals/eval.config.js#L83-L83: test the configured judge model.apps/auth0-evals/eval.config.js#L118-L128: test LiteLLM passthrough and Bedrock mapping branches.
As per coding guidelines, every logic change must have corresponding Vitest tests in the package-local tests/ directory.
📍 Affects 3 files
packages/evals-core/src/config/costs.ts#L8-L8(this comment)packages/evals-core/src/config/settings.ts#L10-L11apps/auth0-evals/eval.config.js#L83-L83apps/auth0-evals/eval.config.js#L118-L128
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/evals-core/src/config/costs.ts` at line 8, Add package-local Vitest
tests covering the runtime migration contracts: in
packages/evals-core/src/config/costs.ts:8-8, verify estimateCost uses the
claude-opus-5 pricing; in packages/evals-core/src/config/settings.ts:10-11,
verify Opus 5 is eligible for effort models; in
apps/auth0-evals/eval.config.js:83-83, verify the configured judge model; and in
apps/auth0-evals/eval.config.js:118-128, verify both LiteLLM passthrough and
Bedrock alias mapping branches. Place the tests in each relevant package-local
tests/ directory.
Source: Coding guidelines
Replaces the retired
claude-opus-4-8alias withclaude-opus-5throughout the framework — the LLM-as-judge model, the--model allknown set, the cost table, the effort-capable model set, and the Bedrock alias map.