@yemi33/minions 0.1.482 → 0.1.483

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.483 (2026-04-07)
+
+### Fixes
+-  harden bug filing — proper escaping, error handling, direct UI button
+
 ## 0.1.482 (2026-04-07)
 
 ### Features

package/dashboard/js/command-center.js

@@ -639,12 +639,12 @@ async function ccExecuteAction(action) {
         break;
       }
       case 'file-bug': {
-        const res = await _ccFetch('/api/issues/create', { title: action.title, description: action.description, project: action.project, labels: action.labels });
+        const res = await _ccFetch('/api/issues/create', { title: action.title, description: action.description, labels: action.labels });
         const d = await res.json();
         if (d.url) {
           status.innerHTML = '&#128027; Bug filed: <a href="' + escHtml(d.url) + '" target="_blank" style="color:var(--blue)">' + escHtml(action.title) + '</a>';
         } else {
-          status.innerHTML = '&#128027; Bug filed: <strong>' + escHtml(action.title) + '</strong>';
+          status.innerHTML = '&#128027; Bug filed: <strong>' + escHtml(action.title) + '</strong> — <a href="https://github.com/yemi33/minions/issues" target="_blank" style="color:var(--blue)">view issues</a>';
         }
         status.style.color = 'var(--green)';
         break;

package/dashboard/js/utils.js

@@ -199,4 +199,42 @@ function renderMd(s) {
   return '<div class="md-content">' + html + '</div>';
 }
 
-window.MinionsUtils = { wakeEngine, escHtml, renderMd, normalizePlanFile, timeAgo, statusColor, llmCopyBtn, copyLlmText };
+function openBugReport() {
+  document.getElementById('modal-title').textContent = 'Report a Bug';
+  document.getElementById('modal-body').innerHTML =
+    '<div style="display:flex;flex-direction:column;gap:12px">' +
+      '<p style="color:var(--muted);font-size:12px;margin:0">File a bug on the Minions repo (yemi33/minions).</p>' +
+      '<label style="color:var(--text);font-size:var(--text-md)">Title' +
+        '<input id="bug-title" style="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)" placeholder="Short description of the bug"></label>' +
+      '<label style="color:var(--text);font-size:var(--text-md)">Description' +
+        '<textarea id="bug-desc" rows="6" style="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);resize:vertical;font-family:inherit" placeholder="Steps to reproduce, expected vs actual behavior..."></textarea></label>' +
+      '<div style="display:flex;justify-content:flex-end;gap:8px">' +
+        '<button onclick="closeModal()" class="pr-pager-btn">Cancel</button>' +
+        '<button onclick="submitBugReport()" style="padding:6px 16px;background:var(--red);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">File Bug</button>' +
+      '</div>' +
+    '</div>';
+  document.getElementById('modal').classList.add('open');
+}
+
+async function submitBugReport() {
+  var title = document.getElementById('bug-title')?.value?.trim();
+  var desc = document.getElementById('bug-desc')?.value?.trim();
+  if (!title) { alert('Title is required'); return; }
+  try { closeModal(); } catch {}
+  showToast('cmd-toast', 'Filing bug...', true);
+  try {
+    var res = await fetch('/api/issues/create', {
+      method: 'POST', headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({ title: title, description: desc || '' })
+    });
+    var d = await res.json();
+    if (!res.ok) throw new Error(d.error || 'Failed');
+    if (d.url) {
+      showToast('cmd-toast', 'Bug filed — <a href="' + d.url + '" target="_blank" style="color:var(--blue)">view issue</a>', true);
+    } else {
+      showToast('cmd-toast', 'Bug filed', true);
+    }
+  } catch (e) { showToast('cmd-toast', 'Error: ' + e.message, false); }
+}
+
+window.MinionsUtils = { wakeEngine, escHtml, renderMd, normalizePlanFile, timeAgo, statusColor, llmCopyBtn, copyLlmText, openBugReport, submitBugReport };

package/dashboard/layout.html

@@ -16,6 +16,7 @@
     <span id="version-banner" style="font-size:9px;color:var(--muted)"></span>
     <div class="timestamp" id="ts">—</div>
     <button onclick="toggleCommandCenter()" style="background:none;border:none;cursor:pointer;font-size:12px;color:var(--blue);padding:4px 10px;line-height:1;font-weight:700;border:1px solid var(--blue);border-radius:4px" title="Command Center" id="cc-toggle-btn">Command Center</button>
+    <button onclick="openBugReport()" style="background:none;border:none;cursor:pointer;font-size:11px;color:var(--muted);padding:4px 8px;line-height:1;border:1px solid var(--border);border-radius:4px" title="Report a bug in Minions">Report Bug</button>
     <button onclick="openSettings()" style="background:none;border:none;cursor:pointer;font-size:18px;color:var(--muted);padding:4px;line-height:1" title="Settings">&#9881;</button>
   </div>
 </header>

package/dashboard.js

@@ -576,7 +576,7 @@ Available action types:
 - **link-pr**: Link an external PR for tracking. Fields: url (PR URL), title (optional), project (optional), autoObserve (bool, default true)
 - **archive-meeting**: Archive a completed meeting. Fields: id (meeting ID)
 - **update-routing**: Update the routing table. Fields: content (full routing.md content)
-- **file-bug**: File a bug on the Minions repo (yemi33/minions). Fields: title (short bug title), description (markdown body with repro steps, expected vs actual behavior), labels (optional array, defaults to ["bug"]). Use when the user says "file a bug", "create an issue", "report this", etc. This is for bugs in Minions itself, not the user's project.
+- **file-bug**: File a bug on the Minions repo (yemi33/minions). Fields: title (short bug title), description (markdown body), labels (optional array, defaults to ["bug"]). Trigger phrases: "file a bug", "file an issue", "report a bug", "create an issue", "this is a bug", "log a bug". This is for bugs in Minions itself, not the user's project. When filing, include repro steps and relevant context from the conversation in the description. If the user hasn't provided repro steps, ask before filing.
 
 ## Rules
 
@@ -3190,15 +3190,40 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const body = await readBody(req);
       if (!body.title) return jsonReply(res, 400, { error: 'title required' });
 
+      // Check gh CLI is available
+      try { shared.exec('gh --version', { encoding: 'utf-8', timeout: 5000, windowsHide: true }); }
+      catch { return jsonReply(res, 500, { error: 'gh CLI not installed. Run: npm install -g gh' }); }
+
       const repo = 'yemi33/minions';
       const labels = (body.labels || ['bug']).join(',');
       const bugBody = (body.description || '') + '\n\n---\n_Filed via Minions dashboard_';
 
-      const cmd = `gh issue create --repo "${repo}" --title "${body.title.replace(/"/g, '\\"')}" --body "${bugBody.replace(/"/g, '\\"')}" --label "${labels}" 2>&1`;
-      const result = shared.exec(cmd, { encoding: 'utf-8', timeout: 30000, windowsHide: true });
-      const urlMatch = result.match(/https:\/\/github\.com\/\S+/);
-      return jsonReply(res, 200, { ok: true, url: urlMatch ? urlMatch[0] : null, output: result.trim() });
-    } catch (e) { return jsonReply(res, 500, { error: e.message }); }
+      // Write body to temp file to avoid shell escaping issues with quotes, backticks, newlines
+      const tmpBody = path.join(ENGINE_DIR, 'tmp', `bug-body-${Date.now()}.md`);
+      safeWrite(tmpBody, bugBody);
+      const safeTitle = body.title.replace(/["`$\\]/g, '');
+      try {
+        const cmd = `gh issue create --repo "${repo}" --title "${safeTitle}" --body-file "${tmpBody}" --label "${labels}" 2>&1`;
+        const result = shared.exec(cmd, { encoding: 'utf-8', timeout: 30000, windowsHide: true });
+        shared.safeUnlink(tmpBody);
+        // Detect gh errors in output
+        if (result.includes('authentication') || result.includes('auth login')) {
+          return jsonReply(res, 401, { error: 'GitHub auth required. Run: gh auth login' });
+        }
+        const urlMatch = result.match(/https:\/\/github\.com\/\S+/);
+        if (!urlMatch) {
+          return jsonReply(res, 500, { error: 'Issue may not have been created: ' + result.trim().slice(0, 200) });
+        }
+        return jsonReply(res, 200, { ok: true, url: urlMatch[0], output: result.trim() });
+      } catch (e) {
+        shared.safeUnlink(tmpBody);
+        throw e;
+      }
+    } catch (e) {
+      const msg = e.message || '';
+      if (msg.includes('ENOENT') || msg.includes('not found')) return jsonReply(res, 500, { error: 'gh CLI not found. Install from https://cli.github.com/' });
+      return jsonReply(res, 500, { error: msg });
+    }
   }
 
   async function handleCommandCenterNewSession(req, res) {

package/package.json

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