npm - @yemi33/minions - Versions diffs - 0.1.2304 → 0.1.2305 - Mend

@yemi33/minions 0.1.2304 → 0.1.2305

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (9) hide show

package/dashboard/js/refresh.js +1 -1
package/dashboard/js/render-work-items.js +2 -1
package/dashboard/layout.html +13 -0
package/dashboard/slim/js/modals-tiles.js +21 -130
package/dashboard/slim/styles.css +9 -14
package/dashboard/styles.css +21 -0
package/dashboard.js +9 -0
package/engine/live-checkout.js +163 -73
package/package.json +1 -1

package/dashboard/js/refresh.js CHANGED Viewed

@@ -147,7 +147,7 @@ const RENDER_VERSIONS = {
   dispatch: 2,
   engineLog: 2,
   metrics: 1,
-  workItems: 5,
+  workItems: 6,
   skills: 1,
   commands: 1,
   mcpServers: 1,

package/dashboard/js/render-work-items.js CHANGED Viewed

@@ -42,6 +42,7 @@ const _WI_ENRICHMENT_FIELDS = [
 //   * already_dispatched — engine reconciles on next tick
 //   * branch_locked — wait for the holding dispatch
 //   * dependency_unmet — auto-clears when upstream dep finishes (engine discovery loop + DEPENDENCY_MET watch)
+//   * live_checkout_busy — one live-mode dispatch at a time; clears when running dispatch finishes
 //
 // Returns { kind, short, full } when the item needs attention, else null.
 // Pure helper (no DOM / escapeHtml deps) so it can be unit-tested in Node.
@@ -219,7 +220,7 @@ function wiRow(item) {
             + escapeHtml(((item._managedSpawnPartial.healthy || []).length) + '/' + (((item._managedSpawnPartial.healthy || []).length) + item._managedSpawnPartial.failed.length))
             + ' healthy</span>'
         : '') +
-      (item._skipReason && item.status === 'pending' ? ' <span style="font-size:var(--text-xs);color:var(--yellow);margin-left:4px" title="Dispatch blocked: ' + escapeHtml(item._skipReason) + (item._blockedBy ? ' (by ' + escapeHtml(item._blockedBy) + ')' : '') + '">' + escapeHtml(item._skipReason.replace(/_/g, ' ')) + (item._blockedBy ? ' <span style="color:var(--muted)">(' + escapeHtml(item._blockedBy) + ')</span>' : '') + '</span>' : '') +
+      (item._skipReason && item.status === 'pending' && item._skipReason !== 'live_checkout_busy' ? ' <span style="font-size:var(--text-xs);color:var(--yellow);margin-left:4px" title="Dispatch blocked: ' + escapeHtml(item._skipReason) + (item._blockedBy ? ' (by ' + escapeHtml(item._blockedBy) + ')' : '') + '">' + escapeHtml(item._skipReason.replace(/_/g, ' ')) + (item._blockedBy ? ' <span style="color:var(--muted)">(' + escapeHtml(item._blockedBy) + ')</span>' : '') + '</span>' : '') +
       (item.status === 'failed' ? ' ' + wiRetryBtn(item) : '') +
     '</td>' +
     '<td>' +

package/dashboard/layout.html CHANGED Viewed

@@ -17,6 +17,19 @@
       document.documentElement.setAttribute('data-font-size', (v && valid[v]) ? v : 'small');
     } catch (e) { /* private mode / disabled storage — fall through to default */ }
   })();
+  // W-mqrdggys000f94a4 — chrome-off "embed" mode. When the SPA is loaded with
+  // ?embed=1 (e.g. inside the slim cockpit's Queued-work iframe pointed at
+  // /work?embed=1), tag <html> so the embed CSS hides the global chrome
+  // (header, sidebar, projects bar, banners) and the iframed page renders as
+  // just its panel. Applied in <head> so the first paint already omits the
+  // chrome (no flash of the full layout).
+  (function() {
+    try {
+      if (/[?&]embed=1(?:&|$)/.test(window.location.search)) {
+        document.documentElement.classList.add('embed');
+      }
+    } catch (e) { /* defensive — never block boot on a URL parse quirk */ }
+  })();
 </script>
 <style>/* __CSS__ */</style>
 </head>

