paperlint 3.1.2 → 3.2.0

package/scripts/test.test.ts

@@ -0,0 +1,78 @@
+/**
+ * `npm test`'s runner: both suites run whatever the first returned, each failure is named, and
+ * the harness arguments reach the harness runner only.
+ */
+import assert from "node:assert/strict";
+import { test } from "vitest";
+import { runSuites, SUITES, type Spawn } from "./test.ts";
+
+/** A spawn that records every call and exits with `status[file]` (`null` for a signal). */
+const fake = (status: Record<string, number | null>) => {
+  const calls: string[][] = [];
+  const spawn: Spawn = (file, args) => {
+    calls.push([file, ...args]);
+    return { status: status[file] ?? null };
+  };
+  return { calls, spawn };
+};
+
+test("a failing vitest does not skip the harnesses, and is named as the failure", () => {
+  const { calls, spawn } = fake({ vitest: 1, vigiles: 0 });
+  const said: string[] = [];
+  assert.equal(
+    runSuites(["--no-skip"], { spawn, err: (s) => said.push(s) }),
+    1,
+  );
+  assert.deepEqual(calls, [
+    ["vitest", "run"],
+    ["vigiles", "test", "--min=1", "--no-skip"],
+  ]);
+  assert.deepEqual(said, [
+    "✗ vitest failed (exit 1) — its own report is above",
+  ]);
+});
+
+test("both passing is 0 and silent; a harness killed by a signal is exit 1, named", () => {
+  const ok = fake({ vitest: 0, vigiles: 0 });
+  assert.equal(runSuites([], { spawn: ok.spawn, err: assert.fail }), 0);
+  const killed = fake({ vitest: 0, vigiles: null });
+  const said: string[] = [];
+  assert.equal(
+    runSuites([], { spawn: killed.spawn, err: (s) => said.push(s) }),
+    1,
+  );
+  assert.deepEqual(said, [
+    "✗ harnesses failed (exit 1) — its own report is above",
+  ]);
+  assert.deepEqual(
+    SUITES.map((s) => s.name),
+    ["vitest", "harnesses"],
+  );
+});
+
+test("every suite runs the SOURCES: the paperlint-source condition is in NODE_OPTIONS, the caller's kept", () => {
+  const seen: (string | undefined)[] = [];
+  const spawn: Spawn = (_file, _args, options) => {
+    seen.push(options.env.NODE_OPTIONS);
+    return { status: 0 };
+  };
+  assert.equal(
+    runSuites([], {
+      spawn,
+      err: assert.fail,
+      env: { NODE_OPTIONS: "--import=./x.ts" },
+    }),
+    0,
+  );
+  assert.deepEqual(seen, [
+    "--conditions=paperlint-source --import=./x.ts",
+    "--conditions=paperlint-source --import=./x.ts",
+  ]);
+  // Without a caller's NODE_OPTIONS there is no stray trailing space.
+  seen.length = 0;
+  runSuites([], { spawn, err: assert.fail, env: {} });
+  assert.deepEqual(seen, [
+    "--conditions=paperlint-source",
+    "--conditions=paperlint-source",
+  ]);
+});

package/scripts/test.ts

