@yemi33/minions 0.1.374 → 0.1.376

package/CHANGELOG.md

@@ -1,8 +1,12 @@
 # Changelog
 
-## 0.1.374 (2026-04-06)
+## 0.1.376 (2026-04-06)
+
+### Features
+- dashboard write locking for work-items.json (#227)
 
 ### Fixes
+-  show red notification dot on Notes & KB when note is created
 -  medium bugs — KB declaration, meeting phantom archive, lifecycle race (#216)
 
 ## 0.1.373 (2026-04-06)

package/dashboard/js/command-center.js

@@ -377,6 +377,8 @@ async function ccExecuteAction(action) {
         await _ccFetch('/api/notes', { title: action.title, what: action.content || action.description, author: 'command-center' });
         status.innerHTML = '&#10003; Note saved: <strong>' + escHtml(action.title) + '</strong>';
         status.style.color = 'var(--green)';
+        var notePageLink = document.querySelector('.sidebar-link[data-page="inbox"]');
+        if (notePageLink && !notePageLink.querySelector('.notif-badge')) { var noteCurPage = document.querySelector('.sidebar-link.active')?.getAttribute('data-page'); if (noteCurPage !== 'inbox') showNotifBadge(notePageLink); }
         break;
       }
       case 'pin': {

package/dashboard/js/render-inbox.js

@@ -201,7 +201,14 @@ async function submitQuickNote() {
       method: 'POST', headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({ title: title || 'Quick note', what: content || title })
     });
-    if (res.ok) { refresh(); }
+    if (res.ok) {
+      refresh();
+      const currentPage = document.querySelector('.sidebar-link.active')?.getAttribute('data-page');
+      if (currentPage !== 'inbox') {
+        const link = document.querySelector('.sidebar-link[data-page="inbox"]');
+        if (link && !link.querySelector('.notif-badge')) showNotifBadge(link);
+      }
+    }
     else { const d = await res.json().catch(() => ({})); alert('Note failed: ' + (d.error || 'unknown')); openQuickNoteModal(); }
   } catch (e) { alert('Error saving note: ' + e.message); openQuickNoteModal(); }
 }

package/dashboard.js

@@ -14,7 +14,7 @@ const shared = require('./engine/shared');
 const queries = require('./engine/queries');
 const os = require('os');
 
