@yemi33/minions 0.1.1558 → 0.1.1560

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 0.1.1560 (2026-04-27)
+
+### Features
+- removeProject also archives plans + PRDs targeting the project
+
+### Fixes
+-  handle project browse cancel
+
 ## 0.1.1558 (2026-04-27)
 
 ### Fixes

package/dashboard/js/render-other.js

@@ -44,6 +44,7 @@ async function projectChipRemove(name) {
     var parts = ['Removed "' + name + '"'];
     if (d.cancelledItems) parts.push(d.cancelledItems + ' WI cancelled');
     if (d.drainedDispatches) parts.push(d.drainedDispatches + ' dispatch drained');
+    if (d.archivedPlans?.length) parts.push(d.archivedPlans.length + ' plan/PRD archived');
     if (d.archivedTo) parts.push('archived');
     if (d.pipelineRefs?.length) parts.push('! pipelines still reference: ' + d.pipelineRefs.join(', '));
     showToast('cmd-toast', parts.join(' — '), true);

package/dashboard/js/settings.js

@@ -424,6 +424,7 @@ async function removeProject(name) {
     if (d.cancelledItems) parts.push(d.cancelledItems + ' WI cancelled');
     if (d.drainedDispatches) parts.push(d.drainedDispatches + ' dispatch drained');
     if (d.cleanedWorktrees) parts.push(d.cleanedWorktrees + ' worktree(s) cleaned');
+    if (d.archivedPlans?.length) parts.push(d.archivedPlans.length + ' plan/PRD archived');
     if (d.archivedTo) parts.push('archived to ' + d.archivedTo);
     if (d.pipelineRefs?.length) parts.push('! pipelines still reference: ' + d.pipelineRefs.join(', '));
     showToast('cmd-toast', parts.join(' — '), true);

package/dashboard.js

@@ -3707,11 +3707,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
   async function handleProjectsBrowse(req, res) {
     try {
-      const { execSync } = require('child_process');
+      const { execSync, execFileSync } = require('child_process');
       let selectedPath = '';
       if (process.platform === 'win32') {
-        // PowerShell STA with topmost window as owner — forces folder dialog to foreground
-        // Write PS script to temp file to avoid shell quoting issues
+        // Launch PowerShell directly (not through cmd.exe) and hide its console so
+        // only the folder picker is visible. Closing the picker should cancel cleanly
+        // instead of surfacing a raw shell "Command failed" error.
         const psScript = [
           'Add-Type -AssemblyName System.Windows.Forms',
           '$f = New-Object System.Windows.Forms.FolderBrowserDialog',
@@ -3730,10 +3731,43 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         fs.mkdirSync(path.dirname(psPath), { recursive: true });
         fs.writeFileSync(psPath, psScript);
         try {
-          selectedPath = execSync(`powershell -STA -NoProfile -ExecutionPolicy Bypass -File "${psPath}"`, { encoding: 'utf8', timeout: 120000 }).trim();
+          selectedPath = execFileSync('powershell.exe', [
+            '-STA',
+            '-NoProfile',
+            '-ExecutionPolicy', 'Bypass',
+            '-File', psPath,
+          ], {
+            encoding: 'utf8',
+            timeout: 120000,
+            windowsHide: true,
+            stdio: ['ignore', 'pipe', 'pipe'],
+          }).trim();
+        } catch (e) {
+          const stdout = String(e.stdout || '').trim();
+          const stderr = String(e.stderr || '').trim();
+          const signal = String(e.signal || '').toUpperCase();
+          const status = Number.isInteger(e.status) ? e.status : null;
+          const interrupted = signal === 'SIGINT' || signal === 'SIGBREAK' || status === 0xC000013A;
+          if (interrupted && !stdout && !stderr) return jsonReply(res, 200, { cancelled: true });
+          throw e;
         } finally { try { fs.unlinkSync(psPath); } catch { /* cleanup */ } }
       } else if (process.platform === 'darwin') {
-        selectedPath = execSync(`osascript -e 'POSIX path of (choose folder with prompt "Select project folder")'`, { encoding: 'utf8', timeout: 120000 }).trim();
+        try {
+          selectedPath = execFileSync('osascript', [
+            '-e',
+            'POSIX path of (choose folder with prompt "Select project folder")',
+          ], {
+            encoding: 'utf8',
+            timeout: 120000,
+            stdio: ['ignore', 'pipe', 'pipe'],
+          }).trim();
+        } catch (e) {
+          const stderr = String(e.stderr || '').trim();
+          const message = String(e.message || '').trim();
+          const cancelled = stderr.includes('User canceled') || message.includes('User canceled') || message.includes('(-128)') || stderr.includes('(-128)');
+          if (cancelled) return jsonReply(res, 200, { cancelled: true });
+          throw e;
+        }
       } else {
         selectedPath = execSync(`zenity --file-selection --directory --title="Select project folder" 2>/dev/null`, { encoding: 'utf8', timeout: 120000 }).trim();
       }

package/engine/projects.js

@@ -44,6 +44,7 @@ function removeProject(target, options = {}) {
     drainedDispatches: 0, // includes active dispatches whose agent processes were killed
     cleanedWorktrees: 0,
     disabledSchedules: 0,
+    archivedPlans: [],
     pipelineRefs: [],
     archivedTo: null,
     purgedDataDir: false,
@@ -110,7 +111,52 @@ function removeProject(target, options = {}) {
     }
   }
 
-  // 5. Surface pipelines that reference this project so the user can review
+  // 5. Archive plans + PRDs targeting this project so they don't keep
+  //    showing in the dashboard after removal. Three signals:
+  //      a) PRD JSON `project` field matches → archive PRD + linked source plan
+  //      b) plan .md content has `Project: <name>` (any markdown emphasis)
+  //      c) plan .md filename contains the project name (case-insensitive)
+  //    The .backup sidecar removal in archivePlan prevents engine restart
+  //    from re-triggering plan completion (mirror of f28162b0 fix).
+  const plansDir = path.join(MINIONS_DIR, 'plans');
+  const plansArchive = path.join(plansDir, 'archive');
+  const prdDir = path.join(MINIONS_DIR, 'prd');
+  const escName = project.name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+  const projectLineRe = new RegExp('^[*_>\\s]{0,4}\\*{0,2}_{0,2}Project:?\\*{0,2}_{0,2}\\s*' + escName + '\\b', 'im');
+  const archivedSourcePlans = new Set();
+  try {
+    if (fs.existsSync(prdDir)) {
+      const { archivePlan } = require('./lifecycle');
+      for (const f of fs.readdirSync(prdDir).filter(f => f.endsWith('.json'))) {
+        try {
+          const prd = JSON.parse(fs.readFileSync(path.join(prdDir, f), 'utf8'));
+          if (prd?.project !== project.name) continue;
+          archivePlan(f, prd, [project], config);
+          summary.archivedPlans.push('prd/' + f);
+          if (prd.source_plan) archivedSourcePlans.add(prd.source_plan);
+        } catch { /* skip unreadable PRD */ }
+      }
+    }
+  } catch (e) { summary.warnings.push('archive PRDs: ' + e.message); }
+  try {
+    if (fs.existsSync(plansDir)) {
+      fs.mkdirSync(plansArchive, { recursive: true });
+      const lowerName = project.name.toLowerCase();
+      for (const f of fs.readdirSync(plansDir).filter(f => f.endsWith('.md'))) {
+        if (archivedSourcePlans.has(f)) continue; // already moved by archivePlan above
+        const fp = path.join(plansDir, f);
+        try {
+          const filenameMatch = f.toLowerCase().includes(lowerName);
+          const content = filenameMatch ? '' : fs.readFileSync(fp, 'utf8');
+          if (!filenameMatch && !projectLineRe.test(content)) continue;
+          fs.renameSync(fp, path.join(plansArchive, f));
+          summary.archivedPlans.push('plans/' + f);
+        } catch { /* skip unreadable plan */ }
+      }
+    }
+  } catch (e) { summary.warnings.push('archive plans: ' + e.message); }
+
+  // 6. Surface pipelines that reference this project so the user can review
   //    them. Don't auto-modify — user intent there is unclear.
   try {
     const { getPipelines } = require('./pipeline');
@@ -125,12 +171,12 @@ function removeProject(target, options = {}) {
     }
   } catch { /* pipelines optional */ }
 
-  // 6. Remove from config.json (and persist any schedule disables)
+  // 7. Remove from config.json (and persist any schedule disables)
   config.projects = (config.projects || []).filter(p => p.name !== project.name);
   try { fs.writeFileSync(configPath, JSON.stringify(config, null, 2)); }
   catch (e) { return { ...summary, error: 'Failed to write config: ' + e.message }; }
 
-  // 7. Move (or purge) projects/<name>/ — preserves PR/work-item history by
+  // 8. Move (or purge) projects/<name>/ — preserves PR/work-item history by
   //    default so a re-add can pick up where it left off.
   const dataDir = path.join(MINIONS_DIR, 'projects', project.name);
   if (fs.existsSync(dataDir)) {

package/minions.js

@@ -221,6 +221,7 @@ function removeProject(target, options = {}) {
   if (result.cancelledItems) lines.push(`    cancelled ${result.cancelledItems} pending work item(s)`);
   if (result.drainedDispatches) lines.push(`    drained ${result.drainedDispatches} dispatch entr${result.drainedDispatches === 1 ? 'y' : 'ies'} (active agents killed)`);
   if (result.cleanedWorktrees) lines.push(`    cleaned ${result.cleanedWorktrees} worktree(s)`);
+  if (result.archivedPlans?.length) lines.push(`    archived ${result.archivedPlans.length} plan/PRD file(s)`);
   if (result.disabledSchedules) lines.push(`    disabled ${result.disabledSchedules} schedule(s)`);
   if (result.archivedTo) lines.push(`    archived data → ${result.archivedTo}`);
   if (result.purgedDataDir) lines.push(`    purged data directory`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1558",
+  "version": "0.1.1560",
   "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"