@yemi33/minions 0.1.1917 → 0.1.1919

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

@@ -161,6 +161,9 @@ function editWorkItem(id, source) {
       '<label style="color:var(--text);font-size:var(--text-md)">Acceptance Criteria (one per line)' +
         '<textarea id="wi-edit-ac" rows="3" 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);font-size:var(--text-md);font-family:inherit;resize:vertical">' + escapeHtml((item.acceptanceCriteria || []).join('\n')) + '</textarea>' +
       '</label>' +
+      '<label style="color:var(--text);font-size:var(--text-md)">Depends On (work-item ids, comma- or newline-separated)' +
+        '<textarea id="wi-edit-depends-on" rows="2" placeholder="W-foo, W-bar" 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);font-size:var(--text-md);font-family:inherit;resize:vertical">' + escapeHtml((Array.isArray(item.depends_on) ? item.depends_on : []).join(', ')) + '</textarea>' +
+      '</label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:8px">' +
         '<button onclick="closeModal()" class="pr-pager-btn" style="padding:6px 16px;font-size:var(--text-md)">Cancel</button>' +
         '<button onclick="submitWorkItemEdit(\'' + escapeHtml(id) + '\',\'' + escapeHtml(source || '') + '\',event)" style="padding:6px 16px;font-size:var(--text-md);background:var(--blue);color:#fff;border:none;border-radius:var(--radius-sm);cursor:pointer">Save</button>' +
@@ -183,13 +186,15 @@ async function submitWorkItemEdit(id, source, e) {
   }).filter(function(r) { return r.url; });
   const acRaw = document.getElementById('wi-edit-ac')?.value || '';
   const acceptanceCriteria = acRaw.split('\n').filter(function(l) { return l.trim(); });
+  const dependsRaw = document.getElementById('wi-edit-depends-on')?.value || '';
+  const depends_on = dependsRaw.split(/[\n,]/).map(function(s) { return s.trim(); }).filter(Boolean);
   if (!title) { if (btn) { btn.disabled = false; btn.textContent = 'Save'; } alert('Title is required'); return; }
   try { closeModal(); } catch { /* may not be open */ }
   showToast('cmd-toast', 'Work item updated', true);
   try {
     const res = await fetch('/api/work-items/update', {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ id, source: source || undefined, title, description, type, priority, agent, references, acceptanceCriteria })
+      body: JSON.stringify({ id, source: source || undefined, title, description, type, priority, agent, references, acceptanceCriteria, depends_on })
     });
     if (res.ok) { refresh(); } else { const d = await res.json().catch(() => ({})); alert('Update failed: ' + (d.error || 'unknown')); editWorkItem(id, source); }
   } catch (e) { alert('Update error: ' + e.message); editWorkItem(id, source); }
