You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
CLAUDE.md bans the em dash (U+2014) repo-wide: "No em dashes ... anywhere: code,
comments, JSDoc, strings, or docs." That is a standing, repo-wide violation on master today, independent of any open PR.
Verified counts (measured directly against master at this triage)
2459 lines across 193 tracked files contain at least one U+2014.
Issue #463 item 3 (fixed by PR #468) covers only em dashes used as the
separator inside @ref gloss lines, and PR #468 lands that fix. This issue is
the broader, repo-wide CLAUDE.md violation: em dashes in ordinary prose,
comments, and docs outside @ref annotations, which #463 never claimed to
cover and which no other open or closed issue tracks (search performed at
filing time: em dash, em-dash, U+2014, 463 all checked).
A related gap worth folding into the fix
While triaging #468 I confirmed the new test/core/llp-ref-hygiene.test.js
em-dash check only inspects the single line containing the @ref token, so
an em dash in a continuation line of a multi-line gloss is invisible to it.
Three live examples exist today, undetected by the new test:
Worth widening that test's scan to the full multi-line gloss (or just running
it as part of whatever mechanism resolves this issue), so the corpus-hygiene
invariant PR #468 establishes doesn't have a standing blind spot.
Backlink
Found and filed during neutral triage of #468 (branch fix/issue-463, head ef04fcd709c145866bcf7d6e439d4ea12b056bb9). Not part of that PR's declared
scope, so filed separately rather than blocking it.
What
CLAUDE.md bans the em dash (U+2014) repo-wide: "No em dashes ... anywhere: code,
comments, JSDoc, strings, or docs." That is a standing, repo-wide violation on
mastertoday, independent of any open PR.Verified counts (measured directly against
masterat this triage)llp/0045-client-attach.design.md: 66 linesllp/0041-central-config-client-actions.design.md: 54 lineshypaware-plugin-kernel-types.d.ts: 36 lineshypaware-core/plugins-workspace/{claude,codex}/skills/hypaware-report-to-html/SKILL.md(72 lines each),llp/0063-login-auto-provision-forward-sink.decision.md(71),llp/0045-client-attach.design.md(66),llp/tombstones/0020-hypaware-design.explainer.md(66),llp/0041-central-config-client-actions.design.md(54),llp/0022-iceberg-export-partitioning.spec.md(51),llp/tombstones/0018-implementation-plan.plan.md(51).Scope note
Issue #463 item 3 (fixed by PR #468) covers only em dashes used as the
separator inside
@refgloss lines, and PR #468 lands that fix. This issue isthe broader, repo-wide CLAUDE.md violation: em dashes in ordinary prose,
comments, and docs outside
@refannotations, which #463 never claimed tocover and which no other open or closed issue tracks (search performed at
filing time:
em dash,em-dash,U+2014,463all checked).A related gap worth folding into the fix
While triaging #468 I confirmed the new
test/core/llp-ref-hygiene.test.jsem-dash check only inspects the single line containing the
@reftoken, soan em dash in a continuation line of a multi-line gloss is invisible to it.
Three live examples exist today, undetected by the new test:
hypaware-core/plugins-workspace/format-iceberg/src/table-format.js:456llp/0067-session-opt-out.design.md:114src/core/runtime/activation.js:64Worth widening that test's scan to the full multi-line gloss (or just running
it as part of whatever mechanism resolves this issue), so the corpus-hygiene
invariant PR #468 establishes doesn't have a standing blind spot.
Backlink
Found and filed during neutral triage of #468 (branch
fix/issue-463, headef04fcd709c145866bcf7d6e439d4ea12b056bb9). Not part of that PR's declaredscope, so filed separately rather than blocking it.