greview-cli 0.13.2 → 0.13.3

package/dist/cli.js

@@ -330,8 +330,8 @@ var Store = class {
     this.path = dbPathFor(repo);
     mkdirSync(dirname(this.path), { recursive: true });
     this.db = new (sqlite()).DatabaseSync(this.path);
-    this.db.exec("PRAGMA journal_mode = WAL");
     this.db.exec("PRAGMA busy_timeout = 5000");
+    this.db.exec("PRAGMA journal_mode = DELETE");
     this.db.exec("PRAGMA foreign_keys = ON");
     this.migrate();
   }
@@ -1719,7 +1719,17 @@ Try: greview help
   process.stdout.write(text);
   return 0;
 }
-function main(argv) {
+var BUSY_ATTEMPTS = 4;
+function isBusy(e) {
+  if (!(e instanceof Error)) return false;
+  const errcode = e.errcode;
+  if (typeof errcode === "number") return (errcode & 255) === 5;
+  return e.message === "database is locked";
+}
+function sleep(ms) {
+  return new Promise((resolve2) => setTimeout(resolve2, ms));
+}
+async function main(argv) {
   let values;
   let positionals;
   try {
@@ -1738,88 +1748,89 @@ function main(argv) {
     return 0;
   }
   if (command === "setup") return cmdSetup(positionals);
-  let ctx = null;
-  try {
-    const repo = findRepo(cwdOf(values));
-    ctx = { repo, store: new Store(repo), json: values.json === true };
-    switch (command) {
-      case "repo":
-        cmdRepo(ctx);
-        break;
-      case "add":
-        cmdAdd(ctx, values);
-        break;
-      case "list":
-      case "ls":
-        cmdList(ctx, values);
-        break;
-      case "show":
-        cmdShow(ctx, values, positionals);
-        break;
-      case "reply":
-        cmdReply(ctx, values, positionals);
-        break;
-      case "edit":
-        cmdEdit(ctx, values, positionals);
-        break;
-      case "resolve":
-        cmdSetStatus(ctx, values, positionals, true);
-        break;
-      case "unresolve":
-      case "reopen":
-        cmdSetStatus(ctx, values, positionals, false);
-        break;
-      case "rm":
-      case "delete":
-        cmdRm(ctx, positionals);
-        break;
-      case "sync":
-        cmdSync(ctx);
-        break;
-      case "stats":
-        cmdStats(ctx);
-        break;
-      case "onsubmit": {
-        const pending = cmdOnsubmit(ctx, values, positionals);
-        if (pending) return pending.then(() => 0);
-        break;
-      }
-      default:
-        process.stderr.write(`greview: unknown command "${command}"
+  for (let attempt = 1; ; attempt++) {
+    let ctx = null;
+    try {
+      const repo = findRepo(cwdOf(values));
+      ctx = { repo, store: new Store(repo), json: values.json === true };
+      switch (command) {
+        case "repo":
+          cmdRepo(ctx);
+          break;
+        case "add":
+          cmdAdd(ctx, values);
+          break;
+        case "list":
+        case "ls":
+          cmdList(ctx, values);
+          break;
+        case "show":
+          cmdShow(ctx, values, positionals);
+          break;
+        case "reply":
+          cmdReply(ctx, values, positionals);
+          break;
+        case "edit":
+          cmdEdit(ctx, values, positionals);
+          break;
+        case "resolve":
+          cmdSetStatus(ctx, values, positionals, true);
+          break;
+        case "unresolve":
+        case "reopen":
+          cmdSetStatus(ctx, values, positionals, false);
+          break;
+        case "rm":
+        case "delete":
+          cmdRm(ctx, positionals);
+          break;
+        case "sync":
+          cmdSync(ctx);
+          break;
+        case "stats":
+          cmdStats(ctx);
+          break;
+        case "onsubmit": {
+          const pending = cmdOnsubmit(ctx, values, positionals);
+          if (pending) return pending.then(() => 0);
+          break;
+        }
+        default:
+          process.stderr.write(`greview: unknown command "${command}"
 Try: greview help
 `);
-        return 2;
-    }
-    return 0;
-  } catch (e) {
-    const message = e instanceof Error ? e.message : String(e);
-    if (values.json) {
-      process.stdout.write(`${JSON.stringify({ ok: false, error: message })}
+          return 2;
+      }
+      return 0;
+    } catch (e) {
+      if (isBusy(e) && attempt < BUSY_ATTEMPTS) {
+        await sleep(attempt * 150 + Math.random() * 100);
+        continue;
+      }
+      const message = e instanceof Error ? e.message : String(e);
+      if (values.json) {
+        process.stdout.write(`${JSON.stringify({ ok: false, error: message })}
 `);
-    } else {
-      process.stderr.write(`greview: ${message}
+      } else {
+        process.stderr.write(`greview: ${message}
 `);
-    }
-    return e instanceof UsageError ? 2 : e instanceof GitError ? 3 : 1;
-  } finally {
-    try {
-      ctx?.store.close();
-    } catch {
+      }
+      return e instanceof UsageError ? 2 : e instanceof GitError ? 3 : 1;
+    } finally {
+      try {
+        ctx?.store.close();
+      } catch {
+      }
     }
   }
 }
-var outcome = main(process.argv.slice(2));
-if (typeof outcome === "number") {
-  process.exitCode = outcome;
-} else {
-  void outcome.then(
-    (code) => {
-      if (process.exitCode === void 0 || process.exitCode === 0) process.exitCode = code;
-    },
-    (e) => {
-      process.stderr.write(`greview: ${e instanceof Error ? e.message : String(e)}
+void main(process.argv.slice(2)).then(
+  (code) => {
+    if (process.exitCode === void 0 || process.exitCode === 0) process.exitCode = code;
+  },
+  (e) => {
+    process.stderr.write(`greview: ${e instanceof Error ? e.message : String(e)}
 `);
-      process.exitCode = 1;
-    }
-  );
-}
+    process.exitCode = 1;
+  }
+);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "greview-cli",
-  "version": "0.13.2",
+  "version": "0.13.3",
   "description": "Review local Git diffs with humans and coding agents",
   "license": "MIT",
   "type": "module",