@yemi33/minions 0.1.361 → 0.1.362

package/CHANGELOG.md

@@ -1,8 +1,9 @@
 # Changelog
 
-## 0.1.361 (2026-04-06)
+## 0.1.362 (2026-04-06)
 
 ### Features
+- Null-safe safeJson wrappers + dashboard crash guards (#203)
 -  pipeline plan stage uses LLM to generate structured plan from meeting
 -  show 'Converting to PRD' status instead of 'In Progress' during plan conversion
 

package/dashboard/js/settings.js

@@ -25,7 +25,7 @@ async function openSettings() {
     '<h3 style="font-size:13px;color:var(--blue);margin-bottom:8px">Engine</h3>' +
     '<div style="display:grid;grid-template-columns:1fr 1fr;gap:8px;margin-bottom:16px">' +
       settingsField('Tick Interval', 'set-tickInterval', e.tickInterval || 60000, 'ms', 'How often the engine runs discovery + dispatch') +
-      settingsField('Max Concurrent Agents', 'set-maxConcurrent', e.maxConcurrent || 3, '', 'Max agents working simultaneously') +
+      settingsField('Max Concurrent Agents', 'set-maxConcurrent', e.maxConcurrent || 5, '', 'Max agents working simultaneously') +
       settingsField('Consolidation Threshold', 'set-inboxConsolidateThreshold', e.inboxConsolidateThreshold || 5, 'notes', 'Inbox notes before auto-consolidation') +
       settingsField('Agent Timeout', 'set-agentTimeout', e.agentTimeout || 18000000, 'ms', 'Kill agent after this duration') +
       settingsField('Max Turns', 'set-maxTurns', e.maxTurns || 100, '', 'Claude CLI --max-turns per agent') +

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, safeUnlink, mutateJsonFileLocked, getProjects: _getProjects, DONE_STATUSES } = shared;
+const { safeRead, safeReadDir, safeWrite, safeJson, safeJsonObj, safeJsonArr, safeUnlink, mutateJsonFileLocked, getProjects: _getProjects, DONE_STATUSES } = shared;
 const { getAgents, getAgentDetail, getPrdInfo, getWorkItems, getDispatchQueue,
   getSkills, getInbox, getNotesWithMeta, getPullRequests,
   getEngineLog, getMetrics, getKnowledgeBaseEntries, timeSince,
@@ -1206,7 +1206,7 @@ const server = http.createServer(async (req, res) => {
       const planPath = resolvePlanPath(body.source);
       if (!fs.existsSync(planPath)) return jsonReply(res, 404, { error: 'plan file not found' });
       // Pre-check: verify item exists before taking the lock
-      const preCheck = safeJson(planPath);
+      const preCheck = safeJsonObj(planPath);
       const preItem = (preCheck.missing_features || []).find(f => f.id === body.itemId);
       if (!preItem) return jsonReply(res, 404, { error: 'item not found in plan' });
 
@@ -1258,7 +1258,7 @@ const server = http.createServer(async (req, res) => {
       if (!body.source || !body.itemId) return jsonReply(res, 400, { error: 'source and itemId required' });
       const planPath = resolvePlanPath(body.source);
       if (!fs.existsSync(planPath)) return jsonReply(res, 404, { error: 'plan file not found' });
-      const plan = safeJson(planPath);
+      const plan = safeJsonObj(planPath);
       const idx = (plan.missing_features || []).findIndex(f => f.id === body.itemId);
       if (idx < 0) return jsonReply(res, 404, { error: 'item not found in plan' });
 
@@ -2048,7 +2048,7 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       if (!body.source) return jsonReply(res, 400, { error: 'source required' });
       const planPath = resolvePlanPath(body.source);
       if (!fs.existsSync(planPath)) return jsonReply(res, 404, { error: 'plan file not found' });
-      const plan = safeJson(planPath);
+      const plan = safeJsonObj(planPath);
       const planItems = plan.missing_features || [];
 
       let reset = 0, kept = 0, newCount = 0;
@@ -2127,7 +2127,7 @@ If nothing to do: { "duplicates": [], "reclassify": [], "remove": [] }`;
       }
       for (const wiPath of wiPaths) {
         try {
-          const items = safeJson(wiPath);
+          const items = safeJsonArr(wiPath);
           const filtered = items.filter(w => w.sourcePlan !== body.file);
           if (filtered.length < items.length) {
             cleaned += items.length - filtered.length;
@@ -2834,7 +2834,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
       if (!fs.existsSync(target)) return jsonReply(res, 400, { error: 'Directory not found: ' + target });
 
       const configPath = path.join(MINIONS_DIR, 'config.json');
-      const config = safeJson(configPath);
+      const config = safeJsonObj(configPath);
       if (!config.projects) config.projects = [];
 
       // Check if already linked

package/engine/shared.js

@@ -73,6 +73,12 @@ function safeJson(p) {
   }
 }
 
+/** Null-safe safeJson wrapper — returns {} when file is missing/corrupt. */
+function safeJsonObj(p) { return safeJson(p) || {}; }
+
+/** Null-safe safeJson wrapper — returns [] when file is missing/corrupt. */
+function safeJsonArr(p) { return safeJson(p) || []; }
+
 /**
  * Monotonic counter for generating unique temp file names within this process.
  * Assumes single-thread execution (no worker_threads). If worker_threads are
@@ -422,7 +428,7 @@ const DEFAULT_AGENTS = {
 
 const DEFAULT_CLAUDE = {
   binary: 'claude',
-  outputFormat: 'json',
+  outputFormat: 'stream-json',
   allowedTools: 'Edit,Write,Read,Bash,Glob,Grep,Agent,WebFetch,WebSearch',
 };
 
@@ -623,7 +629,7 @@ module.exports = {
   log,
   safeRead,
   safeReadDir,
-  safeJson,
+  safeJson, safeJsonObj, safeJsonArr,
   safeWrite,
   safeUnlink,
   withFileLock,

package/engine.js

@@ -1416,7 +1416,12 @@ function discoverFromWorkItems(config, project) {
     // This protects against persisted state drift from old runtime versions.
     try {
       mutateDispatch((dp) => {
-        dp.completed = Array.isArray(dp.completed) ? dp.completed.filter(d => d.meta?.dispatchKey !== key) : [];
+        const prev = Array.isArray(dp.completed) ? dp.completed : [];
+        const next = [];
+        for (let i = 0; i < prev.length; i++) {
+          if (prev[i].meta?.dispatchKey !== key) next.push(prev[i]);
+        }
+        dp.completed = next;
         return dp;
       });
       dispatchCooldowns.delete(key);

package/package.json

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