[finding] `parseJsonFence`'s docblock still says "there are zero `jsonc` fences in the corpus today" — there are 29

[os-steve]

Observation filed out of scope by the dev seat working #7474 (PR #7608), whose dispatch pinned scripts/check-skill-examples.mjs to its MARKED_FLOOR json row only, so this line was left untouched rather than fixed as a rider.

The drift

scripts/check-skill-examples.mjs, in parseJsonFence's docblock:

jsonc gets comments and trailing commas removed first, and nothing else: that is the whole of what the jsonc dialect adds over json for the shapes a guide writes. There are zero jsonc fences in the corpus today; the language is recognised so that adding one is not a silent no-op.

That last sentence was true when the gate landed (#7359) and stopped being true at PR #7473, which retagged 16 comment-carrying fences jsonc. Measured on origin/main at 24e027e with the gate's own scanner: 40 json and 16 jsonc fences under skills/objectui/. PR #7608 moves it further — 41 json and 29 jsonc.

Why it is worth a line of work

It is prose drift, not a defect in the machinery: the parser behaves correctly and every phase reports honestly. The cost is that the sentence is load-bearing for a reader deciding whether the jsonc branch is exercised — it currently reads as "this branch has never run in anger", which is the opposite of the truth, and a reader trusting it could delete or weaken the branch as dead code.

Suggested shape

Replace the stale clause with the population re-derived at the time of the edit, or with a pointer to --measure so the number is not restated in prose at all — the same treatment the file's own header gives its other counts ("Re-derive it with --measure").

Surface note: scripts/** is not a governed surface here, but this file is the enforcement point for one that is, so the fix is a small deliberate PR rather than a drive-by.

Refs: #7474 · PR #7473 · PR #7608 · #7359 (the gate and its marker convention).

Filed by Claude Code on behalf of the skills-lane dev seat, session https://claude.ai/code/session_019RfFHiRCSs3JXLK4cwcfox.

[claude]

Triage — skills seat (session session_019RfFHiRCSs3JXLK4cwcfox, os-steve), 2026-09-04T06:09Z (self-graded under the #7359 precedent while the objectui triage seat is vacant). Graded priority:p3, finding cleared, domain:skills, tooling, state pm:queue. Applied and read back after this comment.

Scope as queued (XS, scripts/check-skill-examples.mjs only, serial behind PR #7608): rewrite parseJsonFence's docblock sentence about the corpus's jsonc population to the measured figure at the head with the date (29 after PR #7608; the count read from the tree, not typed), or to a statement that does not carry a count at all if the docblock's point survives without one; no behaviour change; the self-test and the vitest suites identical before and after.

---

Generated by Claude Code

[claude]

Claim: PM loop round 5
Session: session_019RfFHiRCSs3JXLK4cwcfox
Branch: claude/issue-7610-jsonc-docblock-count
Worktree: objectui-issue-7610
Domain: domain:skills
File surface: scripts/check-skill-examples.mjs only — the one parseJsonFence docblock sentence ("There are zero jsonc fences in the corpus today; …", near L922 at 11edab8) rewritten either to a statement that carries no count (the branch is exercised; re-derive the population with the gate's own measurement) or to the measured figure with its date, the count read from the tree by the gate's scanner, never typed. ⛔ No behaviour change; the --self-test and the vitest suites that name the script identical before and after; ⛔ no other sentence, ⛔ no other file.
Container & model: XS, mode:subagent, model: sonnet (a docblock sentence; floor tier)
Clause-②: no (prose in a script docblock)
Serial constraints cleared: objectui PR #7608 (#7474) MERGED 89d6adf3 10:58:08Z — the corpus the sentence describes is at 41 json + 29 jsonc on main; no open PR touches the file. Not governed (scripts/**) ⇒ draft PR, in-seat contract-tier review, ready + auto-merge by the seat.

Decision re-read (11:27Z): no ruling owed — prose drift measured by the #7474 dev, triaged 06:09Z; the shape is the file's own convention for counts ("Re-derive it with --measure").

---

Generated by Claude Code

[claude]

os-dev-report
{
"issue": 7610,
"status": "done",
"branch": "claude/issue-7610-jsonc-docblock-count",
"pr": "#7631",
"premise_still_valid": true,
"summary": "Rewrote parseJsonFence's docblock sentence in scripts/check-skill-examples.mjs to drop the stale zero-jsonc-fences claim, using the count-free form (the file's own convention: "Re-derive it with --measure"). Confirmed the premise first: on this branch (same tree as origin/main, merge-base 11edab8) node scripts/check-skill-examples.mjs --measure reports 41 json + 29 jsonc fences under skills/objectui, matching the issue's PR #7608 figure. No behaviour change; exactly one sentence in one file touched.",
"tests": "Premise measurement: node scripts/check-skill-examples.mjs --measure (after pnpm exec turbo run build $(node scripts/check-skill-examples.mjs --build-filter) --concurrency=2, exit 0) -> summary line 'Scanned 20 guide(s) under skills, .claude/skills: 121 ts/tsx/typescript fence(s), 70 json/jsonc fence(s).'; per-fence tags counted 41 [json] + 29 [jsonc] = 70, matching the issue body's PR #7608 figure (a grep -c '^```jsonc' control agrees at 29 but was not used as the measurement). Before: 'There are zero jsonc fences in the corpus today; the language is recognised so that adding one is not a silent no-op.' After: 'The jsonc branch is exercised by fences already in the corpus, not merely reserved for a hypothetical one; re-derive the population with --measure.' Gates (all exit 0, redirected-then-captured, verdict lines quoted): node scripts/check-skill-examples.mjs -> 'Every marked skill example holds up against the built types.'; node scripts/check-skill-examples.mjs --self-test -> '52 cases pass' (same count as the pre-edit tree); pnpm exec vitest run scripts/__tests__/check-skill-examples.test.ts scripts/__tests__/check-skill-eval-tokens.test.ts -> 'Test Files 2 passed (2)', 'Tests 120 passed (120)'; pnpm check:control-bytes -> 'OK (scanned 6239 tracked text file(s); skipped 85 binary)'; pnpm check:shell-escape-residue -> 'OK (5/5 root(s) resolved ... 0 occurrence(s) outside a fence)'; pnpm lint (whole repo, NODE_OPTIONS=--max-old-space-size=4096) -> 47/47 tasks successful, 2883 pre-existing warnings (0 errors), none in the edited file; node scripts/check-changeset-presence.mjs -> 'No source or published contract of a released package changed in this range, so no changeset is owed.' Manual control-byte scan of the diff (grep -naP '[\\x00-\\x08\\x0b\\x0c\\x0e-\\x1f\\x7f]') clean. git diff and the PR's own file list both confirm exactly one file, +3/-2, no other sentence touched.",
"mcp_calls": 0,
"open_questions": [],
"out_of_scope_findings": []
}

---

Generated by Claude Code

[claude]

ACCEPT — contract-tier review of PR #7631, head 2dff709 (skills seat, 2026-09-04T11:44Z).

Implemented-by: claude/issue-7610-jsonc-docblock-count (os-dev subagent, task a84fbfaad4d1f2ca6)
Reviewed-by: session_019RfFHiRCSs3JXLK4cwcfox (skills seat)

Verified in a detached compare worktree at 2dff709 (merge base = origin/main 11edab8; git merge-tree clean), not from the report: one file, +3/−2, exactly the parseJsonFence docblock sentence — the stale "zero jsonc fences" claim replaced by the count-free form the file's own convention prescribes ("The jsonc branch is exercised by fences already in the corpus, not merely reserved for a hypothetical one; re-derive the population with --measure"); grep for the old sentence on the head: 0. No behaviour change. The dev measured the premise with the gate's own --measure (41 json + 29 jsonc = 70 fences under skills/objectui, the PR #7608 figure) and ran the gate, its --self-test (52 cases, unchanged), the two vitest suites naming the script (120 tests), check:control-bytes, check:shell-escape-residue, whole-repo lint (0 errors) and check-changeset-presence ("no changeset is owed" — followed). CI on 2dff709 at 11:40Z: 17 success, 3 skipped, 9 in progress.

Landing: pure code (scripts/**, not governed) ⇒ the seat flips ready and arms auto-merge SQUASH once every check is green.

---

Generated by Claude Code