@yemi33/minions 0.1.2002 → 0.1.2003

package/dashboard/js/fre.js

@@ -0,0 +1,196 @@
+// fre.js — First-Run Experience banner for empty-state dashboards (W-mpehchnt000w46da).
+//
+// Renders a two-step onboarding banner at the top of the main content area when
+// the operator has zero projects configured AND has not explicitly dismissed the
+// banner. Self-contained and idempotent — safe to call renderFre() every tick.
+//
+// Trigger: projects.length === 0 AND localStorage.minions_fre_dismissed !== '1'.
+// Auto-hide (no dismissal flag set) when projects.length >= 1 — the banner has
+// served its purpose. If the user later removes all projects we re-show.
+
+const FRE_DISMISS_KEY = 'minions_fre_dismissed';
+const FRE_MOUNT_ID = 'fre-banner';
+
+function _freIsDismissed() {
+  try { return localStorage.getItem(FRE_DISMISS_KEY) === '1'; } catch { return false; }
+}
+
+function dismissFre() {
+  try { localStorage.setItem(FRE_DISMISS_KEY, '1'); } catch { /* expected */ }
+  const mount = document.getElementById(FRE_MOUNT_ID);
+  if (mount) mount.innerHTML = '';
+}
+
+// Reset helper (exposed for tests / debugging only). Not wired to UI.
+function _freReset() {
+  try { localStorage.removeItem(FRE_DISMISS_KEY); } catch { /* expected */ }
+}
+
+// Open the Settings modal and scroll to the Default CLI control (id=set-defaultCli).
+// The modal lives in a separate modal layer, not a sidebar page — openSettings()
+// is async (loads /api/settings) so we await it before scrolling.
+async function openSettingsToDefaultCli() {
+  if (typeof openSettings !== 'function') return;
+  try {
+    await openSettings();
+  } catch { /* settings fetch failure already toasts */ return; }
+  // openSettings injects innerHTML synchronously after the fetch resolves, so
+  // set-defaultCli is in the DOM by now. initRuntimeFleetUI() runs separately
+  // and hydrates options — but the <select> element itself is already present.
+  setTimeout(function() {
+    const el = document.getElementById('set-defaultCli');
+    if (!el) return;
+    try { el.scrollIntoView({ behavior: 'smooth', block: 'center' }); } catch { /* old browser */ }
+    try { el.focus({ preventScroll: true }); } catch { /* expected */ }
+    const prevOutline = el.style.outline;
+    el.style.outline = '2px solid var(--blue)';
+    el.style.outlineOffset = '2px';
+    setTimeout(function() { el.style.outline = prevOutline; el.style.outlineOffset = ''; }, 2000);
+  }, 50);
+}
+
+// Render the FRE banner into #fre-banner. Idempotent — bails out early when:
+//   • mount point missing (layout didn't load yet)
+//   • banner explicitly dismissed
+//   • projects.length >= 1 (auto-hide)
+// Otherwise renders the two-step card.
+//
+// Accepts the current /api/status payload (or just a projects array, for tests).
+// When passed the full payload, autoMode.defaultCli drives the runtime label.
+function renderFre(statusOrProjects) {
+  const mount = document.getElementById(FRE_MOUNT_ID);
+  if (!mount) return;
+  const isArr = Array.isArray(statusOrProjects);
+  const projects = isArr
+    ? statusOrProjects
+    : (statusOrProjects && Array.isArray(statusOrProjects.projects) ? statusOrProjects.projects : []);
+  const status = isArr ? (window._lastStatus || {}) : (statusOrProjects || {});
+  const projectCount = projects.length;
+  // Bail-out paths assign an empty string literal on its own line so the
+  // SEC-03 dynamic-innerHTML counter (test/unit.test.js
+  // DYNAMIC_INNERHTML_BASELINE) treats them as exempt.
+  if (projectCount >= 1) {
+    mount.innerHTML = '';
+    return;
+  }
+  if (_freIsDismissed()) {
+    mount.innerHTML = '';
+    return;
+  }
+
+  // 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'.
+  const auto = (status && status.autoMode) || {};
+  const runtimeCli = String(auto.defaultCli || auto.ccCli || 'claude');
+
+  const cardStyle = [
+    'margin:12px 24px',
+    'padding:16px 20px',
+    'background:var(--surface2)',
+    'border:1px solid var(--blue)',
+    'border-radius:var(--radius-lg)',
+    'color:var(--text)',
+    'font-size:13px',
+    'box-shadow:var(--shadow-md)',
+  ].join(';');
+
+  const stepBox = [
+    'display:flex',
+    'gap:12px',
+    'align-items:flex-start',
+    'padding:10px 12px',
+    'background:var(--surface)',
+    'border:1px solid var(--border)',
+    'border-radius:var(--radius-sm)',
+  ].join(';');
+
+  const stepNum = [
+    'display:inline-flex',
+    'align-items:center',
+    'justify-content:center',
+    'width:22px',
+    'height:22px',
+    'border-radius:var(--radius-full)',
+    'background:var(--blue)',
+    'color:#fff',
+    'font-weight:700',
+    'font-size:11px',
+    'flex-shrink:0',
+  ].join(';');
+
+  const btnPrimary = [
+    'padding:6px 14px',
+    'background:var(--blue)',
+    'color:#fff',
+    'border:none',
+    'border-radius:var(--radius-sm)',
+    'cursor:pointer',
+    'font-size:12px',
+    'font-weight:600',
+    'margin-top:8px',
+  ].join(';');
+
+  const btnGhost = [
+    'padding:6px 14px',
+    'background:transparent',
+    'color:var(--blue)',
+    'border:1px solid var(--blue)',
+    'border-radius:var(--radius-sm)',
+    'cursor:pointer',
+    'font-size:12px',
+    'font-weight:600',
+    'margin-top:8px',
+  ].join(';');
+
+  const dismissBtn = [
+    'background:none',
+    'border:none',
+    'color:var(--muted)',
+    'cursor:pointer',
+    'font-size:11px',
+    'padding:2px 8px',
+  ].join(';');
+
+  // Use textContent-safe interpolation for the runtime label (config-derived, but defensive).
+  const safeRuntime = String(runtimeCli).replace(/[<>&"']/g, function(c) {
+    return ({ '<': '&lt;', '>': '&gt;', '&': '&amp;', '"': '&quot;', "'": '&#39;' })[c];
+  });
+
+  mount.innerHTML =
+    '<div id="fre-card" style="' + cardStyle + '">' +
+      '<div style="display:flex;align-items:center;justify-content:space-between;margin-bottom:10px">' +
+        '<div style="font-size:15px;font-weight:700;color:var(--blue)">&#x1F44B; Welcome to Minions</div>' +
+        '<button onclick="dismissFre()" style="' + dismissBtn + '" title="Hide this banner — clear localStorage.minions_fre_dismissed to re-show">Dismiss</button>' +
+      '</div>' +
+      '<div style="color:var(--muted);font-size:12px;margin-bottom:12px">Two quick steps to get your fleet ready.</div>' +
+      '<div style="display:flex;flex-direction:column;gap:10px">' +
+
+        '<div style="' + stepBox + '">' +
+          '<span style="' + stepNum + '">1</span>' +
+          '<div style="flex:1">' +
+            '<div style="font-weight:600;margin-bottom:2px">Pick your runtime CLI</div>' +
+            '<div style="color:var(--muted);font-size:12px;line-height:1.5">' +
+              'Minions spawns agents through this CLI. Switch via Settings &rarr; Engine &rarr; Default CLI (<code>claude</code> or <code>copilot</code>). ' +
+              'Currently: <code style="background:var(--bg);padding:1px 6px;border-radius:3px;color:var(--text)">' + safeRuntime + '</code>' +
+            '</div>' +
+            '<button onclick="openSettingsToDefaultCli()" style="' + btnGhost + '">Open Settings &rarr; Default CLI</button>' +
+          '</div>' +
+        '</div>' +
+
+        '<div style="' + stepBox + '">' +
+          '<span style="' + stepNum + '">2</span>' +
+          '<div style="flex:1">' +
+            '<div style="font-weight:600;margin-bottom:2px">Add your first project</div>' +
+            '<div style="color:var(--muted);font-size:12px;line-height:1.5">' +
+              'Projects map a local git worktree to a remote repo. Without a project, agents have nowhere to run.' +
+            '</div>' +
+            '<button onclick="addProject()" style="' + btnPrimary + '">+ Add Project</button>' +
+          '</div>' +
+        '</div>' +
+
+      '</div>' +
+    '</div>';
+}
+
+window.MinionsFre = { renderFre, dismissFre, openSettingsToDefaultCli, _freReset, FRE_DISMISS_KEY };

