paperlint 2.1.0 → 3.0.0

package/src/doctor.harness.mjs

@@ -12,10 +12,7 @@
  * Killed by: src/doctor.mutations.mjs
  */
 import assert from "node:assert/strict";
-import {
-  OLD_PAPERS_DIR_FIELD,
-  PAPERS_DIR_FIELD,
-} from "../lib/paper-config.mjs";
+import { PAPERS_DIR_FIELD } from "../lib/paper-config.mjs";
 import {
   mkdtempSync,
   mkdirSync,
@@ -41,8 +38,8 @@ const check = (label, cond) => {
   assert.ok(cond, label);
 };
 
-/** A consumer on disk: a papers directory, and maybe a declaration in package.json. */
-function consumer({ papersDir, pkgKey, makeDir = true }) {
+/** A consumer on disk: a papers directory, and maybe a declaration in the root paperlint.json. */
+function consumer({ papersDir, declared, makeDir = true }) {
   const dir = realpathSync(mkdtempSync(join(tmpdir(), "paperlint-doctor-")));
   if (makeDir && papersDir) {
     mkdirSync(join(dir, papersDir, "some-paper"), { recursive: true });
@@ -51,9 +48,15 @@ function consumer({ papersDir, pkgKey, makeDir = true }) {
       "# s\n",
     );
   }
-  const pkg = { name: "consumer", version: "1.0.0" };
-  if (pkgKey !== undefined) pkg["paperlint"] = { [PAPERS_DIR_FIELD]: pkgKey };
-  writeFileSync(join(dir, "package.json"), JSON.stringify(pkg, null, 2));
+  writeFileSync(
+    join(dir, "package.json"),
+    JSON.stringify({ name: "consumer", version: "1.0.0" }, null, 2),
+  );
+  if (declared !== undefined)
+    writeFileSync(
+      join(dir, "paperlint.json"),
+      JSON.stringify({ [PAPERS_DIR_FIELD]: declared }, null, 2),
+    );
   return dir;
 }
 
@@ -76,7 +79,7 @@ const runDoctor = (
 
 // ── I. A CONSISTENT INSTALL IS SILENT ───────────────────────────────────────────────────────
 {
-  const dir = consumer({ papersDir: "papers", pkgKey: "papers" });
+  const dir = consumer({ papersDir: "papers", declared: "papers" });
   const r = runDoctor(dir, { cliPapers: "papers" });
   check("a consistent install — exit ZERO", r.code === 0);
   check(
@@ -86,30 +89,9 @@ const runDoctor = (
   rmSync(dir, { recursive: true, force: true });
 }
 
-// ── I½. THE OLD FIELD NAME IS A FAILURE, NAMED ─────────────────────────────────────────────
-{
-  const dir = consumer({ papersDir: "papers", pkgKey: "papers" });
-  writeFileSync(
-    join(dir, "package.json"),
-    JSON.stringify({
-      name: "consumer",
-      paperlint: { [OLD_PAPERS_DIR_FIELD]: "papers" },
-    }),
-  );
-  const r = runDoctor(dir, { cliPapers: null });
-  check("the old field name — exit NON-zero", r.code !== 0);
-  check(
-    "and doctor says what the field was renamed to",
-    r.out.includes(
-      `"${OLD_PAPERS_DIR_FIELD}" was renamed to "${PAPERS_DIR_FIELD}" in package.json → "paperlint"`,
-    ),
-  );
-  rmSync(dir, { recursive: true, force: true });
-}
-
 // ── II. THE VERY DEFECT: AN INSTALL FOLLOWING THE DOCS ─────────────────────────────────────
-// The old `init` wrote its own config file and did not touch package.json; the hook reads package.json.
-// Measured 09-18.
+// The CLI lints one directory and the hook guards another: the install that motivated doctor
+// (issue #33). Here the CLI was pointed elsewhere while the hook took the default.
 {
   const dir = consumer({ papersDir: "writing/drafts" });
   const r = runDoctor(dir, { cliPapers: "writing/drafts" });
@@ -137,21 +119,31 @@ const runDoctor = (
   rmSync(dir, { recursive: true, force: true });
 }
 
-// ── II-bis. A MISSING DECLARATION THAT DOES NO HARM YET ────────────────────────────────────
-// The papers live exactly where the hook's default points. The install WORKS — by coincidence.
-// It must not fail here (an error-level false positive costs more than a miss), but it must
-// not stay silent either.
+// ── II-bis. NO paperlint.json — EVERY DEFAULT, AND THAT IS A VALID PROJECT ─────────────────
+// The CLI and the hooks both take `papers` when nothing is declared, so they cannot disagree.
 {
   const dir = consumer({ papersDir: "papers" });
   const r = runDoctor(dir, { cliPapers: "papers" });
-  check("an install that works by coincidence does NOT crash", r.code === 0);
+  check("no paperlint.json — exit ZERO", r.code === 0);
   check(
-    "but the missing declaration is NAMED, not skipped",
-    /⚠ package\.json has no "paperlint"/.test(r.out),
+    "and it says the defaults are in play, naming the directory",
+    /✓ no paperlint\.json — every setting has its default; papers are in "papers"/.test(
+      r.out,
+    ),
   );
+  rmSync(dir, { recursive: true, force: true });
+}
+
+// ── II-quater. A paperlint.json THAT DOES NOT PARSE — the guard refuses every command ─────
+{
+  const dir = consumer({ papersDir: "papers" });
+  writeFileSync(join(dir, "paperlint.json"), '{ "papersDir": <<<<<<< HEAD\n');
+  const r = runDoctor(dir, { cliPapers: "papers" });
+  check("an unparsable paperlint.json — a FAILURE", r.code === 2);
   check(
-    "and it says exactly why that is risky — it works only until the directory moves",
-    /works only while your papers happen to live there/.test(r.out),
+    "named, with the way out (a file edit, not a command)",
+    /✗ paperlint\.json is not valid JSON/.test(r.out) &&
+      /Edit or Write/.test(r.out),
   );
   rmSync(dir, { recursive: true, force: true });
 }
@@ -163,7 +155,7 @@ const runDoctor = (
 {
   const dir = consumer({
     papersDir: "papers",
-    pkgKey: "papers",
+    declared: "papers",
     makeDir: false,
   });
   const r = runDoctor(dir, { cliPapers: "papers" });
@@ -177,42 +169,9 @@ const runDoctor = (
   rmSync(dir, { recursive: true, force: true });
 }
 
-// ── II-quater. THE KEY'S OLD NAME IS READ AND NAMED; TWO DIFFERENT KEYS ARE A FAILURE ──────
-{
-  const dir = consumer({ papersDir: "papers" });
-  writeFileSync(
-    join(dir, "package.json"),
-    JSON.stringify({
-      "research-paper-pipeline": { [PAPERS_DIR_FIELD]: "papers" },
-    }),
-  );
-  const r = runDoctor(dir, { cliPapers: "papers" });
-  check(
-    "the old key — read (not a failure), and named with the new one",
-    r.code === 0 &&
-      /⚠ "research-paper-pipeline" in package\.json is the old name/.test(
-        r.out,
-      ) &&
-      /the hooks will guard papers/.test(r.out),
-  );
-  writeFileSync(
-    join(dir, "package.json"),
-    JSON.stringify({
-      paperlint: { [PAPERS_DIR_FIELD]: "papers" },
-      "research-paper-pipeline": { [PAPERS_DIR_FIELD]: "elsewhere" },
-    }),
-  );
-  const both = runDoctor(dir, { cliPapers: "papers" });
-  check(
-    "both keys with different contents — a failure",
-    both.code === 2 && /✗ package\.json has both/.test(both.out),
-  );
-  rmSync(dir, { recursive: true, force: true });
-}
-
 // ── III. TWO DECLARATIONS HAVE DRIFTED APART ────────────────────────────────────────────────
 {
-  const dir = consumer({ papersDir: "writing/drafts", pkgKey: "papers" });
+  const dir = consumer({ papersDir: "writing/drafts", declared: "papers" });
   mkdirSync(join(dir, "papers"), { recursive: true });
   const r = runDoctor(dir, { cliPapers: "writing/drafts" });
   check("both declarations exist, but differ — a FAILURE", r.code === 2);
@@ -223,12 +182,10 @@ const runDoctor = (
 // This is load-bearing: a copy of the logic would drift silently and print a confident wrong
 // answer.
 {
-  const dir = consumer({ papersDir: "docs/papers", pkgKey: "docs/papers/" });
+  const dir = consumer({ papersDir: "docs/papers", declared: "docs/papers/" });
   const r = runDoctor(dir, { cliPapers: "docs/papers" });
   const fromHook = papersRoot(
-    JSON.stringify({
-      paperlint: { [PAPERS_DIR_FIELD]: "docs/papers/" },
-    }),
+    JSON.stringify({ [PAPERS_DIR_FIELD]: "docs/papers/" }),
   );
   check("the hook itself trims the trailing slash", fromHook === "docs/papers");
   check(
@@ -244,7 +201,7 @@ const runDoctor = (
 
 // ── V. THE HOOK'S REFUSAL IS PASSED ALONG, NOT TURNED INTO A DIRECTORY ─────────────────────
 {
-  const dir = consumer({ papersDir: "papers", pkgKey: "" });
+  const dir = consumer({ papersDir: "papers", declared: "" });
   const r = runDoctor(dir, { cliPapers: "papers" });
   check(
     "an empty string in the declaration — the hook refuses, and doctor NAMES it",
@@ -255,7 +212,7 @@ const runDoctor = (
 
 // ── VI. EXTERNAL PROGRAMS ARE A FACT, NOT A VERDICT ─────────────────────────────────────────
 {
-  const dir = consumer({ papersDir: "papers", pkgKey: "papers" });
+  const dir = consumer({ papersDir: "papers", declared: "papers" });
   const none = runDoctor(dir, { cliPapers: "papers", have: () => 1 });
   check(
     "🔴 a missing tex install does NOT fail the run — a gate on advice would mute the whole thing",
@@ -330,7 +287,7 @@ const runDoctor = (
 // nothing anywhere said so. The link state itself is `link-skills.harness.mjs`'s subject; here
 // only doctor's REPORTING of it is judged, so the state is handed in rather than built on disk.
 {
-  const dir = consumer({ papersDir: "papers", pkgKey: "papers" });
+  const dir = consumer({ papersDir: "papers", declared: "papers" });
   const state = {
     ok: true,
     home: join(dir, ".claude", "skills"),

package/src/doctor.mutations.mjs

@@ -46,8 +46,8 @@ process.exit(
         edits: [
           [
             SRC,
-            "  const hookRoot = rawPkg ? papersRoot(rawPkg) : null;",
-            '  const hookRoot = rawPkg ? (JSON.parse(rawPkg)?.["research-paper-pipeline"]?.papersDir ?? "papers") : null;',
+            '  const hookRoot = papersRoot(raw ?? "{}");',
+            '  const hookRoot = raw === null ? "papers" : (() => { try { return JSON.parse(raw)?.papersDir ?? "papers"; } catch { return "papers"; } })();',
           ],
         ],
       },
@@ -115,18 +115,18 @@ process.exit(
         ],
       },
       {
-        name: "a missing declaration is reported as all-clear",
+        name: "an unparsable paperlint.json is reported as all-clear",
         harness: HARNESS,
-        expect: "the missing declaration is NAMED, not skipped",
+        expect: "an unparsable paperlint.json — a FAILURE",
         disables:
-          "the only trace of issue #33 left on an install that STILL works: `paperlint init` writes " +
-          "one file, the hook reads another, and the directories matching up rests on the " +
-          "default. A cheerful checkmark instead of a warning turns a coincidence into a confirmation",
+          "the one failure left in the settings file: while it does not parse, the edit guard " +
+          "refuses every Bash command. A doctor that counts it as fine sends the reader looking " +
+          "everywhere but the file",
         edits: [
           [
             SRC,
-            '      `  ⚠ package.json has no "${CONFIG_KEY}": { "${PAPERS_DIR_FIELD}": … } — the hooks fall back to "${DEFAULT_PAPERS_ROOT}"`,',
-            "      `  ✓ package.json`,",
+            "        `      the edit guard refuses every Bash command until it parses — fix it with Edit or Write`,\n      ],\n      bad: 1,",
+            "        `      the edit guard refuses every Bash command until it parses — fix it with Edit or Write`,\n      ],\n      bad: 0,",
           ],
         ],
       },

package/src/doctor.ts

@@ -29,19 +29,11 @@ import { spawnSync } from "node:child_process";
 // the hook itself rather than at a shared module.
 import {
   papersRoot,
-  CONFIG_KEY,
   DEFAULT_PAPERS_ROOT,
   PAPERS_DIR_FIELD,
-  OLD_PAPERS_DIR_FIELD,
+  CONFIG_FILE,
 } from "../hooks/paper-edit-guard.hook.mjs";
-import {
-  LEGACY_CONFIG_KEY,
-  LEGACY_KEY_MESSAGE,
-  LEGACY_PAPER_SETTINGS_FILE,
-  PAPER_SETTINGS_FILE,
-  declaredSettings,
-} from "../lib/paper-config.mjs";
-import { PAPER_MARKERS, papersIn } from "./build.ts";
+import { PAPER_MARKERS } from "./build.ts";
 import { linkSkills, SKILLS_HOME, type LinkReport } from "./link-skills.ts";
 import { doctorHooks } from "./hooks-settings.ts";
 
@@ -148,51 +140,44 @@ export function detectPapers(cwd: string, depth = 2): string[] {
 }
 
 /**
- * The declaration's verdict. The old field name and two differing keys are failures (every reader
- * refuses them); the old KEY is read and named; a missing declaration is a warning.
+ * The root `paperlint.json`'s verdict. Absent, or silent on `papersDir`, is fine: the default is
+ * `papers`, and the CLI and the hooks take it alike. Only a file that does not parse is a failure —
+ * the edit guard refuses every Bash command while it cannot read it.
  */
-function declarationVerdict(rawPkg: string): { lines: string[]; bad: number } {
-  const out: string[] = [];
-  const found = (() => {
-    try {
-      return declaredSettings(JSON.parse(rawPkg));
-    } catch {
-      return declaredSettings(undefined);
-    }
-  })();
-  if (found.conflict !== null)
-    return { lines: [`  ✗ ${found.conflict}`], bad: 1 };
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- #49: replace with a real type
-  const settings = found.settings as Record<string, any> | undefined;
-  const key = found.legacy ? LEGACY_CONFIG_KEY : CONFIG_KEY;
-  const declared = settings?.[PAPERS_DIR_FIELD];
-  if (found.legacy) out.push(`  ⚠ ${LEGACY_KEY_MESSAGE}`);
-  // The old field name is a failure, not a warning: every reader refuses it.
-  if (settings && Object.hasOwn(settings, OLD_PAPERS_DIR_FIELD))
+function declarationVerdict(raw: string | null): {
+  lines: string[];
+  bad: number;
+} {
+  if (raw === null)
+    return {
+      lines: [
+        `  ✓ no ${CONFIG_FILE} — every setting has its default; papers are in "${DEFAULT_PAPERS_ROOT}"`,
+      ],
+      bad: 0,
+    };
+  let settings: unknown;
+  try {
+    settings = JSON.parse(raw);
+  } catch (e) {
     return {
       lines: [
-        ...out,
-        `  ✗ "${OLD_PAPERS_DIR_FIELD}" was renamed to "${PAPERS_DIR_FIELD}" in package.json → "${key}"`,
+        `  ✗ ${CONFIG_FILE} is not valid JSON: ${(e as Error).message}`,
+        `      the edit guard refuses every Bash command until it parses — fix it with Edit or Write`,
       ],
       bad: 1,
     };
-  // 🔴 A MISSING DECLARATION IS A WARNING, NOT A REFUSAL, and that is a decision, not an
-  // oversight. Without it the hook takes the `papers` default; if the papers do live there, the
-  // install WORKS — just by coincidence, and it will break silently on the day the directory
-  // moves. Failing on a working install is not allowed here: for an `error`-level check a false
-  // positive costs more than a miss, because people do not fix it, they switch it off — together
-  // with the binary findings below, which the command was written for. A real breakage (the roots
-  // drifted apart, the directory does not exist) is caught where it is binary.
-  if (declared === undefined)
-    out.push(
-      `  ⚠ package.json has no "${CONFIG_KEY}": { "${PAPERS_DIR_FIELD}": … } — the hooks fall back to "${DEFAULT_PAPERS_ROOT}"`,
-      `      it works only while your papers happen to live there; declare it and it keeps working`,
-    );
-  else
-    out.push(
-      `  ✓ package.json → ${key}.${PAPERS_DIR_FIELD} = ${JSON.stringify(declared)}`,
-    );
-  return { lines: out, bad: 0 };
+  }
+  const declared = (settings as Record<string, unknown> | null)?.[
+    PAPERS_DIR_FIELD
+  ];
+  return {
+    lines: [
+      declared === undefined
+        ? `  ✓ ${CONFIG_FILE} names no ${PAPERS_DIR_FIELD} — the default "${DEFAULT_PAPERS_ROOT}"`
+        : `  ✓ ${CONFIG_FILE} → ${PAPERS_DIR_FIELD} = ${JSON.stringify(declared)}`,
+    ],
+    bad: 0,
+  };
 }
 
 /**
@@ -235,28 +220,6 @@ function papersVerdict(
   return { lines: out, bad };
 }
 
-/**
- * A pre-2.1.0 `venue.json` left in a paper: it is no longer read, so that paper's venue checks
- * silently would not run if nothing named it. `init` moves it.
- */
-function leftoverSettingsVerdict(
-  root: string,
-  papersDir: string | null,
-): { lines: string[]; bad: number } {
-  if (papersDir === null) return { lines: [], bad: 0 };
-  const lines = papersIn(resolve(root, papersDir), [
-    ...PAPER_MARKERS,
-    LEGACY_PAPER_SETTINGS_FILE,
-  ])
-    .map((dir) => join(dir, LEGACY_PAPER_SETTINGS_FILE))
-    .filter((f) => existsSync(f))
-    .map(
-      (f) =>
-        `  ✗ ${relative(root, f)} is no longer read — paperlint 2.1.0 renamed it ${PAPER_SETTINGS_FILE}, so this paper's venue checks do not run. \`npx paperlint init\` moves it`,
-    );
-  return { lines, bad: lines.length };
-}
-
 export interface DoctorOptions {
   log?: typeof console.log;
   cwd?: string;
@@ -285,7 +248,7 @@ export function doctor({
   skillLinks = (r: string) => linkSkills(r, { write: false }),
 }: DoctorOptions = {}): number {
   const root = projectDir ?? cwd;
-  const pkgPath = join(root, "package.json");
+  const configPath = join(root, CONFIG_FILE);
   const out: string[] = [
     "",
     "paperlint doctor — what is wired, and what only looks wired",
@@ -293,22 +256,15 @@ export function doctor({
   ];
   let bad = 0;
 
-  out.push("declaration");
-  const rawPkg = existsSync(pkgPath) ? readFileSync(pkgPath, "utf8") : "";
-  if (!rawPkg) {
-    out.push(`  ✗ no package.json at ${root}`);
-    out.push(
-      `      the hooks read their papers directory from there and refuse without it`,
-    );
-    bad++;
-  } else {
-    const declaration = declarationVerdict(rawPkg);
-    out.push(...declaration.lines);
-    bad += declaration.bad;
-  }
+  out.push("settings");
+  const raw = existsSync(configPath) ? readFileSync(configPath, "utf8") : null;
+  const declaration = declarationVerdict(raw);
+  out.push(...declaration.lines);
+  bad += declaration.bad;
 
   out.push("", "papers directory");
-  const hookRoot = rawPkg ? papersRoot(rawPkg) : null;
+  // The hook is given what its provider gives it: the file, or `{}` when there is none.
+  const hookRoot = papersRoot(raw ?? "{}");
   const hookSays = typeof hookRoot === "string" ? hookRoot : null;
   out.push(
     `  the CLI will lint    ${cliPapers ?? "(nothing — no declaration found)"}`,
@@ -317,9 +273,8 @@ export function doctor({
     `  the hooks will guard ${hookSays ?? "(nothing — the guard refuses and says why on first use)"}`,
   );
   const verdict = papersVerdict(root, cliPapers, hookSays);
-  const leftover = leftoverSettingsVerdict(root, cliPapers ?? hookSays);
-  out.push(...verdict.lines, ...leftover.lines);
-  bad += verdict.bad + leftover.bad;
+  out.push(...verdict.lines);
+  bad += verdict.bad;
 
   // A skill that is not linked is ADVISORY, like a missing program: `paperlint lint`, the hooks and CI
   // work without it, and an entry of the same name that `init` refused to replace is the

package/src/facts-file.test.ts

@@ -272,13 +272,6 @@ test("declaredVenue: the label is the preset's, and an extends that resolves now
   assert.throws(() => declaredVenue(typo, PAPER), /shipped presets/);
 });
 
-test("declaredVenue: a venue.json alone is refused with the command that moves it, not read", () => {
-  const files = memoryFiles({
-    [`${PAPER}/venue.json`]: '{"venue":"agenticdev"}',
-  });
-  assert.throws(() => declaredVenue(files, PAPER), /npx paperlint init/);
-});
-
 test("declaredVenue: a paperlint.json with an unknown key is refused, naming the key", () => {
   const files = memoryFiles({
     [`${PAPER}/paperlint.json`]: '{"venu":"agenticdev"}',

package/src/facts-file.ts

@@ -75,8 +75,8 @@ export interface VenueDecl {
 
 /**
  * What a paper's `paperlint.json` declares, or null when it has none. THROWS with a one-line reason
- * when the file does not parse, its `extends` does not resolve, or only the pre-2.1.0 `venue.json`
- * is there: the build reports that as its `facts` failure rather than building against no venue.
+ * when the file does not parse or its `extends` does not resolve: the build reports that as its
+ * `facts` failure rather than building against no venue.
  */
 export function declaredVenue(
   files: Files,

package/src/hooks-settings.harness.mjs

@@ -36,7 +36,6 @@ const {
   shippedWiring,
   wireHooks,
   wiredCounts,
-  pluginEnabledHere,
   doctorHooks,
   readSettings,
   MANAGED_BY,
@@ -83,36 +82,26 @@ try {
   // ── which spelling runs which hook ─────────────────────────────────────────────────────
   const ours = `node "\${CLAUDE_PROJECT_DIR}/${MANAGED_BY}" hook paper-edit-guard`;
   const cases = [
-    [ours, { name: "paper-edit-guard", ours: true, legacy: false }],
+    [ours, { name: "paper-edit-guard", ours: true }],
     [
       `node "$CLAUDE_PROJECT_DIR/${MANAGED_BY}" hook paper-status-gates`,
-      { name: "paper-status-gates", ours: true, legacy: false },
-    ],
-    // Guards: what an older install wrote is recognised as ours-but-stale. Spelled literally, not
-    // built from a constant: these are what 1.x and 2.0.0 actually put in users' settings files.
-    [
-      `node "$CLAUDE_PROJECT_DIR/node_modules/research-paper-pipeline/bin/rpp.mjs" hook paper-edit-guard`,
-      { name: "paper-edit-guard", ours: false, legacy: true },
-    ],
-    [
-      `node "\${CLAUDE_PROJECT_DIR}/node_modules/paperlint/bin/rpp.mjs" hook paper-status-gates`,
-      { name: "paper-status-gates", ours: false, legacy: true },
+      { name: "paper-status-gates", ours: true },
     ],
     [
       `node "$CLAUDE_PROJECT_DIR/node_modules/vigiles/dist/cli.js" hook-runtime run-program "$CLAUDE_PROJECT_DIR/node_modules/paperlint/hooks/paper-edit-guard.hook.mjs"`,
-      { name: "paper-edit-guard", ours: false, legacy: false },
+      { name: "paper-edit-guard", ours: false },
     ],
     [
       `npx paperlint hook paper-skills-nudge`,
-      { name: "paper-skills-nudge", ours: false, legacy: false },
+      { name: "paper-skills-nudge", ours: false },
     ],
     [
       `npx paperlint hook paper-skills-nudge`,
-      { name: "paper-skills-nudge", ours: false, legacy: false },
+      { name: "paper-skills-nudge", ours: false },
     ],
     [
       `node /abs/proj/node_modules/paperlint/bin/paperlint.mjs hook paper-edit-guard`,
-      { name: "paper-edit-guard", ours: false, legacy: false },
+      { name: "paper-edit-guard", ours: false },
     ],
     [`node my-own-lint.mjs`, null],
     [`node node_modules/paperlint/bin/paperlint.mjs lint`, null],
@@ -158,46 +147,6 @@ try {
     );
   }
 
-  // ── what an older install wrote is migrated, the user's own command kept ─────────────────
-  // Two real histories, spelled literally: 1.x wrote the old package directory, 2.0.0 wrote the
-  // new directory with the old entry file name. Neither file exists after an upgrade.
-  for (const [release, stale] of [
-    ["1.x", "node_modules/research-paper-pipeline/bin/rpp.mjs"],
-    ["2.0.0", "node_modules/paperlint/bin/rpp.mjs"],
-  ]) {
-    const legacyWired = JSON.parse(
-      JSON.stringify(merge({}, wiring.compiled, MANAGED_BY)).replaceAll(
-        MANAGED_BY,
-        stale,
-      ),
-    );
-    legacyWired.hooks.PostToolUse[0].hooks.push({
-      type: "command",
-      command: "node my-own-lint.mjs",
-    });
-    const dir = project(`legacy-${release}`, legacyWired);
-    const before = doctorHooks(dir, wiring).join("\n");
-    check(
-      `🔴 doctor names hook commands ${release} left behind — the missing file, and the fix`,
-      before.includes(`run ${stale}, which this version does not install`) &&
-        /npx paperlint init` replaces them/.test(before),
-    );
-    const r = wireHooks(dir, merge, wiring);
-    const s = JSON.parse(text(dir));
-    const counts = [...wiredCounts(s, wiring.names).values()];
-    check(
-      `🔴 init replaces what ${release} wrote: every hook wired once, none left at the old path`,
-      r.status === "written" &&
-        r.replaced === wiring.names.length &&
-        counts.every((c) => c.ours === 1 && c.legacy === 0 && c.other === 0) &&
-        !text(dir).includes(stale),
-    );
-    check(
-      `and the user's own command in the same matcher survives the ${release} migration`,
-      text(dir).includes("node my-own-lint.mjs"),
-    );
-  }
-
   // ── the user's own hooks and keys survive ──────────────────────────────────────────────
   {
     const dir = project("user", {
@@ -250,7 +199,7 @@ try {
               {
                 type: "command",
                 command:
-                  'node "$CLAUDE_PROJECT_DIR/node_modules/vigiles/dist/cli.js" hook-runtime run-program "$CLAUDE_PROJECT_DIR/node_modules/research-paper-pipeline/hooks/paper-edit-guard.hook.mjs"',
+                  'node "$CLAUDE_PROJECT_DIR/node_modules/vigiles/dist/cli.js" hook-runtime run-program "$CLAUDE_PROJECT_DIR/node_modules/paperlint/hooks/paper-edit-guard.hook.mjs"',
               },
             ],
           },
@@ -296,25 +245,6 @@ try {
     );
   }
 
-  // ── the plugin, enabled by the project ─────────────────────────────────────────────────
-  check(
-    "the project's enabledPlugins entry for this package is found",
-    pluginEnabledHere({
-      enabledPlugins: {
-        "research-paper-pipeline@research-paper-pipeline": true,
-        "other@x": true,
-      },
-    }).join() === "research-paper-pipeline@research-paper-pipeline",
-  );
-  check(
-    "a disabled entry does not count",
-    pluginEnabledHere({
-      enabledPlugins: {
-        "research-paper-pipeline@research-paper-pipeline": false,
-      },
-    }).length === 0,
-  );
-
   // ── doctor's section ───────────────────────────────────────────────────────────────────
   const doc = (settings) => {
     const dir = project(`doc-${String(n)}`, settings);
@@ -361,21 +291,6 @@ try {
       handPartial,
     ) && /writes nothing then/.test(handPartial),
   );
-  const pluginText = doc({
-    ...wiredOnce,
-    enabledPlugins: { "research-paper-pipeline@research-paper-pipeline": true },
-  });
-  check(
-    "the project also enabling the plugin → told to uninstall it, with the command",
-    /ALSO enables the plugin/.test(pluginText) &&
-      /\/plugin uninstall research-paper-pipeline@research-paper-pipeline/.test(
-        pluginText,
-      ),
-  );
-  check(
-    "🔴 and it says it cannot see a USER-scope plugin, rather than implying there is none",
-    /USER scope is not visible from here/.test(doc({})),
-  );
   check(
     "an unparsable file is named in doctor, not crashed on",
     /does not parse/.test(

package/src/hooks-settings.mutations.mjs

@@ -24,12 +24,12 @@ process.exit(
         harness: HARNESS,
         // The spelling truth table is the first owner: `npx paperlint hook …` must read as NOT ours,
         // and that row is where it goes red.
-        expect: '→ {"name":"paper-edit-guard","ours":false,"legacy":true}',
+        expect: '→ {"name":"paper-skills-nudge","ours":false}',
         disables:
           "the duplicate guard for every bin spelling but ours. A hook wired by hand as " +
           "`npx paperlint hook …` or through an absolute path to the bin reads as ours, init merges its own " +
           "copy beside it, and the hook runs twice per event",
-        edits: [[SRC, "      ours: t === MANAGED_BY,", "      ours: true,"]],
+        edits: [[SRC, "ours: t === MANAGED_BY,", "ours: true,"]],
       },
       {
         name: "another spelling no longer stops the write",
@@ -94,21 +94,6 @@ process.exit(
           "same thing again — a loop with no exit",
         edits: [[SRC, "  const remedy = handWired", "  const remedy = false"]],
       },
-      {
-        name: "a disabled plugin counts as enabled",
-        harness: HARNESS,
-        expect: "a disabled entry does not count",
-        disables:
-          "the plugin warning's precision. A project that already switched the plugin off would " +
-          "be told to uninstall it",
-        edits: [
-          [
-            SRC,
-            '      ([id, on]) => on === true && id.split("@")[0] === LEGACY_PACKAGE_NAME,',
-            '      ([id]) => id.split("@")[0] === LEGACY_PACKAGE_NAME,',
-          ],
-        ],
-      },
     ],
   }),
 );