planrails 0.1.0 → 0.1.1

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.1 — 2026-09-12
+
+Fixed, found while moving a real project from a vendored copy to the package:
+
+- `npx planrails init` on a project that did not yet hold the package wrote the npx cache's absolute path into every hook command (machine-local, and evicted by npm). Hook commands now name `$CLAUDE_PROJECT_DIR/node_modules/planrails/…` whenever the project holds its own copy, whichever copy runs the installer. `test/hooks-path.test.mjs` reproduces it with a copied package.
+- A CLAUDE.md plans block written by an older copy of the system (a different generator name in its marker) was not recognised, so `init` and `activate` added a second block beside it. Any `<!-- plans:begin` marker now counts as the block. `test/claude-md.test.mjs`.
+- Internal: `npm test` lists the test files by name (`scripts/run-tests.mjs`); Node 20 does not expand the glob the 0.1.0 script used, so its CI jobs failed.
+
 ## 0.1.0 — 2026-09-12
 
 First release, extracted from the project it was built in after a day of use.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "planrails",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "Plans that survive compaction, for Claude Code: gates that prove a task done, conditions answered before it closes, briefs and rules injected by hooks, one fresh session per task.",
   "type": "module",
   "bin": {

package/src/hooks/install.mjs

@@ -15,10 +15,11 @@
  * and every teammate's machine runs the copy in THEIR node_modules. If planrails
  * is not installed under the project's node_modules (a checkout, a global
  * install), the command falls back to the absolute path of this copy and
- * `doctor` says so. Paths are always quoted: an unquoted path with a space runs
+ * `doctor` says so. The project's copy wins whenever it exists, whichever copy
+ * runs the installer (0.1.0 wrote the npx cache path on a first init). Paths are always quoted: an unquoted path with a space runs
  * `node /Users/x/My` and silently disables every hook.
  */
