pan-wizard 3.14.0 → 3.15.0

package/pan-wizard-core/references/ui-brand.md

@@ -1,160 +1,160 @@
-<ui_patterns>
-
-Visual patterns for user-facing PAN output. Orchestrators @-reference this file.
-
-## Stage Banners
-
-Use for major workflow transitions.
-
-```
-━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
- PAN ► {STAGE NAME}
-━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-```
-
-**Stage names (uppercase):**
-- `QUESTIONING`
-- `RESEARCHING`
-- `DEFINING REQUIREMENTS`
-- `CREATING ROADMAP`
-- `PLANNING PHASE {N}`
-- `EXECUTING WAVE {N}`
-- `VERIFYING`
-- `PHASE {N} COMPLETE ✓`
-- `MILESTONE COMPLETE 🎉`
-
----
-
-## Checkpoint Boxes
-
-User action required. 62-character width.
-
-```
-╔══════════════════════════════════════════════════════════════╗
-║  CHECKPOINT: {Type}                                          ║
-╚══════════════════════════════════════════════════════════════╝
-
-{Content}
-
-──────────────────────────────────────────────────────────────
-→ {ACTION PROMPT}
-──────────────────────────────────────────────────────────────
-```
-
-**Types:**
-- `CHECKPOINT: Verification Required` → `→ Type "approved" or describe issues`
-- `CHECKPOINT: Decision Required` → `→ Select: option-a / option-b`
-- `CHECKPOINT: Action Required` → `→ Type "done" when complete`
-
----
-
-## Status Symbols
-
-```
-✓  Complete / Passed / Verified
-✗  Failed / Missing / Blocked
-◆  In Progress
-○  Pending
-⚡ Auto-approved
-⚠  Warning
-🎉 Milestone complete (only in banner)
-```
-
----
-
-## Progress Display
-
-**Phase/milestone level:**
-```
-Progress: ████████░░ 80%
-```
-
-**Task level:**
-```
-Tasks: 2/4 complete
-```
-
-**Plan level:**
-```
-Plans: 3/5 complete
-```
-
----
-
-## Spawning Indicators
-
-```
-◆ Spawning researcher...
-
-◆ Spawning 4 researchers in parallel...
-  → Stack research
-  → Features research
-  → Architecture research
-  → Pitfalls research
-
-✓ Researcher complete: STACK.md written
-```
-
----
-
-## Next Up Block
-
-Always at end of major completions.
-
-```
-───────────────────────────────────────────────────────────────
-
-## ▶ Next Up
-
-**{Identifier}: {Name}** — {one-line description}
-
-`{copy-paste command}`
-
-<sub>`/clear` first → fresh context window</sub>
-
-───────────────────────────────────────────────────────────────
-
-**Also available:**
-- `/pan:alternative-1` — description
-- `/pan:alternative-2` — description
-
-───────────────────────────────────────────────────────────────
-```
-
----
-
-## Error Box
-
-```
-╔══════════════════════════════════════════════════════════════╗
-║  ERROR                                                       ║
-╚══════════════════════════════════════════════════════════════╝
-
-{Error description}
-
-**To fix:** {Resolution steps}
-```
-
----
-
-## Tables
-
-```
-| Phase | Status | Plans | Progress |
-|-------|--------|-------|----------|
-| 1     | ✓      | 3/3   | 100%     |
-| 2     | ◆      | 1/4   | 25%      |
-| 3     | ○      | 0/2   | 0%       |
-```
-
----
-
-## Anti-Patterns
-
-- Varying box/banner widths
-- Mixing banner styles (`===`, `---`, `***`)
-- Skipping `PAN ►` prefix in banners
-- Random emoji (`🚀`, `✨`, `💫`)
-- Missing Next Up block after completions
-
-</ui_patterns>
+<ui_patterns>
+
+Visual patterns for user-facing PAN output. Orchestrators @-reference this file.
+
+## Stage Banners
+
+Use for major workflow transitions.
+
+```
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+ PAN ► {STAGE NAME}
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+```
+
+**Stage names (uppercase):**
+- `QUESTIONING`
+- `RESEARCHING`
+- `DEFINING REQUIREMENTS`
+- `CREATING ROADMAP`
+- `PLANNING PHASE {N}`
+- `EXECUTING WAVE {N}`
+- `VERIFYING`
+- `PHASE {N} COMPLETE ✓`
+- `MILESTONE COMPLETE 🎉`
+
+---
+
+## Checkpoint Boxes
+
+User action required. 62-character width.
+
+```
+╔══════════════════════════════════════════════════════════════╗
+║  CHECKPOINT: {Type}                                          ║
+╚══════════════════════════════════════════════════════════════╝
+
+{Content}
+
+──────────────────────────────────────────────────────────────
+→ {ACTION PROMPT}
+──────────────────────────────────────────────────────────────
+```
+
+**Types:**
+- `CHECKPOINT: Verification Required` → `→ Type "approved" or describe issues`
+- `CHECKPOINT: Decision Required` → `→ Select: option-a / option-b`
+- `CHECKPOINT: Action Required` → `→ Type "done" when complete`
+
+---
+
+## Status Symbols
+
+```
+✓  Complete / Passed / Verified
+✗  Failed / Missing / Blocked
+◆  In Progress
+○  Pending
+⚡ Auto-approved
+⚠  Warning
+🎉 Milestone complete (only in banner)
+```
+
+---
+
+## Progress Display
+
+**Phase/milestone level:**
+```
+Progress: ████████░░ 80%
+```
+
+**Task level:**
+```
+Tasks: 2/4 complete
+```
+
+**Plan level:**
+```
+Plans: 3/5 complete
+```
+
+---
+
+## Spawning Indicators
+
+```
+◆ Spawning researcher...
+
+◆ Spawning 4 researchers in parallel...
+  → Stack research
+  → Features research
+  → Architecture research
+  → Pitfalls research
+
+✓ Researcher complete: STACK.md written
+```
+
+---
+
+## Next Up Block
+
+Always at end of major completions.
+
+```
+───────────────────────────────────────────────────────────────
+
+## ▶ Next Up
+
+**{Identifier}: {Name}** — {one-line description}
+
+`{copy-paste command}`
+
+<sub>`/clear` first → fresh context window</sub>
+
+───────────────────────────────────────────────────────────────
+
+**Also available:**
+- `/pan:alternative-1` — description
+- `/pan:alternative-2` — description
+
+───────────────────────────────────────────────────────────────
+```
+
+---
+
+## Error Box
+
+```
+╔══════════════════════════════════════════════════════════════╗
+║  ERROR                                                       ║
+╚══════════════════════════════════════════════════════════════╝
+
+{Error description}
+
+**To fix:** {Resolution steps}
+```
+
+---
+
+## Tables
+
+```
+| Phase | Status | Plans | Progress |
+|-------|--------|-------|----------|
+| 1     | ✓      | 3/3   | 100%     |
+| 2     | ◆      | 1/4   | 25%      |
+| 3     | ○      | 0/2   | 0%       |
+```
+
+---
+
+## Anti-Patterns
+
+- Varying box/banner widths
+- Mixing banner styles (`===`, `---`, `***`)
+- Skipping `PAN ►` prefix in banners
+- Random emoji (`🚀`, `✨`, `💫`)
+- Missing Next Up block after completions
+
+</ui_patterns>

