lody 0.71.2 → 0.71.4

package/dist/chunks/{review-viewer-CP4bkTWs.js → review-viewer-B9Tm-8O4.js}

@@ -3,7 +3,7 @@ import { readFile, mkdir, writeFile, rename } from "node:fs/promises";
 import os__default from "node:os";
 import path__default from "node:path";
 import process from "node:process";
-const reviewViewerVersion = "0.71.2";
+const reviewViewerVersion = "0.71.4";
 const reviewViewerSha256 = "e395b367a7644fe83bc050d0c646810914a2844824fd2b5d11d24ae4f55b6893";
 const reviewViewerFileName = "standalone.html";
 const DEFAULT_CDN_BASES = ["https://cdn.jsdelivr.net/npm", "https://unpkg.com"];

package/dist/claude-acp.js

@@ -34404,7 +34404,12 @@ ${content.text}
         sessions
       };
     }
+    static TITLE_POLL_INTERVAL_MS = 2e3;
     async maybeUpdateSessionTitle(sessionId, session) {
+      if (session.titleLookupInFlight) {
+        return;
+      }
+      session.titleLookupInFlight = true;
       let info;
       try {
         info = await mGe(sessionId, {
@@ -34413,8 +34418,10 @@ ${content.text}
       } catch (error) {
         this.logger.error(`Session ${sessionId}: failed to read session info: ${error}`);
         return;
+      } finally {
+        session.titleLookupInFlight = false;
       }
-      const rawTitle = info?.customTitle ?? info?.summary;
+      const rawTitle = info?.customTitle;
       if (!rawTitle) {
         return;
       }
@@ -34423,6 +34430,7 @@ ${content.text}
         return;
       }
       session.lastTitle = title;
+      this.stopSessionTitlePolling(session);
       await this.client.sessionUpdate({
         sessionId,
         update: {
@@ -34432,6 +34440,22 @@ ${content.text}
         }
       });
     }
+    startSessionTitlePolling(sessionId, session) {
+      if (session.lastTitle || session.titlePollTimer) {
+        return;
+      }
+      void this.maybeUpdateSessionTitle(sessionId, session);
+      session.titlePollTimer = setInterval(() => {
+        void this.maybeUpdateSessionTitle(sessionId, session);
+      }, ClaudeAcpAgent.TITLE_POLL_INTERVAL_MS);
+      session.titlePollTimer.unref();
+    }
+    stopSessionTitlePolling(session) {
+      if (session.titlePollTimer) {
+        clearInterval(session.titlePollTimer);
+        session.titlePollTimer = void 0;
+      }
+    }
     async authenticate(_params) {
       if (_params.methodId === "gateway" || _params.methodId === "gateway-bedrock") {
         this.gatewayAuthRequest = _params;
@@ -34971,7 +34995,9 @@ Compacting failed${reason}`
                 }
                 case "session_state_changed": {
                   session.lastSessionState = message.state;
-                  if (message.state === "idle") {
+                  if (message.state === "running") {
+                    this.startSessionTitlePolling(params.sessionId, session);
+                  } else if (message.state === "idle") {
                     if (session.cancelled && session.activeTurn && !session.activeTurn.settled) {
                       settleActive({
                         stopReason: "cancelled",
@@ -34989,6 +35015,7 @@ Compacting failed${reason}`
                       this.logger.error(`Session ${params.sessionId}: SDK went idle without emitting a result for the active turn; failing the in-flight prompt (issue #825)`);
                       failActive(RequestError.internalError(errorKindData("no_result"), TURN_NO_RESULT_MESSAGE));
                     }
+                    this.stopSessionTitlePolling(session);
                     await this.maybeUpdateSessionTitle(params.sessionId, session);
                   }
                   break;
@@ -35715,6 +35742,7 @@ ${message.api_refusal_explanation}` : "";
       }
       session.queryClosed = true;
       session.consumer = void 0;
+      this.stopSessionTitlePolling(session);
       session.settingsManager.dispose();
       session.input.end();
       session.query.close();

package/dist/index.js

@@ -4190,7 +4190,7 @@ let __tla = Promise.all([
     }
   }
   const name$1 = "lody";
-  const version$7 = "0.71.2";
+  const version$7 = "0.71.4";
   const description$1 = "Lody Agent CLI tool for managing remote command execution";
   const type$2 = "module";
   const main$4 = "dist/index.js";
@@ -4204,7 +4204,7 @@ let __tla = Promise.all([
     "build": "pnpm run clean && pnpm run prepare:acp-adapters && pnpm run typecheck && pnpm run build:bundle && pnpm run check:published-bundle-imports && pnpm run copy:wasm",
     "build:watch": "pnpm run prepare:review-assets && vite build --watch",
     "build:bundle": "pnpm run prepare:review-assets && vite build",
-    "prepare:acp-adapters": "pnpm --dir ../.. --filter acp-extension-claude build && pnpm --dir ../.. --filter acp-extension-codex build",
+    "prepare:acp-adapters": "corepack pnpm --dir ../.. --filter acp-extension-claude build && corepack pnpm --dir ../.. --filter acp-extension-codex build",
     "prepare:review-assets": "pnpm --dir ../.. --filter lody-code-review-viewer build",
     "check:published-bundle-imports": "node scripts/check-published-bundle-imports.js",
     "copy:wasm": "node scripts/copy-loro-wasm.js",
@@ -4516,7 +4516,7 @@ let __tla = Promise.all([
       displayName: "Codex"
     }
   ];
-  const usesAcpProvidedSessionTitle = (cliType, agentType) => cliType === "builtin" && (agentType === "claude" || agentType === "codex");
+  const usesAcpProvidedSessionTitle = (cliType, agentType) => cliType === "builtin" && agentType === "claude";
   const isBuiltinRuntimeOverrides = (value) => {
     if (!value || typeof value !== "object" || Array.isArray(value)) {
       return false;
@@ -103186,8 +103186,13 @@ ${value}`;
     return proxyFromEnvExports.getProxyForUrl(targetUrl);
   };
   function resolveProxyUrl(targetUrl) {
-    const proxyUrl = getProxyForUrlWithWsFallback(targetUrl);
+    const rawProxyUrl = getProxyForUrlWithWsFallback(targetUrl);
+    if (!rawProxyUrl) {
+      return {};
+    }
+    const proxyUrl = normalizeProxyUrl(rawProxyUrl);
     if (!proxyUrl) {
+      warnUnsupportedProxyUrl(rawProxyUrl);
       return {};
     }
     return {
@@ -103195,6 +103200,33 @@ ${value}`;
       proxyUrlRedacted: redactProxyUrl(proxyUrl)
     };
   }
+  const SCHEME_PREFIX_RE = /^[a-z][a-z0-9+.-]*:\/\//i;
+  const warnedProxyUrls = /* @__PURE__ */ new Set();
+  function normalizeProxyUrl(rawProxyUrl) {
+    const trimmed2 = rawProxyUrl.trim();
+    if (!trimmed2) {
+      return void 0;
+    }
+    const candidate = SCHEME_PREFIX_RE.test(trimmed2) ? trimmed2 : `http://${trimmed2}`;
+    let parsed;
+    try {
+      parsed = new URL(candidate);
+    } catch {
+      return void 0;
+    }
+    if (parsed.protocol !== "http:" && parsed.protocol !== "https:") {
+      return void 0;
+    }
+    return candidate;
+  }
+  function warnUnsupportedProxyUrl(rawProxyUrl) {
+    const redacted = redactProxyUrl(rawProxyUrl);
+    if (warnedProxyUrls.has(redacted)) {
+      return;
+    }
+    warnedProxyUrls.add(redacted);
+    getLogger("proxy").warn(`[proxy] ignoring unsupported proxy URL ${redacted}: only http:// and https:// proxies are supported (e.g. socks proxies are not); using a direct connection instead`);
+  }
   const DEFAULT_CONNECT_TIMEOUT_MS = 15e3;
   const DEFAULT_MAX_CONCURRENT_STREAMS = 100;
   const DISABLED_TRANSPORT_VALUES = /* @__PURE__ */ new Set([
@@ -106445,6 +106477,7 @@ ${value}`;
   const JSON_RPC_VERSION$1 = "2.0";
   const LORO_STREAMS_RPC_VERSION = "1";
   const LORO_STREAMS_RPC_RETENTION_SECONDS = 86400;
+  const DEFAULT_LIVE_IDLE_TIMEOUT_MS = 12e4;
   const LORO_STREAMS_RPC_ERROR_CODES = {
     rpcVersionMismatch: "rpc_version_mismatch",
     methodUnavailable: "method_unavailable",
@@ -106970,57 +107003,101 @@ ${value}`;
       },
       readJsonLive: async (streamId, state2, onBatch, liveOptions) => {
         const { client: streamClient, context: context2 } = getStreamClient(streamId, "live");
-        for await (const event of streamClient.live({
-          offset: state2.nextOffset ?? "now",
-          mode: options.liveMode ?? "sse",
-          signal: liveOptions?.signal
-        })) {
-          switch (event.type) {
-            case "data":
-              state2.nextOffset = event.nextOffset;
-              state2.cursor = event.cursor;
-              await onBatch({
-                messages: parseJsonLivePayload(event.payload, streamId, "live read"),
-                nextOffset: event.nextOffset,
-                cursor: event.cursor,
-                upToDate: false
-              });
-              break;
-            case "up_to_date": {
-              state2.nextOffset = event.nextOffset;
-              state2.cursor = event.cursor;
-              await onBatch({
-                messages: [],
-                nextOffset: event.nextOffset,
-                cursor: event.cursor,
-                upToDate: true
-              });
-              break;
-            }
-            case "eof": {
-              state2.nextOffset = event.nextOffset;
-              state2.cursor = void 0;
-              await onBatch({
-                messages: [],
-                nextOffset: event.nextOffset,
-                cursor: void 0,
-                upToDate: true
-              });
-              return;
+        const watchdog = createLiveIdleWatchdog(options.liveIdleTimeoutMs ?? DEFAULT_LIVE_IDLE_TIMEOUT_MS, liveOptions?.signal);
+        try {
+          for await (const event of streamClient.live({
+            offset: state2.nextOffset ?? "now",
+            mode: options.liveMode ?? "sse",
+            signal: watchdog.signal
+          })) {
+            watchdog.reset();
+            switch (event.type) {
+              case "data":
+                state2.nextOffset = event.nextOffset;
+                state2.cursor = event.cursor;
+                await onBatch({
+                  messages: parseJsonLivePayload(event.payload, streamId, "live read"),
+                  nextOffset: event.nextOffset,
+                  cursor: event.cursor,
+                  upToDate: false
+                });
+                break;
+              case "up_to_date": {
+                state2.nextOffset = event.nextOffset;
+                state2.cursor = event.cursor;
+                await onBatch({
+                  messages: [],
+                  nextOffset: event.nextOffset,
+                  cursor: event.cursor,
+                  upToDate: true
+                });
+                break;
+              }
+              case "eof": {
+                state2.nextOffset = event.nextOffset;
+                state2.cursor = void 0;
+                await onBatch({
+                  messages: [],
+                  nextOffset: event.nextOffset,
+                  cursor: void 0,
+                  upToDate: true
+                });
+                return;
+              }
+              case "reconnecting":
+              case "reconnected":
+                break;
+              case "error":
+                throw toLoroGatewayError("read live updates from", streamId, event.error, context2);
             }
-            case "reconnecting":
-            case "reconnected":
-              break;
-            case "error":
-              throw toLoroGatewayError("read live updates from", streamId, event.error, context2);
           }
+        } finally {
+          watchdog.dispose();
         }
       }
     };
   }
+  function createLiveIdleWatchdog(timeoutMs, parentSignal) {
+    const controller = new AbortController();
+    const armed = Number.isFinite(timeoutMs) && timeoutMs > 0;
+    let timer2;
+    const onParentAbort = () => controller.abort();
+    if (parentSignal) {
+      if (parentSignal.aborted) {
+        controller.abort();
+      } else {
+        parentSignal.addEventListener("abort", onParentAbort, {
+          once: true
+        });
+      }
+    }
+    const reset = () => {
+      if (!armed || controller.signal.aborted) {
+        return;
+      }
+      if (timer2 !== void 0) {
+        clearTimeout(timer2);
+      }
+      timer2 = setTimeout(() => controller.abort(), timeoutMs);
+      timer2.unref?.();
+    };
+    const dispose = () => {
+      if (timer2 !== void 0) {
+        clearTimeout(timer2);
+        timer2 = void 0;
+      }
+      parentSignal?.removeEventListener("abort", onParentAbort);
+    };
+    reset();
+    return {
+      signal: controller.signal,
+      reset,
+      dispose
+    };
+  }
   function selectJsonStreamOperationBaseUrl(baseUrl, streamId, operation, shardUrls) {
     const shardCandidates = operation === "append" ? shardUrls?.largePost ?? shardUrls?.other : operation === "live" ? shardUrls?.catchup ?? shardUrls?.bootstrap ?? shardUrls?.other : shardUrls?.other;
-    const shardBaseUrl = operation === "append" ? selectRandomShardUrl(shardCandidates) : selectStableShardUrl(shardCandidates, streamId);
+    const shardBaseUrl = selectStableShardUrl(shardCandidates, streamId);
     if (!shardBaseUrl) return baseUrl;
     try {
       const selected = new URL(shardBaseUrl);
@@ -107041,16 +107118,6 @@ ${value}`;
     }
     return candidates[(hash2 >>> 0) % candidates.length];
   }
-  function selectRandomShardUrl(candidates) {
-    if (!candidates || candidates.length === 0) return void 0;
-    const crypto2 = globalThis.crypto;
-    if (crypto2?.getRandomValues) {
-      const value = new Uint32Array(1);
-      crypto2.getRandomValues(value);
-      return candidates[(value[0] ?? 0) % candidates.length];
-    }
-    return candidates[Math.floor(Math.random() * candidates.length)];
-  }
   const delay$4 = async (ms2) => {
     await new Promise((resolve2) => setTimeout(resolve2, ms2));
   };
@@ -130961,9 +131028,6 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
     return null;
   };
   const isWorkspaceDirty = async (runGit2) => {
-    if (!await isInsideGitWorktree$1(runGit2)) {
-      return false;
-    }
     try {
       const status = await runGit2([
         "status",
@@ -130971,7 +131035,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
       ]);
       return status.trim().length > 0;
     } catch {
-      return false;
+      return void 0;
     }
   };
   const listUntrackedFiles = async (runGit2, filePaths) => {
@@ -136226,6 +136290,32 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
       };
     }
   }
+  const isOperationStoreBusyError = (error2) => {
+    if (!(error2 instanceof Error)) return false;
+    const code2 = error2.code;
+    return code2 === "SQLITE_BUSY" || code2 === "SQLITE_BUSY_SNAPSHOT";
+  };
+  const BUSY_RETRY_DELAYS_MS = [
+    100,
+    300,
+    900
+  ];
+  const runWithOperationStoreBusyRetry = async (run2, options = {}) => {
+    const delays = options.delaysMs ?? BUSY_RETRY_DELAYS_MS;
+    const sleep2 = options.sleep ?? ((ms2) => new Promise((resolve2) => setTimeout(resolve2, ms2)));
+    for (let attempt = 0; ; attempt += 1) {
+      try {
+        return run2();
+      } catch (error2) {
+        if (!isOperationStoreBusyError(error2)) throw error2;
+        const delayMs = delays[attempt];
+        if (delayMs === void 0) {
+          throw new LodyOperationStoreError("STORE_BUSY", "The local Operation store is busy; retry the command.", true);
+        }
+        await sleep2(delayMs);
+      }
+    }
+  };
   const parseJson = (text, schema2, label2) => {
     let parsed;
     try {
@@ -136361,7 +136451,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
   const getOperationDeliveryId = (requesterSessionId, operationId) => `operation:${requesterSessionId}:${operationId}:completion`;
   const getOperationCompletionTurnId = (requesterSessionId, operationId) => `operation-completion:${requesterSessionId}:${operationId}`;
   class LodyOperationStore {
-    constructor(dbPath = getLodyOperationStorePath(), now2 = Date.now) {
+    constructor(dbPath = getLodyOperationStorePath(), now2 = Date.now, options = {}) {
       this.now = now2;
       mkdirSync$2(path__default$1.dirname(dbPath), {
         recursive: true
@@ -136371,8 +136461,10 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
       this.db.pragma("journal_mode = WAL");
       this.db.pragma("foreign_keys = ON");
       this.migrate();
-      this.repairTerminalDeliveries();
-      this.cleanupExpired(false);
+      if (options.maintenance !== false) {
+        this.repairTerminalDeliveries();
+        this.cleanupExpired(false);
+      }
     }
     db;
     close() {
@@ -136414,7 +136506,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
           claimedItemIndexes
         };
       });
-      const accepted = transaction();
+      const accepted = transaction.immediate();
       this.cleanupExpired(false);
       return accepted;
     }
@@ -136471,7 +136563,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
         return {
           claimed: true
         };
-      })();
+      }).immediate();
     }
     markItemInputDurable(requesterSessionId, operationId, itemIndex, claimToken) {
       return this.db.transaction(() => {
@@ -136492,7 +136584,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
         this.db.prepare(`DELETE FROM operation_item_materializations
            WHERE requester_session_id = ? AND operation_id = ? AND item_index = ?`).run(requesterSessionId, operationId, itemIndex);
         return this.get(requesterSessionId, operationId);
-      })();
+      }).immediate();
     }
     finish(requesterSessionId, operationId, completion, finishedAt = new Date(this.now()).toISOString()) {
       const bounded = this.assertCompletionBound(completion);
@@ -136518,7 +136610,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
         }
         return updated;
       });
-      return transaction();
+      return transaction.immediate();
     }
     cancel(requesterSessionId, operationId) {
       const transaction = this.db.transaction(() => {
@@ -136542,7 +136634,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
           didCancel: true
         };
       });
-      return transaction();
+      return transaction.immediate();
     }
     listPendingDeliveries(workspaceId, requesterSessionId) {
       const rows = requesterSessionId ? this.db.prepare(`SELECT * FROM deliveries
@@ -136690,7 +136782,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
              )`).run(cutoff);
         this.db.prepare(`INSERT INTO orchestration_meta (key, value) VALUES ('last_cleanup_at_ms', ?)
            ON CONFLICT(key) DO UPDATE SET value = excluded.value`).run(String(nowMs2));
-      })();
+      }).immediate();
     }
     migrate() {
       this.db.exec(`
@@ -136774,7 +136866,7 @@ The postId is ${normalizedFeedbackPostId}. Use the feedback-progress-reporter sk
                WHERE deliveries.requester_session_id = operations.requester_session_id
                  AND deliveries.operation_id = operations.operation_id
              )`).run();
-      })();
+      }).immediate();
     }
   }
   const terminalAssistantFor = (history, userTurnId) => history.find((entry) => entry.role === "assistant" && entry.userTurnId === userTurnId && (entry.finished === true || typeof entry.endedAt === "number"));
@@ -141000,12 +141092,6 @@ ${entry.text}`).join("\n\n");
       } : {}
     };
   }
