@yemi33/minions 0.1.2369 → 0.1.2371

package/engine/ado-comment.js

@@ -2,8 +2,7 @@
  * engine/ado-comment.js — Azure DevOps mirror of engine/gh-comment.js.
  *
  * Posts a minions PR-comment thread to ADO with the same hidden marker, the
- * same collapsible "Harnesses used" section, and the same deterministic brand
- * link as the GitHub path — because all three are produced by the ONE neutral
+ * same deterministic brand link as the GitHub path — because both are produced by the ONE neutral
  * builder `engine/comment-format.js#buildMinionsCommentBody`. This module owns
  * only the ADO transport (REST `…/threads` POST) and ADO-specific input
  * validation (orgBase / project / repositoryId / prNumber).
@@ -13,7 +12,7 @@
  * unlinked on ADO (the GitHub backstop in buildMinionsCommentBody never ran).
  * Routing ADO comments through this poster gives ADO the same guarantees as
  * GitHub: marker present (so pollers classify the comment as agent-authored),
- * harness section folded in, brand trailer hyperlinked.
+ * brand trailer hyperlinked.
  *
  * Auth: a bearer token from `az account get-access-token` (via
  * engine/ado-token.js#acquireAdoToken), threaded as `Authorization: Bearer`.
@@ -34,7 +33,7 @@
  * changes) stay `active` so they block/notify the author.
  */
 
-const { buildMinionsCommentBody, buildSkippedSkillSection } = require('./comment-format');
+const { buildMinionsCommentBody } = require('./comment-format');
 const { acquireAdoToken } = require('./ado-token');
 
 const ADO_API_VERSION = '7.1';
