Repository navigation
fix: move project evaluation to claude sonnet 5.5 after bedrock retired sonnet 4 (lfx-cdp-insights-ops#820) - #4996
Conversation
…ed sonnet 4 Refs linuxfoundation/lfx-cdp-insights-ops#820 Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>
PR SummaryMedium Risk Overview That query’s Bedrock args swap
Reviewed by Cursor Bugbot for commit 37c51af. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
🟡 Changes recommended
The pricing configuration understates costs for the selected US geographic inference profile.
1 open finding
What changed in this PR
Moves project evaluation to Claude Sonnet 4.5 to restore evaluations after Sonnet 4 retirement, leaving other query types unchanged.
Changes:
- Adds the Sonnet 4.5 model identifier.
- Registers its region and pricing and selects it for project evaluation.
| File | Description |
|---|---|
| services/libs/types/src/llm.ts | Adds model mappings and switches project evaluation. |
| services/libs/types/src/enums/llm.ts | Defines the Sonnet 4.5 US inference profile. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…ed sonnet 4 Refs linuxfoundation/lfx-cdp-insights-ops#820 Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2804145. Configure here.
…onnet 5.5 Refs linuxfoundation/lfx-cdp-insights-ops#820 Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>
Refs linuxfoundation/lfx-cdp-insights-ops#820 Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>



Summary
Since 2026-10-16 04:00 UTC every project evaluation fails with Bedrock's "This model version has reached the end of its life" for⚠️ something went wrong on our side", and the nightly discovery run produced 15 error rows and nothing else. The same model is behind the member-enrichment workflows (
us.anthropic.claude-sonnet-4-20250514-v1:0. The onboarding bot still parses requests (Haiku 4.5) but every new non-LF project ends in "processMemberSourcesis already failing in cdp-alerts).Step 1 of 2 (this PR): move only
LlmQueryType.PROJECT_EVALUATIONto Claude Sonnet 5.5 as a quick production check of the new model. Step 2: once an evaluation succeeds on it, a follow-up PR moves the other seven query types and the two hard-coded ids in the merge-suggestions workflows, as asked by the CDP team.Why Sonnet 5.5 and not 4.5: on Bedrock Sonnet 4.5 is already "Legacy" with EOL on 2027-04-08; Sonnet 5.5 launched 2026-09-28 with EOL no sooner than 2027-09-28, supports the Invoke API with
anthropic_version: bedrock-2023-05-31from us-east-1 (whatLlmServiceuses), and costs $2/$10 per MTok instead of $3/$15.Changes
enums/llm.ts:CLAUDE_SONNET_5_5 = 'us.anthropic.claude-sonnet-5-5'(US geo inference id from the AWS model card).llm.ts: region and pricing for it;PROJECT_EVALUATIONpoints to it. Every other query type stays on Sonnet 4 for now.Before deploying
Bedrock lists Sonnet 5.5 access as "See Access" rather than "Open": make sure the production AWS account has the model enabled (Bedrock → Model access), e.g.
aws bedrock get-inference-profile --inference-profile-identifier us.anthropic.claude-sonnet-5-5 --region us-east-1. If it is not available, the fallback is Sonnet 5 (us.anthropic.claude-sonnet-5, open access).Test plan
vitesttypes,pnpm lint --deny-warnings,pnpm tsc-checkerrorllmPromptHistoryshows aproject_evaluationrow with modelus.anthropic.claude-sonnet-5-5Refs linuxfoundation/lfx-cdp-insights-ops#820
🤖 Generated with Claude Code