@yemi33/minions 0.1.1766 → 0.1.1768

package/CHANGELOG.md

@@ -1,6 +1,16 @@
 # Changelog
 
-## 0.1.1766 (2026-05-07)
+## 0.1.1768 (2026-05-07)
+
+### Fixes
+- stamp dispatchItem.started_at locally so updateMetrics sees it (#2158)
+- stop test fixtures leaking into live engine/log.json (#2154)
+
+### Other
+- test(engine): add unit tests for PR branch/cause resolution helpers (#2160)
+- test(dispatch): add unit tests for dedup/lock-key + retry-classification helpers (#2159)
+
+## 0.1.1765 (2026-05-07)
 
 ### Features
 - suppress duplicate browser tabs on restart/update
@@ -8,7 +18,6 @@
 - probe every registered runtime adapter, not just claude
 
 ### Fixes
-- silence 'couldn't find remote ref' warnings on worktree reuse for never-pushed branches (#2155)
 - close remaining test isolation leaks + add end-of-run regression guard (#2152)
 - surface legacy .minions dir removal in logs
 - clear session.json when conversation jsonl missing (W-mouugzow00068741) (#2153)

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-05-07T03:30:56.716Z"
+  "cachedAt": "2026-05-07T15:03:20.055Z"
 }

package/engine/dispatch.js

@@ -658,4 +658,11 @@ module.exports = {
   cancelPendingDispatchesForPr,
   cleanDispatchEntries,
   cancelPendingWorkItems,
+  // exported for testing
+  getDispatchProjectKey,
+  getPrDispatchTargetKey,
+  getBranchDispatchLockKey,
+  findActivePrOrBranchLock,
+  normalizeRetryableDecision,
+  isCompletedWorkItemForFailure,
 };

package/engine/shared.js

@@ -168,12 +168,26 @@ function log(level, msg, meta = {}) {
  * That leaked test pollution into the live engine log (e.g. `_test/backoff-*`,
  * `this-playbook-does-not-exist-xyz`, `TEST-EXT-*` meeting IDs).
  *
- * Lazy resolution honors the *current* `MINIONS_TEST_DIR` at flush time, so
- * even unbussed dependents write to the test dir while the test owns it.
+ * Resolution order:
+ *   1. `MINIONS_TEST_DIR` — per-test isolation root (regression tests rely
+ *      on this — log goes to `<tmpdir>/engine/log.json`).
+ *   2. `MINIONS_LOG_PATH` — default test log target. Set at the top of
+ *      `test/unit.test.js` so tests that exercise log() without explicit
+ *      isolation (e.g. github backoff helpers, cooldown helpers) still land
+ *      in a benign tmp file instead of the live `D:/squad/engine/log.json`.
+ *   3. `MINIONS_HOME` / repo root fallback — production behavior.
+ *
+ * Lazy resolution honors the *current* env at flush time, so even unbussed
+ * dependents write to the test dir while the test owns it.
  */
 function _currentLogPath() {
-  const root = process.env.MINIONS_TEST_DIR
-    || (process.env.MINIONS_HOME ? path.resolve(process.env.MINIONS_HOME) : path.resolve(__dirname, '..'));
+  if (process.env.MINIONS_TEST_DIR) {
+    return path.join(path.resolve(process.env.MINIONS_TEST_DIR), 'engine', 'log.json');
+  }
+  if (process.env.MINIONS_LOG_PATH) {
+    return path.resolve(process.env.MINIONS_LOG_PATH);
+  }
+  const root = process.env.MINIONS_HOME ? path.resolve(process.env.MINIONS_HOME) : path.resolve(__dirname, '..');
   return path.join(root, 'engine', 'log.json');
 }
 

package/engine.js

@@ -1692,6 +1692,10 @@ async function spawnAgent(dispatchItem, config) {
     }
     return dispatch;
   });
+  // Also stamp the local ref — mutateDispatch re-reads from disk, so `item`
+  // above is a different object than the `dispatchItem` parameter. updateMetrics
+  // reads dispatchItem.started_at for runtimeMs. (W-moux9nwn0008f923)
+  dispatchItem.started_at = startedAt;
 
   // Atomically stamp dispatched_to/dispatched_at on the originating work item (#402)
   // The discover phase sets these via safeWrite which can race with concurrent writes;
@@ -4709,6 +4713,7 @@ module.exports = {
   parseConflictFiles, pruneAncestorDeps, preflightMergeSimulation, // exported for testing
   isWorktreeRetryableError, removeStaleIndexLock, syncReusedWorktree, // exported for testing
   _maxTurnsForType, buildProjectContext, normalizeAc, _buildAgentSpawnFlags, _classifyAgentFailure, // exported for testing
+  normalizePrBranch, resolvePrBranch, prCausePart, getPrCauseHead, getPrCauseBase, getPrAutomationCauseKey, getPrAutomationDispatchKey, // exported for testing
 
   // Playbooks
   renderPlaybook, validatePlaybookVars, PLAYBOOK_REQUIRED_VARS, buildWorkItemDispatchVars,

package/package.json

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