@yemi33/minions 0.1.2107 → 0.1.2109

package/engine/qa-runs.js

@@ -5,13 +5,19 @@
  * a managed target. Records carry status, timing, work-item linkage, and an
  * artifacts array (screenshots, logs, videos captured during the run).
  *
- * State file: engine/qa-runs.json (single file, all runs across all projects).
- * Artifacts live on disk under engine/qa-artifacts/<runId>/ — the run record
- * stores relative paths (`type`, `path`, `label`, `capturedAt`), and the
- * dashboard exposes them through GET /api/qa/artifacts/<runId>/<file>.
+ * Persistence: SQLite (engine/state.db, `qa_runs` table) as source of truth
+ * since Phase 8 (PRD: migrate-qa-state-to-sqlite). The legacy JSON sidecar
+ * at engine/qa-runs.json is dual-written on every mutation when
+ * `engine.qaDualWriteJson` is true (default) so external tooling and the
+ * rollback path keep working. Artifacts live on disk under
+ * engine/qa-artifacts/<runId>/ — the run record stores relative paths
+ * (`type`, `path`, `label`, `capturedAt`), and the dashboard exposes them
+ * through GET /api/qa/artifacts/<runId>/<file>.
  *
- * Concurrency: every mutation goes through mutateJsonFileLocked per
- * CLAUDE.md best-practice; callbacks are synchronous and never await.
+ * Concurrency: every mutation goes through shared.mutateQaRuns, which routes
+ * to small-state-store.applyQaRunsMutation inside a SQLite transaction
+ * (BEGIN IMMEDIATE) plus a content-hash divergence rehydrate on read.
+ * Callbacks are synchronous and never await.
  *
  * Status state machine:
  *
@@ -20,22 +26,20 @@
  *                       ╲──▶ errored
  *
  * Illegal transitions throw with a descriptive error. Terminal statuses
- * (passed / failed / errored) cannot transition further.
+ * (passed / failed / errored) cannot transition further. `deleteRun` is
+ * only allowed once a run is terminal.
  */
 
 const fs = require('fs');
 const path = require('path');
 const shared = require('./shared');
-const { mutateJsonFileLocked, uid, ts, log } = shared;
+const { mutateQaRuns, uid, ts, log } = shared;
 
