@yemi33/minions 0.1.1638 → 0.1.1639

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.1638 (2026-04-30)
+## 0.1.1639 (2026-04-30)
 
 ### Fixes
+- auto-link agent-created PRs to work items (#1904)
 -  Playbook 'fix' / 'review' gates items forever when pr_branch is unresolved (closes #1899) (#1901)
 
 ## 0.1.1637 (2026-04-30)

package/dashboard.js

@@ -5838,9 +5838,9 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     // /api/prd/regenerate removed — use /api/plans/approve which does diff-aware update
 
     // Agents
-    { method: 'POST', path: '/api/pull-requests/link', desc: 'Manually link an external PR for tracking', params: 'url, title?, project?, autoObserve?, context?', handler: async (req, res) => {
+    { method: 'POST', path: '/api/pull-requests/link', desc: 'Manually link an external PR for tracking', params: 'url, title?, project?, autoObserve?, context?, workItemId?', handler: async (req, res) => {
       const body = await readBody(req);
-      const { url, title, project: projectName, autoObserve, context } = body;
+      const { url, title, project: projectName, autoObserve, context, workItemId } = body;
       if (!url) return jsonReply(res, 400, { error: 'url required' });
 
       // Determine project
@@ -5855,6 +5855,14 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       const prId = shared.getCanonicalPrId(targetProject, prNum, url);
       const contextText = typeof context === 'string' ? context : (context == null ? '' : JSON.stringify(context));
 
+      // Resolve a work-item association from either the top-level workItemId
+      // field (preferred) or context.workItemId (legacy CC payload shape).
+      // Without this, manually-linked PRs end up with prdItems=[] and the
+      // Work Items page renders no PR even though _context records the ID.
+      const linkedItemId = (typeof workItemId === 'string' && workItemId.trim())
+        || (context && typeof context === 'object' && typeof context.workItemId === 'string' && context.workItemId.trim())
+        || '';
+
       // Atomic check-and-insert to prevent duplicates and races with polling loops
       let duplicate = false;
       mutateJsonFileLocked(prPath, (prs) => {
@@ -5871,7 +5879,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           status: 'active',
           created: new Date().toISOString(),
           url,
-          prdItems: [],
+          prdItems: linkedItemId ? [linkedItemId] : [],
           _manual: true,
           _contextOnly: !autoObserve,
           _autoObserve: !!autoObserve,
@@ -5880,6 +5888,16 @@ What would you like to discuss or change? When you're happy, say "approve" and I
         return prs;
       }, { defaultValue: [] });
       if (duplicate) return jsonReply(res, 400, { error: 'PR already tracked' });
+      // Persist the work-item ↔ PR association in pr-links.json so
+      // queries.getWorkItems() can render item._pr / item._prUrl. addPrLink
+      // is idempotent and handles the central / project-scoped split.
+      if (linkedItemId) {
+        try {
+          shared.addPrLink(prId, linkedItemId, { project: targetProject, prNumber: parseInt(prNum, 10) || null, url });
+        } catch (e) {
+          shared.log('warn', `PR link addPrLink failed for ${prId} → ${linkedItemId}: ${e.message}`);
+        }
+      }
       invalidateStatusCache();
       jsonReply(res, 200, { ok: true, id: prId });
 

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-04-30T16:34:18.069Z"
+  "cachedAt": "2026-04-30T16:35:01.979Z"
 }

package/engine/lifecycle.js

@@ -744,13 +744,33 @@ function syncPrsFromOutput(output, agentId, meta, config) {
     }
   } catch {}
 
+  // prId → URL captured from inbox notes. Populated alongside prMatches so
+  // extractPrUrl below has a fallback when the agent's stdout doesn't contain
+  // the URL (the W-moljyu60wuzr / #1902 case — gh pr create ran in a sibling
+  // dispatch and only the inbox note carries the link).
+  const inboxUrls = new Map();
   const today = dateStamp();
   const inboxFiles = getInboxFiles().filter(f => f.includes(agentId) && f.includes(today));
   for (const f of inboxFiles) {
     const content = safeRead(path.join(INBOX_DIR, f));
     if (!content) continue;
-    const prHeaderPattern = /\*\*PR[:\*]*\*?\s*[#-]*\s*(?:(?:visualstudio\.com|dev\.azure\.com)[^\s"]*?pullrequest\/(\d+)|github\.com\/[^\s"]*?\/pull\/(\d+))/gi;
-    while ((match = prHeaderPattern.exec(content)) !== null) prMatches.add(match[1] || match[2]);
+    // Match a PR declaration line in the agent's findings note: optional bold,
+    // optional "Pull Request" spelling, line-anchored so "see PR https://..."
+    // mid-paragraph mentions don't trigger a false-positive. The protocol
+    // and host prefix is optional so "PR: https://github.com/..." ,
+    // "**PR:** github.com/...", etc. all match.
+    const prHeaderPattern = /(?:^|\n)\s*\*{0,2}(?:PR|Pull\s+Request)[:\*]*\*?\s*[#-]*\s*(?:https?:\/\/)?[^\s"]*?(?:(?:visualstudio\.com|dev\.azure\.com)[^\s"]*?pullrequest\/(\d+)|github\.com\/[^\s"]*?\/pull\/(\d+))/gi;
+    while ((match = prHeaderPattern.exec(content)) !== null) {
+      const prId = match[1] || match[2];
+      prMatches.add(prId);
+      // Pull the URL substring out of the matched chunk so we can hand it to
+      // extractPrUrl as a fallback. Prefer the first inbox URL we see for a
+      // given prId — later notes don't override the canonical record.
+      if (!inboxUrls.has(prId)) {
+        const urlMatch = match[0].match(/https?:\/\/[^\s"\\)]+/);
+        if (urlMatch) inboxUrls.set(prId, urlMatch[0].replace(/[.,;:]+$/, ''));
+      }
+    }
   }
 
   if (prMatches.size === 0) return 0;
@@ -773,7 +793,10 @@ function syncPrsFromOutput(output, agentId, meta, config) {
     return defaultProject;
   }
 
-  // Extract PR URL directly from agent output — no manual construction
+  // Extract PR URL directly from agent output — no manual construction.
+  // Falls back to the URL captured from the inbox note when the agent stdout
+  // doesn't contain the link (gh pr create may have run in a sibling dispatch
+  // whose stdout was rotated; the inbox note is the durable artifact).
   function extractPrUrl(prId) {
     // Stop at backslash in addition to whitespace/quotes — raw JSONL encodes newlines as \n (literal
     // backslash-n), so without this the regex would capture e.g. "pull/1804\n/usr/bin/bash".
@@ -781,7 +804,7 @@ function syncPrsFromOutput(output, agentId, meta, config) {
     if (ghMatch) return ghMatch[0].replace(/[.,;:]+$/, '');
     const adoMatch = output.match(new RegExp(`https?://(?:dev\\.azure\\.com|[^/]+\\.visualstudio\\.com)[^\\s"'\\)\\]\\\\]*?pullrequest/${prId}(?:[^\\s"'\\)\\]\\\\]*)`, 'i'));
     if (adoMatch) return adoMatch[0].replace(/[.,;:]+$/, '');
-    return '';
+    return inboxUrls.get(prId) || '';
   }
 
   const agentName = config.agents?.[agentId]?.name || agentId;

package/package.json

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