@yemi33/minions 0.1.2000 → 0.1.2002

package/engine/dispatch.js

@@ -598,7 +598,28 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
               });
               log('warn', `Demoted WI ${item.meta.item.id} from DONE → FAILED${classSuffix} — sidecar acceptance gate rejected after completion-report claimed success`);
             }
-          } catch (e) { log('warn', `force-demote on ${failureClass}: ${e.message}`); }
+          } catch (e) {
+            // P-bf06-force-demote-observability — write failures during
+            // sidecar-rejection force-demote used to log at warn level only,
+            // so they were invisible to dashboards / inbox alerting. Surface
+            // them at error level with the full stack AND a synthetic
+            // agent-failure inbox alert so the failed write is debuggable.
+            // NOTE: do NOT call lifecycle().syncPrdItemStatus from this catch
+            // — the success-path PRD sync at line 688 owns that contract and
+            // adding it here would double-write PRD status under conflicting
+            // assumptions.
+            log('error', `force-demote on ${failureClass} failed: ${e.message}\n${e.stack}`, { stack: e.stack, failureClass });
+            try {
+              // Pass a shimmed dispatch id so writeToInbox's slug-based dedupe
+              // (see shared.writeToInbox prefix collision check) doesn't drop
+              // this alert against the prior agent-failure note that the
+              // ERROR branch already wrote at line 550.
+              const demoteItem = { ...item, id: `${item.id}-demote-write` };
+              writeFailedAgentReport(demoteItem, `force-demote write failed: ${e.message}`, e.stack || '', 'dispatch-demote-write-failed');
+            } catch (reportErr) {
+              log('error', `force-demote write failed-agent-report itself failed: ${reportErr.message}`);
+            }
+          }
         } else {
           log('info', `Dispatch error for ${item.meta.item.id} ignored — work item is already completed`);
         }

package/engine/github.js

@@ -869,6 +869,13 @@ async function pollPrStatus(config) {
             ].filter(Boolean).join('\n') || buildFailReason, 80);
           } else if (allDone && allPassed) {
             buildStatus = 'passing';
+          } else if (allDone) {
+            // Terminal-but-not-passing conclusions (cancelled, action_required,
+            // stale, etc.) — map to 'none' rather than 'failing'. These are not
+            // actionable build failures, so we deliberately do not block merge or
+            // dispatch build-fix agents. Previously fell through to 'running'
+            // forever (P-bf02-github-cancelled-stuck).
+            buildStatus = 'none';
           } else {
             buildStatus = 'running';
           }

package/engine/lifecycle.js