-  async function ensureSessionDocSynced(sessionDoc, reason) {
-    const synced = await sessionDoc.waitUntilSynced();
-    if (!synced) {
-      throw new Error(`Session document changes were not confirmed by Loro Streams (${reason}). Retry the command after checking network connectivity.`);
-    }
-  }
   function buildCliHistoryInputConfig(args2) {
     return {
       prompt: args2.prompt,
@@ -141420,13 +141506,24 @@ ${entry.text}`).join("\n\n");
     } catch {
     }
   }
-  async function writeDispatchPointerAndSync(args2) {
+  async function writeDispatchPointer(args2) {
     await args2.manager.repo.upsertDocMeta(getSessionRoomId(args2.sessionId), {
       latestUserMsgId: args2.userTurnId,
       lastMissingHistoryUserMsgId: void 0
     });
-    await ensureSessionDocSynced(args2.sessionDoc, `${args2.reason}:doc`);
-    await ensureWorkspaceMetaSynced(args2.manager, `${args2.reason}:meta`);
+  }
+  async function confirmDispatchSyncedBestEffort(args2) {
+    const logger2 = args2.logger ?? getLogger("session");
+    try {
+      const synced = await args2.sessionDoc.waitUntilSynced();
+      if (!synced) {
+        logger2.warn(`Session dispatch not yet confirmed by Loro Streams (${args2.reason}:doc); the durable dispatch pointer will converge on reconnect.`);
+        return;
+      }
+      await ensureWorkspaceMetaSynced(args2.manager, `${args2.reason}:meta`);
+    } catch (error2) {
+      logger2.warn(`Session dispatch confirmation did not complete (${args2.reason}); the durable dispatch pointer will converge on reconnect: ${formatErrorMessage(error2)}`);
+    }
   }
   async function listMachineMetasForWorkspace(manager) {
     return (await listAliveDocMetas(manager, isMachineDocRoomId)).map((entry) => entry.meta);
@@ -141954,6 +142051,7 @@ ${entry.text}`).join("\n\n");
     });
     let completionAbortController;
     let completionPromise;