-// Cap qa-runs.json so the file doesn't grow unboundedly over months of nightly
-// QA dispatch. Without a cap, listRuns + summarizeRunsForStatus pay O(N) on
-// every read, and /api/status's fast-state slice runs the summary on every
-// rebuild — at 10k+ historical runs the JSON parse alone starts eating into
-// the W-mpehsyhv event-loop budget that CC SSE isolation depends on. Mirrors
-// the 2500-entry cap on engine/log.json. createRun trims oldest-by-createdAt
-// when crossing the threshold; terminal-status runs that have already shipped
-// completion notifications are safe to drop.
+// Cap qa-runs.json so the file (and qa_runs table) doesn't grow unboundedly
+// over months of nightly QA dispatch. Mirrors the 2500-entry cap on
+// engine/log.json. createRun trims oldest-by-createdAt when crossing the
+// threshold; terminal-status runs that have already shipped completion
+// notifications are safe to drop.
 const QA_RUNS_MAX_RECORDS = 2000;
 
 const QA_RUN_STATUS = Object.freeze({
@@ -80,7 +84,8 @@ const ALLOWED_TRANSITIONS = {
   [QA_RUN_STATUS.ERRORED]: new Set(),
 };
 
-// Dynamic paths — respect MINIONS_TEST_DIR for test isolation.
+// Dynamic paths — respect MINIONS_TEST_DIR for test isolation. Kept exported
+// for back-compat with tests that probe the JSON sidecar on disk.
 function qaRunsPath() {
   return path.join(shared.MINIONS_DIR, 'engine', 'qa-runs.json');
 }
@@ -148,7 +153,7 @@ function createRun({ runbookId, targetName, project, workItemId } = {}) {
     createdAt: ts(),
   };
 
-  mutateJsonFileLocked(qaRunsPath(), (runs) => {
+  mutateQaRuns((runs) => {
     if (!Array.isArray(runs)) runs = [];
     runs.push(run);
     // Rotation: drop oldest-by-createdAt when over the cap. Cheap because
@@ -158,7 +163,7 @@ function createRun({ runbookId, targetName, project, workItemId } = {}) {
       runs = runs.slice(runs.length - QA_RUNS_MAX_RECORDS);
     }
     return runs;
-  }, { defaultValue: [] });
+  });
 
   // Pre-create the artifact directory outside the lock — directory creation
   // is idempotent and slow file I/O must never run while holding the lock.
@@ -178,7 +183,7 @@ function markRunning(id) {
   if (!id) throw new Error('qa-runs: id is required');
   let captured = null;
   let transitionError = null;
-  mutateJsonFileLocked(qaRunsPath(), (runs) => {
+  mutateQaRuns((runs) => {
     if (!Array.isArray(runs)) runs = [];
     const run = runs.find(r => r && r.id === id);
     if (!run) { transitionError = new Error(`qa-runs: run not found: ${id}`); return runs; }
@@ -188,7 +193,7 @@ function markRunning(id) {
     run.startedAt = ts();
     captured = run;
     return runs;
-  }, { defaultValue: [] });
+  });
   if (transitionError) throw transitionError;
   return captured;
 }
@@ -215,7 +220,7 @@ function completeRun(id, { status, summary, artifacts } = {}) {
 
   let captured = null;
   let transitionError = null;
-  mutateJsonFileLocked(qaRunsPath(), (runs) => {
+  mutateQaRuns((runs) => {
     if (!Array.isArray(runs)) runs = [];
     const run = runs.find(r => r && r.id === id);
     if (!run) { transitionError = new Error(`qa-runs: run not found: ${id}`); return runs; }
@@ -230,7 +235,7 @@ function completeRun(id, { status, summary, artifacts } = {}) {
     }
     captured = run;
     return runs;
-  }, { defaultValue: [] });
+  });
   if (transitionError) throw transitionError;
   return captured;
 }
@@ -242,11 +247,22 @@ function completeRun(id, { status, summary, artifacts } = {}) {
  */
 function getRun(id) {
   if (!id) return null;
-  const runs = shared.safeJsonArr(qaRunsPath());
+  const runs = _readRuns();
   const run = runs.find(r => r && r.id === id);
   return run || null;
 }
 
+// Internal helper — reads via the small-state store (SQL → JSON fallback)
+// when available, falling back to direct JSON read if SQLite is unavailable.
+function _readRuns() {
+  try {
+    const store = require('./small-state-store');
+    const arr = store.readQaRuns();
+    if (Array.isArray(arr)) return arr;
+  } catch { /* fall back */ }
+  return shared.safeJsonArr(qaRunsPath());
+}
+
 /**
  * List runs, newest first, optionally filtered by status, capped by limit.
  * @param {object} [opts]
@@ -255,7 +271,7 @@ function getRun(id) {
  * @returns {Array<object>}
  */
 function listRuns({ limit, status } = {}) {
-  let runs = shared.safeJsonArr(qaRunsPath());
+  let runs = _readRuns();
   if (!Array.isArray(runs)) return [];
   if (status) {
     if (!isValidStatus(status)) return [];
@@ -280,7 +296,7 @@ function listRuns({ limit, status } = {}) {
  */
 function getRunsForWorkItem(wi) {
   if (!wi) return [];
-  const runs = shared.safeJsonArr(qaRunsPath());
+  const runs = _readRuns();
   return runs
     .filter(r => r && r.workItemId === wi)
     .sort((a, b) => {
@@ -303,7 +319,7 @@ function getRunsForWorkItem(wi) {
 function setRunWorkItemId(id, workItemId) {
   if (!id) return null;
   let captured = null;
-  mutateJsonFileLocked(qaRunsPath(), (runs) => {
+  mutateQaRuns((runs) => {
     if (!Array.isArray(runs)) runs = [];
     const run = runs.find(r => r && r.id === id);
     if (run) {
@@ -311,10 +327,80 @@ function setRunWorkItemId(id, workItemId) {
       captured = run;
     }
     return runs;
-  }, { defaultValue: [] });
+  });
   return captured;
 }
 
+// W-mpxp90xs000i5732 — sandbox check for run ids reaching deleteQaRun /
+// artifact path operations. Mirrors dashboard.js#_qaIsSafeSegment: rejects
+// empty / overlong / nulls / separators / .. / drive letters. Defense in
+// depth — createRun's uid-suffix is already safe, but operator-driven
+// DELETE endpoints take ids from URL paths, so every callsite re-validates.
+function _isSafeRunId(id) {
+  if (!id || typeof id !== 'string') return false;
+  if (id.length > 128) return false;
+  if (id.indexOf('\0') >= 0) return false;
+  if (id.indexOf('/') >= 0 || id.indexOf('\\') >= 0) return false;
+  if (id.indexOf('..') >= 0) return false;
+  if (/^[a-zA-Z]:/.test(id)) return false;
+  return true;
+}
+
+/**
+ * Hard-delete a terminal-status run record and best-effort remove its
+ * artifact directory. Result envelope (matches PR #3008 shape so the
+ * dashboard DELETE /api/qa/runs/<id> handler stays generic):
+ *
+ *   { ok: true,  id, artifactsRemoved: bool }
+ *   { ok: false, error: 'invalid_id' | 'not_found' | 'not_terminal',
+ *     currentStatus?: string }
+ *
+ * Caller must cancel/complete the run via the lifecycle helpers before
+ * deletion — running/pending runs are refused so an operator can't vacuum a
+ * live QA agent out from under itself. The artifact rmSync runs OUTSIDE
+ * the lock, with a path.resolve + startsWith() belt-and-braces sandbox.
+ *
+ * @param {string} id - run id (must pass _isSafeRunId)
+ * @returns {object} envelope (see above)
+ */
+function deleteQaRun(id) {
+  if (!_isSafeRunId(id)) return { ok: false, error: 'invalid_id' };
+  let captured = null;
+  let notFound = false;
+  let notTerminal = null;
+  mutateQaRuns((runs) => {
+    if (!Array.isArray(runs)) return runs;
+    const idx = runs.findIndex(r => r && r.id === id);
+    if (idx < 0) { notFound = true; return runs; }
+    const run = runs[idx];
+    if (!TERMINAL_STATUSES.has(run.status)) {
+      notTerminal = run.status;
+      return runs;
+    }
+    captured = run;
+    runs.splice(idx, 1);
+    return runs;
+  });
+  if (notFound) return { ok: false, error: 'not_found' };
+  if (notTerminal !== null) return { ok: false, error: 'not_terminal', currentStatus: notTerminal };
+  // Sandbox check + best-effort rm outside the lock.
+  let artifactsRemoved = false;
+  try {
+    const dir = qaArtifactsDirForRun(id);
+    const base = path.resolve(qaArtifactsDir());
+    const resolved = path.resolve(dir);
+    if (resolved.startsWith(base + path.sep) || resolved === base) {
+      if (fs.existsSync(resolved)) {
+        fs.rmSync(resolved, { recursive: true, force: true });
+        artifactsRemoved = true;
+      }
+    }
+  } catch (e) {
+    log('warn', `qa-runs: rm artifacts dir failed for ${id}: ${e.message}`);
+  }
+  return { ok: true, id, artifactsRemoved, run: captured };
+}
+
 /**
  * Cheap summary helper for the dashboard /api/status fast-state slice. Returns
  * `{ total, sig }` without sorting the run list — the sidebar activity-dot
@@ -322,14 +408,12 @@ function setRunWorkItemId(id, workItemId) {
  * (b) when any status flips. `sig` joins id:status across all current runs;
  * any change to either advances the string, which is enough signal for the
  * counter. We deliberately skip the sort that listRuns() does because this
- * runs on every fast-state rebuild (~every 10 s + every mtime-tracked write),
- * and an O(N log N) sort on a 2 k-entry file would eat into the event-loop
- * budget that CC SSE isolation (W-mpehsyhv) depends on.
+ * runs on every fast-state rebuild (~every 10 s + every mtime-tracked write).
  *
  * @returns {{ total: number, sig: string }}
  */
 function summarizeRunsForStatus() {
-  const runs = shared.safeJsonArr(qaRunsPath());
+  const runs = _readRuns();
   if (!Array.isArray(runs) || runs.length === 0) return { total: 0, sig: '' };
   let sig = '';
   for (const r of runs) {
@@ -350,6 +434,8 @@ module.exports = {
   markRunning,
   completeRun,
   setRunWorkItemId,
+  deleteQaRun,
+  _isSafeRunId,
   getRun,
   listRuns,
   getRunsForWorkItem,

package/engine/qa-sessions.js

@@ -28,8 +28,10 @@
  *
  *   (awaiting-approval ──▶ drafting on /edit; drafting ──▶ executing on auto mode.)
  *
- * Concurrency: every mutation goes through mutateJsonFileLocked per the repo
- * convention. Callbacks are synchronous and never await. Slow filesystem work
+ * Concurrency: every mutation goes through shared.mutateQaSessions, which
+ * routes to small-state-store.applyQaSessionsMutation inside a SQLite
+ * transaction (BEGIN IMMEDIATE) plus a content-hash divergence rehydrate on
+ * read. Callbacks are synchronous and never await. Slow filesystem work
  * (qa-tests/<id>/ scaffolding, dispatch enqueueing) runs OUTSIDE the lock.
  *
  * Path-traversal hardening: sessionId is generated by createSession() with a
@@ -38,14 +40,28 @@
  * a filesystem path or a session lookup. Mirrors engine/qa-runbooks.js
  * _isSafeId (PR #2694 review feedback).
  *
- * State file: engine/qa-sessions.json (single file, all sessions across all
- * projects), capped at QA_SESSIONS_MAX_RECORDS via createSession-time rotation.
+ * Persistence: SQLite (engine/state.db, `qa_sessions` table) as source of
+ * truth since Phase 8 (PRD: migrate-qa-state-to-sqlite). The legacy JSON
+ * sidecar at engine/qa-sessions.json is dual-written on every mutation when
+ * `engine.qaDualWriteJson` is true (default). Capped at
+ * QA_SESSIONS_MAX_RECORDS via createSession-time rotation.
  */
 
 const fs = require('fs');
 const path = require('path');
 const shared = require('./shared');
-const { mutateJsonFileLocked, uid, ts, log } = shared;
+const { mutateQaSessions, uid, ts, log } = shared;
+
+// Internal helper — read sessions via the small-state store when available
+// (SQL → JSON fallback inside the store), else direct JSON read.
+function _readSessions() {
+  try {
+    const store = require('./small-state-store');
+    const arr = store.readQaSessions();
+    if (Array.isArray(arr)) return arr;
+  } catch { /* fall back */ }
+  return _readSessions();
+}
 
 // Cap engine/qa-sessions.json. Sessions cost more than runs (3 WIs, a
 // managed-spawn, artifacts) so the operational steady state is meaningfully
@@ -422,7 +438,7 @@ function createSession(spec) {
     completedAt: null,
   };
 
-  mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+  mutateQaSessions( (sessions) => {
     if (!Array.isArray(sessions)) sessions = [];
     sessions.push(session);
     // Rotation: drop oldest-by-createdAt when over cap. Cheap because it runs
@@ -448,7 +464,7 @@ function createSession(spec) {
  */
 function getSession(id) {
   if (!_isSafeSessionId(id)) return null;
-  const sessions = shared.safeJsonArr(qaSessionsPath());
+  const sessions = _readSessions();
   return sessions.find(s => s && s.id === id) || null;
 }
 
@@ -520,7 +536,7 @@ function getSessionTestFile(sessionId) {
  * List sessions, newest first, optionally filtered by state, capped by limit.
  */
 function listSessions({ limit, state } = {}) {
-  let sessions = shared.safeJsonArr(qaSessionsPath());
+  let sessions = _readSessions();
   if (!Array.isArray(sessions)) return [];
   if (state) {
     if (!isValidState(state)) return [];
@@ -547,7 +563,7 @@ function setSessionWorkItem(id, phase, workItemId) {
   if (!_isSafeSessionId(id)) return null;
   if (!Object.values(SESSION_PHASE).includes(phase)) return null;
   let captured = null;
-  mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+  mutateQaSessions( (sessions) => {
     if (!Array.isArray(sessions)) sessions = [];
     const session = sessions.find(s => s && s.id === id);
     if (session) {
@@ -571,7 +587,7 @@ function setSessionWorkItem(id, phase, workItemId) {
 function setSessionQaRunId(id, qaRunId) {
   if (!_isSafeSessionId(id)) return null;
   let captured = null;
-  mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+  mutateQaSessions( (sessions) => {
     if (!Array.isArray(sessions)) sessions = [];
     const session = sessions.find(s => s && s.id === id);
     if (session) {
@@ -607,7 +623,7 @@ function transitionSession(id, toState, patch = {}) {
 
   let captured = null;
   let transitionError = null;
-  mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+  mutateQaSessions( (sessions) => {
     if (!Array.isArray(sessions)) sessions = [];
     const session = sessions.find(s => s && s.id === id);
     if (!session) { transitionError = new Error(`qa-sessions: session not found: ${id}`); return sessions; }
@@ -961,7 +977,7 @@ function queueSetup(sessionId, opts = {}) {
     for (const [key, wiId] of Object.entries(builtWiIds)) {
       if (nextStatus[key]) nextStatus[key] = { ...nextStatus[key], wiId };
     }
-    mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+    mutateQaSessions( (sessions) => {
       if (!Array.isArray(sessions)) sessions = [];
       const s = sessions.find(x => x && x.id === sessionId);
       if (s) {
@@ -1031,7 +1047,7 @@ function handleSetupComplete(sessionId, opts = {}) {
   // Update this project's entry under lock. Capture the merged map so we
   // can decide on a transition without re-reading.
   let mergedStatus = null;
-  mutateJsonFileLocked(qaSessionsPath(), (sessions) => {
+  mutateQaSessions( (sessions) => {
     if (!Array.isArray(sessions)) sessions = [];
     const s = sessions.find(x => x && x.id === sessionId);
     if (!s || !s.setupStatus) return sessions;
@@ -1280,7 +1296,7 @@ function dismissSession(sessionId, { summary } = {}) {
  * @returns {{ total: number, sig: string }}
  */
 function summarizeSessionsForStatus() {
-  const sessions = shared.safeJsonArr(qaSessionsPath());
+  const sessions = _readSessions();
   if (!Array.isArray(sessions) || sessions.length === 0) return { total: 0, sig: '' };
   let sig = '';
   for (const s of sessions) {

package/engine/shared.js

@@ -2072,6 +2072,13 @@ const ENGINE_DEFAULTS = {
   ccUseWorkerPool: false,            // Sub-task C of W-mp2w003600196c51 (CC perf): when true AND CC runtime is copilot, _invokeCcStream routes through engine/cc-worker-pool.js (persistent `copilot --acp` per CC tab) instead of spawning a fresh CLI per turn. Off by default — opt-in feature flag. **Structurally copilot-only**: the pool spawns `copilot --acp` (Agent Client Protocol); Claude Code does not implement ACP, so resolveCcUseWorkerPool returns false on non-copilot CC runtimes even with explicit-true (W-mphlriic00095f69 — prevents silent runtime switch). Engine/agent dispatch path stays per-process regardless.
   maxBudgetUsd: undefined,       // fleet USD ceiling for --max-budget-usd (per-agent override: agents.<id>.maxBudgetUsd). Honors 0 via ?? so a literal cap of $0 works
   disableModelDiscovery: false,  // skip runtime.listModels() REST calls fleet-wide (settings UI falls back to free-text)
+  // Phase 8 (qa-runs.json + qa-sessions.json → SQL). When true, every QA
+  // mutation also writes the JSON sidecar at engine/qa-runs.json and
+  // engine/qa-sessions.json for back-compat with external tooling. SQL is
+  // always the source of truth — flip this OFF once operators trust the
+  // SQL store and want to drop the dual-write cost. Sunset tracked in
+  // docs/deprecated.json (qa-json-sidecars).
+  qaDualWriteJson: true,
   // W-mpmwxkrw000872ec — dashboard global font-size scale. Drives the
   // [data-font-size] attribute on <html> via the inline bootstrap script in
   // layout.html (localStorage fast path) and is reconciled from the server
@@ -2979,6 +2986,77 @@ const mutateWorktreePool = _smallStateMutator({
   defaultValue: () => ({ entries: [] }),
 });
 
+/**
+ * Phase 8 — QA state mutators. Both qa-runs and qa-sessions are top-level
+ * JSON arrays (mutator receives the array; mutates in place or returns a
+ * replacement). The store diffs by `id`. Falls back to mutateJsonFileLocked
+ * on SQLite failure so a node:sqlite-broken install keeps recording QA
+ * state. The JSON mirror is gated by `engine.qaDualWriteJson` (default true)
+ * — turn off once operators trust SQL as the source of truth.
+ */
+function _qaDualWriteEnabled() {
+  try {
+    const cfg = safeJson(path.join(MINIONS_DIR, 'config.json')) || {};
+    const flag = cfg && cfg.engine && cfg.engine.qaDualWriteJson;
+    if (flag === false) return false;
+    return true;
+  } catch { return true; }
+}
+
+function _qaMutator({ filePath, applyMutation, mirror, topic }) {
+  return (mutator) => {
+    // Cross-process serialization: SQLite's BEGIN IMMEDIATE already
+    // serializes the table write across processes, but the JSON sidecar
+    // mirror is best-effort and can race — two concurrent processes can
+    // each snapshot SQL and write the JSON, and an older snapshot can
+    // overwrite a newer one. Wrap SQL apply + mirror in a single file
+    // lock on the JSON path's .lock file so multi-process writers
+    // serialize through the mirror, matching the legacy fully-locked
+    // semantics that test/unit/qa-runs.test.js asserts.
+    return withFileLock(filePath + '.lock', () => {
+      try {
+        const store = require('./small-state-store');
+        const { wrote, result } = store[applyMutation]((arr) => {
+          if (!Array.isArray(arr)) arr = [];
+          return mutator(arr) || arr;
+        });
+        if (wrote) {
+          if (_qaDualWriteEnabled()) {
+            try { store[mirror](filePath); } catch { /* mirror best-effort */ }
+          }
+          try { require('./db-events').emitStateEvent(topic); } catch { /* optional */ }
+        }
+        return result;
+      } catch (e) {
+        if (!e || !/SQLite unavailable|no such table|node:sqlite/.test(String(e.message))) throw e;
+        return mutateJsonFileLocked(filePath, (data) => {
+          if (!Array.isArray(data)) data = [];
+          return mutator(data) || data;
+        }, {
+          defaultValue: [],
+          onWrote: () => {
+            try { require('./db-events').emitStateEvent(topic); } catch { /* optional */ }
+          },
+        });
+      }
+    }, { timeoutMs: 5000, retries: 3 });
+  };
+}
+
+const mutateQaRuns = _qaMutator({
+  filePath: path.join(MINIONS_DIR, 'engine', 'qa-runs.json'),
+  applyMutation: 'applyQaRunsMutation',
+  mirror: '_mirrorQaRunsJson',
+  topic: 'qa_runs',
+});
+
+const mutateQaSessions = _qaMutator({
+  filePath: path.join(MINIONS_DIR, 'engine', 'qa-sessions.json'),
+  applyMutation: 'applyQaSessionsMutation',
+  mirror: '_mirrorQaSessionsJson',
+  topic: 'qa_sessions',
+});
+
 /**
  * Route a watches mutation through the SQL store. Same shape as
  * mutateWorkItems / mutatePullRequests: mutator receives the watches
@@ -5769,7 +5847,7 @@ module.exports = {
   runtimeConfigWarnings,
   projectWorkSourceWarnings,
   backfillProjectWorkSourceDefaults,
-  WI_STATUS, DONE_STATUSES, PLAN_TERMINAL_STATUSES, WORK_TYPE, WORKTREE_REQUIRING_TYPES, VALID_WORK_TYPES, resolveWorkItemTypeFromPrdItem, PLAN_STATUS, PRD_ITEM_STATUS, PRD_MATERIALIZABLE, PR_STATUS, PR_POLLABLE_STATUSES, PR_PENDING_REASON, BUILD_STATUS, REVIEW_STATUS, FETCH_TIMEOUT_MS, RETRY_DELAY_MS, ADO_TOKEN_REFRESH_MAX_RETRIES, DISPATCH_RESULT, mutateMetrics, mutateWatches, mutateScheduleRuns, mutatePipelineRuns, mutateManagedProcesses, mutateWorktreePool, trackReviewMetric, queuePlanToPrd, extractPlanDeclaredProject,
+  WI_STATUS, DONE_STATUSES, PLAN_TERMINAL_STATUSES, WORK_TYPE, WORKTREE_REQUIRING_TYPES, VALID_WORK_TYPES, resolveWorkItemTypeFromPrdItem, PLAN_STATUS, PRD_ITEM_STATUS, PRD_MATERIALIZABLE, PR_STATUS, PR_POLLABLE_STATUSES, PR_PENDING_REASON, BUILD_STATUS, REVIEW_STATUS, FETCH_TIMEOUT_MS, RETRY_DELAY_MS, ADO_TOKEN_REFRESH_MAX_RETRIES, DISPATCH_RESULT, mutateMetrics, mutateWatches, mutateScheduleRuns, mutatePipelineRuns, mutateManagedProcesses, mutateWorktreePool, mutateQaRuns, mutateQaSessions, trackReviewMetric, queuePlanToPrd, extractPlanDeclaredProject,
   WATCH_STATUS, WATCH_TARGET_TYPE, WATCH_CONDITION, WATCH_ABSOLUTE_CONDITIONS, WATCH_ACTION_TYPE,
   WATCH_STALLED_DEFAULT_TICKS, WATCH_STUCK_STAGE_DEFAULT_TICKS,
   PIPELINE_STATUS, STAGE_TYPE, MEETING_STATUS, AGENT_STATUS,