@yemi33/minions 0.1.686 → 0.1.688

package/CHANGELOG.md

@@ -1,14 +1,19 @@
 # Changelog
 
-## 0.1.686 (2026-04-09)
+## 0.1.688 (2026-04-09)
 
 ### Features
+-  PR review/fix cycle pipeline, fix node rendering for never-run pipelines
+-  modal back button for artifact navigation
 -  collapsible doc-chat thread in modal
 
 ### Fixes
 -  add archive fallback for artifact notes when KB entry was swept
 -  link work item artifacts to KB entries, not raw archive copies
 
+### Other
+- refactor: remove dead stageFlow code from pipeline card rendering
+
 ## 0.1.683 (2026-04-09)
 
 ### Features

package/dashboard/js/modal.js

@@ -4,6 +4,7 @@ function closeModal() {
   const modalEl = document.querySelector('#modal .modal');
   if (modalEl) modalEl.classList.remove('modal-wide');
   document.getElementById('modal').classList.remove('open');
+  clearModalBackStack();
   // Hide Q&A section (only shown for document modals)
   document.getElementById('modal-qa').style.display = 'none';
   // Remove settings buttons if present

package/dashboard/js/render-pipelines.js

@@ -241,10 +241,13 @@ function _buildNodeChain(stages, run, options) {
 
   html += '</div>';
 
-  // Loop indicator
-  if (pipeline?.trigger?.cron) {
+  // Loop indicator — only for pipelines with stopWhen or condition stages (repeat-until pattern)
+  var hasStopWhen = !!pipeline?.stopWhen;
+  var hasConditionStage = (pipeline?.stages || []).some(function(s) { return s.type === 'condition'; });
+  if (hasStopWhen || hasConditionStage) {
     var runCount = (pipeline.runs || []).length;
-    html += '<div class="pl-node-loop">\u21BA Loop (' + escHtml(_cronToHuman(pipeline.trigger.cron)) + ')';
+    var cronLabel = pipeline?.trigger?.cron ? _cronToHuman(pipeline.trigger.cron) : 'until condition met';
+    html += '<div class="pl-node-loop">\u21BA Loop (' + escHtml(cronLabel) + ')';
     if (runCount > 0) html += ' \u00b7 Run ' + runCount;
     html += '</div>';
   }
@@ -271,18 +274,10 @@ function renderPipelines(pipelines) {
     const statusLabel = activeRun ? 'Running' : lastRun ? (lastRun.status === 'completed' ? 'Completed' : lastRun.status === 'failed' ? 'Failed' : lastRun.status === 'stopped' ? 'Stopped' : lastRun.status) : 'Never run';
     const trigger = p.trigger?.cron ? _cronToHuman(p.trigger.cron) : 'Manual';
 
-    // Stage flow visualization
-    var stageFlow = (p.stages || []).map(function(s) {
-      var icon = { task: '\u2699', meeting: '\uD83D\uDCAC', plan: '\uD83D\uDCCB', 'merge-prs': '\uD83D\uDD00', api: '\uD83C\uDF10', wait: '\u23F8', parallel: '\u2693', schedule: '\u23F0', condition: '\u2753' }[s.type] || '\u2022';
-      var stageStatus = activeRun?.stages?.[s.id]?.status || 'pending';
-      var color = stageStatus === 'completed' ? 'var(--green)' : stageStatus === 'running' ? 'var(--blue)' : stageStatus === 'failed' ? 'var(--red)' : stageStatus === 'waiting-human' ? 'var(--yellow)' : 'var(--muted)';
-      return '<span style="color:' + color + ';font-size:11px" title="' + escHtml(s.id) + ': ' + escHtml(s.title || s.type) + ' (' + stageStatus + ')">' + icon + ' ' + escHtml(s.id) + '</span>';
-    }).join(' <span style="color:var(--border)">\u2192</span> ');
-
-    // Build step-progress indicator for pipelines with a run
+    // Build node chain (renders for all pipelines, even never-run)
     var progressHtml = '';
     var displayRun = activeRun || lastRun;
-    if (displayRun && (p.stages || []).length > 0) {
+    if ((p.stages || []).length > 0) {
       progressHtml = _buildNodeChain(p.stages || [], displayRun, { compact: true, pipeline: p });
     }
 
@@ -330,7 +325,7 @@ function openPipelineDetail(id) {
 
   // Stage detail with progress bar
   var detailRun = activeRun || (p.runs || []).slice(-1)[0];
-  if (detailRun && (p.stages || []).length > 0) {
+  if ((p.stages || []).length > 0) {
     html += _buildNodeChain(p.stages || [], detailRun, { compact: false, pipeline: p });
   }
   // Pipeline-level monitored resources (full view in detail)

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

@@ -452,26 +452,31 @@ function openWorkItemDetail(id) {
   var artPills = '';
   var pillStyle = 'display:inline-flex;align-items:center;gap:3px;padding:2px 8px;border-radius:10px;font-size:10px;cursor:pointer;background:var(--surface2);border:1px solid var(--border);color:var(--text)';
   // Output log pill removed — raw stream-json output is not human-readable
+  var artBackFn = "pushModalBack(function(){openWorkItemDetail('" + escHtml(item.id) + "')});";
   if (arts.branch) artPills += '<span style="' + pillStyle + ';cursor:default">🌿 ' + escHtml(arts.branch) + '</span> ';
-  if (arts.plan) artPills += '<span onclick="planView(\'' + escHtml(arts.plan) + '\')" style="' + pillStyle + '">📋 Plan</span> ';
-  if (arts.prd) artPills += '<span onclick="planView(\'' + escHtml(arts.prd) + '\')" style="' + pillStyle + '">📄 PRD</span> ';
-  if (arts.sourcePlan) artPills += '<span onclick="planView(\'' + escHtml(arts.sourcePlan) + '\')" style="' + pillStyle + '">📋 Source Plan</span> ';
-  if (arts.notes && arts.notes.length > 0) arts.notes.forEach(function(n) {
-    var noteFile = (n && typeof n === 'object') ? (n.file || n) : String(n || '');
-    if (noteFile.startsWith('kb:')) {
-      var kbParts = noteFile.slice(3).split('/');
-      var kbCat = kbParts[0];
-      var kbFile = kbParts.slice(1).join('/');
-      var kbLabel = kbFile.replace(/\.md$/, '').slice(0, 30);
-      artPills += '<span onclick="kbOpenItem(\'' + escHtml(kbCat) + '\',\'' + escHtml(kbFile) + '\')" style="' + pillStyle + '">📚 ' + escHtml(kbLabel) + '</span> ';
-    } else if (noteFile.startsWith('archive:')) {
-      var archLabel = noteFile.slice(8).replace(/\.md$/, '').replace(/^\d{4}-\d{2}-\d{2}-/, '').slice(0, 30);
-      artPills += '<span onclick="openInboxNote(\'' + escHtml(noteFile.slice(8)) + '\')" style="' + pillStyle + ';opacity:0.7">📄 ' + escHtml(archLabel) + ' <span style="font-size:8px">(archived)</span></span> ';
-    } else {
-      var noteLabel = noteFile.replace(/\.md$/, '').slice(0, 30);
-      artPills += '<span onclick="openInboxNote(\'' + escHtml(noteFile) + '\')" style="' + pillStyle + '">📝 ' + escHtml(noteLabel) + '</span> ';
-    }
-  });
+  if (arts.plan) artPills += '<span onclick="' + artBackFn + 'planView(\'' + escHtml(arts.plan) + '\')" style="' + pillStyle + '">📋 Plan</span> ';
+  if (arts.prd) artPills += '<span onclick="' + artBackFn + 'planView(\'' + escHtml(arts.prd) + '\')" style="' + pillStyle + '">📄 PRD</span> ';
+  if (arts.sourcePlan) artPills += '<span onclick="' + artBackFn + 'planView(\'' + escHtml(arts.sourcePlan) + '\')" style="' + pillStyle + '">📋 Source Plan</span> ';
+  if (arts.notes && arts.notes.length > 0) {
+    var wiId = escHtml(item.id);
+    arts.notes.forEach(function(n) {
+      var noteFile = (n && typeof n === 'object') ? (n.file || n) : String(n || '');
+      var backFn = "pushModalBack(function(){openWorkItemDetail('" + wiId + "')});";
+      if (noteFile.startsWith('kb:')) {
+        var kbParts = noteFile.slice(3).split('/');
+        var kbCat = kbParts[0];
+        var kbFile = kbParts.slice(1).join('/');
+        var kbLabel = kbFile.replace(/\.md$/, '').slice(0, 30);
+        artPills += '<span onclick="' + backFn + 'kbOpenItem(\'' + escHtml(kbCat) + '\',\'' + escHtml(kbFile) + '\')" style="' + pillStyle + '">📚 ' + escHtml(kbLabel) + '</span> ';
+      } else if (noteFile.startsWith('archive:')) {
+        var archLabel = noteFile.slice(8).replace(/\.md$/, '').replace(/^\d{4}-\d{2}-\d{2}-/, '').slice(0, 30);
+        artPills += '<span onclick="' + backFn + 'openInboxNote(\'' + escHtml(noteFile.slice(8)) + '\')" style="' + pillStyle + ';opacity:0.7">📄 ' + escHtml(archLabel) + ' <span style="font-size:8px">(archived)</span></span> ';
+      } else {
+        var noteLabel = noteFile.replace(/\.md$/, '').slice(0, 30);
+        artPills += '<span onclick="' + backFn + 'openInboxNote(\'' + escHtml(noteFile) + '\')" style="' + pillStyle + '">📝 ' + escHtml(noteLabel) + '</span> ';
+      }
+    });
+  }
   if (arts.skills && arts.skills.length > 0) arts.skills.forEach(function(s) { artPills += '<span onclick="openSkill(\'' + escHtml(s) + '\',\'minions\',\'\')" style="' + pillStyle + '">⚙ ' + escHtml(s) + '</span> '; });
   if (artPills) html += field('Artifacts', '<div style="display:flex;flex-wrap:wrap;gap:4px">' + artPills + '</div>');
 

package/dashboard/js/utils.js

@@ -35,6 +35,25 @@ function _togglePinAndRefresh(key, source) {
   else if (source === 'kb') renderKnowledgeBase();
 }
 
+// Modal navigation stack — enables back button when opening a modal from another modal
+var _modalBackStack = [];
+function pushModalBack(reopenFn) {
+  _modalBackStack.push(reopenFn);
+  var btn = document.getElementById('modal-back-btn');
+  if (btn) btn.style.display = '';
+}
+function modalGoBack() {
+  var fn = _modalBackStack.pop();
+  var btn = document.getElementById('modal-back-btn');
+  if (btn && _modalBackStack.length === 0) btn.style.display = 'none';
+  if (fn) fn();
+}
+function clearModalBackStack() {
+  _modalBackStack = [];
+  var btn = document.getElementById('modal-back-btn');
+  if (btn) btn.style.display = 'none';
+}
+
 function updateModalPinBtn() {
   var btn = document.getElementById('modal-pin-btn');
   if (!btn) return;

package/dashboard/layout.html

@@ -98,6 +98,7 @@
     <div class="modal-header">
       <h3 id="modal-title">—</h3>
       <div class="modal-header-actions">
+        <button class="modal-copy" id="modal-back-btn" onclick="modalGoBack()" title="Back" style="display:none">&larr; Back</button>
         <button class="modal-copy" id="modal-edit-btn" onclick="modalToggleEdit()" title="Edit" style="display:none"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M11.013 1.427a1.75 1.75 0 012.474 0l1.086 1.086a1.75 1.75 0 010 2.474l-8.61 8.61c-.21.21-.47.364-.756.445l-3.251.93a.75.75 0 01-.927-.928l.929-3.25c.081-.286.235-.547.445-.758l8.61-8.61zM12.9 2.97L4.288 11.58l-.537 1.878 1.878-.537L14.242 4.31 12.9 2.97z"/></svg> Edit</button>
         <button class="modal-copy" id="modal-save-btn" onclick="modalSaveEdit()" title="Save" style="display:none;color:var(--green);border-color:var(--green)"><svg width="12" height="12" viewBox="0 0 16 16" fill="currentColor"><path d="M13.78 4.22a.75.75 0 010 1.06l-7.25 7.25a.75.75 0 01-1.06 0L2.22 9.28a.75.75 0 011.06-1.06L6 10.94l6.72-6.72a.75.75 0 011.06 0z"/></svg> Save</button>
         <button class="modal-copy" id="modal-cancel-edit-btn" onclick="modalCancelEdit()" title="Cancel edit" style="display:none">Cancel</button>

package/dashboard/styles.css

@@ -194,7 +194,7 @@
   .pl-progress-label { display: flex; gap: 8px; align-items: center; margin-top: 4px; font-size: 10px; }
   .pl-node-chain { display: flex; align-items: flex-start; gap: 2px; overflow-x: auto; padding: 8px 0; }
   .pl-node { display: flex; flex-direction: column; align-items: center; flex-shrink: 0; }
-  .pl-node-box { padding: 4px 8px; border-radius: 6px; border: 2px solid var(--border); background: var(--surface2); text-align: center; font-size: 10px; transition: border-color 0.2s; white-space: nowrap; }
+  .pl-node-box { padding: 4px 8px; border-radius: 6px; border: 1px solid var(--border); background: var(--surface2); text-align: center; font-size: 10px; transition: border-color 0.2s; white-space: nowrap; }
   .pl-node-box.complete { border-color: var(--green); background: rgba(63,185,80,0.08); }
   .pl-node-box.running { border-color: var(--blue); animation: plSegPulse 1.5s ease-in-out infinite; }
   .pl-node-box.failed { border-color: var(--red); background: rgba(248,81,73,0.08); }

package/package.json

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

package/pipelines/pr-review-fix-cycle.json

@@ -0,0 +1,42 @@
+{
+  "id": "pr-review-fix-cycle",
+  "title": "PR Review & Fix Cycle",
+  "description": "Visualizes the default engine flow: review a PR, fix issues if rejected, re-review until approved and build passes.",
+  "trigger": { "manual": true },
+  "stages": [
+    {
+      "id": "review-pr",
+      "type": "task",
+      "taskType": "review",
+      "title": "Review PR",
+      "description": "Review the target PR for correctness, quality, and completeness. Provide actionable feedback if changes are needed."
+    },
+    {
+      "id": "check-review-status",
+      "type": "condition",
+      "title": "Approved?",
+      "check": "allBuildsGreen",
+      "action": "stop",
+      "onMet": "Merge",
+      "onUnmet": "Fix",
+      "dependsOn": ["review-pr"]
+    },
+    {
+      "id": "fix-feedback",
+      "type": "task",
+      "taskType": "fix",
+      "title": "Fix review feedback",
+      "description": "Address all review comments and push fixes to the PR branch.",
+      "dependsOn": ["check-review-status"]
+    },
+    {
+      "id": "re-review",
+      "type": "task",
+      "taskType": "review",
+      "title": "Re-review after fix",
+      "description": "Re-review the PR after fixes have been applied. Verify all prior feedback was addressed.",
+      "dependsOn": ["fix-feedback"]
+    }
+  ],
+  "stopWhen": "allBuildsGreen"
+}