@yemi33/minions 0.1.2325 → 0.1.2327

package/dashboard.js

@@ -9105,6 +9105,29 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     }
   }
 
+  // W-mr5m9inn002d27e1 — safeWrite/safeUnlink can both throw or silently no-op
+  // (disk full, permission error, locked file on Windows). Previously the
+  // archive step's try/catch only logged to console.error while the handler
+  // still returned ok:true, so a failed archive left the note in
+  // notes/inbox/ (risking double-processing) with no operator-visible
+  // signal. Centralize the archive attempt so both call sites report
+  // {archived, archiveError} instead of swallowing the failure.
+  function _archiveInboxNote(inboxPath, archiveDestPath) {
+    try {
+      const _c = safeRead(inboxPath);
+      safeWrite(archiveDestPath, _c);
+      safeUnlink(inboxPath);
+      if (fs.existsSync(inboxPath)) {
+        // safeUnlink swallows its own errors, so a failed removal doesn't
+        // throw — detect it by checking the source file is actually gone.
+        return { archived: false, archiveError: 'inbox note could not be removed after archiving (unlink failed)' };
+      }
+      return { archived: true, archiveError: null };
+    } catch (e) {
+      return { archived: false, archiveError: e.message };
+    }
+  }
+
   async function handleInboxPersist(req, res) {
     try {
       const body = await readBody(req);
@@ -9150,9 +9173,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       // Move to archive
       const archiveDir = path.join(MINIONS_DIR, 'notes', 'archive');
       if (!fs.existsSync(archiveDir)) fs.mkdirSync(archiveDir, { recursive: true });
-      try { const _c = safeRead(inboxPath); safeWrite(path.join(archiveDir, `persisted-${name}`), _c); safeUnlink(inboxPath); } catch (e) { console.error('inbox archive:', e.message); }
+      const { archived, archiveError } = _archiveInboxNote(inboxPath, path.join(archiveDir, `persisted-${name}`));
+      if (!archived) console.error('inbox archive:', archiveError);
 
-      return jsonReply(res, 200, { ok: true, title });
+      return jsonReply(res, 200, archived
+        ? { ok: true, title, archived: true }
+        : { ok: true, title, archived: false, warning: `Persisted to notes.md, but the inbox note could not be archived: ${archiveError}. It remains in notes/inbox and may be reprocessed.` });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 
@@ -9215,9 +9241,12 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       // Move inbox item to archive
       const archiveDir = path.join(MINIONS_DIR, 'notes', 'archive');
       if (!fs.existsSync(archiveDir)) fs.mkdirSync(archiveDir, { recursive: true });
-      try { const _c = safeRead(inboxPath); safeWrite(path.join(archiveDir, `kb-${category}-${name}`), _c); safeUnlink(inboxPath); } catch (e) { console.error('inbox archive:', e.message); }
+      const { archived, archiveError } = _archiveInboxNote(inboxPath, path.join(archiveDir, `kb-${category}-${name}`));
+      if (!archived) console.error('inbox archive:', archiveError);
 
-      return jsonReply(res, 200, { ok: true, category, file: name });
+      return jsonReply(res, 200, archived
+        ? { ok: true, category, file: name, archived: true }
+        : { ok: true, category, file: name, archived: false, warning: `Promoted to knowledge/${category}/${name}, but the inbox note could not be archived: ${archiveError}. It remains in notes/inbox and may be reprocessed.` });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 

package/engine/pull-requests-store.js

@@ -299,11 +299,24 @@ function _mirrorJsonFromSql(scope, filePath) {
   }
 }
 
+// Drop every SQL row for `scope` and forget its mirror-hash. Called by
+// removeProject after the JSON file is archived so safeJson shim reads no
+// longer surface the removed project's records.
+//
+// Per CLAUDE.md's mutator contract, every mutator wrapper must call
+// emitStateEvent() so the dashboard's MAX(events.id) cache-version
+// advances — otherwise the dashboard keeps serving stale cached
+// pull-requests data for other still-active projects. Emit only when rows
+// were actually deleted, matching the wrote-gated emit convention used by
+// applyPullRequestsMutation's callers (see shared.js mutatePullRequests).
 function dropScope(scope) {
   try {
     const { getDb } = require('./db');
-    getDb().prepare('DELETE FROM pull_requests WHERE scope = ?').run(scope);
+    const result = getDb().prepare('DELETE FROM pull_requests WHERE scope = ?').run(scope);
     _lastMirrorHashByScope.delete(scope);
+    if (result && result.changes > 0) {
+      try { require('./db-events').emitStateEvent('pull_requests'); } catch { /* optional */ }
+    }
   } catch { /* db unavailable */ }
 }
 

package/engine/work-items-store.js

@@ -379,11 +379,22 @@ function _mirrorJsonFromSql(scope, filePath) {
 // Drop every SQL row for `scope` and forget its mirror-hash. Called by
 // removeProject after the JSON file is archived so safeJson shim reads no
 // longer surface the removed project's records.
+//
+// Per CLAUDE.md's mutator contract, every mutator wrapper must call
+// emitStateEvent() so the dashboard's MAX(events.id) cache-version
+// advances — otherwise the dashboard keeps serving stale cached
+// work-items data for other still-active projects. Emit only when rows
+// were actually deleted, matching the wrote-gated emit convention used by
+// applyWorkItemsMutation's callers (see shared.js mutateWorkItems).
 function dropScope(scope) {
   try {
     const { getDb } = require('./db');
-    getDb().prepare('DELETE FROM work_items WHERE scope = ?').run(scope);
+    const result = getDb().prepare('DELETE FROM work_items WHERE scope = ?').run(scope);
     _lastMirrorHashByScope.delete(scope);
+    if (result && result.changes > 0) {
+      try { require('./db-events').emitStateEvent('work_items'); } catch { /* optional */ }
+      try { require('./queries').invalidateWorkItemsCache(); } catch { /* queries not loaded */ }
+    }
   } catch { /* db unavailable */ }
 }
 

package/package.json

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