@yemi33/minions 0.1.1550 → 0.1.1552

package/CHANGELOG.md

@@ -1,12 +1,18 @@
 # Changelog
 
-## 0.1.1550 (2026-04-24)
+## 0.1.1552 (2026-04-27)
 
 ### Fixes
--  escape </script> literal in utils.js (closes #1746) (#1751)
+- scan all project subdirs for PR data in PRD view
+
+## 0.1.1551 (2026-04-27)
+
+### Fixes
+- derive PR URL from canonical PR ID scope, not projects[0]
+
+## 0.1.1549 (2026-04-24)
 
 ### Other
-- test(engine): add unit tests for isWorktreeRetryableError, removeStaleIndexLock, _maxTurnsForType, buildProjectContext, normalizeAc (#1752)
 - test(timeout): add unit tests for checkIdleThreshold, checkSteering, checkTimeouts (#1748)
 
 ## 0.1.1548 (2026-04-23)

package/engine/queries.js

@@ -1021,6 +1021,52 @@ function getPrdInfo(config) {
   const allPrs = getPullRequests(config);
   const prById = {};
   for (const pr of allPrs) prById[pr.id] = pr;
+  // Also scan project subdirectories that aren't in config (e.g. removed projects
+  // whose pull-requests.json still holds last-known status). getPrLinks already
+  // reads all subdirs, so without this PRs from those dirs would have status/title
+  // missing in the PRD view and fall back to literal 'active'/'' defaults.
+  try {
+    const projectsDir = path.join(MINIONS_DIR, 'projects');
+    const projectsByName = new Map(projects.map(p => [p.name, p]));
+    for (const d of fs.readdirSync(projectsDir, { withFileTypes: true })) {
+      if (!d.isDirectory() || projectsByName.has(d.name)) continue;
+      const ghostPath = path.join(projectsDir, d.name, 'pull-requests.json');
+      const ghostPrs = safeJson(ghostPath);
+      if (!Array.isArray(ghostPrs)) continue;
+      shared.normalizePrRecords(ghostPrs, null);
+      for (const pr of ghostPrs) {
+        if (pr?.id && !prById[pr.id]) {
+          pr._project = d.name;
+          prById[pr.id] = pr;
+        }
+      }
+    }
+  } catch { /* projects dir missing */ }
+
+  // Build URL for a PR when pr.url isn't set. Prefer the PR ID's own scope
+  // (e.g. `github:owner/repo#123` → github.com URL) so a github PR never gets
+  // an ADO URL just because the only configured project happens to be ADO.
+  // Only use a project's prUrlBase when its host actually matches the PR.
+  const _buildPrUrlFromId = (prId, pr) => {
+    if (pr?.url) return pr.url;
+    const canonical = shared.parseCanonicalPrId(prId);
+    if (canonical) {
+      const [host, rest] = canonical.scope.split(':');
+      if (host === 'github') return `https://github.com/${rest}/pull/${canonical.prNumber}`;
+      if (host === 'ado') {
+        const [org, adoProject, repo] = rest.split('/');
+        if (org && adoProject && repo) {
+          return `https://dev.azure.com/${org}/${adoProject}/_git/${repo}/pullrequest/${canonical.prNumber}`;
+        }
+      }
+    }
+    // Legacy `PR-N` ID with no host scope: only use project's prUrlBase if a
+    // matching project (by name) exists. Never blindly fall back to projects[0].
+    const project = pr?._project ? projects.find(p => p.name === pr._project) : null;
+    const prNumber = shared.getPrNumber(pr || prId);
+    if (project?.prUrlBase && prNumber != null) return project.prUrlBase + prNumber;
+    return '';
+  };
 
   const prdToPr = {};
   const prLinks = shared.getPrLinks(); // { "PR-xxxx": ["P-xxxx", "P-yyyy"] }
@@ -1030,9 +1076,7 @@ function getPrdInfo(config) {
     // (or are typed as verify) and would bleed through as duplicate entries on every
     // constituent item. They are surfaced via renderE2eSection instead. (#1220)
     if ((itemIds || []).length > 1 || pr?.itemType === 'verify' || pr?.title?.startsWith('[E2E]')) continue;
-    const project = projects.find(p => p.name === pr?._project) || projects[0] || null;
-    const prNumber = shared.getPrNumber(pr || prId);
-    const url = pr?.url || (project?.prUrlBase && prNumber != null ? project.prUrlBase + prNumber : '');
+    const url = _buildPrUrlFromId(prId, pr);
     for (const itemId of (itemIds || [])) {
       if (!prdToPr[itemId]) prdToPr[itemId] = [];
       prdToPr[itemId].push({ id: prId, url, title: pr?.title || '', status: pr?.status || 'active', _project: pr?._project || '' });
@@ -1046,8 +1090,7 @@ function getPrdInfo(config) {
     const exactPr = prById[canonicalPrId] || null;
     const displayMatches = exactPr ? [] : Object.values(prById).filter(candidate => shared.getPrDisplayId(candidate) === shared.getPrDisplayId(wi._pr));
     const pr = exactPr || (displayMatches.length === 1 ? displayMatches[0] : null);
-    const prNumber = shared.getPrNumber(pr || wi._pr);
-    const url = pr?.url || (project?.prUrlBase && prNumber != null ? project.prUrlBase + prNumber : '');
+    const url = _buildPrUrlFromId(canonicalPrId || wi._pr, pr);
     prdToPr[wi.id] = [{ id: pr?.id || canonicalPrId || wi._pr, url, title: pr?.title || '', status: pr?.status || 'active', _project: project?.name || '' }];
   }
   // Aggregate sub-task PRs to decomposed parent (sub-tasks aren't PRD items but their PRs should show)
@@ -1060,9 +1103,7 @@ function getPrdInfo(config) {
       const parentId = wi.parent_id;
       if (!prdToPr[parentId]) prdToPr[parentId] = [];
       if (!prdToPr[parentId].some(p => p.id === pr.id)) {
-        const project = projects.find(p => p.name === pr._project) || projects[0] || null;
-        const prNumber = shared.getPrNumber(pr);
-        const url = pr.url || (project?.prUrlBase && prNumber != null ? project.prUrlBase + prNumber : '');
+        const url = _buildPrUrlFromId(pr.id, pr);
         prdToPr[parentId].push({ id: pr.id, url, title: pr.title || '', status: pr.status || 'active', _project: pr._project || '' });
       }
     }

package/package.json

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