@yemi33/minions 0.1.783 → 0.1.785
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +5 -1
- package/engine/lifecycle.js +4 -3
- package/package.json +1 -1
package/CHANGELOG.md
CHANGED
|
@@ -1,10 +1,14 @@
|
|
|
1
1
|
# Changelog
|
|
2
2
|
|
|
3
|
-
## 0.1.
|
|
3
|
+
## 0.1.785 (2026-04-10)
|
|
4
4
|
|
|
5
5
|
### Features
|
|
6
6
|
- cap review→fix cycles at evalMaxIterations (default 3)
|
|
7
7
|
|
|
8
|
+
### Fixes
|
|
9
|
+
- don't overwrite reviewStatus when platform vote hasn't propagated
|
|
10
|
+
- don't overwrite approval with 'waiting' when platform API lags
|
|
11
|
+
|
|
8
12
|
## 0.1.782 (2026-04-10)
|
|
9
13
|
|
|
10
14
|
### Features
|
package/engine/lifecycle.js
CHANGED
|
@@ -730,7 +730,9 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary)
|
|
|
730
730
|
const completedEntry = (dispatch.completed || []).find(d => d.agent === agentId && d.type === 'review');
|
|
731
731
|
|
|
732
732
|
// Check actual review status from the platform (agent may have approved or requested changes)
|
|
733
|
-
|
|
733
|
+
// If platform hasn't propagated the vote yet (returns 'pending'), keep current status unchanged.
|
|
734
|
+
// The poller will pick up the real status on the next cycle (~3 min).
|
|
735
|
+
let postReviewStatus = null; // null = don't change
|
|
734
736
|
try {
|
|
735
737
|
const projectObj = project || shared.getProjects(config)[0];
|
|
736
738
|
if (projectObj) {
|
|
@@ -739,7 +741,6 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary)
|
|
|
739
741
|
? require('./github').checkLiveReviewStatus
|
|
740
742
|
: require('./ado').checkLiveReviewStatus;
|
|
741
743
|
const liveStatus = await checkFn(pr, projectObj);
|
|
742
|
-
// Use live status only if it's a decisive verdict (not 'pending' — review may not have propagated yet)
|
|
743
744
|
if (liveStatus && liveStatus !== 'pending') postReviewStatus = liveStatus;
|
|
744
745
|
}
|
|
745
746
|
} catch (e) { log('warn', `Post-review status check for ${pr.id}: ${e.message}`); }
|
|
@@ -750,7 +751,7 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary)
|
|
|
750
751
|
if (!Array.isArray(prs)) return prs;
|
|
751
752
|
const target = prs.find(p => p.id === pr.id);
|
|
752
753
|
if (!target) return prs;
|
|
753
|
-
target.reviewStatus = postReviewStatus;
|
|
754
|
+
if (postReviewStatus) target.reviewStatus = postReviewStatus; // only update if live check returned decisive result
|
|
754
755
|
target.lastReviewedAt = ts();
|
|
755
756
|
target.minionsReview = {
|
|
756
757
|
reviewer: reviewerName,
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@yemi33/minions",
|
|
3
|
-
"version": "0.1.
|
|
3
|
+
"version": "0.1.785",
|
|
4
4
|
"description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
|
|
5
5
|
"bin": {
|
|
6
6
|
"minions": "bin/minions.js"
|