@locusai/cli 0.25.2 → 0.25.4

package/bin/locus.js

@@ -3352,46 +3352,155 @@ var init_lock = __esm(() => {
 });
 
 // src/skills/installer.ts
-import { existsSync as existsSync11, mkdirSync as mkdirSync8, rmSync, writeFileSync as writeFileSync8 } from "node:fs";
+import {
+  existsSync as existsSync11,
+  mkdirSync as mkdirSync8,
+  readFileSync as readFileSync8,
+  renameSync as renameSync2,
+  rmSync,
+  writeFileSync as writeFileSync8
+} from "node:fs";
+import { tmpdir } from "node:os";
 import { join as join11 } from "node:path";
 async function installSkill(projectRoot, name, content, source) {
   const claudeDir = join11(projectRoot, CLAUDE_SKILLS_DIR, name);
   const agentsDir = join11(projectRoot, AGENTS_SKILLS_DIR, name);
-  mkdirSync8(claudeDir, { recursive: true });
-  mkdirSync8(agentsDir, { recursive: true });
-  writeFileSync8(join11(claudeDir, "SKILL.md"), content, "utf-8");
-  writeFileSync8(join11(agentsDir, "SKILL.md"), content, "utf-8");
-  const lock = readLockFile(projectRoot);
-  lock.skills[name] = {
-    source,
-    sourceType: "github",
-    computedHash: computeSkillHash(content)
-  };
-  writeLockFile(projectRoot, lock);
+  const stagingDir = join11(tmpdir(), `locus-skill-${name}-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`);
+  const stagingClaudeDir = join11(stagingDir, "claude");
+  const stagingAgentsDir = join11(stagingDir, "agents");
+  let wroteClaudeDir = false;
+  let wroteAgentsDir = false;
+  try {
+    try {
+      mkdirSync8(stagingClaudeDir, { recursive: true });
+      mkdirSync8(stagingAgentsDir, { recursive: true });
+      writeFileSync8(join11(stagingClaudeDir, "SKILL.md"), content, "utf-8");
+      writeFileSync8(join11(stagingAgentsDir, "SKILL.md"), content, "utf-8");
+    } catch (err) {
+      throw new SkillInstallError(name, "stage", err);
+    }
+    try {
+      if (!content || content.trim().length === 0) {
+        throw new Error("Skill content is empty");
+      }
+      const expectedHash = computeSkillHash(content);
+      const stagedContent = readFileSync8(join11(stagingClaudeDir, "SKILL.md"), "utf-8");
+      const stagedHash = computeSkillHash(stagedContent);
+      if (stagedHash !== expectedHash) {
+        throw new Error("Staged file hash does not match expected content");
+      }
+    } catch (err) {
+      if (err instanceof SkillInstallError)
+        throw err;
+      throw new SkillInstallError(name, "validate", err);
+    }
+    try {
+      mkdirSync8(join11(projectRoot, CLAUDE_SKILLS_DIR), { recursive: true });
+      mkdirSync8(join11(projectRoot, AGENTS_SKILLS_DIR), { recursive: true });
+      if (existsSync11(claudeDir)) {
+        rmSync(claudeDir, { recursive: true, force: true });
+      }
+      if (existsSync11(agentsDir)) {
+        rmSync(agentsDir, { recursive: true, force: true });
+      }
+      try {
+        renameSync2(stagingClaudeDir, claudeDir);
+        wroteClaudeDir = true;
+      } catch {
+        mkdirSync8(claudeDir, { recursive: true });
+        writeFileSync8(join11(claudeDir, "SKILL.md"), content, "utf-8");
+        wroteClaudeDir = true;
+      }
+      try {
+        renameSync2(stagingAgentsDir, agentsDir);
+        wroteAgentsDir = true;
+      } catch {
+        mkdirSync8(agentsDir, { recursive: true });
+        writeFileSync8(join11(agentsDir, "SKILL.md"), content, "utf-8");
+        wroteAgentsDir = true;
+      }
+    } catch (err) {
+      if (err instanceof SkillInstallError)
+        throw err;
+      throw new SkillInstallError(name, "write", err);
+    }
+    try {
+      const lock = readLockFile(projectRoot);
+      lock.skills[name] = {
+        source,
+        sourceType: "github",
+        computedHash: computeSkillHash(content)
+      };
+      writeLockFile(projectRoot, lock);
+    } catch (err) {
+      throw new SkillInstallError(name, "register", err);
+    }
+  } catch (err) {
+    if (wroteClaudeDir && existsSync11(claudeDir)) {
+      rmSync(claudeDir, { recursive: true, force: true });
+    }
+    if (wroteAgentsDir && existsSync11(agentsDir)) {
+      rmSync(agentsDir, { recursive: true, force: true });
+    }
+    throw err;
+  } finally {
+    if (existsSync11(stagingDir)) {
+      rmSync(stagingDir, { recursive: true, force: true });
+    }
+  }
 }
 async function removeSkill(projectRoot, name) {
-  if (!isSkillInstalled(projectRoot, name)) {
+  const claudeDir = join11(projectRoot, CLAUDE_SKILLS_DIR, name);
+  const agentsDir = join11(projectRoot, AGENTS_SKILLS_DIR, name);
+  const lock = readLockFile(projectRoot);
+  const inLockFile = name in lock.skills;
+  const hasClaude = existsSync11(claudeDir);
+  const hasAgents = existsSync11(agentsDir);
+  if (!inLockFile && !hasClaude && !hasAgents) {
     console.warn(`Skill "${name}" is not installed.`);
     return;
   }
-  const claudeDir = join11(projectRoot, CLAUDE_SKILLS_DIR, name);
-  const agentsDir = join11(projectRoot, AGENTS_SKILLS_DIR, name);
-  if (existsSync11(claudeDir)) {
+  if (hasClaude) {
     rmSync(claudeDir, { recursive: true, force: true });
   }
-  if (existsSync11(agentsDir)) {
+  if (hasAgents) {
     rmSync(agentsDir, { recursive: true, force: true });
   }
-  const lock = readLockFile(projectRoot);
-  delete lock.skills[name];
-  writeLockFile(projectRoot, lock);
+  if (inLockFile) {
+    delete lock.skills[name];
+    writeLockFile(projectRoot, lock);
+  }
 }
 function isSkillInstalled(projectRoot, name) {
   const lock = readLockFile(projectRoot);
   return name in lock.skills;
 }
+function hasOrphanedSkillFiles(projectRoot, name) {
+  const claudeDir = join11(projectRoot, CLAUDE_SKILLS_DIR, name);
+  const agentsDir = join11(projectRoot, AGENTS_SKILLS_DIR, name);
+  const inLockFile = isSkillInstalled(projectRoot, name);
+  return !inLockFile && (existsSync11(claudeDir) || existsSync11(agentsDir));
+}
+var SkillInstallError;
 var init_installer = __esm(() => {
   init_lock();
+  SkillInstallError = class SkillInstallError extends Error {
+    skillName;
+    step;
+    constructor(skillName, step, cause) {
+      const stepLabel = {
+        stage: "staging files to temp directory",
+        validate: "validating skill content",
+        write: "writing skill files to project",
+        register: "updating skills-lock.json"
+      };
+      const reason = cause instanceof Error ? cause.message : String(cause ?? "");
+      super(`Failed to install '${skillName}' during ${stepLabel[step]}${reason ? `: ${reason}` : ""}`);
+      this.skillName = skillName;
+      this.step = step;
+      this.name = "SkillInstallError";
+    }
+  };
 });
 
 // src/skills/registry.ts
@@ -3565,7 +3674,22 @@ async function installRemoteSkill(name) {
   }
   const cwd = process.cwd();
   const source = `${REGISTRY_REPO}/${entry.path}`;
-  await installSkill(cwd, name, content, source);
+  try {
+    await installSkill(cwd, name, content, source);
+  } catch (err) {
+    if (err instanceof SkillInstallError) {
+      process.stderr.write(`${red2("✗")} ${err.message}
+`);
+      process.stderr.write(`  To clean up and retry: ${bold2(`locus skills remove ${name}`)} then ${bold2(`locus skills install ${name}`)}
+`);
+    } else {
+      process.stderr.write(`${red2("✗")} Unexpected error installing skill '${name}'.
+`);
+      process.stderr.write(`  ${dim2(err.message)}
+`);
+    }
+    process.exit(1);
+  }
   process.stderr.write(`
 ${green("✓")} Installed skill '${bold2(name)}' from ${REGISTRY_REPO}
 `);
@@ -3584,7 +3708,9 @@ async function removeInstalledSkill(name) {
     process.exit(1);
   }
   const cwd = process.cwd();
-  if (!isSkillInstalled(cwd, name)) {
+  const installed = isSkillInstalled(cwd, name);
+  const orphaned = hasOrphanedSkillFiles(cwd, name);
+  if (!installed && !orphaned) {
     process.stderr.write(`${red2("✗")} Skill '${bold2(name)}' is not installed.
 `);
     process.stderr.write(`  Run ${bold2("locus skills list --installed")} to see installed skills.
@@ -3592,8 +3718,13 @@ async function removeInstalledSkill(name) {
     process.exit(1);
   }
   await removeSkill(cwd, name);
-  process.stderr.write(`${green("✓")} Removed skill '${bold2(name)}'
+  if (orphaned) {
+    process.stderr.write(`${green("✓")} Cleaned up orphaned skill files for '${bold2(name)}'
+`);
+  } else {
+    process.stderr.write(`${green("✓")} Removed skill '${bold2(name)}'
 `);
+  }
 }
 async function updateSkills(name) {
   const cwd = process.cwd();
@@ -3649,7 +3780,20 @@ async function updateSkills(name) {
       continue;
     }
     const source = `${REGISTRY_REPO}/${entry.path}`;
-    await installSkill(cwd, skillName, content, source);
+    try {
+      await installSkill(cwd, skillName, content, source);
+    } catch (err) {
+      if (err instanceof SkillInstallError) {
+        process.stderr.write(`${red2("✗")} ${err.message}
+`);
+        process.stderr.write(`  To clean up: ${bold2(`locus skills remove ${skillName}`)}
+`);
+      } else {
+        process.stderr.write(`${red2("✗")} Failed to update '${skillName}': ${dim2(err.message)}
+`);
+      }
+      continue;
+    }
     updatedCount++;
     process.stderr.write(`${green("✓")} Updated '${bold2(skillName)}' (hash changed)
 `);
@@ -3743,7 +3887,7 @@ ${bold2("Subcommands:")}
   ${cyan2("list")}              List available skills from the registry
   ${cyan2("list")} ${dim2("--installed")}   List locally installed skills
   ${cyan2("install")} ${dim2("<name>")}    Install a skill from the registry
-  ${cyan2("remove")} ${dim2("<name>")}     Remove an installed skill
+  ${cyan2("remove")} ${dim2("<name>")}     Remove an installed skill (alias: ${cyan2("uninstall")})
   ${cyan2("update")} ${dim2("[name]")}     Update installed skill(s) from registry
   ${cyan2("info")} ${dim2("<name>")}       Show skill metadata and install status
 
@@ -3778,7 +3922,8 @@ async function skillsCommand(args, flags) {
       await installRemoteSkill(skillName);
       break;
     }
-    case "remove": {
+    case "remove":
+    case "uninstall": {
       const skillName = args[1];
       await removeInstalledSkill(skillName);
       break;
@@ -3796,7 +3941,7 @@ async function skillsCommand(args, flags) {
     default:
       process.stderr.write(`${red2("✗")} Unknown subcommand: ${bold2(subcommand)}
 `);
-      process.stderr.write(`  Available: ${bold2("list")}, ${bold2("install")}, ${bold2("remove")}, ${bold2("update")}, ${bold2("info")}
+      process.stderr.write(`  Available: ${bold2("list")}, ${bold2("install")}, ${bold2("remove")} (${bold2("uninstall")}), ${bold2("update")}, ${bold2("info")}
 `);
       process.exit(1);
   }
@@ -3947,7 +4092,7 @@ __export(exports_logs, {
 import {
   existsSync as existsSync12,
   readdirSync as readdirSync2,
-  readFileSync as readFileSync8,
+  readFileSync as readFileSync9,
   statSync as statSync2,
   unlinkSync as unlinkSync2
 } from "node:fs";
@@ -3974,7 +4119,7 @@ async function logsCommand(cwd, options) {
   return viewLog(logFiles[0], options.level, options.lines ?? 50);
 }
 function viewLog(logFile, levelFilter, maxLines) {
-  const content = readFileSync8(logFile, "utf-8");
+  const content = readFileSync9(logFile, "utf-8");
   const lines = content.trim().split(`
 `).filter(Boolean);
   process.stderr.write(`
@@ -4011,7 +4156,7 @@ async function tailLog(logFile, levelFilter) {
 `);
   let lastSize = existsSync12(logFile) ? statSync2(logFile).size : 0;
   if (existsSync12(logFile)) {
-    const content = readFileSync8(logFile, "utf-8");
+    const content = readFileSync9(logFile, "utf-8");
     const lines = content.trim().split(`
 `).filter(Boolean);
     const recent = lines.slice(-10);
@@ -4034,7 +4179,7 @@ async function tailLog(logFile, levelFilter) {
       const currentSize = statSync2(logFile).size;
       if (currentSize <= lastSize)
         return;
-      const content = readFileSync8(logFile, "utf-8");
+      const content = readFileSync9(logFile, "utf-8");
       const allLines = content.trim().split(`
 `).filter(Boolean);
       const oldContent = content.slice(0, lastSize);
@@ -4400,7 +4545,7 @@ var init_stream_renderer = __esm(() => {
 // src/repl/clipboard.ts
 import { execSync as execSync6 } from "node:child_process";
 import { existsSync as existsSync13, mkdirSync as mkdirSync9 } from "node:fs";
-import { tmpdir } from "node:os";
+import { tmpdir as tmpdir2 } from "node:os";
 import { join as join13 } from "node:path";
 function readClipboardImage() {
   if (process.platform === "darwin") {
@@ -4466,12 +4611,12 @@ function readLinuxClipboardImage() {
 }
 var STABLE_DIR;
 var init_clipboard = __esm(() => {
-  STABLE_DIR = join13(tmpdir(), "locus-images");
+  STABLE_DIR = join13(tmpdir2(), "locus-images");
 });
 
 // src/repl/image-detect.ts
 import { copyFileSync, existsSync as existsSync14, mkdirSync as mkdirSync10 } from "node:fs";
-import { homedir as homedir3, tmpdir as tmpdir2 } from "node:os";
+import { homedir as homedir3, tmpdir as tmpdir3 } from "node:os";
 import { basename, extname, join as join14, resolve } from "node:path";
 function detectImages(input) {
   const detected = [];
@@ -4676,7 +4821,7 @@ var init_image_detect = __esm(() => {
     ".tif",
     ".tiff"
   ]);
-  STABLE_DIR2 = join14(tmpdir2(), "locus-images");
+  STABLE_DIR2 = join14(tmpdir3(), "locus-images");
   PLACEHOLDER_ID_PATTERN = /\(locus:\/\/screenshot-(\d+)\)/g;
 });
 
@@ -5689,17 +5834,17 @@ import {
   mkdirSync as mkdirSync11,
   mkdtempSync,
   readdirSync as readdirSync3,
-  readFileSync as readFileSync9,
+  readFileSync as readFileSync10,
   rmSync as rmSync2,
   statSync as statSync3
 } from "node:fs";
-import { tmpdir as tmpdir3 } from "node:os";
+import { tmpdir as tmpdir4 } from "node:os";
 import { dirname as dirname3, join as join15, relative } from "node:path";
 import { promisify } from "node:util";
 function parseIgnoreFile(filePath) {
   if (!existsSync15(filePath))
     return [];
-  const content = readFileSync9(filePath, "utf-8");
+  const content = readFileSync10(filePath, "utf-8");
   const rules = [];
   for (const rawLine of content.split(`
 `)) {
@@ -5810,7 +5955,7 @@ function backupIgnoredFiles(projectRoot) {
     return NOOP_BACKUP;
   let backupDir;
   try {
-    backupDir = mkdtempSync(join15(tmpdir3(), "locus-sandbox-backup-"));
+    backupDir = mkdtempSync(join15(tmpdir4(), "locus-sandbox-backup-"));
   } catch (err) {
     log.debug("Failed to create sandbox backup dir", {
       error: err instanceof Error ? err.message : String(err)
@@ -8028,7 +8173,7 @@ var init_sprint = __esm(() => {
 
 // src/core/prompt-builder.ts
 import { execSync as execSync9 } from "node:child_process";
-import { existsSync as existsSync16, readdirSync as readdirSync4, readFileSync as readFileSync10 } from "node:fs";
+import { existsSync as existsSync16, readdirSync as readdirSync4, readFileSync as readFileSync11 } from "node:fs";
 import { join as join16 } from "node:path";
 function buildExecutionPrompt(ctx) {
   const sections = [];
@@ -8256,7 +8401,7 @@ function readFileSafe(path) {
   try {
     if (!existsSync16(path))
       return null;
-    return readFileSync10(path, "utf-8");
+    return readFileSync11(path, "utf-8");
   } catch {
     return null;
   }
@@ -8825,7 +8970,7 @@ class CombinedCompletion {
 var init_completions = () => {};
 
 // src/repl/input-history.ts
-import { existsSync as existsSync17, mkdirSync as mkdirSync12, readFileSync as readFileSync11, writeFileSync as writeFileSync9 } from "node:fs";
+import { existsSync as existsSync17, mkdirSync as mkdirSync12, readFileSync as readFileSync12, writeFileSync as writeFileSync9 } from "node:fs";
 import { dirname as dirname5, join as join18 } from "node:path";
 
 class InputHistory {
@@ -8873,7 +9018,7 @@ class InputHistory {
     try {
       if (!existsSync17(this.filePath))
         return;
-      const content = readFileSync11(this.filePath, "utf-8");
+      const content = readFileSync12(this.filePath, "utf-8");
       this.entries = content.split(`
 `).map((line) => this.unescape(line)).filter(Boolean);
     } catch {}
@@ -8915,7 +9060,7 @@ import {
   existsSync as existsSync18,
   mkdirSync as mkdirSync13,
   readdirSync as readdirSync6,
-  readFileSync as readFileSync12,
+  readFileSync as readFileSync13,
   unlinkSync as unlinkSync3,
   writeFileSync as writeFileSync10
 } from "node:fs";
@@ -8958,7 +9103,7 @@ class SessionManager {
     const exactPath = this.getSessionPath(idOrPrefix);
     if (existsSync18(exactPath)) {
       try {
-        return JSON.parse(readFileSync12(exactPath, "utf-8"));
+        return JSON.parse(readFileSync13(exactPath, "utf-8"));
       } catch {
         return null;
       }
@@ -8966,7 +9111,7 @@ class SessionManager {
     const matches = files.filter((f) => basename3(f, ".json").startsWith(idOrPrefix));
     if (matches.length === 1) {
       try {
-        return JSON.parse(readFileSync12(matches[0], "utf-8"));
+        return JSON.parse(readFileSync13(matches[0], "utf-8"));
       } catch {
         return null;
       }
@@ -8991,7 +9136,7 @@ class SessionManager {
     const sessions = [];
     for (const file of files) {
       try {
-        const session = JSON.parse(readFileSync12(file, "utf-8"));
+        const session = JSON.parse(readFileSync13(file, "utf-8"));
         sessions.push({
           id: session.id,
           created: session.created,
@@ -9018,7 +9163,7 @@ class SessionManager {
     let pruned = 0;
     const withStats = files.map((f) => {
       try {
-        const session = JSON.parse(readFileSync12(f, "utf-8"));
+        const session = JSON.parse(readFileSync13(f, "utf-8"));
         return { path: f, updated: new Date(session.updated).getTime() };
       } catch {
         return { path: f, updated: 0 };
@@ -9072,7 +9217,7 @@ var init_session_manager = __esm(() => {
 // src/repl/voice.ts
 import { execSync as execSync11, spawn as spawn6 } from "node:child_process";
 import { existsSync as existsSync19, mkdirSync as mkdirSync14, unlinkSync as unlinkSync4 } from "node:fs";
-import { cpus, homedir as homedir4, platform, tmpdir as tmpdir4 } from "node:os";
+import { cpus, homedir as homedir4, platform, tmpdir as tmpdir5 } from "node:os";
 import { join as join20 } from "node:path";
 function getWhisperModelPath() {
   return join20(WHISPER_MODELS_DIR, `ggml-${WHISPER_MODEL}.bin`);
@@ -9284,7 +9429,7 @@ function ensureBuildDeps(pm) {
 }
 function buildWhisperFromSource(pm) {
   const out = process.stderr;
-  const buildDir = join20(tmpdir4(), `locus-whisper-build-${process.pid}`);
+  const buildDir = join20(tmpdir5(), `locus-whisper-build-${process.pid}`);
   if (!ensureBuildDeps(pm)) {
     out.write(`  ${red2("✗")} Could not install build tools (cmake, g++, git).
 `);
@@ -9442,7 +9587,7 @@ class VoiceController {
   onStateChange;
   constructor(options) {
     this.onStateChange = options.onStateChange;
-    this.tempFile = join20(tmpdir4(), `locus-voice-${process.pid}.wav`);
+    this.tempFile = join20(tmpdir5(), `locus-voice-${process.pid}.wav`);
     this.deps = checkDependencies();
   }
   getState() {
@@ -10384,7 +10529,8 @@ ${green("✓")} Issue #${issueNumber} completed ${dim2(`(${timer.formatted()})`)
 `);
   let prNumber;
   if (config.agent.autoPR && !options.skipPR) {
-    prNumber = await createIssuePR(projectRoot, config, issue);
+    const workDir = options.worktreePath ?? projectRoot;
+    prNumber = await createIssuePR(workDir, config, issue);
   }
   if (config.agent.autoLabel) {
     try {
@@ -10465,29 +10611,56 @@ ${aiResult.success ? green("✓") : red2("✗")} Iteration ${aiResult.success ?
     summary: extractSummary(aiResult.output)
   };
 }
-async function createIssuePR(projectRoot, config, issue) {
+async function createIssuePR(workDir, config, issue) {
   try {
+    try {
+      commitDirtySubmodules(workDir, issue.number, issue.title);
+      const status = execSync14("git status --porcelain", {
+        cwd: workDir,
+        encoding: "utf-8",
+        stdio: ["pipe", "pipe", "pipe"]
+      }).trim();
+      if (status) {
+        execSync14("git add -A", {
+          cwd: workDir,
+          encoding: "utf-8",
+          stdio: ["pipe", "pipe", "pipe"]
+        });
+        const message = `chore: complete #${issue.number} - ${issue.title}
+
+Co-Authored-By: LocusAgent <agent@locusai.team>`;
+        execSync14("git commit -F -", {
+          input: message,
+          cwd: workDir,
+          encoding: "utf-8",
+          stdio: ["pipe", "pipe", "pipe"]
+        });
+        process.stderr.write(`  ${dim2(`Committed uncommitted changes for #${issue.number}`)}
+`);
+      }
+    } catch {}
     const currentBranch = execSync14("git rev-parse --abbrev-ref HEAD", {
-      cwd: projectRoot,
+      cwd: workDir,
       encoding: "utf-8",
       stdio: ["pipe", "pipe", "pipe"]
     }).trim();
     const diff = execSync14(`git diff origin/${config.agent.baseBranch}..HEAD --stat`, {
-      cwd: projectRoot,
+      cwd: workDir,
       encoding: "utf-8",
       stdio: ["pipe", "pipe", "pipe"]
     }).trim();
     if (!diff) {
-      getLogger().verbose("No changes to create PR for");
+      process.stderr.write(`  ${yellow2("⚠")} No changes detected — skipping PR creation
+`);
       return;
     }
-    pushSubmoduleBranches(projectRoot);
+    pushSubmoduleBranches(workDir);
     execSync14(`git push -u origin ${currentBranch}`, {
-      cwd: projectRoot,
+      cwd: workDir,
       encoding: "utf-8",
       stdio: ["pipe", "pipe", "pipe"]
     });
-    const submoduleSummary = getSubmoduleChangeSummary(projectRoot, config.agent.baseBranch);
+    const submoduleSummary = getSubmoduleChangeSummary(workDir, config.agent.baseBranch);
     let prBody = `Closes #${issue.number}`;
     if (submoduleSummary) {
       prBody += `
@@ -10502,7 +10675,7 @@ ${submoduleSummary}`;
     const prTitle = `${issue.title} (#${issue.number})`;
     let prNumber;
     try {
-      const existing = execSync14(`gh pr list --head ${currentBranch} --base ${config.agent.baseBranch} --json number --limit 1`, { cwd: projectRoot, encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] }).trim();
+      const existing = execSync14(`gh pr list --head ${currentBranch} --base ${config.agent.baseBranch} --json number --limit 1`, { cwd: workDir, encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] }).trim();
       const parsed = JSON.parse(existing);
       if (Array.isArray(parsed) && parsed.length > 0) {
         prNumber = parsed[0].number;
@@ -10512,7 +10685,7 @@ ${submoduleSummary}`;
       try {
         execSync14(`gh pr edit ${prNumber} --title ${JSON.stringify(prTitle)} --body-file -`, {
           input: prBody,
-          cwd: projectRoot,
+          cwd: workDir,
           encoding: "utf-8",
           stdio: ["pipe", "pipe", "pipe"]
         });
@@ -10522,13 +10695,15 @@ ${submoduleSummary}`;
         getLogger().warn(`Failed to update PR #${prNumber}: ${editErr}`);
       }
     } else {
-      prNumber = createPR(prTitle, prBody, currentBranch, config.agent.baseBranch, { cwd: projectRoot });
+      prNumber = createPR(prTitle, prBody, currentBranch, config.agent.baseBranch, { cwd: workDir });
       process.stderr.write(`  ${green("✓")} Created PR #${prNumber}
 `);
     }
     return prNumber;
   } catch (e) {
-    getLogger().warn(`Failed to create PR: ${e}`);
+    const errorMsg = e instanceof Error ? e.message : String(e);
+    process.stderr.write(`  ${red2("✗")} Failed to create PR: ${errorMsg}
+`);
     return;
   }
 }
@@ -10693,7 +10868,7 @@ var init_conflict = __esm(() => {
 import {
   existsSync as existsSync21,
   mkdirSync as mkdirSync15,
-  readFileSync as readFileSync13,
+  readFileSync as readFileSync14,
   unlinkSync as unlinkSync5,
   writeFileSync as writeFileSync11
 } from "node:fs";
@@ -10716,7 +10891,7 @@ function loadRunState(projectRoot, sprintName) {
   if (!existsSync21(path))
     return null;
   try {
-    return JSON.parse(readFileSync13(path, "utf-8"));
+    return JSON.parse(readFileSync14(path, "utf-8"));
   } catch {
     getLogger().warn("Corrupted run state file, ignoring");
     return null;
@@ -11508,7 +11683,7 @@ async function handleSingleIssue(projectRoot, config, issueNumber, flags, sandbo
 ${bold2("Running sprint issue")} ${cyan2(`#${issueNumber}`)} ${dim2("(sequential)")}
 
 `);
-    await executeIssue(projectRoot, {
+    const result2 = await executeIssue(projectRoot, {
       issueNumber,
       provider: execution.provider,
       model: execution.model,
@@ -11517,6 +11692,11 @@ ${bold2("Running sprint issue")} ${cyan2(`#${issueNumber}`)} ${dim2("(sequential
       sandboxName: execution.sandboxName,
       containerWorkdir: execution.containerWorkdir
     });
+    if (!result2.success) {
+      process.stderr.write(`
+${red2("✗")} Issue #${issueNumber} execution failed${result2.error ? `: ${result2.error}` : ""}
+`);
+    }
     return;
   }
   const randomSuffix = Math.random().toString(36).slice(2, 8);
@@ -11559,6 +11739,15 @@ ${bold2("Running issue")} ${cyan2(`#${issueNumber}`)} ${dim2(`(branch: ${branchN
         });
         log.info(`Checked out ${config.agent.baseBranch}`);
       } catch {}
+      if (result.prNumber) {
+        process.stderr.write(`
+${green("✓")} Issue #${issueNumber} done — PR #${result.prNumber} opened
+`);
+      } else if (config.agent.autoPR) {
+        process.stderr.write(`
+${yellow2("⚠")} Issue #${issueNumber} done — no PR was created (check errors above)
+`);
+      }
     } else {
       process.stderr.write(`  ${yellow2("⚠")} Branch ${dim2(branchName)} preserved for debugging.
 `);
@@ -12302,7 +12491,7 @@ import {
   existsSync as existsSync25,
   mkdirSync as mkdirSync17,
   readdirSync as readdirSync8,
-  readFileSync as readFileSync14,
+  readFileSync as readFileSync15,
   writeFileSync as writeFileSync12
 } from "node:fs";
 import { join as join26 } from "node:path";
@@ -12358,7 +12547,7 @@ function loadPlanFile(projectRoot, id) {
   if (!match)
     return null;
   try {
-    const content = readFileSync14(join26(dir, match), "utf-8");
+    const content = readFileSync15(join26(dir, match), "utf-8");
     return JSON.parse(content);
   } catch {
     return null;
@@ -12443,7 +12632,7 @@ ${bold2("Saved Plans:")}
   for (const file of files) {
     const id = file.replace(".json", "");
     try {
-      const content = readFileSync14(join26(dir, file), "utf-8");
+      const content = readFileSync15(join26(dir, file), "utf-8");
       const plan = JSON.parse(content);
       const date = plan.createdAt ? plan.createdAt.slice(0, 10) : "";
       const issueCount = Array.isArray(plan.issues) ? plan.issues.length : 0;
@@ -12558,7 +12747,7 @@ ${yellow2("⚠")} Plan file was not found at ${bold2(planPathRelative)}.
   }
   let updatedPlan;
   try {
-    const content = readFileSync14(planPath, "utf-8");
+    const content = readFileSync15(planPath, "utf-8");
     updatedPlan = JSON.parse(content);
   } catch {
     process.stderr.write(`
@@ -12701,7 +12890,7 @@ ${yellow2("⚠")} Plan file was not created at ${bold2(planPathRelative)}.
   }
   let plan;
   try {
-    const content = readFileSync14(planPath, "utf-8");
+    const content = readFileSync15(planPath, "utf-8");
     plan = JSON.parse(content);
   } catch {
     process.stderr.write(`
@@ -12875,14 +13064,14 @@ ${directive}${sprintName ? `
 </directive>`);
   const locusPath = join26(projectRoot, ".locus", "LOCUS.md");
   if (existsSync25(locusPath)) {
-    const content = readFileSync14(locusPath, "utf-8");
+    const content = readFileSync15(locusPath, "utf-8");
     parts.push(`<project-context>
 ${content.slice(0, 3000)}
 </project-context>`);
   }
   const learningsPath = join26(projectRoot, ".locus", "LEARNINGS.md");
   if (existsSync25(learningsPath)) {
-    const content = readFileSync14(learningsPath, "utf-8");
+    const content = readFileSync15(learningsPath, "utf-8");
     parts.push(`<past-learnings>
 ${content.slice(0, 2000)}
 </past-learnings>`);
@@ -12936,7 +13125,7 @@ ${feedback}
 </feedback>`);
   const locusPath = join26(projectRoot, ".locus", "LOCUS.md");
   if (existsSync25(locusPath)) {
-    const content = readFileSync14(locusPath, "utf-8");
+    const content = readFileSync15(locusPath, "utf-8");
     parts.push(`<project-context>
 ${content.slice(0, 3000)}
 </project-context>`);
@@ -13118,7 +13307,7 @@ __export(exports_review, {
   reviewCommand: () => reviewCommand
 });
 import { execFileSync as execFileSync2, execSync as execSync19 } from "node:child_process";
-import { existsSync as existsSync26, readFileSync as readFileSync15 } from "node:fs";
+import { existsSync as existsSync26, readFileSync as readFileSync16 } from "node:fs";
 import { join as join27 } from "node:path";
 function printHelp3() {
   process.stderr.write(`
@@ -13290,7 +13479,7 @@ You are an expert code reviewer for the ${config.github.owner}/${config.github.r
 </role>`);
   const locusPath = join27(projectRoot, ".locus", "LOCUS.md");
   if (existsSync26(locusPath)) {
-    const content = readFileSync15(locusPath, "utf-8");
+    const content = readFileSync16(locusPath, "utf-8");
     parts.push(`<project-context>
 ${content.slice(0, 2000)}
 </project-context>`);
@@ -13608,7 +13797,7 @@ import {
   existsSync as existsSync27,
   mkdirSync as mkdirSync18,
   readdirSync as readdirSync9,
-  readFileSync as readFileSync16,
+  readFileSync as readFileSync17,
   unlinkSync as unlinkSync6,
   writeFileSync as writeFileSync13
 } from "node:fs";
@@ -13690,7 +13879,7 @@ ${bold2("Discussions:")}
 `);
   for (const file of files) {
     const id = file.replace(".md", "");
-    const content = readFileSync16(join28(dir, file), "utf-8");
+    const content = readFileSync17(join28(dir, file), "utf-8");
     const titleMatch = content.match(/^#\s+(.+)/m);
     const title = titleMatch ? titleMatch[1] : id;
     const dateMatch = content.match(/\*\*Date:\*\*\s*(.+)/);
@@ -13720,7 +13909,7 @@ function showDiscussion(projectRoot, id) {
 `);
     return;
   }
-  const content = readFileSync16(join28(dir, match), "utf-8");
+  const content = readFileSync17(join28(dir, match), "utf-8");
   process.stdout.write(`${content}
 `);
 }
@@ -13768,7 +13957,7 @@ async function convertDiscussionToPlan(projectRoot, id) {
 `);
     return;
   }
-  const content = readFileSync16(join28(dir, match), "utf-8");
+  const content = readFileSync17(join28(dir, match), "utf-8");
   const titleMatch = content.match(/^#\s+(.+)/m);
   const discussionTitle = titleMatch ? titleMatch[1].trim() : id;
   await planCommand(projectRoot, [
@@ -13911,14 +14100,14 @@ You are a senior software architect and consultant for the ${config.github.owner
 </role>`);
   const locusPath = join28(projectRoot, ".locus", "LOCUS.md");
   if (existsSync27(locusPath)) {
-    const content = readFileSync16(locusPath, "utf-8");
+    const content = readFileSync17(locusPath, "utf-8");
     parts.push(`<project-context>
 ${content.slice(0, 3000)}
 </project-context>`);
   }
   const learningsPath = join28(projectRoot, ".locus", "LEARNINGS.md");
   if (existsSync27(learningsPath)) {
-    const content = readFileSync16(learningsPath, "utf-8");
+    const content = readFileSync17(learningsPath, "utf-8");
     parts.push(`<past-learnings>
 ${content.slice(0, 2000)}
 </past-learnings>`);
@@ -13989,7 +14178,7 @@ __export(exports_artifacts, {
   formatDate: () => formatDate2,
   artifactsCommand: () => artifactsCommand
 });
-import { existsSync as existsSync28, readdirSync as readdirSync10, readFileSync as readFileSync17, statSync as statSync5 } from "node:fs";
+import { existsSync as existsSync28, readdirSync as readdirSync10, readFileSync as readFileSync18, statSync as statSync5 } from "node:fs";
 import { join as join29 } from "node:path";
 function printHelp6() {
   process.stderr.write(`
@@ -14035,7 +14224,7 @@ function readArtifact(projectRoot, name) {
     return null;
   const stat = statSync5(filePath);
   return {
-    content: readFileSync17(filePath, "utf-8"),
+    content: readFileSync18(filePath, "utf-8"),
     info: {
       name: fileName.replace(/\.md$/, ""),
       fileName,
@@ -14392,7 +14581,7 @@ __export(exports_sandbox2, {
 });
 import { execSync as execSync22, spawn as spawn7 } from "node:child_process";
 import { createHash as createHash2 } from "node:crypto";
-import { existsSync as existsSync29, readFileSync as readFileSync18 } from "node:fs";
+import { existsSync as existsSync29, readFileSync as readFileSync19 } from "node:fs";
 import { basename as basename4, join as join30 } from "node:path";
 import { createInterface as createInterface3 } from "node:readline";
 function printSandboxHelp() {
@@ -14923,7 +15112,7 @@ async function handleLogs(projectRoot, args) {
 }
 function detectPackageManager2(projectRoot) {
   try {
-    const raw = readFileSync18(join30(projectRoot, "package.json"), "utf-8");
+    const raw = readFileSync19(join30(projectRoot, "package.json"), "utf-8");
     const pkgJson = JSON.parse(raw);
     if (typeof pkgJson.packageManager === "string") {
       const name = pkgJson.packageManager.split("@")[0];
@@ -15216,7 +15405,7 @@ init_context();
 init_logger();
 init_rate_limiter();
 init_terminal();
-import { existsSync as existsSync30, readFileSync as readFileSync19 } from "node:fs";
+import { existsSync as existsSync30, readFileSync as readFileSync20 } from "node:fs";
 import { join as join31 } from "node:path";
 import { fileURLToPath } from "node:url";
 function getCliVersion() {
@@ -15226,7 +15415,7 @@ function getCliVersion() {
     return fallbackVersion;
   }
   try {
-    const parsed = JSON.parse(readFileSync19(packageJsonPath, "utf-8"));
+    const parsed = JSON.parse(readFileSync20(packageJsonPath, "utf-8"));
     return parsed.version ?? fallbackVersion;
   } catch {
     return fallbackVersion;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@locusai/cli",
-  "version": "0.25.2",
+  "version": "0.25.4",
   "description": "GitHub-native AI engineering assistant",
   "type": "module",
   "bin": {
@@ -36,7 +36,7 @@
   "license": "MIT",
   "dependencies": {},
   "devDependencies": {
-    "@locusai/sdk": "^0.25.2",
+    "@locusai/sdk": "^0.25.4",
     "@types/bun": "latest",
     "typescript": "^5.8.3"
   },