@yemi33/minions 0.1.2194 → 0.1.2196

package/engine/queries.js

@@ -496,6 +496,72 @@ function getInboxFiles() {
   try { return fs.readdirSync(INBOX_DIR).filter(f => f.endsWith('.md')); } catch { return []; }
 }
 
+// P-34fa5d79 — Note ↔ Work Item ↔ KB Entry linkage.
+//
+// Parse the YAML frontmatter from a note body and return the parsed fields.
+// Same minimal contract as engine/kb-sweep.js#_parseFrontmatter — a single
+// shared regex would have been nicer but importing kb-sweep from queries
+// would form a circular dep with lifecycle (queries → kb-sweep → … →
+// queries), so we keep a local copy of the four-line parser.
+function _parseNoteFrontmatter(content) {
+  const m = String(content || '').match(/^---\n([\s\S]*?)\n---\n?/);
+  if (!m) return null;
+  const fm = {};
+  for (const line of m[1].split('\n')) {
+    const lm = line.match(/^([\w-]+):\s*(.*)$/);
+    if (lm) fm[lm[1]] = lm[2].trim();
+  }
+  return fm;
+}
+
+// Extract the WI id that a note's frontmatter cites, or null. Accepts the
+// three field-name conventions used across the repo today:
+//   - `work_item:` (dallas fix-summary template, the common case)
+//   - `wi:` (terser variant, used in a handful of newer notes)
+//   - `sourceItem:` (engine-emitted notes such as pr-auto-link-unverified)
+// `id:` itself is the note's own NOTE-… id and is intentionally NOT matched —
+// it would never carry a WI value in any existing convention.
+function _wiIdFromNoteFrontmatter(fm) {
+  if (!fm) return null;
+  return fm.work_item || fm.wi || fm.sourceItem || null;
+}
+
+// Build a wiId → [filename, ...] map by scanning the inbox + archive dirs.
+// Archive entries are prefixed with `archive:` so the dashboard renderer can
+// tell them apart (matches the convention already used by _artifacts.notes).
+// Pure-function helper exposed for testing.
+function _buildNotesByWiMap() {
+  const out = Object.create(null);
+  const addNote = (wiId, token) => {
+    if (!wiId || !token) return;
+    if (!out[wiId]) out[wiId] = [];
+    if (!out[wiId].includes(token)) out[wiId].push(token);
+  };
+  for (const f of safeReadDir(INBOX_DIR)) {
+    if (!f.endsWith('.md')) continue;
+    const fm = _parseNoteFrontmatter(safeRead(path.join(INBOX_DIR, f)));
+    const wiId = _wiIdFromNoteFrontmatter(fm);
+    if (wiId) addNote(wiId, f);
+  }
+  for (const f of safeReadDir(ARCHIVE_DIR)) {
+    if (!f.endsWith('.md')) continue;
+    const fm = _parseNoteFrontmatter(safeRead(path.join(ARCHIVE_DIR, f)));
+    const wiId = _wiIdFromNoteFrontmatter(fm);
+    if (wiId) addNote(wiId, 'archive:' + f);
+  }
+  return out;
+}
+
+// Public lookup — return the array of note tokens (filenames or
+// `archive:<filename>`) whose frontmatter cites this WI id. Returns [] when
+// nothing matches. Scans inbox + archive each call; for the bulk WI list
+// payload getWorkItems() pre-builds the map once per cache cycle.
+function notesForWorkItem(wiId) {
+  if (!wiId) return [];
+  const map = _buildNotesByWiMap();
+  return map[wiId] || [];
+}
+
 function getInbox() {
   return safeReadDir(INBOX_DIR)
     .filter(f => f.endsWith('.md'))
@@ -1745,6 +1811,9 @@ function getWorkItems(config) {
   // Use snapshot — sync access; cold start before any async warm returns [].
   // Best-effort enrichment for work item _artifacts.notes, not correctness-critical.
   const _kbEntries = getKnowledgeBaseEntriesSnapshot();
+  // P-34fa5d79 — pre-build the wiId → notes map so each item's _notes lookup
+  // is O(1) instead of re-scanning inbox+archive per item.
+  const _notesByWi = _buildNotesByWiMap();
   for (const item of allItems) {
     const arts = shared.isPlainObject(item._artifacts) ? { ...item._artifacts } : {};
     const agentId = item.dispatched_to || item.agent;
@@ -1779,6 +1848,13 @@ function getWorkItems(config) {
     else if (item.planFile) arts.plan = item.planFile;
     if (item._pr) arts.pr = item._pr;
     if (Object.keys(arts).length > 0) item._artifacts = arts;
+    // P-34fa5d79 — Note ↔ WI ↔ KB linkage. Slim string[] field of note
+    // filenames whose YAML frontmatter cites this WI. Archive notes are
+    // prefixed with `archive:` so the dashboard chip renderer can pick the
+    // right open path. Field is left undefined (not []) when empty so the
+    // /api/work-items JSON stays compact for the common no-notes case.
+    const mentions = _notesByWi[item.id];
+    if (mentions && mentions.length > 0) item._notes = mentions;
   }
 
   const statusOrder = {
@@ -2897,6 +2973,11 @@ module.exports = {
 
   // Inbox
   getInboxFiles, getInbox,
+  // P-34fa5d79 — Note ↔ Work Item ↔ KB Entry linkage.
+  notesForWorkItem,
+  _parseNoteFrontmatter, // exported for testing
+  _wiIdFromNoteFrontmatter, // exported for testing
+  _buildNotesByWiMap, // exported for testing
 
   // Agents
   getAgentStatus, getAgentCharter, getAgents, getAgentDetail,

package/engine/scheduler.js

@@ -328,6 +328,12 @@ function createScheduledWorkItem(sched) {
     ...(sched.agentLock === true || sched.hardAgent === true ? { agentLock: true } : {}),
     project: sched.project || null,
     _scheduleId: sched.id,
+    // P-c549d07e — back-link to the schedule that spawned this WI. The WI
+    // detail modal renders this as a clickable chip via renderArtifactLink
+    // so the operator can navigate back to the dispatcher. Kept alongside
+    // _scheduleId (used by queries / lifecycle / consolidation) so we don't
+    // disturb the existing readers — meta.spawnedBy is purely additive.
+    meta: { spawnedBy: 'schedule:' + sched.id },
   };
   // Walk every string-valued field on the work item so vars embedded in
   // nested fields (e.g. _harness.rubric added by callers, references[].url,
@@ -339,7 +345,19 @@ function createScheduledWorkItem(sched) {
 
 function writeScheduleRunEntry(runs, scheduleId, workItemId, extra) {
   const existing = typeof runs[scheduleId] === 'object' && runs[scheduleId] ? runs[scheduleId] : {};
-  runs[scheduleId] = { ...existing, lastRun: ts(), lastWorkItemId: workItemId, ...(extra || {}) };
+  // P-c549d07e — maintain a recentWorkItemIds ring (newest first, hard cap 5,
+  // deduplicated). Surfaces "Recent dispatches" chips in the schedule modal
+  // without forcing the dashboard to scan every WI for _scheduleId matches.
+  // Empty / falsy workItemId leaves the ring untouched (defensive — callers
+  // always pass a real id today, but the engine should never pollute the
+  // history with blanks if a future caller slips up).
+  let recent = Array.isArray(existing.recentWorkItemIds) ? existing.recentWorkItemIds.slice() : [];
+  if (workItemId) {
+    recent = recent.filter((id) => id !== workItemId);
+    recent.unshift(workItemId);
+    if (recent.length > 5) recent = recent.slice(0, 5);
+  }
+  runs[scheduleId] = { ...existing, lastRun: ts(), lastWorkItemId: workItemId, recentWorkItemIds: recent, ...(extra || {}) };
   return runs[scheduleId];
 }
 

package/package.json

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