@nurix/etna 0.20.15 → 0.20.16

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (31) hide show
  1. package/data/harness-index.json +1 -1
  2. package/data/harnesses/code/CLAUDE.md +8 -2
  3. package/data/harnesses/code/agents/bookkeeper.md +1 -0
  4. package/data/harnesses/code/agents/polygraph-finder.md +4 -0
  5. package/data/harnesses/code/harness.json +2 -2
  6. package/data/harnesses/code/hooks/awareness.mjs +15 -2
  7. package/data/harnesses/code/hooks/lib/state.mjs +1 -0
  8. package/data/harnesses/code/hooks/lib/sweep.mjs +20 -2
  9. package/data/harnesses/code/skills/documentation/SKILL.md +28 -0
  10. package/data/harnesses/code/skills/documentation/scripts/comment-strip.mjs +339 -0
  11. package/data/harnesses/code/skills/documentation/scripts/doc-lint.mjs +41 -10
  12. package/data/harnesses/code/skills/documentation/scripts/source-lint.mjs +225 -0
  13. package/data/harnesses/code/skills/documentation/templates/behavioral-checklist.md +1 -2
  14. package/data/harnesses/code/skills/documentation/templates/bookkeeping.md +1 -1
  15. package/data/harnesses/code/skills/polygraph/SKILL.md +1 -1
  16. package/data/harnesses/code/skills/testing/SKILL.md +1 -1
  17. package/data/harnesses/delivery/harness.json +2 -2
  18. package/data/harnesses/delivery/skills/documentation/SKILL.md +28 -0
  19. package/data/harnesses/delivery/skills/documentation/scripts/comment-strip.mjs +339 -0
  20. package/data/harnesses/delivery/skills/documentation/scripts/doc-lint.mjs +41 -10
  21. package/data/harnesses/delivery/skills/documentation/scripts/source-lint.mjs +225 -0
  22. package/data/harnesses/delivery/skills/documentation/templates/behavioral-checklist.md +1 -2
  23. package/data/harnesses/delivery/skills/documentation/templates/bookkeeping.md +1 -1
  24. package/data/harnesses/etna/harness.json +2 -2
  25. package/data/harnesses/etna/skills/documentation/SKILL.md +28 -0
  26. package/data/harnesses/etna/skills/documentation/scripts/comment-strip.mjs +339 -0
  27. package/data/harnesses/etna/skills/documentation/scripts/doc-lint.mjs +41 -10
  28. package/data/harnesses/etna/skills/documentation/scripts/source-lint.mjs +225 -0
  29. package/data/harnesses/etna/skills/documentation/templates/behavioral-checklist.md +1 -2
  30. package/data/harnesses/etna/skills/documentation/templates/bookkeeping.md +1 -1
  31. package/package.json +2 -2
