@yemi33/minions 0.1.2243 → 0.1.2245

package/dashboard/js/refresh.js

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

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

@@ -171,6 +171,14 @@ function wiRow(item) {
     : (item.branchStrategy === 'shared-branch' && item.status === 'done')
       ? '<span style="font-size:var(--text-xs);color:var(--muted)" title="Part of shared branch — aggregate PR created at verify stage">shared branch</span>'
       : '<span style="color:var(--muted)">—</span>';
+  // W-mqrdn6qq — Non-retryable failures surface their red failReason snippet in
+  // the PR column (not the Agent column). The engine prefixes failReason with
+  // "Non-retryable failure:" and stamps item._failureClass on non-retryable
+  // demotion (engine/dispatch.js writeDispatchResult / force-demote path).
+  var isNonRetryableFail = !!item._failureClass || /^Non-retryable failure:/i.test(item.failReason || '');
+  var failSnippet = item.failReason
+    ? '<span style="display:block;font-size:var(--text-xs);color:var(--red)" title="' + escapeHtml(item.failReason) + '">' + escapeHtml(item.failReason.slice(0, 30)) + '</span>'
+    : '';
   // PR follow-up chip (W-mpej3cox00099466) — surfaced when the WI was spun
   // off from a PR comment via meta.pr_followup. Links to the parent PR.
   var prFollowup = item.meta && item.meta.pr_followup;
@@ -183,7 +191,7 @@ function wiRow(item) {
   }
   return '<tr data-wi-id="' + escapeHtml(item.id) + '" style="cursor:pointer" onclick="if(shouldIgnoreSelectionClick(event))return;openWorkItemDetail(\'' + escapeHtml(item.id) + '\')">' +
     '<td><span class="pr-id">' + escapeHtml(item.id || '') + '</span></td>' +
-    '<td style="max-width:220px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap" title="' + escapeHtml((item.title || '').slice(0, 200)) + '">' + escapeHtml(item.title || '') + followupChip + '</td>' +
+    '<td style="min-width:280px;max-width:320px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap" title="' + escapeHtml((item.title || '').slice(0, 200)) + '">' + escapeHtml(item.title || '') + followupChip + '</td>' +
     '<td><span style="font-size:var(--text-sm);color:var(--muted)">' + escapeHtml(item._source || '') + '</span>' +
       (item.scope === 'fan-out' ? ' <span class="pr-badge ' + (item.status === 'done' || item.status === 'failed' ? 'draft' : 'building') + '" style="font-size:var(--text-xs)">fan-out</span>' : '') + '</td>' +
     '<td>' + typeBadge(item.type) + '</td>' +
@@ -217,9 +225,9 @@ function wiRow(item) {
       (item.completedAgents && item.completedAgents.length > 0
         ? '<span class="pr-agent">' + escapeHtml(item.completedAgents.join(', ')) + '</span>'
         : '<span class="pr-agent">' + escapeHtml(item.dispatched_to || item.agent || '—') + '</span>') +
-      (item.failReason ? '<span style="display:block;font-size:var(--text-xs);color:var(--red)" title="' + escapeHtml(item.failReason) + '">' + escapeHtml(item.failReason.slice(0, 30)) + '</span>' : '') +
+      (failSnippet && !isNonRetryableFail ? failSnippet : '') +
     '</td>' +
-    '<td>' + prLink + '</td>' +
+    '<td>' + prLink + (failSnippet && isNonRetryableFail ? failSnippet : '') + '</td>' +
     '<td><span class="pr-date">' + escapeHtml((item.created || '').slice(0, 16).replace('T', ' ')) + '</span></td>' +
     '<td style="white-space:nowrap;font-size:var(--text-xs);color:var(--muted)">' +
       (function() {
@@ -278,7 +286,7 @@ function renderWorkItems(items, opts) {
   const start = wiPage * WI_PER_PAGE;
   const pageItems = items.slice(start, start + WI_PER_PAGE);
 
-  let html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>ID</th><th>Title</th><th>Source</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>';
+  let html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>ID</th><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>';
   html += pageItems.map(wiRow).join('');
   html += '</tbody></table></div>';
 
@@ -479,7 +487,7 @@ async function toggleWorkItemArchive() {
       items.map(function(i) {
         return '<tr style="opacity:0.6">' +
           '<td><span class="pr-id">' + escapeHtml(i.id || '') + '</span></td>' +
-          '<td style="max-width:200px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">' + escapeHtml(i.title || '') + '</td>' +
+          '<td style="min-width:240px;max-width:280px;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">' + escapeHtml(i.title || '') + '</td>' +
           '<td><span class="dispatch-type ' + (i.type || '') + '">' + escapeHtml(i.type || '') + '</span></td>' +
           '<td style="color:' + (i.status === 'done' ? 'var(--green)' : 'var(--red)') + '">' + escapeHtml(i.status || '') + '</td>' +
           '<td>' + escapeHtml(i.dispatched_to || '—') + '</td>' +
@@ -746,7 +754,7 @@ function _wiRenderDetail(item) {
   }
   html += field('Description', '<div id="wi-detail-desc" style="font-size:var(--text-md);max-height:320px;overflow-y:auto;padding:8px 10px;background:var(--surface2);border:1px solid var(--border);border-radius:var(--radius-sm)">' + _descHtml + '</div>');
   html += field('Agent', escapeHtml(item.dispatched_to || item.agent || 'Auto'));
-  html += field('Source', escapeHtml(item._source || 'central'));
+  html += field('Project', escapeHtml(item._source || 'central'));
   // P-714ef144 — surface meta.workdir in the detail modal so operators can
   // confirm the agent's actual cwd. Renders nothing when unset (which is
   // the back-compat default = project root). Code-tagged so the path is
@@ -1053,7 +1061,7 @@ function openWorkItemDetail(id) {
 
 function openAllWorkItems() {
   document.getElementById('modal-title').textContent = 'All Work Items (' + allWorkItems.length + ')';
-  const html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>ID</th><th>Title</th><th>Source</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>' +
+  const html = '<div class="pr-table-wrap"><table class="pr-table"><thead><tr><th>ID</th><th>Title</th><th>Project</th><th>Type</th><th>Priority</th><th>Status</th><th>Agent</th><th>PR</th><th>Created</th><th></th><th></th></tr></thead><tbody>' +
     allWorkItems.map(wiRow).join('') + '</tbody></table></div>';
   // eslint-disable-next-line no-unsanitized/property -- reason: structural HTML is a string literal; all user data wrapped in escapeHtml() by wiRow() (fields: work item id/title/source/status/agent/PR links/dates/follow-up metadata)
   document.getElementById('modal-body').innerHTML = html;

package/dashboard/styles.css

@@ -1125,7 +1125,7 @@
 
   /* Command Center tab bar */
   .cc-tab-scroll { display: flex; gap: 4px; align-items: center; overflow-x: auto; overflow-y: hidden; flex: 1 1 auto; min-width: 0; scrollbar-width: thin; }
-  .cc-tab { padding: 4px 10px; font-size: var(--text-sm); border: 1px solid var(--border); border-bottom: none; border-radius: 6px 6px 0 0; background: var(--surface2); color: var(--muted); cursor: grab; white-space: nowrap; max-width: 140px; display: inline-flex; align-items: center; gap: 2px; flex-shrink: 0; margin-bottom: -1px; position: relative; }
+  .cc-tab { padding: 4px 10px; font-size: var(--text-sm); border: 1px solid var(--border); border-bottom: none; border-radius: 6px 6px 0 0; background: var(--surface2); color: var(--muted); cursor: pointer; white-space: nowrap; max-width: 140px; display: inline-flex; align-items: center; gap: 2px; flex-shrink: 0; margin-bottom: -1px; position: relative; }
   .cc-tab:active { cursor: grabbing; }
   .cc-tab.cc-tab-dragging { opacity: 0.5; cursor: grabbing; }
   .cc-tab.cc-tab-drop-target { box-shadow: inset 2px 0 0 0 var(--blue); }

package/engine/comment-classifier.js

@@ -68,6 +68,14 @@ const _FIREBASE_BRAND_RE = /\bFirebase\s+(?:App\s+Distribution|Hosting)\b/i;
 const _FIREBASE_URL_RE = /\bappdistribution\.firebase\b/i;
 const _APPETIZE_URL_RE = /\bappetize\.io\b/i;
 
+// Firebase Hosting bot posts — "Firebase Hosting preview URL: ...",
+// "Firebase Hosting deployment ...". These never start with a heading but
+// reliably begin a line with "Firebase Hosting". Human feedback about
+// Firebase Hosting issues starts with prose like "The Firebase Hosting ..."
+// or "Our Firebase Hosting ..." so anchoring at the start of a line is a
+// safe discriminator.
+const _FIREBASE_HOSTING_LEAD_RE = /^Firebase\s+Hosting\b/im;
+
 // "deploy ready", "deployment available", "preview succeeded", etc.
 const _DEPLOY_READY_RE =
   /\b(?:deploy|deployment|preview)\s+(?:ready|available|succeeded|complete|completed)\b/i;
@@ -80,6 +88,7 @@ function isPreviewStatusBody(body) {
   if (_PREVIEW_HEADING_RE.test(text)) return true;
   if (_FIREBASE_BRAND_RE.test(text)) return true;
   if (_FIREBASE_URL_RE.test(text)) return true;
+  if (_FIREBASE_HOSTING_LEAD_RE.test(text)) return true;
   if (_APPETIZE_URL_RE.test(text)) return true;
   if (_DEPLOY_READY_RE.test(text)) return true;
   return false;

package/package.json

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