@yemi33/minions 0.1.1918 → 0.1.1920

package/dashboard/js/settings.js

@@ -265,8 +265,7 @@ async function openSettings() {
     '</div>' +
 
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Claude CLI</h3>' +
-    '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:8px">' +
-      settingsField('Output Format', 'set-outputFormat', c.outputFormat || 'stream-json', '', '') +
+    '<div style="display:grid;grid-template-columns:1fr;gap:8px;margin-bottom:8px">' +
       settingsField('Allowed Tools', 'set-allowedTools', c.allowedTools || '', '', 'Claude allow-list passed through for compatibility; runtime bypass flags are adapter-owned.') +
     '</div>' +
     '<div style="font-size:10px;color:var(--muted);margin-bottom:16px;padding:6px 8px;border:1px solid var(--border);border-radius:4px;background:var(--surface-subtle, rgba(130,160,210,0.08))">' +
@@ -635,7 +634,6 @@ async function saveSettings() {
     };
 
     const claudePayload = {
-      outputFormat: document.getElementById('set-outputFormat').value,
       allowedTools: document.getElementById('set-allowedTools').value,
     };
 

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-13T22:31:23.464Z"
+  "cachedAt": "2026-05-13T22:41:19.532Z"
 }

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/engine/shared.js

@@ -1816,7 +1816,11 @@ function pruneDefaultClaudeConfig(config) {
   };
 
   removeKey('permissionMode');
-  for (const key of ['binary', 'outputFormat', 'allowedTools']) {
+  // outputFormat is fully orphan — the Claude adapter hard-codes 'stream-json'
+  // and nothing threads config.claude.outputFormat into it. Strip unconditionally
+  // so stale values stop tripping the deprecation warning.
+  removeKey('outputFormat');
+  for (const key of ['binary', 'allowedTools']) {
     if (Object.prototype.hasOwnProperty.call(claude, key) && claude[key] === DEFAULT_CLAUDE[key]) {
       removeKey(key);
     }

package/package.json

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