@yemi33/minions 0.1.2334 → 0.1.2336

package/docs/harness-transparency.md

@@ -125,7 +125,7 @@ evaluation pass) can see what tooling drove a dispatch:
    and ⚠️-marked), and returns `''` for an empty/absent record. The GitHub
    comment path folds it in via `engine/gh-comment.js#buildMinionsCommentBody`
    (optional `harnessUsed` arg, threaded through `postPrComment` /
-   `postPrReviewComment` / `postPrReview`); the Azure DevOps path mirrors it via
+   `postPrReviewComment`); the Azure DevOps path mirrors it via
    `engine/ado-comment.js#postAdoPrComment` (same builder, same `harnessUsed`
    arg). Both posters are reached from the `minions pr comment` CLI, which turns
    `--harness-file` / `--harness-json` into the grounded record and folds the

package/engine/gh-comment.js

@@ -210,50 +210,6 @@ function postPrReviewComment({
   }
 }
 
-const _REVIEW_EVENT_FLAGS = Object.freeze({
-  APPROVE: '--approve',
-  REQUEST_CHANGES: '--request-changes',
-  COMMENT: '--comment',
-});
-
-function postPrReview({
-  event,
-  repo,
-  prNumber,
-  body,
-  agentId,
-  kind,
-  workItemId,
-  harnessUsed,
-  skillSkipped,
-  timeoutMs = 30000,
-  execFileSync = _execFileSync,
-  resolveTokenForSlug,
-} = {}) {
-  const flag = _REVIEW_EVENT_FLAGS[event];
-  if (!flag) {
-    throw new Error(
-      `invalid event: ${JSON.stringify(event)} (expected APPROVE | REQUEST_CHANGES | COMMENT)`,
-    );
-  }
-  _validateRepo(repo);
-  _validatePrNumber(prNumber);
-  const finalBody = buildMinionsCommentBody({ agentId, kind, workItemId, body, harnessUsed, skillSkipped });
-  const file = _writeTempBodyFile(finalBody);
-  const env = _resolveTokenEnvForRepo(repo, resolveTokenForSlug);
-  try {
-    const output = _runGh(
-      execFileSync,
-      ['pr', 'review', String(prNumber), flag, '--repo', repo, '--body-file', file],
-      timeoutMs,
-      env,
-    );
-    return { output: String(output || '').trim(), bodyFile: file };
-  } finally {
-    try { fs.unlinkSync(file); } catch {}
-  }
-}
-
 module.exports = {
   // Builders / parsers (pure functions — usable from anywhere)
   buildMinionsCommentBody,
@@ -271,7 +227,6 @@ module.exports = {
   // gh wrappers (argv-form, --body-file)
   postPrComment,
   postPrReviewComment,
-  postPrReview,
   // Internal helpers exported for tests / advanced callers
   _buildMarker,
   _writeTempBodyFile,

package/engine/queries.js

@@ -2461,7 +2461,7 @@ function getPrdInfo(config) {
     const t = planTimings[wi.sourcePlan];
     if (wi.dispatched_at) { const d = new Date(wi.dispatched_at).getTime(); if (!t.firstDispatched || d < t.firstDispatched) t.firstDispatched = d; }
     if (wi.completedAt) { const c = new Date(wi.completedAt).getTime(); if (!t.lastCompleted || c > t.lastCompleted) t.lastCompleted = c; }
-    if (wi.status !== 'done') t.allDone = false;
+    if (!DONE_STATUSES.has(wi.status)) t.allDone = false;
   }
 
   const progress = {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.2334",
+  "version": "0.1.2336",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"