@yemi33/minions 0.1.257 → 0.1.258

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.258 (2026-04-03)
+
+### Fixes
+-  remove last evaluate references — eval loop now creates review items
+
 ## 0.1.257 (2026-04-03)
 
 ### Features

package/engine/lifecycle.js

@@ -1266,8 +1266,8 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
           mutateJsonFileLocked(wiPath, (items) => {
             // For fix items, target the original implement parent; for implement, target self
             const evalTargetId = type === 'fix' ? meta.item._evalParentId : meta.item.id;
-            // Dedup: skip if an evaluate item already exists for this parent
-            const existing = items.find(i => i._evalParentId === evalTargetId && i.type === 'evaluate' && i.status === 'pending');
+            // Dedup: skip if a review item already exists for this parent
+            const existing = items.find(i => i._evalParentId === evalTargetId && i.type === 'review' && i.status === 'pending');
             if (existing) {
               log('info', `Eval loop: evaluate item ${existing.id} already exists for ${evalTargetId}, skipping`);
               return items;
@@ -1276,7 +1276,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
             const evalItem = {
               id: 'W-' + shared.uid(),
               title: `Evaluate: ${parentItem?.title || meta.item.title || evalTargetId}`,
-              type: 'evaluate',
+              type: 'review',
               priority: meta.item.priority || 'high',
               status: 'pending',
               created: ts(),
@@ -1302,7 +1302,7 @@ function runPostCompletionHooks(dispatchItem, agentId, code, stdout, config) {
   }
 
   // Evaluate completion: parse verdict and handle eval→fix iteration loop
-  if (isSuccess && type === 'evaluate' && meta?.item?._evalParentId) {
+  if (isSuccess && type === 'review' && meta?.item?._evalParentId) {
     try {
       const verdict = parseEvalVerdict(resultSummary || stdout);
       const evalLoop = config.engine?.evalLoop ?? shared.ENGINE_DEFAULTS.evalLoop;

package/engine/playbook.js

@@ -481,7 +481,7 @@ function selectPlaybook(workType, item) {
   if (workType === 'review' && !item?._pr && !item?.pr_id) {
     return 'work-item';
   }
-  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'evaluate', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
+  const typeSpecificPlaybooks = ['explore', 'review', 'test', 'plan-to-prd', 'plan', 'ask', 'verify', 'decompose', 'meeting-investigate', 'meeting-debate', 'meeting-conclude'];
   return typeSpecificPlaybooks.includes(workType) ? workType : 'work-item';
 }
 

package/package.json

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