paperlint 3.1.0 → 3.1.1

package/src/link-skills.mutations.mjs

@@ -1,140 +0,0 @@
-/**
- * Battery for `link-skills.ts`.
- *
- * Every case below reintroduces a way the skills stop being where Claude Code looks — or, worse,
- * a way the linker destroys something the consumer made. Both fail silently in real life: a
- * missing skill is simply absent from the listing, and a replaced directory is simply gone.
- */
-import { fileURLToPath } from "node:url";
-import { dirname, join, resolve } from "node:path";
-import { runMutations } from "../lib/mutation-driver.mjs";
-
-const HERE = dirname(fileURLToPath(import.meta.url));
-const ROOT = resolve(HERE, "..");
-const SRC = join(HERE, "link-skills.ts");
-const HARNESS = join(HERE, "link-skills.harness.mjs");
-const CONSUMER = join(
-  ROOT,
-  "skills",
-  "paper-pipeline",
-  "scripts",
-  "consumer.mjs",
-);
-
-process.exit(
-  runMutations({
-    root: ROOT,
-    runner: "node",
-    cases: [
-      {
-        // The definition lives in the port since paperlint#62 (`installedSkills` in consumer.mjs), so
-        // the defect is planted THERE and must still die in THIS harness — the evidence that the
-        // linker reads it through the shared function rather than through a copy of its own.
-        name: "any directory counts as a skill",
-        harness: HARNESS,
-        expect: "a directory without SKILL.md is not one",
-        disables:
-          "the definition of a skill. A helper directory beside the skills gets a link, and Claude " +
-          "Code lists an entry with no SKILL.md behind it",
-        edits: [
-          [
-            CONSUMER,
-            '    if (st.isDirectory() && existsSync(join(entry, "SKILL.md")))\n      names.push(name);',
-            "    if (st.isDirectory())\n      names.push(name);",
-          ],
-        ],
-      },
-      {
-        name: "the link is written absolute",
-        harness: HARNESS,
-        expect: "the link is RELATIVE",
-        disables:
-          "portability of the checkout. An absolute link works on the machine that ran init and " +
-          "dangles in every other clone, container and CI runner",
-        edits: [
-          [
-            SRC,
-            '    symlinkSync(target, entry, "dir");',
-            '    symlinkSync(resolve(dirname(entry), target), entry, "dir");',
-          ],
-        ],
-      },
-      {
-        name: "an existing correct link is not recognised",
-        harness: HARNESS,
-        expect: "a second run reports every link as ALREADY present",
-        disables:
-          "idempotency. Every re-run of init would report its own links as someone else's " +
-          "and tell the consumer to move them",
-        edits: [
-          [
-            SRC,
-            '    if (realpathSync(entry) === want) return { status: "present" };',
-            '    if (realpathSync(entry) === "") return { status: "present" };',
-          ],
-        ],
-      },
-      {
-        name: "a foreign entry is replaced instead of reported",
-        harness: HARNESS,
-        expect: "a foreign DIRECTORY is left untouched",
-        disables:
-          "the one promise that makes init safe to run on a real project: a consumer's own skill " +
-          "of the same name, or a link they aimed elsewhere, is deleted without a word",
-        edits: [
-          [
-            SRC,
-            '  symlinkSync,\n} from "node:fs";',
-            '  symlinkSync,\n  rmSync,\n} from "node:fs";',
-          ],
-          [
-            SRC,
-            '  if (!write || seen.status !== "missing")',
-            '  if (!write || seen.status === "present")',
-          ],
-          [
-            SRC,
-            '    symlinkSync(target, entry, "dir");',
-            '    rmSync(entry, { recursive: true, force: true });\n    symlinkSync(target, entry, "dir");',
-          ],
-        ],
-      },
-      {
-        name: "the resolved store path is linked instead of the project's own spelling",
-        harness: HARNESS,
-        expect: "under pnpm the link goes through node_modules/<package>",
-        disables:
-          "the link surviving an upgrade under pnpm. The store directory carries the version in " +
-          "its name; the next install removes it and every skill link dangles until init is re-run",
-        edits: [
-          [
-            SRC,
-            "      if (realpathSync(candidate) === dir) return { dir, spelled: candidate };",
-            '      if (realpathSync(candidate) === "") return { dir, spelled: candidate };',
-          ],
-        ],
-      },
-      {
-        name: "doctor's read-only call writes links",
-        harness: HARNESS,
-        expect:
-          "write:false reports every skill as missing and creates nothing",
-        disables:
-          "the difference between looking and changing. `paperlint doctor` would quietly repair the " +
-          "state it is supposed to report, so the report can never show the gap",
-        edits: [
-          [
-            SRC,
-            '  if (!write || seen.status !== "missing")',
-            '  if (seen.status !== "missing")',
-          ],
-          [
-            SRC,
-            "  if (write && shipped.names.length > 0) {",
-            "  if (shipped.names.length > 0) {",
-          ],
-        ],
-      },
-    ],
-  }),
-);

package/src/new-paper.mutations.mjs

