@yemi33/minions 0.1.1692 → 0.1.1694

package/CHANGELOG.md

@@ -1,10 +1,14 @@
 # Changelog
 
-## 0.1.1692 (2026-05-04)
+## 0.1.1694 (2026-05-04)
+
+### Features
+- remove stale permission mode settings (#2009)
+- sync review verdict PR status (#2008)
+
+## 0.1.1691 (2026-05-04)
 
 ### Features
-- stop Azure auth for GitHub agents (#2005)
-- recover stalled runtime resumes (#2004)
 - preserve doc-chat response fragments (#2003)
 
 ## 0.1.1689 (2026-05-03)

package/README.md

@@ -406,7 +406,7 @@ No bash or shell involved — Node spawns Node directly. Dependency branches are
 - **Working directory** — project root (agent creates worktrees as needed)
 - **MCP servers** — inherited from `~/.claude.json` (no extra config needed)
 - **Full tool access** — all built-in tools plus all MCP tools
-- **Permission mode** — `bypassPermissions` (no interactive prompts)
+- **Permission bypass** — runtime-owned; Claude uses `--dangerously-skip-permissions`, while Copilot uses `--autopilot --allow-all --no-ask-user` (no dashboard `permissionMode` control)
 - **Output format** — `stream-json` (real-time streaming for live dashboard + completion recovery)
 
 ### Post-Completion

package/dashboard/js/render-agents.js

@@ -54,7 +54,6 @@ function renderAgents(agents) {
       })()}
       ${a._blockingToolCall ? `<div style="margin-top:4px;padding:4px 8px;background:rgba(130,160,210,0.13);border:1px solid rgba(130,160,210,0.3);border-radius:4px;font-size:10px;color:var(--muted)">&#x23F3; Blocking tool call (${escapeHtml(a._blockingToolCall.tool)}) &mdash; silent ${Math.round(a._blockingToolCall.silentMs/60000)}min, timeout in ${Math.round(a._blockingToolCall.remainingMs/60000)}min</div>` : ''}
       ${a._warning ? `<div style="margin-top:4px;padding:4px 8px;background:rgba(210,153,34,0.15);border:1px solid rgba(210,153,34,0.3);border-radius:4px;font-size:10px;color:var(--yellow)">&#x26A0; ${escapeHtml(a._warning)}</div>` : ''}
-      ${a._permissionMode && a._permissionMode !== 'bypassPermissions' && !a._warning ? `<div style="margin-top:4px;font-size:9px;color:var(--muted)">Permission mode: ${escapeHtml(a._permissionMode)}</div>` : ''}
       ${a.resultSummary ? `<div class="agent-result" title="${escapeHtml(a.resultSummary)}">${renderMd(a.resultSummary.slice(0, 200))}${a.resultSummary.length > 200 ? '...' : ''}</div>` : ''}
     </div>
   `).join('');

package/dashboard/js/settings.js

@@ -258,20 +258,12 @@ async function openSettings() {
     '</div>' +
 
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Claude CLI</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
+    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:8px">' +
       settingsField('Output Format', 'set-outputFormat', c.outputFormat || 'stream-json', '', '') +
-      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Tools agents can use without permission prompts. Add MCP tools here if using non-bypass mode (e.g. mcp__azure-ado__*)') +
+      settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
     '</div>' +
-    '<div style="margin-bottom:16px">' +
-      '<label style="font-size:10px;color:var(--muted);display:block;margin-bottom:2px">Permission Mode <span style="opacity:0.6">(how agents handle tool approvals)</span></label>' +
-      '<select id="set-permissionMode" style="width:100%;padding:4px 6px;background:var(--surface);border:1px solid var(--border);border-radius:4px;color:var(--text);font-size:12px">' +
-        '<option value="bypassPermissions"' + ((c.permissionMode || 'bypassPermissions') === 'bypassPermissions' ? ' selected' : '') + '>Bypass (recommended) — agents run autonomously without permission prompts</option>' +
-        '<option value="auto"' + ((c.permissionMode) === 'auto' ? ' selected' : '') + '>Auto — auto-approve safe tools, prompt for risky ones (agents may hang on risky tools)</option>' +
-        '<option value="default"' + ((c.permissionMode) === 'default' ? ' selected' : '') + '>Default — prompt for every tool (agents WILL hang — not recommended)</option>' +
-      '</select>' +
-      '<div id="set-permissionMode-warn" style="font-size:9px;margin-top:4px;padding:4px 8px;border-radius:4px;' + ((c.permissionMode && c.permissionMode !== 'bypassPermissions') ? 'display:block;background:rgba(248,81,73,0.1);color:var(--red)' : 'display:none') + '">' +
-        '\u26A0 Tools listed in Allowed Tools above are auto-approved even in non-bypass modes. Agents will only hang if they try to use a tool NOT on that list (e.g. MCP tools). In bypass mode, all tools are approved automatically.' +
-      '</div>' +
+    '<div style="font-size:10px;color:var(--muted);margin-bottom:16px;padding:6px 8px;border:1px solid var(--border);border-radius:4px;background:var(--surface-subtle, rgba(130,160,210,0.08))">' +
+      'Permission bypass is runtime-owned: Claude agents use <code>--dangerously-skip-permissions</code>; Copilot agents use <code>--autopilot --allow-all --no-ask-user</code>. There is no dashboard permission-mode setting.' +
     '</div>' +
 
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Agents</h3>' +
@@ -314,20 +306,6 @@ async function openSettings() {
   document.getElementById('modal-body').style.whiteSpace = '';
   document.getElementById('modal').classList.add('open');
 
-  // Wire permission mode warning toggle
-  var pmSelect = document.getElementById('set-permissionMode');
-  if (pmSelect) pmSelect.addEventListener('change', function() {
-    var warn = document.getElementById('set-permissionMode-warn');
-    if (!warn) return;
-    if (this.value !== 'bypassPermissions') {
-      warn.style.display = 'block';
-      warn.style.background = 'rgba(248,81,73,0.1)';
-      warn.style.color = 'var(--red)';
-    } else {
-      warn.style.display = 'none';
-    }
-  });
-
   // ── Runtime fleet wiring (P-7a5c1f8e) ──────────────────────────────────────
   // 1. Load registered runtimes into the defaultCli + ccCli dropdowns.
   // 2. Load models for the selected defaultCli into the defaultModel input.
@@ -595,7 +573,6 @@ async function saveSettings() {
     const claudePayload = {
       outputFormat: document.getElementById('set-outputFormat').value,
       allowedTools: document.getElementById('set-allowedTools').value,
-      permissionMode: document.getElementById('set-permissionMode').value,
     };
 
     const teamsPayload = {

package/dashboard.js

@@ -5405,6 +5405,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     } catch (e) { return jsonReply(res, e.statusCode || 500, { error: e.message }); }
   }
 
+  function settingsClaudeConfig(config) {
+    const claude = { ...shared.DEFAULT_CLAUDE, ...(config.claude || {}) };
+    delete claude.permissionMode;
+    return claude;
+  }
+
   async function handleSettingsRead(req, res) {
     try {
       const config = queries.getConfig();
@@ -5414,7 +5420,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (engine.prPollCommentsEvery === undefined && engine.adoPollCommentsEvery !== undefined) engine.prPollCommentsEvery = engine.adoPollCommentsEvery;
       return jsonReply(res, 200, {
         engine,
-        claude: { ...shared.DEFAULT_CLAUDE, ...(config.claude || {}) },
+        claude: settingsClaudeConfig(config),
         agents: config.agents || {},
         teams: { ...shared.ENGINE_DEFAULTS.teams, ...(config.teams || {}) },
         projects: (config.projects || []).map(p => ({
@@ -5437,6 +5443,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!config.engine) config.engine = {};
       if (!config.claude) config.claude = {};
       if (!config.agents) config.agents = {};
+      delete config.claude.permissionMode;
 
       const _clamped = [];
       const _engineModelDiscovery = require('./engine/model-discovery');
@@ -5607,10 +5614,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         for (const key of ['allowedTools', 'outputFormat']) {
           if (body.claude[key] !== undefined) config.claude[key] = String(body.claude[key]);
         }
-        if (body.claude.permissionMode !== undefined) {
-          const valid = ['bypassPermissions', 'auto', 'default'];
-          config.claude.permissionMode = valid.includes(body.claude.permissionMode) ? body.claude.permissionMode : 'bypassPermissions';
-        }
       }
 
       if (body.agents) {

package/docs/auto-discovery.md

@@ -250,10 +250,12 @@ Combines:
 ```bash
 claude -p --system-prompt-file <sysprompt-file> \
   --output-format stream-json --max-turns 100 --verbose \
-  --permission-mode bypassPermissions
+  --dangerously-skip-permissions
 # Prompt text is piped via stdin (not passed as an arg).
 # Agent dispatches route through engine/spawn-agent.js; CC / doc-chat use a direct
 # spawn path in engine/llm.js that bypasses spawn-agent.js entirely.
+# Permission bypass is runtime-owned: Copilot uses
+# --autopilot --allow-all --no-ask-user instead.
 ```
 
 - Process runs in the worktree directory (or rootDir for reviews)

package/docs/blog-first-successful-dispatch.md

@@ -63,9 +63,9 @@ proc.stdin.end();
 
 Fix: switched to `--output-format stream-json` — streams events as they happen.
 
-### Permission Mode
+### Permission Bypass
 
-Agents would hang waiting for permission prompts (invisible in headless mode). Added `--permission-mode bypassPermissions`.
+Agents would hang waiting for permission prompts (invisible in headless mode). Modern Minions keeps bypass behavior inside runtime adapters: Claude emits `--dangerously-skip-permissions`; Copilot emits `--autopilot --allow-all --no-ask-user`. The legacy `config.claude.permissionMode` key is ignored.
 
 ### CLAUDECODE Environment Variable
 
@@ -85,7 +85,7 @@ When the engine restarts, the in-memory `activeProcesses` Map is lost. Active di
 
 **Dispatch 1773292681199** — Dallas, central work item, auto-route:
 
-1. Engine spawns `node spawn-agent.js prompt.md sysprompt.md --output-format stream-json --verbose --permission-mode bypassPermissions`
+1. Engine spawns `node spawn-agent.js prompt.md sysprompt.md --runtime claude --output-format stream-json --verbose`
 2. spawn-agent.js resolves `cli.js`, spawns `node cli.js -p --system-prompt <content> ...`
 3. Prompt piped via stdin — no shell interpretation
 4. MCP servers connect (azure-ado, azure-kusto, mobile, DevBox)

package/docs/deprecated.json

@@ -22,5 +22,13 @@
     "reason": "These cadence settings gate both ADO and GitHub PR polling, so the ADO-prefixed names are misleading.",
     "locations": ["engine.js read-side fallback from config.engine.adoPollStatusEvery/adoPollCommentsEvery", "dashboard.js handleSettingsRead/handleSettingsUpdate alias fallback for old keys"],
     "cleanup": "Remove the adoPoll* alias fallback reads after existing configs have been migrated to prPollStatusEvery/prPollCommentsEvery."
+  },
+  {
+    "id": "config-claude-permission-mode",
+    "summary": "config.claude.permissionMode ignored; runtime adapters own permission bypass flags",
+    "deprecated": "2026-05-04",
+    "reason": "Claude and Copilot require different non-interactive bypass flags, so a shared Claude config field was misleading and no longer controls spawns.",
+    "locations": ["dashboard.js settings update strips config.claude.permissionMode", "dashboard/js/settings.js no longer renders a Permission Mode selector"],
+    "cleanup": "After old configs have been rewritten through settings, remove the deprecated-field preflight warning entry for permissionMode."
   }
 ]

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-04T01:58:58.614Z"
+  "cachedAt": "2026-05-04T03:31:17.601Z"
 }

package/engine/github.js

@@ -39,6 +39,19 @@ function getRepoSlug(project) {
   return `${org}/${repo}`;
 }
 
+function _hasMinionsReviewVerdict(body) {
+  return /(?:^|\n)\s*\*{0,2}VERDICT[:\s]+\*{0,2}(?:APPROVE|REQUEST[_\s-]?CHANGES)\*{0,2}\b/i.test(String(body || ''));
+}
+
+function _isAgentComment(c) {
+  const body = c.body || '';
+  if (_hasMinionsReviewVerdict(body)) return true;
+  if (/\bMinions\s*\(/i.test(body)) return true;
+  if (/\bby\s+Minions\b/i.test(body)) return true;
+  if (/\[minions\]/i.test(body)) return true;
+  return false;
+}
+
 // ─── Per-Repo Poll Backoff ──────────────────────────────────────────────────
 // Tracks consecutive poll failures per repo slug to avoid spamming logs when
 // a repo is inaccessible. Backoff doubles each failure: 2min, 4min, 8min, 16min, max 30min.
@@ -572,13 +585,6 @@ async function pollPrHumanComments(config) {
       if (/!\[.*\]\(https?:\/\/.*badge/i.test(body)) return true;
       return false;
     }
-    function _isAgentComment(c) {
-      const body = c.body || '';
-      if (/\bMinions\s*\(/i.test(body)) return true;
-      if (/\bby\s+Minions\b/i.test(body)) return true;
-      if (/\[minions\]/i.test(body)) return true;
-      return false;
-    }
     const actionableComments = allComments.filter(c => !_isIgnoredComment(c));
 
     const cutoffStr = pr.humanFeedback?.lastProcessedCommentDate || pr.created || '1970-01-01';
@@ -907,4 +913,6 @@ module.exports = {
   GH_MAX_BUFFER, // exported for testing
   GH_POLL_BACKOFF_BASE_MS, // exported for testing
   GH_POLL_BACKOFF_MAX_MS, // exported for testing
+  _hasMinionsReviewVerdict, // exported for testing
+  _isAgentComment, // exported for testing
 };

package/engine/lifecycle.js

@@ -1266,11 +1266,61 @@ function isReviewBailout(text) {
   return /bail(ing)?\s+out/i.test(text) || /already\s+posted/i.test(text);
 }
 
-async function updatePrAfterReview(agentId, pr, project, config, resultSummary, structuredCompletion = null) {
+function reviewPrRefFromCompletion(completion) {
+  if (!completion || typeof completion !== 'object') return null;
+  const value = String(completion.pr || completion.pull_request || completion.pullRequest || '').trim();
+  if (!value || /^N\/?A$/i.test(value)) return null;
+  return value;
+}
 
-  if (!pr?.id) return;
+function centralPrPath() {
+  return path.join(path.resolve(MINIONS_DIR, '..'), '.minions', 'pull-requests.json');
+}
+
+function resolveReviewPrContext(pr, project, config, structuredCompletion = null) {
+  const refs = [pr, reviewPrRefFromCompletion(structuredCompletion)].filter(Boolean);
+  if (refs.length === 0) return null;
+
+  const projects = shared.getProjects(config);
+  const projectCandidates = [];
+  if (project) projectCandidates.push(project);
+  for (const p of projects) {
+    if (!projectCandidates.some(existing => existing?.name === p.name)) projectCandidates.push(p);
+  }
+
+  for (const candidateProject of projectCandidates) {
+    const prPath = shared.projectPrPath(candidateProject);
+    const prs = safeJson(prPath) || [];
+    for (const ref of refs) {
+      const target = shared.findPrRecord(prs, ref, candidateProject);
+      if (target) return { pr: { ...target }, project: candidateProject, prPath };
+    }
+  }
+
+  const centralPath = centralPrPath();
+  const centralPrs = safeJson(centralPath) || [];
+  for (const ref of refs) {
+    const target = shared.findPrRecord(centralPrs, ref, null);
+    if (target) return { pr: { ...target }, project: null, prPath: centralPath };
+  }
+
+  return pr?.id
+    ? { pr, project: project || null, prPath: project ? shared.projectPrPath(project) : centralPath }
+    : null;
+}
+
+async function updatePrAfterReview(agentId, pr, project, config, resultSummary, structuredCompletion = null) {
 
   if (!config) config = getConfig();
+  const reviewContext = resolveReviewPrContext(pr, project, config, structuredCompletion);
+  if (!reviewContext?.pr?.id) {
+    const reportedPr = reviewPrRefFromCompletion(structuredCompletion);
+    if (reportedPr) log('warn', `Review completion reported PR ${reportedPr}, but no tracked PR record was found`);
+    return;
+  }
+  const reviewPr = reviewContext.pr;
+  const reviewProject = reviewContext.project;
+  const prPath = reviewContext.prPath;
   const reviewerName = config.agents?.[agentId]?.name || agentId;
   const dispatch = getDispatch();
   const completedEntry = (dispatch.completed || []).find(d => d.agent === agentId && d.type === 'review');
@@ -1280,31 +1330,30 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary,
   // The poller will pick up the real status on the next cycle (~3 min).
   let postReviewStatus = null; // null = don't change
   try {
-    const projectObj = project || shared.getProjects(config)[0];
+    const projectObj = reviewProject || shared.getProjects(config)[0];
     if (projectObj) {
       const host = projectObj.repoHost || 'ado';
       const checkFn = host === 'github'
         ? require('./github').checkLiveReviewStatus
         : require('./ado').checkLiveReviewStatus;
-      const liveStatus = await checkFn(pr, projectObj);
+      const liveStatus = await checkFn(reviewPr, projectObj);
       if (liveStatus && liveStatus !== 'pending') postReviewStatus = liveStatus;
     }
-  } catch (e) { log('warn', `Post-review status check for ${pr.id}: ${e.message}`); }
+  } catch (e) { log('warn', `Post-review status check for ${reviewPr.id}: ${e.message}`); }
 
   // Fallback: if live check returned pending (e.g., GitHub self-approval blocked), use the agent's completion report.
   if (!postReviewStatus) {
     const verdict = reviewVerdictFromCompletion(structuredCompletion) || parseReviewVerdict(resultSummary);
     if (verdict) {
       postReviewStatus = verdict;
-      log('info', `Read review verdict from agent completion for ${pr.id}: ${verdict}`);
+      log('info', `Read review verdict from agent completion for ${reviewPr.id}: ${verdict}`);
     }
   }
 
-  const prPath = project ? shared.projectPrPath(project) : path.join(path.resolve(MINIONS_DIR, '..'), '.minions', 'pull-requests.json');
   let updatedTarget = null;
   shared.mutateJsonFileLocked(prPath, (prs) => {
     if (!Array.isArray(prs)) return prs;
-    const target = shared.findPrRecord(prs, pr, project);
+    const target = shared.findPrRecord(prs, reviewPr, reviewProject);
     if (!target) return prs;
     // Once approved, stays approved — only changes-requested can override
     if (postReviewStatus) {
@@ -1323,12 +1372,12 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary,
       // Drop it when reviewer requests changes again — that starts a new fix cycle.
       ...(target.minionsReview?.fixedAt && postReviewStatus !== 'changes-requested' ? { fixedAt: target.minionsReview.fixedAt } : {}),
     };
-    updatedTarget = { ...pr, ...target };
+    updatedTarget = { ...reviewPr, ...target };
     return prs;
   }, { defaultValue: [] });
 
   // Track reviewer for metrics purposes (separate file, separate lock)
-  const authorAgentId = (pr.agent || '').toLowerCase();
+  const authorAgentId = (reviewPr.agent || '').toLowerCase();
   if (authorAgentId && config.agents?.[authorAgentId]) {
     shared.mutateJsonFileLocked(path.join(ENGINE_DIR, 'metrics.json'), (metrics) => {
       if (!metrics[authorAgentId]) metrics[authorAgentId] = { ...DEFAULT_AGENT_METRICS };
@@ -1338,7 +1387,7 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary,
     }, { defaultValue: {} });
   }
 
-  log('info', `Updated ${pr.id} → minions review: ${postReviewStatus || 'waiting'} by ${reviewerName}`);
+  log('info', `Updated ${reviewPr.id} → minions review: ${postReviewStatus || 'waiting'} by ${reviewerName}`);
   if (updatedTarget) createReviewFeedbackForAuthor(agentId, updatedTarget, config);
 }
 

package/engine/queries.js

@@ -428,25 +428,11 @@ function getAgentStatus(agentId) {
     if (active._blockingToolCall) {
       result._blockingToolCall = active._blockingToolCall;
     }
-    // Detect permission-waiting and in-flight tools: read only head+tail of live-output.log (max 2KB total)
+    // Detect in-flight tools: read only head+tail of live-output.log (max 2KB total)
     try {
       const liveLogPath = path.join(AGENTS_DIR, agentId, 'live-output.log');
       const { head, tail } = readHeadTail(liveLogPath, 1024);
       if (head) {
-        // Check init message (in head) for permission mode
-        const initMatch = head.match(/"permissionMode"\s*:\s*"([^"]+)"/);
-        if (initMatch && initMatch[1] !== 'bypassPermissions') {
-          result._permissionMode = initMatch[1];
-        }
-        // Check if agent has been silent for >60s (use tail for recent activity)
-        const lastLine = tail.trimEnd().split('\n').pop();
-        if (lastLine && lastLine.includes('"type":"assistant"') && lastLine.includes('"tool_use"')) {
-          const liveStat = fs.statSync(liveLogPath);
-          const silentMs = Date.now() - liveStat.mtimeMs;
-          if (silentMs > 60000 && result._permissionMode) {
-            result._warning = 'Possibly waiting for permission approval — agent is not in bypass mode';
-          }
-        }
         // Detect in-flight tool calls (task_started with no task_notification)
         const inFlight = detectInFlightTool(tail);
         if (inFlight && inFlight.description) {
@@ -570,7 +556,6 @@ function getAgents(config) {
       completed_at: s.completed_at || null,
       _blockingToolCall: s._blockingToolCall || null,
       _warning: s._warning || null,
-      _permissionMode: s._permissionMode || null,
       chartered, inboxCount: inboxFiles.length + steeringInboxFiles.length
     };
   });

package/engine/shared.js

@@ -851,7 +851,7 @@ const ENGINE_DEFAULTS = {
   // Backward-compat: keep `engine.claude.*` field family deprecation tracker. Listed here so preflight
   // knows which subkeys to flag as deprecated. Do not consume `claude.*` in new code — use the runtime
   // adapter system (engine/runtimes/) and the resolveAgent*/resolveCc* helpers instead.
-  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'maxTurns', 'effort', 'budgetCap'],
+  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'permissionMode', 'maxTurns', 'effort', 'budgetCap'],
   // Teams integration — config.teams shape: { enabled, appId, appPassword, certPath, privateKeyPath, tenantId, notifyEvents, ccMirror, inboxPollInterval }
   // Auth modes: (1) appId + appPassword (client secret), or (2) appId + certPath + privateKeyPath + tenantId (certificate)
   teams: {
@@ -1310,14 +1310,27 @@ function projectPrPath(project) {
   return path.join(projectStateDir(project), 'pull-requests.json');
 }
 
+function comparablePath(filePath) {
+  const resolved = path.resolve(filePath);
+  try {
+    return fs.realpathSync.native(resolved);
+  } catch {
+    try {
+      return path.join(fs.realpathSync.native(path.dirname(resolved)), path.basename(resolved));
+    } catch {
+      return resolved;
+    }
+  }
+}
+
 function resolveProjectForPrPath(filePath, config = null) {
-  const resolvedPaths = new Set([path.resolve(filePath)]);
+  const resolvedPaths = new Set([comparablePath(filePath)]);
   if (filePath && !path.isAbsolute(filePath)) {
-    resolvedPaths.add(path.resolve(MINIONS_DIR, filePath));
+    resolvedPaths.add(comparablePath(path.resolve(MINIONS_DIR, filePath)));
   }
   const projects = getProjects(config);
   for (const project of projects) {
-    if (resolvedPaths.has(path.resolve(projectPrPath(project)))) return project;
+    if (resolvedPaths.has(comparablePath(projectPrPath(project)))) return project;
   }
   if (projects.length === 1) return projects[0];
   return null;

package/minions.js

@@ -470,6 +470,7 @@ async function initMinions({ skipScan = false, scanRoot, scanDepth } = {}) {
     if (config.engine[k] === undefined) config.engine[k] = v;
   }
   if (!config.claude) config.claude = {};
+  delete config.claude.permissionMode;
   for (const [k, v] of Object.entries(DEFAULT_CLAUDE)) {
     if (config.claude[k] === undefined) config.claude[k] = v;
   }

package/package.json

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