@yemi33/minions 0.1.885 → 0.1.887

package/CHANGELOG.md

@@ -1,11 +1,13 @@
 # Changelog
 
-## 0.1.885 (2026-04-11)
+## 0.1.887 (2026-04-11)
 
 ### Features
 -  add 13 missing CC action types for full dashboard parity
 
 ### Fixes
+- require explicit vote step in review playbook
+-  CC queued message silently retries on 429 after abort
 - track e2e/ and fix/ branches in PR reconciler
 - show stale banner on paused PRDs (#905) (#909)
 - regenerate-plan now dispatches diff-aware plan-to-prd agent

package/dashboard/js/command-center.js

@@ -531,6 +531,14 @@ async function _ccDoSend(message, skipUserMsg) {
     });
 
     if (!res.ok) {
+      // 429 = server still processing previous request (abort race) — retry silently up to 3 times
+      if (res.status === 429 && (!activeTab._429retries || activeTab._429retries < 3)) {
+        activeTab._429retries = (activeTab._429retries || 0) + 1;
+        _cleanupStreamDiv();
+        await new Promise(function(r) { setTimeout(r, 1500); });
+        return await _ccDoSend(message, true);
+      }
+      activeTab._429retries = 0;
       _cleanupStreamDiv();
       var errText = await res.text();
       addMsg('assistant', '<span style="color:var(--red)">' + escHtml(errText || 'CC error') + '</span>' +
@@ -648,7 +656,7 @@ async function _ccDoSend(message, skipUserMsg) {
         ' <button onclick="ccNewTab()" style="margin-top:6px;padding:4px 12px;background:var(--surface2);border:1px solid var(--border);border-radius:4px;color:var(--muted);cursor:pointer;font-size:11px">New Session</button>');
     }
   } finally {
-    if (activeTab) { activeTab._sending = false; activeTab._abortController = null; delete activeTab._streamedText; delete activeTab._toolsUsed; delete activeTab._sendStartedAt; }
+    if (activeTab) { activeTab._sending = false; activeTab._abortController = null; activeTab._429retries = 0; delete activeTab._streamedText; delete activeTab._toolsUsed; delete activeTab._sendStartedAt; }
     _ccSending = (_ccTabs.some(function(t) { return t._sending; }));
     ccRenderTabBar();
     try { clearInterval(phaseTimer); } catch { /* may not be defined if error before reader */ }

package/package.json

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

package/playbooks/review.md

@@ -40,26 +40,28 @@ Use subagents only for genuinely parallel, independent tasks (e.g., reviewing un
    - Verdict: **APPROVE**
    - Note any minor non-blocking suggestions
 
-## Post Review — Comment AND Vote on PR
+## Post Review — Two required actions, both mandatory
 
-Do both of the following:
+You MUST complete BOTH steps below. The review is NOT done until both are confirmed. A comment without a vote is an incomplete review.
 
-### Step 1: Leave a detailed review comment
+### Step 1: Post a detailed review comment
 
 {{pr_comment_instructions}}
-- pullRequestId: `{{pr_number}}`
 - content: Your full review with verdict, findings, and sign-off
 - Sign: `Review by Minions ({{agent_name}} — {{agent_role}})`
 
-### Step 2: Set your vote on the PR
+### Step 2: Submit a formal review vote — THIS IS REQUIRED
+
+**This is a separate action from Step 1.** Posting a comment does NOT submit a vote. You must explicitly run the vote command:
 
 {{pr_vote_instructions}}
-- pullRequestId: `{{pr_number}}`
-- If your verdict is **APPROVE**: vote `10` (approved)
-- If your verdict is **REQUEST_CHANGES**: vote `-10` (rejected)
-- If you have minor non-blocking suggestions: vote `5` (approved with suggestions)
+- If your verdict is **APPROVE**: use `--approve`
+- If your verdict is **REQUEST_CHANGES**: use `--request-changes`
+- If you have minor non-blocking suggestions only: use `--approve` with a note
+
+**Do not stop after Step 1.** The task is incomplete until Step 2 is done.
 
-This vote is visible to human reviewers in the PR UI and helps them understand the minions's assessment.
+After running the vote command, confirm it succeeded (check the command output for errors). If it fails, retry once.
 
 ## Handling Merge Conflicts
 If you encounter merge conflicts (e.g., the PR shows conflicts):
@@ -69,5 +71,9 @@ If you encounter merge conflicts (e.g., the PR shows conflicts):
 
 ## When to Stop
 
-Your task is complete once you have: (1) posted your review comment on the PR, and (2) cast your vote. Do NOT continue reading unrelated files or performing additional analysis. Stop after voting.
+Your task is complete only when BOTH of these are true:
+1. Review comment posted (Step 1)
+2. Formal vote submitted via `gh pr review --approve` or `--request-changes` (Step 2)
+
+Do NOT stop after only posting a comment. Do NOT continue reading unrelated files after voting.