@forwardimpact/libwiki 0.2.20 → 0.2.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@forwardimpact/libwiki",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "description": "Wiki lifecycle primitives — stable memory for agent teams so coordination persists across sessions.",
   "keywords": [
     "wiki",

package/src/audit/rules.js

@@ -75,16 +75,22 @@ const columnCount = (expected) => (s) =>
 const exists = (s) => (s.exists ? null : {});
 const expired = (s, ctx) => (s.expires_at < ctx.today ? {} : null);
 
+// The heading must equal `requiredLine` exactly — a suffixed variant like
+// "### Decision — <summary>" does not satisfy it, but is reported as a
+// near miss so the writer fixes the heading instead of hunting for a
+// "missing" line that is right there.
 function entryHasDecision(lines, startIdx, requiredLine, stopRe) {
   let seen = 0;
+  let nearMiss = null;
   for (let j = startIdx + 1; j < lines.length && seen < 5; j++) {
     const ln = lines[j].trim();
     if (ln === "") continue;
     seen++;
-    if (ln === requiredLine) return true;
-    if (stopRe.test(lines[j])) return false;
+    if (ln === requiredLine) return { found: true };
+    if (nearMiss === null && ln.startsWith(requiredLine)) nearMiss = ln;
+    if (stopRe.test(lines[j])) break;
   }
-  return false;
+  return { found: false, nearMiss };
 }
 
 const decisionWithin5 =
@@ -92,12 +98,9 @@ const decisionWithin5 =
   (s) => {
     const offenders = [];
     for (let i = 0; i < s.fileLines.length; i++) {
-      if (
-        entryRe.test(s.fileLines[i]) &&
-        !entryHasDecision(s.fileLines, i, requiredLine, stopRe)
-      ) {
-        offenders.push({ lineNo: i + 1 });
-      }
+      if (!entryRe.test(s.fileLines[i])) continue;
+      const res = entryHasDecision(s.fileLines, i, requiredLine, stopRe);
+      if (!res.found) offenders.push({ lineNo: i + 1, nearMiss: res.nearMiss });
     }
     return offenders.length === 0 ? null : offenders;
   };
@@ -254,7 +257,7 @@ export const RULES = [
     remediation: "rotate",
     check: lineBudget(WEEKLY_LOG_LINE_BUDGET),
     message: (_s, r) => `${r.value} lines (limit ${WEEKLY_LOG_LINE_BUDGET})`,
-    hint: "run `bunx fit-wiki rotate` to seal this file as a sealed part and start a fresh weekly log",
+    hint: "run `bunx fit-wiki rotate --agent <agent>` (agent = this filename's prefix) to seal this file as a sealed part and start a fresh weekly log",
   },
   {
     id: "weekly-log.word-budget",
@@ -263,7 +266,7 @@ export const RULES = [
     remediation: "rotate",
     check: wordBudget(WEEKLY_LOG_WORD_BUDGET),
     message: (_s, r) => `${r.value} words (limit ${WEEKLY_LOG_WORD_BUDGET})`,
-    hint: "run `bunx fit-wiki rotate` to seal this file as a sealed part and start a fresh weekly log",
+    hint: "run `bunx fit-wiki rotate --agent <agent>` (agent = this filename's prefix) to seal this file as a sealed part and start a fresh weekly log",
   },
   {
     id: "weekly-log.h1-agent-matches-filename",
@@ -283,8 +286,11 @@ export const RULES = [
       requiredLine: DECISION_HEADING,
       stopRe: /^##\s/,
     }),
-    message: () => "Entry lacks leading '### Decision'",
-    hint: "open each '## YYYY-MM-DD' entry with a '### Decision' subheading that summarises the decision recorded in the entry's own narrative — do not invent rationale the entry does not support",
+    message: (_s, r) =>
+      r.nearMiss
+        ? `Entry opens with '${r.nearMiss}'; the heading must be exactly '${DECISION_HEADING}' — move the suffix into the body`
+        : `Entry lacks a line that is exactly '${DECISION_HEADING}'`,
+    hint: `open each '## YYYY-MM-DD' entry with a line containing exactly '${DECISION_HEADING}' (no suffix — the check is an exact match); put the one-line summary in the body below it, drawn from the entry's own narrative — do not invent rationale the entry does not support`,
   },
 
   // -- Weekly logs (sealed parts) --

package/src/commands/rotate.js

@@ -1,4 +1,4 @@
-import { rotateIfOverBudget } from "../weekly-log.js";
+import { rotateIfOverBudget, weeklyLogPath } from "../weekly-log.js";
 import { currentDayIso } from "../util/clock.js";
 import { resolveWikiRoot } from "../util/wiki-dir.js";
 
@@ -16,6 +16,12 @@ export function runRotateCommand(ctx) {
   }
   const wikiRoot = resolveWikiRoot(runtime, options);
   const today = options.today || currentDayIso(runtime);
+  // Name the resolved target before any seal: the file follows from agent +
+  // current week, not from any audit finding, so an env-default agent can
+  // silently select a different file than the one the operator has in mind.
+  runtime.proc.stdout.write(
+    `target → ${weeklyLogPath(wikiRoot, agent, today)}\n`,
+  );
 
   let result;
   try {

package/src/weekly-log.js

@@ -281,6 +281,13 @@ export function rotateIfOverBudget(
   const { force = false } = options;
   if (!fs.existsSync(filePath)) return { status: "noop", fromPath: filePath };
   const text = fs.readFileSync(filePath, "utf-8");
+  // A header-only (or empty) log has nothing to seal. Without this floor,
+  // force-rotating a freshly-reset main would mint an empty `(part 1 of 1)`
+  // file and reset the main again — once per invocation, forever.
+  const nl = text.indexOf("\n");
+  if ((nl === -1 ? "" : text.slice(nl + 1)).trim() === "") {
+    return { status: "noop", fromPath: filePath };
+  }
   const current = countLines(text);
   if (!force && current + appendLines <= WEEKLY_LOG_LINE_BUDGET) {
     return { status: "noop", fromPath: filePath };