Repository navigation
test(grok): PR #844 left command-catalog and media-census baselines stale #1016
Description
Expected
After merging a provider integration, characterization/census test baselines are updated in the same PR.
Actual
On main @ 5fda2f0 (post PR #844), 4 unit tests fail:
src/renderer/src/features/command-palette/catalog.test.ts— 3 failures: catalog now registers 134 commands; the characterization baseline says 132 (grok added palette commands without bumping the governed list and the arithmetic target: 104 vs 102).src/providers/shared/renderer/protocols/media/imageAttachment.test.ts— "every fixture is traceable to a census row and a real session": new grok fixtures (e.g.testing/fixtures/grok-session-feed/) are not recorded in the census.
Impact
Red unit CI for everyone after the merge; hides real regressions in these suites.
Fix direction
Update the governed command list / arithmetic in catalog.test.ts (verify the two new commands are intentional and correctly registered) and add the grok fixtures to the media census. Pre-existing on main, discovered while verifying #1014's fix (which is unrelated).
Activity
Juliusolsson05 commentedon Sep 19, 2026
Triage update (2026-09-19): of the two failures listed, only the command-catalog one is real on CI. It is fixed by #1019.
catalog.test.ts: 3 failures on main since feat(grok): build native provider integration #844, from the generatedgrok-vertical/grok-horizontal. fix(command-palette): pin the generated Grok split commands in the catalog snapshot #1019 pins them, moves the count to 134 and the governance subtrahend 35 → 37, and keeps the 102 right-hand side, per the file's own rule. The OpenCode session that filed this issue started the same edit in.worktrees/fix-command-catalog-grok-baselineand hit its usage limit before committing. fix(command-palette): pin the generated Grok split commands in the catalog snapshot #1019 is equivalent, so that worktree can go once fix(command-palette): pin the generated Grok split commands in the catalog snapshot #1019 merges.imageAttachment.test.ts"every fixture is traceable to a census row and a real session": not a Grok regression, and not failing on CI. It passes 17/17 in main's red run 35422088849 (c3614cb8). Locally it fails withatp-codex-image-inside-claude-transcript cites a missing session: ~/.claude/projects/-Users-juliusolsson-Desktop-Development-klay/c0c60d3d-….jsonl. That is a Claude transcript from another project, deleted from this machine. This is the known local-environment failure of that census check (it verifies source sessions exist on the author's disk), so it predates feat(grok): build native provider integration #844 and is unrelated to the new Grok fixtures.