+    let dispatched = false;
     try {
       const modeId = effectiveDispatchConfig.modeId;
       const modelId = effectiveDispatchConfig.modelId;
@@ -141981,11 +142079,15 @@ ${entry.text}`).join("\n\n");
       await manager.repo.upsertDocMeta(sessionRoomId, {
         status: SessionStatusFactory.idle()
       });
-      await writeDispatchPointerAndSync({
+      await writeDispatchPointer({
         manager,
-        sessionDoc,
         sessionId,
-        userTurnId,
+        userTurnId
+      });
+      dispatched = true;
+      await confirmDispatchSyncedBestEffort({
+        manager,
+        sessionDoc,
         reason: `session.create:${sessionId}`
       });
       await dispatchTurnFastPath({
@@ -142018,7 +142120,9 @@ ${entry.text}`).join("\n\n");
     } catch (error2) {
       completionAbortController?.abort();
       await completionPromise?.catch(() => void 0);
-      await rollbackPendingSessionCreate(manager, sessionId);
+      if (!dispatched) {
+        await rollbackPendingSessionCreate(manager, sessionId);
+      }
       throw error2;
     }
   }
@@ -142085,13 +142189,18 @@ ${entry.text}`).join("\n\n");
       signal: completionAbortController?.signal,
       onEvent: structuredOutput.onEvent
     }) : void 0;
+    let dispatched = false;
     try {
       await updateSessionActivityTimestampsBestEffort(manager, sessionId);
-      await writeDispatchPointerAndSync({
+      await writeDispatchPointer({
         manager,
-        sessionDoc,
         sessionId,
-        userTurnId,
+        userTurnId
+      });
+      dispatched = true;
+      await confirmDispatchSyncedBestEffort({
+        manager,
+        sessionDoc,
         reason: `session.chat:${sessionId}:${userTurnId}`
       });
       await dispatchTurnFastPath({
@@ -142114,11 +142223,13 @@ ${entry.text}`).join("\n\n");
     } catch (error2) {
       completionAbortController?.abort();
       await completionPromise?.catch(() => void 0);
-      await removeHistoryEntryById(sessionDoc, userTurnId);
-      await manager.repo.upsertDocMeta(getSessionRoomId(sessionId), {
-        latestUserMsgId: void 0,
-        lastMissingHistoryUserMsgId: void 0
-      }).catch(() => void 0);
+      if (!dispatched) {
+        await removeHistoryEntryById(sessionDoc, userTurnId);
+        await manager.repo.upsertDocMeta(getSessionRoomId(sessionId), {
+          latestUserMsgId: void 0,
+          lastMissingHistoryUserMsgId: void 0
+        }).catch(() => void 0);
+      }
       throw error2;
     }
   }