@@ -1,65 +0,0 @@
-/**
- * Battery for `new-paper.ts`.
- *
- * Every case reintroduces a way `paperlint new` damages a paper folder or produces one the linter
- * refuses — the first a loss nobody notices until the text is gone, the second the exact
- * "missing file on the first run" this command was written to end.
- */
-import { fileURLToPath } from "node:url";
-import { dirname, join, resolve } from "node:path";
-import { runMutations } from "../lib/mutation-driver.mjs";
-
-const HERE = dirname(fileURLToPath(import.meta.url));
-const ROOT = resolve(HERE, "..");
-const SRC = join(HERE, "new-paper.ts");
-const HARNESS = join(HERE, "new-paper.harness.mjs");
-
-process.exit(
-  runMutations({
-    root: ROOT,
-    runner: "node",
-    cases: [
-      {
-        name: "an existing file is overwritten",
-        harness: HARNESS,
-        expect: "a second run overwrites nothing",
-        disables:
-          "the one promise that makes `paperlint new` safe on an existing folder. A scorecard with a " +
-          "year of journal in it is replaced by the empty template",
-        edits: [
-          [
-            SRC,
-            '    if (existsSync(join(dir, file))) {\n      files.push({ file, status: "kept" });\n      continue;\n    }\n',
-            "",
-          ],
-          [SRC, '{ encoding: "utf8", flag: "wx" }', '{ encoding: "utf8" }'],
-        ],
-      },
-      {
-        name: "a second source is added beside an existing one",
-        harness: HARNESS,
-        // The first owner is the re-run on a .tex paper asked for md: it must add no paper.md.
-        expect: "adds no second source",
-        disables:
-          "the migration of an old folder: a paper written in Markdown gains a stub paper.tex, " +
-          "and every tex rule starts linting a file the author never wrote",
-        edits: [
-          [
-            SRC,
-            "  return hasSource ? [STATUS_FILE] : [STATUS_FILE, SOURCE_FILE[format]];",
-            "  return [STATUS_FILE, SOURCE_FILE[format]];",
-          ],
-        ],
-      },
-      {
-        name: "a leading dot is accepted",
-        harness: HARNESS,
-        expect: "discovery skips dot-directories",
-        disables:
-          "the refusal of a name that would be created and then never linted — the scaffold " +
-          "would sit beside the papers, invisible to every check",
-        edits: [[SRC, '  if (name.startsWith("."))', "  if (false)"]],
-      },
-    ],
-  }),
-);

package/src/structure.mutations.mjs

@@ -1,83 +0,0 @@
-/**
- * Battery for `structure.ts` — the package's only check whose subject is a file's ABSENCE.
- *
- * A check like this is especially vulnerable to silent failure: it reports on what is missing,
- * so a broken one looks exactly like a clean corpus. This battery exists because the mutation gate
- * refused to accept a harness that nothing could kill.
- */
-import { fileURLToPath } from "node:url";
-import { dirname, join, resolve } from "node:path";
-import { runMutations } from "../lib/mutation-driver.mjs";
-
-const HERE = dirname(fileURLToPath(import.meta.url));
-const ROOT = resolve(HERE, "..");
-const SRC = join(HERE, "structure.ts");
-const HARNESS = join(HERE, "structure.harness.mjs");
-
-process.exit(
-  runMutations({
-    root: ROOT,
-    runner: "node",
-    cases: [
-      {
-        name: "generous detection is removed — any directory counts as a paper",
-        harness: HARNESS,
-        expect:
-          "a directory WITHOUT a single pipeline marker is left alone entirely",
-        disables:
-          "the condition the check uses at all to decide it is looking at a paper. Without it, " +
-          "findings would rain down on `data/`, `figures/` and any neighboring directory — and a " +
-          "rule that yells at the innocent gets turned off entirely, real findings and all",
-        edits: [
-          [
-            SRC,
-            "      if (!rules.markers.some((m: string) => existsSync(join(dir, m))))\n        continue;",
-            "      if (false) continue;",
-          ],
-        ],
-      },
-      {
-        name: "`ignore` stops exempting",
-        harness: HARNESS,
-        expect: "`ignore` exempts a directory by name",
-        disables:
-          'the consumer\'s only way to say "this directory is not a paper". Without it, an ' +
-          "exemption has to be expressed by renaming the directory",
-        edits: [
-          [SRC, "      if (rules.ignore.includes(name)) continue;\n", ""],
-        ],
-      },
-      {
-        name: "the second accepted source form falls out of the defaults",
-        harness: HARNESS,
-        expect: "`paper.md` counts on equal footing with `paper.tex`",
-        disables:
-          "the second source form the live corpus keeps. A paper on `paper.md` would become a " +
-          '"no source" finding while its source is right there — i.e. the check would yell at ' +
-          "the innocent, and such rules get turned off entirely, real findings and all",
-        edits: [
-          [
-            SRC,
-            'requireOneOf: [["paper.tex", "paper.md"]]',
-            'requireOneOf: [["paper.tex"]]',
-          ],
-        ],
-      },
-      {
-        name: "the path in a finding goes absolute again",
-        harness: HARNESS,
-        expect: "and the path is RELATIVE",
-        disables:
-          "the readability of the finding. An absolute path to the run's temp directory tells " +
-          "the reader nothing and on top of that makes the output useless for comparing across machines",
-        edits: [
-          [
-            SRC,
-            '    return rel &&\n      rel !== ".." &&\n      !rel.startsWith(`..${sep}`) &&\n      !isAbsolute(rel)\n      ? rel\n      : p;',
-            "    return p;",
-          ],
-        ],
-      },
-    ],
-  }),
-);