pan-wizard 3.27.0 → 3.29.0

package/bin/install.js

@@ -46,7 +46,7 @@ const pkg = require('../package.json');
 // phrased by capability, not by name; these are the "switch to this" examples
 // that keep the advice actionable. Nothing in PAN gates on these values.
 const RECOMMENDED_MODELS = {
-  flagship: 'claude-fable-5',
+  flagship: 'claude-fable-5-1',
   reasoningTier: 'claude-opus-5 / claude-opus-4-8',
 };
 
@@ -565,21 +565,15 @@ function copyCommandsAsUnifiedSkills(srcDir, skillsDir, prefix, pathPrefix, core
       fs.mkdirSync(skillDir, { recursive: true });
 
       let content = fs.readFileSync(srcPath, 'utf8');
-      // Core + agent-definition references → shared .agents/ copies (specific,
-      // before the generic rewrites); everything else .claude-scoped → the
-      // installing runtime. Agent refs point at the canonical reference copies
-      // shipped with the shared core — the runtime's own agents dir may carry
-      // a different format (Codex TOML, Copilot .agent.md).
-      content = content.replace(/~\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-      content = content.replace(/\.\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-      content = content.replace(/~\/\.claude\/agents\//g, `${corePrefix}pan-wizard-core/agents/`);
-      content = content.replace(/\.\/\.claude\/agents\//g, `${corePrefix}pan-wizard-core/agents/`);
-      content = content.replace(/~\/\.claude\//g, pathPrefix);
-      content = content.replace(/\.\/\.claude\//g, `./${getDirName(runtime)}/`);
-      // Not every runtime puts a `pan-tools` bin on PATH — invoke via node.
-      const panToolsPath = `${corePrefix}pan-wizard-core/bin/pan-tools.cjs`;
-      content = content.replace(/\bpan-tools\b(?=\s+[a-z])/g, `node ${panToolsPath}`);
-      content = processAttribution(content, getCommitAttribution(runtime));
+      // The path rewrite lives in install-lib (rewriteUnifiedSkillCommandContent)
+      // because the Agent Plugins bundle builder runs the SAME function — one
+      // converter, several call sites, never a second copy (ADR-0028, ADR-0045).
+      content = lib.rewriteUnifiedSkillCommandContent(content, {
+        corePrefix,
+        pathPrefix,
+        projectDirPrefix: `./${getDirName(runtime)}/`,
+        attribution: getCommitAttribution(runtime),
+      });
       content = convertClaudeCommandToUnifiedSkill(content, skillName);
 
       fs.writeFileSync(path.join(skillDir, 'SKILL.md'), content);
@@ -670,17 +664,14 @@ function copySharedCore(srcDir, destDir, corePrefix, runtimePathPrefix, runtime)
         recurse(srcPath, destPath);
       } else if (entry.name.endsWith('.md')) {
         try {
-          let content = fs.readFileSync(srcPath, 'utf8');
-          content = content.replace(/~\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-          content = content.replace(/\.\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-          // Agent-definition refs → the canonical reference copies in the
-          // shared core (runtime agents dirs carry runtime-specific formats).
-          content = content.replace(/~\/\.claude\/agents\//g, `${corePrefix}pan-wizard-core/agents/`);
-          content = content.replace(/\.\/\.claude\/agents\//g, `${corePrefix}pan-wizard-core/agents/`);
-          content = content.replace(/~\/\.claude\//g, runtimePathPrefix);
-          content = content.replace(/\.\/\.claude\//g, `./${dirName}/`);
-          content = processAttribution(content, getCommitAttribution(runtime));
-          content = convertSlashCommandsToCopilotSkillMentions(content);
+          // Shared with the Agent Plugins bundle builder (install-lib) — see
+          // rewriteSharedCoreMarkdown for the rewrite order and rationale.
+          const content = lib.rewriteSharedCoreMarkdown(fs.readFileSync(srcPath, 'utf8'), {
+            corePrefix,
+            pathPrefix: runtimePathPrefix,
+            projectDirPrefix: `./${dirName}/`,
+            attribution: getCommitAttribution(runtime),
+          });
           fs.writeFileSync(destPath, content);
         } catch (err) {
           pushInstallWarning('copySharedCore(md)', destPath, err);
@@ -728,21 +719,13 @@ function stripInternalFromLearningsIndex(indexPath) {
     if (err.code !== 'ENOENT') pushInstallWarning('stripInternalLearnings', 'learnings/index.json', err);
     return;
   }
-  if (!parsed || !Array.isArray(parsed.topics)) return;
-
-  const kept = parsed.topics.filter(t => t && t.scope !== 'internal');
-  if (kept.length === parsed.topics.length) return; // nothing internal to drop
-
-  parsed.topics = kept;
-  if (parsed.totals && typeof parsed.totals === 'object') {
-    parsed.totals.topics = kept.length;
-    parsed.totals.patterns = kept.reduce((n, t) => n + (Array.isArray(t.patterns) ? t.patterns.length : 0), 0);
-    parsed.totals.size_bytes = kept.reduce((n, t) => n + (t.size_bytes || 0), 0);
-    parsed.totals.size_tokens_est = kept.reduce((n, t) => n + (t.size_tokens_est || 0), 0);
-  }
+  // The transform is pure and shared with the bundle builders (install-lib):
+  // null means "not an index" or "nothing internal to drop" — both no-ops here.
+  const stripped = lib.stripInternalLearningsTopics(parsed);
+  if (!stripped) return;
 
   try {
-    fs.writeFileSync(indexPath, JSON.stringify(parsed, null, 2) + '\n');
+    fs.writeFileSync(indexPath, JSON.stringify(stripped, null, 2) + '\n');
   } catch (err) {
     pushInstallWarning('stripInternalLearnings', 'learnings/index.json', err);
   }
@@ -2230,10 +2213,8 @@ function install(isGlobal, runtime = 'claude') {
         fs.mkdirSync(agentsRefDir, { recursive: true });
         const agentsSrc = path.join(src, 'agents');
         for (const f of fs.readdirSync(agentsSrc).filter(n => n.endsWith('.md'))) {
-          let content = fs.readFileSync(path.join(agentsSrc, f), 'utf8');
-          content = content.replace(/~\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-          content = content.replace(/\.\/\.claude\/pan-wizard-core\//g, `${corePrefix}pan-wizard-core/`);
-          content = convertSlashCommandsToCopilotSkillMentions(content);
+          // Shared with the Agent Plugins bundle builder (install-lib).
+          const content = lib.rewriteAgentReferenceCopy(fs.readFileSync(path.join(agentsSrc, f), 'utf8'), corePrefix);
           fs.writeFileSync(path.join(agentsRefDir, f), content);
         }
       } catch (e) {

package/commands/pan/army.md

@@ -92,7 +92,7 @@ Every cap the conductor enforces applies to the campaign, scaled up:
 | `--push` | off | Push approved merges to origin (still human-gated). |
 | `--clean-seal` | off | One clean build + full verification after the last item (commands from config). |
 | `--schedule` | off | Arm a self-resuming campaign at this cadence (`hourly`/`daily`/`weekly`/`Nh`/`Nd`) instead of running once — writes the schedule descriptor (ADR-0034). Pair with `--daily-budget`. |
-| `--daily-budget` | 300 | Per-day point budget for a scheduled campaign. Advisory by default (an indicator of the day's spend); it only pauses the day's run when `budget.enforce`/`enforce_budget` is set. |
+| `--daily-budget` | 300 | Per-day point budget for a scheduled campaign. Advisory by default (an indicator of the day's spend); it only pauses the day's run when `enforce_budget: true` is set by hand in the schedule descriptor (`schedule.json`); no flag or config key sets it. |
 | `--dry-run` | off | Plan + squad delegation preview only; STOP. |
 | `--continue` / `--stop` / `--status` | — | Resume / halt / report from `.planning/orchestration/` + focus-auto state. |
 

package/commands/pan/cost.md

@@ -2,7 +2,7 @@
 name: pan:cost
 group: Observability
 description: Show token usage and estimated cost across PAN commands and agents
-argument-hint: "[report|append|clear] [--format json|table|chart] [--since YYYY-MM-DD] [--until YYYY-MM-DD]"
+argument-hint: "[report|append|clear|rebuild] [--format json|table|chart] [--since YYYY-MM-DD] [--until YYYY-MM-DD] [--apply] [--no-main-thread]"
 allowed-tools:
   - Read
   - Bash
@@ -45,7 +45,10 @@ pan-tools cost report [--format json|table|chart] [--since YYYY-MM-DD] [--until
     "cache_read_tokens": 50000,
     "cache_write_tokens": 5000,
     "cost_usd": 2.1234,
-    "cost_unknown": 0
+    "cost_unknown": 0,
+    "suspect_excluded": 0,
+    "empty_excluded": 0,
+    "malformed_skipped": 0
   },
   "cache_hit_rate_pct": 40.5,
   "by_agent": { "pan-planner": { "calls": 8, "input": 50000, ... } },
@@ -78,6 +81,15 @@ Delete the cost log. Useful at the start of a billing cycle.
 pan-tools cost clear
 ```
 
+### `rebuild`
+
+Rebuild the ledger from Claude Code's own transcripts (session file plus the per-agent files under `<session>/subagents/`, Workflow-tool subagents one level down). Rows written by hooks before v3.29 booked a slice of the parent session to whichever subagent stopped and counted turns once per content block; the rebuild replaces them with one exact row per agent transcript, typed from the main thread's `Agent` calls, plus one row per session for the main thread's own usage (`--no-main-thread` omits it). Rows whose session transcript is gone, and caller-appended rows, are kept. Dry-run by default — show the user the per-session before → after (and any `warnings`), then apply only on their say-so; the previous ledger is kept beside the new one as `tokens.jsonl.rebuilt-<date>` (a later copy never overwrites an earlier one). Run it **before** `/pan:hygiene --apply` on a poisoned ledger: quarantine moves the whole file aside, and a rebuild afterwards has no rows left to keep. The main-thread row is dated to the session's last record, so `--since`/`--until` windows and the per-day view see a session's own usage on its final day.
+
+```
+pan-tools cost rebuild              # dry run
+pan-tools cost rebuild --apply      # write the rebuilt ledger
+```
+
 </subcommands>
 
 <rate_table>

package/commands/pan/preview.md

@@ -34,9 +34,9 @@ Consolidates Spec B v1's architect + simulate + predict-milestone into one entry
 **What it does:**
 1. `pan-tools preview phase <N>` returns `{files_mentioned, test_files_mentioned, risk_signals, risk_score, plans[], status}`.
 2. Spawn `pan-previewer` with the payload as `<preview_input>`.
-3. Agent writes `.planning/phases/<N>/preview.md` with files touched / tests at risk / migration steps / risk assessment / bottom line.
+3. Agent writes `.planning/phases/<NN-slug>/preview.md` with files touched / tests at risk / migration steps / risk assessment / bottom line.
 
-**Output:** `.planning/phases/<N>/preview.md`
+**Output:** `.planning/phases/<NN-slug>/preview.md`
 
 ### `phases` — Cross-phase dependency graph
 

package/hooks/dist/pan-check-update.js

@@ -113,6 +113,10 @@ function defaultFetchLatest() {
     encoding: 'utf8',
     timeout: 10000,
     windowsHide: true,
+    // Own the silence here rather than relying on the caller's stdio: 'ignore'. npm
+    // writes registry and PATH failures to stderr, and a hook that lets them through
+    // puts its own diagnostics in front of the user mid-session.
+    stdio: ['ignore', 'pipe', 'ignore'],
   }).trim();
 }