From 2dff709215ec5af0ca17e150a0ac3af8d4a64a1b Mon Sep 17 00:00:00 2001 From: Claude Date: Fri, 4 Sep 2026 11:40:45 +0000 Subject: [PATCH] docs(check-skill-examples): drop the stale zero-jsonc-fences claim MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `parseJsonFence`'s docblock said "There are zero `jsonc` fences in the corpus today" as the reason the dialect is recognised. That stopped being true at PR #7473 (16 comment-carrying fences retagged `jsonc`) and PR #7608 moved it further (41 `json` / 29 `jsonc` today, confirmed with `node scripts/check-skill-examples.mjs --measure`). The sentence read as "this branch has never run in anger" — the opposite of the truth, and a risk that a reader deletes or weakens the branch as dead code. Rewritten to a count-free statement, in the same convention the file's own header uses for its other counts ("Re-derive it with `--measure`"): the branch is exercised by fences already in the corpus, not merely reserved for a hypothetical one. No behaviour change. Co-Authored-By: Claude Fable 5.1 Claude-Session: https://claude.ai/code/session_019RfFHiRCSs3JXLK4cwcfox --- scripts/check-skill-examples.mjs | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/check-skill-examples.mjs b/scripts/check-skill-examples.mjs index 0480b44ed8..c83e1af746 100644 --- a/scripts/check-skill-examples.mjs +++ b/scripts/check-skill-examples.mjs @@ -919,8 +919,9 @@ function stripTrailingCommas(source) { * tolerant parser here would bless a file no `JSON.parse` in the product would * accept. `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. + * shapes a guide writes. The `jsonc` branch is exercised by fences already in + * the corpus, not merely reserved for a hypothetical one; re-derive the + * population with `--measure`. */ export function parseJsonFence(body, language) { const text = language === 'jsonc' ? stripTrailingCommas(stripJsonComments(body)) : body;