-import { existsSync, readFileSync, writeFileSync, mkdirSync, copyFileSync, realpathSync, unlinkSync } from "node:fs";
+import { existsSync, readFileSync, writeFileSync, mkdirSync, copyFileSync, unlinkSync } from "node:fs";
 import { join, dirname } from "node:path";
 import { spawnSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
@@ -43,11 +44,16 @@ export const OPTIONAL = {
 const ALL_FILES = new Set([...MANAGED, ...Object.values(OPTIONAL)].map((m) => m.file));
 const SKILL_SRC = join(HOOKS_DIR, "..", "plan", "skill", "SKILL.md");
 
-function same(a, b) { try { return realpathSync(a) === realpathSync(b); } catch { return false; } }
-/** Portable when the package sits in the project's node_modules; absolute otherwise. */
+/** Does the PROJECT hold its own copy of this hook under node_modules? That copy is what the hook command names. */
+function installedLocally(file) { return existsSync(join(projectRoot(), "node_modules", "planrails", "src", "hooks", file)); }
+/**
+ * Portable ($CLAUDE_PROJECT_DIR/node_modules/planrails/…) whenever the project holds its own copy, WHICHEVER copy runs
+ * this installer. 0.1.0 compared the running copy with the project's, so `npx planrails init` on a project that did not
+ * yet hold the package (the documented first step) wrote the npx cache's absolute path into every hook — machine-local,
+ * and evicted by npm. Absolute only when the project has no copy of its own.
+ */
 export function commandFor(m) {
-  const viaNodeModules = same(packageRoot(), join(projectRoot(), "node_modules", "planrails"));
-  const path = viaNodeModules ? `$CLAUDE_PROJECT_DIR/node_modules/planrails/src/hooks/${m.file}` : join(HOOKS_DIR, m.file);
+  const path = installedLocally(m.file) ? `$CLAUDE_PROJECT_DIR/node_modules/planrails/src/hooks/${m.file}` : join(HOOKS_DIR, m.file);
   return `${m.runner} "${path}"`;
 }
 function entryFor(m) {
@@ -122,7 +128,7 @@ export function install({ dryRun = false, withNeverDelete = false, remove = fals
     }
   } else if (!remove) lines.push(`(dry run) would install /plan skill → ${skillDest()}`);
   if (wanted.some((m) => m.file === "guard-never-delete.sh") && spawnSync("which", ["jq"]).status !== 0) lines.push("warn: jq not found — the never-delete guard needs it (brew install jq / apt install jq)");
-  if (!remove && !same(packageRoot(), join(projectRoot(), "node_modules", "planrails"))) lines.push(`note: planrails is not installed under ${join(projectRoot(), "node_modules")} — hook commands use the absolute path of this copy (${packageRoot()}), which only works on this machine. Run: npm install --save-dev planrails, then ${cli} hooks install`);
+  if (!remove && !installedLocally("plan-session-start.mjs")) lines.push(`note: planrails is not installed under ${join(projectRoot(), "node_modules")} — hook commands use the absolute path of this copy (${packageRoot()}), which only works on this machine. Run: npm install --save-dev planrails, then ${cli} hooks install`);
   lines.push(remove ? "Restart Claude Code (or run /hooks) so the removal takes effect." : "Restart Claude Code (or run /hooks) so the new hooks load.");
   process.stdout.write(lines.join("\n") + "\n");
   return { changed, replaced };

package/src/init.mjs

@@ -57,7 +57,7 @@ export async function init({ dir = null, force = false, noInstall = false, withN
   const installed = existsSync(join(root, "node_modules", "planrails", "package.json"));
   const declared = Boolean(pkg.devDependencies?.planrails || pkg.dependencies?.planrails);
   if (installed && declared) note(false, `planrails is a dependency (${readJson(join(root, "node_modules", "planrails", "package.json"), {}).version || "?"})`);
-  else if (noInstall || dryRun) note(false, `planrails not installed as a dependency (${noInstall ? "--no-install" : "dry run"}); hooks will use this copy's absolute path`);
+  else if (noInstall || dryRun) note(false, `planrails not installed as a dependency (${noInstall ? "--no-install" : "dry run"}); hooks name the project's own copy if it has one, else this copy's absolute path`);
   else {
     log(`installing planrails@${pkgVersion()} as a devDependency…`);
     const r = spawnSync("npm", ["install", "--save-dev", `planrails@${pkgVersion()}`], { cwd: root, stdio: "inherit" });

package/src/plan/lib/claude-md.mjs

@@ -13,7 +13,7 @@
  */
 import { existsSync, readFileSync, writeFileSync } from "node:fs";
 
-export const BEGIN = "<!-- plans:begin — generated by src/plan/plan.mjs; do not hand-edit this block -->";
+export const BEGIN = "<!-- plans:begin — generated by planrails; do not hand-edit this block -->";
 export const END = "<!-- plans:end -->";
 
 export function renderBlock(activePlans) {
@@ -28,13 +28,15 @@ export function renderBlock(activePlans) {
   return lines.join("\n");
 }
 
+/** Any begin marker counts, whatever generator name it carries: an older copy of this system wrote a different one, and a second block must never be added beside it. */
+const BEGIN_PREFIX = "<!-- plans:begin";
 export function readBlock(md) {
-  const a = md.indexOf(BEGIN); const b = md.indexOf(END);
+  const a = md.indexOf(BEGIN_PREFIX); const b = md.indexOf(END);
   if (a === -1 || b === -1 || b < a) return null;
   return md.slice(a, b + END.length);
 }
 /** How many blocks the file holds. More than one is a merge accident; validate fails on it. */
-export function blockCount(md) { return md.split(BEGIN).length - 1; }
+export function blockCount(md) { return md.split(BEGIN_PREFIX).length - 1; }
 
 /** Plan ids currently listed in the block (for the drift check). */
 export function idsInBlock(md) {