@chuckssmith/agentloom 1.0.0 → 1.1.0

package/dist/commands/collect.js

@@ -3,6 +3,15 @@ import { join } from 'path';
 import { existsSync } from 'fs';
 import { spawnSync } from 'child_process';
 import { STATE_DIR, readSession } from '../state/session.js';
+function isProcessAlive(pid) {
+    try {
+        process.kill(pid, 0);
+        return true;
+    }
+    catch {
+        return false;
+    }
+}
 const WORKERS_DIR = join(STATE_DIR, 'workers');
 export async function collect(args) {
     if (!existsSync(WORKERS_DIR)) {
@@ -17,6 +26,21 @@ export async function collect(args) {
         console.log('Workers still running? Check: loom logs');
         return;
     }
+    // Warn if any workers are still running — synthesis will be incomplete
+    const pidFiles = files.filter(f => f.endsWith('.pid'));
+    const stillRunning = [];
+    for (const pidFile of pidFiles) {
+        const id = pidFile.replace('.pid', '');
+        if (existsSync(join(WORKERS_DIR, `${id}-result.md`)))
+            continue;
+        const pid = parseInt(await readFile(join(WORKERS_DIR, pidFile), 'utf8').catch(() => ''), 10);
+        if (!isNaN(pid) && isProcessAlive(pid))
+            stillRunning.push(id);
+    }
+    if (stillRunning.length > 0) {
+        console.log(`⚠  Workers still running: ${stillRunning.join(', ')} — results will be incomplete`);
+        console.log(`   Wait for completion or run: loom stop\n`);
+    }
     console.log(`\nCollecting results from ${resultFiles.length} worker(s)...\n`);
     const results = [];
     for (const f of resultFiles) {

package/dist/commands/crew.js

@@ -145,7 +145,7 @@ async function launchSerial(sessionId, specs, tasks, contextPath, forcePermissio
             console.log(`  → Worker ${workerId} (${agentType}) starting...`);
             const claudeArgs = [
                 '--print',
-                ...(forcePermissions || !READ_ONLY_ROLES.has(agentType) ? ['--dangerously-skip-permissions'] : []),
+                ...(forcePermissions && !READ_ONLY_ROLES.has(agentType) ? ['--dangerously-skip-permissions'] : []),
                 '-p',
                 prompt,
             ];
@@ -190,11 +190,16 @@ async function launchBackground(sessionId, specs, tasks, contextPath, forcePermi
             // Build args declaratively — no positional splicing
             const claudeArgs = [
                 '--print',
-                ...(forcePermissions || !READ_ONLY_ROLES.has(agentType) ? ['--dangerously-skip-permissions'] : []),
+                ...(forcePermissions && !READ_ONLY_ROLES.has(agentType) ? ['--dangerously-skip-permissions'] : []),
                 '-p',
                 prompt,
             ];
             const log = await open(logFile, 'w');
+            let logClosed = false;
+            const closeLog = () => { if (!logClosed) {
+                logClosed = true;
+                log.close().catch(() => { });
+            } };
             const child = spawn('claude', claudeArgs, {
                 detached: true,
                 stdio: ['ignore', log.fd, log.fd],
@@ -202,9 +207,9 @@ async function launchBackground(sessionId, specs, tasks, contextPath, forcePermi
             });
             child.on('error', async (err) => {
                 await writeFile(join(STATE_DIR, 'workers', `${workerId}-result.md`), `# Launch Error\n\nFailed to start worker: ${err.message}\n`).catch(() => { });
-                log.close().catch(() => { });
+                closeLog();
             });
-            child.on('close', () => { log.close().catch(() => { }); });
+            child.on('close', () => { closeLog(); });
             if (child.pid != null) {
                 await writeFile(pidFile, String(child.pid));
             }
@@ -221,11 +226,9 @@ async function launchTmux(sessionId, specs, tasks, contextPath, forcePermissions
         console.error(`tmux session "${tmuxSession}" already exists. Run: tmux kill-session -t ${tmuxSession}`);
         process.exit(1);
     }
-    try {
-        execSync(`tmux new-session -d -s ${tmuxSession} -x 220 -y 50`);
-    }
-    catch (err) {
-        console.error(`Failed to create tmux session: ${err instanceof Error ? err.message : err}`);
+    const newSession = spawnSync('tmux', ['new-session', '-d', '-s', tmuxSession, '-x', '220', '-y', '50'], { stdio: 'ignore' });
+    if (newSession.status !== 0) {
+        console.error(`Failed to create tmux session: ${newSession.stderr?.toString().trim() ?? 'unknown error'}`);
         process.exit(1);
     }
     await mkdir(join(STATE_DIR, 'workers'), { recursive: true });
@@ -238,18 +241,23 @@ async function launchTmux(sessionId, specs, tasks, contextPath, forcePermissions
             workerIdx++;
             const prompt = buildWorkerPrompt(subtask, contextPath, sessionId, workerId, agentType);
             const promptFile = join(STATE_DIR, 'workers', `${workerId}-prompt.md`);
+            const logFile = join(STATE_DIR, 'workers', `${workerId}.log`);
+            const pidFile = join(STATE_DIR, 'workers', `${workerId}.pid`);
             const scriptFile = join(STATE_DIR, 'workers', `${workerId}-run.mjs`);
             await writeFile(promptFile, prompt);
-            // Node.js runner — JSON.stringify safely encodes all values, no shell expansion possible
+            // Node.js runner — JSON.stringify safely encodes all values, no shell expansion possible.
+            // Writes PID and logs to the same files as launchBackground so loom stop/watch/status work.
+            const skipPerms = forcePermissions && !READ_ONLY_ROLES.has(agentType);
             await writeFile(scriptFile, [
-                `import { readFileSync } from 'fs'`,
+                `import { readFileSync, writeFileSync, openSync } from 'fs'`,
                 `import { spawnSync } from 'child_process'`,
                 `process.env.AGENTLOOM_WORKER_ID = ${JSON.stringify(workerId)}`,
                 `process.env.AGENTLOOM_SESSION = ${JSON.stringify(sessionId)}`,
+                `writeFileSync(${JSON.stringify(pidFile)}, String(process.pid))`,
+                `const logFd = openSync(${JSON.stringify(logFile)}, 'w')`,
                 `const prompt = readFileSync(${JSON.stringify(promptFile)}, 'utf8')`,
-                `const args = ['--print', ${(forcePermissions || !READ_ONLY_ROLES.has(agentType)) ? `'--dangerously-skip-permissions', ` : ``}'${'-p'}', prompt]`,
-                `const r = spawnSync('claude', args, { stdio: 'inherit' })`,
-                `console.log('[worker done]')`,
+                `const args = ['--print', ${skipPerms ? `'--dangerously-skip-permissions', ` : ``}'${'-p'}', prompt]`,
+                `const r = spawnSync('claude', args, { stdio: ['ignore', logFd, logFd] })`,
                 `process.exit(r.status ?? 0)`,
             ].join('\n'));
             if (workerIdx > 1) {

package/dist/commands/setup.js

@@ -2,13 +2,13 @@ import { mkdir, copyFile, readdir } from 'fs/promises';
 import { join, dirname } from 'path';
 import { homedir } from 'os';
 import { fileURLToPath } from 'url';
+import { execSync } from 'child_process';
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const SKILLS_SRC = join(__dirname, '../../skills');
 const SKILLS_DEST = join(homedir(), '.claude', 'skills');
 export async function setup() {
     console.log('agentloom setup\n');
     // 1. Validate claude CLI exists
-    const { execSync } = await import('child_process');
     try {
         execSync('claude --version', { stdio: 'ignore' });
         console.log('✓ claude CLI found');

package/dist/commands/watch.js

@@ -2,9 +2,9 @@ import { readdir, stat, readFile } from 'fs/promises';
 import { join } from 'path';
 import { existsSync } from 'fs';
 import { STATE_DIR } from '../state/session.js';
+import { loadConfig } from '../config.js';
 const WORKERS_DIR = join(STATE_DIR, 'workers');
 const POLL_MS = 800;
-const STALE_TIMEOUT_MS = 15 * 60 * 1000; // 15 min no log growth + dead PID = give up
 const COLORS = ['\x1b[36m', '\x1b[33m', '\x1b[35m', '\x1b[32m', '\x1b[34m', '\x1b[31m'];
 const RESET = '\x1b[0m';
 const DIM = '\x1b[2m';
@@ -21,8 +21,10 @@ function isProcessAlive(pid) {
 export async function watch(_args) {
     if (!existsSync(WORKERS_DIR)) {
         console.log('No active session. Run: loom crew "<task>"');
-        process.exit(1);
+        return;
     }
+    const config = await loadConfig();
+    const STALE_TIMEOUT_MS = config.staleMinutes * 60 * 1000;
     console.log(`${DIM}Watching worker logs. Ctrl+C to stop.${RESET}\n`);
     const offsets = {};
     const lastGrowth = {};
@@ -102,7 +104,7 @@ export async function watch(_args) {
                 }
             }
             if (staleWorkers.length > 0 && staleWorkers.length + workersDone.length === logFiles.length) {
-                console.log(`\n${YELLOW}Workers stalled (dead PID, no output for 15min): ${staleWorkers.join(', ')}${RESET}`);
+                console.log(`\n${YELLOW}Workers stalled (dead PID, no output for ${config.staleMinutes}min): ${staleWorkers.join(', ')}${RESET}`);
                 console.log(`${DIM}Run: loom logs <workerId>  to inspect. loom collect to gather what's available.${RESET}`);
                 break;
             }

package/dist/config.js

@@ -6,7 +6,9 @@ const DEFAULTS = {
     agentType: 'general-purpose',
     claimTtlMinutes: 30,
     staleMinutes: 10,
-    dangerouslySkipPermissions: false,
+    // Default true: background workers must skip permission prompts to run non-interactively.
+    // Set to false in .loomrc to require interactive approval (workers will pause on each action).
+    dangerouslySkipPermissions: true,
 };
 export async function loadConfig() {
     if (!existsSync(LOOMRC))

package/dist/state/session.d.ts

@@ -1,4 +1,4 @@
-export declare const STATE_DIR = ".claude-team";
+export declare const STATE_DIR: string;
 export type TaskStatus = 'pending' | 'claimed' | 'done' | 'failed';
 export type Task = {
     id: string;

package/dist/state/session.js

@@ -1,6 +1,8 @@
-import { readFile, writeFile, mkdir, readdir } from 'fs/promises';
+import { readFile, writeFile, mkdir, readdir, unlink } from 'fs/promises';
 import { join } from 'path';
-export const STATE_DIR = '.claude-team';
+// Absolute path so loom works correctly regardless of which subdirectory it's invoked from.
+// NOTE: resolves to cwd at process start — does not walk up to find project root.
+export const STATE_DIR = join(process.cwd(), '.claude-team');
 export async function ensureStateDir() {
     await mkdir(join(STATE_DIR, 'tasks'), { recursive: true });
     await mkdir(join(STATE_DIR, 'workers'), { recursive: true });
@@ -18,13 +20,32 @@ export async function readSession() {
     }
 }
 export async function writeTask(task) {
-    await writeFile(join(STATE_DIR, 'tasks', `${task.id}-${task.status}.json`), JSON.stringify(task, null, 2));
+    const dir = join(STATE_DIR, 'tasks');
+    // Remove any existing files for this task id to prevent ghost duplicates on status change.
+    try {
+        const files = await readdir(dir);
+        await Promise.all(files
+            .filter(f => f.startsWith(`${task.id}-`) && f.endsWith('.json'))
+            .map(f => unlink(join(dir, f)).catch(() => { })));
+    }
+    catch { /* dir may not exist yet — ensureStateDir handles this */ }
+    await writeFile(join(dir, `${task.id}-${task.status}.json`), JSON.stringify(task, null, 2));
 }
 export async function readTasks() {
     const dir = join(STATE_DIR, 'tasks');
     try {
         const files = await readdir(dir);
-        return Promise.all(files.filter(f => f.endsWith('.json')).map(async (f) => JSON.parse(await readFile(join(dir, f), 'utf8'))));
+        const tasks = await Promise.all(files.filter(f => f.endsWith('.json')).map(async (f) => JSON.parse(await readFile(join(dir, f), 'utf8'))));
+        // Deduplicate by id — keep highest-priority status in case of ghost duplicates.
+        const STATUS_PRIORITY = { done: 4, failed: 3, claimed: 2, pending: 1 };
+        const byId = new Map();
+        for (const task of tasks) {
+            const existing = byId.get(task.id);
+            if (!existing || (STATUS_PRIORITY[task.status] ?? 0) > (STATUS_PRIORITY[existing.status] ?? 0)) {
+                byId.set(task.id, task);
+            }
+        }
+        return [...byId.values()];
     }
     catch {
         return [];

package/dist/team/orchestrator.js

@@ -28,7 +28,7 @@ export function parseWorkerSpec(args, defaultWorkers = 2, defaultAgentType = 'ge
 export async function initSession(description, workerCount) {
     await ensureStateDir();
     const session = {
-        id: randomUUID().slice(0, 8),
+        id: randomUUID().replace(/-/g, '').slice(0, 16),
         description,
         status: 'running',
         workerCount,
@@ -52,7 +52,7 @@ export async function decomposeTasks(task, specs, dryRun = false) {
     for (const spec of specs) {
         for (let i = 0; i < spec.count; i++) {
             const t = {
-                id: randomUUID().slice(0, 8),
+                id: randomUUID().replace(/-/g, '').slice(0, 16),
                 description: subtasks[idx] ?? task,
                 agentType: spec.agentType,
                 status: 'pending',
@@ -80,10 +80,12 @@ Task: "${task}"`;
         if (result.status !== 0 || !result.stdout) {
             throw new Error(result.stderr ?? 'no output');
         }
-        const match = result.stdout.match(/\[[\s\S]*\]/);
-        if (!match)
+        // Use first '[' and last ']' to avoid greedy regex over-capturing trailing brackets
+        const start = result.stdout.indexOf('[');
+        const end = result.stdout.lastIndexOf(']');
+        if (start === -1 || end === -1 || end < start)
             throw new Error('No JSON array in response');
-        const parsed = JSON.parse(match[0]);
+        const parsed = JSON.parse(result.stdout.slice(start, end + 1));
         if (!Array.isArray(parsed) || parsed.length === 0)
             throw new Error('Empty array');
         const subtasks = parsed.map(String);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chuckssmith/agentloom",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "A workflow layer for Claude Code — reusable roles, persistence loops, and multi-agent crew coordination",
   "keywords": [
     "ai",