@yemi33/minions 0.1.287 → 0.1.288

package/.gitattributes

@@ -0,0 +1,2 @@
+* text=auto eol=lf
+*.ps1 text eol=crlf

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.287 (2026-04-03)
+## 0.1.288 (2026-04-03)
 
 ### Features
 -  all doc-chats use Sonnet with full tools (agent change)
 
 ### Fixes
+-  cross-platform compatibility — signal handling, paths, home dir
 -  engine sidebar badge only triggers on new dispatch errors
 
 ## 0.1.285 (2026-04-03)

package/bin/minions.js

@@ -666,7 +666,7 @@ if (!cmd || cmd === 'help' || cmd === '--help' || cmd === '-h') {
 
   // 2. Remove minions-authored skills from ~/.claude/skills/
   try {
-    const claudeSkills = path.join(process.env.HOME || process.env.USERPROFILE || '', '.claude', 'skills');
+    const claudeSkills = path.join(os.homedir(), '.claude', 'skills');
     if (fs.existsSync(claudeSkills)) {
       for (const dir of fs.readdirSync(claudeSkills)) {
         const skillFile = path.join(claudeSkills, dir, 'SKILL.md');

package/dashboard.js

@@ -163,7 +163,7 @@ function getEngineState() { return queries.getControl(); }
 
 function getMcpServers() {
   try {
-    const home = process.env.USERPROFILE || process.env.HOME || '';
+    const home = os.homedir();
     const claudeJsonPath = path.join(home, '.claude.json');
     const data = JSON.parse(safeRead(claudeJsonPath) || '{}');
     const servers = data.mcpServers || {};
@@ -2736,7 +2736,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
     }
     if (!content) {
       // Fallback: search Claude Code skills, then project skills
-      const home = process.env.HOME || process.env.USERPROFILE || '';
+      const home = os.homedir();
       const claudePath = path.join(home, '.claude', 'skills', file.replace('.md', '').replace('SKILL', ''), 'SKILL.md');
       content = safeRead(claudePath) || '';
       if (!content) {

package/engine/cleanup.js

@@ -230,7 +230,7 @@ function runCleanup(config, verbose = false) {
     const activeIds = new Set((dispatch.active || []).map(d => d.id));
     for (const [id, info] of activeProcesses.entries()) {
       if (!activeIds.has(id)) {
-        try { if (info.proc) info.proc.kill('SIGTERM'); } catch { /* process may be dead */ }
+        shared.killImmediate(info.proc);
         activeProcesses.delete(id);
         cleaned.zombies++;
       }

package/engine/consolidation.js

@@ -179,10 +179,8 @@ function consolidateWithLLM(items, existingNotes, files, config) {
 
   const timeout = setTimeout(() => {
     log('warn', 'LLM consolidation timed out after 3m — killing and falling back to regex');
-    try { proc.kill('SIGTERM'); } catch { /* process may be dead */ }
-    // Escalate to SIGKILL after 10s if process doesn't exit
+    shared.killGracefully(proc, 10000);
     setTimeout(() => {
-      try { proc.kill('SIGKILL'); } catch { /* process may be dead */ }
       if (_consolidationInFlight) {
         _consolidationInFlight = false;
         _processingFiles.clear();

package/engine/lifecycle.js

@@ -5,6 +5,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const os = require('os');
 const shared = require('./shared');
 const { safeRead, safeJson, safeWrite, safeReadDir, execSilent, projectPrPath, getPrLinks, addPrLink,
   mutateJsonFileLocked, log, ts, dateStamp } = shared;
@@ -200,11 +201,12 @@ function checkPlanCompletion(meta, config) {
 
     // Build per-project checkout commands: one worktree, merge all PR branches into it
     const checkoutBlocks = Object.entries(projectPrs).map(([name, { project: p, prs, mainBranch }]) => {
-      const wtPath = `${p.localPath}/../worktrees/verify-${name}-${planSlug}-${shared.uid()}`;
+      const localPath = p.localPath.replace(/\\/g, '/');
+      const wtPath = `${localPath}/../worktrees/verify-${name}-${planSlug}-${shared.uid()}`;
       const branches = prs.map(pr => pr.branch).filter(Boolean);
       const lines = [
         `# ${name} — merge ${branches.length} PR branch(es) into one worktree`,
-        `cd "${p.localPath}"`,
+        `cd "${localPath}"`,
         `git fetch origin ${branches.map(b => `"${b}"`).join(' ')} "${mainBranch}"`,
         `git worktree add "${wtPath}" "origin/${mainBranch}" 2>/dev/null || (cd "${wtPath}" && git checkout "${mainBranch}" && git pull origin "${mainBranch}")`,
         `cd "${wtPath}"`,
@@ -901,7 +903,7 @@ function extractSkillsFromOutput(output, agentId, dispatchItem, config) {
       }
     } else {
       // Write in Claude Code native format: ~/.claude/skills/<name>/SKILL.md
-      const claudeSkillsDir = path.join(process.env.HOME || process.env.USERPROFILE || '', '.claude', 'skills');
+      const claudeSkillsDir = path.join(os.homedir(), '.claude', 'skills');
       const skillDir = path.join(claudeSkillsDir, name.replace(/[^a-z0-9-]/g, '-'));
       const skillPath = path.join(skillDir, 'SKILL.md');
       if (!fs.existsSync(skillPath)) {

package/engine/llm.js

@@ -72,7 +72,7 @@ function callLLM(promptText, sysPromptText, { timeout = 120000, label = 'llm', m
     proc.stdout.on('data', d => { stdout += d.toString(); });
     proc.stderr.on('data', d => { stderr += d.toString(); });
 
-    const timer = setTimeout(() => { try { proc.kill('SIGTERM'); } catch { /* process may be dead */ } }, timeout);
+    const timer = setTimeout(() => { shared.killImmediate(proc); }, timeout);
 
     proc.on('close', (code) => {
       clearTimeout(timer);
@@ -180,7 +180,7 @@ function callLLMStreaming(promptText, sysPromptText, { timeout = 120000, label =
     });
     proc.stderr.on('data', d => { stderr += d.toString(); });
 
-    const timer = setTimeout(() => { try { proc.kill('SIGTERM'); } catch {} }, timeout);
+    const timer = setTimeout(() => { shared.killImmediate(proc); }, timeout);
 
     proc.on('close', (code) => {
       clearTimeout(timer);

package/engine/queries.js

@@ -6,6 +6,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const os = require('os');
 const shared = require('./shared');
 
 const { safeRead, safeReadDir, safeJson, safeWrite, getProjects,
@@ -19,7 +20,7 @@ const ENGINE_DIR = path.join(MINIONS_DIR, 'engine');
 const INBOX_DIR = path.join(MINIONS_DIR, 'notes', 'inbox');
 const PLANS_DIR = path.join(MINIONS_DIR, 'plans');
 const PRD_DIR = path.join(MINIONS_DIR, 'prd');
-const SKILLS_DIR = path.join(process.env.HOME || process.env.USERPROFILE || '', '.claude', 'skills');
+const SKILLS_DIR = path.join(os.homedir(), '.claude', 'skills');
 const KNOWLEDGE_DIR = path.join(MINIONS_DIR, 'knowledge');
 const ARCHIVE_DIR = path.join(MINIONS_DIR, 'notes', 'archive');
 
@@ -332,7 +333,7 @@ function collectSkillFiles(config) {
   const seen = new Set(); // dedup by name
 
   // 1. Claude Code native skills: ~/.claude/skills/<name>/SKILL.md
-  const homeDir = process.env.HOME || process.env.USERPROFILE || '';
+  const homeDir = os.homedir();
   const claudeSkillsDir = path.join(homeDir, '.claude', 'skills');
   try {
     const dirs = fs.readdirSync(claudeSkillsDir).filter(d => {

package/engine/shared.js

@@ -536,6 +536,32 @@ function addPrLink(prId, itemId) {
   safeWrite(PR_LINKS_PATH, links);
 }
 
+// ─── Cross-Platform Process Kill Helpers ─────────────────────────────────────
+
+function killGracefully(proc, graceMs = 5000) {
+  if (!proc || !proc.pid) return;
+  if (process.platform === 'win32') {
+    try { _execSync(`taskkill /PID ${proc.pid} /T`, { stdio: 'pipe', timeout: 3000, windowsHide: true }); } catch { /* process may be dead */ }
+    setTimeout(() => {
+      try { _execSync(`taskkill /PID ${proc.pid} /F /T`, { stdio: 'pipe', timeout: 3000, windowsHide: true }); } catch { /* process may be dead */ }
+    }, graceMs);
+  } else {
+    try { proc.kill('SIGTERM'); } catch { /* process may be dead */ }
+    setTimeout(() => {
+      try { proc.kill('SIGKILL'); } catch { /* process may be dead */ }
+    }, graceMs);
+  }
+}
+
+function killImmediate(proc) {
+  if (!proc || !proc.pid) return;
+  if (process.platform === 'win32') {
+    try { _execSync(`taskkill /PID ${proc.pid} /F /T`, { stdio: 'pipe', timeout: 3000, windowsHide: true }); } catch { /* process may be dead */ }
+  } else {
+    try { proc.kill('SIGKILL'); } catch { /* process may be dead */ }
+  }
+}
+
 module.exports = {
   MINIONS_DIR,
   PR_LINKS_PATH,
@@ -580,6 +606,8 @@ module.exports = {
   validatePid,
   parseSkillFrontmatter,
   sleepMs,
+  killGracefully,
+  killImmediate,
   LOCK_STALE_MS,
 };
 

package/engine/timeout.js

@@ -72,7 +72,7 @@ function checkSteering(config) {
     log('info', `Steering: killing ${info.agentId} (${id}) for session resume with human message`);
 
     // Kill current process
-    try { info.proc.kill('SIGTERM'); } catch { /* process may be dead */ }
+    shared.killImmediate(info.proc);
 
     // Store steering context for re-spawn on close
     info._steeringMessage = message;
@@ -97,10 +97,7 @@ function checkTimeouts(config) {
     const elapsed = Date.now() - new Date(info.startedAt).getTime();
     if (elapsed > itemTimeout) {
       log('warn', `Agent ${info.agentId} (${id}) hit hard timeout after ${Math.round(elapsed / 1000)}s — killing`);
-      try { info.proc.kill('SIGTERM'); } catch { /* process may be dead */ }
-      setTimeout(() => {
-        try { info.proc.kill('SIGKILL'); } catch { /* process may be dead */ }
-      }, 5000);
+      shared.killGracefully(info.proc, 5000);
     }
   }
 
@@ -149,7 +146,7 @@ function checkTimeouts(config) {
         runPostCompletionHooks(item, item.agent, isSuccess ? 0 : 1, liveLog, config);
 
         if (hasProcess) {
-          try { activeProcesses.get(item.id)?.proc.kill('SIGTERM'); } catch { /* process may be dead */ }
+          shared.killImmediate(activeProcesses.get(item.id)?.proc);
           activeProcesses.delete(item.id);
         }
         continue; // Skip orphan/hung detection — we handled it
@@ -208,8 +205,7 @@ function checkTimeouts(config) {
       log('warn', `Hung agent: ${item.agent} (${item.id}) — process exists but no output for ${silentSec}s${isBlocking ? ' (blocking timeout exceeded)' : ''}`);
       const procInfo = activeProcesses.get(item.id);
       if (procInfo) {
-        try { procInfo.proc.kill('SIGTERM'); } catch { /* process may be dead */ }
-        setTimeout(() => { try { procInfo.proc.kill('SIGKILL'); } catch { /* process may be dead */ } }, 5000);
+        shared.killGracefully(procInfo.proc, 5000);
         activeProcesses.delete(item.id);
       }
       deadItems.push({ item, reason: `Hung — no output for ${silentSec}s` });

package/package.json

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