@yemi33/minions 0.1.247 → 0.1.249

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.249 (2026-04-03)
+
+### Features
+-  mandatory test validation before PR creation in implement and fix playbooks
+
+## 0.1.248 (2026-04-03)
+
+### Fixes
+-  skip worktrees in project scan, make scan optional during init
+
 ## 0.1.245 (2026-04-03)
 
 ### Features

package/minions.js

@@ -263,6 +263,14 @@ function findGitRepos(rootDir, maxDepth = 3) {
 
       const gitDir = path.join(dir, '.git');
       if (fs.existsSync(gitDir)) {
+        // Skip git worktrees — their .git is a file containing "gitdir: ..." pointing to the main repo
+        const stat = fs.statSync(gitDir);
+        if (stat.isFile()) {
+          try {
+            const content = fs.readFileSync(gitDir, 'utf8');
+            if (content.trimStart().startsWith('gitdir:')) return; // worktree, skip
+          } catch {}
+        }
         repos.push(dir);
         return; // Don't recurse into git repos (they may have nested submodules)
       }
@@ -459,9 +467,15 @@ async function initMinions({ skipScan = false, scanRoot, scanDepth } = {}) {
     console.log('  Run "minions scan" or "minions add <dir>" to link projects.\n');
     rl.close();
   } else {
-    // Auto-chain into scan (scanAndAdd closes rl)
-    console.log('  Now let\'s find your repos...\n');
-    await scanAndAdd({ root: scanRoot, depth: scanDepth });
+    // Offer to scan for repos (optional)
+    console.log('  You can link projects now or skip and add them later.\n');
+    const doScan = await ask('Scan for projects? (Y/n/skip)', 'Y');
+    if (doScan.toLowerCase() === 'n' || doScan.toLowerCase() === 'skip') {
+      console.log('  Skipped. Run "minions scan" or "minions add <dir>" to link projects later.\n');
+      rl.close();
+    } else {
+      await scanAndAdd({ root: scanRoot, depth: scanDepth });
+    }
   }
 }
 

package/package.json

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

package/playbooks/fix.md

@@ -43,6 +43,15 @@ If you encounter merge conflicts (e.g., during `git pull` or when the PR shows c
 - content: Explain what was fixed, reference each review finding
 - Sign: `Fixed by Minions ({{agent_name}} — {{agent_role}})`
 
+## Test Validation (MANDATORY before pushing)
+
+Before pushing your fix, run the project's test suite:
+
+1. Find the test command (check `package.json` scripts, CLAUDE.md, or README — usually `npm test`)
+2. Run the full test suite
+3. If any tests fail due to your changes, fix them before pushing
+4. Do NOT push code that breaks existing tests
+
 ## Signal Completion
 
 **Note:** Do NOT write to `agents/*/status.json` — the engine manages your status automatically.

package/playbooks/implement.md

@@ -68,3 +68,16 @@ After building, verify the build succeeded. If the build fails:
 2. Fix the issue
 3. Re-run the build
 4. If it fails 3 times, report the build errors in your findings file and stop
+
+## Test Validation (MANDATORY before PR)
+
+Before creating a PR, you MUST run the project's test suite and ensure all existing tests pass:
+
+1. Find the test command (check `package.json` scripts, CLAUDE.md, or README — usually `npm test`)
+2. Run the full test suite
+3. If any tests fail:
+   - Determine if YOUR changes caused the failure (compare with the failing test's assertions)
+   - Fix any regressions you introduced
+   - Re-run tests until all pass
+4. If tests were already failing before your changes (pre-existing failures), note them in the PR description but do NOT block on them
+5. Do NOT create a PR with failing tests that you introduced