@@ -0,0 +1,339 @@
1
+ #!/usr/bin/env node
2
+ // skills/documentation/scripts/comment-strip.mjs - Proves a comment sweep changed no executable code.
3
+ // Strips comments from a git revision of each file and from the working copy, then compares the two
4
+ // stripped forms; a difference is a code change hiding inside a comments-only pass. String and
5
+ // template literals, character literals, regex literals, and raw strings are honoured, so a `//`
6
+ // inside a string is code and stays.
7
+
8
+ import { execFileSync } from "node:child_process";
9
+ import { readFileSync, existsSync, statSync } from "node:fs";
10
+ import { readdirSync } from "node:fs";
11
+ import { join, extname } from "node:path";
12
+
13
+ // Comment syntax per extension. `block` is null where the language has none.
14
+ const DIALECTS = {
15
+ ".ts": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
16
+ ".tsx": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
17
+ ".mts": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
18
+ ".js": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
19
+ ".jsx": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
20
+ ".mjs": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
21
+ ".cjs": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: true , sameLineQuotes: true },
22
+ ".css": { line: null, block: ["/*", "*/"], strings: `'"`, regex: false , sameLineQuotes: true },
23
+ ".rs": { line: "//", block: ["/*", "*/"], strings: `"`, regex: false, rustRaw: true , sameLineQuotes: true },
24
+ ".go": { line: "//", block: ["/*", "*/"], strings: `'"\``, regex: false , sameLineQuotes: true },
25
+ ".java": { line: "//", block: ["/*", "*/"], strings: `'"`, regex: false , sameLineQuotes: true },
26
+ ".swift": { line: "//", block: ["/*", "*/"], strings: `"`, regex: false , sameLineQuotes: true },
27
+ ".kt": { line: "//", block: ["/*", "*/"], strings: `"`, regex: false , sameLineQuotes: true },
28
+ ".sql": { line: "--", block: ["/*", "*/"], strings: `'"`, regex: false },
29
+ ".py": { line: "#", block: null, strings: `'"`, regex: false },
30
+ ".sh": { line: "#", block: null, strings: `'"`, regex: false },
31
+ ".zsh": { line: "#", block: null, strings: `'"`, regex: false },
32
+ ".yml": { line: "#", block: null, strings: `'"`, regex: false },
33
+ ".yaml": { line: "#", block: null, strings: `'"`, regex: false },
34
+ ".toml": { line: "#", block: null, strings: `'"`, regex: false },
35
+ };
36
+
37
+ // Whether a `/` at this position opens a regex literal rather than dividing — decided by the last
38
+ // significant character, the same way a JavaScript tokenizer decides it.
39
+ function regexCanStart(prev) {
40
+ if (prev === "") return true;
41
+ return !/[\w$)\]]/.test(prev);
42
+ }
43
+
44
+ // The index just past a template literal starting at `i`, following `${…}` interpolations and the
45
+ // templates and quotes nested inside them. A scan that stops at the first backtick it meets closes the
46
+ // OUTER template on an INNER one — `${x ? `<b>${y}</b>` : ""}` — and every quote and comment downstream
47
+ // is then read in the wrong context, which reports a whole file as changed code.
48
+ function templateEnd(source, i) {
49
+ const n = source.length;
50
+ let j = i + 1;
51
+ while (j < n) {
52
+ const c = source[j];
53
+ if (c === "\\") {
54
+ j += 2;
55
+ continue;
56
+ }
57
+ if (c === "`") return j + 1;
58
+ if (c === "$" && source[j + 1] === "{") {
59
+ let depth = 1;
60
+ j += 2;
61
+ while (j < n && depth > 0) {
62
+ const d = source[j];
63
+ if (d === "\\") {
64
+ j += 2;
65
+ continue;
66
+ }
67
+ if (d === "`") {
68
+ j = templateEnd(source, j);
69
+ continue;
70
+ }
71
+ if (d === "'" || d === '"') {
72
+ const quote = d;
73
+ j++;
74
+ while (j < n && source[j] !== quote && source[j] !== "\n") {
75
+ if (source[j] === "\\") j++;
76
+ j++;
77
+ }
78
+ j++;
79
+ continue;
80
+ }
81
+ if (d === "{") depth++;
82
+ else if (d === "}") depth--;
83
+ j++;
84
+ }
85
+ continue;
86
+ }
87
+ j++;
88
+ }
89
+ return n;
90
+ }
91
+
92
+ // Remove every comment from `source`, preserving all other bytes including newlines.
93
+ export function stripComments(source, dialect) {
94
+ const { line, block, strings, regex, rustRaw } = dialect;
95
+ const out = [];
96
+ let i = 0;
97
+ let lastSignificant = "";
98
+ const n = source.length;
99
+ while (i < n) {
100
+ const ch = source[i];
101
+ const two = source.slice(i, i + 2);
102
+
103
+ if (rustRaw && ch === "r" && (source[i + 1] === "#" || source[i + 1] === '"')) {
104
+ let j = i + 1;
105
+ let hashes = 0;
106
+ while (source[j] === "#") {
107
+ hashes++;
108
+ j++;
109
+ }
110
+ if (source[j] === '"') {
111
+ const close = '"' + "#".repeat(hashes);
112
+ const end = source.indexOf(close, j + 1);
113
+ const stop = end === -1 ? n : end + close.length;
114
+ out.push(source.slice(i, stop));
115
+ lastSignificant = '"';
116
+ i = stop;
117
+ continue;
118
+ }
119
+ }
120
+
121
+ if (line && two.slice(0, line.length) === line) {
122
+ const nl = source.indexOf("\n", i);
123
+ i = nl === -1 ? n : nl;
124
+ continue;
125
+ }
126
+
127
+ if (block && two === block[0]) {
128
+ const end = source.indexOf(block[1], i + 2);
129
+ const stop = end === -1 ? n : end + block[1].length;
130
+ // A JSX expression container holding only a comment — `{/* … */}` — is comment syntax
131
+ // whole: the braces exist to carry it, so they leave with it. Left behind, a multi-line
132
+ // container collapses to `{}` and reads as a code change.
133
+ let after = stop;
134
+ while (after < n && /\s/.test(source[after])) after++;
135
+ let brace = out.length - 1;
136
+ while (brace >= 0 && out[brace].trim() === "") brace--;
137
+ if (regex && source[after] === "}" && brace >= 0 && out[brace] === "{") {
138
+ out.splice(brace, 1);
139
+ for (const c of source.slice(i, after + 1)) if (c === "\n") out.push("\n");
140
+ i = after + 1;
141
+ continue;
142
+ }
143
+ for (const c of source.slice(i, stop)) if (c === "\n") out.push("\n");
144
+ i = stop;
145
+ continue;
146
+ }
147
+
148
+ if (ch === "`" && regex) {
149
+ const stop = templateEnd(source, i);
150
+ out.push(source.slice(i, stop));
151
+ lastSignificant = "`";
152
+ i = stop;
153
+ continue;
154
+ }
155
+
156
+ if (strings.includes(ch)) {
157
+ // A quote opens a literal only where one can actually close. In the C family and in Rust a
158
+ // `'` or `"` literal may not contain a raw newline, so requiring the partner on the same
159
+ // line is not a heuristic — it is the grammar, and it is what keeps an apostrophe in JSX
160
+ // text (`Couldn't load`) and a Rust char literal holding a quote (`trim_matches('"')`) from
161
+ // opening a string that swallows the rest of the file. Backticks span lines by design.
162
+ if (ch !== "`" && dialect.sameLineQuotes) {
163
+ let j = i + 1;
164
+ let closed = false;
165
+ while (j < n && source[j] !== "\n") {
166
+ if (source[j] === "\\") {
167
+ j += 2;
168
+ continue;
169
+ }
170
+ if (source[j] === ch) {
171
+ closed = true;
172
+ break;
173
+ }
174
+ j++;
175
+ }
176
+ if (!closed) {
177
+ out.push(ch);
178
+ lastSignificant = ch;
179
+ i++;
180
+ continue;
181
+ }
182
+ out.push(source.slice(i, j + 1));
183
+ lastSignificant = ch;
184
+ i = j + 1;
185
+ continue;
186
+ }
187
+ out.push(ch);
188
+ i++;
189
+ while (i < n) {
190
+ if (source[i] === "\\") {
191
+ out.push(source.slice(i, i + 2));
192
+ i += 2;
193
+ continue;
194
+ }
195
+ out.push(source[i]);
196
+ if (source[i] === ch) {
197
+ i++;
198
+ break;
199
+ }
200
+ i++;
201
+ }
202
+ lastSignificant = ch;
203
+ continue;
204
+ }
205
+
206
+ if (regex && ch === "/" && regexCanStart(lastSignificant)) {
207
+ let j = i + 1;
208
+ let inClass = false;
209
+ let closed = false;
210
+ while (j < n) {
211
+ const c = source[j];
212
+ if (c === "\\") {
213
+ j += 2;
214
+ continue;
215
+ }
216
+ if (c === "\n") break;
217
+ if (c === "[") inClass = true;
218
+ else if (c === "]") inClass = false;
219
+ else if (c === "/" && !inClass) {
220
+ closed = true;
221
+ j++;
222
+ break;
223
+ }
224
+ j++;
225
+ }
226
+ if (closed) {
227
+ out.push(source.slice(i, j));
228
+ lastSignificant = "/";
229
+ i = j;
230
+ continue;
231
+ }
232
+ }
233
+
234
+ out.push(ch);
235
+ if (!/\s/.test(ch)) lastSignificant = ch;
236
+ i++;
237
+ }
238
+ // Collapse whitespace so a re-indent left behind by a removed block comment is not a finding.
239
+ return out
240
+ .join("")
241
+ .split("\n")
242
+ .map((l) => l.replace(/\s+/g, " ").trim())
243
+ .filter((l) => l !== "")
244
+ .join("\n");
245
+ }
246
+
247
+ // Comment-only lines in a source: the non-blank lines it has, less the lines that survive stripping.
248
+ // Both sides exclude blanks, so the difference is comments alone; a trailing comment on a code line is
249
+ // correctly not counted, because the line itself survives.
250
+ function countCommentLines(source, dialect) {
251
+ const nonBlank = source.split("\n").filter((l) => l.trim() !== "").length;
252
+ const stripped = stripComments(source, dialect);
253
+ return nonBlank - (stripped === "" ? 0 : stripped.split("\n").length);
254
+ }
255
+
256
+ function gitShow(rev, path) {
257
+ try {
258
+ return execFileSync("git", ["show", `${rev}:${path}`], { encoding: "utf8", maxBuffer: 1 << 28 });
259
+ } catch {
260
+ return null;
261
+ }
262
+ }
263
+
264
+ function walk(dir, acc = []) {
265
+ for (const entry of readdirSync(dir, { withFileTypes: true })) {
266
+ if (entry.name === "node_modules" || entry.name === ".git" || entry.name === "dist") continue;
267
+ const full = join(dir, entry.name);
268
+ if (entry.isDirectory()) walk(full, acc);
269
+ else if (DIALECTS[extname(entry.name)]) acc.push(full);
270
+ }
271
+ return acc;
272
+ }
273
+
274
+ function main(argv) {
275
+ let rev = "HEAD";
276
+ // With `--to`, both sides come from git and the working tree is not read — the mode that
277
+ // audits a sweep somebody already committed.
278
+ let to = null;
279
+ const paths = [];
280
+ for (let i = 0; i < argv.length; i++) {
281
+ if (argv[i] === "--rev") rev = argv[++i];
282
+ else if (argv[i] === "--to") to = argv[++i];
283
+ else if (argv[i] === "--changed") {
284
+ const out = execFileSync("git", ["diff", "--name-only", rev], { encoding: "utf8" });
285
+ paths.push(...out.split("\n").filter(Boolean));
286
+ } else paths.push(argv[i]);
287
+ }
288
+ if (paths.length === 0) {
289
+ const out = execFileSync("git", ["diff", "--name-only", rev], { encoding: "utf8" });
290
+ paths.push(...out.split("\n").filter(Boolean));
291
+ }
292
+
293
+ const files = [];
294
+ for (const p of paths) {
295
+ // A revision-to-revision run names paths that need not exist in the checkout.
296
+ if (to !== null) {
297
+ if (DIALECTS[extname(p)]) files.push(p);
298
+ continue;
299
+ }
300
+ if (!existsSync(p)) continue;
301
+ if (statSync(p).isDirectory()) files.push(...walk(p));
302
+ else if (DIALECTS[extname(p)]) files.push(p);
303
+ }
304
+
305
+ let changed = 0;
306
+ let checked = 0;
307
+ let commentDelta = 0;
308
+ for (const file of [...new Set(files)]) {
309
+ const dialect = DIALECTS[extname(file)];
310
+ const before = gitShow(rev, file);
311
+ if (before === null) continue;
312
+ const after = to === null ? readFileSync(file, "utf8") : gitShow(to, file);
313
+ if (after === null) continue;
314
+ const a = stripComments(before, dialect);
315
+ const b = stripComments(after, dialect);
316
+ checked++;
317
+ // Count comment lines directly. Differencing raw length against stripped length would fold in the
318
+ // blank lines the strip also drops, and report a comment+blank delta under a comment label.
319
+ const cb = countCommentLines(before, dialect);
320
+ const ca = countCommentLines(after, dialect);
321
+ commentDelta += ca - cb;
322
+ if (a !== b) {
323
+ changed++;
324
+ const al = a.split("\n");
325
+ const bl = b.split("\n");
326
+ let k = 0;
327
+ while (k < al.length && k < bl.length && al[k] === bl[k]) k++;
328
+ console.log(`CODE CHANGED ${file}`);
329
+ console.log(` ${rev}: ${al[k] === undefined ? "(end of file)" : al[k].slice(0, 120)}`);
330
+ console.log(` now: ${bl[k] === undefined ? "(end of file)" : bl[k].slice(0, 120)}`);
331
+ }
332
+ }
333
+
334
+ const verdict = changed === 0 ? "no executable code changed" : `${changed} file(s) changed code`;
335
+ console.log(`\n${checked} file(s) compared against ${rev} — ${verdict}; comment lines ${commentDelta >= 0 ? "+" : ""}${commentDelta}.`);
336
+ process.exit(changed === 0 ? 0 : 1);
337
+ }
338
+
339
+ if (import.meta.url === `file://${process.argv[1]}`) main(process.argv.slice(2));
@@ -17,19 +17,35 @@ const RULES = [
17
17
  { id: 'status-glyph', severity: 'warn', scope: 'truth', exempt: /(^|\/)docs\/(tests|operations)\//, pattern: /(^|\s)(✅|⏳|- \[[ x]\])/u, message: 'status glyph / checklist in a truth doc — tracking belongs in a record' },
18
18
  // Truth docs are supposed to cite records, and record filenames are dated — so a date inside a link
19
19
  // target or a code span is a path, not narration. Strip those before matching.
20
- { id: 'date-in-truth-doc', severity: 'warn', scope: 'truth', strip: /\[[^\]]*\]\([^)]*\)|`[^`]*`/g, pattern: /\b20\d{2}-\d{2}-\d{2}\b/, message: 'date in a truth doc — only the AD Decision Changelog and structure.md Health Log tails carry dates' },
20
+ { id: 'date-in-truth-doc', severity: 'block', scope: 'truth', strip: /\[[^\]]*\]\([^)]*\)|`[^`]*`|^(last-reviewed|last-updated|rev|harness-version):.*$/gm, pattern: /\b20\d{2}-\d{2}-\d{2}\b/, message: 'date in a truth doc — cite the record by name and link; the closed exceptions are the frontmatter tooling keys, an AD Decision Changelog tail and structure.md\'s Health Log tail' },
21
21
  ];
