@neriros/ralphy 3.10.0 → 3.10.1

package/dist/mcp/index.js

@@ -6615,20 +6615,33 @@ import {
   writeFileSync,
   existsSync,
   unlinkSync,
+  renameSync,
   mkdirSync,
   readdirSync
 } from "fs";
 import { dirname } from "path";
 
 class FileSystemProvider {
+  writeSeq = 0;
   read(path) {
     if (!existsSync(path))
       return null;
     return readFileSync(path, "utf-8");
   }
   write(path, content) {
-    mkdirSync(dirname(path), { recursive: true });
-    writeFileSync(path, content, "utf-8");
+    const dir = dirname(path);
+    mkdirSync(dir, { recursive: true });
+    const tmp = `${path}.tmp-${process.pid}-${this.writeSeq++}`;
+    try {
+      writeFileSync(tmp, content, "utf-8");
+      renameSync(tmp, path);
+    } catch (err) {
+      try {
+        if (existsSync(tmp))
+          unlinkSync(tmp);
+      } catch {}
+      throw err;
+    }
   }
   remove(path) {
     if (!existsSync(path))

package/dist/shell/index.js

@@ -18928,8 +18928,8 @@ import { readFileSync } from "fs";
 import { resolve } from "path";
 function getVersion() {
   try {
-    if ("3.10.0")
-      return "3.10.0";
+    if ("3.10.1")
+      return "3.10.1";
   } catch {}
   const dirsToTry = [];
   try {
@@ -81370,8 +81370,7 @@ var init_fields = __esm(() => {
       id: "linear.syncSpecsAsAttachments",
       label: "Upload proposal.md / design.md as attachments?",
       description: "Upload the OpenSpec planning docs (proposal.md, design.md) to the issue as attachments. OpenSpec is Ralphy's spec-driven planning format.",
-      spec: yes(),
-      when: isOn("linear.syncTasksToComment")
+      spec: yes()
     },
     {
       id: "linear.specAttachmentFormats",
@@ -82780,6 +82779,14 @@ function buildFromAnswers(mode, answers, build = buildWorkflowMarkdown) {
     else
       delete values2["linear.indicators"];
   }
+  if (values2["linear.confirmationMode.enabled"] === true && values2["linear.indicators"] && typeof values2["linear.indicators"] === "object") {
+    const map3 = { ...values2["linear.indicators"] };
+    if (!("getApproved" in map3)) {
+      map3.getApproved = { filter: [{ type: "label", value: "approved" }] };
+      map3.clearApproved = { type: "label", value: "approved" };
+      values2["linear.indicators"] = map3;
+    }
+  }
   let bodyOverride;
   if (PROMPT_BODY_FIELD_ID in values2) {
     const body = values2[PROMPT_BODY_FIELD_ID];
@@ -83785,12 +83792,6 @@ var init_SetupWizard = __esm(async () => {
     }
   ];
   CONFIRMATION_STATES = [
-    {
-      key: "confirmGate",
-      label: "Confirmation gate",
-      description: "Opt-in: only require human approval for issues that match this.",
-      slots: ["getConfirmGate"]
-    },
     {
       key: "autoApprove",
       label: "Auto-approve",
@@ -84079,20 +84080,33 @@ import {
   writeFileSync,
   existsSync as existsSync2,
   unlinkSync,
+  renameSync,
   mkdirSync,
   readdirSync
 } from "fs";
 import { dirname as dirname2 } from "path";
 
 class FileSystemProvider {
+  writeSeq = 0;
   read(path) {
     if (!existsSync2(path))
       return null;
     return readFileSync3(path, "utf-8");
   }
   write(path, content) {
-    mkdirSync(dirname2(path), { recursive: true });
-    writeFileSync(path, content, "utf-8");
+    const dir = dirname2(path);
+    mkdirSync(dir, { recursive: true });
+    const tmp = `${path}.tmp-${process.pid}-${this.writeSeq++}`;
+    try {
+      writeFileSync(tmp, content, "utf-8");
+      renameSync(tmp, path);
+    } catch (err) {
+      try {
+        if (existsSync2(tmp))
+          unlinkSync(tmp);
+      } catch {}
+      throw err;
+    }
   }
   remove(path) {
     if (!existsSync2(path))
@@ -99807,7 +99821,9 @@ function useLoop(opts) {
         startingStatus: currentState.status
       });
       while (!cancelled) {
-        currentState = readState(stateDir);
+        const { state: polled } = tryReadStateRaw(stateDir);
+        if (polled !== null)
+          currentState = polled;
         setState(currentState);
         if (!actor.getSnapshot().matches("running"))
           break;
@@ -101475,7 +101491,16 @@ function branchForChange(changeName) {
 function worktreeDirNameForIssue(issue2) {
   return issue2.identifier.toLowerCase();
 }
-async function createWorktree(projectRoot, changeName, baseBranch, runner) {
+function withRepoLock(projectRoot, fn) {
+  const prev = repoWorktreeLocks.get(projectRoot) ?? Promise.resolve();
+  const result2 = prev.then(fn, fn);
+  repoWorktreeLocks.set(projectRoot, result2.then(() => {}, () => {}));
+  return result2;
+}
+function createWorktree(projectRoot, changeName, baseBranch, runner) {
+  return withRepoLock(projectRoot, () => provisionWorktree(projectRoot, changeName, baseBranch, runner));
+}
+async function provisionWorktree(projectRoot, changeName, baseBranch, runner) {
   const dir = worktreesDir2(projectRoot);
   const cwd2 = join20(dir, changeName);
   const branch = branchForChange(changeName);
@@ -101570,7 +101595,7 @@ async function seedWorktreeMcpConfig(projectRoot, worktreeCwd) {
   await Bun.write(dst, JSON.stringify(parsed, null, 2) + `
 `);
 }
-var PRE_PUSH_HOOK_SCRIPT = `#!/usr/bin/env bash
+var repoWorktreeLocks, PRE_PUSH_HOOK_SCRIPT = `#!/usr/bin/env bash
 # Installed by ralphy createWorktree (RLF-107).
 # Rejects any push whose remote ref is not refs/heads/ralph/*,
 # and rejects force pushes unless RALPH_ALLOW_FORCE_PUSH=1.
@@ -101590,7 +101615,9 @@ while read local_ref local_sha remote_ref remote_sha; do
 done
 exit 0
 `;
-var init_worktree = () => {};
+var init_worktree = __esm(() => {
+  repoWorktreeLocks = new Map;
+});
 
 // apps/agent/src/shared/capabilities/git.ts
 var createWorktree2, removeWorktree2, seedWorktreeMcpConfig2, git;
@@ -105811,7 +105838,10 @@ var init_runners = __esm(() => {
       const stderr = await new Response(proc.stderr).text();
       const code = await proc.exited;
       if (code !== 0) {
-        const err = new Error("git command failed");
+        const firstStderrLine = stderr.trim().split(`
+`)[0] ?? "";
+        const summary = firstStderrLine ? `: ${firstStderrLine}` : "";
+        const err = new Error(`git \`${args.join(" ")}\` failed${summary}`);
         err.stderr = stderr;
         err.code = code;
         throw err;
@@ -107542,7 +107572,7 @@ var init_linear_sync = __esm(() => {
 
 // apps/agent/src/agent/linear-sync/comment-sync.ts
 import { dirname as dirname12, join as join30 } from "path";
-import { mkdir as mkdir11 } from "fs/promises";
+import { mkdir as mkdir11, rename, unlink as unlink2 } from "fs/promises";
 async function readStateJson(statePath) {
   const file2 = Bun.file(statePath);
   if (!await file2.exists())
@@ -107555,8 +107585,15 @@ async function readStateJson(statePath) {
 }
 async function writeStateJson(statePath, state) {
   await mkdir11(dirname12(statePath), { recursive: true });
-  await Bun.write(statePath, JSON.stringify(state, null, 2) + `
+  const tmp = `${statePath}.tmp-${process.pid}-${writeStateSeq++}`;
+  try {
+    await Bun.write(tmp, JSON.stringify(state, null, 2) + `
 `);
+    await rename(tmp, statePath);
+  } catch (err) {
+    await unlink2(tmp).catch(() => {});
+    throw err;
+  }
 }
 function readComments(state) {
   const raw = state?.linearComments ?? {};
@@ -107763,7 +107800,7 @@ ${deps.message.trim()}`;
     iteration: deps.iteration
   });
 }
-var PLAN_COMMENT_TITLE = "\uD83D\uDCCB Ralph plan", STEERING_COMMENT_TITLE = "\uD83E\uDDED Ralph steering";
+var PLAN_COMMENT_TITLE = "\uD83D\uDCCB Ralph plan", STEERING_COMMENT_TITLE = "\uD83E\uDDED Ralph steering", writeStateSeq = 0;
 var init_comment_sync = __esm(() => {
   init_linear_sync();
 });
@@ -260251,7 +260288,9 @@ var init_spec_attachments = __esm(() => {
 // apps/agent/src/agent/wire/comment-sync.ts
 function createCommentSyncHooks(input) {
   const { apiKey, cfg, projectRoot, onLog, diag, cwdByChange, issueByChange } = input;
-  const enabled2 = Boolean(cfg.linear.syncTasksToComment && apiKey);
+  const commentsEnabled = Boolean(cfg.linear.syncTasksToComment && apiKey);
+  const specAttachmentsEnabled = Boolean(cfg.linear.syncSpecsAsAttachments && apiKey);
+  const enabled2 = commentsEnabled || specAttachmentsEnabled;
   if (!enabled2)
     return { enabled: false };
   const commentMutations = {
@@ -260265,7 +260304,6 @@ function createCommentSyncHooks(input) {
     deleteAttachment,
     findIssueAttachmentByTitle
   };
-  const specAttachmentsEnabled = Boolean(cfg.linear.syncSpecsAsAttachments);
   return {
     enabled: enabled2,
     syncTasks: async (worker, iteration) => {
@@ -260273,27 +260311,29 @@ function createCommentSyncHooks(input) {
       const layout = projectLayout(root);
       const changeDir = layout.changeDir(worker.changeName);
       const statePath = layout.stateFile(worker.changeName);
-      if (!specAttachmentsEnabled) {
-        await postPlanCommentOnce({
+      if (commentsEnabled) {
+        if (!specAttachmentsEnabled) {
+          await postPlanCommentOnce({
+            apiKey,
+            issueId: worker.issueId,
+            statePath,
+            changeDir,
+            changeName: worker.changeName,
+            log: onLog,
+            mutations: commentMutations
+          });
+        }
+        await postOrUpdateTasksComment({
           apiKey,
           issueId: worker.issueId,
           statePath,
           changeDir,
           changeName: worker.changeName,
+          iteration,
           log: onLog,
           mutations: commentMutations
         });
       }
-      await postOrUpdateTasksComment({
-        apiKey,
-        issueId: worker.issueId,
-        statePath,
-        changeDir,
-        changeName: worker.changeName,
-        iteration,
-        log: onLog,
-        mutations: commentMutations
-      });
       if (specAttachmentsEnabled) {
         await syncSpecAttachments({
           apiKey,
@@ -260307,37 +260347,39 @@ function createCommentSyncHooks(input) {
         });
       }
     },
-    onSteeringAppended: async (changeName, message) => {
-      const root = cwdByChange.get(changeName) ?? projectRoot;
-      const layout = projectLayout(root);
-      const changeDir = layout.changeDir(changeName);
-      const statePath = layout.stateFile(changeName);
-      const issue2 = issueByChange.get(changeName) ?? null;
-      const issueId = issue2?.id ?? null;
-      if (!issueId) {
-        diag("comment-sync", `  comment-sync: no Linear issue cached for ${changeName}; skipping steering refresh`, "gray");
-        return;
-      }
-      let iteration = 0;
-      try {
-        const f2 = Bun.file(statePath);
-        if (await f2.exists()) {
-          const json2 = await f2.json();
-          iteration = json2.iteration ?? 0;
+    ...commentsEnabled ? {
+      onSteeringAppended: async (changeName, message) => {
+        const root = cwdByChange.get(changeName) ?? projectRoot;
+        const layout = projectLayout(root);
+        const changeDir = layout.changeDir(changeName);
+        const statePath = layout.stateFile(changeName);
+        const issue2 = issueByChange.get(changeName) ?? null;
+        const issueId = issue2?.id ?? null;
+        if (!issueId) {
+          diag("comment-sync", `  comment-sync: no Linear issue cached for ${changeName}; skipping steering refresh`, "gray");
+          return;
         }
-      } catch {}
-      await postSteeringAndRefreshTasks({
-        apiKey,
-        issueId,
-        statePath,
-        changeDir,
-        changeName,
-        iteration,
-        message,
-        log: onLog,
-        mutations: commentMutations
-      });
-    }
+        let iteration = 0;
+        try {
+          const f2 = Bun.file(statePath);
+          if (await f2.exists()) {
+            const json2 = await f2.json();
+            iteration = json2.iteration ?? 0;
+          }
+        } catch {}
+        await postSteeringAndRefreshTasks({
+          apiKey,
+          issueId,
+          statePath,
+          changeDir,
+          changeName,
+          iteration,
+          message,
+          log: onLog,
+          mutations: commentMutations
+        });
+      }
+    } : {}
   };
 }
 var init_comment_sync2 = __esm(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neriros/ralphy",
-  "version": "3.10.0",
+  "version": "3.10.1",
   "description": "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   "keywords": [
     "agent",