@@ -143370,7 +143481,7 @@ ${entry.text}`).join("\n\n");
       } catch (error2) {
         this.deps.logger.debug(`[${sessionId}] Failed to compute git diff stats:`, error2);
       }
-      let workspaceDirty = false;
+      let workspaceDirty;
       try {
         workspaceDirty = await isWorkspaceDirty(runGit2);
         this.deps.logger.debug(`[${sessionId}] Workspace dirty: ${workspaceDirty}`);
@@ -143381,9 +143492,11 @@ ${entry.text}`).join("\n\n");
         const sessionDoc = await this.deps.workspaceDocument.getOrCreateSessionDoc(sessionId);
         const workspace = await this.resolveWorkspaceSessionContext(sessionId, sessionDoc);
         const metaPatch = {
-          diffStats,
-          workspaceDirty
+          diffStats
         };
+        if (workspaceDirty !== void 0) {
+          metaPatch.workspaceDirty = workspaceDirty;
+        }
         await this.deps.workspaceDocument.repo.upsertDocMeta(workspace.ownerRoomId, metaPatch);
       } catch (error2) {
         this.deps.logger.debug(`[${sessionId}] Failed to persist session meta diffStats: ${formatErrorMessage(error2)}`);
@@ -144451,11 +144564,11 @@ ${entry.text}`).join("\n\n");
         });
       }
       const workspaceRoot = path__default$1.resolve(resolvedWorkspace.workspaceRoot);
-      const workspacePath = normalizeWorkspacePath(requestedPath, options);
+      const workspacePath = normalizeWorkspacePath(resolveWorkspaceRelativeRequestPath(workspaceRoot, requestedPath), options);
       const requestedAbsolutePath = path__default$1.resolve(workspaceRoot, workspacePath);
       const absolutePath = await resolveExistingPathWithoutConflicts(workspaceRoot, workspacePath, requestedPath) ?? requestedAbsolutePath;
       const relative2 = path__default$1.relative(workspaceRoot, absolutePath);
-      if (relative2.startsWith("..") || path__default$1.isAbsolute(relative2)) {
+      if (isPathOutsideRoot(relative2)) {
         throw new CodeCollabV2ServiceError("invalid_path", "Path escapes workspace root.", {
           path: requestedPath
         });
@@ -145140,6 +145253,21 @@ ${entry.text}`).join("\n\n");
     }
     return normalized;
   }