22
22
 
23
23
  // Sections inside a truth doc where dated history is sanctioned (the two bridges).
24
24
  const BRIDGE_HEADINGS = /^#{1,4}\s+(decision changelog|health log)\b/i;
25
- // Instruction files and generated trees are a different genre — out of this linter's scope.
26
- const SKIP = /(^|\/)(\.claude|node_modules|skills|rules|agents|harness|packages\/[^/]+\/data)\/|(^|\/)(CLAUDE|AGENTS)\.md$|\.bkp\./;
27
- const RECORD = /(^|\/)docs\/(history|research|memory)\//;
25
+ // Instruction files and generated trees are out of scope; the authoring roots are anchored to the repo root.
26
+ const SKIP = /^(\.claude|skills|rules|agents|harness)\/|(^|\/)(node_modules|packages\/[^/]+\/data)\/|(^|\/)(CLAUDE|AGENTS)\.md$|\.bkp\./;
27
+ // `operations/issues/YYYY-MM-DD-{slug}.md` is a sanctioned record genre (`type: issue`, the
28
+ // `issue-bridge` skill's local pack) living outside the record folders, so it is named here too —
29
+ // graded as a truth document it fails the date rule for carrying the date its filename already does.
30
+ const RECORD = /(^|\/)docs\/(history|research|memory)\/|(^|\/)docs\/operations\/issues\//;
28
31
 
