@yemi33/minions 0.1.1829 → 0.1.1831

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.1.1831 (2026-05-09)
+
+### Features
+- aggressively delete tech debt — drop adoPoll* aliases and permissionMode preflight warning (#2285)
+- hoist targetUrl in fetchAdoBuildErrorLog to avoid ReferenceError (#2284)
+- revert CC action-result Context: suffix (#2283)
+
 ## 0.1.1829 (2026-05-09)
 
 ### Fixes

package/dashboard/js/command-center.js

@@ -1039,16 +1039,15 @@ function _ccActionResultLine(action, result) {
   var type = _ccActionResultType(action, result);
   var subject = _ccActionResultSubject(action, result);
   var label = escHtml(type) + (subject ? ' <strong>' + escHtml(subject) + '</strong>' : '');
-  var contextSuffix = result && result.actionContext ? _ccActionContextSuffix(result.actionContext) : '';
   if (result && result.error) {
-    return '<li class="cc-action-feedback-row cc-action-feedback-error">&#10007; ' + label + ': ' + escHtml(result.error) + contextSuffix + '</li>';
+    return '<li class="cc-action-feedback-row cc-action-feedback-error">&#10007; ' + label + ': ' + escHtml(result.error) + '</li>';
   }
   if (result && result.warning) {
-    return '<li class="cc-action-feedback-row cc-action-feedback-warning">&#9888; ' + label + ': ' + escHtml(result.warning) + contextSuffix + '</li>';
+    return '<li class="cc-action-feedback-row cc-action-feedback-warning">&#9888; ' + label + ': ' + escHtml(result.warning) + '</li>';
   }
   if (result && result.ok) {
     var duplicate = result.duplicate ? ' <span style="color:var(--orange)">already exists</span>' : '';
-    return '<li class="cc-action-feedback-row cc-action-feedback-ok">&#10003; ' + label + duplicate + contextSuffix + '</li>';
+    return '<li class="cc-action-feedback-row cc-action-feedback-ok">&#10003; ' + label + duplicate + '</li>';
   }
   return '';
 }
@@ -1098,28 +1097,6 @@ function _ccRunServerActionSideEffects(action) {
   refresh();
 }
 
-var CC_ACTION_CONTEXT_STALE_MS = 2 * 60 * 1000;
-function _ccActionContextIsStale(ctx, nowMs) {
-  if (!ctx || !ctx.requestedAt) return false;
-  var ts = Date.parse(ctx.requestedAt);
-  if (!Number.isFinite(ts)) return false;
-  var now = Number.isFinite(nowMs) ? nowMs : Date.now();
-  return now - ts > CC_ACTION_CONTEXT_STALE_MS;
-}
-
-function _ccActionContextSuffix(ctx, nowMs) {
-  if (!ctx || typeof ctx !== 'object') return '';
-  var parts = [];
-  if (ctx.title) parts.push('action "' + escHtml(ctx.title) + '"');
-  if (ctx.request) parts.push('request "' + escHtml(ctx.request) + '"');
-  if (ctx.requestedAt) parts.push('started ' + escHtml(ctx.requestedAt));
-  if (parts.length === 0) return '';
-  var stale = _ccActionContextIsStale(ctx, nowMs)
-    ? ' <strong style="color:var(--orange)">possibly stale</strong>'
-    : '';
-  return '<div style="font-size:10px;color:var(--muted);margin-top:2px">Context: ' + parts.join(' | ') + stale + '</div>';
-}
-
 // Tag actions that the server already executed so ccExecuteAction skips the API call
 function _tagServerExecuted(actions, actionResults) {
   if (!actionResults || !Array.isArray(actionResults)) return;
@@ -1130,12 +1107,10 @@ function _tagServerExecuted(actions, actionResults) {
       if (r.id) actions[i]._serverId = r.id;
       if (r.warning) actions[i]._serverWarning = r.warning;
       if (r.duplicate) actions[i]._serverDuplicate = true;
-      if (r.actionContext) actions[i]._serverContext = r.actionContext;
       if (r.reusedFromAction !== undefined) actions[i]._serverHidden = true;
     } else if (r && r.error) {
       actions[i]._serverExecuted = true;
       actions[i]._serverError = r.error;
-      if (r.actionContext) actions[i]._serverContext = r.actionContext;
     }
     // clientExecuted: false means server didn't handle it — frontend must execute
   }
@@ -1150,8 +1125,7 @@ async function ccExecuteAction(action, targetTabId, opts) {
   if (action._serverExecuted) {
     if (action._serverHidden) return;
     if (action._serverError) {
-      status.innerHTML = '&#10007; ' + escHtml(action.type) + ' failed: ' + escHtml(action._serverError) +
-        _ccActionContextSuffix(action._serverContext);
+      status.innerHTML = '&#10007; ' + escHtml(action.type) + ' failed: ' + escHtml(action._serverError);
       status.style.color = 'var(--red)';
     } else {
       var label = action._serverId ? escHtml(action._serverId) : escHtml(action.title || action.type);
@@ -1159,8 +1133,7 @@ async function ccExecuteAction(action, targetTabId, opts) {
       var successLabel = serverActionType === 'dispatch' ? 'Dispatched' : serverActionType;
       status.innerHTML = '&#10003; ' + escHtml(successLabel) + ': <strong>' + label + '</strong>' +
         (action._serverDuplicate ? '<div style="font-size:10px;color:var(--orange);margin-top:2px">Already existed from a previous request; no duplicate work item was created.</div>' : '') +
-        (action._serverWarning ? '<div style="font-size:10px;color:var(--muted);margin-top:2px">' + escHtml(action._serverWarning) + '</div>' : '') +
-        _ccActionContextSuffix(action._serverContext);
+        (action._serverWarning ? '<div style="font-size:10px;color:var(--muted);margin-top:2px">' + escHtml(action._serverWarning) + '</div>' : '');
       status.style.color = 'var(--green)';
     }
     if (opts && opts.originMessageId && _ccAppendHtmlToMessage(targetTabId || _ccActiveTabId, opts.originMessageId, status.outerHTML)) {

package/dashboard/js/settings.js

@@ -114,8 +114,8 @@ async function openSettings() {
         settingsToggle('Auto-fix Human Comments', 'set-autoFixHumanComments', e.autoFixHumanComments !== false, 'Shared dispatch gate: fix agent for actionable human PR comments; also requires that PR provider polling is enabled') +
       '</div>' +
       '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-top:10px">' +
-        settingsField('PR Status Poll Frequency', 'set-prPollStatusEvery', e.prPollStatusEvery ?? e.adoPollStatusEvery ?? 12, 'ticks', 'Poll PR build/review/merge status every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
-        settingsField('PR Comments Poll Frequency', 'set-prPollCommentsEvery', e.prPollCommentsEvery ?? e.adoPollCommentsEvery ?? 12, 'ticks', 'Poll PR human comments every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
+        settingsField('PR Status Poll Frequency', 'set-prPollStatusEvery', e.prPollStatusEvery ?? 12, 'ticks', 'Poll PR build/review/merge status every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
+        settingsField('PR Comments Poll Frequency', 'set-prPollCommentsEvery', e.prPollCommentsEvery ?? 12, 'ticks', 'Poll PR human comments every N ticks for both ADO and GitHub (~12 min at default tick rate)') +
       '</div>' +
     '</div>' +
 

package/dashboard.js

@@ -2281,34 +2281,6 @@ function _actionsWithIntentFallback(actions, opts = {}) {
   return [action];
 }
 
-function _ccActionContextValue(action) {
-  if (!action || typeof action !== 'object') return '';
-  const candidates = [action.title, action.id, action.file, action.target, action.pr, action.endpoint];
-  for (const value of candidates) {
-    const cleaned = _ccCleanIntentString(value, 160);
-    if (cleaned) return cleaned;
-  }
-  return '';
-}
-
-function _annotateCCActionResults(actions, actionResults, opts = {}) {
-  if (!Array.isArray(actionResults)) return actionResults;
-  const requestedAt = _ccCleanIntentString(opts.requestedAt || new Date().toISOString(), 80);
-  const request = _ccCleanIntentString(opts.message, 180);
-  return actionResults.map((result, idx) => {
-    if (!result || typeof result !== 'object') return result;
-    const action = Array.isArray(actions) ? normalizeCCAction(actions[idx]) : null;
-    const context = {
-      type: _ccCleanIntentString(result.type || action?.type || 'action', 80) || 'action',
-    };
-    const title = _ccActionContextValue(action);
-    if (title) context.title = title;
-    if (request) context.request = request;
-    if (requestedAt) context.requestedAt = requestedAt;
-    return { ...result, actionContext: context };
-  });
-}
-
 function parseCCActions(text) {
   let actions = [];
   let displayText = stripCCActionsForDisplay(text);
@@ -7312,7 +7284,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     try {
       const body = await readBody(req);
       if (!body.message) return jsonReply(res, 400, { error: 'message required' });
-      const actionRequestedAt = new Date().toISOString();
 
       // Per-tab concurrency guard
       tabId = body.tabId || 'default';
@@ -7368,11 +7339,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           { message: body.message, intentMetadata: body.intentMetadata, source: 'command-center', answerText: parsed.text, toolUses }
         );
         if (parsed.actions.length > 0) {
-          parsed.actionResults = _annotateCCActionResults(
-            parsed.actions,
-            await executeCCActions(parsed.actions),
-            { message: body.message, requestedAt: actionRequestedAt }
-          );
+          parsed.actionResults = await executeCCActions(parsed.actions);
           parsed.actionResultsAt = new Date().toISOString();
         }
         // Mirror only user-facing text to Teams; never send the internal action block.
@@ -7477,7 +7444,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const body = await readBody(req);
       if (!body.message && !body.reconnect) { res.statusCode = 400; res.end('message required'); return; }
       tabId = body.tabId || 'default';
-      const actionRequestedAt = new Date().toISOString();
       if (body.reconnect) {
         const live = _getCcLiveStream(tabId);
         if (!live) { res.statusCode = 409; res.end('No live command-center response to reconnect'); return; }
@@ -7707,11 +7673,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         let actionResults;
         let actionResultsAt;
         if (actions.length > 0) {
-          actionResults = _annotateCCActionResults(
-            actions,
-            await executeCCActions(actions),
-            { message: body.message, requestedAt: actionRequestedAt }
-          );
+          actionResults = await executeCCActions(actions);
           actionResultsAt = new Date().toISOString();
         }
         const donePayload = { type: 'done', text: displayText, actions, actionResults, actionResultsAt, sessionId: responseSessionId, newSession: !wasResume };
@@ -8018,8 +7980,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const config = queries.getConfig();
       const routing = safeRead(path.join(MINIONS_DIR, 'routing.md')) || '';
       const engine = { ...shared.ENGINE_DEFAULTS, ...(config.engine || {}) };
-      if (engine.prPollStatusEvery === undefined && engine.adoPollStatusEvery !== undefined) engine.prPollStatusEvery = engine.adoPollStatusEvery;
-      if (engine.prPollCommentsEvery === undefined && engine.adoPollCommentsEvery !== undefined) engine.prPollCommentsEvery = engine.adoPollCommentsEvery;
       return jsonReply(res, 200, {
         engine,
         claude: settingsClaudeConfig(config),
@@ -8071,8 +8031,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (body.engine) {
         const e = body.engine;
         const D = shared.ENGINE_DEFAULTS;
-        if (e.prPollStatusEvery === undefined && e.adoPollStatusEvery !== undefined) e.prPollStatusEvery = e.adoPollStatusEvery;
-        if (e.prPollCommentsEvery === undefined && e.adoPollCommentsEvery !== undefined) e.prPollCommentsEvery = e.adoPollCommentsEvery;
         // Numeric fields: { key: [min, max?] }
         const numericFields = {
           tickInterval: [10000], maxConcurrent: [1, 50], inboxConsolidateThreshold: [1],
@@ -8095,8 +8053,6 @@ What would you like to discuss or change? When you're happy, say "approve" and I
             _setEngineConfig(key, val);
           }
         }
-        _deleteEngineConfig('adoPollStatusEvery');
-        _deleteEngineConfig('adoPollCommentsEvery');
         // String fields
         if (e.worktreeRoot !== undefined) _setEngineConfig('worktreeRoot', String(e.worktreeRoot || D.worktreeRoot));
 

package/docs/deprecated.json

@@ -1,36 +1,4 @@
 [
-  {
-    "id": "evaluate-work-type",
-    "summary": "evaluate work type removed — eval loop uses review exclusively",
-    "deprecated": "2026-04-02",
-    "reason": "Consolidated into review + evalLoop config. No separate evaluate dispatch.",
-    "locations": ["routing.md: evaluate row removed", "CLAUDE.md: evaluate references removed"],
-    "cleanup": "Already cleaned — routing.md and CLAUDE.md updated 2026-04-02"
-  },
-  {
-    "id": "autoReview-flag",
-    "summary": "autoReview ENGINE_DEFAULT consolidated into evalLoop",
-    "deprecated": "2026-04-15",
-    "reason": "Redundant with evalLoop; evalLoop is the single gate for the review+fix cycle",
-    "locations": ["engine/shared.js ENGINE_DEFAULTS.autoReview", "engine.js discoverFromPrs autoReview variable"],
-    "cleanup": "Removed from ENGINE_DEFAULTS, removed autoReview variable from engine.js, replaced with reviewEnabled = evalLoopEnabled && pollEnabled"
-  },
-  {
-    "id": "ado-poll-frequency-keys",
-    "summary": "adoPollStatusEvery and adoPollCommentsEvery renamed to prPollStatusEvery and prPollCommentsEvery",
-    "deprecated": "2026-04-16",
-    "reason": "These cadence settings gate both ADO and GitHub PR polling, so the ADO-prefixed names are misleading.",
-    "locations": ["engine.js read-side fallback from config.engine.adoPollStatusEvery/adoPollCommentsEvery", "dashboard.js handleSettingsRead/handleSettingsUpdate alias fallback for old keys"],
-    "cleanup": "Remove the adoPoll* alias fallback reads after existing configs have been migrated to prPollStatusEvery/prPollCommentsEvery."
-  },
-  {
-    "id": "config-claude-permission-mode",
-    "summary": "config.claude.permissionMode ignored; runtime adapters own permission bypass flags",
-    "deprecated": "2026-05-04",
-    "reason": "Claude and Copilot require different non-interactive bypass flags, so a shared Claude config field was misleading and no longer controls spawns.",
-    "locations": ["dashboard.js settings update strips config.claude.permissionMode", "dashboard/js/settings.js no longer renders a Permission Mode selector"],
-    "cleanup": "After old configs have been rewritten through settings, remove the deprecated-field preflight warning entry for permissionMode."
-  },
   {
     "id": "project-local-minions-state",
     "summary": "Project-local .minions directories are migrated to central projects/<name>/ state and removed",

package/engine/ado.js

@@ -511,8 +511,9 @@ async function fetchAdoBuildErrorLog(orgBase, project, failedStatus, token, pr,
   try {
     // Use pre-resolved buildId if available (from builds API query), else parse from targetUrl
     let buildId = failedStatus?._buildId || null;
+    let targetUrl = '';
     if (!buildId) {
-      const targetUrl = failedStatus?.targetUrl || '';
+      targetUrl = failedStatus?.targetUrl || '';
       const buildIdMatch = targetUrl.match(/buildId=(\d+)/);
       if (buildIdMatch) buildId = buildIdMatch[1];
     }

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-09T23:39:05.707Z"
+  "cachedAt": "2026-05-09T23:56:38.102Z"
 }

package/engine/shared.js

@@ -1143,7 +1143,7 @@ const ENGINE_DEFAULTS = {
   // Backward-compat: keep `engine.claude.*` field family deprecation tracker. Listed here so preflight
   // knows which subkeys to flag as deprecated. Do not consume `claude.*` in new code — use the runtime
   // adapter system (engine/runtimes/) and the resolveAgent*/resolveCc* helpers instead.
-  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'permissionMode', 'maxTurns', 'effort', 'budgetCap'],
+  _deprecatedConfigClaudeFields: ['binary', 'outputFormat', 'allowedTools', 'maxTurns', 'effort', 'budgetCap'],
   // Teams integration — config.teams shape: { enabled, appId, appPassword, certPath, privateKeyPath, tenantId, notifyEvents, ccMirror, inboxPollInterval }
   // Auth modes: (1) appId + appPassword (client secret), or (2) appId + certPath + privateKeyPath + tenantId (certificate)
   teams: {

package/engine.js

@@ -4427,11 +4427,11 @@ async function tickInner() {
   const ghPollEnabled = config.engine?.ghPollEnabled ?? ENGINE_DEFAULTS.ghPollEnabled;
   const prPollStatusEvery = Math.max(
     1,
-    Number(config.engine?.prPollStatusEvery ?? config.engine?.adoPollStatusEvery) || ENGINE_DEFAULTS.prPollStatusEvery
+    Number(config.engine?.prPollStatusEvery) || ENGINE_DEFAULTS.prPollStatusEvery
   );
   const prPollCommentsEvery = Math.max(
     1,
-    Number(config.engine?.prPollCommentsEvery ?? config.engine?.adoPollCommentsEvery) || ENGINE_DEFAULTS.prPollCommentsEvery
+    Number(config.engine?.prPollCommentsEvery) || ENGINE_DEFAULTS.prPollCommentsEvery
   );
 
   // 2.6. Poll PR status: build, review, merge (every prPollStatusEvery tick-equivalents, default ~12 minutes)

package/package.json

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