Repository navigation
Conversation
|
Thanks for catching this and refreshing the catalog. I reviewed exact head Local validation passed:
I also independently checked the changed prices against the official pricing page, including Haiku 5.5's >100K tier, Mistral Large 4, Glyph's free preview, and Sonnet 5.5's $0.10/M cache-read rate. A local offline probe accepts One related CI hardening item would help prevent a repeat: the catalog steps use The log shows Could you add explicit Two optional coverage improvements, not merge blockers:
|
The Compare and Synchronize steps piped '... | tee' without pipefail, so tee's exit 0 masked a failing catalog check and the job stayed green. Run them with shell: bash so GitHub invokes bash with -o pipefail and a failed check fails the job. Also trigger the workflow when the new pricing fixture changes.
Build the off-omission stream test from thinkingMetadataForModel so it pins the canonical thinking.effortMap path instead of only the legacy level map, and add a minimal Haiku 5.5 pricing fixture with its <=100K / >100K bands. The new pricing test compares the fixture against the real MODEL_COSTS, so a drift between the page bands and the runtime policy fails.
Set contextWindow to 1000000 and individual-goat to true so the snapshot row mirrors the live props.rows entry verbatim. Neither field feeds the parser, but a fixture should not carry known-wrong values.
|
Thanks for addressing the three follow-ups on new head The Local validation on the new head passed:
Approving — CI is green on this head and the scope matches the reviewed catalog refresh. |
pierreraby
left a comment
There was a problem hiding this comment.
Approved on exact head 8645883: catalog refresh to command-code@1.79.2, off recognition, and reviewed pricing verified locally (full npm test incl. pi 1.1.0 + OMP 18.4.2 against mocks, live catalog/pricing checks PASS, format/diff-check clean). Follow-up commits address the reported pipefail masking and add the effortMap + Haiku-band coverage.
Adds
claude-haiku-5-5from the latest Command Code catalog and clears the drift that had accumulated since the pinnedcommand-code@1.72.4snapshot.What changed:
command-code@1.79.2: newclaude-haiku-5-5,mistral/mistral-large-4(image input, 262K output limit), and freestealth/glyph-cluster:free; retiredstealth/pixel-canaryandstealth/space-bunny-alpha.offreasoning effort now published for the DeepSeek V4 and V4.1 models. The daily catalog check was failing on it (Unexpected reasoning efforts for deepseek/deepseek-v4-pro: off, high, max) becauseVALID_EFFORTSand the generated type union did not knowoff. The generate transport still never forwardsoffasreasoning_effort; the Provider API and Oh My Pi adapters translate the level on their own wire.claude-haiku-5-5base plus its 100K long-context tier,mistral/mistral-large-4, and freestealth/glyph-cluster:free; dropped the retiredstealth/space-bunny-alpha; correctedclaude-sonnet-5-5cache-read to $0.10/M. The snapshot now covers all 87 advertised models.MODEL_EFFORT_OVERRIDESstill added manual levels (the map is empty).Validation:
npm test(exit 0),npm run check:commandcode-catalog(no drift),npm run check:commandcode-pricing(PASS),npm run typecheck,npm run format:check,git diff --check.