package/pan-wizard-core/templates/config.json

@@ -1,38 +1,38 @@
-{
-  "mode": "interactive",
-  "depth": "standard",
-  "workflow": {
-    "research": true,
-    "plan_check": true,
-    "verifier": true,
-    "auto_advance": false,
-    "nyquist_validation": false,
-    "phase_record_compact": false
-  },
-  "planning": {
-    "commit_docs": true,
-    "search_gitignored": false
-  },
-  "parallelization": {
-    "enabled": true,
-    "plan_level": true,
-    "task_level": false,
-    "skip_checkpoints": true,
-    "max_concurrent_agents": 3,
-    "min_plans_for_parallel": 2
-  },
-  "gates": {
-    "confirm_project": true,
-    "confirm_phases": true,
-    "confirm_roadmap": true,
-    "confirm_breakdown": true,
-    "confirm_plan": true,
-    "execute_next_plan": true,
-    "issues_review": true,
-    "confirm_transition": true
-  },
-  "safety": {
-    "always_confirm_destructive": true,
-    "always_confirm_external_services": true
-  }
-}
+{
+  "mode": "interactive",
+  "depth": "standard",
+  "workflow": {
+    "research": true,
+    "plan_check": true,
+    "verifier": true,
+    "auto_advance": false,
+    "nyquist_validation": false,
+    "phase_record_compact": false
+  },
+  "planning": {
+    "commit_docs": true,
+    "search_gitignored": false
+  },
+  "parallelization": {
+    "enabled": true,
+    "plan_level": true,
+    "task_level": false,
+    "skip_checkpoints": true,
+    "max_concurrent_agents": 3,
+    "min_plans_for_parallel": 2
+  },
+  "gates": {
+    "confirm_project": true,
+    "confirm_phases": true,
+    "confirm_roadmap": true,
+    "confirm_breakdown": true,
+    "confirm_plan": true,
+    "execute_next_plan": true,
+    "issues_review": true,
+    "confirm_transition": true
+  },
+  "safety": {
+    "always_confirm_destructive": true,
+    "always_confirm_external_services": true
+  }
+}