@@ -0,0 +1,87 @@
+/**
+ * `npm test`: the unit tests (vitest), then the harnesses (`vigiles test`). BOTH always run.
+ *
+ * Until #52 this was `vitest run && vigiles test --min=1`. Under `npm run coverage` a single
+ * failing vitest test then skipped every harness, and what the run printed last was c8's summary:
+ * lines, branches and functions each some six points below the floor. The one failing test sat
+ * above that, looking like a consequence rather than the cause. Running both suites keeps the
+ * coverage report whole, so a failing test is reported as itself and the coverage figures still
+ * mean what they say.
+ *
+ * Arguments are the harness runner's (`npm test -- --no-skip`), as they were before.
+ *
+ * Both suites run the SOURCES: every child gets `--conditions=paperlint-source`, so `#lib/*` and
+ * `#eslint-rules/*` resolve to `.ts`, not to `dist/`. This runner is the one owner of that
+ * setting. When only `npm run coverage` passed it, a plain `npm test` loaded the rules twice —
+ * `dist/` through `eslint.config.mjs`, `.ts` through the harness — and a harness comparing the two
+ * failed on macOS CI, the one job that runs `npm test` bare.
+ */
+import { spawnSync } from "node:child_process";
+import { isMain } from "../skills/paper-pipeline/scripts/consumer.mjs";
+
+/** The two suites, in order: what a failure is named as, and the command. */
+export interface Suite {
+  readonly name: string;
+  /** The program, then its arguments. */
+  readonly command: readonly [string, ...string[]];
+}
+
+/** What `runSuites` needs from a finished process: its exit status, `null` when a signal ended it. */
+export type Spawn = (
+  file: string,
+  args: readonly string[],
+  options: { stdio: "inherit"; env: NodeJS.ProcessEnv },
+) => { readonly status: number | null };
+
+/** The condition that points package `imports` at the TypeScript sources instead of `dist/`. */
+const SOURCE_CONDITION = "--conditions=paperlint-source";
+
+/** `env` with the source condition first in NODE_OPTIONS, whatever the caller already had kept. */
+export function withSources(env: NodeJS.ProcessEnv): NodeJS.ProcessEnv {
+  const own = env.NODE_OPTIONS?.trim();
+  return {
+    ...env,
+    NODE_OPTIONS: own ? `${SOURCE_CONDITION} ${own}` : SOURCE_CONDITION,
+  };
+}
+
+export const SUITES: readonly Suite[] = [
+  { name: "vitest", command: ["vitest", "run"] },
+  { name: "harnesses", command: ["vigiles", "test", "--min=1"] },
+];
+
+/**
+ * Run every suite, whatever the one before it returned, and name the ones that failed.
+ * @returns 0 when every suite passed; otherwise the first failing suite's exit code (1 for a signal).
+ */
+export function runSuites(
+  args: readonly string[],
+  {
+    spawn = spawnSync,
+    err = console.error,
+    suites = SUITES,
+    env = process.env,
+  }: {
+    spawn?: Spawn;
+    err?: (line: string) => void;
+    suites?: readonly Suite[];
+    env?: NodeJS.ProcessEnv;
+  } = {},
+): number {
+  const failed = suites.flatMap(({ name, command }) => {
+    const [file, ...rest] = command;
+    const r = spawn(file, [...rest, ...(name === "harnesses" ? args : [])], {
+      stdio: "inherit",
+      env: withSources(env),
+    });
+    return r.status === 0 ? [] : [{ name, code: r.status ?? 1 }];
+  });
+  for (const f of failed)
+    err(
+      `✗ ${f.name} failed (exit ${String(f.code)}) — its own report is above`,
+    );
+  const [first] = failed;
+  return first === undefined ? 0 : first.code;
+}
+
+if (isMain(import.meta.url)) process.exit(runSuites(process.argv.slice(2)));

package/skills/grade-paper-writing/prose-lint.mjs

@@ -70,7 +70,7 @@ import {
   stripFences,
   requireMarkdown,
   stripFrontmatter,
-} from "../../lib/markdown.mjs";
+} from "#lib/markdown";
 
 // Markup is parsed with a PARSER (`CLAUDE.md`, 2026-08-11). We fail rather than degrade, even
 // though this file is advisory: without a parser the paper's body would collapse to empty, and from

package/skills/grade-paper-writing/{prose-lint.test.mjs → prose-lint.test.ts}