-const { safeRead, safeReadDir, safeWrite, safeJson, safeJsonObj, safeJsonArr, safeUnlink, mutateJsonFileLocked, getProjects: _getProjects, DONE_STATUSES } = shared;
+const { safeRead, safeReadDir, safeWrite, safeJson, safeJsonObj, safeJsonArr, safeUnlink, mutateJsonFileLocked, getProjects: _getProjects, DONE_STATUSES, WI_STATUS } = shared;
 const { getAgents, getAgentDetail, getPrdInfo, getWorkItems, getDispatchQueue,
   getSkills, getInbox, getNotesWithMeta, getPullRequests,
   getEngineLog, getMetrics, getKnowledgeBaseEntries, timeSince,
@@ -972,7 +972,7 @@ const server = http.createServer(async (req, res) => {
       }, { defaultValue: [] });
       if (!found) return jsonReply(res, 404, { error: 'item not found' });
 
-      // Clean dispatch entries + kill running agent
+      // Clean dispatch entries + kill running agent (outside lock)
       const dispatchRemoved = cleanDispatchEntries(d =>
         d.meta?.item?.id === id ||
         d.meta?.dispatchKey?.endsWith(id)
@@ -1011,21 +1011,24 @@ const server = http.createServer(async (req, res) => {
       }
       if (!wiPath) return jsonReply(res, 404, { error: 'source not found' });
 
-      const items = JSON.parse(safeRead(wiPath) || '[]');
-      const idx = items.findIndex(i => i.id === id);
-      if (idx === -1) return jsonReply(res, 404, { error: 'item not found' });
-
-      const item = items.splice(idx, 1)[0];
-      item.archivedAt = new Date().toISOString();
+      let archivedItem = null;
+      mutateJsonFileLocked(wiPath, (items) => {
+        if (!Array.isArray(items)) items = [];
+        const idx = items.findIndex(i => i.id === id);
+        if (idx === -1) return items;
+        archivedItem = items.splice(idx, 1)[0];
+        archivedItem.archivedAt = new Date().toISOString();
+        return items;
+      });
+      if (!archivedItem) return jsonReply(res, 404, { error: 'item not found' });
 
-      // Append to archive file
+      // Append to archive file (outside lock)
       const archivePath = wiPath.replace('.json', '-archive.json');
-      let archive = [];
-      const existing = safeRead(archivePath);
-      if (existing) { try { archive = JSON.parse(existing); } catch {} }
-      archive.push(item);
-      safeWrite(archivePath, archive);
-      safeWrite(wiPath, items);
+      mutateJsonFileLocked(archivePath, (archive) => {
+        if (!Array.isArray(archive)) archive = [];
+        archive.push(archivedItem);
+        return archive;
+      }, { defaultValue: [] });
 
       // Clean dispatch entries for archived item
       const sourcePrefix = (!source || source === 'central') ? 'central-work-' : `work-${source}-`;
@@ -1069,14 +1072,11 @@ const server = http.createServer(async (req, res) => {
         // Write to central queue — agent decides which project
         wiPath = path.join(MINIONS_DIR, 'work-items.json');
       }
-      let items = [];
-      const existing = safeRead(wiPath);
-      if (existing) { try { items = JSON.parse(existing); } catch {} }
       const id = 'W-' + shared.uid();
       const item = {
         id, title: body.title, type: body.type || 'implement',
         priority: body.priority || 'medium', description: body.description || '',
-        status: 'pending', created: new Date().toISOString(), createdBy: 'dashboard',
+        status: WI_STATUS.PENDING, created: new Date().toISOString(), createdBy: 'dashboard',
       };
       if (body.scope) item.scope = body.scope;
       if (body.agent) item.agent = body.agent;
@@ -1084,8 +1084,11 @@ const server = http.createServer(async (req, res) => {
       if (body.references) item.references = body.references;
       if (body.acceptanceCriteria) item.acceptanceCriteria = body.acceptanceCriteria;
       if (body.skipPr) item.skipPr = true;
-      items.push(item);
-      safeWrite(wiPath, items);
+      mutateJsonFileLocked(wiPath, (items) => {
+        if (!Array.isArray(items)) items = [];
+        items.push(item);
+        return items;
+      });
       return jsonReply(res, 200, { ok: true, id });
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
@@ -1107,29 +1110,32 @@ const server = http.createServer(async (req, res) => {
       }
       if (!wiPath) return jsonReply(res, 404, { error: 'source not found' });
 
-      const items = JSON.parse(safeRead(wiPath) || '[]');
-      const item = items.find(i => i.id === id);
-      if (!item) return jsonReply(res, 404, { error: 'item not found' });
-
-      if (item.status === 'dispatched') {
-        return jsonReply(res, 400, { error: 'Cannot edit dispatched items' });
-      }
-
-      if (title !== undefined) item.title = title;
-      if (description !== undefined) item.description = description;
-      if (type !== undefined) item.type = type;
-      if (priority !== undefined) item.priority = priority;
-      if (agent !== undefined) {
-        item.agent = agent || null;
-        // Clear stale pending dispatch entries so the engine re-queues with the new agent
-        cleanDispatchEntries(d => d.meta?.item?.id === id);
-      }
-      if (body.references !== undefined) item.references = body.references;
-      if (body.acceptanceCriteria !== undefined) item.acceptanceCriteria = body.acceptanceCriteria;
-      item.updatedAt = new Date().toISOString();
-
-      safeWrite(wiPath, items);
-      return jsonReply(res, 200, { ok: true, item });
+      let result = null;
+      let agentChanged = false;
+      mutateJsonFileLocked(wiPath, (items) => {
+        if (!Array.isArray(items)) items = [];
+        const item = items.find(i => i.id === id);
+        if (!item) { result = { code: 404, body: { error: 'item not found' } }; return items; }
+        if (item.status === WI_STATUS.DISPATCHED) { result = { code: 400, body: { error: 'Cannot edit dispatched items' } }; return items; }
+
+        if (title !== undefined) item.title = title;
+        if (description !== undefined) item.description = description;
+        if (type !== undefined) item.type = type;
+        if (priority !== undefined) item.priority = priority;
+        if (agent !== undefined) {
+          item.agent = agent || null;
+          agentChanged = true;
+        }
+        if (body.references !== undefined) item.references = body.references;
+        if (body.acceptanceCriteria !== undefined) item.acceptanceCriteria = body.acceptanceCriteria;
+        item.updatedAt = new Date().toISOString();
+        result = { code: 200, body: { ok: true, item } };
+        return items;
+      });
+      if (!result) return jsonReply(res, 500, { error: 'unexpected state' });
+      // Clear stale pending dispatch entries outside lock
+      if (agentChanged) cleanDispatchEntries(d => d.meta?.item?.id === id);
+      return jsonReply(res, result.code, result.body);
     } catch (e) { return jsonReply(res, 400, { error: e.message }); }
   }
 

package/package.json

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