@@ -95,7 +94,7 @@ async function _defaultAcquireToken() {
 /**
  * Post a minions PR comment to an Azure DevOps pull request.
  *
- * Mirrors engine/gh-comment.js#postPrComment: builds the marked + harness +
+ * Mirrors engine/gh-comment.js#postPrComment: builds the marked +
  * brand-linked body via the shared builder, then POSTs a new active thread.
  *
  * @param {object} args
@@ -107,8 +106,6 @@ async function _defaultAcquireToken() {
  * @param {string} args.agentId        minions agent id (marker)
  * @param {string} args.kind           comment kind (marker)
  * @param {string} [args.workItemId]   originating work-item id (marker)
- * @param {object} [args.harnessUsed]  grounded harnessUsed record (folded into body)
- * @param {object} [args.skillSkipped] skipped-project-skill record `{name, reason}` (folded into body, visible)
  * @param {number} [args.timeoutMs=30000]
  * @param {Function} [args.acquireToken]  () => Promise<string> — injectable token source
  * @param {Function} [args.fetchImpl=fetch] injectable fetch
@@ -123,8 +120,6 @@ async function postAdoPrComment({
   agentId,
   kind,
   workItemId,
-  harnessUsed,
-  skillSkipped,
   resolved = false,
   timeoutMs = 30000,
   acquireToken = _defaultAcquireToken,
@@ -136,7 +131,7 @@ async function postAdoPrComment({
   _validatePrNumber(prNumber);
 
   // buildMinionsCommentBody validates marker fields and throws on bad input.
-  const finalBody = buildMinionsCommentBody({ agentId, kind, workItemId, body, harnessUsed, skillSkipped });
+  const finalBody = buildMinionsCommentBody({ agentId, kind, workItemId, body });
 
   const token = await acquireToken();
   if (!token || typeof token !== 'string') {
@@ -186,7 +181,6 @@ module.exports = {
   // Re-export the neutral builder so ADO callers have a single import surface,
   // mirroring engine/gh-comment.js.
   buildMinionsCommentBody,
-  buildSkippedSkillSection,
   // Internal validators exported for tests.
   _validateOrgBase,
   _validateProject,

package/engine/ado.js

@@ -409,11 +409,13 @@ function applyAdoPrMetadata(pr, prData) {
     updated = true;
   }
 
-  // W-mpej044m00076d63: backfill `pr.created` from ADO's real creation
-  // timestamp. lifecycle.js attach paths omit `created` so this poll is the
-  // single source of truth for fresh records, and historical records missing
-  // `created` get backfilled the next time they're polled. Idempotent.
-  if (!pr.created && prData.creationDate) {
+  // W-mpej044m00076d63 / W-mrezh0yb0007f733: keep `pr.created` aligned with
+  // ADO's real creationDate. The platform is authoritative for when a PR was
+  // opened, so adopt its value whenever the stored one is missing, a legacy
+  // date-only stamp, or a link/attach-time `now()` default (the bug where
+  // manually-linked PRs showed a "created today" date). `shouldAdoptPlatformCreated`
+  // compares instants so equivalent representations don't churn. Idempotent.
+  if (shared.shouldAdoptPlatformCreated(pr.created, prData.creationDate)) {
     pr.created = prData.creationDate;
     updated = true;
   }
@@ -2606,6 +2608,9 @@ async function fetchAdoPrMetadata(prNum, adoOrg, adoProj, adoRepo, project) {
     description: pr.description || '',
     branch: stripRefsHeads(pr.sourceRefName),
     author: pr.createdBy?.displayName || '',
+    // W-mrezh0yb0007f733 — surface ADO's real creationDate so manual-link paths
+    // can seed `pr.created` with the true value instead of link-time now().
+    created: pr.creationDate || '',
   };
 }
 

package/engine/cleanup.js

@@ -1388,23 +1388,13 @@ async function runCleanup(config, verbose = false) {
   }
   // PRD items (missing_features[].status)
   try {
-    let prdDirEntries;
-    try {
-      prdDirEntries = fs.readdirSync(PRD_DIR);
-    } catch (e) {
-      log('warn', `migrate PRD statuses: failed to read ${PRD_DIR} — ${e.message}`);
-      prdDirEntries = [];
-    }
-    const prdFiles = prdDirEntries.filter(f => f.endsWith('.json'));
+    const prdStore = require('./prd-store');
+    const prdFiles = prdStore.listPrdRows().filter(row => !row.archived).map(row => row.filename);
     for (const pf of prdFiles) {
       const prdPath = path.join(PRD_DIR, pf);
+      const prd = prdStore.readPrd(prdPath);
       let migrated = 0;
-      shared.withFileLock(`${prdPath}.lock`, () => {
-        // safeJsonNoRestore: PRDs are terminal artifacts. If the active PRD
-        // was archived but a stale .backup sidecar remains, never resurrect
-        // it during a routine migration sweep (W-mouptdh1000h9f39).
-        const prd = safeJsonNoRestore(prdPath);
-        if (!prd?.missing_features) return;
+      if (prd?.missing_features) {
         for (const feat of prd.missing_features) {
           if (LEGACY_DONE_ALIASES.has(feat.status)) {
             feat.status = shared.WI_STATUS.DONE;
@@ -1414,8 +1404,8 @@ async function runCleanup(config, verbose = false) {
             migrated++;
           }
         }
-        if (migrated > 0) safeWrite(prdPath, prd);
-      });
+      }
+      if (migrated > 0) prdStore.writePrd(prdPath, prd);
       if (migrated > 0) {
         log('info', `Migrated ${migrated} legacy PRD item status(es) in ${pf}`);
       }
@@ -1425,36 +1415,27 @@ async function runCleanup(config, verbose = false) {
   // Reset orphaned PRD item statuses — dispatched/failed with no matching work item (#779)
   cleaned.orphanedPrdStatuses = 0;
   try {
-    const wiIds = new Set();
-    for (const project of projects) {
-      const items = safeJsonArr(projectWorkItemsPath(project));
-      for (const wi of items) { if (wi?.id) wiIds.add(wi.id); }
-    }
-    try {
-      const centralWi = safeJsonArr(path.join(MINIONS_DIR, 'work-items.json'));
-      for (const wi of centralWi) { if (wi?.id) wiIds.add(wi.id); }
-    } catch { /* optional */ }
+    const wiIds = new Set(
+      queries.getWorkItems(config, { enrich: false }).map(wi => wi?.id).filter(Boolean)
+    );
 
