@yemi33/minions 0.1.2045 → 0.1.2047

package/README.md

@@ -11,8 +11,8 @@ Inspired by and initially scaffolded from [Brady Gaster's Squad](https://bradyga
 ## Prerequisites
 
 - **Node.js** 18+ (LTS recommended)
-- **Claude Code CLI** — install with `npm install -g @anthropic-ai/claude-code`
-- **Anthropic API key** or Claude Max subscription (agents spawn Claude Code sessions)
+- **A supported runtime CLI** — Minions defaults to GitHub Copilot CLI (`npm install -g @github/copilot`). Claude Code CLI (`npm install -g @anthropic-ai/claude-code`) is also supported; switch with `minions config set-cli claude` or per-agent `cli` overrides.
+- **Auth for your runtime** — GitHub Copilot subscription (Copilot CLI handles its own auth) or an Anthropic API key / Claude Max subscription
 - **Git** — agents create worktrees for all code changes
 
 > **Note:** you do **not** need to configure your CLI for "autopilot" / "bypass permissions" / "dangerous mode". Minions passes the right bypass flag per spawn (`--dangerously-skip-permissions` for Claude; `--autopilot --allow-all --no-ask-user` for Copilot), independent of your global CLI config. Run `minions doctor` to verify your installed CLI accepts those flags.

package/dashboard/js/fre.js

@@ -80,9 +80,10 @@ function renderFre(statusOrProjects) {
 
   // Resolve the currently-configured runtime CLI for the explainer copy.
   // /api/status surfaces this as autoMode.defaultCli (resolveAgentCli(null, engine)).
-  // Fall back to autoMode.ccCli (also defaultCli-derived when ccCli unset) then 'claude'.
+  // Fall back to autoMode.ccCli (also defaultCli-derived when ccCli unset) then 'copilot'
+  // (matches ENGINE_DEFAULTS.defaultCli — W-mpmwxkk40007c995).
   const auto = (status && status.autoMode) || {};
-  const runtimeCli = String(auto.defaultCli || auto.ccCli || 'claude');
+  const runtimeCli = String(auto.defaultCli || auto.ccCli || 'copilot');
 
   const cardStyle = [
     'margin:12px 24px',

package/dashboard/js/render-prs.js

@@ -22,6 +22,16 @@ function _countPrFollowups(pr) {
   return n;
 }
 
+// W-mpmwxkzm0009ba0b — Parse a canonical PR id like `github:octo/repo#123`
+// or `ado:org/proj/repo#456` into the {host, slug, number} triple used by
+// POST /api/pull-requests/observe. Returns null when the id doesn't match.
+function _parseCanonicalPrId(id) {
+  if (!id || typeof id !== 'string') return null;
+  var m = id.match(/^(github|ado):(.+)#(\d+)$/);
+  if (!m) return null;
+  return { host: m[1], slug: m[2], number: parseInt(m[3], 10) };
+}
+
 function prRow(pr) {
   // Minions review (agent) state — separate from ADO human review
   const sq = pr.minionsReview || {};
@@ -61,6 +71,29 @@ function prRow(pr) {
         ? '<span class="pr-agent" title="' + escapeHtml(pr.reviewedBy.join(', ')) + '">' + escapeHtml(pr.reviewedBy.join(', ')) + '</span>'
         : '<span style="color:var(--muted);font-size:11px">—</span>';
   const createdLabel = (pr.created || '—').slice(0, 16).replace('T', ' ');
+  // Per-row auto-observe toggle (W-mpmwxkzm0009ba0b). _contextOnly === true
+  // means the engine polls status/comments but does NOT dispatch review/fix
+  // agents against this PR. Default (undefined / false) = observed.
+  var observe = pr._contextOnly !== true;
+  var observeClass = observe ? 'pr-observe-on' : 'pr-observe-off';
+  var observeLabel = observe ? 'observed' : 'context';
+  var observeTitle = observe
+    ? 'Auto-observe ON — engine reviews this PR and dispatches fix agents on review/build feedback. Click to switch to context-only.'
+    : 'Context only — engine polls status and comments but will not dispatch review/fix agents. Click to enable auto-observe.';
+  // pr.id is canonical: "<host>:<slug>#<number>" (engine/shared.js getCanonicalPrId).
+  // Parse here so the toggle handler can post {host, slug, number} without
+  // shipping additional fields per row.
+  var observeBtn = '';
+  var idMatch = typeof pr.id === 'string' ? pr.id.match(/^([^:]+):(.+)#(\d+)$/) : null;
+  if (idMatch) {
+    observeBtn = '<button class="pr-observe-toggle ' + observeClass + '" '
+      + 'data-pr-host="' + escapeHtml(idMatch[1]) + '" '
+      + 'data-pr-slug="' + escapeHtml(idMatch[2]) + '" '
+      + 'data-pr-number="' + escapeHtml(idMatch[3]) + '" '
+      + 'data-pr-observe="' + (observe ? '1' : '0') + '" '
+      + 'title="' + escapeHtml(observeTitle) + '" '
+      + 'onclick="event.stopPropagation();togglePrObserve(this)">' + observeLabel + '</button> ';
+  }
   // Title attrs live on the inner element (link/span/badge) so hovering the
   // ellipsis-truncated content reveals the full text. Cell tags stay bare so
   // the header-to-cell count assertion in test/unit.test.js continues to
@@ -74,6 +107,7 @@ function prRow(pr) {
     '<td>' + reviewerCell + '</td>' +
     '<td><span class="pr-badge ' + buildClass + '" title="' + escapeHtml(buildTitle || buildLabel) + '">' + escapeHtml(buildLabel) + '</span></td>' +
     '<td><span class="pr-badge ' + statusClass + '" title="' + escapeHtml(statusLabel) + '">' + escapeHtml(statusLabel) + '</span></td>' +
+    '<td>' + (observeBtn || '<span style="color:var(--muted);font-size:11px">—</span>') + '</td>' +
     '<td><span class="pr-date" title="' + escapeHtml(createdLabel) + '">' + escapeHtml(createdLabel) + '</span></td>' +
     '<td><button class="pr-pager-btn" style="font-size:9px;padding:1px 5px;color:var(--red);border-color:var(--red)" data-pr-id="' + escapeHtml(String(prId)) + '" onclick="event.stopPropagation();unlinkPr(this.dataset.prId)" title="Remove from tracking">x</button></td>' +
     '</tr>';
@@ -93,6 +127,7 @@ const PRS_COLGROUP =
     '<col style="width:140px">' +   // Signed Off By
     '<col style="width:130px">' +   // Build
     '<col style="width:110px">' +   // Status
+    '<col style="width:100px">' +   // Observe
     '<col style="width:130px">' +   // Created
     '<col style="width:50px">' +    // Actions
   '</colgroup>';
@@ -101,7 +136,7 @@ function prTableHtml(rows) {
   return '<div class="pr-table-wrap pr-table-wrap--prs"><table class="pr-table pr-table--prs">' +
     PRS_COLGROUP +
     '<thead><tr>' +
-    '<th>PR</th><th>Title</th><th>Agent</th><th>Branch</th><th>Review</th><th>Signed Off By</th><th>Build</th><th>Status</th><th>Created</th><th></th>' +
+    '<th>PR</th><th>Title</th><th>Agent</th><th>Branch</th><th>Review</th><th>Signed Off By</th><th>Build</th><th>Status</th><th>Observe</th><th>Created</th><th></th>' +
     '</tr></thead><tbody>' + rows + '</tbody></table></div>';
 }
 
@@ -249,4 +284,49 @@ async function unlinkPr(id) {
   } catch (e) { clearDeleted('pr:' + id); showToast('pr-toast', 'Error: ' + e.message, false); refresh(); }
 }
 
-window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, unlinkPr };
+// Per-row auto-observe toggle (W-mpmwxkzm0009ba0b). Optimistic UI: flip
+// the button immediately, only revert on API error. The engine consumes
+// `_contextOnly` to gate review/fix dispatch (engine/shared.js
+// isAutoManagedPrRecord + discoverFromPrs).
+async function togglePrObserve(btn) {
+  if (!btn || btn.disabled) return;
+  const host = btn.dataset.prHost;
+  const slug = btn.dataset.prSlug;
+  const number = parseInt(btn.dataset.prNumber, 10);
+  if (!host || !slug || !Number.isFinite(number)) return;
+  const wasObserve = btn.dataset.prObserve === '1';
+  const next = !wasObserve;
+  // Optimistic flip
+  btn.dataset.prObserve = next ? '1' : '0';
+  btn.classList.toggle('pr-observe-on', next);
+  btn.classList.toggle('pr-observe-off', !next);
+  btn.textContent = next ? 'observed' : 'context';
+  btn.disabled = true;
+  try {
+    const res = await fetch('/api/pull-requests/observe', {
+      method: 'POST', headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ host, slug, number, observe: next })
+    });
+    if (!res.ok) {
+      const d = await res.json().catch(() => ({}));
+      // Revert
+      btn.dataset.prObserve = wasObserve ? '1' : '0';
+      btn.classList.toggle('pr-observe-on', wasObserve);
+      btn.classList.toggle('pr-observe-off', !wasObserve);
+      btn.textContent = wasObserve ? 'observed' : 'context';
+      showToast('pr-toast', 'Failed: ' + (d.error || 'unknown'), false);
+      return;
+    }
+    showToast('pr-toast', next ? 'Auto-observe enabled' : 'Switched to context only', true);
+  } catch (e) {
+    btn.dataset.prObserve = wasObserve ? '1' : '0';
+    btn.classList.toggle('pr-observe-on', wasObserve);
+    btn.classList.toggle('pr-observe-off', !wasObserve);
+    btn.textContent = wasObserve ? 'observed' : 'context';
+    showToast('pr-toast', 'Error: ' + e.message, false);
+  } finally {
+    btn.disabled = false;
+  }
+}
+
+window.MinionsPrs = { prRow, prTableHtml, renderPrs, prPrev, prNext, openAllPrs, openModal, openAddPrModal, unlinkPr, togglePrObserve };

package/dashboard/js/settings.js

@@ -49,7 +49,7 @@ async function openSettings() {
   // Per-agent override placeholders surface the inherited fleet defaults as
   // muted text — operators see exactly what each agent will resolve to without
   // chasing config files. Empty input clears the override → re-inherit fleet.
-  const fleetCliLabel = e.defaultCli || 'claude';
+  const fleetCliLabel = e.defaultCli || 'copilot';
   const fleetModelLabel = e.defaultModel ? String(e.defaultModel) : 'CLI default';
   const agentRows = Object.entries(agents).map(function([id, a]) {
     return '<tr>' +
@@ -406,10 +406,10 @@ async function initRuntimeFleetUI(engineCfg, agentsCfg) {
     runtimes = Array.isArray(d.runtimes) ? d.runtimes : [];
   } catch { /* ignore — we'll surface a free-text-only path below */ }
 
-  // Always include 'claude' as a fallback option even if /api/runtimes is empty;
+  // Always include 'copilot' as a fallback option even if /api/runtimes is empty;
   // legacy installs without the registry endpoint should still see something pickable.
-  const names = runtimes.length ? runtimes.map(rt => rt.name) : ['claude'];
-  const currentDefault = engineCfg.defaultCli || 'claude';
+  const names = runtimes.length ? runtimes.map(rt => rt.name) : ['copilot'];
+  const currentDefault = engineCfg.defaultCli || 'copilot';
   const currentCc = engineCfg.ccCli || '';
   cliSelect.innerHTML = names.map(n =>
     '<option value="' + escHtml(n) + '"' + (n === currentDefault ? ' selected' : '') + '>' + escHtml(n) + '</option>'
@@ -440,7 +440,7 @@ async function initRuntimeFleetUI(engineCfg, agentsCfg) {
   // this the input was free-text and a user could (and did) save an agent
   // with cli=claude + model=<some gpt> — invalid combination that crashed
   // dispatch. Refreshing on CLI change clears stale model values.
-  const fleetDefaultCli = engineCfg.defaultCli || 'claude';
+  const fleetDefaultCli = engineCfg.defaultCli || 'copilot';
   for (const cell of cliCells) {
     const agentId = cell.getAttribute('data-runtime-cli');
     const agent = (agentsCfg || {})[agentId] || {};

package/dashboard/styles.css

@@ -322,6 +322,17 @@
   .pr-page-info { font-size: var(--text-base); color: var(--muted); }
   .pr-date { font-size: var(--text-base); color: var(--muted); }
 
+  .pr-observe-toggle {
+    font-size: 9px; padding: 1px 6px; border-radius: var(--radius-sm); cursor: pointer;
+    text-transform: uppercase; letter-spacing: 0.5px; font-weight: 600;
+    background: var(--surface2); transition: all var(--transition-base);
+  }
+  .pr-observe-toggle:disabled { opacity: 0.6; cursor: wait; }
+  .pr-observe-toggle.pr-observe-on { color: var(--green); border: 1px solid var(--green); background: rgba(63,185,80,0.12); }
+  .pr-observe-toggle.pr-observe-on:hover:not(:disabled) { background: rgba(63,185,80,0.22); }
+  .pr-observe-toggle.pr-observe-off { color: var(--muted); border: 1px dashed var(--border); }
+  .pr-observe-toggle.pr-observe-off:hover:not(:disabled) { color: var(--text); border-color: var(--text); }
+
   .archive-btn {
     background: var(--surface2); border: 1px solid var(--border); color: var(--muted);
     font-size: var(--text-base); padding: var(--space-2) var(--space-5); border-radius: var(--radius-sm); cursor: pointer; transition: all var(--transition-base); margin-left: auto;