+  function resolveWorkspaceRelativeRequestPath(workspaceRoot, input2) {
+    if (!path__default$1.isAbsolute(input2)) {
+      return input2;
+    }
+    const relative2 = path__default$1.relative(workspaceRoot, path__default$1.resolve(input2));
+    if (isPathOutsideRoot(relative2)) {
+      throw new CodeCollabV2ServiceError("invalid_path", "Path escapes workspace root.", {
+        path: input2
+      });
+    }
+    return relative2.split(path__default$1.sep).join("/");
+  }
+  function isPathOutsideRoot(relativePath) {
+    return relativePath === ".." || relativePath.startsWith(`..${path__default$1.sep}`) || path__default$1.isAbsolute(relativePath);
+  }
   async function stableReadFile(resolved, maxBytes) {
     let lastTransientError;
     for (let attempt = 0; attempt < 3; attempt += 1) {
@@ -145256,7 +145384,7 @@ ${entry.text}`).join("\n\n");
       throw mapNodeReadError(error2, resolved.workspacePath);
     });
     const relative2 = path__default$1.relative(rootRealpath, fileRealpath);
-    if (relative2.startsWith("..") || path__default$1.isAbsolute(relative2)) {
+    if (isPathOutsideRoot(relative2)) {
       throw new CodeCollabV2ServiceError("invalid_path", "Resolved path escapes workspace root.", {
         path: resolved.workspacePath
       });
@@ -153340,7 +153468,10 @@ ${escapeHtmlScriptContent(VISUAL_ANNOTATION_INSPECTOR_BROWSER_SCRIPT)}
               return {
                 ...entry,
                 userTurnId: entry.userTurnId ?? userTurnId,
-                modelInfo: modelInfo ?? entry.modelInfo
+                modelInfo: modelInfo ?? entry.modelInfo,
+                finished: false,
+                endedAt: void 0,
+                permissionWaitMs: void 0
               };
             });
           }
@@ -155590,7 +155721,9 @@ ${escapeHtmlScriptContent(VISUAL_ANNOTATION_INSPECTOR_BROWSER_SCRIPT)}
         this.logger.debug(`[${sessionId}] Failed to delete session doc (may already be deleted): ${formatErrorMessage(error2)}`);
       }
       try {
-        const operationStore = new LodyOperationStore(getLodyOperationStorePath(this.machineId));
+        const operationStore = new LodyOperationStore(getLodyOperationStorePath(this.machineId), void 0, {
+          maintenance: false
+        });
         try {
           operationStore.deleteRequesterSession(sessionId);
         } finally {
@@ -203332,7 +203465,7 @@ ${page}${helpTipBottom}${choiceDescription}${ansiEscapes.cursorHide}`;
       data = createReviewBundleSnapshot(bundle);
     }
     const { injectReviewSnapshot } = await import("./chunks/index-VoI6Ds2-.js");
-    const { resolveReviewViewerTemplate } = await import("./chunks/review-viewer-CP4bkTWs.js");
+    const { resolveReviewViewerTemplate } = await import("./chunks/review-viewer-B9Tm-8O4.js");
     const template = await resolveReviewViewerTemplate();
     const html = injectReviewSnapshot(template, data);
     const outputPath = options.output ? path__default$1.resolve(options.output) : defaultHtmlOutputPath(inputPath);
@@ -221230,14 +221363,14 @@ ${result.stderr}`;
     localControlSocketPath: readOptionalEnv("LODY_MCP_SOCKET_PATH", "LODY_PREVIEW_MCP_SOCKET_PATH"),
     workdir: readOptionalEnv("LODY_MCP_WORKDIR", "LODY_PREVIEW_MCP_WORKDIR") ?? process.cwd()
   });
-  const withOperationStore = (fn) => {
-    const store = new LodyOperationStore();
-    try {
-      return fn(store);
-    } finally {
-      store.close();
-    }
+  let sharedOperationStore;
+  const getSharedOperationStore = () => {
+    sharedOperationStore ??= new LodyOperationStore(void 0, void 0, {
+      maintenance: false
+    });
+    return sharedOperationStore;
   };
+  const withOperationStore = (fn) => runWithOperationStoreBusyRetry(() => fn(getSharedOperationStore()));
   const assertBatchSize = (length2, maximum) => {
     if (length2 < 1 || length2 > maximum) {
       throw new LodyOperationStoreError("BATCH_TOO_LARGE", `Batch item count must be between 1 and ${maximum}; received ${length2}.`, false);
@@ -222074,8 +222207,8 @@ ${result.stderr}`;
           deadlineSeconds: args2.deadlineSeconds
         } : {}
       };