29
- // Classify a path: skipped genre, record, or truth.
30
- function scopeOf(path) {
32
+ const RECORD_TYPES = /^(changelog-entry|plan|migration|migration-runbook|research|brainstorm|journal|record|ledger|issue|triage|project-memory|agent-memory)$/;
33
+ const TRUTH_LOCATIONS = /(^|\/)docs\/(features|architecture|tests|acceptance)\//;
34
+
35
+ // The genre a document declares for itself, from its own frontmatter `type:` key, or null.
36
+ function declaredType(text) {
37
+ if (!text.startsWith('---')) return null;
38
+ const end = text.indexOf('\n---', 3);
39
+ const m = /^type:\s*(\S+)\s*$/m.exec(end === -1 ? text : text.slice(0, end));
40
+ return m ? m[1] : null;
41
+ }
42
+
43
+ // Classify a document: record or truth, the declared genre outranking the path.
44
+ function scopeOf(path, text) {
31
45
  if (SKIP.test(path)) return 'skip';
32
- return RECORD.test(path) ? 'record' : 'truth';
46
+ if (RECORD.test(path)) return 'record';
47
+ const declared = text === undefined ? null : declaredType(text);
48
+ return declared && RECORD_TYPES.test(declared) ? 'record' : 'truth';
33
49
  }
34
50
 
35
51
  // Yield prose lines only: frontmatter, fenced code blocks, and bridge sections drop out.
@@ -37,13 +53,21 @@ function proseLines(text) {
37
53
  const lines = text.split('\n');
38
54
  const out = [];
39
55
  let inFence = false;
56
+ let fenceMarker = '';
40
57
  let inFrontmatter = lines[0] === '---';
41
58
  let inBridge = false;
42
59
  let bridgeLevel = 0;
43
60
  for (let i = inFrontmatter ? 1 : 0; i < lines.length; i++) {
44
61
  const line = lines[i];
45
62
  if (inFrontmatter) { if (line === '---') inFrontmatter = false; continue; }
46
- if (/^\s*(```|~~~|````)/.test(line)) { inFence = !inFence; continue; }
63
+ // A fence closes only on a marker at least as long as the one that opened it, so a nested ```yaml
64
+ // inside a ```markdown block does not invert the state and expose the outer block as prose.
65
+ const fence = /^\s*(`{3,}|~{3,})/.exec(line);
66
+ if (fence) {
67
+ if (!inFence) { inFence = true; fenceMarker = fence[1]; }
68
+ else if (fence[1][0] === fenceMarker[0] && fence[1].length >= fenceMarker.length) inFence = false;
69
+ continue;
70
+ }
47
71
  if (inFence) continue;
48
72
  const heading = line.match(/^(#{1,4})\s/);
49
73
  if (heading) {
@@ -64,9 +88,9 @@ function fileDisables(text) {
64
88
  }
65
89
 
66
90
  function lintFile(path, findings) {
67
- const scope = scopeOf(path);
68
- if (scope === 'skip' || !existsSync(path)) return;
91
+ if (SKIP.test(path) || !existsSync(path)) return;
69
92
  const text = readFileSync(path, 'utf8');
93
+ const scope = scopeOf(path, text);
70
94
  const disabled = fileDisables(text);
71
95
  const lines = proseLines(text);
72
96
 
@@ -85,6 +109,13 @@ function lintFile(path, findings) {
85
109
  }
86
110
  }
87
111
 
112
+ // A record genre declared in a settled truth location is a contradiction, not a classification.
113
+ if (TRUTH_LOCATIONS.test(path)) {
114
+ const declared = declaredType(text);
115
+ if (declared && RECORD_TYPES.test(declared) && !disabled.has('record-type-in-truth-location'))
116
+ findings.push({ path, n: 1, severity: 'block', id: 'record-type-in-truth-location', message: `\`type: ${declared}\` is a record genre declared in a truth location — move the file to \`docs/history/\` if it is a record, or give it its location's genre` });
117
+ }
118
+
88
119
  // A folder index (Readme.md/README.md) is a browse page, not a feature doc — it owns no code.
89
120
  const isFolderIndex = /(^|\/)(readme)\.md$/i.test(path);
90
121
  if (scope === 'truth' && !isFolderIndex && /(^|\/)docs\/features\/[^/]+\.md$/.test(path)) {
@@ -0,0 +1,225 @@
1
+ // skills/documentation/scripts/source-lint.mjs - mechanical checks over source comments (header path, density, tense, shouting)
2
+ // Usage: node source-lint.mjs <file | dir ...> | node source-lint.mjs --changed (git-changed + untracked source files)
3
+ // Exit 1 when any block-severity finding survives. Escape hatch: a `source-lint-disable: id[, id]` comment,
4
+ // file-wide in the first 20 lines or on the offending line; the reason goes in a second comment beside it.
5
+ //
6
+ // The convention is the harness `CLAUDE.md` § File Structure and is not restated here: a repo-relative path
7
+ // header of two or three present-tense sentences, one third-person sentence above each non-obvious function,
8
+ // rationale in the owning doc. This script checks only what is mechanical; voice and placement stay judgement.
9
+
10
+ import { readFileSync, existsSync, statSync, readdirSync } from 'node:fs';
11
+ import { execSync, spawnSync } from 'node:child_process';
12
+ import { basename, extname } from 'node:path';
13
+
14
+ // Comment syntax per extension, and whether a leading `#!` shebang may precede the header.
15
+ const DIALECTS = {
16
+ '.ts': '//', '.tsx': '//', '.mts': '//', '.cts': '//',
17
+ '.js': '//', '.jsx': '//', '.mjs': '//', '.cjs': '//',
18
+ '.rs': '//', '.go': '//', '.java': '//', '.kt': '//', '.swift': '//', '.c': '//', '.h': '//', '.cpp': '//',
19
+ '.css': '/*', '.scss': '//',
20
+ '.sql': '--',
21
+ '.py': '#', '.sh': '#', '.zsh': '#', '.bash': '#', '.rb': '#',
22
+ };
23
+
24
+ // Generated trees, vendored copies, and formats with no comment syntax are a different genre. `.claude/`
25
+ // is harness-owned in every consumer — its files are overwritten on the next roll, so a finding there
26
+ // names work nobody in that repo can land. The forge lints its own copies at `harness/`, not here.
27
+ const SKIP = /(^|\/)(node_modules|dist|build|target|\.git|\.claude|coverage|__screenshots__|data\/harnesses)\/|\.min\.|\.d\.ts$|(^|\/)(pnpm-lock|package-lock)\./;
28
+ // A generated file says so in its own first lines; the convention exempts it.
29
+ const GENERATED = /@generated|DO NOT EDIT|AUTO-GENERATED|This file was generated/i;
30
+ // Below this a file is too small for a density ratio to mean anything.
31
+ const RATIO_MIN_LINES = 40;
32
+ // A file-level ceiling, not a target. The house style sits near 14%; a file past this is carrying prose.
33
+ // It grades INHERITED density, so it only warns: a file somebody else wrote at 42% is not this session's
34
+ // debt, and a wall that blocks on it stops every commit in the repo until a rewrite pass lands.
35
+ const RATIO_CEILING = 0.35;
36
+ // The ceiling that actually holds the line, because it grades only the lines THIS change adds. The measured
37
+ // regression that prompted the rule ran at 28% on new files while the tree sat at 14% — a file-level 35%
38
+ // ceiling passes that cleanly, so the file-level rule cannot be what stops it. You own what you add.
39
+ const ADDED_RATIO_CEILING = 0.25;
40
+ // Below this an added-line ratio is noise — a two-line fix with one comment is not a prose problem.
41
+ const ADDED_MIN_LINES = 20;
42
+ // The header's own ceiling in wrapped lines, from the measured p99 of the convention's own sweep.
43
+ const HEADER_MAX_LINES = 6;
44
+ // A comment block longer than this is a document living in a source file.
45
+ const BLOCK_MAX_LINES = 12;
46
+
47
+ const RULES = {
48
+ 'header-missing': { severity: 'block', message: 'no file-path header comment — add one naming this file\'s repo-relative path' },
49
+ 'header-path-stale': { severity: 'block', message: 'the header names another path — a header a move left behind reads as current' },
50
+ 'header-too-long': { severity: 'warn', message: `header past ${HEADER_MAX_LINES} lines — two or three sentences, and the fuller account goes in its doc` },
51
+ 'comment-ratio': { severity: 'warn', message: `comment lines over ${Math.round(RATIO_CEILING * 100)}% of the file — the rationale belongs in the owning doc` },
52
+ 'comment-ratio-added': { severity: 'block', message: `over ${Math.round(ADDED_RATIO_CEILING * 100)}% of the lines this change adds are comments — one sentence per symbol, and the rationale goes in the owning doc` },
53
+ 'comment-block-long': { severity: 'warn', message: `a comment block past ${BLOCK_MAX_LINES} lines — one sentence per symbol; a paragraph is a doc, unless it names a booby trap` },
54
+ 'comment-narration': { severity: 'warn', message: 'history, a ticket or decision-record reference, or a dated note in a comment — the journey belongs in a record' },
55
+ 'comment-shouting': { severity: 'warn', message: 'an ALL-CAPS clause opener — comments are third-person present tense, not emphasis' },
56
+ };
57
+
58
+ const NARRATION = /\bused to\b|\bno longer\b|\bpreviously\b(?!-)|\bwas (renamed|removed|extracted|folded|migrated|replaced|ported)\b|\bported from\b|\bfor now\b|\bnot yet implemented\b|\b(AD|ADR)-\d+\b|\b20\d{2}-\d{2}-\d{2}\b/i;
59
+ const SHOUTING = /(?:^|[.:;]\s+)([A-Z][A-Z0-9 ,'`\-]{14,})(?=[.,:]|\s+[a-z])/;
60
+
61
+ // The comment token for a path, or null when the format has none.
62
+ function tokenFor(path) {
63
+ return DIALECTS[extname(path)] ?? null;
64
+ }
65
+
66
+ // True when the line is a comment line in this dialect (an open block, a continuation, or a line comment).
67
+ function isCommentLine(line, token) {
68
+ const t = line.trim();
69
+ if (t === '') return false;
70
+ if (token === '//') return t.startsWith('//') || t.startsWith('/*') || t.startsWith('*') || t.startsWith('*/');
71
+ if (token === '/*') return t.startsWith('/*') || t.startsWith('*') || t.startsWith('*/');
72
+ if (token === '--') return t.startsWith('--') || t.startsWith('/*') || t.startsWith('*');
73
+ return t.startsWith(token);
74
+ }
75
+
76
+ // The leading comment block: its lines, skipping a shebang and any blank lines above it.
77
+ function leadingBlock(lines, token) {
78
+ let i = 0;
79
+ if (lines[0]?.startsWith('#!')) i = 1;
80
+ while (i < lines.length && lines[i].trim() === '') i++;
81
+ const block = [];
82
+ while (i < lines.length && isCommentLine(lines[i], token)) {
83
+ block.push(lines[i]);
84
+ i++;
85
+ }
86
+ return block;
87
+ }
88
+
89
+ // Rule ids disabled file-wide (first 20 lines) or on one line.
90
+ function disabledIds(text) {
91
+ const head = text.split('\n').slice(0, 20).join('\n');
92
+ const ids = new Set();
93
+ for (const m of head.matchAll(/source-lint-disable:\s*([a-z0-9,\s-]+)/g))
94
+ for (const id of m[1].split(',')) ids.add(id.trim());
95
+ return ids;
96
+ }
97
+
98
+ // The comment ratio of the lines this change ADDS, or null where there is nothing to grade. An untracked
99
+ // file is added whole; a tracked one is graded on its `+` lines against HEAD. Reading the diff rather than
100
+ // the file is what makes the rule fair: inherited prose is somebody else's row, and a session that adds
101
+ // twenty clean lines to a 60%-comment file passes.
102
+ function addedRatio(path, token) {
103
+ let added;
104
+ try {
105
+ const tracked = spawnSync('git', ['ls-files', '--error-unmatch', path], { encoding: 'utf8' }).status === 0;
106
+ if (!tracked) added = readFileSync(path, 'utf8').split('\n');
107
+ else {
108
+ const out = spawnSync('git', ['diff', '-U0', '--no-color', 'HEAD', '--', path], { encoding: 'utf8' });
109
+ if (out.status !== 0) return null;
110
+ added = out.stdout.split('\n').filter((l) => l.startsWith('+') && !l.startsWith('+++')).map((l) => l.slice(1));
111
+ }
112
+ } catch {
113
+ return null;
114
+ }
115
+ const nonBlank = added.filter((l) => l.trim() !== '');
116
+ if (nonBlank.length < ADDED_MIN_LINES) return null;
117
+ const comments = nonBlank.filter((l) => isCommentLine(l, token)).length;
118
+ return { ratio: comments / nonBlank.length, comments, lines: nonBlank.length };
119
+ }
120
+
121
+ function lintFile(path, findings) {
122
+ if (SKIP.test(path)) return false;
123
+ const token = tokenFor(path);
124
+ if (!token) return false;
125
+ let text;
126
+ try {
127
+ text = readFileSync(path, 'utf8');
128
+ } catch {
129
+ return false;
130
+ }
131
+ if (GENERATED.test(text.split('\n').slice(0, 5).join('\n'))) return false;
132
+
133
+ const lines = text.split('\n');
134
+ const disabled = disabledIds(text);
135
+ const add = (id, n, excerpt) => {
136
+ if (disabled.has(id)) return;
137
+ const lineDisable = /source-lint-disable:\s*([a-z0-9,\s-]+)/.exec(lines[n - 1] ?? '');
138
+ if (lineDisable && lineDisable[1].split(',').some((x) => x.trim() === id)) return;
139
+ findings.push({ path, n, id, ...RULES[id], excerpt });
140
+ };
141
+
142
+ // --- the header ---------------------------------------------------------------------------------
143
+ const block = leadingBlock(lines, token);
144
+ if (block.length === 0) add('header-missing', 1);
145
+ else {
146
+ const head = block.join('\n');
147
+ // The FIRST path-shaped token, compared exactly — containment would pass a header reading
148
+ // `packages/old-name/src/app.ts` for a file at `src/app.ts`, which is the rename this catches.
149
+ const named = /([\w.@-]+(?:\/[\w.@-]+)*\.[A-Za-z]{1,5})\b/.exec(head);
150
+ if (!named) add('header-missing', 1);
151
+ else if (named[1] !== path) {
152
+ if (basename(named[1]) === basename(path)) add('header-path-stale', 1, named[1]);
153
+ else add('header-missing', 1, `names ${named[1]}`);
154
+ }
155
+ if (block.length > HEADER_MAX_LINES) add('header-too-long', 1, `${block.length} lines`);
156
+ }
157
+
158
+ // --- density, block length, and comment prose --------------------------------------------------
159
+ let commentLines = 0;
160
+ let nonBlank = 0;
161
+ let runStart = 0;
162
+ let run = 0;
163
+ lines.forEach((line, i) => {
164
+ if (line.trim() !== '') nonBlank++;
165
+ if (isCommentLine(line, token)) {
166
+ commentLines++;
167
+ if (run === 0) runStart = i + 1;
168
+ run++;
169
+ const body = line.replace(/^\s*(\/\/+|\/\*+|\*+\/?|--+|#+)\s?/, '');
170
+ if (NARRATION.test(body)) add('comment-narration', i + 1, body.slice(0, 90));
171
+ const shout = SHOUTING.exec(body);
172
+ if (shout) add('comment-shouting', i + 1, shout[1].slice(0, 60));
173
+ return;
174
+ }
175
+ if (run > BLOCK_MAX_LINES) add('comment-block-long', runStart, `${run} lines`);
176
+ run = 0;
177
+ });
178
+ if (run > BLOCK_MAX_LINES) add('comment-block-long', runStart, `${run} lines`);
179
+ if (nonBlank >= RATIO_MIN_LINES) {
180
+ const ratio = commentLines / nonBlank;
181
+ if (ratio > RATIO_CEILING) add('comment-ratio', 1, `${Math.round(ratio * 100)}% of ${nonBlank} lines`);
182
+ }
183
+ const grown = addedRatio(path, token);
184
+ if (grown && grown.ratio > ADDED_RATIO_CEILING)
185
+ add('comment-ratio-added', 1, `${grown.comments} of ${grown.lines} added lines`);
186
+ return true;
187
+ }
188
+
189
+ function expandDirs(paths) {
190
+ const out = [];
191
+ for (const p of paths) {
192
+ if (!existsSync(p)) continue;
193
+ if (!statSync(p).isDirectory()) {
194
+ out.push(p);
195
+ continue;
196
+ }
197
+ for (const e of readdirSync(p, { withFileTypes: true })) {
198
+ if (e.name === 'node_modules' || e.name.startsWith('.')) continue;
199
+ out.push(...expandDirs([`${p.replace(/\/$/, '')}/${e.name}`]));
200
+ }
201
+ }
202
+ return out;
203
+ }
204
+
205
+ const args = process.argv.slice(2);
206
+ let files = args.filter((a) => a !== '--changed');
207
+ if (args.includes('--changed')) {
208
+ const out = execSync('git diff --name-only HEAD; git ls-files --others --exclude-standard', { encoding: 'utf8', shell: '/bin/bash' });
209
+ files = [...new Set([...files, ...out.split('\n').filter(Boolean)])];
210
+ }
211
+ files = expandDirs(files);
212
+ if (!files.length) {
213
+ console.error('usage: node source-lint.mjs <file | dir ...> | --changed');
214
+ process.exit(2);
215
+ }
216
+
217
+ const findings = [];
218
+ let checked = 0;
219
+ for (const f of files) if (lintFile(f, findings)) checked++;
220
+
221
+ for (const f of findings.sort((a, b) => (a.severity === b.severity ? 0 : a.severity === 'block' ? -1 : 1)))
222
+ console.log(`${f.path}:${f.n} [${f.severity}] ${f.id} — ${f.message}${f.excerpt ? `: "${f.excerpt}"` : ''}`);
223
+ const blocks = findings.filter((f) => f.severity === 'block').length;
224
+ console.log(`\n${findings.length} finding(s), ${blocks} blocking, ${checked} file(s) checked.`);
225
+ process.exit(blocks ? 1 : 0);