@yemi33/minions 0.1.687 → 0.1.688

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.687 (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
 
@@ -10,6 +11,9 @@
 -  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/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/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.687",
+  "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"
+}