@@ -8,14 +8,14 @@ import { join } from "node:path";
 import { dirname } from "node:path";
 import { fileURLToPath } from "node:url";
 import { expect, test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../test/support.ts";
 
 const SCRIPT = join(dirname(fileURLToPath(import.meta.url)), "prose-lint.mjs");
 const root = useTempDir("prose-lint-");
 
 const LONG_CAPTION_SENTENCE = Array.from(
   { length: 45 },
-  (_, i) => `word${i}`,
+  (_, i) => `word${String(i)}`,
 ).join(" ");
 const PAPER = `---
 title: A title the linter must not read as a sentence

package/skills/paper-pipeline/description-language.eval.mjs

@@ -82,7 +82,7 @@ import { assertPromptDiversity, skillResolved, skip } from "vigiles";
 import { paid_measureTriggerRate as measureTriggerRate } from "vigiles/eval";
 import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
-import { frontmatterBlock } from "../../lib/markdown.mjs";
+import { frontmatterBlock } from "#lib/markdown";
 import { parseFm } from "../../lib/skill-corpus.mjs";
 import { installedSkills } from "./scripts/consumer.mjs";
 

package/skills/paper-pipeline/framing-vs-vocabulary.eval.mjs

@@ -250,7 +250,7 @@ import { assertPromptDiversity, skip } from "vigiles";
 import { paid_measureTriggerRate as measureTriggerRate } from "vigiles/eval";
 import { existsSync, readFileSync, writeFileSync, mkdirSync } from "node:fs";
 import { join } from "node:path";
-import { frontmatterBlock } from "../../lib/markdown.mjs";
+import { frontmatterBlock } from "#lib/markdown";
 import { parseFm } from "../../lib/skill-corpus.mjs";
 import { installedSkills } from "./scripts/consumer.mjs";
 

package/skills/paper-pipeline/grade-paper-writing-ablation.eval.mjs

@@ -375,7 +375,7 @@ import {
   realpathSync,
 } from "node:fs";
 import { join } from "node:path";
-import { frontmatterBlock } from "../../lib/markdown.mjs";
+import { frontmatterBlock } from "#lib/markdown";
 import { parseFm } from "../../lib/skill-corpus.mjs";
 import { installedSkills } from "./scripts/consumer.mjs";
 import { execFileSync } from "node:child_process";

package/skills/paper-pipeline/pipeline-firing.eval.mjs

@@ -180,7 +180,7 @@ import {
 import { paid_measureTriggerRate as measureTriggerRate } from "vigiles/eval";
 import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
-import { frontmatterBlock } from "../../lib/markdown.mjs";
+import { frontmatterBlock } from "#lib/markdown";
 import { parseFm } from "../../lib/skill-corpus.mjs";
 import { installedSkills } from "./scripts/consumer.mjs";
 

package/skills/paper-pipeline/pipeline-language.eval.mjs

@@ -137,7 +137,7 @@ import {
 import { paid_measureTriggerRate as measureTriggerRate } from "vigiles/eval";
 import { existsSync, readFileSync } from "node:fs";
 import { join } from "node:path";
-import { frontmatterBlock } from "../../lib/markdown.mjs";
+import { frontmatterBlock } from "#lib/markdown";
 import { parseFm } from "../../lib/skill-corpus.mjs";
 import { installedSkills } from "./scripts/consumer.mjs";
 

package/skills/paper-pipeline/scripts/README.md

@@ -172,7 +172,7 @@ that fires on correct text is muted within a day, which is worse than one that m
 | `generated-code.harness.mjs`                   | `generated-code.mjs`                                                                | an analysis script that draws randomness and never seeds it · one that hard-codes an absolute path · one that reads and then overwrites its own input (literal and `argv` forms). Plus every quiet case, which is where this checker lives or dies: a seed threaded through `--seed`, a lowercase Express route, a third party's relative path, a read-here-write-there script, a variable name reused across two loops — and the released bundle, which `check-anon.sh` cat. 5 owns and this one must not enter                                                                                                                    |
 | `pipeline-check.harness.mjs`                   | `pipeline-check.mjs`                                                                | six defects, **one at a time**, asserting the exact finding set. 🔴 Since 2026-08-26 here only checks whose input is OUTSIDE the file (git · clock · `reviews/` · `process.env`); sixteen others moved to `eslint-rules/pipeline-status.mjs` and are checked by `eslint-rules/pipeline-status.harness.mjs`                                                                                                                                                                                                                                                                                                                          |
 | `paper-lint.harness.mjs`                       | `paper-lint.mjs`                                                                    | text moving under an unchanged scorecard · appendix outweighing the body · a shaved passage                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                         |
-| `round-diff.harness.mjs`                       | `paper-pipeline/scripts/round-diff.mjs`                                             | a section changed outside the round's declaration (and one added, and one **removed**) · a cite and a bare bibliography entry arriving mid-round · a numeric literal arriving mid-round, with restating an existing one asserted FREE · the body over its budget · **three rounds compounding past the sum of their budgets while the open round is inside its own** · hedge density rising · a `touches:` list covering the paper · a base that does not resolve · edits with every round closed. Plus the clean case: a legal round must produce total silence                                                                    |
+| `round-diff.harness.ts`                        | `paper-pipeline/scripts/round-diff.mjs`                                             | a section changed outside the round's declaration (and one added, and one **removed**) · a cite and a bare bibliography entry arriving mid-round · a numeric literal arriving mid-round, with restating an existing one asserted FREE · the body over its budget · **three rounds compounding past the sum of their budgets while the open round is inside its own** · hedge density rising · a `touches:` list covering the paper · a base that does not resolve · edits with every round closed. Plus the clean case: a legal round must produce total silence                                                                    |
 | `delivered-pdf.harness.mjs`                    | `repro/delivered_pdf.py`                                                            | a quantity deleted from the built page · **a superscript minus lost in typesetting, compiled and read back** · a value moved away from its claim · an occurrence whose prose the extractor lost · a missing, unreadable and stale PDF · and all six normalisation rules, one assertion each                                                                                                                                                                                                                                                                                                                                         |
 | `bound-numbers.harness.mjs`                    | `repro/paper_numbers.py`, `md2submission.py`, `latex-build.sh`                      | a digit in a `\newcommand` name · two registry keys mangling to one macro · an unescaped `%` in the generated values · a converter handling the digits again · a missing `\input` · an unbound name that must stop the build with **no PDF** · a failing pass that leaves its PDF on disk. Plus the CONTROL: a hand-typed literal still compiles, **asserted** so nobody writes that this makes a wrong number unrepresentable                                                                                                                                                                                                      |
 | `textidote.harness.mjs`                        | `repro/textidote_check.py` (TeXtidote v0.9)                                         | a misspelling planted into real prose from the paper, live through the pinned jar · a coinage with a committed row staying silent in the same run · the flagged span truncated by one letter · the caret row leaking into the message · each of the three exclusions (`{{macro}}`, `` `code` ``, the reference list) · **an appendix AFTER `## References` still being judged** · a key that stops discriminating by rule, by word, or by case · a tolerated row moved 40 lines down and indented · the tolerated set, its reasons and the stale rows going unprinted · a missing jar or JRE answering with an empty findings array |

package/skills/paper-pipeline/scripts/announce.test.mjs

@@ -8,7 +8,7 @@ import { cpSync, readFileSync, rmSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { afterEach, test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 
 const HERE = dirname(fileURLToPath(import.meta.url));
 const SCRIPT = join(HERE, "announce.mjs");

package/skills/paper-pipeline/scripts/artifact-coverage.mjs

@@ -65,7 +65,7 @@ import {
   headings as mdHeadings,
   splitSections,
   requireMarkdown,
-} from "../../../lib/markdown.mjs";
+} from "#lib/markdown";
 
 // Markup is parsed with a parser (`CLAUDE.md`, 2026-08-11). We fail rather than degrade: without
 // the parser NOT A SINGLE section would be found in the paper, and the check "every section with a

package/skills/paper-pipeline/scripts/{artifact-coverage.test.mjs → artifact-coverage.test.ts}

@@ -8,14 +8,14 @@ import assert from "node:assert/strict";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 
 const SCRIPT = join(
   dirname(fileURLToPath(import.meta.url)),
   "artifact-coverage.mjs",
 );
 const root = useTempDir("artifact-coverage-");
-const run = (...args) => runNode(SCRIPT, args);
+const run = (...args: string[]) => runNode(SCRIPT, args);
 
 test("no argument prints the usage; a paper with no markdown source is a named skip", () => {
   assert.deepEqual(run(), {

package/skills/paper-pipeline/scripts/consumer.mjs

@@ -85,11 +85,7 @@ import { fileURLToPath, pathToFileURL } from "node:url";
 
 // Re-exported, not re-declared — see the note in `lib/paper-config.mjs`. This file is not a hook
 // and never needed its own copy.
-import {
-  CONFIG_FILE,
-  CONFIG_KEY,
-  settingsOf,
-} from "../../../lib/paper-config.mjs";
+import { CONFIG_FILE, CONFIG_KEY, settingsOf } from "#lib/paper-config";
 export { CONFIG_KEY, settingsOf };
 
 /**

package/skills/paper-pipeline/scripts/consumer.test.mjs

@@ -6,7 +6,7 @@ import assert from "node:assert/strict";
 import { mkdirSync, symlinkSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { afterEach, test } from "vitest";
-import { useTempDir, writeTree } from "../../../test/support.mjs";
+import { useTempDir, writeTree } from "../../../test/support.ts";
 import {
   consumerContactEmail,
   consumerTimezone,

package/skills/paper-pipeline/scripts/extract-ref-facts.harness.mjs

@@ -25,7 +25,7 @@ import {
   DEFAULT_PAPERS_ROOT,
   PAPERS_DIR_FIELD,
   settingsOf,
-} from "../../../lib/paper-config.mjs";
+} from "#lib/paper-config";
 import {
   existsSync,
   mkdirSync,

package/skills/paper-pipeline/scripts/extract-ref-facts.mjs

@@ -71,10 +71,7 @@ import {
 } from "node:fs";
 import { createHash } from "node:crypto";
 import { join, resolve, dirname, basename } from "node:path";
-import {
-  headings as mdHeadings,
-  requireMarkdown,
-} from "../../../lib/markdown.mjs";
+import { headings as mdHeadings, requireMarkdown } from "#lib/markdown";
 import { isMain } from "./consumer.mjs";
 
 // Markup is parsed with a parser (`CLAUDE.md`, 2026-08-11). We fail rather than degrade: without

package/skills/paper-pipeline/scripts/extract-ref-facts.test.mjs

@@ -8,7 +8,7 @@ import { readFileSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 import {
   buildFacts,
   extractIds,

package/skills/paper-pipeline/scripts/fixtures/{scorecard.mjs → scorecard.ts}

@@ -4,6 +4,34 @@
  */
 export const FAR = "2027-12-31";
 
+/** What a test may replace in one row: any cell, and for a GATES row its `Requires` cell. */
+export interface RowOverride {
+  readonly status?: string;
+  readonly date?: string;
+  readonly skill?: string;
+  readonly result?: string;
+  readonly requires?: string;
+}
+
+/** Per-row overrides by row id, plus the four free-text parts of the file. */
+export interface ScorecardOverrides {
+  readonly verdict?: string;
+  readonly deadline?: string;
+  readonly tail?: string;
+  readonly preamble?: string;
+  readonly [id: string]: RowOverride | string | undefined;
+}
+
+interface Row extends RowOverride {
+  readonly id: string;
+  readonly status: string;
+  readonly date: string;
+}
+
+/** A row's overrides; the free-text keys never name a row, so a string here is none. */
+const cellsOf = (v: RowOverride | string | undefined): RowOverride =>
+  typeof v === "object" ? v : {};
+
 /**
  * A scorecard with the four sections the template requires, all green. `over` replaces individual
  * cells: `{ cites: { date: "2020-01-01" } }`, `{ verdict: "…" }`, `{ deadline: "2026-08-18" }`,
@@ -11,22 +39,23 @@ export const FAR = "2027-12-31";
  * inserted BEFORE the first heading (the only way to plant a row under no heading at all).
  */
 export function scorecard(
-  over = {},
-  today = new Date().toISOString().slice(0, 10),
-) {
-  const r = (id, status = "☑", date = today) => ({
+  over: ScorecardOverrides = {},
+  today: string = new Date().toISOString().slice(0, 10),
+): string {
+  const r = (id: string, status = "☑", date = today): Row => ({
     id,
     status,
     date,
-    ...(over[id] ?? {}),
+    ...cellsOf(over[id]),
   });
-  const row = (o, ...mid) =>
+  const row = (o: Row, ...mid: string[]): string =>
     `| ${o.id} | work | ${o.skill ?? "skill"} | ${mid.length ? mid.join(" | ") + " | " : ""}${o.status} | ${o.date} | ${o.result ?? "result"} | — |`;
   // The GATES `Requires` cell, canonical by default so the baseline is silent — `{ structure: {
   // requires: "render" } }` plants a dropped edge. `harden` deliberately declares four of its five
   // canonical inputs: the fifth (`priordelta`) has no row in this fixture, and the clean case asserts that the checker stays
   // quiet about an edge there is nowhere to point at.
-  const req = (id, dflt) => over[id]?.requires ?? dflt;
+  const req = (id: string, dflt: string): string =>
+    cellsOf(over[id]).requires ?? dflt;
   const verdict =
     over.verdict ??
     "Submit-ready — every gate green, artifact reproduces clean, nothing blocking.";

package/skills/paper-pipeline/scripts/{generated-code.test.mjs → generated-code.test.ts}

@@ -8,14 +8,14 @@ import { symlinkSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 
 const SCRIPT = join(
   dirname(fileURLToPath(import.meta.url)),
   "generated-code.mjs",
 );
 const root = useTempDir("generated-code-");
-const run = (...args) => runNode(SCRIPT, args);
+const run = (...args: string[]) => runNode(SCRIPT, args);
 
 test("no argument prints the usage; a directory that does not exist scanned nothing, and says so", () => {
   assert.deepEqual(run(), {
@@ -41,7 +41,7 @@ test("repro/: a dangling link is passed over; skipped trees are named, more than
   const dir = writeTree(join(root, "paper"), {
     ...deep,
     "repro/.cache/x.py": "",
-    // eslint-disable-next-line port/js-install-path -- a fixture directory the scan must skip, not an install location
+    // A fixture directory the scan must skip, not an install location.
     "repro/node_modules/x.js": "",
     "repro/__pycache__/x.py": "",
     // Draws randomness unseeded: a NO_SEED finding, which the reasonless row below waves through.

package/skills/paper-pipeline/scripts/ledger.cli.test.mjs

@@ -11,7 +11,7 @@ import { cpSync, rmSync, writeFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { afterEach, test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 
 const HERE = dirname(fileURLToPath(import.meta.url));
 const SCRIPT = join(HERE, "ledger.mjs");

package/skills/paper-pipeline/scripts/pipeline-check.cli.test.mjs

@@ -10,10 +10,10 @@ import { utimesSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 import { check, newestSourceDate, parseStatus } from "./pipeline-check.mjs";
 import { readdirSync } from "node:fs";
-import { scorecard } from "./fixtures/scorecard.mjs";
+import { scorecard } from "./fixtures/scorecard.ts";
 
 const SCRIPT = join(
   dirname(fileURLToPath(import.meta.url)),

package/skills/paper-pipeline/scripts/pipeline-check.harness.mjs

@@ -5,7 +5,7 @@
  * moved to `eslint-rules/pipeline-status.mjs`, and their cases — to `eslint-rules/pipeline-status.harness.mjs`.
  * `npx vigiles test .claude/skills/paper-pipeline/scripts/pipeline-check.harness.mjs`.
  *
- * WHY THIS EXISTS ALONGSIDE THE FIXTURE TEST IN hooks.harness.mjs. That one runs the checker over
+ * WHY THIS EXISTS ALONGSIDE THE FIXTURE TEST IN hooks.harness.ts. That one runs the checker over
  * `scripts/fixtures/dirty/`, a scorecard carrying every defect at once, and asserts each kind
  * appears somewhere in the output. It cannot answer the question that matters when a check breaks:
  * WHICH defect produced WHICH finding. A checker that reports `stale-continuous` for the wrong
@@ -40,7 +40,7 @@ import { tmpdir } from "node:os";
 import { join, dirname } from "node:path";
 import { fileURLToPath } from "node:url";
 import { consumerRoot } from "./consumer.mjs";
-import { scorecard as makeScorecard } from "./fixtures/scorecard.mjs";
+import { scorecard as makeScorecard } from "./fixtures/scorecard.ts";
 const HERE = dirname(fileURLToPath(import.meta.url));
 
 const ROOT = consumerRoot();

package/skills/paper-pipeline/scripts/pipeline-check.mjs

@@ -109,10 +109,7 @@ import { readFileSync, readdirSync, statSync, existsSync } from "node:fs";
 import { dirname, join, resolve } from "node:path";
 import { execFileSync } from "node:child_process";
 import MarkdownIt from "markdown-it";
-import {
-  headings as mdHeadings,
-  requireMarkdown,
-} from "../../../lib/markdown.mjs";
+import { headings as mdHeadings, requireMarkdown } from "#lib/markdown";
 import { isMain } from "./consumer.mjs";
 
 // Markup is parsed with a parser (`CLAUDE.md`, 2026-08-11). This script already parsed tables

package/skills/paper-pipeline/scripts/population-map.mjs

@@ -48,10 +48,7 @@
  */
 import { readFileSync, existsSync } from "node:fs";
 import { join } from "node:path";
-import {
-  headings as mdHeadings,
-  requireMarkdown,
-} from "../../../lib/markdown.mjs";
+import { headings as mdHeadings, requireMarkdown } from "#lib/markdown";
 import { isMain } from "./consumer.mjs";
 
 // Markup is parsed with a parser (`CLAUDE.md`, 2026-08-11). We fail rather than degrade: without

package/skills/paper-pipeline/scripts/population-map.test.mjs

@@ -11,7 +11,7 @@ import assert from "node:assert/strict";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 import { test } from "vitest";
-import { runNode, useTempDir, writeTree } from "../../../test/support.mjs";
+import { runNode, useTempDir, writeTree } from "../../../test/support.ts";
 import { bodyOf, findings } from "./population-map.mjs";
 
 const SCRIPT = join(