-      const retry2 = withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_create", canonicalCommand));
-      if (retry2) return withOperationStore((store) => store.snapshot(retry2));
+      const retry2 = await withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_create", canonicalCommand));
+      if (retry2) return await withOperationStore((store) => store.snapshot(retry2));
       const targetMachineId = args2.machineId ?? currentSession.machineId;
       await assertMachineOnlineForSingleCommand(manager, targetMachineId, ctx);
       const createOptions = buildMcpCreateOptions(args2, ctx);
@@ -222096,7 +222229,7 @@ ${result.stderr}`;
       const preallocatedUserTurnId = randomUUID();
       const materializationClaimToken = randomUUID();
       const timing = operationDeadline(args2.deadlineSeconds);
-      const accepted = withOperationStore((store) => store.accept({
+      const accepted = await withOperationStore((store) => store.accept({
         workspaceId: workspace.id,
         ownerMachineId: ctx.machineId,
         requesterSessionId: ctx.sessionId,
@@ -222119,7 +222252,7 @@ ${result.stderr}`;
         materializationClaimToken
       }));
       if (accepted.operation.state === "finished") {
-        return withOperationStore((store) => store.snapshot(accepted.operation));
+        return await withOperationStore((store) => store.snapshot(accepted.operation));
       }
       const pendingItem = accepted.operation.items[0];
       if (!pendingItem || pendingItem.status !== "active") {
@@ -222133,7 +222266,7 @@ ${result.stderr}`;
         if (result.sessionId !== pendingItem.target.sessionId || result.userTurnId !== pendingItem.target.userTurnId) {
           throw new Error("Create result did not preserve preallocated target ids.");
         }
-        withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, 0, materializationClaimToken));
+        await withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, 0, materializationClaimToken));
       }
       return snapshotOperation(ctx.sessionId, args2.operationId);
     });
@@ -222158,8 +222291,8 @@ ${result.stderr}`;
           deadlineSeconds: args2.deadlineSeconds
         } : {}
       };
