planrails 0.1.0 → 0.1.2

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.1.2 — 2026-09-12
+
+- `npx planrails update` now also rewrites the plans block in CLAUDE.md from the plans on disk, so a marker or wording written by an older version is replaced on upgrade. `test/claude-md.test.mjs`.
+- Release checks around `npm publish` (`scripts/release-check.mjs`, wired as `prepublishOnly` and `postpublish`): before uploading, refuse a version the registry already has, a missing CHANGELOG entry, a dirty tree or an unpushed HEAD; after uploading, wait until the registry serves the version. Why: a web-authenticated publish is staged and finalizes about a minute later; in that window a second publish fails with E409 and an install with ETARGET, which is how 0.1.1's release went. CONTRIBUTING.md § Release. `test/release-check.test.mjs`.
+
+## 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.2",
   "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": {
@@ -22,7 +22,9 @@
     "selftest": "node bin/planrails.mjs selftest && node src/hooks/selftest.mjs",
     "acceptance": "node scripts/acceptance.mjs",
     "trial": "node scripts/trial/run.mjs",
-    "prepack": "npm test"
+    "prepack": "npm test",
+    "prepublishOnly": "node scripts/release-check.mjs before",
+    "postpublish": "node scripts/release-check.mjs after"
   },
   "dependencies": {
     "zod": "^4.4.3"

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" });
@@ -130,7 +130,7 @@ export async function init({ dir = null, force = false, noInstall = false, withN
   return problems ? 1 : 0;
 }
 
-/** Refresh the guide, the skill and the hook entries to this version. Plans are never touched. */
+/** Refresh the guide, the skill, the hook entries and the CLAUDE.md plans block to this version. Plans are never touched. */
 export async function update({ dir = null, log = console.log } = {}) {
   if (dir) process.env.PLAN_PROJECT_ROOT = resolve(dir);
   const root = projectRoot();
@@ -138,6 +138,12 @@ export async function update({ dir = null, log = console.log } = {}) {
   const guideDest = join(root, "docs", "PLANNING_GUIDE.md");
   if (!existsSync(guideDest) || readFileSync(guideDest, "utf8") !== readFileSync(guideSrc, "utf8")) { mkdirSync(join(root, "docs"), { recursive: true }); copyFileSync(guideSrc, guideDest); log("refreshed docs/PLANNING_GUIDE.md"); }
   else log("docs/PLANNING_GUIDE.md is current");
+  // The plans block, rewritten from the plans on disk: a marker or wording written by an older version is replaced.
+  try {
+    const { activePlanIds, loadPlanCheap } = await import("./plan/lib/store.mjs");
+    const active = activePlanIds().map((id) => ({ id, title: loadPlanCheap(id).state?.title || id }));
+    if (existsSync(claudeMdPath())) log(writeBlock(claudeMdPath(), active) ? "refreshed the plans block in CLAUDE.md" : "the plans block in CLAUDE.md is current");
+  } catch (e) { log(`could not refresh the plans block in CLAUDE.md: ${e.message}`); }
   const { install, status } = await import("./hooks/install.mjs");
   install({});
   const { problems } = status({ print: false });

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) {