@@ -3465,6 +3465,9 @@ function handleDecompositionResult(stdout, meta, config, runtimeName) {
       // Mark parent as decomposed
       p.status = WI_STATUS.DECOMPOSED;
       p._decomposed = true;
+      // P-bf04: timestamp used by engine.js areDependenciesMet() to classify
+      // zero-children failures as recent vs stale in logs (both cascade 'failed').
+      p._decomposedAt = ts();
       delete p._decomposing;
       // Sync to PRD so dashboard shows decomposed status
       if (p.sourcePlan) syncPrdItemStatus(p.id, WI_STATUS.DECOMPOSED, p.sourcePlan);

package/engine/pipeline.js

@@ -756,15 +756,23 @@ function executeMergePrsStage(stage, stageState, run, config) {
 function executeScheduleStage(stage, stageState, config) {
   // Create/update schedules in config
   const schedules = stage.schedules || [{ id: stage.id + '-sched', cron: stage.cron, title: stage.title, type: routing.normalizeWorkType(stage.taskType, WORK_TYPE.IMPLEMENT) }];
-  // Write to config via shared
-  for (const sched of schedules) {
-    const existing = (config.schedules || []).find(s => s.id === sched.id);
-    if (!existing) {
-      config.schedules = config.schedules || [];
-      config.schedules.push({ ...sched, enabled: true });
+  // P-bf05-config-write-race — atomic locked write so concurrent
+  // addProject/removeProject/dashboard-settings writers can't clobber the
+  // schedules we just appended. The prior `safeWrite(CONFIG_PATH, config)`
+  // last-write-wins corrupted config.json on bootstrap (no `.backup`
+  // self-heal). Mutate the fresh on-disk snapshot inside the lock so we
+  // don't drop concurrent edits.
+  mutateJsonFileLocked(CONFIG_PATH, (cfg) => {
+    if (!cfg || typeof cfg !== 'object' || Array.isArray(cfg)) return cfg;
+    for (const sched of schedules) {
+      const existing = (cfg.schedules || []).find(s => s.id === sched.id);
+      if (!existing) {
+        cfg.schedules = cfg.schedules || [];
+        cfg.schedules.push({ ...sched, enabled: true });
+      }
     }
-  }
-  safeWrite(CONFIG_PATH, config);
+    return cfg;
+  });
   return { status: PIPELINE_STATUS.COMPLETED, completedAt: ts() };
 }
 
@@ -1151,7 +1159,7 @@ module.exports = {
   getPipelineRuns, getActiveRun, startRun, updateRunStage, completeRun,
   discoverPipelineWork,
   evaluateCondition, // exported for testing
-  executeTaskStage, executePlanStage, isStageComplete, resolveTemplate, // exported for testing
+  executeTaskStage, executePlanStage, executeScheduleStage, isStageComplete, resolveTemplate, // exported for testing
   _resolvePipelineProjects, // exported for testing
   _findMeetingsInRun, _findExistingPlanForMeeting, _findExistingPrdForPlan, // exported for testing
 };

package/engine/playbook.js

@@ -467,18 +467,74 @@ function buildQaValidateContextBlock({ runId, runbook, target, artifactsDir }) {
 
 // ─── Playbook Renderer ──────────────────────────────────────────────────────
 
+/**
+ * Parse YAML-ish frontmatter at the top of a playbook file.
+ * Returns { meta: {key:value}, body: string }. Recognized values:
+ *   true|yes|1  → boolean true
+ *   false|no|0  → boolean false
+ *   everything else → raw string
+ */
+function _parsePlaybookFrontmatter(content) {
+  const m = String(content || '').match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?([\s\S]*)$/);
+  if (!m) return { meta: {}, body: content || '' };
+  const meta = {};
+  for (const line of m[1].split(/\r?\n/)) {
+    const lm = line.match(/^([\w-]+):\s*(.*)$/);
+    if (!lm) continue;
+    const key = lm[1];
+    const raw = lm[2].trim();
+    if (/^(true|yes|1)$/i.test(raw)) meta[key] = true;
+    else if (/^(false|no|0)$/i.test(raw)) meta[key] = false;
+    else meta[key] = raw;
+  }
+  return { meta, body: m[2].replace(/^\r?\n+/, '') };
+}
+
 function renderPlaybook(type, vars) {
   const projectName = vars.project_name || null;
   const pbPath = resolvePlaybookPath(projectName, type);
-  let content;
-  try { content = fs.readFileSync(pbPath, 'utf8'); } catch {
+  let rawContent;
+  try { rawContent = fs.readFileSync(pbPath, 'utf8'); } catch {
     log('warn', `Playbook not found: ${type}`);
     return null;
   }
 
-  // Inject shared rules (apply to all playbooks)
+  // P-bf07 — parse frontmatter and enforce requiresProjectContext before any
+  // substitution work. Hard-fail dispatch when the playbook requires a
+  // project but none is configured (i.e. fresh install) so the agent prompt
+  // can never expand {{project_name}} / {{repo_name}} / {{ado_org}} /
+  // {{github_org}} to empty strings or the bogus 'Unknown' defaults.
+  const { meta: playbookMeta, body: bodyAfterFm } = _parsePlaybookFrontmatter(rawContent);
+  let content = bodyAfterFm;
+
+  const configEarly = getConfig();
+  const projectsEarly = getProjects(configEarly);
+  const matchedProject = (vars.repo_id && projectsEarly.find(p => p.repositoryId === vars.repo_id))
+    || (vars.repo_name && projectsEarly.find(p => p.repoName === vars.repo_name))
+    || projectsEarly[0]
+    || null;
+  // Callers (PR dispatches, fan-out, unit tests) may supply project context
+  // in vars even when on-disk config has zero projects. Treat that as
+  // "project context provided" — the only true bug is when buildBaseVars
+  // fell back to the placeholder defaults ('Unknown Project' / 'Unknown')
+  // because no project object was available.
+  const callerSuppliedProjectContext = !!(
+    vars.repo_id
+    || (vars.repo_name && vars.repo_name !== 'Unknown')
+    || (vars.project_name && vars.project_name !== 'Unknown Project')
+  );
+  if (!matchedProject && !callerSuppliedProjectContext && playbookMeta.requiresProjectContext === true) {
+    const err = new Error(`Playbook ${type} requires a project but none is configured. Add a project via \`minions project add\`.`);
+    err.code = 'PLAYBOOK_REQUIRES_PROJECT';
+    err.playbook = type;
+    throw err;
+  }
+
+  // Inject shared rules (apply to all playbooks). Strip any frontmatter so the
+  // P-bf07 audit metadata doesn't leak into the rendered prompt.
   try {
-    const sharedRules = fs.readFileSync(path.join(PLAYBOOKS_DIR, 'shared-rules.md'), 'utf8');
+    const sharedRulesRaw = fs.readFileSync(path.join(PLAYBOOKS_DIR, 'shared-rules.md'), 'utf8');
+    const { body: sharedRules } = _parsePlaybookFrontmatter(sharedRulesRaw);
     if (sharedRules) content += '\n\n' + sharedRules;
   } catch { /* optional — shared rules file may not exist */ }
 
@@ -625,12 +681,13 @@ function renderPlaybook(type, vars) {
   content += `- **SOURCE REFERENCES for every finding** — file paths with line numbers, PR URLs, API endpoints, config keys. Format: \`(source: path/to/file.ts:42)\` or \`(source: PR-12345)\`. Without references, findings cannot be verified.\n\n`;
 
   // Inject project-level variables from config
-  const config = getConfig();
-  const projects = getProjects(config);
-  // Find the specific project being dispatched (match by repo_id or repo_name from vars)
-  const dispatchProject = (vars.repo_id && projects.find(p => p.repositoryId === vars.repo_id))
-    || (vars.repo_name && projects.find(p => p.repoName === vars.repo_name))
-    || projects[0] || {};
+  const config = configEarly;
+  const projects = projectsEarly;
+  // Find the specific project being dispatched (match by repo_id or repo_name from vars).
+  // P-bf07 — when no project resolves we keep the legacy `{}` fallback ONLY for
+  // playbooks that opted out (requiresProjectContext: false). Required-context
+  // playbooks already short-circuited above with a structured error.
+  const dispatchProject = matchedProject || {};
   const renderProject = {
     ...dispatchProject,
     adoOrg: vars.ado_org || vars.adoOrg || dispatchProject.adoOrg,
@@ -997,6 +1054,7 @@ module.exports = {
   validatePlaybookVars,
   PLAYBOOK_REQUIRED_VARS,
   PLAYBOOK_OPTIONAL_VARS,
+  _parsePlaybookFrontmatter,
   buildSystemPrompt,
   buildAgentContext,
   selectPlaybook,

package/engine/projects.js

@@ -203,16 +203,13 @@ function removeProject(target, options = {}) {
     } catch (e) { summary.warnings.push('worktree cleanup: ' + e.message); }
   }
 
-  // 4. Disable schedules whose `project` field targets this project
-  //    specifically. Don't touch schedules with project='any' or unset.
-  if (Array.isArray(config.schedules)) {
-    for (const s of config.schedules) {
-      if (_projectRefMatches(s.project, project, projects) && s.enabled !== false) {
-        s.enabled = false;
-        summary.disabledSchedules++;
-      }
-    }
-  }
+  // 4. Schedule disables happen atomically with the project unlink in
+  //    section 7's mutateJsonFileLocked callback so concurrent writers
+  //    (addProject, dashboard settings update, pipeline schedule stage)
+  //    don't clobber each other. The previous pattern mutated the unlocked
+  //    in-memory `config` snapshot here and relied on the section 7 write
+  //    to persist it — that lost concurrent updates that landed between
+  //    the initial read and the write.
 
   // 5. Archive plans + PRDs targeting this project so they don't keep
   //    showing in the dashboard after removal. Three signals:
@@ -277,10 +274,31 @@ function removeProject(target, options = {}) {
     }
   } catch { /* pipelines optional */ }
 
-  // 7. Remove from config.json (and persist any schedule disables)
-  config.projects = (config.projects || []).filter(p => !_sameProjectName(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. Remove from config.json and persist schedule disables atomically
+  //    under the config.json file lock. P-bf05-config-write-race —
+  //    the prior `fs.writeFileSync` (and section 4's unlocked in-memory
+  //    mutation) raced with addProject, dashboard settings updates, and
+  //    pipeline schedule-stage writers, corrupting config.json on fresh
+  //    bootstrap (which has no `.backup` self-heal). Both mutations now
+  //    run inside the same mutateJsonFileLocked callback against the
+  //    fresh on-disk snapshot.
+  let writeError = null;
+  try {
+    shared.mutateJsonFileLocked(configPath, (cfg) => {
+      if (!cfg || typeof cfg !== 'object' || Array.isArray(cfg)) return cfg;
+      cfg.projects = (cfg.projects || []).filter(p => !_sameProjectName(p?.name, project.name));
+      if (Array.isArray(cfg.schedules)) {
+        for (const s of cfg.schedules) {
+          if (_projectRefMatches(s.project, project, projects) && s.enabled !== false) {
+            s.enabled = false;
+            summary.disabledSchedules++;
+          }
+        }
+      }
+      return cfg;
+    });
+  } catch (e) { writeError = e; }
+  if (writeError) return { ...summary, error: 'Failed to write config: ' + writeError.message };
 
   // 8. Move (or purge) projects/<name>/ — preserves PR/work-item history by
   //    default so a re-add can pick up where it left off.

package/engine/shared.js

@@ -1678,6 +1678,13 @@ const ENGINE_DEFAULTS = {
   shutdownTimeout: 300000, // 5min — max wait for active agents during graceful shutdown
   allowTempAgents: false, // opt-in: spawn ephemeral agents when all permanent agents are busy
   autoDecompose: true, // auto-decompose implement:large items into sub-tasks
+  // P-bf04-decompose-zero-children: when a DECOMPOSED parent has zero children
+  // and no decompose dispatch is in flight, areDependenciesMet treats that as
+  // a silent decompose failure and cascades `'failed'` to dependents. This
+  // window splits the failure log into "recent" vs "stale" categories — both
+  // return `'failed'`; the threshold is for monitoring/log classification only,
+  // not behavior. See engine.js areDependenciesMet() DECOMPOSED branch.
+  decomposeZeroChildrenStaleMinutes: 30,
   autoApprovePlans: false, // auto-approve PRDs without waiting for human approval
   autoArchive: false, // opt-in: auto-archive plans after verify completes (false = mark ready, user archives manually)
   autoFixConflicts: true, // auto-dispatch fix agents when a PR has merge conflicts

package/engine.js

@@ -3061,7 +3061,43 @@ function areDependenciesMet(item, config) {
       const children = allWorkItems.filter(w => w.parent_id === depId);
       if (children.length > 0 && children.every(c => PRD_MET_STATUSES.has(c.status))) continue; // all children done
       if (children.length > 0) return false; // children still in progress
-      continue; // no children found — treat as met (decomposition may be in flight)
+
+      // P-bf04-decompose-zero-children: parent is DECOMPOSED but no children
+      // exist. Previously this silently `continue`d (treat as met), so a crashed
+      // decompose (parser error, empty output, race) cascaded into dependents
+      // running against a parent whose work never happened. New three-branch logic:
+      //
+      //   (1) IN-FLIGHT HEDGE — a decompose dispatch for this parent is active
+      //       or pending → return false (caller waits, no cascade)
+      //   (2) RECENT zero-children — decomposedAt within the last N min
+      //   (3) STALE  zero-children — decomposedAt older than N min (or unknown)
+      //
+      // (2) and (3) both return 'failed'; the split exists for log/monitor
+      // classification (and test scaffolding), not for caller behavior.
+      // PLAN_TO_PRD dispatches are intentionally NOT consulted here — they
+      // materialize plan items into work items, they do not decompose an
+      // existing DECOMPOSED parent into children.
+      try {
+        const dispatch = queries.getDispatch();
+        const inFlight = [...(dispatch.pending || []), ...(dispatch.active || [])]
+          .some(d => d?.type === WORK_TYPE.DECOMPOSE && d?.meta?.item?.id === depId);
+        if (inFlight || depItem._decomposing === true) {
+          log('warn', `Dep ${depId} (DECOMPOSED, zero children) — decompose dispatch still in flight; treating as unmet`);
+          return false;
+        }
+      } catch (e) { log('warn', `areDependenciesMet bf04 in-flight check failed: ${e.message}`); }
+
+      const staleMin = ENGINE_DEFAULTS.decomposeZeroChildrenStaleMinutes;
+      const parsedDecomposedAt = depItem._decomposedAt ? Date.parse(depItem._decomposedAt) : NaN;
+      const ageMs = Number.isFinite(parsedDecomposedAt) ? (Date.now() - parsedDecomposedAt) : null;
+      const isRecent = ageMs !== null && ageMs < staleMin * 60000;
+      if (isRecent) {
+        log('warn', `Dep ${depId} (DECOMPOSED) produced zero children RECENTLY (${Math.round(ageMs / 60000)}m ago) — cascading dependency_failed to ${item.id}`);
+      } else {
+        const ageLabel = ageMs == null ? 'no _decomposedAt' : `${Math.round(ageMs / 60000)}m ago`;
+        log('warn', `Dep ${depId} (DECOMPOSED) produced zero children STALE (${ageLabel}) — cascading dependency_failed to ${item.id}`);
+      }
+      return 'failed';
     }
     if (!PRD_MET_STATUSES.has(depItem.status)) return false; // Pending, dispatched, or retrying — wait
   }
@@ -4077,7 +4113,7 @@ async function discoverFromPrs(config, project) {
               if (target && target.reviewStatus !== 'approved') target.reviewStatus = liveStatus;
               return data;
             });
-          } catch {}
+          } catch (e) { log('warn', `persist live vote-check for ${pr.id}: ${e.message}`); }
           continue;
         }
       } catch (e) { log('warn', `Pre-dispatch vote check for ${pr.id}: ${e.message} — skipping dispatch`); continue; }
@@ -4237,7 +4273,7 @@ async function discoverFromPrs(config, project) {
               if (target && target.reviewStatus !== 'approved') target.reviewStatus = liveStatus;
               return data;
             });
-          } catch {}
+          } catch (e) { log('warn', `persist live re-review vote-check for ${pr.id}: ${e.message}`); }
           continue;
         }
       } catch (e) { log('warn', `Pre-dispatch vote check for ${pr.id}: ${e.message} — skipping dispatch`); continue; }
