Replace retired Claude model id in the translation hooks - #1924
Open
kasparovabi wants to merge 1 commit into
Open
Replace retired Claude model id in the translation hooks#1924kasparovabi wants to merge 1 commit into
kasparovabi wants to merge 1 commit into
Conversation
claude-3-5-haiku-20241022 was retired on 19 February 2026. Use claude-haiku-4-5-20251001, the form already used elsewhere in this repo.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The translation hooks call
claude-3-5-haiku-20241022. That id was retired on 19 February 2026, and the documented replacement is Claude Haiku 4.5.The rest of the repo is already current.
src/core/llm/types.tsmapshaikutoclaude-haiku-4-5-20251001, and the pricing registry tests use the same string, so I used that exact form here instead of the bare alias.plugins/specweave/lib/hooks/translate-file.tsplugins/specweave/lib/hooks/translate-file.jsplugins/specweave/lib/hooks/invoke-translator-skill.tsplugins/specweave/lib/hooks/invoke-translator-skill.jsFour of the six are the
modelfield passed toanthropic.messages.create. The other two are theconsole.loglines that report which model ran, updated so the verbose output stays accurate.I changed the
.jsfiles alongside the.tsones.tsconfig.jsonexcludesplugins/**/lib/hooks/**/*.ts, sonpm run builddoes not regenerate them, and leaving one side alone would let the two drift apart.There is one thing I deliberately left alone. The cost line beside each call multiplies by 0.25 and 1.25 per million tokens, which is Haiku 3.5 pricing. Haiku 4.5 is $1 and $5 per million, the rates already encoded in
tests/unit/pricing-constants.test.ts. Changing the model id therefore leaves that estimate low by a factor of four. That is a pricing correction rather than an id fix, so I kept it out of this diff, but you may want it. The same figures appear in a comment inplugins/specweave/lib/vendor/utils/translation.d.ts.I also left
src/core/llm/providers/bedrock-provider.tsuntouched, whereclaude-3-5-sonnetappears in a pricing lookup. Retired models still need pricing history, so that string should stay.Reference: Model deprecations
I read this diff line by line before opening it, and I am happy to adjust or drop any part of it. What brought me here was a checker I maintain that flags retired Claude model ids, so I am mentioning that up front rather than leaving you to wonder. I have not run the hooks against the API to reproduce a failure, so the claim rests on the published retirement date. If you would rather handle this your own way, or would rather not get this kind of contribution at all, tell me and I will not send another.