@forwardimpact/libwiki 0.2.19 → 0.2.21

package/package.json

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

package/src/audit/rules.js

@@ -254,7 +254,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 +263,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",

package/src/commands/fix.js

@@ -48,9 +48,10 @@ function invariantContract(findings) {
 }
 
 /**
- * The opening task: the findings, the invariant contract, and the two things
- * the rule hints don't cover — where trimmed history goes, and to prefer a
- * single Write.
+ * The opening task: the findings, the invariant contract, and the things the
+ * rule hints don't cover — where trimmed history goes (only existing
+ * weekly-log files; rotation owns minting new ones), and to prefer a single
+ * Write.
  */
 function composeTask(findings, wikiRoot, projectRoot) {
   return [
@@ -62,9 +63,13 @@ function composeTask(findings, wikiRoot, projectRoot) {
     `by breaking another:`,
     ...invariantContract(findings),
     ``,
-    `Move history out of an over-budget summary into the agent's weekly-log`,
-    `file (wiki/<agent>-YYYY-Www.md), never a new summary section. Prefer a`,
-    `single Write over many Edits.`,
+    `Move history out of an over-budget summary into the agent's existing`,
+    `weekly-log file or its current part (wiki/<agent>-YYYY-Www[-partN].md) —`,
+    `never a new summary section, and never a new file: rotation tooling owns`,
+    `when part files are created, so do not mint filenames yourself. If the`,
+    `trimmed narrative already exists in the weekly log, replace it in the`,
+    `summary with a pointer to that file instead of copying it anywhere.`,
+    `Prefer a single Write over many Edits.`,
   ].join("\n");
 }
 

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 };