-      const retry2 = withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_chat", canonicalCommand));
-      if (retry2) return withOperationStore((store) => store.snapshot(retry2));
+      const retry2 = await withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_chat", canonicalCommand));
+      if (retry2) return await withOperationStore((store) => store.snapshot(retry2));
       const targetSession = await readCurrentSessionMeta(manager, args2.sessionId);
       if (!targetSession) {
         throw new LodyOperationStoreError("SESSION_NOT_FOUND", `Target Session not found: ${args2.sessionId}`, false);
@@ -222181,7 +222314,7 @@ ${result.stderr}`;
       const preallocatedUserTurnId = randomUUID();
       const materializationClaimToken = randomUUID();
       const timing = operationDeadline(args2.deadlineSeconds);
-      const accepted = withOperationStore((store) => store.accept({
+      const accepted = await withOperationStore((store) => store.accept({
         workspaceId: workspace.id,
         ownerMachineId: ctx.machineId,
         requesterSessionId: ctx.sessionId,
@@ -222204,7 +222337,7 @@ ${result.stderr}`;
         materializationClaimToken
       }));
       if (accepted.operation.state === "finished") {
-        return withOperationStore((store) => store.snapshot(accepted.operation));
+        return await withOperationStore((store) => store.snapshot(accepted.operation));
       }
       const pendingItem = accepted.operation.items[0];
       if (!pendingItem || pendingItem.status !== "active") {
@@ -222218,7 +222351,7 @@ ${result.stderr}`;
         if (result.userTurnId !== pendingItem.target.userTurnId) {
           throw new Error("Chat result did not preserve the preallocated target turn id.");
         }
-        withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, 0, materializationClaimToken));
+        await withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, 0, materializationClaimToken));
       }
       return snapshotOperation(ctx.sessionId, args2.operationId);
     });
@@ -222261,11 +222394,11 @@ ${result.stderr}`;
     } : {},
     error: makeLodyError(code2, message, retryable)
   });
-  const finishOperationWhenEveryItemIsTerminal = (requesterSessionId, operationId, items2) => {
+  const finishOperationWhenEveryItemIsTerminal = async (requesterSessionId, operationId, items2) => {
     if (items2.some((item) => item.status === "active")) {
       return;
     }
-    withOperationStore((store) => store.finish(requesterSessionId, operationId, {
+    await withOperationStore((store) => store.finish(requesterSessionId, operationId, {
       type: "result",
       value: {
         items: items2
@@ -222293,8 +222426,8 @@ ${result.stderr}`;
           deadlineSeconds: args2.deadlineSeconds
         } : {}
       };
-      const retry2 = withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_create_many", canonicalCommand));
-      if (retry2) return withOperationStore((store) => store.snapshot(retry2));
+      const retry2 = await withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_create_many", canonicalCommand));
+      if (retry2) return await withOperationStore((store) => store.snapshot(retry2));
       const invoking = await resolveInvokingTurnContext(manager, requester);
       const initialItems = await mapWithConcurrency(expanded, 5, async (item) => {
         const label2 = item.label;
@@ -222341,7 +222474,7 @@ ${result.stderr}`;
       });
       const materializationClaimToken = randomUUID();
       const timing = operationDeadline(args2.deadlineSeconds);
-      const accepted = withOperationStore((store) => store.accept({
+      const accepted = await withOperationStore((store) => store.accept({
         workspaceId: workspace.id,
         ownerMachineId: ctx.machineId,
         requesterSessionId: ctx.sessionId,
@@ -222362,7 +222495,7 @@ ${result.stderr}`;
         materializationClaimToken
       }));
       if (accepted.operation.state === "finished") {
-        return withOperationStore((store) => store.snapshot(accepted.operation));
+        return await withOperationStore((store) => store.snapshot(accepted.operation));
       }
       const nextItems = await mapWithConcurrency(accepted.operation.items, 5, async (storedItem, index) => {
         if (storedItem.status !== "active" || storedItem.inputDurable || !accepted.claimedItemIndexes.includes(index)) {
@@ -222396,13 +222529,13 @@ ${result.stderr}`;
           options.userTurnId = storedItem.target.userTurnId;
           options.chainDepth = invoking.chainDepth + 1;
           await createSessionResult(auth, workspace, manager, expandedItem.prompt, options, resolveTurnDispatchConfig({}));
-          withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, index, materializationClaimToken));
+          await withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, index, materializationClaimToken));
           return markOperationItemInputDurable(storedItem);
         } catch {
           return storedItem;
         }
       });
-      finishOperationWhenEveryItemIsTerminal(ctx.sessionId, args2.operationId, nextItems);
+      await finishOperationWhenEveryItemIsTerminal(ctx.sessionId, args2.operationId, nextItems);
       return snapshotOperation(ctx.sessionId, args2.operationId);
     });
   };
@@ -222427,8 +222560,8 @@ ${result.stderr}`;
           deadlineSeconds: args2.deadlineSeconds
         } : {}
       };
-      const retry2 = withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_chat_many", canonicalCommand));
-      if (retry2) return withOperationStore((store) => store.snapshot(retry2));
+      const retry2 = await withOperationStore((store) => store.findMatchingRetry(ctx.sessionId, args2.operationId, "session_chat_many", canonicalCommand));
+      if (retry2) return await withOperationStore((store) => store.snapshot(retry2));
       const invoking = await resolveInvokingTurnContext(manager, requester);
       const initialItems = await mapWithConcurrency(expanded, 5, async (item) => {
         if (!item.sessionId || !item.prompt) {
@@ -222459,7 +222592,7 @@ ${result.stderr}`;
       });
       const materializationClaimToken = randomUUID();
       const timing = operationDeadline(args2.deadlineSeconds);
-      const accepted = withOperationStore((store) => store.accept({
+      const accepted = await withOperationStore((store) => store.accept({
         workspaceId: workspace.id,
         ownerMachineId: ctx.machineId,
         requesterSessionId: ctx.sessionId,
@@ -222480,7 +222613,7 @@ ${result.stderr}`;
         materializationClaimToken
       }));
       if (accepted.operation.state === "finished") {
-        return withOperationStore((store) => store.snapshot(accepted.operation));
+        return await withOperationStore((store) => store.snapshot(accepted.operation));
       }
       const nextItems = await mapWithConcurrency(accepted.operation.items, 5, async (storedItem, index) => {
         if (storedItem.status !== "active" || storedItem.inputDurable || !accepted.claimedItemIndexes.includes(index)) {
@@ -222495,13 +222628,13 @@ ${result.stderr}`;
             userTurnId: storedItem.target.userTurnId,
             chainDepth: invoking.chainDepth + 1
           });
-          withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, index, materializationClaimToken));
+          await withOperationStore((store) => store.markItemInputDurable(ctx.sessionId, args2.operationId, index, materializationClaimToken));
           return markOperationItemInputDurable(storedItem);
         } catch {
           return storedItem;
         }
       });
