@yemi33/minions 0.1.110 → 0.1.112

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.1.112 (2026-04-01)
+
+### Engine
+- engine/ado.js
+- engine/github.js
+- engine/lifecycle.js
+
+### Dashboard
+- dashboard.js
+- dashboard/js/live-stream.js
+- dashboard/js/modal-qa.js
+- dashboard/js/utils.js
+
 ## 0.1.110 (2026-04-01)
 
 ### Dashboard

package/dashboard/js/live-stream.js

@@ -19,7 +19,7 @@ function renderLiveChatMessage(raw) {
         if (block.type === 'tool_use') {
           el.innerHTML += '<div style="background:var(--surface);border:1px solid var(--border);padding:4px 8px;border-radius:4px;margin:2px 0;font-size:10px;color:var(--muted);cursor:pointer" onclick="this.nextElementSibling.style.display=this.nextElementSibling.style.display===\'none\'?\'block\':\'none\'">' +
             '\u{1F527} ' + escHtml(block.name || 'tool') + '</div>' +
-            '<div style="display:none;background:var(--bg);padding:4px 8px;border-radius:4px;margin:0 0 4px;font-size:10px;font-family:monospace;white-space:pre-wrap;max-height:200px;overflow-y:auto;color:var(--muted)">' + escHtml(JSON.stringify(block.input, null, 2).slice(0, 500)) + '</div>';
+            '<div style="display:none;background:var(--bg);padding:4px 8px;border-radius:4px;margin:0 0 4px;font-size:10px;font-family:monospace;white-space:pre-wrap;max-height:200px;overflow-y:auto;color:var(--muted)">' + escHtml(JSON.stringify(block.input || {}, null, 2).slice(0, 500)) + '</div>';
         }
       }
     }

package/dashboard/js/modal-qa.js

@@ -227,8 +227,8 @@ async function _processQaMessage(message, selection) {
         _modalDocContext.content = display;
       }
 