package/scripts/build-hooks.js

@@ -1,51 +1,51 @@
-#!/usr/bin/env node
-/**
- * Copy PAN hooks from hooks/ to hooks/dist/ for installation.
- *
- * This is COPY-ONLY. PAN's hooks are pure Node.js with zero runtime
- * dependencies, so no bundling step is needed. The script is named
- * "build-hooks" for npm-script convention, but the work is `cp`.
- *
- * (See docs/IMPROVEMENT-TODO.md P2 for the rationale.)
- */
-
-const fs = require('fs');
-const path = require('path');
-
-const HOOKS_DIR = path.join(__dirname, '..', 'hooks');
-const DIST_DIR = path.join(HOOKS_DIR, 'dist');
-
-// Hooks to copy (pure Node.js, no bundling needed)
-const HOOKS_TO_COPY = [
-  'pan-check-update.js',
-  'pan-context-monitor.js',
-  'pan-statusline.js',
-  'pan-cost-logger.js',
-  'pan-trace-logger.js'
-];
-
-function build() {
-  // Ensure dist directory exists
-  if (!fs.existsSync(DIST_DIR)) {
-    fs.mkdirSync(DIST_DIR, { recursive: true });
-  }
-
-  // Copy hooks to dist
-  for (const hook of HOOKS_TO_COPY) {
-    const src = path.join(HOOKS_DIR, hook);
-    const dest = path.join(DIST_DIR, hook);
-
-    if (!fs.existsSync(src)) {
-      console.warn(`Warning: ${hook} not found, skipping`);
-      continue;
-    }
-
-    console.log(`Copying ${hook}...`);
-    fs.copyFileSync(src, dest);
-    console.log(`  → ${dest}`);
-  }
-
-  console.log('\nBuild complete.');
-}
-
-build();
+#!/usr/bin/env node
+/**
+ * Copy PAN hooks from hooks/ to hooks/dist/ for installation.
+ *
+ * This is COPY-ONLY. PAN's hooks are pure Node.js with zero runtime
+ * dependencies, so no bundling step is needed. The script is named
+ * "build-hooks" for npm-script convention, but the work is `cp`.
+ *
+ * (See docs/IMPROVEMENT-TODO.md P2 for the rationale.)
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const HOOKS_DIR = path.join(__dirname, '..', 'hooks');
+const DIST_DIR = path.join(HOOKS_DIR, 'dist');
+
+// Hooks to copy (pure Node.js, no bundling needed)
+const HOOKS_TO_COPY = [
+  'pan-check-update.js',
+  'pan-context-monitor.js',
+  'pan-statusline.js',
+  'pan-cost-logger.js',
+  'pan-trace-logger.js'
+];
+
+function build() {
+  // Ensure dist directory exists
+  if (!fs.existsSync(DIST_DIR)) {
+    fs.mkdirSync(DIST_DIR, { recursive: true });
+  }
+
+  // Copy hooks to dist
+  for (const hook of HOOKS_TO_COPY) {
+    const src = path.join(HOOKS_DIR, hook);
+    const dest = path.join(DIST_DIR, hook);
+
+    if (!fs.existsSync(src)) {
+      console.warn(`Warning: ${hook} not found, skipping`);
+      continue;
+    }
+
+    console.log(`Copying ${hook}...`);
+    fs.copyFileSync(src, dest);
+    console.log(`  → ${dest}`);
+  }
+
+  console.log('\nBuild complete.');
+}
+
+build();

package/scripts/release-check.js

@@ -116,40 +116,30 @@ process.stderr.write('\n[release-check] Gate 5/7: links validate\n');
 }
 
 
-// npm runs lifecycle scripts (prepare) before pack; any of their stdout noise
-// lands ahead of the --json payload. npm pretty-prints the JSON array starting
-// on its own line — parse from there.
-function parseNpmJson(stdout) {
-  try { return JSON.parse(stdout); } catch { /* fall through to extraction */ }
-  const m = stdout.search(/^[[{]s*$/m);
-  if (m === -1) throw new Error('no JSON payload found in npm output');
-  return JSON.parse(stdout.slice(m));
-}
-
-// Gate 6: npm pack dry-run
-process.stderr.write('\n[release-check] Gate 6/7: npm pack --dry-run\n');
+// NOTE: we deliberately do NOT parse `npm pack --json` stdout. Under
+// `npm publish` the runner routes the child pack's lifecycle-script output onto
+// stdout (foreground-scripts), and that noise can include a decoy JSON object —
+// any string heuristic then picks the wrong payload (Gate 7 crashed on
+// packJson[0].filename with "0 files"). Instead, pack into a temp dir and read
+// the .tgz npm actually wrote: a filesystem op that stdout noise cannot corrupt.
+
+// Gate 6: npm pack — produces a non-empty, sanely-sized tarball (read the file,
+// never parse stdout)
+process.stderr.write('\n[release-check] Gate 6/7: npm pack (size sanity)\n');
 {
-  const r = run('npm', ['pack', '--dry-run', '--json'], { capture: true });
-  if (r.status !== 0) {
-    logGate('npm pack dry-run', false, `exit ${r.status}`);
+  const tmp6 = fs.mkdtempSync(path.join(os.tmpdir(), 'pan-release-pack-'));
+  const r = run('npm', ['pack', '--pack-destination', tmp6], { capture: true });
+  const tgz = r.status === 0 ? fs.readdirSync(tmp6).find(f => f.endsWith('.tgz')) : null;
+  const size = tgz ? fs.statSync(path.join(tmp6, tgz)).size : 0;
+  const sizeMB = (size / 1024 / 1024).toFixed(2);
+  // Sane = a non-empty tarball under 50MB (large for a zero-runtime-dep tool)
+  const ok = r.status === 0 && !!tgz && size > 0 && size < 50 * 1024 * 1024;
+  logGate('npm pack', ok, tgz ? `${sizeMB}MB tarball` : `no tarball (exit ${r.status})`);
+  fs.rmSync(tmp6, { recursive: true, force: true });
+  if (!ok) {
     process.stderr.write((r.stderr || '') + '\n');
     process.exit(1);
   }
-  // Parse the JSON output to check size
-  try {
-    const parsed = parseNpmJson(r.stdout);
-    const entry = Array.isArray(parsed) ? parsed[0] : parsed;
-    const size = entry.size || 0;
-    const fileCount = entry.files ? entry.files.length : 0;
-    const sizeMB = (size / 1024 / 1024).toFixed(2);
-    // Flag if pack size exceeds 50MB — large for a zero-runtime-dep tool
-    const ok = size < 50 * 1024 * 1024;
-    logGate('npm pack dry-run', ok, `${sizeMB}MB, ${fileCount} files`);
-    if (!ok) process.exit(1);
-  } catch (err) {
-    logGate('npm pack dry-run', false, 'JSON parse failed: ' + err.message);
-    process.exit(1);
-  }
 }
 
 // Gate 7: smoke install — pack and install into temp dir, run pan-tools
@@ -159,18 +149,19 @@ if (SKIP_SMOKE) {
   process.stderr.write('\n[release-check] Gate 7/7: smoke install (npm pack + install + sanity)\n');
   const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'pan-release-smoke-'));
   try {
-    // Pack
-    const pack = run('npm', ['pack', '--pack-destination', tmpDir, '--json'], { capture: true });
+    // Pack — read the .tgz npm writes to tmpDir; never parse its stdout (see note).
+    const pack = run('npm', ['pack', '--pack-destination', tmpDir], { capture: true });
     if (pack.status !== 0) {
       logGate('smoke install (pack)', false, `exit ${pack.status}`);
+      process.stderr.write((pack.stderr || '') + '\n');
       process.exit(1);
     }
-    const packJson = parseNpmJson(pack.stdout);
-    const tarball = path.join(tmpDir, packJson[0].filename);
-    if (!fs.existsSync(tarball)) {
-      logGate('smoke install (pack)', false, `tarball not found at ${tarball}`);
+    const tgz = fs.readdirSync(tmpDir).find(f => f.endsWith('.tgz'));
+    if (!tgz) {
+      logGate('smoke install (pack)', false, `no .tgz produced in ${tmpDir}`);
       process.exit(1);
     }
+    const tarball = path.join(tmpDir, tgz);
     // Install into a separate fake project dir
     const installDir = path.join(tmpDir, 'install-target');
     fs.mkdirSync(installDir, { recursive: true });