@yemi33/minions 0.1.2424 → 0.1.2425

package/bin/minions.js

@@ -11,6 +11,7 @@
  *   minions update [--no-wait]       Update to latest version (--no-wait backgrounds the post-update restart)
  *   minions version                  Show installed and package versions
  *   minions doctor [--harness]       Check prerequisites and runtime health (--harness: print runtime harness inventory)
+ *   minions state <subcmd>            Check, backup, or export SQLite state
  *   minions restart [--open]         Stop + start engine + dashboard (--open forces a new browser tab)
  *   minions start [--open]           Start engine + dashboard if not already running (no-op when up)
  *   minions stop                     Stop the engine
@@ -856,6 +857,15 @@ function getPkgVersionFresh() {
   try { return JSON.parse(fs.readFileSync(path.join(PKG_ROOT, 'package.json'), 'utf8')).version; } catch { return null; }
 }
 
+// Which npm distribution is this CLI? Derived from the installed package
+// manifest (public `@yemi33/minions` vs an internal rename produced by
+// tools/prepare-package.js) so `minions version` compares against the right
+// registry channel and `minions update` reinstalls ITSELF instead of pulling
+// in a second package that fights over the shared `minions` bin (npm EEXIST).
+function getPkgName() {
+  return shared.resolvePackageName(PKG_ROOT);
+}
+
 function getInstalledVersion() {
   try {
     const vFile = path.join(MINIONS_HOME, '.minions-version');
@@ -1201,7 +1211,8 @@ function runPostUpdateRestartDetached() {
 function showVersion() {
   const pkg = getPkgVersion();
   const installed = getInstalledVersion();
-  console.log(`\n  Package version:   ${pkg}`);
+  console.log(`\n  Package:           ${getPkgName()}`);
+  console.log(`  Package version:   ${pkg}`);
   console.log(`  Runtime root:      ${MINIONS_HOME}`);
   if (installed) {
     console.log(`  Installed version: ${installed}`);
@@ -1216,7 +1227,7 @@ function showVersion() {
 
   // Check npm registry for latest version (best-effort, non-blocking)
   try {
-    const latest = execSync('npm view @yemi33/minions version', { encoding: 'utf8', timeout: 5000, windowsHide: true }).trim();
+    const latest = execSync(shared.buildNpmViewVersionCommand(getPkgName()), { encoding: 'utf8', timeout: 5000, windowsHide: true }).trim();
     if (latest && latest !== pkg) {
       console.log(`\n  Latest on npm:     ${latest}`);
       console.log('  To update: minions update');
@@ -1269,6 +1280,14 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
     minions update [--no-wait]       Update to latest version (--no-wait backgrounds the restart)
     minions version                  Show installed vs package version
     minions doctor [--harness]       Check prerequisites and runtime health (--harness: print runtime harness inventory)
+    minions state check              Run SQLite integrity and invariant checks
+    minions state backup <file>      Checkpoint and create a consistent SQLite backup
+    minions state export <file>      Write a human-readable diagnostic JSON export
+    minions state import-legacy-json <dir> --confirm
+                                      Offline recovery import from a legacy home
+    minions memory backfill-review-learnings [--dry-run] [--limit <N>] [--project <p>]
+                                      Strict offline backfill of review-learning
+                                      lessons from legacy artifacts (no LLM inference)
     minions add <project-dir>        Link a project (interactive)
     minions remove <project-dir>     Unlink a project
     minions list                     List linked projects
@@ -1342,6 +1361,13 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
     console.log(`  Version synced to ${getPkgVersion()} (dev/symlink install — pull from git to update code)`);
   } else {
     const preVersion = getPkgVersionFresh();
+    // Resolve the ACTIVE distribution first: an internal build must update
+    // itself, not the public package. Installing `@yemi33/minions` on top of an
+    // `@opg-microsoft/minions` install adds a second package claiming the same
+    // `minions` bin, which npm rejects with EEXIST (W-ms1j8m54000g4a4a).
+    // Reinstalling the same package name is an in-place upgrade, so no bin
+    // collision is possible.
+    const pkgName = getPkgName();
     // Resolve the target version explicitly BEFORE installing, and pin the
     // install to that exact version instead of a floating `@latest` tag. This
     // avoids a race where `npm view` (used here and by `minions version`) and
@@ -1353,18 +1379,18 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
     // though `npm view pkg version` reports a newer one (W-mr9k4a5x00090184).
     let targetVersion;
     try {
-      targetVersion = execSync('npm view @yemi33/minions version', { encoding: 'utf8', timeout: 15000, windowsHide: true }).trim();
+      targetVersion = execSync(shared.buildNpmViewVersionCommand(pkgName), { encoding: 'utf8', timeout: 15000, windowsHide: true }).trim();
     } catch (e) {
-      console.error('  Failed to resolve the latest version from the npm registry:', e.message);
+      console.error(`  Failed to resolve the latest version of ${pkgName} from the npm registry:`, e.message);
       process.exit(1);
     }
     if (!targetVersion) {
-      console.error('  Failed to resolve the latest version from the npm registry (empty response).');
+      console.error(`  Failed to resolve the latest version of ${pkgName} from the npm registry (empty response).`);
       process.exit(1);
     }
 
     try {
-      execSync(`npm install -g @yemi33/minions@${targetVersion}`, { stdio: 'inherit', timeout: 120000 });
+      execSync(shared.buildNpmGlobalInstallCommand(pkgName, targetVersion), { stdio: 'inherit', timeout: 120000 });
     } catch (e) {
       console.error('  npm update failed:', e.message);
       process.exit(1);
@@ -1615,6 +1641,7 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
 
   console.log('\n  Factory reset complete. Run "minions init" to link projects and start fresh.\n');
 } else if (cmd === 'uninstall') {
+  const uninstallPkgName = getPkgName();
   if (!rest.includes('--confirm')) {
     console.log(`
   Uninstall Minions — removes EVERYTHING.
@@ -1623,7 +1650,7 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
     1. Kill all running engine, dashboard, and agent processes
     2. Delete the entire ${MINIONS_HOME} directory (all state, config, agents, knowledge)
     3. Remove extracted skills from ~/.claude/skills/ (minions-authored only)
-    4. Uninstall the npm package (@yemi33/minions)
+    4. Uninstall the npm package (${uninstallPkgName})
 
   This is irreversible. Your project repos are NOT affected.
 
@@ -1670,11 +1697,11 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
 
   // 4. Uninstall npm package
   console.log('  Uninstalling npm package...');
-  try { execSync('npm uninstall -g @yemi33/minions', { stdio: 'inherit', timeout: 60000 }); } catch {}
+  try { execSync(shared.buildNpmGlobalUninstallCommand(uninstallPkgName), { stdio: 'inherit', timeout: 60000 }); } catch {}
 
   console.log('\n  Minions uninstalled. Your project repos were not touched.');
   console.log('\n  To re-install later, run:');
-  console.log('    npm install -g @yemi33/minions && minions init\n');
+  console.log(`    npm install -g ${uninstallPkgName} && minions init\n`);
 } else if (cmd === 'state') {
   ensureInstalled();
   const operations = require(path.join(MINIONS_HOME, 'engine', 'state-operations'));
@@ -1707,6 +1734,45 @@ ${fs.existsSync(path.join(PKG_ROOT, '.git')) ? `
     console.error(`\n  State ${sub || 'operation'} failed: ${e.message}\n`);
     process.exit(1);
   }
+} else if (cmd === 'memory') {
+  ensureInstalled();
+  const sub = rest[0];
+  if (sub !== 'backfill-review-learnings') {
+    console.error('\n  Usage: minions memory backfill-review-learnings [--dry-run] [--limit <N>] [--project <name>]\n');
+    process.exit(2);
+  }
+  const dryRun = rest.includes('--dry-run');
+  const limitIdx = rest.indexOf('--limit');
+  const limit = limitIdx >= 0 ? Number(rest[limitIdx + 1]) : 100;
+  if (limitIdx >= 0 && (!Number.isFinite(limit) || limit <= 0)) {
+    console.error('\n  --limit requires a positive integer\n');
+    process.exit(2);
+  }
+  const projIdx = rest.indexOf('--project');
+  const project = projIdx >= 0 ? rest[projIdx + 1] : null;
+  // A real run mutates memory_records; refuse while the engine is live so we do
+  // not race live capture. Dry-run is read-only and always allowed.
+  if (!dryRun) {
+    const enginePid = readEnginePid(MINIONS_HOME);
+    if (enginePid) {
+      try {
+        process.kill(enginePid, 0);
+        console.error(`\n  Engine PID ${enginePid} is running; stop Minions before a real backfill (or use --dry-run)\n`);
+        process.exit(1);
+      } catch (e) {
+        if (!e.code) throw e;
+      }
+    }
+  }
+  try {
+    const { backfillReviewLearnings } = require(path.join(MINIONS_HOME, 'engine', 'review-learning-backfill'));
+    const result = backfillReviewLearnings({ dryRun, limit, project });
+    console.log(JSON.stringify(result, null, 2));
+    process.exit(0);
+  } catch (e) {
+    console.error(`\n  memory backfill-review-learnings failed: ${e.message}\n`);
+    process.exit(1);
+  }
 } else if (cmd === 'doctor') {
   ensureInstalled();
   const { doctor, runHarnessDoctor } = require(path.join(MINIONS_HOME, 'engine', 'preflight'));

package/dashboard/docs/typography.md

@@ -40,7 +40,7 @@ two stay in lockstep.
 
 | Token              | Value | Common use                                                |
 | ------------------ | ----- | --------------------------------------------------------- |
-| `--text-xs`        | 10px  | Smallest legitimate text; sidebar badges, count pills     |
+| `--text-xs`        | 10px  | Reserved chrome only — not for body/label/button text (see floor note) |
 | `--text-sm`        | 12px  | Status badges, chip / tag labels                          |
 | `--text-base`      | 12px  | Captions, secondary metadata, dense table cells           |
 | `--text-md`        | 13px  | Default for slim chat / detail tables / button labels     |
@@ -57,24 +57,39 @@ two stay in lockstep.
 > token that breaks the "all px" rule and is explicitly allowlisted in
 > the tripwire.
 
-### 12px readability floor (W-mr3xgejt0008d3a5)
+### 12px readability floor (W-mr3xgejt0008d3a5, W-mryzdch9000f069c)
 
 `--text-sm` was raised from an 11px base to **12px** so it clears the 12px
 readability floor in both stylesheets: classic renders it at `12 × 1.05 =
 12.6px`, slim at `12 × 1 = 12px`. `--text-xs` (10px base) stays below the
-floor and is reserved for the very smallest chrome. To lift the
-highest-signal small-text callsites off `--text-xs`, these selectors were
-reassigned to `--text-sm`:
-
-- `.sidebar-count` — primary nav count pill
-- `.dispatch-stuck` — uppercase red "stuck" status badge
-- `.pl-node-meta` — muted plan-node secondary text
-- `.error-details-btn` — uppercase error-details button label
+floor.
+
+An initial pass (W-mr3xgejt0008d3a5) reassigned four high-signal callsites
+(`.sidebar-count`, `.dispatch-stuck`, `.pl-node-meta`, `.error-details-btn`)
+from `--text-xs` to `--text-sm`. **W-mryzdch9000f069c then bumped the
+remaining ~190 `--text-xs` *usages* across `dashboard/**`** (CSS rules,
+inline `style="font-size:var(--text-xs)"`, and JS template strings) to
+`--text-sm`, so almost all dashboard text now renders at or above the floor.
+The `--text-xs` token declaration itself is unchanged (still
+`calc(10px * var(--minions-text-scale))` in both stylesheets) — this was a
+usage migration, not a value change.
+
+`--text-xs` is now retained at only these genuinely-dense chrome callsites,
+where 12px would clip or misalign a fixed-size component:
+
+- `.token-chart-labels span` — token-usage chart axis labels squeezed into
+  fixed `min-width:8px; max-width:24px` columns with `overflow:hidden`; 12px
+  would clip the labels.
+- `.qa-phase-arrow` — the decorative `→` separator glyph between QA phase
+  chips (a micro-glyph, not readable text).
+
+The `.text-xs` utility class (`.text-xs { font-size: var(--text-xs); }`)
+stays declared as the token's escape hatch but currently has no callsites.
 
 The `--text-sm` selectors already on the token (`.table th`, `.pr-table th`,
 `.token-agent-table th`, `.status-badge`, `.sidebar-link`, slim
-`.tile-item-subtle`) now clear the floor automatically via the token bump —
-no per-callsite change needed.
+`.tile-item-subtle`) clear the floor automatically via the token bump — no
+per-callsite change needed.
 
 ## Role aliases (preferred for new code)
 

package/dashboard/js/command-center.js

@@ -636,10 +636,12 @@ function _ccSetButtonProcessing(on) {
 }
 
 function _ccRenderSuggestedPrompts() {
-  var el = document.getElementById('cc-messages');
+  var host = document.getElementById('cc-suggestions-host');
   var tab = _ccActiveTab();
-  if (!el || !tab || tab.messages.length !== 0 || tab._sending || el.children.length !== 0) return;
-  renderCcSuggestedPrompts(el, function(prompt) {
+  if (!host) return;
+  clearCcSuggestedPrompts(host);
+  if (!tab || tab.messages.length !== 0 || tab._sending) return;
+  renderCcSuggestedPrompts(host, function(prompt) {
     var input = document.getElementById('cc-input');
     if (!input) return;
     input.value = prompt;
@@ -738,7 +740,7 @@ function ccSwitchTab(id) {
       for (var pi = phases.length - 1; pi >= 0; pi--) { if (ms >= phases[pi][0]) { label = phases[pi][1]; break; } }
       var secs = Math.floor(ms / 1000);
       html += '<div style="margin-top:' + (text ? '6px' : '0') + ';display:flex;align-items:center;gap:6px"><span style="color:var(--muted);font-size:var(--text-base)">' + label + '</span>' + dotPulse + '<span style="margin-left:auto;font-size:var(--text-sm);color:var(--muted)">' + secs + 's</span>' +
-        '<button onclick="ccAbort()" style="font-size:var(--text-xs);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer;margin-left:4px">Stop</button></div>';
+        '<button onclick="ccAbort()" style="font-size:var(--text-sm);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer;margin-left:4px">Stop</button></div>';
       return html;
     }
     ccAddMessage('assistant', _restoreStreamHtml(), true);
@@ -933,7 +935,7 @@ function ccRestoreMessages() {
       thinking.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:var(--text-base);color:var(--muted);align-self:flex-start;display:flex;align-items:center;gap:8px';
       // eslint-disable-next-line no-unsanitized/property -- reason: composed from elapsed timer and compile-time thinking UI strings (no user data flows in)
       thinking.innerHTML = '<span class="dot-pulse" style="display:inline-flex;gap:3px"><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.2s"></span><span style="width:4px;height:4px;background:var(--blue);border-radius:50%;animation:dotPulse 1.2s infinite;animation-delay:0.4s"></span></span> <span id="cc-thinking-text">Still working...</span> <span id="cc-thinking-time" style="font-size:var(--text-sm);color:var(--border)">' + Math.floor(elapsed / 1000) + 's</span>' +
-        ' <button onclick="ccNewTab()" style="font-size:var(--text-xs);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer">Reset</button>';
+        ' <button onclick="ccNewTab()" style="font-size:var(--text-sm);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer">Reset</button>';
       el.appendChild(thinking);
       el.scrollTop = el.scrollHeight;
       // Update timer
@@ -1012,7 +1014,7 @@ function ccAddMessage(role, html, skipSave, targetTabId, meta) {
   var isVisible = !targetTabId || targetTabId === _ccActiveTabId;
   if (isVisible) {
     var el = document.getElementById('cc-messages');
-    clearCcSuggestedPrompts(el);
+    clearCcSuggestedPrompts(document.getElementById('cc-suggestions-host'));
     var div = document.createElement('div');
     div.className = isAssistant ? 'cc-msg-assistant' : isAction ? 'cc-msg-action' : '';
     if (messageId) {
@@ -1099,7 +1101,7 @@ function _renderQueueIndicator() {
     el.className = 'cc-queue-item';
     el.style.cssText = 'padding:8px 12px;border-radius:8px;font-size:var(--text-md);line-height:1.6;max-width:95%;align-self:flex-end;background:var(--blue);color:#fff;opacity:0.5;order:9999';
     // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; queued message text wrapped in escHtml() (fields: m.message)
-    el.innerHTML = escHtml(typeof m === 'string' ? m : m.message) + '<div style="font-size:var(--text-xs);opacity:0.7;font-style:italic;margin-top:2px">queued</div>';
+    el.innerHTML = escHtml(typeof m === 'string' ? m : m.message) + '<div style="font-size:var(--text-sm);opacity:0.7;font-style:italic;margin-top:2px">queued</div>';
     msgs.appendChild(el);
   });
   if (msgs.scrollHeight - msgs.scrollTop - msgs.clientHeight < 150) msgs.scrollTop = msgs.scrollHeight;
@@ -1211,7 +1213,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
       }
       var secs = Math.floor(elapsed / 1000);
       return '<div style="display:flex;align-items:center;gap:6px"><span style="color:var(--muted);font-size:var(--text-base)">' + label + '</span>' + dotPulse + '<span style="margin-left:auto;font-size:var(--text-sm);color:var(--muted)">' + secs + 's</span>' +
-        '<button onclick="ccAbort()" style="font-size:var(--text-xs);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer;margin-left:4px">Stop</button></div>';
+        '<button onclick="ccAbort()" style="font-size:var(--text-sm);padding:2px 8px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--red);cursor:pointer;margin-left:4px">Stop</button></div>';
     }
     function updateStreamDiv() {
       // Skip DOM updates if user switched to a different tab (restore interval handles that tab)
@@ -1239,7 +1241,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
     }
     function _ccElapsedFooter(label) {
       var seconds = Math.round((Date.now() - ccStartTime) / 1000);
-      return '<div style="font-size:var(--text-xs);color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + label.replace('{seconds}', seconds) + '</div>';
+      return '<div style="font-size:var(--text-sm);color:var(--muted);margin-top:6px;display:flex;justify-content:flex-end;padding-right:30px">' + label.replace('{seconds}', seconds) + '</div>';
     }
     function _ccRetryControls(retryRequest, extraHtml, showReload) {
       var retryTabId = retryRequest && retryRequest.tabId ? retryRequest.tabId : activeTabId;
@@ -1396,7 +1398,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
         var ccErrCode = typeof evt.code === 'string' ? evt.code : '';
         var ccRetry = _ccStoreRetryRequest(activeTab, activeTabId, message);
         var codeChip = ccErrCode
-          ? '<span style="display:inline-block;margin-left:6px;padding:1px 6px;font-size:var(--text-xs);color:var(--muted);background:var(--surface2);border:1px solid var(--border);border-radius:3px;font-family:monospace">' + escHtml(ccErrCode) + '</span>'
+          ? '<span style="display:inline-block;margin-left:6px;padding:1px 6px;font-size:var(--text-sm);color:var(--muted);background:var(--surface2);border:1px solid var(--border);border-radius:3px;font-family:monospace">' + escHtml(ccErrCode) + '</span>'
           : '';
         var availList = Array.isArray(evt.availableModels) && evt.availableModels.length
           ? '<div style="font-size:var(--text-sm);color:var(--muted);margin-top:6px">Available models: ' + escHtml(evt.availableModels.slice(0, 8).join(', ')) + (evt.availableModels.length > 8 ? '…' : '') + '</div>'
@@ -1547,7 +1549,7 @@ async function _ccDoSend(message, skipUserMsg, forceTabId, intentMetadata) {
       var errorRendered;
       if (ccEnv) {
         var ccCodeChip = ccEnv.code
-          ? '<span style="display:inline-block;margin-left:6px;padding:1px 6px;font-size:var(--text-xs);color:var(--muted);background:var(--surface2);border:1px solid var(--border);border-radius:3px;font-family:monospace">' + escHtml(ccEnv.code) + '</span>'
+          ? '<span style="display:inline-block;margin-left:6px;padding:1px 6px;font-size:var(--text-sm);color:var(--muted);background:var(--surface2);border:1px solid var(--border);border-radius:3px;font-family:monospace">' + escHtml(ccEnv.code) + '</span>'
           : '';
         var ccAvail = Array.isArray(ccEnv.availableModels) && ccEnv.availableModels.length
           ? '<div style="font-size:var(--text-sm);color:var(--muted);margin-top:6px">Available models: ' + escHtml(ccEnv.availableModels.slice(0, 8).join(', ')) + (ccEnv.availableModels.length > 8 ? '…' : '') + '</div>'

package/dashboard/js/live-stream.js

@@ -352,7 +352,7 @@ async function sendSteering() {
   if (region) {
     // eslint-disable-next-line no-unsanitized/method -- reason: structural HTML is a string literal; all user data wrapped in escHtml() (fields: steering message)
     region.insertAdjacentHTML('beforeend', '<div style="align-self:flex-end;background:var(--blue);color:#fff;padding:6px 12px;border-radius:12px 12px 2px 12px;max-width:80%;margin:4px 0;font-size:var(--text-md)">' + escHtml(message) +
-      '<div id="steer-pending" style="font-size:var(--text-xs);opacity:0.7;margin-top:2px">\u2197 Sending...</div></div>');
+      '<div id="steer-pending" style="font-size:var(--text-sm);opacity:0.7;margin-top:2px">\u2197 Sending...</div></div>');
     if (container) container.scrollTop = container.scrollHeight;
   }
   showToast('cmd-toast', 'Steering message sent to ' + currentAgentId, true);