@yemi33/minions 0.1.1835 → 0.1.1837

package/dashboard.js

@@ -1602,13 +1602,6 @@ function renderDocChatSystemPromptForTurn(turnId) {
     .replace(/\{\{dashboard_port\}\}/g, String(PORT));
 }
 
-// Doc-chat document rewrite delimiter constants. Kept for backwards-compat
-// while the system migrates to using Edit/Write tools directly (see
-// prompts/cc-system.md "Document edits" section). Once CC always uses
-// Edit/Write, these can be retired.
-const DOC_CHAT_DOCUMENT_DELIMITER = '---MINIONS-DOC-CHAT-DOCUMENT-v1-6f2f90e3---';
-const LEGACY_DOC_CHAT_DOCUMENT_DELIMITER = '---DOCUMENT---';
-
 // Per-CC-turn correlation. CC's tool calls (Bash curl to /api/...) attach an
 // X-CC-Turn-Id header; the mutating endpoints record creations into this map
 // keyed by the turn ID so the CC handler can surface them as confirmation
@@ -1700,6 +1693,25 @@ function _buildSyntheticActionResultsForTurn(turnId, message, requestedAt) {
   return { actions, results };
 }
 
+// Cheap mtime+size snapshot used by doc-chat to detect whether CC's `Edit` /
+// `Write` tool calls actually changed the target file during a turn. A diff
+// triggers a "Document saved" chip via the same _ccTurnCreations surface used
+// by mutating /api/* endpoints.
+function _snapshotDocFile(fullPath) {
+  if (!fullPath) return null;
+  try {
+    const st = fs.statSync(fullPath);
+    return { mtimeMs: st.mtimeMs, size: st.size };
+  } catch { return { mtimeMs: 0, size: 0 }; } // missing file → all changes look like edits
+}
+
+function _docFileChanged(before, fullPath) {
+  if (!fullPath) return false;
+  const after = _snapshotDocFile(fullPath);
+  if (!before || !after) return false;
+  return before.mtimeMs !== after.mtimeMs || before.size !== after.size;
+}
+
 function _ccTurnEntryToActionType(kind) {
   switch (kind) {
     case 'work-item': return 'dispatch';
@@ -2073,135 +2085,25 @@ For all state files, look under \`${MINIONS_DIR}\`.${indexSection}`;
   return result;
 }
 
-function findCCActionsDelimiter(text) {
-  const header = findCCActionsHeader(text);
-  return header && header.parseable ? header.index : -1;
-}
-
-// Single helper that handles both the strict (well-formed) and loose forms of
-// the ===ACTIONS=== delimiter. `parseable` is true only for the strict form
-// that parseCCActions can JSON.parse; loose matches still split display text
-// but are not parsed (they shouldn't reach the user as actions).
-function findCCActionsHeader(text) {
-  if (!text) return null;
-  // Tier 1 — strict, parseable: the exact canonical delimiter on its own line.
-  const strict = /(?:^|\r?\n)===ACTIONS===[ \t]*(?=\r?\n|$)/m.exec(text);
-  if (strict) {
-    const headerStart = strict.index + strict[0].indexOf('===ACTIONS');
-    const headerMatch = text.slice(headerStart).match(/^===ACTIONS===[ \t]*/);
-    return {
-      index: headerStart,
-      headerLength: headerMatch ? headerMatch[0].length : '===ACTIONS==='.length,
-      parseable: true,
-    };
-  }
-  // Tier 2 — loose: sentinel-looking malformed delimiters such as ===ACTIONS ->
-  // should still be hidden, but prose like "===ACTIONS are documented" must render.
-  const loose = /(?:^|\r?\n)===ACTIONS(?:[ \t]*(?:[-=]>?|={1,}|$)|[^A-Za-z0-9_\s\r\n][^\r\n]*)(?=\r?\n|$)/m.exec(text);
-  if (loose) {
-    const headerStart = loose.index + loose[0].indexOf('===ACTIONS');
-    return { index: headerStart, headerLength: 0, parseable: false };
-  }
-  // Tier 3 — very loose: 2+ leading equals, optional whitespace, ACTIONS keyword
-  // (case-insensitive), optional trailing equals. Catches the common model
-  // fumbles: ====ACTIONS===, ===actions===, ===ACTIONS=====, ==ACTIONS==. Must
-  // still be a complete line (preceded by line start, followed by EOL or EOS)
-  // so prose like "the answer is 2 + 3 = 5" or "==Important==" doesn't match.
-  const veryLoose = /(?:^|\r?\n)={2,}[ \t]*ACTIONS[ \t]*={0,}[ \t]*(?=\r?\n|$)/im.exec(text);
-  if (veryLoose) {
-    // Skip the leading newline (if any) so headerStart points to first '='.
-    const offset = veryLoose[0].search(/=/);
-    const headerStart = veryLoose.index + (offset >= 0 ? offset : 0);
-    return { index: headerStart, headerLength: 0, parseable: false };
-  }
-  return null;
-}
+// The ===ACTIONS=== delimiter parser tiers (findCCActionsHeader,
+// findCCActionsPartialDelimiter, stripCCActionsForStream/Display) and the
+// _extractActionsJson Copilot fence-stripper were retired with the move to
+// direct /api/* tool calls. Action surfacing now flows entirely through the
+// X-CC-Turn-Id header → _ccTurnCreations → _buildSyntheticActionResultsForTurn.
 
-function findCCActionsPartialDelimiter(text) {
-  if (!text) return -1;
-  const delimiter = '===ACTIONS===';
-  const lineStart = Math.max(text.lastIndexOf('\n'), text.lastIndexOf('\r')) + 1;
-  const trailingLine = text.slice(lineStart).trimEnd();
-  if (!trailingLine) return -1;
-  // Pure "=" run of any length 1+ is a likely partial of any delimiter tier.
-  // The next chunk arrives within milliseconds and the strip is restored or
-  // completed — false-positives at chunk EOL self-heal.
-  if (/^=+$/.test(trailingLine)) return lineStart;
-  // Strict prefix of the canonical "===ACTIONS===" (case-insensitive so
-  // "===act" and "===ACT" both strip).
-  if (trailingLine.length >= 1 && trailingLine.length < delimiter.length
-      && delimiter.toLowerCase().startsWith(trailingLine.toLowerCase())) {
-    return lineStart;
-  }
-  return -1;
-}
-
-function stripCCActionsForStream(text) {
-  if (!text) return '';
-  // Fast path: 95% of streamed chunks contain no '=' before any actions block
-  // appears. Skip all regex work in that case.
-  if (text.indexOf('===') < 0) return text;
-  const header = findCCActionsHeader(text);
-  if (header) return text.slice(0, header.index).trim();
-  const partialIdx = findCCActionsPartialDelimiter(text);
-  if (partialIdx >= 0) return text.slice(0, partialIdx).trimEnd();
-  return text;
-}
-
-function stripCCActionsForDisplay(text) {
-  if (!text) return '';
-  const header = findCCActionsHeader(text);
-  if (header) return text.slice(0, header.index).trim();
-  const partialIdx = findCCActionsPartialDelimiter(text);
-  if (partialIdx >= 0) return text.slice(0, partialIdx).trimEnd();
-  return text;
-}
-
-// Issue #1834: non-Claude runtimes (Copilot/GPT) routinely wrap the action JSON
-// in ```json fences or append trailing prose ("Let me know if that helps!").
-// JSON.parse on the raw segment fails silently → actions dropped, user sees
-// inert text. This extractor pulls out the balanced JSON value (array or
-// object) regardless of fences, leading whitespace, or trailing junk so the
-// downstream parse can succeed. Returns null if no plausible JSON value is
-// present (caller surfaces the failure via _actionParseError).
-function _extractActionsJson(segment) {
-  if (!segment) return null;
-  let body = segment.trim();
-  // Strip ```json / ``` fences (open + close). The model sometimes only emits
-  // an opening fence (truncation), so handle both halves independently.
-  body = body.replace(/^```[a-zA-Z0-9_-]*\s*\r?\n?/, '').replace(/\r?\n?```\s*$/, '').trim();
-  if (!body) return null;
-  const first = body.indexOf('[');
-  const firstObj = body.indexOf('{');
-  let start = -1;
-  let openCh = '';
-  let closeCh = '';
-  if (first >= 0 && (firstObj < 0 || first <= firstObj)) {
-    start = first; openCh = '['; closeCh = ']';
-  } else if (firstObj >= 0) {
-    start = firstObj; openCh = '{'; closeCh = '}';
-  }
-  if (start < 0) return null;
-  let depth = 0;
-  let inString = false;
-  let escape = false;
-  for (let i = start; i < body.length; i++) {
-    const ch = body[i];
-    if (escape) { escape = false; continue; }
-    if (ch === '\\') { escape = true; continue; }
-    if (ch === '"') { inString = !inString; continue; }
-    if (inString) continue;
-    if (ch === openCh) depth++;
-    else if (ch === closeCh) {
-      depth--;
-      if (depth === 0) return body.slice(start, i + 1);
-    }
+function normalizeMeetingParticipants(participants) {
+  if (!Array.isArray(participants)) return [];
+  const seen = new Set();
+  const normalized = [];
+  for (const participant of participants) {
+    const id = String(participant || '').trim();
+    if (!id || seen.has(id)) continue;
+    seen.add(id);
+    normalized.push(id);
   }
-  return null;
+  return normalized;
 }
 
-const CC_DISPATCH_ACTION_ALIASES = new Set(['fix', 'explore', 'review', 'test', 'implement', 'implement:large', 'ask', 'verify']);
-const CC_ACTION_INTENT_WORK_TYPES = new Set(['fix', 'implement', 'implement:large', 'explore', 'review', 'test', 'ask', 'verify']);
 
 function getWorkItemPrRef(input) {
   if (!input || typeof input !== 'object') return null;
@@ -2229,1677 +2131,6 @@ function extractPrRefFromText(value) {
   return numberMatch ? numberMatch[1] : null;
 }
 
-function normalizeCCAction(action) {
-  if (!action || typeof action !== 'object') return action;
-  if (typeof action.type !== 'string') return action;
-  const type = action.type.trim().toLowerCase();
-  if (type === 'dispatch') {
-    return action.type === 'dispatch' ? action : { ...action, type: 'dispatch' };
-  }
-  if (!CC_DISPATCH_ACTION_ALIASES.has(type)) return action;
-  return { ...action, type: 'dispatch', workType: action.workType || type };
-}
-
-function _ccCleanIntentString(value, max = 500) {
-  if (typeof value !== 'string') return '';
-  let out = '';
-  let lastWasSpace = true;
-  for (const ch of value) {
-    const isSpace = ch === '\0' || ch === ' ' || ch === '\n' || ch === '\r' || ch === '\t';
-    if (isSpace) {
-      if (!lastWasSpace) out += ' ';
-      lastWasSpace = true;
-    } else {
-      out += ch;
-      lastWasSpace = false;
-    }
-  }
-  return out.trim().slice(0, max);
-}
-
-function _ccNormalizeIntentMetadata(meta) {
-  if (!meta || typeof meta !== 'object' || Array.isArray(meta)) return {};
-  const out = {};
-  for (const key of ['intent', 'type', 'title', 'description', 'priority', 'project', 'branchStrategy', 'branch_strategy', 'pr', 'targetPr', 'prId', 'prNumber', 'pullRequest', 'sourcePr', 'prUrl']) {
-    const value = _ccCleanIntentString(meta[key], key === 'description' ? 2000 : 300);
-    if (value) out[key] = value;
-  }
-  if (Array.isArray(meta.agents)) {
-    out.agents = meta.agents.map(a => _ccCleanIntentString(a, 80)).filter(Boolean);
-  }
-  if (Array.isArray(meta.projects)) {
-    out.projects = meta.projects.map(p => _ccCleanIntentString(p, 120)).filter(Boolean);
-  }
-  if (meta.fanout === true) out.fanout = true;
-  return out;
-}
-
-function _messageExplicitlyRequestsMinionsOrchestration(message) {
-  const normalized = _normalizeIntentText(message);
-  if (!normalized.trim()) return false;
-  if (_messageHasDelegationIntent(message)) return true;
-  if (_intentHasAnyToken(normalized, ['minions']) &&
-      (_messageHasMediumLargeWorkIntent(message) || _messageRequestsPlanIntent(message))) return true;
-  if (_intentHasAnyToken(normalized, ['agent', 'agents']) && _messageHasMediumLargeWorkIntent(message)) return true;
-  return _intentHasVerbObject(normalized, ['create', 'make', 'draft', 'write'], ['plan']) &&
-    _intentHasAnyToken(normalized, ['minion', 'minions']);
-}
-
-function _ccInferMessageActionIntent(message, source = 'command-center') {
-  const normalized = _normalizeIntentText(message);
-  if (!normalized.trim()) return null;
-  if (source === 'doc-chat' && !_messageExplicitlyRequestsMinionsOrchestration(message)) return null;
-  if (_messageRequestsPlanIntent(message)) {
-    return { kind: 'plan' };
-  }
-  if (!_messageHasDelegationIntent(message) && !_messageHasMediumLargeWorkIntent(message, { source })) return null;
-  return { kind: 'work-item', workType: _inferDelegatedWorkType(message) };
-}
-
-function _ccInferMetadataActionIntent(meta) {
-  if (meta.intent === 'plan') return { kind: 'plan' };
-  if (meta.intent === 'work-item' && CC_ACTION_INTENT_WORK_TYPES.has(String(meta.type || '').toLowerCase())) {
-    return { kind: 'work-item', workType: String(meta.type).toLowerCase() };
-  }
-  return null;
-}
-
-function _ccActionIntentTargetText(message, intent) {
-  const tokens = _intentTokens(_normalizeIntentText(_ccCleanIntentString(message, 500)));
-  let start = 0;
-  const skip = (terms) => {
-    if (terms.includes(tokens[start])) {
-      start++;
-      return true;
-    }
-    return false;
-  };
-  skip(['please']);
-  if (tokens[start] && tokens[start].startsWith('/')) start++;
-  if (['dispatch', 'delegate', 'assign', 'queue', 'enqueue'].includes(tokens[start])) {
-    start++;
-    while (['a', 'an', 'the', 'work', 'item', 'task', 'agent', 'to'].includes(tokens[start])) start++;
-  } else if (['create', 'open', 'file'].includes(tokens[start]) && tokens[start + 1] && ['a', 'an', 'the', 'work', 'item', 'task'].includes(tokens[start + 1])) {
-    start++;
-    while (['a', 'an', 'the', 'work', 'item', 'task', 'to', 'for'].includes(tokens[start])) start++;
-  } else if (['have', 'ask', 'tell'].includes(tokens[start])) {
-    start++;
-    while (tokens[start] && !['to', 'fix', 'implement', 'review', 'test', 'explore', 'investigate', 'audit'].includes(tokens[start])) start++;
-    if (tokens[start] === 'to') start++;
-  }
-  if (intent?.kind === 'plan') {
-    while (['make', 'create', 'draft', 'write', 'design', 'plan', 'out', 'a', 'an', 'the', 'for', 'to', 'about', 'how'].includes(tokens[start])) start++;
-  } else if (intent?.workType) {
-    while (['fix', 'repair', 'resolve', 'patch', 'debug', 'implement', 'build', 'add', 'create', 'ship', 'review', 'inspect', 'audit', 'test', 'verify', 'validate', 'run', 'write', 'explore', 'investigate', 'research', 'analyze', 'analyse', 'understand', 'look', 'map', 'survey', 'a', 'an', 'the', 'for', 'to', 'of', 'on', 'about'].includes(tokens[start])) start++;
-  }
-  while (['for', 'to', 'of', 'on', 'about', 'a', 'an', 'the'].includes(tokens[start])) start++;
-  return tokens.slice(start).join(' ');
-}
-
-function _ccIntentHasConcreteTarget(message, metadataTitle, intent) {
-  const raw = String(message || '');
-  const normalized = _normalizeIntentText(raw);
-  const tokens = _intentTokens(normalized);
-  if (raw.includes('http://') || raw.includes('https://')) return true;
-  for (let i = 0; i < tokens.length; i++) {
-    if (['pr', 'issue'].includes(tokens[i]) && tokens[i + 1] && _intentTokenIsNumeric(tokens[i + 1])) return true;
-    if (tokens[i] === 'pull' && tokens[i + 1] === 'request' && tokens[i + 2] && _intentTokenIsNumeric(tokens[i + 2])) return true;
-    if (tokens[i] === 'w' && tokens[i + 1] && tokens[i + 1].length >= 2) return true;
-  }
-  const target = _ccActionIntentTargetText(metadataTitle || raw, intent).trim().toLowerCase();
-  if (!target) return false;
-  const targetTokens = _intentTokens(_normalizeIntentText(target));
-  const generic = new Set(['this', 'that', 'it', 'these', 'those', 'same', 'above', 'here', 'there', 'one', 'thing', 'issue', 'bug', 'task', 'request', 'change', 'pr', 'pull', 'code', 'test', 'plan', 'doc', 'document', 'file']);
-  if (targetTokens.length > 0 && targetTokens.every(token => generic.has(token))) return false;
-  return target.length >= 3;
-}
-
-function _ccIntentTitle(message, metadataTitle, intent) {
-  const metaTitle = _ccCleanIntentString(metadataTitle, 300);
-  if (metaTitle && _ccIntentHasConcreteTarget(message, metaTitle, intent)) return metaTitle;
-  const cleaned = _ccCleanIntentString(message, 300);
-  if (!cleaned) return '';
-  const title = intent?.kind === 'plan'
-    ? cleaned
-    : _delegatedWorkTitle(cleaned, intent?.workType || 'ask');
-  return title.charAt(0).toUpperCase() + title.slice(1);
-}
-
-function _ccFallbackMissingTargetError(intent) {
-  const label = intent?.kind === 'plan' ? 'plan' : (intent?.workType || 'dispatch');
-  return `Missing target for ${label} request. Specify a concrete title, PR/work item, file, or feature to ${label}.`;
-}
-
-function _actionsWithIntentFallback(actions, opts = {}) {
-  const { message = '', intentMetadata = null, source = 'command-center', filePath = null, title: docTitle = null, answerText = '', toolUses = [] } = opts;
-  const existing = (Array.isArray(actions) ? actions.map(normalizeCCAction) : [])
-    .filter(action => !_ccShouldSuppressAnsweredAskDispatch(action, { message, source, answerText, toolUses }));
-  if (_messageRequestsDirectHandling(message, { answerText, toolUses })) return existing.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
-  if (existing.some(a => normalizeCCAction(a)?.type === 'dispatch')) return existing;
-  if (existing.length > 0) return existing;
-  const meta = _ccNormalizeIntentMetadata(intentMetadata);
-  const messageIntent = _ccInferMessageActionIntent(message, source);
-  const metadataIntent = source === 'doc-chat' ? null : _ccInferMetadataActionIntent(meta);
-  const intent = messageIntent || metadataIntent;
-  if (!intent) return _ensureDelegationForIntent(existing, { message, source, filePath, title: docTitle, answerText, toolUses });
-
-  const title = _ccIntentTitle(message, meta.title, intent);
-  const hasTarget = _ccIntentHasConcreteTarget(message, meta.title, intent);
-  const description = meta.description || _ccCleanIntentString(message, 2000);
-  const common = {
-    ...(title ? { title } : {}),
-    ...(description ? { description } : {}),
-    ...(meta.priority ? { priority: meta.priority } : {}),
-    ...(meta.project ? { project: meta.project } : {}),
-    ...(Array.isArray(meta.agents) && meta.agents.length ? { agents: meta.agents } : {}),
-    ...(meta.fanout ? { scope: 'fan-out' } : {}),
-  };
-  if (intent.kind === 'plan') {
-    const action = {
-      type: 'plan',
-      ...common,
-      branchStrategy: meta.branchStrategy || meta.branch_strategy || 'parallel',
-    };
-    if (!hasTarget) action._intentFallbackError = _ccFallbackMissingTargetError(intent);
-    return [action];
-  }
-  const action = {
-    type: 'dispatch',
-    workType: intent.workType,
-    ...common,
-  };
-  const prRef = getWorkItemPrRef(meta) || extractPrRefFromText([message, meta.title, meta.description].filter(Boolean).join('\n'));
-  if (prRef && isPrTargetedWorkType(intent.workType)) action.pr = prRef;
-  if (!hasTarget) action._intentFallbackError = _ccFallbackMissingTargetError(intent);
-  if (_ccShouldSuppressAnsweredAskDispatch(action, { message, source, answerText, toolUses })) return existing;
-  return [action];
-}
-
-function parseCCActions(text) {
-  let actions = [];
-  let displayText = stripCCActionsForDisplay(text);
-  let parseError = null;
-  const header = findCCActionsHeader(text);
-  let segment = '';
-  if (header) {
-    displayText = text.slice(0, header.index).trim();
-    if (header.parseable) {
-      segment = text.slice(header.index + header.headerLength);
-      const jsonStr = _extractActionsJson(segment);
-      if (jsonStr) {
-        try {
-          const parsed = JSON.parse(jsonStr);
-          if (Array.isArray(parsed)) {
-            actions = parsed;
-          } else {
-            parseError = 'actions JSON must be an array after ===ACTIONS=== delimiter';
-          }
-        } catch (e) {
-          parseError = e.message || 'invalid JSON';
-        }
-      } else if (segment.trim()) {
-        parseError = 'no JSON value found after ===ACTIONS=== delimiter';
-      }
-    } else {
-      // Loose/very-loose match: delimiter present but malformed (e.g. extra
-      // equals, lowercase, trailing punct). Surface so the client banner fires
-      // instead of silently dropping actions — user resends with the strict
-      // shape.
-      parseError = 'Malformed ===ACTIONS=== delimiter (extra equals, lowercase, or trailing punctuation). Actions silently discarded — fix the model output.';
-    }
-  }
-  actions = actions.map(normalizeCCAction);
-  const result = { text: displayText, actions };
-  if (parseError && actions.length === 0) {
-    result._actionParseError = parseError;
-    // Visibility for the engine log — shared.log applies SEC-09 redaction before persistence.
-    try {
-      const snippet = (segment.trim() || '').slice(0, 200);
-      if (typeof shared !== 'undefined' && shared && typeof shared.log === 'function') {
-        shared.log('warn', `CC action JSON parse failed: ${parseError} — segment: ${snippet}`);
-      }
-    } catch { /* logging is best-effort */ }
-  }
-  return result;
-}
-
-const DELEGATION_ACTION_TERMS = ['dispatch', 'delegate', 'assign', 'queue', 'enqueue'];
-const DELEGATION_REQUEST_PREFIX_TERMS = ['please', 'can', 'could', 'would', 'will', 'should', 'you', 'cc', 'minions', 'minion'];
-const DELEGATION_PERSON_ACTION_TERMS = ['ask', 'tell', 'have'];
-// Subset of delegation action terms that double as common nouns in status
-// questions (e.g. "queue status", "dispatch contents"). When followed by a
-// status-noun follow-on these are noun usage, not imperative delegation.
-const DELEGATION_AMBIGUOUS_NOUN_TERMS = new Set(['dispatch', 'queue']);
-const DELEGATION_NON_VERB_FOLLOW_ONS = new Set([
-  'status', 'contents', 'content', 'state', 'count', 'list', 'items', 'item',
-  'size', 'length', 'depth', 'health', 'info', 'order', 'queue', 'history',
-  'log', 'logs', 'summary', 'overview', 'report', 'is', 'was', 'are', 'were',
-  'has', 'have', 'looks', 'shows',
-]);
-const DELEGATION_MINIONS_PHRASES = ['have minions', 'ask minions', 'tell minions', 'hand off', 'hand it off', 'hand this off', 'send to agent', 'send it to agent', 'send this to agent'];
-const MEDIUM_INVESTIGATION_TERMS = ['audit', 'investigate', 'research', 'explore', 'analyze', 'analyse'];
-const MEDIUM_INVESTIGATION_PHRASES = ['deep dive', 'root cause'];
-const MEDIUM_IMPLEMENT_TERMS = ['implement', 'fix', 'debug', 'repair', 'refactor', 'migrate', 'harden', 'redesign'];
-const MEDIUM_BUILD_VERBS = ['build', 'add', 'create'];
-const MEDIUM_BUILD_OBJECTS = ['feature', 'support', 'endpoint', 'api', 'ui', 'page', 'component', 'flow', 'integration', 'automation', 'test', 'tests', 'tool', 'command', 'dashboard', 'service', 'module'];
-const MEDIUM_REVIEW_TERMS = ['review', 'test', 'verify', 'validate'];
-const MEDIUM_SIZE_TERMS = ['medium', 'large', 'larger'];
-const MEDIUM_SIZE_PHRASES = ['cross cutting', 'multi file', 'multi step', 'multi stage', 'multi module'];
-const PLAN_INTENT_TERMS = ['plan'];
-const PLAN_CREATE_TERMS = ['make', 'create', 'draft', 'write', 'design'];
-const PLAN_CREATE_PHRASES = ['come up with'];
-const DOC_CHAT_DIRECT_DOC_TERMS = ['summarize', 'summary', 'quote', 'extract', 'rewrite', 'reword', 'copyedit', 'proofread', 'format', 'typo'];
-const DOC_CHAT_DOC_ACTION_TERMS = ['edit', 'update', 'change', 'review', 'check'];
-const DOC_CHAT_DOC_OBJECTS = ['document', 'doc', 'paragraph', 'section', 'selection', 'text', 'wording'];
-const DIRECT_QUICK_TERMS = ['quick', 'simple', 'small'];
-const DIRECT_QUICK_OBJECTS = ['question', 'answer', 'lookup', 'check'];
-const DIRECT_REPLY_TERMS = ['answer', 'respond', 'reply'];
-const DIRECT_REPLY_TARGETS = ['directly', 'inline', 'here'];
-const DIRECT_SELF_TERMS = ['do', 'handle', 'answer', 'fix', 'edit', 'update', 'change', 'review', 'check'];
-const DIRECT_SELF_TARGETS = ['yourself', 'directly'];
-const DIRECT_HANDLING_PHRASES = [
-  'you do it yourself', 'you handle it yourself', 'you handle this yourself',
-  'cc do it yourself', 'doc chat do it yourself', 'doc chat handle it yourself',
-  'do not dispatch', 'dont dispatch', 'do not delegate', 'dont delegate',
-  'do not assign', 'dont assign', 'do not queue', 'dont queue',
-  'do not enqueue', 'dont enqueue', 'do not create work item', 'dont create work item',
-  'no dispatch', 'no delegate', 'no delegation', 'no work item',
-  'without dispatching', 'without delegating', 'without creating work item',
-];
-const ANSWERED_ASK_MIN_CHARS = 80;
-const ANSWERED_ASK_INCOMPLETE_PHRASES = [
-  'i will dispatch', 'ill dispatch', 'i will delegate', 'ill delegate',
-  'i will ask', 'ill ask', 'need to dispatch', 'needs to dispatch',
-  'need to delegate', 'needs to delegate', 'need an agent', 'needs an agent',
-  'hand off', 'hand this off', 'deeper investigation', 'further investigation',
-  'cannot answer', 'cant answer', 'could not answer', 'couldnt answer',
-  'not enough information',
-];
-
-function _isIntentWordChar(ch) {
-  const code = ch.charCodeAt(0);
-  return (code >= 48 && code <= 57) || (code >= 97 && code <= 122);
-}
-
-function _normalizeIntentText(message) {
-  const raw = String(message || '').toLowerCase();
-  let out = ' ';
-  let lastWasSpace = true;
-  for (const ch of raw) {
-    if (ch === '\'' || ch === '\u2019') continue;
-    if (_isIntentWordChar(ch)) {
-      out += ch;
-      lastWasSpace = false;
-    } else if (!lastWasSpace) {
-      out += ' ';
-      lastWasSpace = true;
-    }
-  }
-  return lastWasSpace ? out : out + ' ';
-}
-
-function _intentTokens(normalized) {
-  const text = String(normalized || '').trim();
-  return text ? text.split(' ') : [];
-}
-
-function _intentHasAnyToken(normalized, terms) {
-  const tokens = new Set(_intentTokens(normalized));
-  return terms.some(term => tokens.has(term));
-}
-
-function _intentHasPhrase(normalized, phrase) {
-  return String(normalized || '').includes(` ${phrase} `);
-}
-
-function _intentHasAnyPhrase(normalized, phrases) {
-  return phrases.some(phrase => _intentHasPhrase(normalized, phrase));
-}
-
-function _intentTokenIsNumeric(token) {
-  if (!token) return false;
-  for (const ch of String(token)) {
-    const code = ch.charCodeAt(0);
-    if (code < 48 || code > 57) return false;
-  }
-  return true;
-}
-
-function _intentHasVerbObject(normalized, verbs, objects) {
-  const verbSet = new Set(verbs);
-  const objectSet = new Set(objects);
-  const filler = new Set(['a', 'an', 'the', 'this', 'that', 'it', 'new', 'proper', 'some']);
-  const tokens = _intentTokens(normalized);
-  for (let i = 0; i < tokens.length; i++) {
-    if (!verbSet.has(tokens[i])) continue;
-    for (let j = i + 1; j < tokens.length && j <= i + 4; j++) {
-      if (filler.has(tokens[j])) continue;
-      if (objectSet.has(tokens[j])) return true;
-      break;
-    }
-  }
-  return false;
-}
-
-function _messageRequestsPlanIntent(message) {
-  const normalized = _normalizeIntentText(message);
-  const tokens = _intentTokens(normalized);
-  if (!tokens.length) return false;
-  if (tokens[0] === 'plan') return true;
-  if (_intentHasAnyPhrase(normalized, PLAN_CREATE_PHRASES) && _intentHasAnyToken(normalized, PLAN_INTENT_TERMS)) return true;
-  if (_intentHasVerbObject(normalized, PLAN_CREATE_TERMS, PLAN_INTENT_TERMS)) return true;
-  return _intentHasAnyToken(normalized, PLAN_INTENT_TERMS) &&
-    _intentHasAnyToken(normalized, ['this', 'that', 'it', 'for', 'how', 'out']);
-}
-
-function _messageHasDelegationIntent(message) {
-  const normalized = _normalizeIntentText(message);
-  if (!normalized.trim()) return false;
-  if (_intentHasAnyToken(normalized, DELEGATION_ACTION_TERMS)) return true;
-  if (_intentHasPhrase(normalized, 'work item') && _intentHasAnyToken(normalized, ['create', 'open', 'add'])) return true;
-  return _intentHasAnyPhrase(normalized, DELEGATION_MINIONS_PHRASES);
-}
-
-function _delegationTokenIsImperativeVerb(tokens, idx) {
-  const term = tokens[idx];
-  if (!term) return false;
-  if (!DELEGATION_AMBIGUOUS_NOUN_TERMS.has(term)) return true;
-  const next = tokens[idx + 1];
-  if (!next) return true;
-  return !DELEGATION_NON_VERB_FOLLOW_ONS.has(next);
-}
-
-function _messageExplicitlyRequestsDelegation(message) {
-  const normalized = _normalizeIntentText(message);
-  const tokens = _intentTokens(normalized);
-  if (!tokens.length) return false;
-  if (_intentHasAnyPhrase(normalized, DELEGATION_MINIONS_PHRASES)) return true;
-  if (_intentHasPhrase(normalized, 'work item') && _intentHasAnyToken(normalized, ['create', 'open', 'add'])) return true;
-
-  let i = 0;
-  while (DELEGATION_REQUEST_PREFIX_TERMS.includes(tokens[i])) i++;
-  if (DELEGATION_ACTION_TERMS.includes(tokens[i]) && _delegationTokenIsImperativeVerb(tokens, i)) return true;
-  if (DELEGATION_PERSON_ACTION_TERMS.includes(tokens[i])) {
-    return tokens.slice(i + 1, i + 7).includes('to');
-  }
-  for (let j = 0; j < tokens.length && j <= 6; j++) {
-    if (!DELEGATION_ACTION_TERMS.includes(tokens[j])) continue;
-    if (!_delegationTokenIsImperativeVerb(tokens, j)) continue;
-    const prefix = tokens.slice(0, j);
-    if (prefix.includes('you') || prefix.includes('minions') || prefix.includes('minion')) return true;
-  }
-  return false;
-}
-
-function _messageRequestsDirectHandling(message, opts = {}) {
-  const normalized = _normalizeIntentText(message);
-  if (!normalized.trim()) return false;
-  if (_messageRequestsInformationOnly(message) && !_unansweredInvestigationSignal(opts)) return true;
-  if (_intentHasAnyPhrase(normalized, DIRECT_HANDLING_PHRASES)) return true;
-  if (_intentHasAnyToken(normalized, DIRECT_QUICK_TERMS) && _intentHasAnyToken(normalized, DIRECT_QUICK_OBJECTS)) return true;
-  if (_intentHasVerbObject(normalized, DIRECT_REPLY_TERMS, DIRECT_REPLY_TARGETS)) return true;
-  return _intentHasVerbObject(normalized, DIRECT_SELF_TERMS, DIRECT_SELF_TARGETS);
-}
-
-function _unansweredInvestigationSignal({ answerText = '', toolUses = [] } = {}) {
-  const answerLen = String(answerText || '').trim().length;
-  const toolCount = Array.isArray(toolUses) ? toolUses.length : 0;
-  return toolCount >= 4 && answerLen < ANSWERED_ASK_MIN_CHARS;
-}
-
-function _messageRequestsInformationOnly(message) {
-  const normalized = _normalizeIntentText(message);
-  const tokens = _intentTokens(normalized);
-  if (!tokens.length) return false;
-  if (['what', 'which', 'who', 'when', 'where', 'why', 'how'].includes(tokens[0])) return true;
-  if (['show', 'list'].includes(tokens[0])) return true;
-  if (tokens[0] === 'tell' && tokens[1] === 'me') return true;
-  if (String(message || '').trim().endsWith('?') && !_intentHasAnyToken(normalized, DELEGATION_ACTION_TERMS)) return true;
-  return false;
-}
-
-function _messageIsSmallDocOnlyRequest(normalized) {
-  if (_intentHasAnyToken(normalized, DOC_CHAT_DIRECT_DOC_TERMS)) return true;
-  return _intentHasVerbObject(normalized, DOC_CHAT_DOC_ACTION_TERMS, DOC_CHAT_DOC_OBJECTS);
-}
-
-function _messageHasMediumLargeWorkIntent(message, { source = 'command-center' } = {}) {
-  const normalized = _normalizeIntentText(message);
-  if (!normalized.trim()) return false;
-  if (_messageRequestsDirectHandling(message)) return false;
-  if (source === 'doc-chat' && _messageIsSmallDocOnlyRequest(normalized)) return false;
-  return _intentHasAnyToken(normalized, MEDIUM_INVESTIGATION_TERMS) ||
-    _intentHasAnyPhrase(normalized, MEDIUM_INVESTIGATION_PHRASES) ||
-    _intentHasAnyToken(normalized, MEDIUM_IMPLEMENT_TERMS) ||
-    _intentHasVerbObject(normalized, MEDIUM_BUILD_VERBS, MEDIUM_BUILD_OBJECTS) ||
-    _intentHasAnyToken(normalized, MEDIUM_REVIEW_TERMS) ||
-    _intentHasAnyToken(normalized, MEDIUM_SIZE_TERMS) ||
-    _intentHasAnyPhrase(normalized, MEDIUM_SIZE_PHRASES);
-}
-
-function _inferDelegatedWorkType(message) {
-  const normalized = _normalizeIntentText(message);
-  if (_intentHasAnyToken(normalized, ['test', 'verify', 'validate']) || _intentHasPhrase(normalized, 'build and test')) return 'test';
-  if (_intentHasAnyToken(normalized, ['review']) || _intentHasPhrase(normalized, 'code review')) return 'review';
-  if (_intentHasAnyToken(normalized, ['fix', 'debug', 'repair', 'bug', 'broken', 'failing', 'failure', 'regression']) || _intentHasPhrase(normalized, 'root cause')) return 'fix';
-  if (_intentHasAnyToken(normalized, ['implement', 'refactor', 'migrate', 'harden', 'redesign']) ||
-      _intentHasVerbObject(normalized, MEDIUM_BUILD_VERBS, MEDIUM_BUILD_OBJECTS)) return 'implement';
-  if (_intentHasAnyToken(normalized, MEDIUM_INVESTIGATION_TERMS) || _intentHasAnyPhrase(normalized, MEDIUM_INVESTIGATION_PHRASES)) return 'explore';
-  return 'ask';
-}
-
-function _ccAnswerLooksCompleteForAsk(answerText, toolUses = []) {
-  if (!Array.isArray(toolUses) || toolUses.length === 0) return false;
-  const answer = _collapseWhitespace(stripCCActionSyntax(answerText || ''));
-  if (answer.length < ANSWERED_ASK_MIN_CHARS) return false;
-  const normalized = _normalizeIntentText(answer);
-  if (_intentHasAnyPhrase(normalized, ANSWERED_ASK_INCOMPLETE_PHRASES)) return false;
-  if (_intentHasVerbObject(normalized, ['need', 'needs', 'require', 'requires'], ['agent', 'delegation', 'investigation', 'research'])) return false;
-  return true;
-}
-
-function _ccShouldSuppressAnsweredAskDispatch(action, opts = {}) {
-  const normalized = normalizeCCAction(action);
-  if (normalized?.type !== 'dispatch') return false;
-  if (String(normalized.workType || '').trim().toLowerCase() !== 'ask') return false;
-  if (_messageExplicitlyRequestsDelegation(opts.message)) return false;
-  return _ccAnswerLooksCompleteForAsk(opts.answerText, opts.toolUses);
-}
-
-function _collapseWhitespace(text) {
-  let out = '';
-  let lastWasSpace = true;
-  for (const ch of String(text || '')) {
-    const isSpace = ch === ' ' || ch === '\n' || ch === '\r' || ch === '\t';
-    if (isSpace) {
-      if (!lastWasSpace) out += ' ';
-      lastWasSpace = true;
-    } else {
-      out += ch;
-      lastWasSpace = false;
-    }
-  }
-  return out.trim();
-}
-
-function _stripEdgeQuotes(text) {
-  const quotes = new Set(['"', "'", '`']);
-  let start = 0;
-  let end = String(text || '').length;
-  while (start < end && quotes.has(text[start])) start++;
-  while (end > start && quotes.has(text[end - 1])) end--;
-  return text.slice(start, end);
-}
-
-function _delegatedWorkTitle(message, workType) {
-  const compact = _stripEdgeQuotes(_collapseWhitespace(String(message || '')));
-  const trimmed = compact.length > 90 ? compact.slice(0, 87).trimEnd() + '...' : compact;
-  const fallback = {
-    fix: 'Fix requested issue',
-    review: 'Review requested work',
-    test: 'Test requested work',
-    implement: 'Implement requested work',
-    'implement:large': 'Implement requested work',
-    explore: 'Investigate requested work',
-    ask: 'Answer requested query',
-    verify: 'Verify requested work',
-  }[workType] || 'Handle requested work';
-  if (!trimmed) return fallback;
-  const prefix = {
-    fix: 'Fix',
-    review: 'Review',
-    test: 'Test',
-    implement: 'Implement',
-    'implement:large': 'Implement',
-    explore: 'Investigate',
-    ask: 'Answer',
-    verify: 'Verify',
-  }[workType] || 'Handle';
-  const lower = trimmed.toLowerCase();
-  const prefixLower = prefix.toLowerCase();
-  if (lower === prefixLower || lower.startsWith(prefixLower + ' ')) return trimmed;
-  return `${prefix}: ${trimmed}`;
-}
-
-function _priorityFromDelegationMessage(message) {
-  const normalized = _normalizeIntentText(message);
-  if (_intentHasAnyToken(normalized, ['urgent', 'asap', 'critical', 'blocker', 'p0', 'p1']) || _intentHasPhrase(normalized, 'high priority')) return 'high';
-  if (_intentHasPhrase(normalized, 'low priority') || _intentHasPhrase(normalized, 'when you can') || _intentHasAnyToken(normalized, ['whenever'])) return 'low';
-  return 'medium';
-}
-
-function _inferDelegationActionFromMessage(message, { source = 'command-center', filePath = null, title = null, answerText = '', toolUses = [] } = {}) {
-  if (_messageRequestsDirectHandling(message, { answerText, toolUses })) return null;
-  const explicit = _messageHasDelegationIntent(message);
-  const mediumLarge = _messageHasMediumLargeWorkIntent(message, { source });
-  const toolCount = Array.isArray(toolUses) ? toolUses.length : 0;
-  if (!explicit && !mediumLarge && toolCount < 4) return null;
-
-  const workType = _inferDelegatedWorkType(message);
-  const inferredProject = source === 'doc-chat' && filePath && typeof _inferDocChatProject === 'function'
-    ? _inferDocChatProject(filePath)
-    : null;
-  const context = [];
-  context.push(`${source === 'doc-chat' ? 'Doc-chat' : 'Command Center'} detected ${explicit ? 'an explicit dispatch/delegation request' : (toolCount >= 4 ? 'a medium-sized tool-using request' : 'a medium/large work request')} and routed it through the engine.`);
-  if (filePath) context.push(`Document: ${filePath}`);
-  if (title && title !== filePath) context.push(`Title: ${title}`);
-  if (answerText && String(answerText).trim()) context.push(`Assistant draft before delegation:\n${String(answerText).trim().slice(0, 1200)}`);
-  context.push(`Original request:\n${String(message || '').trim()}`);
-
-  return {
-    type: 'dispatch',
-    title: _delegatedWorkTitle(message, workType),
-    workType,
-    priority: _priorityFromDelegationMessage(message),
-    description: context.join('\n\n'),
-    ...(inferredProject ? { project: inferredProject } : {}),
-    _autoDelegated: true,
-  };
-}
-
-function _ensureDelegationForIntent(actions, opts = {}) {
-  const list = (Array.isArray(actions) ? actions.map(normalizeCCAction) : [])
-    .filter(action => !_ccShouldSuppressAnsweredAskDispatch(action, opts));
-  if (_messageRequestsDirectHandling(opts.message, opts)) return list.filter(a => normalizeCCAction(a)?.type !== 'dispatch');
-  if (list.some(a => normalizeCCAction(a)?.type === 'dispatch')) return list;
-  if (list.length > 0) return list;
-  const inferred = _inferDelegationActionFromMessage(opts.message, opts);
-  if (_ccShouldSuppressAnsweredAskDispatch(inferred, opts)) return list;
-  return inferred ? [...list, inferred] : list;
-}
-
-function stripCCActionSyntax(text) {
-  if (!text) return '';
-  let displayText = text;
-  const header = findCCActionsHeader(text);
-  if (header) {
-    displayText = text.slice(0, header.index).trim();
-  } else {
-    // C-2: doc-chat streaming must also catch partial ===ACTIONS===
-    // delimiters at the chunk tail — without this, a tail like `===ACT`
-    // leaks raw to the modal until the next chunk completes the header.
-    const partialIdx = findCCActionsPartialDelimiter(displayText);
-    if (partialIdx >= 0) displayText = displayText.slice(0, partialIdx).trimEnd();
-  }
-  return displayText.replace(/`{3,}\s*action\s*\r?\n[\s\S]*?`{3,}\n?/g, '').trim();
-}
-
-function _escapeRegExp(str) {
-  return String(str).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}
-
-function findLineBoundedDelimiter(text, delimiter) {
-  const re = new RegExp(`(?:^|\\r?\\n)${_escapeRegExp(delimiter)}[ \\t]*(?=\\r?\\n|$)`);
-  const match = re.exec(text || '');
-  if (!match) return null;
-  return {
-    index: match.index + match[0].indexOf(delimiter),
-    length: delimiter.length,
-  };
-}
-
-function findDocChatDocumentDelimiter(text) {
-  return findLineBoundedDelimiter(text, DOC_CHAT_DOCUMENT_DELIMITER)
-    || findLineBoundedDelimiter(text, LEGACY_DOC_CHAT_DOCUMENT_DELIMITER);
-}
-
-function markdownFenceFor(content) {
-  const runs = String(content || '').match(/`+/g) || [];
-  const maxRun = runs.reduce((max, run) => Math.max(max, run.length), 0);
-  return '`'.repeat(Math.max(4, maxRun + 1));
-}
-
-function fencedUntrustedBlock(label, content) {
-  const value = String(content || '');
-  const fence = markdownFenceFor(value);
-  return `### ${label}\n${fence}text\n${value}\n${fence}`;
-}
-
-function _messageExplicitlyRequestsMonitoring(message) {
-  const text = String(message || '').toLowerCase();
-  if (!text.trim()) return false;
-  if (/\b(?:do\s+not|don't|dont|never|without|no\s+need\s+to)\s+(?:monitor(?:ing)?|watch(?:ing)?|poll(?:ing)?|check(?:ing)?|notify(?:ing)?|ping(?:ing)?|keep(?:ing)?\s+an\s+eye)\b/.test(text)) {
-    return false;
-  }
-  return [
-    /\bmonitor(?:ing)?\b/,
-    /\bwatch(?:ing)?\b/,
-    /\bkeep(?:ing)?\s+an\s+eye\s+on\b/,
-    /\bcheck(?:ing)?\s+(?:it|this|that|[^.?!\n]+)\s+periodically\b/,
-    /\bperiodically\s+check\b/,
-    /\bcheck(?:ing)?\s+[^.?!\n]+\bevery\s+\d+\s*(?:s|sec|seconds?|m|min|minutes?|h|hr|hours?)\b/,
-    /\bevery\s+\d+\s*(?:s|sec|seconds?|m|min|minutes?|h|hr|hours?)\b[^.?!\n]+\b(?:check|poll|monitor|watch)\b/,
-    /\bping\s+(?:me\s+)?(?:on|when|after)\b/,
-    /\bping\s+on\s+completion\b/,
-    /\bnotify\s+(?:me\s+)?(?:on|when|after)\b/,
-    /\blet\s+me\s+know\s+(?:when|once|if)\b/,
-    /\bpoll(?:ing)?\b/,
-  ].some(pattern => pattern.test(text));
-}
-
-function _isDispatchLikeCCAction(action) {
-  const type = String(action?.type || '').trim().toLowerCase();
-  return type === 'dispatch' || CC_DISPATCH_ACTION_ALIASES.has(type);
-}
-
-function _filterImplicitPostDispatchActions(actions, humanMessage) {
-  if (!Array.isArray(actions) || actions.length === 0) return [];
-  if (_messageExplicitlyRequestsMonitoring(humanMessage)) return actions;
-  let sawDispatch = false;
-  const filtered = [];
-  for (const action of actions) {
-    if (_isDispatchLikeCCAction(action)) {
-      sawDispatch = true;
-      filtered.push(action);
-      continue;
-    }
-    if (!sawDispatch) filtered.push(action);
-  }
-  return filtered;
-}
-
-// ── /loop → create-watch safety net ──────────────────────────────────────────
-// CC sometimes invokes the /loop skill instead of emitting a create-watch action.
-// This pure function detects /loop invocation in CC response text and synthesizes
-// a create-watch action as a fallback. Returns null if no conversion needed.
-
-function _detectLoopInvocation(text, actions, toolUses, humanMessage) {
-  const observedToolUses = Array.isArray(toolUses) ? toolUses : [];
-  if (!text && observedToolUses.length === 0) return null;
-  // If a create-watch action was already emitted, no fallback needed
-  if (actions && actions.some(a => a.type === 'create-watch')) return null;
-  if (humanMessage !== undefined && !_messageExplicitlyRequestsMonitoring(humanMessage)) return null;
-
-  function _extractTargetFromValue(value, keyHint) {
-    if (value == null) return null;
-    const hint = String(keyHint || '').toLowerCase();
-    if (Array.isArray(value)) {
-      for (const item of value) {
-        const nested = _extractTargetFromValue(item, hint);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    if (typeof value === 'object') {
-      for (const [k, v] of Object.entries(value)) {
-        const nested = _extractTargetFromValue(v, k);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    const str = String(value).trim();
-    if (!str) return null;
-    const prUrlMatch = str.match(/\/pull\/(\d+)\b/i) || str.match(/\/pullrequest\/(\d+)\b/i);
-    if (prUrlMatch) return { target: prUrlMatch[1], targetType: 'pr' };
-    const prMatch = str.match(/\bPR[- #:]?(\d+)\b/i) || str.match(/\bpull[- ]request[- #:]?(\d+)\b/i);
-    if (prMatch) return { target: prMatch[1], targetType: 'pr' };
-    const wiMatch = str.match(/\bW-([a-z0-9]+)\b/i);
-    if (wiMatch) return { target: 'W-' + wiMatch[1], targetType: 'work-item' };
-    if ((hint.includes('pr') || hint.includes('pull')) && /^\d+$/.test(str)) return { target: str, targetType: 'pr' };
-    if ((hint.includes('work') || hint.includes('item') || hint === 'id') && /^W-[a-z0-9]+$/i.test(str)) return { target: str.toUpperCase().startsWith('W-') ? 'W-' + str.slice(2) : str, targetType: 'work-item' };
-    if (hint.includes('target')) {
-      if (/^\d+$/.test(str)) return { target: str, targetType: 'pr' };
-      if (/^W-[a-z0-9]+$/i.test(str)) return { target: 'W-' + str.slice(2), targetType: 'work-item' };
-    }
-    return null;
-  }
-
-  function _extractIntervalFromValue(value, keyHint) {
-    if (value == null) return null;
-    const hint = String(keyHint || '').toLowerCase();
-    if (Array.isArray(value)) {
-      for (const item of value) {
-        const nested = _extractIntervalFromValue(item, hint);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    if (typeof value === 'object') {
-      for (const [k, v] of Object.entries(value)) {
-        const nested = _extractIntervalFromValue(v, k);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    if (!(hint.includes('interval') || hint.includes('every') || hint.includes('frequency'))) return null;
-    const str = String(value).trim().toLowerCase();
-    if (!str) return null;
-    if (/^\d+$/.test(str)) return str;
-    const match = str.match(/^(\d+(?:\.\d+)?)\s*(s|sec|seconds?|m|min|minutes?|h|hr|hours?)$/i);
-    if (!match) return null;
-    return match[1] + match[2][0].toLowerCase();
-  }
-
-  function _extractConditionFromValue(value, keyHint) {
-    if (value == null) return null;
-    const hint = String(keyHint || '').toLowerCase();
-    if (Array.isArray(value)) {
-      for (const item of value) {
-        const nested = _extractConditionFromValue(item, hint);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    if (typeof value === 'object') {
-      for (const [k, v] of Object.entries(value)) {
-        const nested = _extractConditionFromValue(v, k);
-        if (nested) return nested;
-      }
-      return null;
-    }
-    if (!(hint.includes('condition') || hint.includes('until') || hint.includes('goal') || hint.includes('status'))) return null;
-    const str = String(value).trim().toLowerCase();
-    if (['merged', 'build-pass', 'build-fail', 'completed', 'failed', 'status-change', 'any', 'new-comments', 'vote-change'].includes(str)) return str;
-    if (/\b(?:pass(?:es|ing|ed)?|green|succeed(?:s|ed)?|success)\b/i.test(str)) return 'build-pass';
-    if (/\b(?:fail(?:s|ing|ed)?|red|broken|broke)\b/i.test(str)) return 'build-fail';
-    if (/\bmerge(?:d)?\b/i.test(str)) return 'merged';
-    if (/\bcomplete(?:d)?\b/i.test(str)) return 'completed';
-    if (/\bfail(?:ed)?\b/i.test(str)) return 'failed';
-    if (/\bcomment/i.test(str)) return 'new-comments';
-    if (/\bvote|review/i.test(str)) return 'vote-change';
-    if (/\bstatus/i.test(str)) return 'any';
-    return null;
-  }
-
-  const loopToolSeen = observedToolUses.some(t => /\bloop\b/i.test(String(t?.name || '')));
-  const toolText = observedToolUses.map(t => {
-    try { return [String(t?.name || ''), JSON.stringify(t?.input || {})].filter(Boolean).join(' '); }
-    catch { return String(t?.name || ''); }
-  }).join('\n');
-  const combinedText = [text || '', toolText].filter(Boolean).join('\n');
-
-  // Check for /loop invocation patterns in CC response
-  const loopPatterns = [
-    /\/loop\b/i,
-    /\bloop skill\b/i,
-    /\bSkill.*\bloop\b/i,
-    /\bstarted.*\bloop\b/i,
-    /\bmonitoring.*\bloop\b/i,
-    /\binvok(?:e|ed|ing).*\bloop\b/i,
-  ];
-  if (!loopToolSeen && !loopPatterns.some(p => p.test(combinedText))) return null;
-
-  // Extract target — PR number or work item ID
-  const directTarget = observedToolUses.map(t => _extractTargetFromValue(t && t.input, t && t.name)).find(Boolean);
-  const prMatch = combinedText.match(/\/pull\/(\d+)\b/i) ||
-    combinedText.match(/\/pullrequest\/(\d+)\b/i) ||
-    combinedText.match(/\bPR[- #:]?(\d+)\b/i) ||
-    combinedText.match(/\bpull[- ]request[- #:]?(\d+)/i);
-  const wiMatch = combinedText.match(/\bW-([a-z0-9]+)\b/i);
-
-  let target = null, targetType = 'pr';
-  if (directTarget) {
-    target = directTarget.target;
-    targetType = directTarget.targetType;
-  } else if (prMatch) {
-    target = prMatch[1];
-    targetType = 'pr';
-  } else if (wiMatch) {
-    target = 'W-' + wiMatch[1];
-    targetType = 'work-item';
-  }
-  if (!target) return null; // Can't synthesize without a target
-
-  // Extract interval (e.g. "every 15 minutes", "every 5m")
-  const directInterval = observedToolUses.map(t => _extractIntervalFromValue(t && t.input, t && t.name)).find(Boolean);
-  const intervalMatch = combinedText.match(/every\s+(\d+)\s*(s|sec|seconds?|m|min|minutes?|h|hr|hours?)\b/i);
-  let interval = '5m';
-  if (directInterval) interval = directInterval;
-  else if (intervalMatch) interval = intervalMatch[1] + intervalMatch[2][0];
-
-  // Infer condition from keywords
-  let condition = observedToolUses.map(t => _extractConditionFromValue(t && t.input, t && t.name)).find(Boolean) || 'any';
-  if (condition === 'any') {
-    if (/\bbuild\b/i.test(combinedText) && /\b(?:pass(?:es|ing|ed)?|green|succeed(?:s|ed)?|success)\b/i.test(combinedText)) condition = 'build-pass';
-    else if (/\bbuild\b/i.test(combinedText) && /\b(?:fail(?:s|ing|ed)?|red|broken|broke)\b/i.test(combinedText)) condition = 'build-fail';
-    else if (/\bmerge[d]?\b/i.test(combinedText)) condition = 'merged';
-    else if (/\bcomplete[d]?\b/i.test(combinedText)) condition = 'completed';
-  }
-
-  return {
-    type: 'create-watch',
-    target,
-    targetType,
-    condition,
-    interval,
-    owner: 'human',
-    description: 'Auto-converted from /loop invocation',
-    stopAfter: condition === 'any' ? 0 : 1,
-  };
-}
-
-function _extractToolUsesFromRaw(raw) {
-  const toolUses = [];
-  if (!raw) return toolUses;
-  for (const line of String(raw).split('\n')) {
-    const trimmed = line.trim();
-    if (!trimmed || !trimmed.startsWith('{')) continue;
-    try {
-      const obj = JSON.parse(trimmed);
-      if (obj.type !== 'assistant' || !Array.isArray(obj.message?.content)) continue;
-      for (const block of obj.message.content) {
-        if (block?.type === 'tool_use' && block.name) toolUses.push({ name: block.name, input: block.input || {} });
-      }
-    } catch {}
-  }
-  return toolUses;
-}
-
-// ── Server-side CC action execution ──────────────────────────────────────────
-// Actions are executed server-side so all clients (frontend, curl, Teams) get the same behavior.
-// The frontend still shows status toasts but no longer needs to fire the API calls.
-
-// Parse interval from CC action — accepts ms number, "15m", "1h", "30s", or null (default 5m).
-function _parseWatchInterval(val) {
-  if (!val) return 300000;
-  if (typeof val === 'number') return Math.max(60000, val);
-  const s = String(val).trim().toLowerCase();
-  if (/^\d+$/.test(s)) { const n = parseInt(s, 10); return Math.max(60000, n >= 1000 ? n : n * 1000); }
-  const m = s.match(/^(\d+(?:\.\d+)?)\s*(s|sec|m|min|h|hr|hours?)$/);
-  if (!m) return 300000;
-  const n = parseFloat(m[1]), u = m[2][0];
-  return Math.max(60000, Math.round(u === 's' ? n * 1000 : u === 'm' ? n * 60000 : n * 3600000));
-}
-
-function normalizeMeetingParticipants(participants) {
-  if (!Array.isArray(participants)) return [];
-  const seen = new Set();
-  const normalized = [];
-  for (const participant of participants) {
-    const id = String(participant || '').trim();
-    if (!id || seen.has(id)) continue;
-    seen.add(id);
-    normalized.push(id);
-  }
-  return normalized;
-}
-
-function meetingParticipantsFromAction(action) {
-  return normalizeMeetingParticipants(
-    Array.isArray(action?.participants) && action.participants.length > 0
-      ? action.participants
-      : action?.agents
-  );
-}
-
-function normalizePipelineForCompare(pipeline) {
-  if (!pipeline || typeof pipeline !== 'object') return null;
-  return {
-    title: pipeline.title || '',
-    stages: Array.isArray(pipeline.stages) ? pipeline.stages : [],
-    trigger: pipeline.trigger && typeof pipeline.trigger === 'object' ? pipeline.trigger : {},
-    enabled: pipeline.enabled !== false,
-    project: pipeline.project !== undefined ? pipeline.project : null,
-    projects: Array.isArray(pipeline.projects) ? pipeline.projects : [],
-    stopWhen: pipeline.stopWhen || null,
-    monitoredResources: Array.isArray(pipeline.monitoredResources) ? pipeline.monitoredResources : [],
-  };
-}
-
-function buildPipelineFromAction(action) {
-  const pipeline = {
-    id: String(action.id || '').trim(),
-    title: String(action.title || '').trim(),
-    stages: action.stages,
-    trigger: action.trigger && typeof action.trigger === 'object' ? action.trigger : {},
-    enabled: action.enabled !== false,
-  };
-  if (action.project !== undefined) pipeline.project = action.project;
-  if (Array.isArray(action.projects)) pipeline.projects = action.projects;
-  if (action.stopWhen) pipeline.stopWhen = action.stopWhen;
-  if (Array.isArray(action.monitoredResources) && action.monitoredResources.length > 0) {
-    pipeline.monitoredResources = action.monitoredResources;
-  }
-  return pipeline;
-}
-
-function pipelineDefinitionsEqual(a, b) {
-  return JSON.stringify(normalizePipelineForCompare(a)) === JSON.stringify(normalizePipelineForCompare(b));
-}
-
-function createPipelineFromAction(action) {
-  const { savePipeline, getPipeline } = require('./engine/pipeline');
-  const pipeline = buildPipelineFromAction(action);
-  const projectError = validatePipelineProjects(pipeline);
-  if (projectError) {
-    return { type: 'create-pipeline', id: pipeline.id, error: projectError };
-  }
-  const existing = getPipeline(pipeline.id);
-  if (existing) {
-    if (pipelineDefinitionsEqual(existing, pipeline)) {
-      return {
-        type: 'create-pipeline',
-        id: pipeline.id,
-        ok: true,
-        duplicate: true,
-        duplicateOf: pipeline.id,
-        warning: `Pipeline "${pipeline.id}" already exists; no changes made.`,
-      };
-    }
-    return {
-      type: 'create-pipeline',
-      id: pipeline.id,
-      error: `Pipeline "${pipeline.id}" already exists with a different definition. Use edit-pipeline to update it.`,
-    };
-  }
-  savePipeline(pipeline);
-  const persisted = getPipeline(pipeline.id);
-  if (!persisted) {
-    return { type: 'create-pipeline', id: pipeline.id, error: `Pipeline "${pipeline.id}" was not persisted.` };
-  }
-  if (!pipelineDefinitionsEqual(persisted, pipeline)) {
-    return { type: 'create-pipeline', id: pipeline.id, error: `Pipeline "${pipeline.id}" persisted with unexpected contents.` };
-  }
-  invalidateStatusCache();
-  return { type: 'create-pipeline', id: pipeline.id, ok: true, created: true };
-}
-
-// Required-field validator for CC actions. Returns null when valid, an error string when not.
-// Centralises field-required checks so the model can't quietly emit a malformed action and have
-// the server silently fall back to placeholder values (e.g. "Untitled"). The handler invokes this
-// before `try` to avoid filling `results` with cryptic per-handler error messages.
-function _ccValidateAction(action) {
-  if (!action || typeof action !== 'object' || !action.type) return 'action is missing required field: type';
-  const normalized = normalizeCCAction(action);
-  switch (normalized.type) {
-    case 'dispatch':
-      if (!normalized.title || typeof normalized.title !== 'string' || !normalized.title.trim()) return `${action.type} action missing required field: title`;
-      return null;
-    case 'implement':
-      return 'Unsupported action type "implement"; use type="dispatch" with workType="implement".';
-    case 'build-and-test':
-      if (!action.pr) return 'build-and-test action missing required field: pr';
-      return null;
-    case 'note':
-      if (!action.title) return 'note action missing required field: title';
-      if (!action.content && !action.description) return 'note action missing required field: content (or description)';
-      return null;
-    case 'knowledge':
-      if (!action.title) return 'knowledge action missing required field: title';
-      if (!action.content) return 'knowledge action missing required field: content';
-      if (!action.category) return 'knowledge action missing required field: category';
-      return null;
-    case 'pin-to-pinned':
-      if (!action.title || !action.content) return 'pin-to-pinned action missing title or content';
-      return null;
-    case 'plan':
-      if (!action.title) return 'plan action missing required field: title';
-      return null;
-    case 'create-meeting': {
-      if (!action.title || typeof action.title !== 'string' || !action.title.trim()) return 'create-meeting action missing required field: title';
-      if (!action.agenda || typeof action.agenda !== 'string' || !action.agenda.trim()) return 'create-meeting action missing required field: agenda';
-      if (meetingParticipantsFromAction(action).length < 2) return 'create-meeting action requires at least 2 participants';
-      return null;
-    }
-    case 'create-pipeline':
-      if (!action.id || typeof action.id !== 'string' || !action.id.trim()) return 'create-pipeline action missing required field: id';
-      if (!action.title || typeof action.title !== 'string' || !action.title.trim()) return 'create-pipeline action missing required field: title';
-      if (!Array.isArray(action.stages) || action.stages.length === 0) return 'create-pipeline action requires non-empty stages array';
-      return null;
-    default:
-      return null; // unknown types fall through to existing handler / generic fallback
-  }
-}
-
-let _ccLocalApiInvokerForTest = null;
-
-function _setCcLocalApiInvokerForTest(fn) {
-  _ccLocalApiInvokerForTest = typeof fn === 'function' ? fn : null;
-}
-
-function _ccRouteMethodsForPath(pathname) {
-  if (!Array.isArray(_ccApiRoutesMeta) || _ccApiRoutesMeta.length === 0) return null;
-  const methods = new Set();
-  for (const route of _ccApiRoutesMeta) {
-    if (route._pathRegex instanceof RegExp) {
-      route._pathRegex.lastIndex = 0;
-      if (route._pathRegex.test(pathname)) methods.add(String(route.method || '').toUpperCase());
-    } else if (route.path === pathname) {
-      methods.add(String(route.method || '').toUpperCase());
-    }
-  }
-  return methods;
-}
-
-function _ccValidateLocalApiFallback(endpoint, method) {
-  if (typeof endpoint !== 'string' || !endpoint.trim()) return 'generic API fallback requires endpoint';
-  const raw = endpoint.trim();
-  if (!(raw === '/api' || raw.startsWith('/api/'))) return 'generic API fallback endpoint must be a local /api/ path';
-  if (/[\0\r\n\\]/.test(raw) || raw.includes('..') || /%2e/i.test(raw) || /%5c/i.test(raw)) {
-    return 'generic API fallback endpoint is unsafe';
-  }
-  let parsed;
-  try {
-    parsed = new URL(raw, 'http://127.0.0.1');
-  } catch {
-    return 'generic API fallback endpoint is invalid';
-  }
-  if (parsed.origin !== 'http://127.0.0.1' || !(parsed.pathname === '/api' || parsed.pathname.startsWith('/api/'))) {
-    return 'generic API fallback endpoint must be a local /api/ path';
-  }
-  if (CC_API_FALLBACK_BLOCKED_PREFIXES.some(prefix => parsed.pathname === prefix || parsed.pathname.startsWith(prefix + '/'))) {
-    return 'generic API fallback cannot call Command Center, doc-chat, or bot endpoints';
-  }
-  if (/stream/i.test(parsed.pathname) || parsed.pathname === '/api/hot-reload') {
-    return 'generic API fallback cannot call streaming endpoints';
-  }
-  const normalizedMethod = String(method || 'POST').toUpperCase();
-  if (!CC_API_FALLBACK_METHODS.has(normalizedMethod)) {
-    return `generic API fallback method ${normalizedMethod} is not allowed`;
-  }
-  const routeMethods = _ccRouteMethodsForPath(parsed.pathname);
-  if (routeMethods && routeMethods.size > 0 && !routeMethods.has(normalizedMethod)) {
-    return `API endpoint ${parsed.pathname} does not allow ${normalizedMethod}; allowed methods: ${[...routeMethods].join(', ')}`;
-  }
-  if (routeMethods && routeMethods.size === 0) {
-    return `API endpoint ${parsed.pathname} is not in the local API index`;
-  }
-  return null;
-}
-
-function _ccBuildQueryString(params) {
-  if (!params || typeof params !== 'object' || Array.isArray(params)) return '';
-  const search = new URLSearchParams();
-  for (const [key, value] of Object.entries(params)) {
-    if (value === undefined || value === null) continue;
-    if (Array.isArray(value)) {
-      for (const item of value) search.append(key, String(item));
-    } else if (typeof value === 'object') {
-      search.append(key, JSON.stringify(value));
-    } else {
-      search.append(key, String(value));
-    }
-  }
-  const text = search.toString();
-  return text ? '?' + text : '';
-}
-
-function _ccRequestPath(endpoint, method, params) {
-  const parsed = new URL(endpoint, 'http://127.0.0.1');
-  if (method === 'GET') {
-    const extra = _ccBuildQueryString(params);
-    if (extra) {
-      const glue = parsed.search ? '&' : '?';
-      return parsed.pathname + parsed.search + glue + extra.slice(1);
-    }
-  }
-  return parsed.pathname + parsed.search;
-}
-
-async function _ccInvokeLocalApi({ method, endpoint, params }) {
-  if (_ccLocalApiInvokerForTest) return _ccLocalApiInvokerForTest({ method, endpoint, params });
-  const requestPath = _ccRequestPath(endpoint, method, params);
-  return new Promise((resolve, reject) => {
-    const body = method === 'GET' ? null : JSON.stringify(params || {});
-    const req = http.request({
-      hostname: '127.0.0.1',
-      port: PORT,
-      method,
-      path: requestPath,
-      timeout: CC_API_FALLBACK_TIMEOUT_MS,
-      headers: body ? {
-        'Content-Type': 'application/json',
-        'Content-Length': Buffer.byteLength(body),
-      } : {},
-    }, res => {
-      let text = '';
-      res.setEncoding('utf8');
-      res.on('data', chunk => { text += chunk; });
-      res.on('end', () => {
-        let data = text;
-        try { data = text ? JSON.parse(text) : {}; } catch { /* non-JSON API response */ }
-        resolve({ status: res.statusCode || 0, data });
-      });
-    });
-    req.on('timeout', () => {
-      req.destroy(new Error(`local API fallback timed out after ${CC_API_FALLBACK_TIMEOUT_MS}ms`));
-    });
-    req.on('error', reject);
-    if (body) req.write(body);
-    req.end();
-  });
-}
-
-function _ccApiRequest(endpoint, params = {}, method = 'POST') {
-  return { endpoint, params, method };
-}
-
-function _ccMappedApiRequests(action) {
-  switch (action.type) {
-    case 'pin':
-    case 'pin-to-pinned':
-      return _ccApiRequest('/api/pinned', { title: action.title, content: action.content || action.description, level: action.level || '' });
-    case 'plan': {
-      const branchStrategy = action.branch_strategy || action.branchStrategy || 'parallel';
-      return _ccApiRequest('/api/plan', {
-        title: action.title, description: action.description || '', priority: action.priority,
-        project: action.project, agent: action.agent, branch_strategy: branchStrategy,
-      });
-    }
-    case 'cancel':
-      return _ccApiRequest('/api/agents/cancel', {
-        agent: action.agent || action.agentId,
-        task: action.task || action.cancelTask,
-        reason: action.reason || 'Cancelled via command center',
-      });
-    case 'retry':
-      return (action.ids || []).map(id => _ccApiRequest('/api/work-items/retry', { id, source: action.source || '' }));
-    case 'pause-plan':
-      return _ccApiRequest('/api/plans/pause', { file: action.file });
-    case 'approve-plan':
-      return _ccApiRequest('/api/plans/approve', { file: action.file });
-    case 'reject-plan':
-      return _ccApiRequest('/api/plans/reject', { file: action.file, reason: action.reason || '' });
-    case 'archive-plan':
-      return _ccApiRequest('/api/plans/archive', { file: action.file });
-    case 'unarchive-plan':
-      return _ccApiRequest('/api/plans/unarchive', { file: action.file });
-    case 'execute-plan':
-      return _ccApiRequest('/api/plans/execute', { file: action.file, project: action.project || '' });
-    case 'trigger-verify':
-      return _ccApiRequest('/api/plans/trigger-verify', { file: action.file });
-    case 'regenerate-plan':
-      return _ccApiRequest('/api/plans/approve', { file: action.file, forceRegen: true });
-    case 'revise-plan':
-      return _ccApiRequest('/api/plans/revise', { file: action.file, feedback: action.feedback || action.description, requestedBy: 'command-center' });
-    case 'edit-prd-item':
-      return _ccApiRequest('/api/prd-items/update', {
-        source: action.source, itemId: action.itemId, name: action.name, description: action.description,
-        priority: action.priority, estimated_complexity: action.estimated_complexity || action.complexity,
-      });
-    case 'remove-prd-item':
-      return _ccApiRequest('/api/prd-items/remove', { source: action.source, itemId: action.itemId });
-    case 'reopen-prd-item':
-      return _ccApiRequest('/api/prd-items/update', { source: action.file, itemId: action.id, status: 'updated' });
-    case 'delete-work-item':
-      return _ccApiRequest('/api/work-items/delete', { id: action.id, source: action.source || '' });
-    case 'cancel-work-item':
-      return _ccApiRequest('/api/work-items/cancel', { id: action.id, source: action.source || '', reason: action.reason || 'cc' });
-    case 'archive-work-item':
-      return _ccApiRequest('/api/work-items/archive', { id: action.id });
-    case 'work-item-feedback':
-      return _ccApiRequest('/api/work-items/feedback', { id: action.id, rating: action.rating || 'up', comment: action.comment || '' });
-    case 'schedule':
-      return _ccApiRequest(action._update ? '/api/schedules/update' : '/api/schedules', {
-        id: action.id, title: action.title, cron: action.cron, type: action.workType || 'implement',
-        project: action.project, agent: action.agent, description: action.description,
-        priority: action.priority, enabled: action.enabled !== false,
-      });
-    case 'delete-schedule':
-      return _ccApiRequest('/api/schedules/delete', { id: action.id });
-    case 'edit-pipeline':
-      return _ccApiRequest('/api/pipelines/update', {
-        id: action.id, title: action.title, stages: action.stages,
-        trigger: action.trigger, enabled: action.enabled, stopWhen: action.stopWhen,
-        monitoredResources: action.monitoredResources,
-      });
-    case 'delete-pipeline':
-      return _ccApiRequest('/api/pipelines/delete', { id: action.id });
-    case 'trigger-pipeline':
-      return _ccApiRequest('/api/pipelines/trigger', { id: action.id });
-    case 'continue-pipeline':
-      return _ccApiRequest('/api/pipelines/continue', { id: action.id, stageId: action.stageId });
-    case 'abort-pipeline':
-      return _ccApiRequest('/api/pipelines/abort', { id: action.id });
-    case 'retrigger-pipeline':
-      return _ccApiRequest('/api/pipelines/retrigger', { id: action.id });
-    case 'add-meeting-note':
-      return _ccApiRequest('/api/meetings/note', { id: action.id, note: action.note || action.content });
-    case 'advance-meeting':
-      return _ccApiRequest('/api/meetings/advance', { id: action.id });
-    case 'end-meeting':
-      return _ccApiRequest('/api/meetings/end', { id: action.id });
-    case 'archive-meeting':
-      return _ccApiRequest('/api/meetings/archive', { id: action.id });
-    case 'unarchive-meeting':
-      return _ccApiRequest('/api/meetings/unarchive', { id: action.id });
-    case 'delete-meeting':
-      return _ccApiRequest('/api/meetings/delete', { id: action.id });
-    case 'set-config':
-      return _ccApiRequest('/api/settings', { engine: { [action.setting]: action.value } });
-    case 'update-routing':
-      return _ccApiRequest('/api/settings/routing', { content: action.content });
-    case 'steer-agent':
-      return _ccApiRequest('/api/agents/steer', { agent: action.agent, message: action.message || action.content });
-    case 'link-pr':
-      return _ccApiRequest('/api/pull-requests/link', { url: action.url, title: action.title || '', project: action.project || '', autoObserve: action.autoObserve !== false });
-    case 'delete-pr':
-      return _ccApiRequest('/api/pull-requests/delete', { id: action.id, project: action.project || '' });
-    case 'file-bug':
-      return _ccApiRequest('/api/issues/create', { title: action.title, description: action.description, labels: action.labels });
-    case 'promote-to-kb':
-      return _ccApiRequest('/api/inbox/promote-kb', { name: action.file, category: action.category || 'project-notes' });
-    case 'kb-sweep':
-      return _ccApiRequest('/api/knowledge/sweep', {});
-    case 'toggle-kb-pin':
-      return _ccApiRequest('/api/kb-pins/toggle', { key: action.key });
-    case 'unpin':
-      return _ccApiRequest('/api/pinned' + '/remove', { title: action.title });
-    case 'add-project':
-      return _ccApiRequest('/api/projects/add', {
-        path: action.path || action.localPath, name: action.name || '',
-        repoHost: action.repoHost || 'github', allowNonRepo: action.allowNonRepo,
-        confirmToken: action.confirmToken,
-      });
-    case 'restart-engine':
-      return _ccApiRequest('/api/engine/restart', {});
-    case 'reset-settings':
-      return _ccApiRequest('/api/settings/reset', {});
-    default:
-      if (action.endpoint) return _ccApiRequest(action.endpoint, action.params || {}, action.method || 'POST');
-      return null;
-  }
-}
-
-async function _ccExecuteLocalApiAction(action) {
-  const mapped = _ccMappedApiRequests(action);
-  if (!mapped) return null;
-  const requests = Array.isArray(mapped) ? mapped : [mapped];
-  if (requests.length === 0) throw new Error(`${action.type} action has no API requests to execute`);
-  const apiResults = [];
-  for (const request of requests) {
-    const method = String(request.method || 'POST').toUpperCase();
-    const endpoint = String(request.endpoint || '').trim();
-    const params = request.params || {};
-    const validationError = _ccValidateLocalApiFallback(endpoint, method);
-    if (validationError) throw new Error(validationError);
-    const response = await _ccInvokeLocalApi({ method, endpoint, params });
-    const status = Number(response?.status) || 0;
-    const data = response?.data === undefined ? {} : response.data;
-    if (status < 200 || status >= 300) {
-      const detail = data && typeof data === 'object' && data.error ? data.error : `HTTP ${status}`;
-      throw new Error(`${method} ${endpoint} failed: ${detail}`);
-    }
-    if (data && typeof data === 'object' && data.error) throw new Error(`${method} ${endpoint} failed: ${data.error}`);
-    apiResults.push({ status, data, endpoint, method });
-  }
-  const firstData = apiResults[0]?.data && typeof apiResults[0].data === 'object' ? apiResults[0].data : {};
-  return {
-    type: action.type,
-    ok: true,
-    endpoint: apiResults[0]?.endpoint,
-    method: apiResults[0]?.method,
-    status: apiResults[0]?.status,
-    ...(firstData.id ? { id: firstData.id } : {}),
-    ...(firstData.file ? { file: firstData.file } : {}),
-    ...(firstData.message ? { message: firstData.message } : {}),
-    ...(apiResults.length > 1 ? { count: apiResults.length, results: apiResults.map(r => r.data) } : { data: firstData }),
-  };
-}
-
-function _ccActionFailureLabel(action) {
-  const type = String(action?.type || 'action').trim() || 'action';
-  const title = String(action?.title || action?.id || action?.file || action?.target || '').replace(/\s+/g, ' ').trim();
-  return title ? `${type} failed for '${title.slice(0, 120)}'` : `${type} failed`;
-}
-
-function _ccMultiProjectRequiredError(action) {
-  return `${_ccActionFailureLabel(action)}: project field is required when ${PROJECTS.length} projects are configured: ${PROJECTS.map(p => p.name).join(', ')}`;
-}
-
-async function executeCCActions(actions, { source = 'command-center', inferredProject = null } = {}) {
-  const results = [];
-  const dispatchIdsCreatedInThisCall = new Map();
-  for (let actionIndex = 0; actionIndex < actions.length; actionIndex++) {
-    const rawAction = actions[actionIndex];
-    const action = normalizeCCAction(rawAction);
-    if (action?._intentFallbackError) {
-      results.push({
-        type: action.type || 'dispatch',
-        ...(action.workType ? { workType: action.workType } : {}),
-        error: action._intentFallbackError,
-        missingTarget: true,
-      });
-      continue;
-    }
-    const validationError = _ccValidateAction(action);
-    if (validationError) {
-      results.push({ type: action?.type || 'unknown', error: validationError });
-      continue;
-    }
-    try {
-      switch (action.type) {
-        case 'dispatch': {
-          const workType = routing.normalizeWorkType(action.workType || (action.type !== 'dispatch' ? action.type : WORK_TYPE.IMPLEMENT), WORK_TYPE.IMPLEMENT);
-          const id = 'W-' + shared.uid();
-          const project = action.project || '';
-          const prTargetedWorkType = isPrTargetedWorkType(workType);
-          let prRef = getWorkItemPrRef(action);
-          let linkedPr = null;
-          let allPrsForAction = null;
-          if (!prRef && prTargetedWorkType) {
-            allPrsForAction = getPullRequests().filter(p => !p._ghost);
-            linkedPr = inferActionPrRecord(action, allPrsForAction, null);
-            if (linkedPr) prRef = linkedPr.id || linkedPr.url || linkedPr.prNumber;
-          }
-
-          // Strict project resolution. Silent fallback to PROJECTS[0] when the model named an unknown
-          // project caused work items to land in the wrong repo. Now: unknown name → error; ambiguous
-          // (multiple projects + no field) → error; single-project deployments fall through; zero
-          // projects → root-level work-items.json (orchestration system standalone use).
-          let targetProject = null;
-          if (project) {
-            const target = resolveProjectSourceTarget(project, PROJECTS, { allowCentral: false });
-            targetProject = target.project;
-            if (target.error) {
-              results.push({ type: action.type, error: target.error });
-              break;
-            }
-          } else if (prRef) {
-            const allPrs = allPrsForAction || getPullRequests().filter(p => !p._ghost);
-            if (!linkedPr) linkedPr = shared.findPrRecord(allPrs, prRef) || null;
-            if (linkedPr?._project && linkedPr._project !== 'central') {
-              targetProject = resolveProjectSourceTarget(linkedPr._project, PROJECTS, { allowCentral: false }).project || null;
-            }
-          } else if (inferredProject) {
-            // Doc-chat fallback: filePath-derived project when the LLM omits the field. Validated against
-            // PROJECTS upstream by _inferDocChatProject — a stale lookup would just yield null here.
-            targetProject = resolveProjectSourceTarget(inferredProject, PROJECTS, { allowCentral: false }).project || null;
-          }
-          if (!targetProject) {
-            if (PROJECTS.length > 1) {
-              results.push({ type: action.type, error: _ccMultiProjectRequiredError(action) });
-              break;
-            }
-            if (PROJECTS.length === 1) targetProject = PROJECTS[0];
-          }
-          // PROJECTS.length === 0 → targetProject stays null, falls back to root work-items.json (existing behavior).
-
-          if (targetProject && (!linkedPr || !prRef)) {
-            const projectPrs = shared.safeJson(shared.projectPrPath(targetProject)) || [];
-            shared.normalizePrRecords(projectPrs, targetProject);
-            if (!prRef && prTargetedWorkType) {
-              linkedPr = inferActionPrRecord(action, projectPrs, targetProject);
-              if (linkedPr) prRef = linkedPr.id || linkedPr.url || linkedPr.prNumber;
-            } else if (prRef && !linkedPr) {
-              linkedPr = shared.findPrRecord(projectPrs, prRef, targetProject) || null;
-            }
-          }
-          if (prRef && prTargetedWorkType && !linkedPr) {
-            results.push({ type: action.type, error: `PR not found: ${prRef}` });
-            break;
-          }
-
-          const wiPath = targetProject ? shared.projectWorkItemsPath(targetProject) : shared.centralWorkItemsPath(MINIONS_DIR);
-
-          // Promote `agent` (singular) → `agents` (array). Models emit either shape and the prior code
-          // only read `action.agents`, silently dropping `agent: "lambert"` style hints.
-          const agentHints = (() => {
-            if (Array.isArray(action.agents) && action.agents.length > 0) return action.agents.map(String).filter(Boolean);
-            if (typeof action.agent === 'string' && action.agent) return [action.agent];
-            return [];
-          })();
-          const knownAgents = Object.keys(CONFIG.agents || {});
-          const unknownAgent = agentHints.find(a => !knownAgents.includes(a));
-          if (unknownAgent) {
-            results.push({ type: action.type, error: `Unknown agent "${unknownAgent}". Configured agents: ${knownAgents.join(', ') || '(none)'}` });
-            break;
-          }
-
-          // Issue #1772: CC review/explore/test are human-initiated one-offs.
-          // Mark oneShot so any discovered PR is tagged _contextOnly (skips eval loop).
-          const ccOneShotTypes = new Set([WORK_TYPE.REVIEW, WORK_TYPE.EXPLORE, WORK_TYPE.TEST, WORK_TYPE.ASK, WORK_TYPE.VERIFY]);
-          const isOneShot = action.oneShot === true || (action.oneShot !== false && ccOneShotTypes.has(workType));
-          const item = {
-            id, title: action.title.trim(), type: workType,
-            priority: action.priority || 'medium', description: action.description || '',
-            status: WI_STATUS.PENDING, created: new Date().toISOString(),
-            createdBy: source, project: targetProject?.name || project,
-            ...(action.scope ? { scope: action.scope } : {}),
-            ...(agentHints.length ? { preferred_agent: agentHints[0], agents: agentHints } : {}),
-            ...(isOneShot ? { oneShot: true } : {}),
-          };
-          copyWorkItemPrFields(item, action, linkedPr);
-          const createResult = createWorkItemWithDedup(wiPath, item);
-          if (!createResult.created) {
-            const duplicateId = createResult.duplicateOf || createResult.item?.id;
-            if (duplicateId && dispatchIdsCreatedInThisCall.has(duplicateId)) {
-              results.push({
-                type: action.type,
-                id: duplicateId,
-                ok: true,
-                reusedFromAction: dispatchIdsCreatedInThisCall.get(duplicateId),
-              });
-              break;
-            }
-            results.push({ type: action.type, id: duplicateId, ok: true, duplicate: true, duplicateOf: duplicateId });
-            break;
-          }
-          dispatchIdsCreatedInThisCall.set(id, actionIndex);
-          results.push({ type: action.type, id, ok: true });
-
-          // Pre-flight routing check: warn the user if no agent is currently available so the new
-          // item won't sit pending invisibly. Routing failure is non-fatal — the WI was created.
-          try {
-            const resolvedAgent = routing.resolveAgent(workType, CONFIG, { agentHints, dryRun: true });
-            if (!resolvedAgent) {
-              const lastResult = results[results.length - 1];
-              lastResult.warning = `Created ${id} but no agent is currently available to dispatch (routing returned no match for workType=${workType}${agentHints.length ? ', hints=' + agentHints.join(',') : ''}). Item will sit pending until an agent becomes available.`;
-            }
-          } catch (e) {
-            shared.log('warn', `CC dispatch routing pre-flight: ${e.message}`);
-          }
-          break;
-        }
-        case 'build-and-test': {
-          // Resolve PR by number, ID, or URL — same lookup that drives the link-pr / PR-row paths.
-          const allPrs = getPullRequests().filter(p => !p._ghost);
-          const pr = shared.findPrRecord(allPrs, action.pr) || null;
-          if (!pr) {
-            results.push({ type: 'build-and-test', error: `PR not found: ${action.pr}` });
-            break;
-          }
-          // Resolve project: explicit param wins, else PR's _project. No silent first-project fallback —
-          // unresolved → error so build-and-test can't accidentally run against the wrong repo.
-          const projectName = action.project || pr._project || null;
-          const project = projectName
-            ? resolveProjectSourceTarget(projectName, PROJECTS, { allowCentral: false }).project
-            : null;
-          if (!project) {
-            results.push({ type: 'build-and-test', error: `Project not found for PR ${pr.id}: ${projectName || '(none)'}` });
-            break;
-          }
-          // Pick agent: explicit param wins; else routing for 'test' work type.
-          let agentId = action.agent && CONFIG.agents?.[action.agent] ? action.agent : null;
-          if (!agentId) {
-            agentId = routing.resolveAgent('test', CONFIG, { authorAgent: pr.agent });
-          }
-          if (!agentId) {
-            results.push({ type: 'build-and-test', error: 'No available agent for test routing' });
-            break;
-          }
-          const prNumber = shared.getPrNumber(pr);
-          const dispatchKey = `cc-bt-${project.name}-${pr.id}`;
-          const item = playbook.buildPrDispatch(agentId, CONFIG, project, pr, 'test', {
-            pr_id: pr.id, pr_number: prNumber, pr_title: pr.title || '', pr_branch: pr.branch || '',
-            pr_author: pr.agent || '', pr_url: pr.url || '',
-            project_path: project.localPath || '',
-            task: `Build & test ${pr.id}: ${pr.title || ''}`,
-          }, `Build & test ${pr.id}: ${pr.title || ''}`,
-           { dispatchKey, source: 'cc-build-and-test', pr, branch: pr.branch, project: { name: project.name, localPath: project.localPath } });
-          if (!item) {
-            if (agentId?.startsWith('temp-')) {
-              routing.tempAgents.delete(agentId);
-              routing._claimedAgents.delete(agentId);
-            }
-            results.push({ type: 'build-and-test', error: 'Failed to render build-and-test playbook' });
-            break;
-          }
-          const id = dispatchMod.addToDispatch(item);
-          if (agentId?.startsWith('temp-')) {
-            routing.tempAgents.delete(agentId);
-            routing._claimedAgents.delete(agentId);
-          }
-          results.push({ type: 'build-and-test', id, agent: agentId, pr: pr.id, ok: true });
-          break;
-        }
-        case 'note': {
-          shared.writeToInbox('command-center', shared.slugify(action.title || 'note'), `# ${action.title || 'Note'}\n\n${action.content || action.description || ''}`);
-          results.push({ type: 'note', ok: true });
-          break;
-        }
-        case 'knowledge': {
-          const validCategories = ['architecture', 'conventions', 'project-notes', 'build-reports', 'reviews'];
-          const category = action.category || 'project-notes';
-          if (!validCategories.includes(category)) { results.push({ type: 'knowledge', error: 'Invalid category: ' + category }); break; }
-          const slug = shared.slugify(action.title || 'entry');
-          const kbDir = path.join(MINIONS_DIR, 'knowledge', category);
-          if (!fs.existsSync(kbDir)) fs.mkdirSync(kbDir, { recursive: true });
-          shared.safeWrite(path.join(kbDir, slug + '.md'), `# ${action.title}\n\n${action.content || action.description || ''}`);
-          queries.invalidateKnowledgeBaseCache();
-          results.push({ type: 'knowledge', ok: true });
-          break;
-        }
-        case 'reopen-work-item': {
-          const project = action.project || '';
-          let targetProject = null;
-          if (project) {
-            const target = resolveProjectSourceTarget(project, PROJECTS, { allowCentral: false });
-            targetProject = target.project;
-            if (target.error) {
-              results.push({ type: 'reopen-work-item', id: action.id, error: target.error });
-              break;
-            }
-          } else if (inferredProject) {
-            targetProject = resolveProjectSourceTarget(inferredProject, PROJECTS, { allowCentral: false }).project || null;
-          }
-          if (!targetProject) {
-            if (PROJECTS.length > 1) {
-              results.push({ type: 'reopen-work-item', id: action.id, error: _ccMultiProjectRequiredError({ ...action, type: 'reopen-work-item' }) });
-              break;
-            }
-            if (PROJECTS.length === 1) targetProject = PROJECTS[0];
-          }
-          const wiPath = targetProject ? shared.projectWorkItemsPath(targetProject) : shared.centralWorkItemsPath(MINIONS_DIR);
-          let reopenResult = null;
-          mutateJsonFileLocked(wiPath, items => {
-            if (!Array.isArray(items)) items = [];
-            const item = items.find(i => i.id === action.id);
-            if (!item) { reopenResult = { error: 'item not found' }; return items; }
-            if (item.status !== WI_STATUS.DONE && item.status !== WI_STATUS.FAILED && !DONE_STATUSES.has(item.status)) {
-              reopenResult = { error: 'can only reopen done or failed items' }; return items;
-            }
-            reopenWorkItem(item);
-            if (action.description) item.description = action.description;
-            reopenResult = { ok: true };
-            return items;
-          }, { defaultValue: [] });
-          if (reopenResult?.ok) {
-            // Clear dispatch history outside lock
-            const sourcePrefix = targetProject ? `work-${targetProject.name}-` : 'central-work-';
-            const dispatchKey = sourcePrefix + action.id;
-            try {
-              const dispatchPath = path.join(MINIONS_DIR, 'engine', 'dispatch.json');
-              mutateJsonFileLocked(dispatchPath, dispatch => {
-                dispatch.completed = Array.isArray(dispatch.completed) ? dispatch.completed : [];
-                dispatch.completed = dispatch.completed.filter(d => d.meta?.dispatchKey !== dispatchKey);
-                dispatch.completed = dispatch.completed.filter(d => !d.meta?.parentKey || d.meta.parentKey !== dispatchKey);
-                return dispatch;
-              }, { defaultValue: { pending: [], active: [], completed: [] } });
-            } catch { /* best effort */ }
-            invalidateStatusCache();
-          }
-          results.push({ type: 'reopen-work-item', id: action.id, ...(reopenResult || { error: 'unexpected' }) });
-          break;
-        }
-        case 'create-watch': {
-          const intervalMs = _parseWatchInterval(action.interval);
-          const watch = watchesMod.createWatch({
-            target: action.target,
-            targetType: action.targetType || 'pr',
-            condition: action.condition || 'build-pass',
-            interval: intervalMs,
-            owner: action.owner || 'human',
-            description: action.description || null,
-            project: action.project || null,
-            notify: 'inbox',
-            stopAfter: Number(action.stopAfter) || 0,
-            onNotMet: action.onNotMet || null,
-          });
-          results.push({ type: 'create-watch', id: watch.id, ok: true });
-          break;
-        }
-        case 'create-meeting': {
-          const { createMeeting } = require('./engine/meeting');
-          const meeting = createMeeting({
-            title: action.title.trim(),
-            agenda: action.agenda.trim(),
-            participants: meetingParticipantsFromAction(action),
-          });
-          invalidateStatusCache();
-          results.push({ type: 'create-meeting', id: meeting.id, ok: true });
-          break;
-        }
-        case 'create-pipeline': {
-          results.push(createPipelineFromAction(action));
-          break;
-        }
-        case 'delete-watch': {
-          const deleted = watchesMod.deleteWatch(action.id);
-          if (deleted) invalidateStatusCache();
-          results.push({ type: 'delete-watch', id: action.id, ok: deleted });
-          break;
-        }
-        case 'pause-watch': {
-          const paused = watchesMod.updateWatch(action.id, { status: shared.WATCH_STATUS.PAUSED });
-          if (paused) invalidateStatusCache();
-          results.push({ type: 'pause-watch', id: action.id, ok: !!paused });
-          break;
-        }
-        case 'resume-watch': {
-          const resumed = watchesMod.updateWatch(action.id, { status: shared.WATCH_STATUS.ACTIVE });
-          if (resumed) invalidateStatusCache();
-          results.push({ type: 'resume-watch', id: action.id, ok: !!resumed });
-          break;
-        }
-        default: {
-          const apiResult = await _ccExecuteLocalApiAction(action);
-          if (apiResult) {
-            results.push(apiResult);
-          } else {
-            // Server didn't handle — frontend must execute.
-            results.push({ type: action.type });
-          }
-          break;
-        }
-      }
-    } catch (e) {
-      results.push({ type: action.type, error: e.message });
-    }
-  }
-  return results;
-}
-
-async function executeDocChatActions(actions, { filePath = null } = {}) {
-  if (!Array.isArray(actions) || actions.length === 0) return undefined;
-  return executeCCActions(actions, { source: 'doc-chat', inferredProject: _inferDocChatProject(filePath) });
-}
-
-const DOC_CHAT_WORK_ITEM_ACTION_TYPES = new Set(['dispatch', 'fix', 'implement', 'implement:large', 'explore', 'review', 'test', 'ask', 'verify']);
-
-function _buildDocChatActionFeedback(actions, actionResults) {
-  if (!Array.isArray(actions) || !Array.isArray(actionResults)) return [];
-  const feedback = [];
-  for (let i = 0; i < actions.length && i < actionResults.length; i++) {
-    const action = actions[i] || {};
-    const result = actionResults[i] || {};
-    const type = String(result.type || action.type || '').trim();
-    if (!DOC_CHAT_WORK_ITEM_ACTION_TYPES.has(type)) continue;
-    if (result.error) {
-      feedback.push({ type, error: String(result.error) });
-      continue;
-    }
-    if (result.reusedFromAction !== undefined) continue;
-    const id = result.id || result.duplicateOf;
-    if (!result.ok || !id) continue;
-    const item = { type, id: String(id), ok: true };
-    if (result.duplicate) item.duplicate = true;
-    if (result.duplicateOf) item.duplicateOf = String(result.duplicateOf);
-    if (result.warning) item.warning = String(result.warning);
-    feedback.push(item);
-  }
-  return feedback;
-}
 
 // ── Shared LLM call core — used by CC panel and doc modals ──────────────────
 
@@ -4318,33 +2549,11 @@ function contentFingerprint(str) {
   return str.length + ':' + str.charCodeAt(0) + ':' + str.charCodeAt(mid) + ':' + str.charCodeAt(str.length - 1);
 }
 
-function _parseDocChatResultText(text) {
-  const docDelimiter = findDocChatDocumentDelimiter(text);
-  if (docDelimiter) {
-    const answerPart = text.slice(0, docDelimiter.index).trim();
-    const parsedActions = parseCCActions(answerPart);
-    const { text: answer, actions } = parsedActions;
-    let content = text.slice(docDelimiter.index + docDelimiter.length).trim();
-    content = content.replace(/^```\w*\n?/, '').replace(/\n?```$/, '').trim();
-    return {
-      answer,
-      content,
-      actions,
-      ...(parsedActions._actionParseError ? { actionParseError: parsedActions._actionParseError } : {}),
-    };
-  }
-  const parsedActions = parseCCActions(text);
-  const { text: stripped, actions } = parsedActions;
-  return {
-    answer: stripped,
-    content: null,
-    actions,
-    ...(parsedActions._actionParseError ? { actionParseError: parsedActions._actionParseError } : {}),
-  };
-}
-
+// _parseDocChatResultText / _docChatDisplayText were retired together with
+// the ---DOCUMENT--- and ===ACTIONS=== delimiters. Doc-chat now returns the
+// raw text answer; file edits flow through the Edit/Write tools.
 function _docChatDisplayText(text) {
-  return _parseDocChatResultText(text).answer;
+  return String(text || '').trim();
 }
 
 function _inferDocChatProject(filePath) {
@@ -4368,8 +2577,9 @@ function _formatDocChatContext({ document, title, filePath, selection, canEdit,
   // the file path explicitly so the model knows which file Edit can target.
   const editInstructions = canEdit
     ? `\n\nIf editing is requested:\n` +
-      `- Prefer the runtime \`Edit\` tool against \`${filePath}\` for localized changes (typo fixes, single sections, ≲30% of the file). After Edit succeeds, just describe what you changed in plain text — do NOT also echo the document delimiter, the server reads the updated file from disk.\n` +
-      `- For wholesale rewrites or when an edit would invalidate JSON, fall back to the explanation followed by ${DOC_CHAT_DOCUMENT_DELIMITER} on its own line and the COMPLETE updated file. Do not use ${LEGACY_DOC_CHAT_DOCUMENT_DELIMITER} unless continuing an older session.\n` +
+      `- Use the runtime \`Edit\` tool against \`${filePath}\` for localized changes (typo fixes, single sections, ≲30% of the file).\n` +
+      `- Use the runtime \`Write\` tool against \`${filePath}\` for wholesale rewrites or whenever an Edit would invalidate document structure (JSON, code).\n` +
+      `- After your tool calls return, just describe what you changed in plain text — the server re-reads the file from disk and refreshes the editor view. Do NOT echo the document content back, do NOT emit any delimiter (\`---DOCUMENT---\` and similar are gone).\n` +
       `- Never edit any file other than \`${filePath}\`.`
     : '\n\nRead-only — answer questions only.';
   let context = `## Document Context\n**${safeTitle}**${location}${isJson ? ' (JSON)' : ''}${projectHint}\n\n`;
@@ -4478,8 +2688,6 @@ function _docChatFailureResponse(label, filePath, result, sessionPreserved = fal
   console.error(`[${label}] Failed: code=${result.code}, errorClass=${result.errorClass || 'null'}, sessionPreserved=${sessionPreserved}, empty=${!result.text}, tools=${toolUses.length}, filePath=${filePath}, stderr=${envelope.stderr.slice(0, 200)}`);
   return {
     answer: _docChatErrorMessage(result.errorClass, sessionPreserved, toolUses, result.errorMessage || null),
-    content: null,
-    actions: [],
     toolUses,
     error: envelope,
   };
@@ -4493,18 +2701,11 @@ function _docChatFailureResponse(label, filePath, result, sessionPreserved = fal
 // Returns null when there's nothing parseable; caller falls through to failure.
 function _recoverPartialDocChatResponse(result, sessionKey) {
   if (!result || !result.text || !result.text.trim()) return null;
-  const parsed = _parseDocChatResultText(result.text);
-  const hasActions = Array.isArray(parsed.actions) && parsed.actions.length > 0;
-  const hasAnswer = typeof parsed.answer === 'string' && !!parsed.answer.trim();
-  const hasContent = typeof parsed.content === 'string' && !!parsed.content.trim();
-  if (!hasAnswer && !hasContent && !hasActions) return null;
   return {
-    ...parsed,
+    answer: result.text,
     partial: true,
     warning: _docChatPartialWarning(result.errorClass, result.errorMessage || result.stderr || null),
     toolUses: Array.isArray(result.toolUses) ? result.toolUses : [],
-    // Recovery path still attaches the raw runtime failure — the answer landed
-    // despite a non-zero exit; users still benefit from seeing why.
     error: _buildDocChatErrorEnvelope(result),
   };
 }
@@ -4617,45 +2818,23 @@ function _finalizeDocChatEdit({ filePath, fullPath, isJson, canEdit, originalCon
   return { edited: true, content: diskContent };
 }
 
-// Wraps the streaming onChunk so that once the document delimiter is observed
-// in the growing text, subsequent chunks reuse the locked answer instead of
-// re-scanning the tail. The model emits "<explanation> ---DOCUMENT--- <full file>"
-// — the answer portion can't grow after the delimiter, so re-parsing every
-// chunk through the regenerated file body is wasted O(n²) work.
-//
-// Also dedups identical post-strip answers: the upstream accumulator dedups
-// against the raw growing text, but that text keeps changing as the document
-// body streams in even though the visible answer is locked. Without this
-// guard the SSE writer fires a duplicate `chunk` event for every doc-body
-// delta, which triggers a client DOM rerender and localStorage write each time.
+// Dedupes identical post-strip answers so the SSE writer doesn't fire a
+// duplicate `chunk` event for every doc-body delta when the model streams
+// `Edit`/`Write` tool input alongside narration. Since the ===ACTIONS=== /
+// ---DOCUMENT--- delimiters are gone, we just pass text through untouched
+// with a last-sent guard.
 function _makeDocChatStreamStripper(onChunk) {
   if (!onChunk) return undefined;
-  let lockedAnswer = null;
   let lastSent;
   return (text) => {
-    let answer;
-    if (lockedAnswer !== null) {
-      answer = lockedAnswer;
-    } else if (text.indexOf('---') < 0) {
-      // Fast path for typical Q&A: the doc delimiter starts with "---", so
-      // when the chunk doesn't contain that substring it can't possibly
-      // contain ---MINIONS-DOC-CHAT-DOCUMENT-…--- or ---DOCUMENT---. Skip the
-      // regex-heavy delimiter scan; we still need the actions stripper to
-      // hide partial ===ACTIONS=== while the model is mid-emission.
-      answer = stripCCActionsForStream(text);
-    } else {
-      const parsed = _parseDocChatResultText(text);
-      if (parsed.content !== null) lockedAnswer = parsed.answer;
-      answer = parsed.answer;
-    }
-    if (answer === lastSent) return;
-    lastSent = answer;
-    onChunk(answer);
+    if (text === lastSent) return;
+    lastSent = text;
+    onChunk(text);
   };
 }
 
 // Doc-specific wrapper — adds document context, parses ---DOCUMENT---
-async function ccDocCall({ message, document, title, filePath, selection, canEdit, isJson, model, freshSession, transcript, onAbortReady }) {
+async function ccDocCall({ message, document, title, filePath, selection, canEdit, isJson, model, freshSession, transcript, onAbortReady, systemPrompt = DOC_CHAT_SYSTEM_PROMPT }) {
   const sessionKey = filePath || title;
   const docSlice = String(document || '');
 
@@ -4675,13 +2854,10 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
       store: 'doc', sessionKey,
       extraContext, label: 'doc-chat',
       timeout: DOC_CHAT_TIMEOUT_MS,
-      // Match Command Center's full tool surface and turn budget so doc-chat
-      // can take action (read/write/edit/dispatch) instead of being limited
-      // to Q&A. The doc-chat sysprompt scopes orchestration to human message
-      // intent (explicit dispatch or medium/larger work), and ---DOCUMENT---
-      // remains the only whole-file document edit channel.
+      // Match Command Center's full tool surface so doc-chat can take action
+      // (read/write/edit/dispatch) instead of being limited to Q&A.
       allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
-      systemPrompt: DOC_CHAT_SYSTEM_PROMPT,
+      systemPrompt,
       transcript,
       ...(model ? { model } : {}),
       onAbortReady,
@@ -4717,10 +2893,15 @@ async function ccDocCall({ message, document, title, filePath, selection, canEdi
     return _docChatFailureResponse('doc-chat', filePath, result, sessionPreserved);
   }
 
-  return _parseDocChatResultText(result.text);
+  // No more ===ACTIONS=== / ---DOCUMENT--- parsing — CC's text answer is the
+  // visible reply; file edits land via Edit/Write tools (validated in
+  // _finalizeDocChatEdit by re-reading disk); state mutations land via direct
+  // /api/* calls (correlated by X-CC-Turn-Id, surfaced as synthetic chips
+  // server-side).
+  return { answer: result.text, toolUses: Array.isArray(result.toolUses) ? result.toolUses : [] };
 }
 
-async function ccDocCallStreaming({ message, document, title, filePath, selection, canEdit, isJson, model, freshSession, transcript, onAbortReady, onChunk, onToolUse, onRetry }) {
+async function ccDocCallStreaming({ message, document, title, filePath, selection, canEdit, isJson, model, freshSession, transcript, onAbortReady, onChunk, onToolUse, onRetry, systemPrompt = DOC_CHAT_SYSTEM_PROMPT }) {
   const sessionKey = filePath || title;
   const docSlice = String(document || '');
   const streamStripper = _makeDocChatStreamStripper(onChunk);
@@ -4737,11 +2918,9 @@ async function ccDocCallStreaming({ message, document, title, filePath, selectio
       store: 'doc', sessionKey,
       extraContext, label: 'doc-chat',
       timeout: DOC_CHAT_TIMEOUT_MS,
-      // Match Command Center's full tool surface — see ccDocCall for
-      // rationale. Both wrappers must share the same policy so the streaming
-      // variant doesn't diverge from the non-streaming one.
+      // Match Command Center's full tool surface — see ccDocCall for rationale.
       allowedTools: 'Bash,Read,Write,Edit,Glob,Grep,WebFetch,WebSearch',
-      systemPrompt: DOC_CHAT_SYSTEM_PROMPT,
+      systemPrompt,
       transcript,
       ...(model ? { model } : {}),
       onAbortReady,
@@ -4776,7 +2955,12 @@ async function ccDocCallStreaming({ message, document, title, filePath, selectio
     return _docChatFailureResponse('doc-chat-stream', filePath, result, sessionPreserved);
   }
 
-  return _parseDocChatResultText(result.text);
+  // No more ===ACTIONS=== / ---DOCUMENT--- parsing — CC's text answer is the
+  // visible reply; file edits land via Edit/Write tools (validated in
+  // _finalizeDocChatEdit by re-reading disk); state mutations land via direct
+  // /api/* calls (correlated by X-CC-Turn-Id, surfaced as synthetic chips
+  // server-side).
+  return { answer: result.text, toolUses: Array.isArray(result.toolUses) ? result.toolUses : [] };
 }
 
 // -- POST helpers --
@@ -6843,25 +5027,39 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         }
       }
 
-      let { answer, content, actions, actionParseError, partial, warning, toolUses, error: ccError } = await ccDocCall({
+      // Per-turn correlation: render the doc-chat sysprompt with a fresh
+      // {{cc_turn_id}} so any /api/* calls CC makes during the turn surface
+      // as chips, plus snapshot the target file so we can detect Edit/Write
+      // tool changes and emit a "Document saved" chip.
+      const ccTurnId = 'cct-' + shared.uid();
+      const turnSystemPrompt = renderDocChatSystemPromptForTurn(ccTurnId);
+      const docFileBefore = _snapshotDocFile(fullPath);
+
+      let { answer, partial, warning, toolUses, error: ccError } = await ccDocCall({
         message: body.message, document: currentContent, title: body.title,
         filePath: body.filePath, selection: body.selection, canEdit, isJson,
         model: body.model || undefined,
         freshSession: !!body.freshSession,
         transcript: body.transcript,
         onAbortReady: (abort) => { _docAbort = abort; },
+        systemPrompt: turnSystemPrompt,
       });
-      const delegatedActions = _ensureDelegationForIntent(actions, {
-        message: body.message, source: 'doc-chat', filePath: body.filePath, title: body.title, answerText: answer, toolUses,
-      });
-      const actionResults = await executeDocChatActions(delegatedActions, { filePath: body.filePath });
-      const actionFeedback = _buildDocChatActionFeedback(delegatedActions, actionResults);
       const finalize = _finalizeDocChatEdit({
         filePath: body.filePath, fullPath, isJson, canEdit,
-        originalContent: currentContent, delimiterContent: content,
+        originalContent: currentContent, delimiterContent: null,
       });
+      // Record a document-saved entry under this turn ID if the file mtime/size
+      // changed during the call — surfaces as a "Document saved" chip.
+      if (canEdit && _docFileChanged(docFileBefore, fullPath)) {
+        _recordCcTurnCreation(ccTurnId, { kind: 'document-saved', path: body.filePath });
+      }
+      const _synthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, new Date().toISOString());
       const payload = _buildDocChatResponsePayload({
-        answer, actions: delegatedActions, actionResults, actionFeedback, actionParseError, ccError, partial, warning, toolUses, finalize,
+        answer,
+        actions: _synthetic.actions,
+        actionResults: _synthetic.results,
+        actionFeedback: null, actionParseError: null,
+        ccError, partial, warning, toolUses, finalize,
       });
       _docDone = true;
       return jsonReply(res, 200, payload);
@@ -6938,7 +5136,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
 
       try {
 
-        let { answer, content, actions, actionParseError, partial, warning, toolUses, error: ccError } = await ccDocCallStreaming({
+        // Per-turn correlation: see handleDocChat for the matching pattern.
+        const ccTurnId = 'cct-' + shared.uid();
+        const turnSystemPrompt = renderDocChatSystemPromptForTurn(ccTurnId);
+        const docFileBefore = _snapshotDocFile(fullPath);
+
+        let { answer, partial, warning, toolUses, error: ccError } = await ccDocCallStreaming({
           message: body.message, document: currentContent, title: body.title,
           filePath: body.filePath, selection: body.selection, canEdit, isJson,
           model: body.model || undefined,
@@ -6948,18 +5151,22 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           onChunk: (text) => { writeDocEvent({ type: 'chunk', text }); },
           onToolUse: (name, input) => { writeDocEvent({ type: 'tool', name, input: _lightToolInput(input) }); },
           onRetry: (attempt) => { writeDocEvent({ type: 'progress', attempt }); },
+          systemPrompt: turnSystemPrompt,
         });
-        const delegatedActions = _ensureDelegationForIntent(actions, {
-          message: body.message, source: 'doc-chat', filePath: body.filePath, title: body.title, answerText: answer, toolUses,
-        });
-        const actionResults = await executeDocChatActions(delegatedActions, { filePath: body.filePath });
-        const actionFeedback = _buildDocChatActionFeedback(delegatedActions, actionResults);
         const finalize = _finalizeDocChatEdit({
           filePath: body.filePath, fullPath, isJson, canEdit,
-          originalContent: currentContent, delimiterContent: content,
+          originalContent: currentContent, delimiterContent: null,
         });
+        if (canEdit && _docFileChanged(docFileBefore, fullPath)) {
+          _recordCcTurnCreation(ccTurnId, { kind: 'document-saved', path: body.filePath });
+        }
+        const _streamSynthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, new Date().toISOString());
         const payload = _buildDocChatResponsePayload({
-          answer, actions: delegatedActions, actionResults, actionFeedback, actionParseError, ccError, partial, warning, toolUses, finalize,
+          answer,
+          actions: _streamSynthetic.actions,
+          actionResults: _streamSynthetic.results,
+          actionFeedback: null, actionParseError: null,
+          ccError, partial, warning, toolUses, finalize,
         });
         const { answer: finalAnswer, ...donePayload } = payload;
         writeDocEvent({
@@ -7486,45 +5693,24 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           });
         }
 
-        const toolUses = Array.isArray(result.toolUses) ? result.toolUses : _extractToolUsesFromRaw(result.raw);
-        const parsed = parseCCActions(result.text);
-        // Safety net: detect /loop invocation and convert to create-watch
-        const _loopWatch = _detectLoopInvocation(parsed.text, parsed.actions, toolUses, body.message);
-        if (_loopWatch) {
-          parsed.actions.push(_loopWatch);
-          console.warn('[CC] /loop invocation detected — converted to create-watch');
-          try { shared.log('warn', '/loop invocation detected in CC response — auto-converted to create-watch'); } catch {}
-        }
-        parsed.actions = _actionsWithIntentFallback(
-          _filterImplicitPostDispatchActions(parsed.actions, body.message),
-          { message: body.message, intentMetadata: body.intentMetadata, source: 'command-center', answerText: parsed.text, toolUses }
-        );
-        if (parsed.actions.length > 0) {
-          parsed.actionResults = await executeCCActions(parsed.actions);
-          parsed.actionResultsAt = new Date().toISOString();
-        }
-        // Merge synthetic action results from CC's direct /api/* tool calls
-        // (correlated via the X-CC-Turn-Id header). Replaces the ===ACTIONS===
-        // executor for the canonical "CC calls APIs directly" path while
-        // leaving the legacy parser available as a fallback during transition.
-        const _synthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, parsed.actionResultsAt || new Date().toISOString());
-        if (_synthetic.actions.length > 0) {
-          parsed.actions = (parsed.actions || []).concat(_synthetic.actions);
-          parsed.actionResults = (parsed.actionResults || []).concat(_synthetic.results);
-          parsed.actionResultsAt = parsed.actionResultsAt || new Date().toISOString();
-        }
-        // Mirror only user-facing text to Teams; never send the internal action block.
+        // CC mutates state via direct /api/* calls (correlated via X-CC-Turn-Id);
+        // any creations during this turn surface as synthetic action results
+        // for the existing chip renderer.
+        const _synthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, new Date().toISOString());
+        const replyBody = {
+          text: result.text,
+          actions: _synthetic.actions,
+          actionResults: _synthetic.results,
+          ...(_synthetic.actions.length > 0 ? { actionResultsAt: new Date().toISOString() } : {}),
+          sessionId: ccSession.sessionId,
+          newSession: !wasResume,
+        };
+        // Mirror user-facing text to Teams (skip Teams-originated turns).
         if (!tabId.startsWith('teams-')) {
-          teams.teamsPostCCResponse(body.message, parsed.text).catch(() => {});
+          teams.teamsPostCCResponse(body.message, result.text).catch(() => {});
         }
-        // Issue #1834: rename _actionParseError → actionParseError (public field)
-        // so the client can surface a warning when the model emitted ===ACTIONS===
-        // but the JSON couldn't be recovered.
-        const { _actionParseError, ...parsedReply } = parsed;
-        const reply = { ...parsedReply, sessionId: ccSession.sessionId, newSession: !wasResume };
-        if (_actionParseError) reply.actionParseError = _actionParseError;
-        if (sessionReset) reply.sessionReset = true;
-        return jsonReply(res, 200, reply);
+        if (sessionReset) replyBody.sessionReset = true;
+        return jsonReply(res, 200, replyBody);
       } finally {
         _releaseCCTab(tabId);
       }
@@ -7559,9 +5745,8 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       engineConfig,
       onChunk: (text) => {
         _touchCcLiveStream(liveState);
-        const display = stripCCActionsForStream(text);
-        liveState.text = display;
-        if (liveState.writer) liveState.writer({ type: 'chunk', text: display });
+        liveState.text = text;
+        if (liveState.writer) liveState.writer({ type: 'chunk', text });
       },
       onToolUse: (name, input) => {
         _touchCcLiveStream(liveState);
@@ -7835,36 +6020,13 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           } catch { /* non-critical */ }
         }
 
-        // Send final result with actions — execute server-side first
-        let { text: displayText, actions, _actionParseError } = parseCCActions(result.text);
-        // Safety net: detect /loop invocation and convert to create-watch
-        const _loopWatch = _detectLoopInvocation(displayText, actions, toolUses, body.message);
-        if (_loopWatch) {
-          actions.push(_loopWatch);
-          console.warn('[CC] /loop invocation detected — converted to create-watch');
-          try { shared.log('warn', '/loop invocation detected in CC response — auto-converted to create-watch'); } catch {}
-        }
-        actions = _actionsWithIntentFallback(
-          _filterImplicitPostDispatchActions(actions, body.message),
-          { message: body.message, intentMetadata: body.intentMetadata, source: 'command-center', answerText: displayText, toolUses }
-        );
-        let actionResults;
-        let actionResultsAt;
-        if (actions.length > 0) {
-          actionResults = await executeCCActions(actions);
-          actionResultsAt = new Date().toISOString();
-        }
-        // Merge synthetic action results from CC's direct /api/* tool calls.
-        const _streamSynthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, actionResultsAt || new Date().toISOString());
-        if (_streamSynthetic.actions.length > 0) {
-          actions = (actions || []).concat(_streamSynthetic.actions);
-          actionResults = (actionResults || []).concat(_streamSynthetic.results);
-          actionResultsAt = actionResultsAt || new Date().toISOString();
-        }
+        // CC mutates state via direct /api/* tool calls; surface those as chips.
+        const displayText = result.text;
+        const _streamSynthetic = _buildSyntheticActionResultsForTurn(ccTurnId, body.message, new Date().toISOString());
+        const actions = _streamSynthetic.actions;
+        const actionResults = _streamSynthetic.results;
+        const actionResultsAt = actions.length > 0 ? new Date().toISOString() : undefined;
         const donePayload = { type: 'done', text: displayText, actions, actionResults, actionResultsAt, sessionId: responseSessionId, newSession: !wasResume };
-        // Issue #1834: surface action JSON parse failures so the UI can warn
-        // instead of silently dropping. Client renders this as a small notice.
-        if (_actionParseError) donePayload.actionParseError = _actionParseError;
         if (sessionReset) {
           donePayload.sessionReset = true;
           if (sessionResetReason) donePayload.sessionResetReason = sessionResetReason;
@@ -9492,12 +7654,8 @@ module.exports = {
   readBody,
   _filterCcTabSessions,
   _getVersionCheckInterval,
-  _parseWatchInterval,
   _normalizeMeetingParticipants: normalizeMeetingParticipants,
-  _meetingParticipantsFromAction: meetingParticipantsFromAction,
   parsePinnedEntries,
-  _parseDocChatResultText,
-  _buildDocChatActionFeedback,
   _formatDocChatContext,
   _isCompletedMeetingJson,
   _finalizeDocChatEdit,
@@ -9511,18 +7669,15 @@ module.exports = {
   _shouldSuppressDocChatPostPatchError,
   _buildDocChatResponsePayload,
   _inferDocChatProject,
-  _messageHasDelegationIntent,
-  _messageRequestsDirectHandling,
-  _messageHasMediumLargeWorkIntent,
-  _inferDelegationActionFromMessage,
-  _ensureDelegationForIntent,
   _linkPullRequestForTracking: linkPullRequestForTracking,
   _resolveSkillReadPath,
-  DOC_CHAT_DOCUMENT_DELIMITER,
-  _ccValidateAction,
-  _actionsWithIntentFallback,
-  _messageExplicitlyRequestsMonitoring,
-  _filterImplicitPostDispatchActions,
+  // Per-CC-turn correlation surface (replaces ===ACTIONS=== protocol)
+  _ccTurnCreations,
+  _recordCcTurnCreation,
+  _consumeCcTurnCreations,
+  _readCcTurnIdHeader,
+  _buildSyntheticActionResultsForTurn,
+  CC_TURN_CREATION_TTL_MS,
   _findDuplicateWorkItemCreate: findDuplicateWorkItemCreate,
   _createWorkItemWithDedup: createWorkItemWithDedup,
   _resolveWorkItemsCreateTarget: resolveWorkItemsCreateTarget,
@@ -9530,12 +7685,6 @@ module.exports = {
   _buildManualPrdItemPlan: buildManualPrdItemPlan,
   _resolveScheduleProjectValue: resolveScheduleProjectValue,
   _collectArchivedWorkItems: collectArchivedWorkItems,
-  _createPipelineFromAction: createPipelineFromAction,
-  _setCcLocalApiInvokerForTest,
-  _resetCcApiRoutesMetaForTest,
-  _ccValidateLocalApiFallback,
-  executeCCActions,
-  executeDocChatActions,
   buildCCStatePreamble,
   _routesAsMeta,
   _buildTranscriptCarryover,