@yemi33/minions 0.1.987 → 0.1.989

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.987 (2026-04-15)
+## 0.1.989 (2026-04-15)
 
 ### Features
+- remove DEFAULTS alias from timeout.js — standardize to ENGINE_DEFAULTS
 - complete watches management — new conditions, CC actions, PROJECTS fix (#1103)
 - fix misleading poll frequency field labels in settings modal (#1099)
 - add Auto-complete PRs toggle to settings modal (#1095)
@@ -22,7 +23,6 @@
 - update detail-panel.js Output Log tab to use renderAgentOutput
 -  Update command-center.js to capture tool inputs and use formatted summaries
 - certificate-based auth for Teams integration (#1027)
--  Create render-utils.js with shared formatting helpers
 
 ### Fixes
 -  updatePrAfterReview preserves fixedAt when writing minionsReview

package/engine/lifecycle.js

@@ -909,7 +909,9 @@ async function updatePrAfterReview(agentId, pr, project, config, resultSummary)
       reviewer: reviewerName,
       reviewedAt: ts(),
       note: resultSummary || completedEntry?.task || '',
-      ...(target.minionsReview?.fixedAt ? { fixedAt: target.minionsReview.fixedAt } : {}),
+      // Preserve fixedAt across re-reviews so the poller guard knows a fix was pushed.
+      // Drop it when reviewer requests changes again — that starts a new fix cycle.
+      ...(target.minionsReview?.fixedAt && postReviewStatus !== 'changes-requested' ? { fixedAt: target.minionsReview.fixedAt } : {}),
     };
     updatedTarget = { ...pr, ...target };
     return prs;

package/engine/timeout.js

@@ -9,7 +9,7 @@ const shared = require('./shared');
 const queries = require('./queries');
 
 const { safeRead, safeWrite, safeJson, mutateJsonFileLocked, getProjects, projectWorkItemsPath, log, ts,
-  ENGINE_DEFAULTS: DEFAULTS, WI_STATUS, WORK_TYPE, DISPATCH_RESULT, AGENT_STATUS } = shared;
+  ENGINE_DEFAULTS, WI_STATUS, WORK_TYPE, DISPATCH_RESULT, AGENT_STATUS } = shared;
 const { getDispatch, getAgentStatus } = queries;
 const AGENTS_DIR = queries.AGENTS_DIR;
 const MINIONS_DIR = shared.MINIONS_DIR;
@@ -131,11 +131,11 @@ function checkTimeouts(config) {
   const { completeDispatch } = dispatch();
   const { runPostCompletionHooks } = require('./lifecycle');
 
-  const timeout = config.engine?.agentTimeout || DEFAULTS.agentTimeout;
-  const defaultHeartbeatTimeout = config.engine?.heartbeatTimeout || DEFAULTS.heartbeatTimeout;
+  const timeout = config.engine?.agentTimeout || ENGINE_DEFAULTS.agentTimeout;
+  const defaultHeartbeatTimeout = config.engine?.heartbeatTimeout || ENGINE_DEFAULTS.heartbeatTimeout;
 
   // Per-type heartbeat timeouts: merge ENGINE_DEFAULTS ← config overrides
-  const perTypeTimeouts = { ...DEFAULTS.heartbeatTimeouts, ...(config.engine?.heartbeatTimeouts || {}) };
+  const perTypeTimeouts = { ...ENGINE_DEFAULTS.heartbeatTimeouts, ...(config.engine?.heartbeatTimeouts || {}) };
 
   // 1. Check tracked processes for hard timeout (supports per-item deadline from fan-out)
   for (const [id, info] of activeProcesses.entries()) {
@@ -389,7 +389,7 @@ function checkTimeouts(config) {
           (dispatchData.active || []).some(d => d.meta?.item?.id === item.id);
         if (!isActive) {
           const retries = (item._retryCount || 0);
-          const maxRetries = DEFAULTS.maxRetries;
+          const maxRetries = ENGINE_DEFAULTS.maxRetries;
           if (retries < maxRetries) {
             log('info', `Reconcile: work item ${item.id} agent died — auto-retry ${retries + 1}/${maxRetries}`);
             item.status = WI_STATUS.PENDING;

package/package.json

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