@@ -4353,7 +4389,7 @@ async function discoverFromPrs(config, project) {
               target._buildStatusStale = true;
               if (live.buildStatusDetail) target._buildStatusDetail = live.buildStatusDetail;
             });
-          } catch {}
+          } catch (e) { log('warn', `persist live build-stale-marker for ${pr.id}: ${e.message}`); }
         }
         if (live && live.buildStatus && live.buildStatus !== 'failing') {
           log('info', `Pre-dispatch build check: ${pr.id} build is ${live.buildStatus} (cached was failing) — skipping build-fix`);
@@ -4374,7 +4410,7 @@ async function discoverFromPrs(config, project) {
                 }
               }
             });
-          } catch {}
+          } catch (e) { log('warn', `persist live build-status for ${pr.id}: ${e.message}`); }
           continue;
         }
       } catch (e) { log('warn', `Pre-dispatch build check for ${pr.id}: ${e.message} — skipping dispatch`); continue; }
@@ -4452,7 +4488,7 @@ async function discoverFromPrs(config, project) {
                 delete target._mergeConflict;
                 delete target._conflictFixedAt;
               });
-            } catch {}
+            } catch (e) { log('warn', `persist live conflict-clear for ${pr.id}: ${e.message}`); }
             liveSkip = true;
           }
         } catch (e) { log('warn', `Pre-dispatch conflict check for ${pr.id}: ${e.message} — skipping dispatch`); liveSkip = true; }