-    let orphanPrdEntries;
-    try { orphanPrdEntries = fs.readdirSync(PRD_DIR).filter(f => f.endsWith('.json')); }
-    catch { orphanPrdEntries = []; }
+    const prdStore = require('./prd-store');
+    const orphanPrdEntries = prdStore.listPrdRows().filter(row => !row.archived).map(row => row.filename);
     for (const pf of orphanPrdEntries) {
       const prdPath = path.join(PRD_DIR, pf);
-      // safeJsonNoRestore — peek must not auto-restore archived PRDs
-      // from a stale .backup sidecar (W-mouptdh1000h9f39).
-      const peek = safeJsonNoRestore(prdPath);
+      const peek = prdStore.readPrd(prdPath);
       if (!peek?.missing_features) continue;
       let reset = 0;
-      mutateJsonFileLocked(prdPath, (prd) => {
-        if (!prd?.missing_features) return prd;
+      const prd = peek;
+      if (prd?.missing_features) {
         for (const feat of prd.missing_features) {
           if ((feat.status === shared.WI_STATUS.DISPATCHED || feat.status === shared.WI_STATUS.FAILED) && !wiIds.has(feat.id)) {
             feat.status = shared.WI_STATUS.PENDING;
             reset++;
           }
         }
-        return prd;
-      }, { skipWriteIfUnchanged: true });
+      }
+      if (reset > 0) prdStore.writePrd(prdPath, prd);
       if (reset > 0) {
         log('info', `Reset ${reset} orphaned PRD item status(es) → pending in ${pf}`);
         cleaned.orphanedPrdStatuses += reset;
@@ -1669,7 +1650,6 @@ async function runCleanup(config, verbose = false) {
  *  that were written before that guard existed. */
 function scrubStaleMetrics() {
   const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
-  if (!fs.existsSync(metricsPath)) return;
   mutateJsonFileLocked(metricsPath, metrics => {
     for (const key of Object.keys(metrics)) {
       if (key.startsWith('temp-') || key === 'agent1' || key.startsWith('_test')) {

package/engine/cli.js

@@ -247,10 +247,10 @@ const CLI_COMMAND_DOCS = Object.freeze({
   kill:       { args: '',                      summary: 'Kill all active agents, reset to pending' },
   complete:   { args: '<dispatch-id>',         summary: 'Mark a dispatch as done' },
   cleanup:    { args: '',                      summary: 'Clean temp files, worktrees, zombies' },
-  'mcp-sync': { args: '',                      summary: 'Print harness propagation diagnostic (same source as `minions doctor --harness`; read-only, no writes)' },
-  doctor:     { args: '[--harness]',           summary: 'Check prerequisites and runtime health (--harness: print harness propagation diagnostic)' },
+  'mcp-sync': { args: '',                      summary: 'Print runtime harness inventory (same source as `minions doctor --harness`; read-only, no writes)' },
+  doctor:     { args: '[--harness]',           summary: 'Check prerequisites and runtime health (--harness: print runtime harness inventory)' },
   config:     { args: 'set-cli <R> [--model M]', summary: 'Persist defaultCli/defaultModel without starting' },
-  pr:         { args: 'comment <repo> <prNumber> --agent <id> --kind <k> [--wi <id>] [--resolved] [--harness-file <f>|--harness-json <j>] [--skill-skipped-file <f>|--skill-skipped-json <j>] [--body-file <f>|--body <text>]', summary: 'Post a marker-prepended PR comment via gh (ADO: --resolved posts pre-closed)' },
+  pr:         { args: 'comment <repo> <prNumber> --agent <id> --kind <k> [--wi <id>] [--resolved] [--body-file <f>|--body <text>]', summary: 'Post a marker-prepended PR comment via gh (ADO: --resolved posts pre-closed)' },
   bridge:     { args: 'status|health|enable|disable',         summary: 'Constellation bridge: toggle and inspect the read-only cross-repo feed' },
 });
 
@@ -413,111 +413,6 @@ function _applyRuntimeFlags({ cli, model, modelExplicit }) {
   return { warnings, applied: true };
 }
 
-// resolveHarnessUsedForComment(flags) — P-7a3c9e21. Turn the `minions pr comment`
-// harness flags into a grounded `harnessUsed` record suitable for the posters'
-// `harnessUsed` param (engine/gh-comment.js / engine/ado-comment.js), which fold
-// it via buildMinionsCommentBody -> buildHarnessUsedSection.
-//
-// Source precedence: `--harness-file <path>` (JSON on disk) wins over the inline
-// `--harness-json <json>` convenience; neither supplied -> returns undefined so
-// the comment body is byte-identical to today (no section). Read/parse failures
-// are HARD errors (process.exit(2) with the offending message) — never a silent
-// drop. Grounding uses the spawn-time `_harnessPropagated` manifest located by
-// dispatch id (explicit `--dispatch-id` -> else basename(MINIONS_COMPLETION_REPORT,
-// '.json')); an unresolvable manifest passes null, marking every entry
-// grounded:false — the honest "no record" state, not a fatal condition.
-function resolveHarnessUsedForComment(flags) {
-  const fileArg = flags['harness-file'];
-  const jsonArg = flags['harness-json'];
-
-  let raw;
-  if (fileArg !== undefined) {
-    try {
-      raw = fs.readFileSync(fileArg, 'utf8');
-    } catch (e) {
-      console.error(`error: could not read --harness-file ${fileArg}: ${e.message}`);
-      process.exit(2);
-    }
-  } else if (jsonArg !== undefined) {
-    raw = String(jsonArg);
-  } else {
-    return undefined;
-  }
-
-  let parsed;
-  try {
-    parsed = JSON.parse(raw);
-  } catch (e) {
-    const src = fileArg !== undefined ? `--harness-file ${fileArg}` : '--harness-json';
-    console.error(`error: invalid JSON in ${src}: ${e.message}`);
-    process.exit(2);
-  }
-
-  // Resolve the grounding manifest best-effort: explicit id -> completion-report
-  // basename. A missing/unresolvable manifest is non-fatal (groundHarnessUsed
-  // with null propagated marks everything grounded:false).
-  const dispatchId = flags['dispatch-id']
-    || path.basename(process.env.MINIONS_COMPLETION_REPORT || '', '.json')
-    || undefined;
-  let propagated = null;
-  if (dispatchId) {
-    try {
-      const dispatch = getDispatch();
-      for (const queue of ['active', 'completed', 'pending']) {
-        const list = Array.isArray(dispatch?.[queue]) ? dispatch[queue] : null;
-        if (!list) continue;
-        const found = list.find(d => d && d.id === dispatchId);
-        if (found && found._harnessPropagated) { propagated = found._harnessPropagated; break; }
-      }
-    } catch { propagated = null; }
-  }
-
-  return shared.groundHarnessUsed(parsed, propagated);
-}
-
-// resolveSkillSkippedForComment(flags) — W-mr287sc0000uc214. Turn the
-// `minions pr comment` skip flags into a canonical `{name, reason}` record for
-// the posters' `skillSkipped` param (engine/gh-comment.js / engine/ado-comment.js),
-// which fold it via buildMinionsCommentBody -> buildSkippedSkillSection into a
-// VISIBLE callout in the same comment as the verdict.
-//
-// Source precedence mirrors the harness resolver: `--skill-skipped-file <path>`
-// (JSON on disk) wins over the inline `--skill-skipped-json <json>`; neither
-// supplied -> returns undefined so the comment body is byte-identical to today
-// (no callout). Read/parse failures are HARD errors (process.exit(2)). Accepts
-// the raw `{name, reason}` shape or a `meta.skill` / `meta.review` wrapper; a
-// record with no non-empty `name` yields undefined (nothing to surface).
-function resolveSkillSkippedForComment(flags) {
-  const fileArg = flags['skill-skipped-file'];
-  const jsonArg = flags['skill-skipped-json'];
-
-  let raw;
-  if (fileArg !== undefined) {
-    try {
-      raw = fs.readFileSync(fileArg, 'utf8');
-    } catch (e) {
-      console.error(`error: could not read --skill-skipped-file ${fileArg}: ${e.message}`);
-      process.exit(2);
-    }
-  } else if (jsonArg !== undefined) {
-    raw = String(jsonArg);
-  } else {
-    return undefined;
-  }
-
-  let parsed;
-  try {
-    parsed = JSON.parse(raw);
-  } catch (e) {
-    const src = fileArg !== undefined ? `--skill-skipped-file ${fileArg}` : '--skill-skipped-json';
-    console.error(`error: invalid JSON in ${src}: ${e.message}`);
-    process.exit(2);
-  }
-
-  const { _extractSkillSkipped } = require('./comment-format');
-  return _extractSkillSkipped(parsed) || undefined;
-}
-
 const commands = {
   start(...startArgs) {
     // Apply --cli / --model fleet flags before any engine wiring touches
@@ -539,18 +434,6 @@ const commands = {
       throw err;
     }
 
-    // Startup state-file size guard (#1167): dispatch.json / cooldowns.json
-    // bloated past 100 MB silently OOMed V8 on JSON.parse at startup. Fail fast
-    // with an actionable message instead.
-    try {
-      for (const fp of [DISPATCH_PATH, path.join(ENGINE_DIR, 'cooldowns.json')]) {
-        shared.assertStateFileSize(fp);
-      }
-    } catch (stateErr) {
-      console.error('\n[engine] STARTUP ABORTED — ' + stateErr.message + '\n');
-      process.exit(78); // 78 = configuration error
-    }
-
     // Run preflight checks (warn but don't block — engine may still be useful)
     try {
       const { runPreflight, printPreflight } = require('./preflight');
@@ -667,10 +550,7 @@ const commands = {
       }
     } catch (err) { e.log('warn', `note-link backfill failed: ${err.message}`); }
 
-    // Backfill work_items.prd_item_id (the WI↔PRD-item FK) for rows that predate
-    // the dual-write stamp (#546) or that a JSON re-hydrate transiently zeroed.
-    // Set-based, only touches NULL FKs, no-op once everything is linked — safe on
-    // every boot. Lets the Phase-10 render join move off the feature-id string.
+    // Backfill work_items.prd_item_id for rows created before FK stamping.
     try {
       const fk = require('./prd-store').backfillWorkItemPrdItemIds();
       const filled = (fk.byWorkItemId || 0) + (fk.bySourcePlan || 0);
@@ -1211,22 +1091,7 @@ const commands = {
     const _watchedFiles = new Set();
     let _globalDebounce = null;
     function watchForWorkChanges() {
-      const filesToWatch = [
-        path.join(MINIONS_DIR, 'work-items.json'),
-        // dispatch.json excluded — it changes every tick, causing a feedback loop
-      ];
-      // Watch project-specific work-items.json
-      const { getProjects } = require('./shared');
-      for (const p of getProjects(config)) {
-        filesToWatch.push(shared.projectWorkItemsPath(p));
-      }
-      // Watch PRD files
-      const prdDir = path.join(MINIONS_DIR, 'prd');
-      try {
-        for (const f of fs.readdirSync(prdDir).filter(f => f.endsWith('.json'))) {
-          filesToWatch.push(path.join(prdDir, f));
-        }
-      } catch { /* optional */ }
+      const filesToWatch = [];
 
       for (const filePath of filesToWatch) {
         if (_watchedFiles.has(filePath)) continue;
@@ -1465,8 +1330,8 @@ const commands = {
     }
     const flagFields = [
       'claudeBareMode', 'claudeFallbackModel',
-      'copilotDisableBuiltinMcps', 'copilotSuppressAgentsMd', 'copilotStreamMode', 'copilotReasoningSummaries',
-      'harnessPropagateProjectLocal', 'claudePreApproveWorkspaceMcps', 'hermeticHarness',
+      'copilotDisableBuiltinMcps', 'copilotStreamMode', 'copilotReasoningSummaries',
+      'claudePreApproveWorkspaceMcps',
       'maxBudgetUsd', 'disableModelDiscovery',
       'ccUseWorkerPool',
     ];
@@ -1488,14 +1353,13 @@ const commands = {
     }
 
     console.log('');
-    const metrics = shared.safeJsonObj(path.join(ENGINE_DIR, 'metrics.json'));
+    const metrics = queries.getMetrics();
     const lifetimeCompleted = Object.entries(metrics).filter(([k]) => !k.startsWith('_')).reduce((sum, [, m]) => sum + (m.tasksCompleted || 0) + (m.tasksErrored || 0), 0);
     console.log(`Dispatch: ${dispatch.pending.length} pending | ${(dispatch.active || []).length} active | ${lifetimeCompleted} completed`);
     const pidSummary = summarizeActiveDispatchPids(dispatch.active || []);
     console.log(`Active dispatch PID files: ${pidSummary.live} live | ${pidSummary.dead} dead | ${pidSummary.missing} missing`);
 
-    const metricsPath = path.join(ENGINE_DIR, 'metrics.json');
-    const metricsData = safeJson(metricsPath);
+    const metricsData = queries.getMetrics();
     if (metricsData && Object.keys(metricsData).length > 0) {
       console.log('\nMetrics:');
       console.log(`  ${'Agent'.padEnd(12)} ${'Done'.padEnd(6)} ${'Err'.padEnd(6)} ${'PRs'.padEnd(6)} ${'Appr'.padEnd(6)} ${'Rej'.padEnd(6)} ${'Reviews'.padEnd(8)} ${'Cost'.padEnd(8)}`);
@@ -1989,7 +1853,7 @@ const commands = {
   doctor(...doctorArgs) {
     const { doctor, runHarnessDoctor } = require('./preflight');
     if (doctorArgs.includes('--harness')) {
-      // P-a3f9b2c1 — harness propagation diagnostic. See
+      // Read-only runtime harness inventory. See
       // docs/harness-propagation.md and the parallel branch in bin/minions.js.
       const ok = runHarnessDoctor(MINIONS_DIR);
       if (!ok) process.exit(1);
@@ -2086,19 +1950,9 @@ const commands = {
       console.log('Usage:');
       console.log('  GitHub: minions pr comment <repo> <prNumber> --agent <id> --kind <k> [--wi <id>] (--body-file <path> | --body <text>)');
       console.log('  ADO:    minions pr comment <prNumber> --host ado --ado-org <org> --ado-project <proj> --repo-id <id> --agent <id> --kind <k> [--wi <id>] (--body-file <path> | --body <text>)');
-      console.log('');
-      console.log('Optional harness self-report (folded into the collapsible "Harnesses used" section):');
-      console.log('  --harness-file <path>   JSON file with the { skills, mcpServers, commands, docs } record (wins over --harness-json)');
-      console.log('  --harness-json <json>   same record inline as a JSON string');
-      console.log('  --dispatch-id <id>      grounding manifest override (else basename of $MINIONS_COMPLETION_REPORT)');
-      console.log('');
-      console.log('Skipped in-scope project skill (folded in as a VISIBLE "⚠️ Skipped project skill" callout):');
-      console.log('  --skill-skipped-file <path>   JSON file with the { name, reason } skip record (wins over --skill-skipped-json)');
-      console.log('  --skill-skipped-json <json>   same record inline as a JSON string (also accepts a meta.skill/meta.review wrapper)');
       console.log('  --resolved              (ADO only) post the thread pre-resolved/closed — for non-actionable FYI/praise notes');
       console.log('');
-      console.log('Posts a PR comment with the hidden minions marker, the collapsible');
-      console.log('"Harnesses used" section, and the brand link folded in by the shared');
+      console.log('Posts a PR comment with the hidden minions marker and brand link folded in by the shared');
       console.log('builder — so engine classifiers identify agent-authored comments by');
       console.log('structure, and the body is byte-identical on GitHub (`gh pr comment`)');
       console.log('and Azure DevOps (REST `…/threads`).');
@@ -2162,18 +2016,6 @@ const commands = {
       process.exit(2);
     }
 
-    // P-7a3c9e21 — thread the consulted-sources footprint into both posters so
-    // the auto-folded "Harnesses used" <details> section the help already
-    // promises is actually rendered. Resolved once (host-neutral); absent flags
-    // -> undefined -> body byte-identical to today.
-    const harnessUsed = resolveHarnessUsedForComment(flags);
-
-    // W-mr287sc0000uc214 — thread the skipped-in-scope-project-skill self-report
-    // into both posters so it's surfaced as a VISIBLE callout in the same comment
-    // as the verdict (not buried in the completion-report JSON). Absent flags
-    // -> undefined -> body byte-identical to today.
-    const skillSkipped = resolveSkillSkippedForComment(flags);
-
     // ── Azure DevOps: <prNumber> --host ado --ado-org --ado-project --repo-id ──
     if (isAdo) {
       const [prNumberRaw] = positional;
@@ -2193,7 +2035,7 @@ const commands = {
       const orgBase = flags['org-base'] || `https://dev.azure.com/${adoOrg}`;
       const adoComment = require('./ado-comment');
       adoComment.postAdoPrComment({
-        orgBase, project, repositoryId, prNumber, body, agentId, kind, workItemId, harnessUsed, skillSkipped,
+        orgBase, project, repositoryId, prNumber, body, agentId, kind, workItemId,
         resolved: flags.resolved === true,
       }).then((result) => {
         if (result && result.threadId) {
@@ -2232,7 +2074,7 @@ const commands = {
 
     try {
       const result = ghComment.postPrComment({
-        repo, prNumber, body, agentId, kind, workItemId, harnessUsed, skillSkipped,
+        repo, prNumber, body, agentId, kind, workItemId,
       });
       if (result.output) console.log(result.output);
     } catch (e) {
@@ -2371,10 +2213,6 @@ module.exports = {
   _readDispatchPid: readDispatchPid,
   _normalizeSessionBranch: normalizeSessionBranch,
   _dispatchSessionBranch: dispatchSessionBranch,
-  // P-7a3c9e21 — harness-flag resolver exported for CLI-threading tests.
-  _resolveHarnessUsedForComment: resolveHarnessUsedForComment,
-  // W-mr287sc0000uc214 — skipped-project-skill flag resolver exported for tests.
-  _resolveSkillSkippedForComment: resolveSkillSkippedForComment,
   // W-mpcyvff6000pf828 (#2653) — heartbeat writer + factory exported for tests
   _writeHeartbeatNow: writeHeartbeatNow,
   _createHeartbeatInterval: createHeartbeatInterval,