@yemi33/minions 0.1.232 → 0.1.234

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.234 (2026-04-03)
+
+### Fixes
+-  clean _pendingReason on all status→done transitions, not just updateWorkItemStatus
+
+## 0.1.233 (2026-04-03)
+
+### Fixes
+-  combine nuke and reset into single 'minions nuke --confirm'
+
 ## 0.1.232 (2026-04-03)
 
 ### Fixes

package/bin/minions.js

@@ -458,7 +458,7 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
     minions spawn <agent> <prompt>   Manually spawn an agent
     minions plan <file|text> [proj]  Run a plan
     minions cleanup                  Clean temp files, worktrees, zombies
-    minions reset --confirm          Factory reset (delete all state, keep config)
+    minions nuke --confirm           Factory reset (delete state, reset config to defaults)
 
   Dashboard:
     minions dash                     Start web dashboard (default :7331)
@@ -497,68 +497,114 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
   dashProc.unref();
   console.log(`  Dashboard started (PID: ${dashProc.pid})`);
   console.log('  Dashboard: http://localhost:7331\n');
-} else if (cmd === 'reset') {
+} else if (cmd === 'nuke') {
   ensureInstalled();
   if (!rest.includes('--confirm')) {
     console.log(`
-  This will DELETE all runtime state:
-    - Work items, dispatch queue, PRDs, plans
+  Factory reset — kills all processes and deletes all runtime state.
+
+  DELETED:
+    - Work items, dispatch queue, PRDs, plans, pipelines
     - Agent history, sessions, output logs
     - Notes, knowledge base, pinned notes
-    - Metrics, cooldowns, schedules
+    - Metrics, cooldowns, schedules, meetings
+    - Project state (PR tracking, per-project work items)
+
+  RESET to defaults:
+    - config.json (engine settings, agents — project links removed)
+    - routing.md
 
-  Config.json and agent charters are PRESERVED.
+  PRESERVED:
+    - Agent charters
+    - Playbooks
 
-  Run: minions reset --confirm
+  Run: minions nuke --confirm
 `);
     process.exit(0);
   }
-  // Stop engine + dashboard
+
+  console.log('\n  Minions Factory Reset\n');
+
+  // 1. Kill all processes
   try { execSync(`node "${path.join(MINIONS_HOME, 'engine.js')}" stop`, { stdio: 'ignore', cwd: MINIONS_HOME }); } catch {}
+  // Kill dashboard
+  try {
+    if (process.platform === 'win32') {
+      const out = execSync('wmic process where "name=\'node.exe\'" get processid,commandline /format:csv', { encoding: 'utf8', timeout: 10000, windowsHide: true });
+      for (const line of out.split('\n')) {
+        if (line.includes('minions') && (line.includes('engine.js') || line.includes('dashboard.js') || line.includes('spawn-agent.js'))) {
+          const pid = line.split(',').pop()?.trim();
+          if (pid && pid !== String(process.pid)) {
+            try { process.kill(parseInt(pid)); } catch {}
+          }
+        }
+      }
+    } else {
+      try { execSync('lsof -ti:7331 | xargs kill -9 2>/dev/null', { timeout: 5000 }); } catch {}
+    }
+  } catch {}
+  console.log('  Killed all processes');
+
+  // 2. Delete runtime state
   const glob = (dir, pattern) => { try { return fs.readdirSync(dir).filter(f => pattern.test(f)).map(f => path.join(dir, f)); } catch { return []; } };
   const rm = (f) => { try { fs.unlinkSync(f); } catch {} };
   const rmDir = (d) => { try { fs.rmSync(d, { recursive: true, force: true }); } catch {} };
   const engineDir = path.join(MINIONS_HOME, 'engine');
+
   // Engine state
-  for (const f of ['dispatch.json', 'control.json', 'log.json', 'metrics.json', 'cooldowns.json', 'schedule-runs.json', 'kb-checkpoint.json', 'cc-session.json', 'doc-sessions.json']) rm(path.join(engineDir, f));
+  for (const f of ['dispatch.json', 'control.json', 'log.json', 'metrics.json', 'cooldowns.json', 'schedule-runs.json', 'kb-checkpoint.json', 'cc-session.json', 'doc-sessions.json', 'pipeline-runs.json']) rm(path.join(engineDir, f));
   glob(engineDir, /^pid-.*\.pid$/).forEach(rm);
   rmDir(path.join(engineDir, 'tmp'));
+
   // Work items + PRs
   rm(path.join(MINIONS_HOME, 'work-items.json'));
   rm(path.join(MINIONS_HOME, 'work-items-archive.json'));
   rm(path.join(MINIONS_HOME, 'pull-requests.json'));
-  // Plans + PRDs
-  glob(path.join(MINIONS_HOME, 'plans'), /\.md$/).forEach(rm);
-  glob(path.join(MINIONS_HOME, 'prd'), /\.json$/).forEach(rm);
-  rmDir(path.join(MINIONS_HOME, 'prd', 'archive'));
-  rmDir(path.join(MINIONS_HOME, 'prd', 'guides'));
+
+  // Plans + PRDs + Pipelines + Meetings
+  rmDir(path.join(MINIONS_HOME, 'plans'));
+  rmDir(path.join(MINIONS_HOME, 'prd'));
+  rmDir(path.join(MINIONS_HOME, 'pipelines'));
+  rmDir(path.join(MINIONS_HOME, 'meetings'));
+  fs.mkdirSync(path.join(MINIONS_HOME, 'plans'), { recursive: true });
+  fs.mkdirSync(path.join(MINIONS_HOME, 'prd'), { recursive: true });
+
   // Notes + KB
   rm(path.join(MINIONS_HOME, 'notes.md'));
   rm(path.join(MINIONS_HOME, 'pinned.md'));
-  rmDir(path.join(MINIONS_HOME, 'notes', 'inbox'));
-  rmDir(path.join(MINIONS_HOME, 'notes', 'archive'));
+  rmDir(path.join(MINIONS_HOME, 'notes'));
+  rmDir(path.join(MINIONS_HOME, 'knowledge'));
   fs.mkdirSync(path.join(MINIONS_HOME, 'notes', 'inbox'), { recursive: true });
-  fs.mkdirSync(path.join(MINIONS_HOME, 'notes', 'archive'), { recursive: true });
-  for (const cat of ['architecture', 'conventions', 'project-notes', 'build-reports', 'reviews']) {
-    const catDir = path.join(MINIONS_HOME, 'knowledge', cat);
-    glob(catDir, /\.md$/).forEach(rm);
-  }
+
   // Agent state (preserve charters)
   const agentsDir = path.join(MINIONS_HOME, 'agents');
   try {
     for (const agent of fs.readdirSync(agentsDir)) {
       const agentDir = path.join(agentsDir, agent);
       if (!fs.statSync(agentDir).isDirectory()) continue;
-      for (const f of ['live-output.log', 'session.json', 'history.md', 'steer.md', 'status.json']) rm(path.join(agentDir, f));
-      glob(agentDir, /^output.*\.log$/).forEach(rm);
+      for (const f of fs.readdirSync(agentDir)) {
+        if (f === 'charter.md') continue;
+        rm(path.join(agentDir, f));
+      }
     }
   } catch {}
+
   // Projects state
   rmDir(path.join(MINIONS_HOME, 'projects'));
   fs.mkdirSync(path.join(MINIONS_HOME, 'projects'), { recursive: true });
 
-  console.log('\n  Reset complete. Config and charters preserved.');
-  console.log('  Run: minions up\n');
+  // 3. Reset config.json and routing.md to defaults
+  const tmplPath = path.join(MINIONS_HOME, 'config.template.json');
+  const configPath = path.join(MINIONS_HOME, 'config.json');
+  if (fs.existsSync(tmplPath)) {
+    fs.copyFileSync(tmplPath, configPath);
+  } else {
+    fs.writeFileSync(configPath, JSON.stringify({ projects: [], engine: {}, claude: {}, agents: {} }, null, 2));
+  }
+  // Re-run init to populate defaults (agents, engine settings)
+  try { execSync(`node "${path.join(MINIONS_HOME, 'minions.js')}" init --skip-scan`, { stdio: 'inherit' }); } catch {}
+
+  console.log('\n  Factory reset complete. Run "minions init" to link projects and start fresh.\n');
 } else if (cmd === 'doctor') {
   ensureInstalled();
   const { doctor } = require(path.join(MINIONS_HOME, 'engine', 'preflight'));

package/engine/cleanup.js

@@ -364,6 +364,7 @@ function runCleanup(config, verbose = false) {
       for (const item of items) {
         if (LEGACY_DONE_STATUSES.has(item.status)) {
           item.status = 'done';
+          delete item._pendingReason;
           migrated++;
         }
       }
@@ -381,6 +382,7 @@ function runCleanup(config, verbose = false) {
     for (const item of centralItems) {
       if (LEGACY_DONE_STATUSES.has(item.status)) {
         item.status = 'done';
+        delete item._pendingReason;
         migrated++;
       }
     }

package/engine/lifecycle.js

@@ -284,23 +284,33 @@ function checkPlanCompletion(meta, config) {
     shared.safeWrite(planPath, plan);
   }
 
-  // Also archive the source .md plan if it exists
+  // Also archive the source .md plan if it exists (use source_plan field, not content matching)
   const planArchiveDir = path.join(PLANS_DIR, 'archive');
   if (!fs.existsSync(planArchiveDir)) fs.mkdirSync(planArchiveDir, { recursive: true });
-  try {
-    const mdFiles = fs.readdirSync(PLANS_DIR).filter(f => f.endsWith('.md'));
-    for (const md of mdFiles) {
-      const mdContent = shared.safeRead(path.join(PLANS_DIR, md)) || '';
-      // Match by project name or plan summary appearing in the .md content
-      if (mdContent.includes(projectName) || mdContent.includes(plan.plan_summary?.slice(0, 40) || '___nomatch___')) {
-        try {
-          fs.renameSync(path.join(PLANS_DIR, md), path.join(planArchiveDir, md));
-          log('info', `Archived source plan: plans/archive/${md}`);
-        } catch (err) { log('warn', `Failed to archive plan ${md}: ${err.message}`); }
-        break;
-      }
+  if (plan.source_plan) {
+    const mdPath = path.join(PLANS_DIR, plan.source_plan);
+    if (fs.existsSync(mdPath)) {
+      try {
+        fs.renameSync(mdPath, path.join(planArchiveDir, plan.source_plan));
+        log('info', `Archived source plan: plans/archive/${plan.source_plan}`);
+      } catch (err) { log('warn', `Failed to archive source plan ${plan.source_plan}: ${err.message}`); }
     }
-  } catch (err) { log('warn', `Plan archive scan: ${err.message}`); }
+  } else {
+    // Fallback: scan for matching .md files (legacy PRDs without source_plan)
+    try {
+      const mdFiles = fs.readdirSync(PLANS_DIR).filter(f => f.endsWith('.md'));
+      for (const md of mdFiles) {
+        const mdContent = shared.safeRead(path.join(PLANS_DIR, md)) || '';
+        if (mdContent.includes(projectName) || mdContent.includes(plan.plan_summary?.slice(0, 40) || '___nomatch___')) {
+          try {
+            fs.renameSync(path.join(PLANS_DIR, md), path.join(planArchiveDir, md));
+            log('info', `Archived source plan: plans/archive/${md}`);
+          } catch (err) { log('warn', `Failed to archive plan ${md}: ${err.message}`); }
+          break;
+        }
+      }
+    } catch (err) { log('warn', `Plan archive scan: ${err.message}`); }
+  }
 
   // 6. Clean up ALL worktrees created for this plan's work items (shared-branch + per-item)
   try {
@@ -791,6 +801,7 @@ async function handlePostMerge(pr, project, config, newStatus) {
             log('info', `Post-merge: marking work item ${mergedItemId} as done (was ${item.status}) for ${pr.id}`);
             item.status = 'done';
             item.completedAt = ts();
+            delete item._pendingReason;
             item._mergedVia = pr.id;
             found = true;
           }

package/minions.js

@@ -465,105 +465,6 @@ async function initMinions({ skipScan = false, scanRoot, scanDepth } = {}) {
   }
 }
 
-function nukeMinions() {
-  console.log('\n  Minions Factory Reset');
-  console.log('  ===================\n');
-
-  // 1. Kill engine process
-  const controlPath = path.join(MINIONS_HOME, 'engine', 'control.json');
-  try {
-    const control = JSON.parse(fs.readFileSync(controlPath, 'utf8'));
-    if (control.pid) {
-      try {
-        process.kill(control.pid);
-        console.log(`  Killed engine (PID: ${control.pid})`);
-      } catch { console.log(`  Engine process ${control.pid} already dead`); }
-    }
-  } catch {}
-
-  // 2. Kill dashboard (port 7331)
-  try {
-    if (process.platform === 'win32') {
-      const { execSync } = require('child_process');
-      const out = execSync('netstat -ano | findstr :7331 | findstr LISTENING', { encoding: 'utf8', timeout: 5000, windowsHide: true });
-      const pids = [...new Set(out.split('\n').map(l => l.trim().split(/\s+/).pop()).filter(p => p && p !== '0'))];
-      for (const pid of pids) {
-        try { execSync(`taskkill /F /PID ${pid}`, { windowsHide: true, timeout: 5000 }); console.log(`  Killed dashboard (PID: ${pid})`); } catch {}
-      }
-    } else {
-      const { execSync } = require('child_process');
-      try { execSync('lsof -ti:7331 | xargs kill -9 2>/dev/null', { timeout: 5000 }); console.log('  Killed dashboard'); } catch {}
-    }
-  } catch {}
-
-  // 3. Kill all agent processes (PID files in engine/tmp/)
-  const pidDir = path.join(MINIONS_HOME, 'engine', 'tmp');
-  try {
-    const pidFiles = fs.readdirSync(pidDir).filter(f => f.endsWith('.pid'));
-    for (const f of pidFiles) {
-      try {
-        const pid = parseInt(fs.readFileSync(path.join(pidDir, f), 'utf8').trim());
-        if (pid) { try { process.kill(pid); console.log(`  Killed agent process (PID: ${pid})`); } catch {} }
-      } catch {}
-    }
-  } catch {}
-
-  // 4. Kill any remaining minions-related node processes
-  try {
-    const { execSync } = require('child_process');
-    if (process.platform === 'win32') {
-      // Find node processes with minions in their command line
-      const out = execSync('wmic process where "name=\'node.exe\'" get processid,commandline /format:csv', { encoding: 'utf8', timeout: 10000, windowsHide: true });
-      for (const line of out.split('\n')) {
-        if (line.includes('minions') && (line.includes('engine.js') || line.includes('dashboard.js') || line.includes('spawn-agent.js'))) {
-          const pid = line.split(',').pop()?.trim();
-          if (pid && pid !== String(process.pid)) {
-            try { execSync(`taskkill /F /PID ${pid}`, { windowsHide: true, timeout: 5000 }); console.log(`  Killed minions process (PID: ${pid})`); } catch {}
-          }
-        }
-      }
-    }
-  } catch {}
-
-  // 5. Delete runtime state (NOT the source code)
-  console.log('\n  Cleaning runtime state...');
-  const runtimeDirs = ['projects', 'plans', 'prd', 'knowledge', 'skills', 'notes', 'identity'];
-  const runtimeFiles = ['config.json', 'work-items.json', 'notes.md', 'routing.md'];
-  const engineRuntimeFiles = ['control.json', 'dispatch.json', 'log.json', 'metrics.json', 'cooldowns.json', 'kb-checkpoint.json', 'cc-session.json', 'doc-sessions.json'];
-
-  for (const dir of runtimeDirs) {
-    const p = path.join(MINIONS_HOME, dir);
-    if (fs.existsSync(p)) { try { fs.rmSync(p, { recursive: true, force: true }); console.log(`  Deleted ${dir}/`); } catch {} }
-  }
-  for (const f of runtimeFiles) {
-    const p = path.join(MINIONS_HOME, f);
-    if (fs.existsSync(p)) { try { fs.unlinkSync(p); console.log(`  Deleted ${f}`); } catch {} }
-  }
-  const engineDir = path.join(MINIONS_HOME, 'engine');
-  for (const f of engineRuntimeFiles) {
-    const p = path.join(engineDir, f);
-    if (fs.existsSync(p)) { try { fs.unlinkSync(p); console.log(`  Deleted engine/${f}`); } catch {} }
-  }
-  // Clean engine/tmp/
-  const tmpDir = path.join(engineDir, 'tmp');
-  if (fs.existsSync(tmpDir)) { try { fs.rmSync(tmpDir, { recursive: true, force: true }); console.log('  Deleted engine/tmp/'); } catch {} }
-  // Clean agent history and output logs (preserve charters)
-  const agentsDir = path.join(MINIONS_HOME, 'agents');
-  if (fs.existsSync(agentsDir)) {
-    for (const agent of fs.readdirSync(agentsDir)) {
-      const agentDir = path.join(agentsDir, agent);
-      try { if (!fs.statSync(agentDir).isDirectory()) continue; } catch { continue; }
-      for (const f of fs.readdirSync(agentDir)) {
-        if (f === 'charter.md') continue; // preserve charters
-        try { fs.unlinkSync(path.join(agentDir, f)); } catch {}
-      }
-    }
-    console.log('  Cleaned agent state (charters preserved)');
-  }
-
-  console.log('  Factory reset complete. Run "minions init" to start fresh.\n');
-  rl.close();
-}
 
 const commands = {
   init: () => {
@@ -586,7 +487,6 @@ const commands = {
   list: () => listProjects(),
   scan: () => scanAndAdd({ root: rest[0], depth: rest[1] })
     .catch(e => { console.error(e); process.exit(1); }),
-  nuke: () => nukeMinions(),
 };
 
 if (cmd && commands[cmd]) {
@@ -599,7 +499,6 @@ if (cmd && commands[cmd]) {
   console.log('    scan [dir] [depth]      Scan for git repos and multi-select to add');
   console.log('    add <project-dir>       Link a single project');
   console.log('    remove <project-dir>    Unlink a project');
-  console.log('    list                    List linked projects');
-  console.log('    nuke                    Factory reset — kill all processes, delete ~/.minions/\n');
+  console.log('    list                    List linked projects\n');
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.232",
+  "version": "0.1.234",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"