-      finishOperationWhenEveryItemIsTerminal(ctx.sessionId, args2.operationId, nextItems);
+      await finishOperationWhenEveryItemIsTerminal(ctx.sessionId, args2.operationId, nextItems);
       return snapshotOperation(ctx.sessionId, args2.operationId);
     });
   };
@@ -222783,7 +222916,7 @@ ${lines2.join("\n")}` : ""}${suffix}`);
     }, async (args2) => {
       try {
         const ctx = getSessionContext();
-        return jsonTextResult(snapshotOperation(ctx.sessionId, args2.operationId));
+        return jsonTextResult(await snapshotOperation(ctx.sessionId, args2.operationId));
       } catch (error2) {
         return mcpErrorResult(error2);
       }
@@ -222796,13 +222929,13 @@ ${lines2.join("\n")}` : ""}${suffix}`);
       try {
         const ctx = getSessionContext();
         const requesterSessionId = ctx.sessionId;
-        const before = withOperationStore((store) => store.get(requesterSessionId, args2.operationId));
-        const cancellation = withOperationStore((store) => store.cancel(requesterSessionId, args2.operationId));
+        const before = await withOperationStore((store) => store.get(requesterSessionId, args2.operationId));
+        const cancellation = await withOperationStore((store) => store.cancel(requesterSessionId, args2.operationId));
         if (cancellation.didCancel && before.state === "active") {
           const startedTargets = before.items.filter((item) => item.status === "active" && item.inputDurable);
           await Promise.allSettled(startedTargets.map((item) => item.status === "active" ? runLodyCliJson(buildOperationTargetCancelArgs(getMcpWorkspaceId(ctx), item)) : Promise.resolve()));
         }
-        return jsonTextResult(withOperationStore((store) => store.snapshot(cancellation.operation)));
+        return jsonTextResult(await withOperationStore((store) => store.snapshot(cancellation.operation)));
       } catch (error2) {
         return mcpErrorResult(error2);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "lody",
-  "version": "0.71.2",
+  "version": "0.71.4",
   "description": "Lody Agent CLI tool for managing remote command execution",
   "type": "module",
   "main": "dist/index.js",
@@ -76,14 +76,14 @@
     "zod": "^4.1.5",
     "zstd-stream": "^1.1.0",
     "@lody/cli-supervisor": "0.0.1",
-    "@lody/code-review-helper": "0.0.0",
-    "@lody/shared": "0.0.1",
     "@lody/convex": "0.0.1",
     "@lody/loro-streams-rpc": "0.0.1",
+    "@lody/shared": "0.0.1",
+    "@lody/code-review-helper": "0.0.0",
     "acp-extension-claude": "0.59.0",
+    "lody-code-review-viewer": "0.71.4",
     "acp-extension-core": "0.0.5",
-    "acp-extension-codex": "1.2.1",
-    "lody-code-review-viewer": "0.71.2"
+    "acp-extension-codex": "1.2.1"
   },
   "files": [
     "dist",
@@ -106,7 +106,7 @@
     "build": "pnpm run clean && pnpm run prepare:acp-adapters && pnpm run typecheck && pnpm run build:bundle && pnpm run check:published-bundle-imports && pnpm run copy:wasm",
     "build:watch": "pnpm run prepare:review-assets && vite build --watch",
     "build:bundle": "pnpm run prepare:review-assets && vite build",
-    "prepare:acp-adapters": "pnpm --dir ../.. --filter acp-extension-claude build && pnpm --dir ../.. --filter acp-extension-codex build",
+    "prepare:acp-adapters": "corepack pnpm --dir ../.. --filter acp-extension-claude build && corepack pnpm --dir ../.. --filter acp-extension-codex build",
     "prepare:review-assets": "pnpm --dir ../.. --filter lody-code-review-viewer build",
     "check:published-bundle-imports": "node scripts/check-published-bundle-imports.js",
     "copy:wasm": "node scripts/copy-loro-wasm.js",