@yemi33/minions 0.1.1612 → 0.1.1613

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.1612 (2026-04-28)
+## 0.1.1613 (2026-04-28)
 
 ### Features
+- fix npm test failures (#1846)
 - hard-pin agent assignment when CC names a specific agent
 - auto-detect available CLI runtimes and pin engine.defaultCli
 - match runtime tags to actual logos (pixel-crab Claude, mascot Copilot)

package/dashboard/js/utils.js

@@ -183,8 +183,8 @@ function copyLlmText(btn) {
   const clone = container.cloneNode(true);
   clone.querySelectorAll('.llm-copy-btn').forEach(b => b.remove());
   navigator.clipboard.writeText(clone.textContent.trim());
-  btn.innerHTML = '✓';
-  setTimeout(() => { btn.innerHTML = '⎘'; }, 1500);
+  btn.textContent = '\u2713';
+  setTimeout(() => { btn.textContent = '\u2398'; }, 1500);
 }
 
 /**

package/engine/queries.js

@@ -11,7 +11,7 @@ const shared = require('./shared');
 
 const { safeRead, safeReadDir, safeJson, safeWrite, getProjects, mutateJsonFileLocked,
   projectWorkItemsPath, projectPrPath, parseSkillFrontmatter, KB_CATEGORIES,
-  WI_STATUS, DONE_STATUSES, PRD_ITEM_STATUS, PR_STATUS, ENGINE_DEFAULTS } = shared;
+  WI_STATUS, DONE_STATUSES, PRD_ITEM_STATUS, PR_STATUS, ENGINE_DEFAULTS, DEFAULT_AGENT_METRICS } = shared;
 
 /**
  * Read the first `bytes` and last `bytes` of a file efficiently using byte offsets.
@@ -196,6 +196,14 @@ function getEngineLog() {
 function getMetrics() {
   const metrics = safeJson(path.join(ENGINE_DIR, 'metrics.json')) || {};
 
+  for (const [agentId, m] of Object.entries(metrics)) {
+    if (agentId.startsWith('_')) continue;
+    metrics[agentId] = {
+      ...DEFAULT_AGENT_METRICS,
+      ...(m && typeof m === 'object' && !Array.isArray(m) ? m : {}),
+    };
+  }
+
   // Enrich agent PR counts from pull-requests.json (source of truth)
   const allPrs = getPullRequests();
   const prCountByAgent = {};

package/engine/routing.js

@@ -116,7 +116,15 @@ function setTempBudget(n) {
 }
 function getTempBudget() { return _tempBudget; }
 
-function resolveAgent(workType, config, authorAgent = null) {
+function normalizeAgentHints(agentHints, authorAgent = null) {
+  const raw = Array.isArray(agentHints) ? agentHints : (agentHints ? [agentHints] : []);
+  return raw
+    .map(id => id === '_author_' ? authorAgent : id)
+    .map(id => typeof id === 'string' ? id.trim().toLowerCase() : '')
+    .filter(Boolean);
+}
+
+function resolveAgent(workType, config, authorAgent = null, agentHints = null) {
   const routes = getRoutingTableCached();
   const route = routes[workType] || routes['implement'];
   const agents = config.agents || {};
@@ -145,6 +153,14 @@ function resolveAgent(workType, config, authorAgent = null) {
     return null;
   };
 
+  const hintedAgents = normalizeAgentHints(agentHints, authorAgent);
+  if (hintedAgents.length > 0) {
+    for (const id of hintedAgents) {
+      if (isAvailable(id)) { _claimedAgents.add(id); return id; }
+    }
+    return null;
+  }
+
   // Resolve _any_ token — pick any available agent (#480)
   if (preferred === '_any_') { const pick = pickAnyIdle(); if (pick) return pick; }
   else if (preferred && isAvailable(preferred)) { _claimedAgents.add(preferred); return preferred; }
@@ -192,4 +208,5 @@ module.exports = {
   resolveAgent,
   setTempBudget,
   getTempBudget,
+  normalizeAgentHints,
 };

package/engine.js

@@ -2441,7 +2441,8 @@ function discoverFromWorkItems(config, project) {
       item._decomposing = true;
       needsWrite = true;
     }
-    const agentId = item.agent || resolveAgent(workType, config);
+    const agentHints = item.preferred_agent || item.agents || null;
+    const agentId = item.agent || resolveAgent(workType, config, null, agentHints);
     if (!agentId) {
       // Check if reason is budget
       const cfgAgents = config.agents || {};
@@ -2960,7 +2961,8 @@ function discoverCentralWorkItems(config) {
 
     } else {
       // ─── Normal: single agent dispatch ──────────────────────────────
-      const agentId = item.agent || resolveAgent(workType, config);
+      const agentHints = item.preferred_agent || item.agents || null;
+      const agentId = item.agent || resolveAgent(workType, config, null, agentHints);
       if (!agentId) continue;
 
       const agentName = config.agents[agentId]?.name || agentId;
@@ -3602,7 +3604,8 @@ async function tickInner() {
     // be of type string. Received undefined` and re-queues — every tick. Try to
     // resolve a fallback via routing; if none is available, skip this tick.
     if (!item.agent || typeof item.agent !== 'string') {
-      const fallback = resolveAgent(item.type || WORK_TYPE.FIX, config);
+      const agentHints = item.meta?.item?.preferred_agent || item.meta?.item?.agents || null;
+      const fallback = resolveAgent(item.type || WORK_TYPE.FIX, config, null, agentHints);
       if (!fallback) {
         log('warn', `Pending dispatch ${item.id} has no agent and routing returned no fallback — skipping`);
         continue;
@@ -3660,7 +3663,7 @@ async function tickInner() {
       // Agent busy reassignment: if item has been waiting on a busy agent past the threshold,
       // try to find an alternative agent via routing. Skip explicitly assigned items.
       const reassignMs = config.engine?.agentBusyReassignMs ?? ENGINE_DEFAULTS.agentBusyReassignMs;
-      const isExplicitReassign = !!item.meta?.item?.agent;
+      const isExplicitReassign = !!(item.meta?.item?.agent || item.meta?.item?.preferred_agent || item.meta?.item?.agents?.length);
       if (!isExplicitReassign && reassignMs > 0 && item._agentBusySince) {
         const busySinceMs = new Date(item._agentBusySince).getTime();
         if (Date.now() - busySinceMs > reassignMs) {

package/minions.js

@@ -465,6 +465,7 @@ async function initMinions({ skipScan = false, scanRoot, scanDepth } = {}) {
   // Merge defaults — fills in new fields from upgrades while preserving user customizations
   if (!config.engine) config.engine = {};
   for (const [k, v] of Object.entries(ENGINE_DEFAULTS)) {
+    if (k === 'defaultCli') continue;
     if (config.engine[k] === undefined) config.engine[k] = v;
   }
   if (!config.claude) config.claude = {};

package/package.json

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