@forwardimpact/libwiki 0.2.6 → 0.2.7

package/package.json

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

package/src/audit/rules.js

@@ -4,6 +4,8 @@ import {
   DECISION_HEADING,
   MEMO_INBOX_MARKER,
   PRIORITY_INDEX_HEADING,
+  STORYBOARD_LINE_BUDGET,
+  STORYBOARD_WORD_BUDGET,
   SUMMARY_LINE_BUDGET,
   SUMMARY_WORD_BUDGET,
   WEEKLY_LOG_LINE_BUDGET,
@@ -23,11 +25,15 @@ const CLAIMS_HEADER_RE =
 const CLAIMS_SEPARATOR_RE =
   /^\|\s*---\s*\|\s*---\s*\|\s*---\s*\|\s*---\s*\|\s*---\s*\|\s*---\s*\|/m;
 const ISO_DATE_RE = /^\d{4}-\d{2}-\d{2}$/;
-const XMR_OPEN_RE = /^<!--\s*xmr:([^:\s]+):([^\s]+)\s*-->\s*$/;
-const XMR_CLOSE_RE = /^<!--\s*\/xmr\s*-->\s*$/;
+// Marker regexes (mirror of marker-scanner.js): tolerate optional trailing
+// text inside the marker so an open marker can carry an inline notice like
+// "Do not edit. Auto-generated." without breaking the audit's balance check.
+const XMR_OPEN_RE = /^<!--\s*xmr:([^:\s]+):(\S+)(?:\s+[^>]*?)?\s*-->\s*$/;
+const XMR_CLOSE_RE = /^<!--\s*\/xmr(?:\s+[^>]*?)?\s*-->\s*$/;
 const ISSUE_OPEN_RE =
-  /^<!--\s*(obstacles|experiments):(open|closed)(?::(\d+d))?\s*-->\s*$/;
-const ISSUE_CLOSE_RE = /^<!--\s*\/(obstacles|experiments)\s*-->\s*$/;
+  /^<!--\s*(obstacles|experiments):(open|closed)(?::(\d+d))?(?:\s+[^>]*?)?\s*-->\s*$/;
+const ISSUE_CLOSE_RE =
+  /^<!--\s*\/(obstacles|experiments)(?:\s+[^>]*?)?\s*-->\s*$/;
 
 // improvement-coach is the storyboard facilitator and carries no domain
 // metrics; only the five domain agents need their own H3.
@@ -419,6 +425,24 @@ export const RULES = [
     check: allRequiredLines(AGENT_H3_REQUIREMENTS),
     message: (s, r) => `storyboard: ${s.path} missing '### ${r.label}' H3`,
   },
+  {
+    id: "storyboard.line-budget",
+    scope: "storyboard",
+    severity: "fail",
+    when: storyboardExists,
+    check: lineBudget(STORYBOARD_LINE_BUDGET),
+    message: (s, r) =>
+      `storyboard: ${s.path} has ${r.value} lines (limit ${STORYBOARD_LINE_BUDGET})`,
+  },
+  {
+    id: "storyboard.word-budget",
+    scope: "storyboard",
+    severity: "fail",
+    when: storyboardExists,
+    check: wordBudget(STORYBOARD_WORD_BUDGET),
+    message: (s, r) =>
+      `storyboard: ${s.path} has ${r.value} words (limit ${STORYBOARD_WORD_BUDGET})`,
+  },
   {
     id: "storyboard.markers-balanced.xmr",
     scope: "storyboard",

package/src/audit/scopes.js

@@ -107,6 +107,8 @@ function loadStoryboard(wikiRoot, today) {
     fileLines: text.split("\n"),
     exists,
     yearMonth: `${yyyy}-M${mm}`,
+    lines: countLines(text),
+    words: countWords(text),
   };
 }
 

package/src/block-renderer.js

@@ -32,11 +32,8 @@ export function renderBlock({
     throw new BlockRenderError(`metric-not-found: ${metric}`);
   }
 
-  const latestValue = m.latest?.value ?? m.values[m.values.length - 1] ?? "—";
-  const status = m.status;
-
   let chartLines;
-  if (status === "insufficient_data") {
+  if (m.status === "insufficient_data") {
     chartLines = [
       `Insufficient data: ${m.n} points (need at least ${MIN_POINTS}).`,
     ];
@@ -45,16 +42,12 @@ export function renderBlock({
     chartLines = chartText.split("\n");
   }
 
-  const signalLine = formatSignals(m.signals);
-
   return [
-    `**Latest:** ${latestValue} · **Status:** ${status}`,
-    "",
     "```",
     ...chartLines,
     "```",
     "",
-    `**Signals:** ${signalLine}`,
+    `**Signals:** ${formatSignals(m.signals)}`,
   ];
 }
 

package/src/commands/refresh.js

@@ -1,10 +1,12 @@
 import { readFileSync, writeFileSync } from "node:fs";
+import { spawnSync } from "node:child_process";
 import path from "node:path";
 import fsAsync from "node:fs/promises";
 import { Finder } from "@forwardimpact/libutil";
+import { createScriptConfig } from "@forwardimpact/libconfig";
 import { scanMarkers } from "../marker-scanner.js";
 import { renderBlock, BlockRenderError } from "../block-renderer.js";
-import { renderIssueList } from "../issue-list-renderer.js";
+import { renderIssueList, parseRepoSlug } from "../issue-list-renderer.js";
 
 function currentStoryboardPath() {
   const now = new Date();
@@ -13,7 +15,17 @@ function currentStoryboardPath() {
   return `wiki/storyboard-${yyyy}-M${mm}.md`;
 }
 
-function renderForBlock(block, projectRoot) {
+function deriveParentRepo(parentDir) {
+  if (process.env.FIT_GH_REPO) return process.env.FIT_GH_REPO;
+  const r = spawnSync("git", ["-C", parentDir, "remote", "get-url", "origin"], {
+    encoding: "utf-8",
+    stdio: "pipe",
+  });
+  if (r.status !== 0) return null;
+  return parseRepoSlug(r.stdout);
+}
+
+function renderForBlock(block, projectRoot, ghContext) {
   if (block.kind === "xmr") {
     return renderBlock({
       metric: block.metric,
@@ -26,6 +38,9 @@ function renderForBlock(block, projectRoot) {
       topic: block.topic,
       state: block.state,
       window: block.window,
+      cwd: ghContext.cwd,
+      repo: ghContext.repo,
+      token: ghContext.token,
     });
   }
   return null;
@@ -40,7 +55,7 @@ function spliceBlock(lines, block, rendered) {
 }
 
 /** Re-render XmR chart blocks and issue-list blocks in a storyboard file. */
-export function runRefreshCommand(values, args, _cli) {
+export async function runRefreshCommand(values, args, _cli) {
   const logger = { debug() {} };
   const finder = new Finder(fsAsync, logger, process);
   const projectRoot = finder.findProjectRoot(process.cwd());
@@ -53,13 +68,33 @@ export function runRefreshCommand(values, args, _cli) {
   const blocks = scanMarkers(text);
   if (blocks.length === 0) return;
 
+  const config = await createScriptConfig("wiki");
+  let token = null;
+  try {
+    token = config.ghToken();
+  } catch {
+    // Missing token is non-fatal; issue-list renders will fail with a stderr
+    // warning and the block will collapse to the notice line.
+  }
+  // Spawn `gh` from the project root so it resolves the monorepo's origin
+  // instead of whatever git context the caller's cwd happens to be in (the
+  // wiki sibling repo, a subagent worktree, a service dir, etc.). Also
+  // resolve an explicit owner/repo slug so `gh` works when origin has been
+  // rewritten to a proxy URL (sandbox environments) — `FIT_GH_REPO` env
+  // overrides the parsed origin.
+  const ghContext = {
+    cwd: projectRoot,
+    repo: deriveParentRepo(projectRoot),
+    token,
+  };
+
   const lines = text.split("\n");
   let spliced = false;
 
   for (let i = blocks.length - 1; i >= 0; i--) {
     const block = blocks[i];
     try {
-      const rendered = renderForBlock(block, projectRoot);
+      const rendered = renderForBlock(block, projectRoot, ghContext);
       if (!rendered) continue;
       spliceBlock(lines, block, rendered);
       spliced = true;

package/src/constants.js

@@ -13,10 +13,14 @@ export const PRIORITY_INDEX_TABLE_HEADER =
   "| Item | Agents | Owner | Status | Added |";
 export const DECISION_HEADING = "### Decision";
 
-// Cap derivation: ≤2.5% of a 1M-token context window = 25k tokens;
-// converted via ≈42 tokens/line empirical proxy, then 64-aligned.
-// See spec 1060 design-a.md § Decision area 2 for the full anchor.
+// Unified budgets for the three audited surfaces (summary, weekly-log,
+// storyboard). They share the same numeric limits today so the
+// context-tax floor is symmetric across surfaces; each surface keeps
+// its own audit rule pair so the limits can diverge later if the
+// context-tax model says one surface should be looser or tighter.
+export const SUMMARY_LINE_BUDGET = 496;
+export const SUMMARY_WORD_BUDGET = 2048;
 export const WEEKLY_LOG_LINE_BUDGET = 496;
-export const SUMMARY_LINE_BUDGET = 72;
 export const WEEKLY_LOG_WORD_BUDGET = 6400;
-export const SUMMARY_WORD_BUDGET = 12800;
+export const STORYBOARD_LINE_BUDGET = 496;
+export const STORYBOARD_WORD_BUDGET = 6400;

package/src/issue-list-renderer.js

@@ -1,9 +1,14 @@
 import { spawnSync } from "node:child_process";
 
-function defaultGh(args) {
+function defaultGh(args, options) {
+  const env = options?.token
+    ? { ...process.env, GH_TOKEN: options.token }
+    : undefined;
   return spawnSync("gh", args, {
     encoding: "utf-8",
     stdio: ["ignore", "pipe", "pipe"],
+    cwd: options?.cwd,
+    env,
   });
 }
 
@@ -13,18 +18,30 @@ function daysAgo(today, n) {
   return d.toISOString().slice(0, 10);
 }
 
-/** Render an issue-list block for an obstacles/experiments marker. Returns markdown lines. */
+/** Parse `owner/repo` from a git origin URL. Tolerates http(s), ssh, and proxy-rewritten URLs (e.g. `http://host/git/owner/repo`) by taking the last two path segments after stripping `.git`. Returns null when nothing parseable is found. */
+export function parseRepoSlug(originUrl) {
+  if (!originUrl) return null;
+  const stripped = originUrl.trim().replace(/\.git$/, "");
+  const match = stripped.match(/([^/:]+)\/([^/:]+)$/);
+  if (!match) return null;
+  return `${match[1]}/${match[2]}`;
+}
+
+/** Render an issue-list block for an obstacles/experiments marker. Returns markdown lines. `cwd` should be the parent monorepo's project root so `gh` resolves the correct origin; `repo` is an explicit `owner/name` slug used when the origin remote is unparseable by `gh` (e.g. sandbox proxy URLs); `token` is the resolved GH token (e.g. via `Config.ghToken()`). */
 export function renderIssueList({
   topic,
   state,
   window,
+  cwd,
+  repo,
+  token,
   today = new Date(),
   gh = defaultGh,
 }) {
   const ghState = state === "closed" ? "closed" : "open";
-  const result = gh([
-    "issue",
-    "list",
+  const args = ["issue", "list"];
+  if (repo) args.push("--repo", repo);
+  args.push(
     "--label",
     topic.replace(/s$/, ""),
     "--state",
@@ -33,7 +50,8 @@ export function renderIssueList({
     "number,title,labels,closedAt",
     "--limit",
     "100",
-  ]);
+  );
+  const result = gh(args, { cwd, token });
   if (result.status !== 0) {
     process.stderr.write(
       `refresh: gh issue list failed for ${topic}:${state}\n`,
@@ -62,8 +80,7 @@ export function renderIssueList({
 
   const lines = [];
   for (const issue of issues) {
-    const tag = topic === "experiments" ? "Exp" : "Obs";
-    lines.push(`- **${tag} #${issue.number} — ${issue.title}**`);
+    lines.push(`- #${issue.number} ${issue.title}`);
   }
   return lines;
 }

package/src/marker-scanner.js

@@ -1,8 +1,12 @@
-const XMR_OPEN_RE = /^<!--\s*xmr:([^:\s]+):([^\s]+)\s*-->\s*$/;
+// Markers tolerate optional trailing text after the tag (typically an inline
+// "Do not edit. Generated from fit-wiki refresh." notice), so an open or close
+// marker can carry its own warning without needing a separate notice line.
+const XMR_OPEN_RE = /^<!--\s*xmr:([^:\s]+):(\S+)(?:\s+[^>]*?)?\s*-->\s*$/;
 const ISSUE_OPEN_RE =
-  /^<!--\s*(obstacles|experiments):(open|closed)(?::(\d+d))?\s*-->\s*$/;
-const XMR_CLOSE_RE = /^<!--\s*\/xmr\s*-->\s*$/;
-const ISSUE_CLOSE_RE = /^<!--\s*\/(obstacles|experiments)\s*-->\s*$/;
+  /^<!--\s*(obstacles|experiments):(open|closed)(?::(\d+d))?(?:\s+[^>]*?)?\s*-->\s*$/;
+const XMR_CLOSE_RE = /^<!--\s*\/xmr(?:\s+[^>]*?)?\s*-->\s*$/;
+const ISSUE_CLOSE_RE =
+  /^<!--\s*\/(obstacles|experiments)(?:\s+[^>]*?)?\s*-->\s*$/;
 
 function openLabel(open) {
   return open.kind === "xmr" ? open.metric : open.topic;