-      // If editing paused an active PRD, show re-execute actions
-      if (data.pausedPrd && capturedFilePath) {
+      // If editing paused an active PRD, show re-execute actions (only for plan .md files)
+      if (data.pausedPrd && capturedFilePath && /^plans\/.*\.md$/.test(capturedFilePath)) {
         const planFile = capturedFilePath.replace(/^plans\//, '');
         const esc = planFile.replace(/'/g, "\\'");
         const actionDiv = document.createElement('div');

package/dashboard/js/utils.js

@@ -67,7 +67,10 @@ function renderMd(s) {
   html = html.replace(/(?<!\w)\*([^*\n]+)\*(?!\w)/g, '<em>$1</em>');
   html = html.replace(/(?<!\w)_([^_\n]+)_(?!\w)/g, '<em>$1</em>');
   html = html.replace(/~~(.+?)~~/g, '<s>$1</s>');
-  html = html.replace(/\[([^\]]+)\]\(([^)]+)\)/g, '<a href="$2" target="_blank" style="color:var(--blue)">$1</a>');
+  html = html.replace(/\[([^\]]+)\]\(([^)]+)\)/g, function(_, text, href) {
+    if (/^(javascript|data|vbscript):/i.test(href)) return text;
+    return '<a href="' + href + '" target="_blank" rel="noopener" style="color:var(--blue)">' + text + '</a>';
+  });
 
   // 3. Block-level processing (line by line)
   var lines = html.split('\n');

package/dashboard.js

@@ -1453,55 +1453,50 @@ const server = http.createServer(async (req, res) => {
         manifest.push({ category: e.cat, file: e.file, title: e.title, agent: e.agent, date: e.date, content: content.slice(0, 3000) });
       }
 
-      const prompt = `You are a knowledge base curator. Analyze these ${manifest.length} knowledge base entries and produce a cleanup plan.
+      const { callLLM, trackEngineUsage } = require('./engine/llm');
+      const BATCH_SIZE = 30; // ~30 entries per batch to stay within Haiku context
+      const batches = [];
+      for (let i = 0; i < manifest.length; i += BATCH_SIZE) {
+        batches.push(manifest.slice(i, i + BATCH_SIZE));
+      }
+
+      const plan = { duplicates: [], reclassify: [], remove: [] };
+      for (let b = 0; b < batches.length; b++) {
+        const batch = batches[b];
+        const offset = b * BATCH_SIZE;
+        const prompt = `You are a knowledge base curator. Analyze these ${batch.length} entries (batch ${b + 1}/${batches.length}, indices ${offset}-${offset + batch.length - 1}) and produce a cleanup plan.
 
 ## Entries
 
-${manifest.map((m, i) => `<entry index="${i}" category="${m.category}" file="${m.file}" date="${m.date}" agent="${m.agent || 'unknown'}">
-${m.title}
-${m.content.slice(0, 1500)}
-</entry>`).join('\n\n')}
+${batch.map((m, i) => `[${offset + i}] ${m.category}/${m.file} | ${m.title} | ${m.date} | ${m.agent || '?'} | ${(m.content || '').slice(0, 200).replace(/\n/g, ' ')}`).join('\n')}
 
 ## Instructions
 
-1. **Find duplicates**: entries with substantially the same content or insights (same findings from different agents or dispatch runs). List pairs/groups by index. When choosing which to keep, prefer the more recent entry (later date) as it likely reflects the current state of the codebase.
-
-2. **Find misclassified**: entries in the wrong category. Common: build reports in conventions, reviews in architecture.
+1. **Find duplicates**: entries with substantially the same content (same findings, different agents/runs). List pairs by index. Prefer keeping the more recent entry.
+2. **Find misclassified**: entries in the wrong category.
+3. **Find stale/empty**: entries with no actionable content (boilerplate, bail-out notes, "no changes needed").
 
-3. **Find stale/empty**: entries with no actionable content (boilerplate, "no changes needed", bail-out notes).
-
-## Output Format
-
-Respond with ONLY valid JSON (no markdown fences, no preamble):
-
-{
-  "duplicates": [
-    { "keep": 0, "remove": [1, 5], "reason": "same PR review findings" }
-  ],
-  "reclassify": [
-    { "index": 3, "from": "conventions", "to": "build-reports", "reason": "..." }
-  ],
-  "remove": [
-    { "index": 7, "reason": "empty bail-out note" }
-  ]
-}
+Respond with ONLY valid JSON: { "duplicates": [{ "keep": N, "remove": [N], "reason": "..." }], "reclassify": [{ "index": N, "from": "cat", "to": "cat", "reason": "..." }], "remove": [{ "index": N, "reason": "..." }] }
+If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
 
-If nothing to do, return: { "duplicates": [], "reclassify": [], "remove": [] }`;
-
-      const { callLLM, trackEngineUsage } = require('./engine/llm');
-      const result = await callLLM(prompt, 'You are a concise knowledge curator. Output only JSON.', {
-        timeout: 180000, label: 'kb-sweep', model: 'haiku', maxTurns: 1
-      });
-      trackEngineUsage('kb-sweep', result.usage);
+        const result = await callLLM(prompt, 'Output only JSON.', {
+          timeout: 120000, label: 'kb-sweep', model: 'haiku', maxTurns: 1
+        });
+        trackEngineUsage('kb-sweep', result.usage);
 
-      let plan;
-      try {
-        let jsonStr = result.text.trim();
-        const fenceMatch = jsonStr.match(/```(?:json)?\s*([\s\S]*?)```/);
-        if (fenceMatch) jsonStr = fenceMatch[1].trim();
-        plan = JSON.parse(jsonStr);
-      } catch {
-        return jsonReply(res, 200, { ok: false, error: 'LLM returned invalid JSON', raw: result.text.slice(0, 500) });
+        let batchPlan;
+        try {
+          let jsonStr = (result.text || '').trim();
+          const fenceMatch = jsonStr.match(/```(?:json)?\s*([\s\S]*?)```/);
+          if (fenceMatch) jsonStr = fenceMatch[1].trim();
+          batchPlan = JSON.parse(jsonStr);
+        } catch {
+          console.log(`[kb-sweep] batch ${b + 1}/${batches.length} returned invalid JSON, skipping`);
+          continue;
+        }
+        if (batchPlan.duplicates) plan.duplicates.push(...batchPlan.duplicates);
+        if (batchPlan.reclassify) plan.reclassify.push(...batchPlan.reclassify);
+        if (batchPlan.remove) plan.remove.push(...batchPlan.remove);
       }
 
       let removed = 0, reclassified = 0, merged = 0;

package/engine/ado.js

@@ -389,7 +389,19 @@ async function reconcilePrs(config) {
       e.log('info', `PR reconciliation: added ${prId} (branch: ${branch}${confirmedItemId ? ', linked to ' + confirmedItemId : ''}) to ${project.name}`);
     }
 
-    if (projectAdded > 0 || projectUpdated > 0) {
+    // Backfill prdItems from pr-links for any PR with empty array
+    const prLinks = shared.getPrLinks();
+    let backfilled = 0;
+    for (const pr of existingPrs) {
+      const linked = prLinks[pr.id];
+      if (linked && !(pr.prdItems || []).includes(linked)) {
+        pr.prdItems = Array.isArray(pr.prdItems) ? pr.prdItems : [];
+        pr.prdItems.push(linked);
+        backfilled++;
+      }
+    }
+
+    if (projectAdded > 0 || projectUpdated > 0 || backfilled > 0) {
       shared.safeWrite(prPath, existingPrs);
       totalAdded += projectAdded;
       if (projectUpdated > 0) e.log('info', `PR reconciliation: linked ${projectUpdated} existing PR(s) to PRD items in ${project.name}`);

package/engine/github.js

@@ -376,7 +376,19 @@ async function reconcilePrs(config) {
       e.log('info', `GitHub PR reconciliation: added ${prId} (branch: ${branch}${confirmedItemId ? ', linked to ' + confirmedItemId : ''}) to ${project.name}`);
     }
 
-    if (projectAdded > 0) {
+    // Backfill prdItems from pr-links for any PR with empty array
+    const prLinks = getPrLinks();
+    let backfilled = 0;
+    for (const pr of existingPrs) {
+      const linked = prLinks[pr.id];
+      if (linked && !(pr.prdItems || []).includes(linked)) {
+        pr.prdItems = Array.isArray(pr.prdItems) ? pr.prdItems : [];
+        pr.prdItems.push(linked);
+        backfilled++;
+      }
+    }
+
+    if (projectAdded > 0 || backfilled > 0) {
       safeWrite(prPath, existingPrs);
       totalAdded += projectAdded;
     }

package/engine/lifecycle.js

@@ -724,6 +724,9 @@ async function handlePostMerge(pr, project, config, newStatus) {
 
   if (newStatus !== 'merged') return;
 
+  // Mark review as approved since it was merged
+  pr.reviewStatus = 'approved';
+
   // Resolve linked work item from pr-links or PR branch name
   let mergedItemId = getPrLinks()[pr.id];
   if (!mergedItemId && pr.branch) {
@@ -761,7 +764,7 @@ async function handlePostMerge(pr, project, config, newStatus) {
         if (item && item.status !== 'done') {
           log('info', `Post-merge: marking work item ${mergedItemId} as done (was ${item.status}) for ${pr.id}`);
           item.status = 'done';
-          item.completedAt = e.ts();
+          item.completedAt = ts();
           item._mergedVia = pr.id;
           shared.safeWrite(wiPath, items);
           break;

package/package.json

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