fix(test): bump registration-coverage snapshot to 27 ocs atoms#357
Merged
Conversation
The `ocs_get_chatbot_pipeline_id` atom landed in 569b57e (PR #350) without updating the registration-coverage snapshot. The atom is legitimate — wraps `PlaywrightBackend.pipelineIdFor` so `/ace:sweep ocs` can pair orphan chatbots with their per-opp Pipeline rows before deletion — so the fix is to bump the snapshot, not remove the atom. Snapshot count: 26 → 27. CLAUDE.md atom breakdown: "23 atoms (Authoring 11 + Observation 12)" → "27 atoms (Authoring 15 + Observation 12)" — also stale; the capability-map already had Authoring 15, just the prose comment + top-level doc were behind. capability-map.ts: "Authoring (11)" comment → "Authoring (15)". Full suite green: 1239 passed, 43 skipped. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
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.
Summary
test/mcp/registration-coverage.test.tswas failing locally withexpected 26, got 27. Pre-existing on origin/main —clean-installCI doesn't gate on this test so it slipped, but adds noise to local runs.Diagnosis: the 27th atom is
ocs_get_chatbot_pipeline_id, landed in569b57e(PR #350) without bumping the snapshot. Atom is legitimate (wraps existingPlaywrightBackend.pipelineIdForfor/ace:sweep ocsto pair orphan chatbots with their per-opp Pipeline rows). Direction: bump snapshot, not remove atom.Fix:
test/mcp/registration-coverage.test.ts: ocsexpectedCount26 → 27mcp/ocs/capability-map.ts: header comment "Authoring (11)" → "Authoring (15)" (was already stale before this PR — the map had 15 authoring atoms, just the comment was behind)CLAUDE.md: "23 atoms (Authoring 11 + Observation 12)" → "27 atoms (Authoring 15 + Observation 12)"Small follow-up after the 10-PR Phase 6 session, just resolving pre-existing snapshot drift.
Test plan
npm test— 1239 passed, 43 skipped (was 1238 passing + 1 failing)npx vitest run test/mcp/registration-coverage.test.ts— 14/14 passclean-installCI passes🤖 Generated with Claude Code