package/dashboard/js/refresh.js

@@ -79,6 +79,12 @@ function _processStatusUpdate(data) {
   if (_changed('prdProgress', data.prdProgress) || _changed('prdPrs', data.pullRequests?.length)) { renderPrdProgress(data.prdProgress); _cachePrdItems(data.prdProgress); }
   if (_changed('inbox', data.inbox)) renderInbox(data.inbox || []);
   if (_changed('projects', data.projects)) { cmdUpdateProjectList(data.projects || []); renderProjects(data.projects || []); }
+  // FRE banner — safe to call every tick (idempotent + cheap). Pass the full
+  // status payload so the runtime-CLI explainer reads autoMode.defaultCli from
+  // THIS tick rather than the previous one (window._lastStatus is set later).
+  if (typeof renderFre === 'function') {
+    try { renderFre(data); } catch { /* expected on first load */ }
+  }
   if (_changed('notes', data.notes)) renderNotes(data.notes);
   if (_changed('prd', [data.prd, data.prdProgress])) renderPrd(data.prd, data.prdProgress);
   if (_changed('prs', data.pullRequests)) renderPrs(data.pullRequests || []);

package/dashboard/js/render-prs.js

@@ -135,15 +135,27 @@ function openModal(i) {
 }
 
 function openAddPrModal() {
-  const projOpts = (typeof cmdProjects !== 'undefined' ? cmdProjects : []).map(p => {
+  const projects = (typeof cmdProjects !== 'undefined' ? cmdProjects : []) || [];
+  const projOpts = projects.map(p => {
     const name = typeof p === 'object' ? p.name : p;
     return '<option value="' + escapeHtml(name) + '">' + escapeHtml(name) + '</option>';
   }).join('');
   const inputStyle = 'display:block;width:100%;margin-top:4px;padding:6px 8px;background:var(--bg);border:1px solid var(--border);border-radius:var(--radius-sm);color:var(--text);font-size:var(--text-md);font-family:inherit';
 
+  // No-projects warning (W-mpehchnt000w46da) — surfaced when the operator hasn't
+  // added any project yet. We still allow the link (read-only tracking is valid),
+  // but warn that auto-fix / auto-review can't dispatch without a project worktree.
+  const noProjectsWarning = projects.length === 0
+    ? '<div id="pr-link-no-projects-warning" style="padding:8px 12px;background:rgba(210,153,34,0.15);border:1px solid rgba(210,153,34,0.3);border-radius:var(--radius-sm);color:var(--yellow);font-size:11px;line-height:1.5;margin-bottom:4px">' +
+        '<div style="font-weight:700;margin-bottom:2px">&#x26A0; No project configured.</div>' +
+        '<div style="color:var(--text)">This PR will be tracked, but auto-fix and auto-review cannot dispatch against it &mdash; agents need a project worktree to operate in. Add a project first (Projects tab) to enable automation.</div>' +
+      '</div>'
+    : '';
+
   document.getElementById('modal-title').textContent = 'Link Pull Request';
   document.getElementById('modal-body').innerHTML =
     '<div style="display:flex;flex-direction:column;gap:10px">' +
+      noProjectsWarning +
       '<label style="color:var(--text);font-size:var(--text-md)">PR URL <input id="pr-link-url" style="' + inputStyle + '" placeholder="https://github.com/org/repo/pull/123"></label>' +
       '<label style="color:var(--text);font-size:var(--text-md)">Title <input id="pr-link-title" style="' + inputStyle + '" placeholder="Short description (optional — auto-detected from URL)"></label>' +
       '<label style="color:var(--text);font-size:var(--text-md)">Project <select id="pr-link-project" style="' + inputStyle + '"><option value="">Auto-detect from URL (central if no unique match)</option>' + projOpts + '</select></label>' +

package/dashboard/layout.html

@@ -78,7 +78,10 @@
     <a class="sidebar-link" data-page="meetings" href="/meetings">Meetings</a>
     <a class="sidebar-link" data-page="engine" href="/engine">Engine</a>
   </nav>
-  <div class="page-content" id="page-content"><!-- __PAGES__ --></div>
+  <div class="page-content" id="page-content">
+    <div id="fre-banner"></div>
+    <!-- __PAGES__ -->
+  </div>
 </div>
 
 <!-- Floating toast — lives outside .page divs so showToast() works on every page -->

package/dashboard-build.js

@@ -34,7 +34,7 @@ function buildDashboardHtml() {
     'render-prs', 'render-plans', 'render-inbox', 'render-kb', 'render-skills',
     'render-other', 'render-managed', 'render-schedules', 'render-watches', 'render-pipelines', 'render-meetings', 'render-pinned',
     'command-parser', 'command-input', 'command-center', 'command-history',
-    'modal', 'modal-qa', 'settings', 'qa', 'refresh'
+    'modal', 'modal-qa', 'settings', 'qa', 'fre', 'refresh'
   ];
   let jsHtml = '';
   for (const f of jsFiles) {

package/dashboard.js

@@ -980,7 +980,7 @@ function buildDashboardHtml() {
     'render-prs', 'render-plans', 'render-inbox', 'render-kb', 'render-skills',
     'render-other', 'render-managed', 'render-schedules', 'render-watches', 'render-pipelines', 'render-meetings', 'render-pinned',
     'command-parser', 'command-input', 'command-center', 'command-history',
-    'modal', 'modal-qa', 'settings', 'qa', 'refresh'
+    'modal', 'modal-qa', 'settings', 'qa', 'fre', 'refresh'
   ];
   let jsHtml = '';
   for (const f of jsFiles) {
@@ -1640,6 +1640,7 @@ function _buildStatusSlowState() {
       ccCli: shared.resolveCcCli(CONFIG.engine),
       ccModel: shared.resolveCcModel(CONFIG.engine),
       ccEffort: CONFIG.engine?.ccEffort || shared.ENGINE_DEFAULTS.ccEffort,
+      defaultCli: shared.resolveAgentCli(null, CONFIG.engine),
     },
     initialized: !!(CONFIG.agents && Object.keys(CONFIG.agents).length > 0),
     installId: safeRead(path.join(MINIONS_DIR, '.install-id')).trim() || null,

package/package.json

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