@yemi33/minions 0.1.2151 → 0.1.2152

package/engine/lifecycle.js

@@ -546,6 +546,64 @@ function resolveWorkItemPath(meta) {
   return null;
 }
 
+function getWorkItemPaths(config) {
+  const paths = [path.join(MINIONS_DIR, 'work-items.json')];
+  for (const p of shared.getProjects(config || {})) {
+    if (p && p.name) paths.push(shared.projectWorkItemsPath(p));
+  }
+  return paths;
+}
+
+function findWorkItemRecord(itemId, config) {
+  if (!itemId) return null;
+  for (const wiPath of getWorkItemPaths(config)) {
+    try {
+      const items = readOptionalJsonStrict(wiPath, 'work-items', Array.isArray) || [];
+      const item = items.find(i => i && i.id === itemId);
+      if (item) return { item, path: wiPath };
+    } catch { /* best-effort lineage lookup */ }
+  }
+  return null;
+}
+
+function resolveWorkItemScheduleContext(item, config) {
+  if (!item || typeof item !== 'object') return {};
+  const seen = new Set();
+  let current = item;
+  let scheduleId = current._scheduleId || current._sourceScheduleId || '';
+  let scheduleWorkItemId = current._scheduleId ? current.id : (current._scheduleWorkItemId || '');
+  let projectName = current.project || '';
+  let parentItemId = current.parent_id || current.parentId || '';
+
+  while ((!scheduleId || !projectName) && parentItemId && !seen.has(parentItemId)) {
+    seen.add(parentItemId);
+    const found = findWorkItemRecord(parentItemId, config);
+    if (!found?.item) break;
+    current = found.item;
+    if (!scheduleId && (current._scheduleId || current._sourceScheduleId)) {
+      scheduleId = current._scheduleId || current._sourceScheduleId;
+      scheduleWorkItemId = current._scheduleId ? current.id : (current._scheduleWorkItemId || current.id);
+    }
+    if (!projectName && current.project) projectName = current.project;
+    parentItemId = current.parent_id || current.parentId || '';
+  }
+
+  return {
+    scheduleId: scheduleId || '',
+    scheduleWorkItemId: scheduleWorkItemId || '',
+    projectName: projectName || '',
+  };
+}
+
+function applyScheduleContextToPrEntry(entry, item, config) {
+  const ctx = resolveWorkItemScheduleContext(item, config);
+  if (ctx.scheduleId) {
+    entry._sourceScheduleId = ctx.scheduleId;
+    if (ctx.scheduleWorkItemId) entry._scheduleWorkItemId = ctx.scheduleWorkItemId;
+  }
+  return entry;
+}
+
 /** Check if a work item is in a terminal completed state. */
 function isItemCompleted(item) {
   if (!item || typeof item !== 'object') return false;
@@ -920,6 +978,7 @@ function syncPrsFromOutput(output, agentId, meta, config, opts = {}) {
       sourcePlan: meta?.item?.sourcePlan || '',
       itemType: meta?.item?.itemType || ''
     };
+    applyScheduleContextToPrEntry(entry, meta?.item, config);
     // Issue #1772: one-off dispatches (e.g. human-initiated "review this PR" via CC)
     // must not enroll the discovered PR into the auto eval loop. Tag _contextOnly so
     // discoverFromPrs skips it for review/fix dispatch (still polled for status/comments).
@@ -1318,6 +1377,11 @@ function resolvePrFallbackProject(meta, config) {
     const match = shared.resolveProjectSource(meta.item.project, projects, { allowCentral: false }).project;
     if (match) return match;
   }
+  const scheduleContext = resolveWorkItemScheduleContext(meta?.item, config);
+  if (scheduleContext.projectName) {
+    const match = shared.resolveProjectSource(scheduleContext.projectName, projects, { allowCentral: false }).project;
+    if (match) return match;
+  }
   return projects.length === 1 ? projects[0] : null;
 }
 
@@ -1488,6 +1552,7 @@ function _attachFoundPrToWi(found, meta, agentId, resultSummary, config) {
     sourcePlan: meta.item?.sourcePlan || '',
     itemType: meta.item?.itemType || '',
   };
+  applyScheduleContextToPrEntry(entry, meta?.item, config);
   shared.upsertPullRequestRecord(shared.projectPrPath(found.project), entry, {
     project: found.project,
     itemId: meta.item.id,
@@ -4184,6 +4249,9 @@ function processCompletionFollowups(completion, agentId, dispatchItem, config) {
   }
   if (raw.length === 0) return [];
   const wiId = dispatchItem?.meta?.item?.id || 'N/A';
+  const parentItem = dispatchItem?.meta?.item || null;
+  const parentProject = dispatchItem?.meta?.project?.name || parentItem?.project || '';
+  const parentSchedule = resolveWorkItemScheduleContext(parentItem, config);
   let existingIds = null;
   function loadExistingIds() {
     if (existingIds !== null) return existingIds;
@@ -4217,6 +4285,27 @@ function processCompletionFollowups(completion, agentId, dispatchItem, config) {
     if (allIds.size > 0 && !allIds.has(followupWiId)) {
       log('warn', `Followup audit (${agentId || 'unknown'} wi=${wiId}): claimed followup ${followupWiId} not found in any work-items.json — dispatch may have been deduped, reverted, or never landed`);
     }
+    if (parentItem && (parentSchedule.scheduleId || parentProject)) {
+      let stamped = false;
+      for (const wiPath of getWorkItemPaths(config)) {
+        mutateWorkItems(wiPath, items => {
+          const target = Array.isArray(items) ? items.find(i => i && i.id === followupWiId) : null;
+          if (!target) return items;
+          if (!target.parent_id && target.id !== wiId) target.parent_id = wiId;
+          if (!target.project && parentProject) target.project = parentProject;
+          if (parentSchedule.scheduleId) {
+            if (!target._sourceScheduleId) target._sourceScheduleId = parentSchedule.scheduleId;
+            if (!target._scheduleWorkItemId) target._scheduleWorkItemId = parentSchedule.scheduleWorkItemId || wiId;
+          }
+          stamped = true;
+          return items;
+        });
+        if (stamped) break;
+      }
+      if (stamped && parentSchedule.scheduleId) {
+        log('info', `Followup audit (${agentId || 'unknown'} wi=${wiId}): linked ${followupWiId} to schedule ${parentSchedule.scheduleId}`);
+      }
+    }
     accepted.push({
       wi_id: followupWiId,
       title,

package/package.json

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