package/dashboard/slim/js/modals-tiles.js CHANGED Viewed

@@ -123,135 +123,23 @@
     });
   }
-  // Collapsible "Completed work" section appended to the queued popout (only).
-  // Collapsed by DEFAULT; clicking (or Enter/Space on) the header toggles the
-  // list. Sources data.dispatch.completed — the same recent window the slim
-  // History feed reads (capped to the last 20 by the /api/status snapshot) —
-  // so it adds no new API calls. Status semantics reuse completionStatus /
-  // completionTitle for parity with the History feed.
-  function renderCompletedSection(body, data) {
-    var dispatch = data.dispatch || {};
-    // Snapshot delivers completed oldest→newest; reverse for most-recent-first.
-    var completed = Array.isArray(dispatch.completed) ? dispatch.completed.slice().reverse() : [];
-    var section = document.createElement('div');
-    section.className = 'tile-completed-section collapsed';
-    var head = document.createElement('div');
-    head.className = 'tile-completed-section-head';
-    head.setAttribute('role', 'button');
-    head.setAttribute('tabindex', '0');
-    head.setAttribute('aria-expanded', 'false');
-    var caret = document.createElement('span');
-    caret.className = 'tile-caret';
-    caret.textContent = '▸';
-    var label = document.createElement('span');
-    label.className = 'tile-completed-section-title';
-    label.textContent = 'Completed work (' + completed.length + ')';
-    head.appendChild(caret);
-    head.appendChild(label);
-    var listWrap = document.createElement('div');
-    listWrap.className = 'tile-completed-section-body';
-    listWrap.hidden = true;
-    if (!completed.length) {
-      tileEmpty(listWrap, 'No completed work yet.');
-    } else {
-      completed.forEach(function(c) {
-        var who = c.agentName || c.agent || c.dispatched_to || '—';
-        var when = c.completed_at || c.completedAt || c.endedAt || c.started_at || c.startedAt;
-        var status = completionStatus(c);
-        var chip = status === 'ok' ? { text: 'done', cls: 'green' }
-                 : status === 'warn' ? { text: 'partial', cls: 'amber' }
-                 : { text: 'failed', cls: 'red' };
-        listWrap.appendChild(tileCard({
-          title: completionTitle(c),
-          meta: who + (when ? ' · completed ' + relTime(when) : ''),
-          chip: chip,
-        }));
-      });
-    }
-    function toggle() {
-      var open = !section.classList.toggle('collapsed');
-      head.setAttribute('aria-expanded', open ? 'true' : 'false');
-      caret.textContent = open ? '▾' : '▸';
-      listWrap.hidden = !open;
-    }
-    head.addEventListener('click', toggle);
-    head.addEventListener('keydown', function(ev) {
-      if (ev.key === 'Enter' || ev.key === ' ') { ev.preventDefault(); toggle(); }
-    });
-    section.appendChild(head);
-    section.appendChild(listWrap);
-    body.appendChild(section);
-  }
-  // Collapsible "Pending work items" section appended to the queued popout
-  // (W-mqv1xbcm). Sources data.workItems filtered to status === 'pending' — the
-  // same /api/work-items slice the slim poll merges onto the snapshot — so it
-  // adds no new API calls and refreshes whenever the popout is re-opened.
-  // Expanded by DEFAULT (more prominent than the collapsed Completed section)
-  // since these are the queued WIs the operator most wants to scan. Mirrors
-  // renderCompletedSection's structure + tileCard usage; all text via
-  // textContent (no innerHTML) to keep the SEC-03 baseline clean. Each card
-  // shows title, id, project (project || _source), type, and the engine's
-  // _pendingReason as a chip when set.
-  function renderPendingSection(body, data) {
-    var pending = (Array.isArray(data.workItems) ? data.workItems : []).filter(function(w) {
-      return w && w.status === 'pending';
-    });
-    var section = document.createElement('div');
-    section.className = 'tile-completed-section';
-    var head = document.createElement('div');
-    head.className = 'tile-completed-section-head';
-    head.setAttribute('role', 'button');
-    head.setAttribute('tabindex', '0');
-    head.setAttribute('aria-expanded', 'true');
-    var caret = document.createElement('span');
-    caret.className = 'tile-caret';
-    caret.textContent = '▾';
-    var label = document.createElement('span');
-    label.className = 'tile-completed-section-title';
-    label.textContent = 'Pending work items (' + pending.length + ')';
-    head.appendChild(caret);
-    head.appendChild(label);
-    var listWrap = document.createElement('div');
-    listWrap.className = 'tile-completed-section-body';
-    if (!pending.length) {
-      tileEmpty(listWrap, 'No pending work items.');
-    } else {
-      pending.forEach(function(w) {
-        var project = w.project || w._source;
-        var meta = [w.id || null, project || null, w.type || null].filter(Boolean).join(' · ');
-        var card = { title: w.title || w.id || '(untitled)', meta: meta };
-        if (w._pendingReason) {
-          card.chip = { text: String(w._pendingReason).replace(/_/g, ' '), cls: 'amber' };
-        }
-        listWrap.appendChild(tileCard(card));
-      });
-    }
-    function toggle() {
-      var open = !section.classList.toggle('collapsed');
-      head.setAttribute('aria-expanded', open ? 'true' : 'false');
-      caret.textContent = open ? '▾' : '▸';
-      listWrap.hidden = !open;
-    }
-    head.addEventListener('click', toggle);
-    head.addEventListener('keydown', function(ev) {
-      if (ev.key === 'Enter' || ev.key === ' ') { ev.preventDefault(); toggle(); }
-    });
-    section.appendChild(head);
-    section.appendChild(listWrap);
-    body.appendChild(section);
+  // Queued-work tile body — reuses the LITERAL classic dashboard Work Items
+  // screen instead of a slim-specific list, so there is one work-items UI, not
+  // two (W-mqrdggys000f94a4 / "reuse, don't fork"). Slim and classic are two
+  // separate IIFE bundles with no shared scope, so we can't call the classic
+  // renderWorkItems() in-process; instead we embed the real /work screen in an
+  // iframe with the chrome-off ?embed=1 mode. The iframed page IS the classic
+  // screen — full filter/sort + every row action (edit/cancel/retry/archive/
+  // delete/feedback), backed by the same /api/work-items data + endpoints —
+  // with zero duplicated rendering logic. Classic is reachable at /work even
+  // with slim-ux ON (only / is taken over). Built with createElement (no
+  // innerHTML) to satisfy the dashboard no-unsanitized lint gate.
+  function renderQueuedWorkBody(body) {
+    var frame = document.createElement('iframe');
+    frame.className = 'slim-work-embed';
+    frame.src = '/work?embed=1';
+    frame.title = 'Work Items';
+    body.appendChild(frame);
   }
   function renderPrTileBody(body, data) {
@@ -375,7 +263,7 @@
   var TILE_VIEWS = {
     engine:     { title: 'Engine',            render: renderEngineTileBody },
     dispatches: { title: 'Active dispatches', render: function(body, data) { renderDispatchTileBody(body, data, true); } },
-    queued:     { title: 'Queued work',       render: function(body, data) { renderDispatchTileBody(body, data, false); renderPendingSection(body, data); renderCompletedSection(body, data); } },
+    queued:     { title: 'Queued work',       render: function(body) { renderQueuedWorkBody(body); } },
     prs:        { title: 'Pull requests',     render: renderPrTileBody },
     watches:    { title: 'Watches',           render: renderWatchTileBody },
   };
@@ -400,6 +288,9 @@
     // The "+ Link PR" header chip only applies to the PR list.
     var headerChip = document.getElementById('slim-tile-modal-linkpr');
     if (headerChip) headerChip.style.display = (key === 'prs') ? '' : 'none';
+    // The queued tile embeds the full classic /work screen in an iframe — widen
+    // the modal + drop the body padding so the embedded table gets real estate.
+    modal.classList.toggle('tile-modal--work', key === 'queued');
     view.render(body, lastStatusData || {});
     modal.classList.add('open');
   }

package/dashboard/slim/styles.css CHANGED Viewed

@@ -680,6 +680,15 @@
     /* Cockpit-tile detail modal: list of dispatches / PRs / watches, or a
        key/value engine readout (reuses .agent-detail-row). */
     #slim-tile-modal .modal { width: 720px; max-width: calc(100vw - 32px); }
+    /* W-mqrdggys000f94a4 — the Queued-work tile embeds the full classic /work
+       screen in an iframe; widen the modal + remove body padding so the
+       embedded work-items table gets the full width/height. */
+    #slim-tile-modal.tile-modal--work .modal { width: 1180px; }
+    #slim-tile-modal.tile-modal--work .modal-body { padding: 0; }
+    .slim-work-embed {
+      display: block; width: 100%; height: calc(100vh - 160px); min-height: 360px;
+      border: 0; background: var(--bg);
+    }
     .tile-item {
       display: block;
       text-decoration: none;
@@ -712,20 +721,6 @@
     .tile-chip.blue  { background: rgba(88, 166, 255, 0.12); color: var(--blue);  border-color: var(--blue); }
     .tile-empty { color: var(--muted); font-style: italic; font-size: var(--text-md); }
-    /* Collapsible "Completed work" section in the queued popout. Collapsed by
-       default; the caret rotates and the body unhides on toggle. */
-    .tile-completed-section { margin-top: 12px; border-top: 1px solid var(--border); padding-top: 10px; }
-    .tile-completed-section-head {
-      display: flex; align-items: center; gap: 6px;
-      cursor: pointer; user-select: none;
-      font-size: var(--text-md); font-weight: 600; color: var(--text);
-      padding: 2px 0;
-    }
-    .tile-completed-section-head:hover { color: var(--blue); }
-    .tile-completed-section-head:focus-visible { outline: 1px solid var(--blue); outline-offset: 2px; border-radius: var(--radius); }
-    .tile-caret { font-size: var(--text-sm); color: var(--muted); width: 12px; display: inline-block; text-align: center; }
-    .tile-completed-section-body { margin-top: 8px; }
     /* Collapsible PR sections (Active / Merged-Completed / Abandoned). Uses
        native <details>/<summary> — no JS wiring needed for the toggle. */
     .tile-section { margin-bottom: 10px; }

package/dashboard/styles.css CHANGED Viewed

@@ -1547,3 +1547,24 @@
   .cc-error strong { color: var(--red); }
   .cc-error code { font-family: monospace; font-size: var(--text-xs);
     color: var(--muted); }
+  /* W-mqrdggys000f94a4 — chrome-off "embed" mode (?embed=1). Strips the global
+     dashboard chrome so a single page (e.g. /work?embed=1) can be iframed inside
+     the slim cockpit's Queued-work modal and render as just its panel. The
+     html.embed class is applied in layout.html's head before first paint. */
+  html.embed body > header,
+  html.embed #projects-bar,
+  html.embed #setup-banner,
+  html.embed #fre-banner,
+  html.embed .engine-alert,
+  html.embed .paused-banner,
+  html.embed .sidebar,
+  html.embed #cc-drawer,
+  html.embed #cc-overlay { display: none !important; }
+  /* Do NOT override .page-layout to display:block here — that detaches
+     .page-content from its bounded-height flex parent, so it grows to full
+     content height and the fixed-height iframe clips it with no scrollbar (the
+     Queued-work popup "doesn't scroll"). The sidebar is hidden above, so the
+     base .page-layout flex row already gives .page-content full width plus a
+     bounded height to scroll within. */
+  html.embed .page-content { padding: 12px 16px; overflow-y: auto; }

package/dashboard.js CHANGED Viewed

@@ -14463,6 +14463,15 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     "img-src 'self' data:; " +
     "connect-src 'self'"
   );
+  // The Slim UX "Queued work" tile (W-mqrdggys000f94a4) embeds this classic SPA
+  // at /work?embed=1 in a SAME-ORIGIN iframe. buildSecurityHeaders() stamped
+  // X-Frame-Options: DENY on every response above, which makes the browser
+  // refuse to render the iframe ("localhost refused to connect"). Relax to
+  // SAMEORIGIN for the chrome-off embed request only — the standalone classic
+  // dashboard keeps DENY. Mirrors serveSlimUx's embeddable relaxation for /slim.
+  if (/[?&]embed=1(?:&|$)/.test(req.url)) {
+    res.setHeader('X-Frame-Options', 'SAMEORIGIN');
+  }
   if (req.headers['if-none-match'] === HTML_ETAG) {
     res.statusCode = 304;
     res.end();

package/engine/live-checkout.js CHANGED Viewed

@@ -132,6 +132,62 @@ function _isPartialCloneBlobError(message = '') {
   );
 }
+// W-mqva907q — auto-clean safe untracked build artifacts before a live-checkout
+// dirty bail. A `git status --porcelain` line for a stray `__pycache__/` (or
+// other regenerable cache) should NOT block dispatch the way a real edit does.
+// These patterns describe untracked-only paths that are always safe to delete
+// because the build/test toolchain regenerates them. Matching is intentionally
+// implemented with plain string checks (no glob dep — Minions ships zero runtime
+// deps). dist/ and build/ are deliberately EXCLUDED: they can hold real,
+// non-regenerable output, so auto-deleting them would be unsafe.
+const LIVE_CHECKOUT_AUTO_CLEAN_PATTERNS = Object.freeze([
+  '**/__pycache__/',
+  '**/*.pyc',
+  '**/*.pyo',
+  '**/.pytest_cache/',
+  '**/node_modules/.cache/',
+  '**/*.egg-info/',
+  '.tox/',
+]);
+// Returns true when a single untracked path (relative, git-porcelain form) is a
+// known regenerable build artifact safe to delete. Normalizes separators and a
+// trailing slash, and conservatively unquotes git's C-style quoting for paths
+// with special chars. Directory artifacts match on their final path segment so
+// nested copies (e.g. a/b/__pycache__/) are covered like the `**/` globs above.
+function _isAutoCleanableArtifact(rawPath) {
+  if (!rawPath || typeof rawPath !== 'string') return false;
+  let p = rawPath.trim();
+  if (p.length >= 2 && p.startsWith('"') && p.endsWith('"')) {
+    try { p = JSON.parse(p); } catch { /* keep raw on parse failure */ }
+  }
+  p = p.replace(/\\/g, '/').replace(/\/+$/, '');
+  if (!p) return false;
+  if (p.endsWith('.pyc') || p.endsWith('.pyo')) return true;
+  if (p.endsWith('.egg-info')) return true;
+  if (p === 'node_modules/.cache' || p.endsWith('/node_modules/.cache')) return true;
+  const seg = p.split('/').pop();
+  if (seg === '__pycache__' || seg === '.pytest_cache' || seg === '.tox') return true;
+  return false;
+}
+// Classifies the porcelain dirty lines for auto-clean. Returns the list of
+// untracked artifact paths ONLY when EVERY dirty entry is an untracked (`?? `)
+// line matching a safe artifact pattern; returns null otherwise (a tracked
+// modification, a staged/deleted change, or any untracked path that is NOT a
+// known artifact). Null means "do not auto-clean — fail dirty as before", which
+// enforces the all-or-nothing contract: partial dirt is never silently cleaned.
+function _collectAutoCleanablePaths(dirtyFiles) {
+  const cleanable = [];
+  for (const line of dirtyFiles) {
+    if (!line.startsWith('?? ')) return null;
+    const p = line.slice(3);
+    if (!_isAutoCleanableArtifact(p)) return null;
+    cleanable.push(p);
+  }
+  return cleanable.length > 0 ? cleanable : null;
+}
 // W-mqvejug6000eeb20 — production default for resolving the live-checkout
 // auto-reset decision when the caller (engine.js spawnAgent) does not pass an
 // explicit `autoReset` boolean. engine.js is intentionally not threading the
@@ -178,6 +234,7 @@ async function prepareLiveCheckout(opts = {}) {
                 // defers to `_resolveAutoReset`.
     _git,       // private injection for testing — defaults to shared.shellSafeGit
     _exists,    // private injection for testing — defaults to fs.existsSync
+    _rm,        // private injection for testing — defaults to fs.rmSync (recursive, force)
     _resolveAutoReset, // private injection for testing — defaults to config-based resolver
     _writeInboxNote,   // private injection for testing — defaults to dispatch.writeInboxAlert
   } = opts;
@@ -206,6 +263,8 @@ async function prepareLiveCheckout(opts = {}) {
   const git = (typeof _git === 'function') ? _git : shared.shellSafeGit;
   const exists = (typeof _exists === 'function') ? _exists : fs.existsSync;
+  const rm = (typeof _rm === 'function') ? _rm : ((absPath) => fs.rmSync(absPath, { recursive: true, force: true }));
+  const logFn = (typeof log === 'function') ? log : () => {};
   // W-mqvejug6000eeb20 — default maxBuffer (50 MB) so a large `git status
   // --porcelain` never overflows execFile's 1 MB ceiling. A caller-supplied
   // gitOpts.maxBuffer still wins (spread after the default).
@@ -230,91 +289,114 @@ async function prepareLiveCheckout(opts = {}) {
   // before the dirtyFiles check so callers receive it as `branchInfo`.
   if (!skipDirtyCheck) {
     const statusRaw = await git(['status', '--porcelain=v1', '-b'], baseOpts);
-    const statusStr = typeof statusRaw === 'string' ? statusRaw : '';
-    const statusLines = statusStr
-      .split(/\r?\n/)
-      .map((line) => line.replace(/\s+$/, ''))
-      .filter((line) => line.length > 0);
-    const branchInfo = statusLines.find((line) => line.startsWith('## ')) || '';
-    const dirtyFiles = statusLines.filter((line) => !line.startsWith('## '));
+    const parsePorcelain = (raw) => {
+      const lines = (typeof raw === 'string' ? raw : '')
+        .split(/\r?\n/)
+        .map((line) => line.replace(/\s+$/, ''))
+        .filter((line) => line.length > 0);
+      return {
+        branchInfo: lines.find((line) => line.startsWith('## ')) || '',
+        dirtyFiles: lines.filter((line) => !line.startsWith('## ')),
+      };
+    };
+    let { branchInfo, dirtyFiles } = parsePorcelain(statusRaw);
     if (dirtyFiles.length > 0) {
-      // W-mqvejug6000eeb20 — opt-in auto-reset. Default behavior is to bail with
-      // reason:'dirty' (spawnAgent translates to non-retryable LIVE_CHECKOUT_DIRTY
-      // and alerts the operator). When auto-reset is enabled — either the caller
-      // passed an explicit `autoReset` boolean, or the config-based resolver says
-      // so — we DISCARD the dirty state via `git fetch origin` + `git reset --hard
-      // origin/<branch>` and continue. This is destructive (the operator's
-      // uncommitted work is gone), which is why it is strictly opt-in.
-      let wantAutoReset = false;
-      if (typeof autoReset === 'boolean') {
-        wantAutoReset = autoReset;
-      } else {
-        const resolver = (typeof _resolveAutoReset === 'function') ? _resolveAutoReset : _defaultResolveAutoReset;
-        try { wantAutoReset = !!resolver(localPath); } catch { wantAutoReset = false; }
-      }
-      if (!wantAutoReset) {
-        return { ok: false, reason: 'dirty', dirtyFiles, branchInfo };
+      // W-mqva907q — auto-clean safe untracked build artifacts BEFORE bailing. This
+      // runs unconditionally (no config flag — deleting regenerable caches is always
+      // safe) and BEFORE the spawnAgent auto-stash path. Only fires when EVERY dirty
+      // entry is an untracked artifact (all-or-nothing); a single tracked edit or
+      // non-artifact untracked file means zero cleaning and the original dirty bail.
+      const cleanablePaths = _collectAutoCleanablePaths(dirtyFiles);
+      if (cleanablePaths) {
+        for (const rel of cleanablePaths) {
+          const abs = path.isAbsolute(rel) ? rel : path.join(localPath, rel);
+          rm(abs);
+        }
+        logFn(`[live-checkout] auto-cleaned ${cleanablePaths.length} artifact path(s) before dirty check: ${cleanablePaths.join(', ')}`, 'debug');
+        // Re-run the dirty check. If cleaning fully resolved the tree we fall
+        // through and proceed; if anything remains (e.g. a race re-created a file)
+        // we still bail dirty with the post-clean snapshot.
+        const reRaw = await git(['status', '--porcelain=v1', '-b'], baseOpts);
+        ({ branchInfo, dirtyFiles } = parsePorcelain(reRaw));
       }
+      if (dirtyFiles.length > 0) {
+        // W-mqvejug6000eeb20 — opt-in auto-reset. Default behavior is to bail with
+        // reason:'dirty' (spawnAgent translates to non-retryable LIVE_CHECKOUT_DIRTY
+        // and alerts the operator). When auto-reset is enabled — either the caller
+        // passed an explicit `autoReset` boolean, or the config-based resolver says
+        // so — we DISCARD the dirty state via `git fetch origin` + `git reset --hard
+        // origin/<branch>` and continue. This is destructive (the operator’s
+        // uncommitted work is gone), which is why it is strictly opt-in.
+        let wantAutoReset = false;
+        if (typeof autoReset === 'boolean') {
+          wantAutoReset = autoReset;
+        } else {
+          const resolver = (typeof _resolveAutoReset === 'function') ? _resolveAutoReset : _defaultResolveAutoReset;
+          try { wantAutoReset = !!resolver(localPath); } catch { wantAutoReset = false; }
+        }
+        if (!wantAutoReset) {
+          return { ok: false, reason: 'dirty', dirtyFiles, branchInfo };
+        }
-      if (typeof log === 'function') {
-        log(`live-checkout auto-reset: discarding ${dirtyFiles.length} dirty path(s) on '${branchName}' via fetch + reset --hard origin/${branchName}`);
-      }
-      let resetOk = true;
-      try {
-        await git(['fetch', 'origin'], baseOpts);
-        await git(['reset', '--hard', `origin/${branchName}`], baseOpts);
-      } catch (e) {
-        resetOk = false;
         if (typeof log === 'function') {
-          log(`live-checkout auto-reset FAILED (fetch/reset): ${e && e.message ? e.message : e}`);
+          log(`live-checkout auto-reset: discarding ${dirtyFiles.length} dirty path(s) on '${branchName}' via fetch + reset --hard origin/${branchName}`);
         }
-      }
-      // Re-run the porcelain preflight once. If the tree is now clean we proceed;
-      // otherwise (reset failed, or something is still dirty) we fall back to the
-      // safe dirty refusal so we never dispatch onto an unexpected tree.
-      let stillDirty = dirtyFiles;
-      if (resetOk) {
+        let resetOk = true;
         try {
-          const recheckRaw = await git(['status', '--porcelain=v1', '-b'], baseOpts);
-          const recheckStr = typeof recheckRaw === 'string' ? recheckRaw : '';
-          stillDirty = recheckStr
-            .split(/\r?\n/)
-            .map((line) => line.replace(/\s+$/, ''))
-            .filter((line) => line.length > 0 && !line.startsWith('## '));
+          await git(['fetch', 'origin'], baseOpts);
+          await git(['reset', '--hard', `origin/${branchName}`], baseOpts);
         } catch (e) {
           resetOk = false;
           if (typeof log === 'function') {
-            log(`live-checkout auto-reset re-check FAILED: ${e && e.message ? e.message : e}`);
+            log(`live-checkout auto-reset FAILED (fetch/reset): ${e && e.message ? e.message : e}`);
           }
         }
-      }
-      if (!resetOk || stillDirty.length > 0) {
-        return { ok: false, reason: 'dirty', dirtyFiles: stillDirty, branchInfo };
-      }
+        // Re-run the porcelain preflight once. If the tree is now clean we proceed;
+        // otherwise (reset failed, or something is still dirty) we fall back to the
+        // safe dirty refusal so we never dispatch onto an unexpected tree.
+        let stillDirty = dirtyFiles;
+        if (resetOk) {
+          try {
+            const recheckRaw = await git(['status', '--porcelain=v1', '-b'], baseOpts);
+            const recheckStr = typeof recheckRaw === 'string' ? recheckRaw : '';
+            stillDirty = recheckStr
+              .split(/\r?\n/)
+              .map((line) => line.replace(/\s+$/, ''))
+              .filter((line) => line.length > 0 && !line.startsWith('## '));
+          } catch (e) {
+            resetOk = false;
+            if (typeof log === 'function') {
+              log(`live-checkout auto-reset re-check FAILED: ${e && e.message ? e.message : e}`);
+            }
+          }
+        }
-      // Audit trail: record the discarded paths so the operator can recover from
-      // reflog / understand why their tree changed. Best-effort, never throws.
-      try {
-        const writeNote = (typeof _writeInboxNote === 'function') ? _writeInboxNote : _defaultWriteInboxNote;
-        const slug = `live-checkout-autoreset-${wiId || dispatchId || 'unknown'}`;
-        const body = [
-          `# Live-checkout auto-reset on '${branchName}'`,
-          '',
-          `⚠️ The live-checkout tree at \`${localPath}\` was dirty at dispatch time and`,
-          '`liveCheckoutAutoReset` is enabled, so it was force-reset to',
-          `\`origin/${branchName}\`. **The following uncommitted changes were DISCARDED**`,
-          '(recover from `git reflog` / `git fsck --lost-found` if needed):',
-          '',
-          '```',
-          ...dirtyFiles,
-          '```',
-        ].join('\n');
-        writeNote(slug, body);
-      } catch { /* best-effort audit note */ }
-      // Fall through — tree is now clean, continue with normal preparation.
+        if (!resetOk || stillDirty.length > 0) {
+          return { ok: false, reason: 'dirty', dirtyFiles: stillDirty, branchInfo };
+        }
+        // Audit trail: record the discarded paths so the operator can recover from
+        // reflog / understand why their tree changed. Best-effort, never throws.
+        try {
+          const writeNote = (typeof _writeInboxNote === 'function') ? _writeInboxNote : _defaultWriteInboxNote;
+          const slug = `live-checkout-autoreset-${wiId || dispatchId || 'unknown'}`;
+          const body = [
+            `# Live-checkout auto-reset on '${branchName}'`,
+            '',
+            `⚠️ The live-checkout tree at \`${localPath}\` was dirty at dispatch time and`,
+            '`liveCheckoutAutoReset` is enabled, so it was force-reset to',
+            `\`origin/${branchName}\`. **The following uncommitted changes were DISCARDED**`,
+            '(recover from `git reflog` / `git fsck --lost-found` if needed):',
+            '',
+            '```',
+            ...dirtyFiles,
+            '```',
+          ].join('\n');
+          writeNote(slug, body);
+        } catch { /* best-effort audit note */ }
+        // Fall through — tree is now clean, continue with normal preparation.
+      }
     }
   }
@@ -817,4 +899,12 @@ async function maybeRestoreLiveCheckoutFromRecord(opts = {}) {
   }
 }
-module.exports = { prepareLiveCheckout, restoreLiveCheckoutAtDispatchEnd, maybeRestoreLiveCheckoutFromRecord, _isPartialCloneBlobError };
+module.exports = {
+  prepareLiveCheckout,
+  restoreLiveCheckoutAtDispatchEnd,
+  maybeRestoreLiveCheckoutFromRecord,
+  _isPartialCloneBlobError,
+  _isAutoCleanableArtifact,
+  _collectAutoCleanablePaths,
+  LIVE_CHECKOUT_AUTO_CLEAN_PATTERNS,
+};

package/package.json CHANGED Viewed

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