@@ -6626,6 +6662,7 @@ module.exports = {
 
   // Shared helpers (used by lifecycle.js and tests)
   reconcileItemsWithPrs, detectDependencyCycles,
+  areDependenciesMet, // exported for testing (P-bf04-decompose-zero-children)
   parseConflictFiles, pruneAncestorDeps, preflightMergeSimulation, // exported for testing
   buildDepConflictFixItem, // exported for testing (W-mpcwojgr000a0244)
   isWorktreeRetryableError, removeStaleIndexLock, syncReusedWorktree, assertCleanSharedWorktree, // exported for testing

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2000",
+  "version": "0.1.2002",
   "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/ask.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Ask Playbook
 
 > Agent: {{agent_name}} | Question from user | ID: {{task_id}}

package/playbooks/build-and-test.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Build & Test: PR {{pr_id}}
 
 > Agent: {{agent_name}} ({{agent_role}}) | Project: {{project_name}}

package/playbooks/decompose.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Task Decomposition
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/docs.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Docs Playbook
 
 > Agent: {{agent_name}} ({{agent_role}}) | Task: {{item_name}} | ID: {{item_id}}

package/playbooks/explore.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Explore Playbook
 
 > Agent: {{agent_name}} | Task: {{task_description}} | ID: {{task_id}}

package/playbooks/fix.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Fix Review Issues
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/implement-shared.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Implement (Shared Branch)
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/implement.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Implement
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/meeting-conclude.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: false
+---
+
 # Meeting: Conclusion
 
 You are {{agent_name}} ({{agent_role}}), synthesizing the team meeting results.

package/playbooks/meeting-debate.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: false
+---
+
 # Meeting: Debate Round
 
 You are {{agent_name}} ({{agent_role}}) in Round 2 of a team meeting.

package/playbooks/meeting-investigate.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: false
+---
+
 # Meeting: Investigation Round
 
 You are {{agent_name}} ({{agent_role}}) participating in a team meeting.

package/playbooks/plan-to-prd.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Plan → PRD
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/plan.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Feature Plan
 
 > Agent: {{agent_name}} | Task: {{task_description}} | ID: {{task_id}}

package/playbooks/qa-validate.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: QA Validate
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/review.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Review
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/setup.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Playbook: Setup
 
 You are {{agent_name}}, the {{agent_role}} on the {{project_name}} project.

package/playbooks/shared-rules.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: false
+---
+
 ## Operating Principle
 
 Treat a Minions assignment like the user typed the same task directly into a capable CLI agent. Optimize for the requested outcome and use the repo's own tools, conventions, and acceptance criteria.

package/playbooks/test.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Test / Build / Run: {{item_name}}
 
 > Agent: {{agent_name}} ({{agent_role}}) | ID: {{item_id}} | Priority: {{item_priority}}

package/playbooks/verify.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Plan Verification: {{item_name}}
 
 > Agent: {{agent_name}} ({{agent_role}}) | Team root: {{team_root}}

package/playbooks/work-item.md

@@ -1,3 +1,7 @@
+---
+requiresProjectContext: true
+---
+
 # Work Item: {{item_name}}
 
 > Agent: {{agent_name}} ({{agent_role}}) | ID: {{item_id}} | Priority: {{item_priority}} | Type: {{work_type}}