@@ -375,6 +380,7 @@ function openCreateWorkItemModal() {
         '<label style="flex:1;color:var(--text);font-size:var(--text-md)">Project <select id="wi-new-project" style="' + inputStyle + '"><option value="">Central</option>' + projOpts + '</select></label>' +
       '</div>' +
       '<label style="color:var(--text);font-size:var(--text-md)">Acceptance Criteria <textarea id="wi-new-ac" rows="2" style="' + inputStyle + ';resize:vertical" placeholder="One criterion per line (optional)"></textarea></label>' +
+      '<label style="color:var(--text);font-size:var(--text-md)">Depends On <textarea id="wi-new-depends-on" rows="2" style="' + inputStyle + ';resize:vertical" placeholder="W-foo, W-bar — comma- or newline-separated work-item ids (optional)"></textarea></label>' +
       '<label style="color:var(--text);font-size:var(--text-md)">References <textarea id="wi-new-refs" rows="2" style="' + inputStyle + ';resize:vertical" placeholder="url | title | type — one per line (optional)"></textarea></label>' +
       '<label id="wi-new-skippr-row" style="color:var(--text);font-size:var(--text-md);display:flex;gap:8px;align-items:center;cursor:pointer"><input type="checkbox" id="wi-new-skippr"> Skip PR creation (push branch only)</label>' +
       '<div style="display:flex;justify-content:flex-end;gap:8px;margin-top:4px">' +
@@ -406,6 +412,8 @@ async function _submitCreateWorkItem(e) {
   const project = document.getElementById('wi-new-project')?.value || '';
   const acRaw = document.getElementById('wi-new-ac')?.value || '';
   const acceptanceCriteria = acRaw.split('\n').map(l => l.trim()).filter(Boolean);
+  const dependsRaw = document.getElementById('wi-new-depends-on')?.value || '';
+  const depends_on = dependsRaw.split(/[\n,]/).map(s => s.trim()).filter(Boolean);
   const refsRaw = document.getElementById('wi-new-refs')?.value || '';
   const references = refsRaw.split('\n').filter(l => l.trim()).map(l => {
     const parts = l.split('|').map(s => s.trim());
@@ -418,6 +426,7 @@ async function _submitCreateWorkItem(e) {
     if (project) body.project = project;
     if (acceptanceCriteria.length) body.acceptanceCriteria = acceptanceCriteria;
     if (references.length && references[0].url) body.references = references;
+    if (depends_on.length) body.depends_on = depends_on;
     const skipPr = document.getElementById('wi-new-skippr')?.checked || false;
     if (skipPr) body.skipPr = true;
 

package/dashboard.js

@@ -3779,6 +3779,10 @@ const server = http.createServer(async (req, res) => {
     try {
       const body = await readBody(req);
       if (!body.title || !body.title.trim()) return jsonReply(res, 400, { error: 'title is required' });
+      if (body.depends_on !== undefined) {
+        if (!Array.isArray(body.depends_on)) return jsonReply(res, 400, { error: 'depends_on must be an array of strings' });
+        if (!body.depends_on.every(s => typeof s === 'string')) return jsonReply(res, 400, { error: 'depends_on entries must be strings' });
+      }
       const target = resolveWorkItemsCreateTarget(body.project);
       if (target.error) return jsonReply(res, 400, { error: target.error });
       const wiPath = target.wiPath;
@@ -3789,6 +3793,9 @@ const server = http.createServer(async (req, res) => {
         priority: body.priority || 'medium', description: body.description || '',
         status: WI_STATUS.PENDING, created: new Date().toISOString(), createdBy: 'dashboard',
       };
+      if (Array.isArray(body.depends_on)) {
+        item.depends_on = body.depends_on.map(s => s.trim()).filter(Boolean);
+      }
       if (targetProject) item.project = targetProject.name;
       if (body.scope) item.scope = body.scope;
       // Agent assignment normalization: `agent` and `agents` are routing hints.
@@ -3832,6 +3839,10 @@ const server = http.createServer(async (req, res) => {
       const body = await readBody(req);
       const { id, source, title, description, type, priority, agent } = body;
       if (!id) return jsonReply(res, 400, { error: 'id required' });
+      if (body.depends_on !== undefined) {
+        if (!Array.isArray(body.depends_on)) return jsonReply(res, 400, { error: 'depends_on must be an array of strings' });
+        if (!body.depends_on.every(s => typeof s === 'string')) return jsonReply(res, 400, { error: 'depends_on entries must be strings' });
+      }
 
       const target = resolveProjectSourceTarget(source, PROJECTS);
       if (target.error) return jsonReply(res, 404, { error: target.error });
@@ -3855,6 +3866,9 @@ const server = http.createServer(async (req, res) => {
         }
         if (body.references !== undefined) item.references = body.references;
         if (body.acceptanceCriteria !== undefined) item.acceptanceCriteria = body.acceptanceCriteria;
+        if (Array.isArray(body.depends_on)) {
+          item.depends_on = body.depends_on.map(s => s.trim()).filter(Boolean);
+        }
         item.updatedAt = new Date().toISOString();
         result = { code: 200, body: { ok: true, item } };
         return items;

package/engine/features.js

@@ -3,6 +3,23 @@
  */
 
 // Entry shape: id → { description, default: bool, addedIn?: version, expires?: ISO-date }
+//
+// INTENTIONAL EMPTY REGISTRY — DO NOT DELETE.
+//
+// The `FEATURES = {}` literal below is intentional scaffolding, not dead code.
+// The surrounding framework (isFeatureOn, listFeatures, hasFeature, env-var
+// override resolver, expiration timestamps) is load-bearing: dashboard.js
+// boot wires it into /api/features (list), /api/features/toggle, and the
+// `window.MINIONS_FEATURES` client bootstrap. Deleting the registry — even
+// while empty — would break dashboard startup and the experimental-flags UI.
+//
+// New flags belong here. Register them in this object instead of removing
+// the empty literal. The first real entry will replace the example below;
+// until then the empty object is the correct, expected shape.
+//
+// Decision logged in the 2026-05-13 daily architecture & bug review meeting
+// (knowledge/architecture/2026-05-13-ripley-meeting-conclusion-daily-architecture-bug-review-2.md,
+// PR-C, Option B — keep framework, document the empty registry).
 const FEATURES = {
   // Example:
   // 'ux-sidebar-v2': { description: '…', default: false, addedIn: '0.1.1738', expires: '2026-06-01' },

package/engine/lifecycle.js

@@ -2597,6 +2597,12 @@ function parseAgentOutput(stdout, runtimeName) {
  * Agents produce a ```completion fenced block with key: value pairs.
  * Returns parsed object or null if not found / malformed.
  * If multiple blocks exist, the last one wins (agent may retry).
+ *
+ * DEPRECATED — slated for removal once telemetry shows zero hits over a 14-day
+ * window. Telemetry: see _engine.completionFallbacks in metrics.json (the JSON
+ * sidecar at MINIONS_COMPLETION_REPORT is the documented contract; this fenced
+ * fallback exists only to support runtimes/agents that have not yet adopted it).
+ * Removal is tracked by the P-c8f5e1b3 follow-up plan.
  */
 function parseStructuredCompletion(stdout, runtimeName) {
   if (!stdout || typeof stdout !== 'string') return null;
@@ -2692,6 +2698,13 @@ function parseCompletionKeyValues(text) {
   return result;
 }
 
+/**
+ * DEPRECATED — slated for removal once telemetry shows zero hits over a 14-day
+ * window. Telemetry: see _engine.completionFallbacks in metrics.json (the JSON
+ * sidecar at MINIONS_COMPLETION_REPORT is the documented contract; this prose
+ * fallback only fires when both the sidecar AND the fenced ```completion block
+ * are missing). Removal is tracked by the P-c8f5e1b3 follow-up plan.
+ */
 function parseCompletionFieldSummary(text) {
   if (!text || typeof text !== 'string') return null;
 
@@ -3203,6 +3216,28 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
   const summaryCompletion = reportCompletion || fencedCompletion ? null : parseCompletionFieldSummary(resultSummary);
   const fallbackCompletion = fencedCompletion || summaryCompletion;
   const fallbackSource = fencedCompletion && hasCompletionFence(stdout, runtimeName) ? 'fenced-completion' : 'summary-completion';
+  // P-c8f5e1b3 — telemetry: completion-fallback. Emit a grep-able log + bump a
+  // counter in metrics.json whenever the fenced or summary fallback fires (i.e.
+  // the documented sidecar contract was missing). Used to confirm zero adoption
+  // gaps before removing parseStructuredCompletion / parseCompletionFieldSummary
+  // in a follow-up. Sidecar (happy) path is silent — no log, no increment.
+  if (!reportCompletion && fallbackCompletion) {
+    const counterKey = fallbackSource === 'fenced-completion' ? 'fenced' : 'summary';
+    const wiId = dispatchItem.meta?.item?.id || null;
+    log('info', `telemetry: completion-fallback agent=${agentId || 'unknown'} wi=${wiId || 'N/A'} runtime=${runtimeName} source=${fallbackSource}`);
+    try {
+      const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
+      mutateJsonFileLocked(metricsPath, (metrics) => {
+        metrics = metrics || {};
+        if (!metrics._engine) metrics._engine = {};
+        if (!metrics._engine.completionFallbacks) metrics._engine.completionFallbacks = { fenced: 0, summary: 0 };
+        metrics._engine.completionFallbacks[counterKey] = (metrics._engine.completionFallbacks[counterKey] || 0) + 1;
+        return metrics;
+      });
+    } catch (err) {
+      log('warn', `telemetry: completion-fallback metrics write failed: ${err.message}`);
+    }
+  }
   const structuredCompletion = reportCompletion || persistCompletionReport(dispatchItem, fallbackCompletion, fallbackSource);
   if (structuredCompletion) {
     if (structuredCompletion.summary) resultSummary = String(structuredCompletion.summary);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1917",
+  "version": "0.1.1919",
   "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/engine/copilot-models.json

@@ -1,5 +0,0 @@
-{
-  "runtime": "copilot",
-  "models": null,
-  "cachedAt": "2026-05-13T22:15:26.685Z"
-}