@withone/cli 1.52.2 → 1.52.3

package/dist/{chunk-FKHUH223.js → chunk-IKYXS7EI.js}

@@ -216,6 +216,32 @@ function listSyncedPlatforms() {
   if (!fs3.existsSync(DATA_DIR)) return [];
   return fs3.readdirSync(DATA_DIR).filter((f) => f.endsWith(".db")).map((f) => f.replace(/\.db$/, ""));
 }
+function isDriverFault(err) {
+  const code = err?.code;
+  if (code === "ERR_DLOPEN_FAILED") return true;
+  const msg = err instanceof Error ? err.message : String(err ?? "");
+  return /NODE_MODULE_VERSION|compiled against a different Node\.js version|Could not locate the bindings file|invalid ELF header|wrong ELF class|symbol not found|image not found|not a valid Win32 application/i.test(msg);
+}
+function passesIntegrityCheck(Ctor, dbPath) {
+  let probe;
+  try {
+    probe = new Ctor(dbPath, { readonly: true, fileMustExist: true });
+    const result = probe.pragma("quick_check");
+    const first = Array.isArray(result) ? result[0] : result;
+    const verdict = typeof first === "string" ? first : first?.quick_check;
+    return verdict === "ok";
+  } catch {
+    return false;
+  } finally {
+    try {
+      probe?.close();
+    } catch {
+    }
+  }
+}
+function backupPathFor(dbPath, now = /* @__PURE__ */ new Date()) {
+  return `${dbPath}.bak.${now.toISOString().replace(/[:.]/g, "-")}`;
+}
 async function openDatabase(platform, opts = {}) {
   const Database = await loadSqlite();
   fs3.mkdirSync(DATA_DIR, { recursive: true });
@@ -226,13 +252,38 @@ async function openDatabase(platform, opts = {}) {
   let db;
   try {
     db = new Database(dbPath);
-  } catch {
-    const backupPath = dbPath + ".bak";
-    if (fs3.existsSync(dbPath)) {
-      fs3.renameSync(dbPath, backupPath);
-      process.stderr.write(`Database corrupted, starting fresh. Backup saved at ${backupPath}
-`);
+  } catch (err) {
+    if (isDriverFault(err)) {
+      const detail = err instanceof Error ? err.message.split("\n")[0] : String(err);
+      throw new Error(
+        `The local sync engine (better-sqlite3) could not load in this Node process.
+Your database was NOT modified.
+
+This usually means the CLI is running under a different Node than the one
+better-sqlite3 was built for \u2014 currently node ${process.version} (NODE_MODULE_VERSION ${process.versions.modules}). \`one\` is a #!/usr/bin/env node shim, so a minimal PATH under cron, launchd,
+or an agent runner can select a different interpreter than your shell does.
+
+Rebuild it against this Node with:
+  one sync install
+
+Underlying error: ${detail}`
+      );
     }
+    if (!fs3.existsSync(dbPath)) throw err;
+    if (passesIntegrityCheck(Database, dbPath)) {
+      const detail = err instanceof Error ? err.message.split("\n")[0] : String(err);
+      throw new Error(
+        `Could not open ${dbPath}, but it passes SQLite's integrity check \u2014 so it is not corrupt and has been left untouched.
+
+Underlying error: ${detail}`
+      );
+    }
+    const backupPath = backupPathFor(dbPath);
+    fs3.renameSync(dbPath, backupPath);
+    process.stderr.write(
+      `Database at ${dbPath} failed its integrity check. Backup saved at ${backupPath}, starting fresh.
+`
+    );
     db = new Database(dbPath);
   }
   db.pragma("journal_mode = WAL");

package/dist/index.js

@@ -65,7 +65,7 @@ import {
   writeDraftProfile,
   writePageToMemory,
   writeProfile
-} from "./chunk-FKHUH223.js";
+} from "./chunk-IKYXS7EI.js";
 import {
   getByDotPath
 } from "./chunk-44CV5IMX.js";
@@ -7731,6 +7731,7 @@ async function syncDoctorCommand() {
     checks.push({ name: "better-sqlite3 loads", ok: false, detail: err instanceof Error ? err.message : String(err) });
   }
   const allOk = checks.every((c) => c.ok);
+  if (!allOk) process.exitCode = 1;
   if (isAgentMode()) {
     json({ ok: allOk, checks });
     return;
@@ -8286,7 +8287,7 @@ async function maybeAutoMigrateLegacy(platform, models) {
       `  detected legacy .one/sync/data/${platform}.db (${dbSize}) \u2014 auto-migrating into memory before sync.
 `
     );
-    const { memMigrateCommand: memMigrateCommand3 } = await import("./migrate-LA7I3OQN.js");
+    const { memMigrateCommand: memMigrateCommand3 } = await import("./migrate-U2MZVGU6.js");
     await memMigrateCommand3({ platform, yes: true });
     return;
   }
@@ -8295,7 +8296,7 @@ async function maybeAutoMigrateLegacy(platform, models) {
     initialValue: true
   });
   if (p7.isCancel(shouldMigrate) || !shouldMigrate) return;
-  const { memMigrateCommand: memMigrateCommand2 } = await import("./migrate-LA7I3OQN.js");
+  const { memMigrateCommand: memMigrateCommand2 } = await import("./migrate-U2MZVGU6.js");
   await memMigrateCommand2({ platform, yes: true });
 }
 async function syncSuggestSearchableCommand(platformModel, options = {}) {
@@ -10903,7 +10904,7 @@ Every \`sync X\` command is also exposed as \`mem sync X\` \u2014 same handlers,
 | Command | What it does |
 |---------|-------------|
 | \`sync profiles [platform]\` | List built-in pre-validated profiles |
-| \`sync doctor\` | Verify sync engine health |
+| \`sync doctor\` | Verify sync engine health. **Exits non-zero when not ready**, so \`one sync doctor && one sync run <platform>\` is a safe gate \u2014 run it first from cron/launchd, where a bare PATH can select a Node the native driver wasn't built for |
 | \`sync models <platform>\` | Discover available models |
 | \`sync init <plat> <model>\` | Create/patch profile (seeds from built-in, auto-tests) |
 | \`sync test <plat>/<model>\` | Validate profile. \`--show-searchable\` previews embedded text across 5 samples with per-path hit rates |

package/dist/{migrate-LA7I3OQN.js → migrate-U2MZVGU6.js}

@@ -3,7 +3,7 @@ import {
   dotPathToJsonbExpr,
   memMigrateCommand,
   reviveStringifiedJson
-} from "./chunk-FKHUH223.js";
+} from "./chunk-IKYXS7EI.js";
 import "./chunk-44CV5IMX.js";
 import "./chunk-GNSR3NYN.js";
 import "./chunk-DDCDPVJH.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@withone/cli",
-  "version": "1.52.2",
+  "version": "1.52.3",
   "description": "CLI for managing One",
   "type": "module",
   "files": [