@yemi33/minions 0.1.193 → 0.1.194

package/CHANGELOG.md

@@ -1,8 +1,11 @@
 # Changelog
 
-## 0.1.193 (2026-04-02)
+## 0.1.194 (2026-04-02)
 
 ### Engine
+- engine/ado.js
+- engine/github.js
+- engine/lifecycle.js
 - engine/queries.js
 - engine/shared.js
 
@@ -14,6 +17,9 @@
 ### Documentation
 - README.md
 
+### Other
+- test/unit.test.js
+
 ## 0.1.189 (2026-04-02)
 
 ### Dashboard

package/engine/ado.js

@@ -5,7 +5,7 @@
 
 const path = require('path');
 const shared = require('./shared');
-const { exec, getAdoOrgBase, addPrLink, log, dateStamp } = shared;
+const { exec, getAdoOrgBase, log, dateStamp } = shared;
 const { getPrs } = require('./queries');
 
 // Lazy require to avoid circular dependency — only needed for engine().handlePostMerge
@@ -366,9 +366,8 @@ async function reconcilePrs(config) {
       const confirmedItemId = linkedItem ? linkedItemId : null;
 
       if (existingIds.has(prId)) {
-        // PR already tracked — write link to pr-links.json if we can extract an ID
+        // PR already tracked — update prdItems if we can extract an ID
         if (confirmedItemId) {
-          addPrLink(prId, confirmedItemId);
           const existing = existingPrs.find(p => p.id === prId);
           if (existing && !(existing.prdItems || []).includes(confirmedItemId)) {
             existing.prdItems = Array.isArray(existing.prdItems) ? existing.prdItems : [];
@@ -391,7 +390,6 @@ async function reconcilePrs(config) {
         url: prUrl,
         prdItems: confirmedItemId ? [confirmedItemId] : [],
       });
-      if (confirmedItemId) addPrLink(prId, confirmedItemId);
       existingIds.add(prId);
       projectAdded++;
       log('info', `PR reconciliation: added ${prId} (branch: ${branch}${confirmedItemId ? ', linked to ' + confirmedItemId : ''}) to ${project.name}`);

package/engine/github.js

@@ -5,7 +5,7 @@
  */
 
 const shared = require('./shared');
-const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR, addPrLink, log, dateStamp } = shared;
+const { exec, getProjects, projectPrPath, projectWorkItemsPath, safeJson, safeWrite, MINIONS_DIR, log, dateStamp } = shared;
 const { getPrs } = require('./queries');
 const path = require('path');
 
@@ -341,7 +341,6 @@ async function reconcilePrs(config) {
 
       if (existingIds.has(prId)) {
         if (confirmedItemId) {
-          addPrLink(prId, confirmedItemId);
           const existing = existingPrs.find(p => p.id === prId);
           if (existing && !(existing.prdItems || []).includes(confirmedItemId)) {
             existing.prdItems = Array.isArray(existing.prdItems) ? existing.prdItems : [];
@@ -364,7 +363,6 @@ async function reconcilePrs(config) {
         url: prUrl,
         prdItems: confirmedItemId ? [confirmedItemId] : [],
       });
-      if (confirmedItemId) addPrLink(prId, confirmedItemId);
       existingIds.add(prId);
       projectAdded++;
 

package/engine/lifecycle.js

@@ -6,7 +6,7 @@
 const fs = require('fs');
 const path = require('path');
 const shared = require('./shared');
-const { safeRead, safeJson, safeWrite, safeReadDir, execSilent, projectPrPath, getPrLinks, addPrLink,
+const { safeRead, safeJson, safeWrite, safeReadDir, execSilent, projectPrPath, getPrLinks,
   mutateJsonFileLocked, log, ts, dateStamp } = shared;
 const { trackEngineUsage } = require('./llm');
 const queries = require('./queries');
@@ -735,7 +735,7 @@ async function handlePostMerge(pr, project, config, newStatus) {
   // Mark review as approved since it was merged
   pr.reviewStatus = 'approved';
 
-  // Resolve linked work item from pr-links or PR branch name
+  // Resolve linked work item from PR.prdItems or PR branch name
   let mergedItemId = getPrLinks()[pr.id];
   if (!mergedItemId && pr.branch) {
     const branchMatch = pr.branch.match(/(P-[a-z0-9]{6,})/i) || pr.branch.match(/(W-[a-z0-9]+)/i);

package/engine/queries.js

@@ -582,7 +582,7 @@ function getPrdInfo(config) {
     for (const wi of centralWi) { if (!wi.id) { console.warn('[queries] Skipping central work item without id:', JSON.stringify(wi).slice(0, 120)); continue; } if (wi.sourcePlan && !wiById[wi.id]) wiById[wi.id] = wi; }
   } catch { /* optional */ }
 
-  // PR-to-PRD linking — primary source is pr-links.json (single-writer, never clobbered by polling)
+  // PR-to-PRD linking — derived from PR.prdItems (single source of truth)
   const allPrs = getPullRequests(config);
   const prById = {};
   for (const pr of allPrs) prById[pr.id] = pr;

package/engine/shared.js

@@ -8,7 +8,6 @@ const path = require('path');
 
 const MINIONS_DIR = path.resolve(__dirname, '..');
 const CENTRAL_WI_PATH = path.join(MINIONS_DIR, 'work-items.json');
-const PR_LINKS_PATH = path.join(MINIONS_DIR, 'engine', 'pr-links.json');
 const LOG_PATH = path.join(__dirname, 'log.json');
 
 // ── Timestamps & Logging ────────────────────────────────────────────────────
@@ -502,7 +501,7 @@ function parseSkillFrontmatter(content, filename) {
 // Never touched by polling loops — only written when a PR is first linked to a PRD item.
 
 function getPrLinks() {
-  // Derive from PR.prdItems (single source of truth) + legacy pr-links.json as fallback
+  // Derive from PR.prdItems (single source of truth)
   const links = {};
   try {
     const projects = getProjects();
@@ -515,24 +514,9 @@ function getPrLinks() {
       }
     }
   } catch { /* optional */ }
-  // Merge legacy pr-links.json for items not yet in PR.prdItems
-  try {
-    const legacy = JSON.parse(require('fs').readFileSync(PR_LINKS_PATH, 'utf8'));
-    for (const [prId, itemId] of Object.entries(legacy)) {
-      if (!links[prId]) links[prId] = itemId;
-    }
-  } catch { /* optional */ }
   return links;
 }
 
-function addPrLink(prId, itemId) {
-  if (!prId || !itemId) return;
-  const links = getPrLinks();
-  if (links[prId] === itemId) return; // already correct, no write needed
-  links[prId] = itemId;
-  safeWrite(PR_LINKS_PATH, links);
-}
-
 /**
  * Locked mutation of a project's pull-requests.json.
  * Single source of truth for PR data including prdItems links.
@@ -563,7 +547,6 @@ function linkPrToItem(project, prId, itemId) {
 module.exports = {
   MINIONS_DIR,
   CENTRAL_WI_PATH,
-  PR_LINKS_PATH,
   LOG_PATH,
   ts,
   logTs,
@@ -597,7 +580,6 @@ module.exports = {
   projectWorkItemsPath,
   projectPrPath,
   getPrLinks,
-  addPrLink,
   mutatePrs,
   linkPrToItem,
   nextWorkItemId,

package/package.json

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