@yemi33/minions 0.1.2112 → 0.1.2113

package/engine/check-status.js

@@ -4,7 +4,7 @@ const dir = path.resolve(__dirname, '..');
 
 console.log('=== Work Items (non-done) ===');
 let items = [];
-try { items = JSON.parse(fs.readFileSync(path.join(dir, 'work-items.json'), 'utf8')); } catch {}
+try { items = require('./queries').getWorkItems() || []; } catch {}
 items.filter(i => i.status !== 'done').forEach(i => {
   console.log(i.id, (i.status || '').padEnd(12), (i.type || '').padEnd(12), (i.title || '').slice(0, 60));
 });

package/engine/shared.js

@@ -4970,33 +4970,24 @@ function getPrLinks() {
     if (!knownPrIdsByDisplay.has(displayId)) knownPrIdsByDisplay.set(displayId, new Set());
     knownPrIdsByDisplay.get(displayId).add(pr.id);
   };
-  // Primary source: derive from all projects/*/pull-requests.json prdItems
-  const projectsDir = path.join(MINIONS_DIR, 'projects');
+  // Primary source: derive from all per-project + central PR records via the
+  // SQL-canonical store (Phase 9). `readAllPullRequests` decorates each row
+  // with `_scope` so we can look up the owning project for legacy-ID
+  // normalization.
   const projectsByName = new Map(getProjects().map(project => [project.name || path.basename(project.localPath || ''), project]));
   try {
-    for (const d of fs.readdirSync(projectsDir, { withFileTypes: true })) {
-      if (!d.isDirectory()) continue;
-      try {
-        const prs = JSON.parse(fs.readFileSync(path.join(projectsDir, d.name, 'pull-requests.json'), 'utf8'));
-        const project = projectsByName.get(d.name) || null;
-        normalizePrRecords(prs, project);
-        for (const pr of prs) {
-          if (!pr.id) continue;
-          registerPrId(pr);
-          mergePrLinkItems(links, pr.id, pr.prdItems || []);
-        }
-      } catch { /* missing or invalid */ }
-    }
-  } catch { /* projects dir missing */ }
-  try {
-    const centralPrs = JSON.parse(fs.readFileSync(path.join(MINIONS_DIR, 'pull-requests.json'), 'utf8'));
-    normalizePrRecords(centralPrs, null);
-    for (const pr of centralPrs) {
-      if (!pr.id) continue;
+    const store = require('./pull-requests-store');
+    const allPrs = store.readAllPullRequests() || [];
+    for (const pr of allPrs) {
+      if (!pr?.id) continue;
+      const scope = pr._scope;
+      delete pr._scope;
+      const project = scope && scope !== 'central' ? (projectsByName.get(scope) || null) : null;
+      normalizePrRecords([pr], project);
       registerPrId(pr);
       mergePrLinkItems(links, pr.id, pr.prdItems || []);
     }
-  } catch { /* central file optional */ }
+  } catch { /* SQL unavailable — skip primary source */ }
   // Fallback: static pr-links.json for entries not covered above
   try {
     const static_ = JSON.parse(fs.readFileSync(PR_LINKS_PATH, 'utf8'));
@@ -5054,27 +5045,21 @@ function addPrLink(prId, itemId, { project = null, url = '', prNumber = null } =
   if (!project) return;
   const prPath = projectPrPath(project);
   const effectivePrNumber = getPrNumber(prNumber ?? effectivePrId);
-  const prLockPath = `${prPath}.lock`;
-  withFileLock(prLockPath, () => {
-    if (!fs.existsSync(prPath)) return;
-    let prs = safeJson(prPath);
-    if (!Array.isArray(prs)) prs = [];
+  // Phase 9 SQL routing: mutatePullRequests writes through pull-requests-store
+  // so SQL stays canonical; the JSON mirror is regenerated by the store.
+  mutatePullRequests(prPath, (prs) => {
+    if (!Array.isArray(prs) || prs.length === 0) return prs;
     normalizePrRecords(prs, project);
     const existingPr = prs.find(pr =>
       pr?.id === effectivePrId
       || (url && pr?.url === url)
       || (effectivePrNumber != null && getPrNumber(pr) === effectivePrNumber)
     );
-    if (!existingPr) return;
-    const backupPath = prPath + '.backup';
-    try { if (fs.existsSync(prPath)) fs.copyFileSync(prPath, backupPath); } catch { /* backup is best-effort */ }
+    if (!existingPr) return prs;
     existingPr.prdItems = Array.isArray(existingPr.prdItems) ? existingPr.prdItems : [];
-    if (existingPr.prdItems.includes(itemId)) return;
+    if (existingPr.prdItems.includes(itemId)) return prs;
     existingPr.prdItems.push(itemId);
-    safeWrite(prPath, prs);
-  }, {
-    retries: ENGINE_DEFAULTS.lockRetries,
-    retryBackoffMs: ENGINE_DEFAULTS.lockRetryBackoffMs
+    return prs;
   });
 }
 

package/package.json

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