threadnote 0.7.3 → 0.7.4

package/dist/threadnote.cjs

@@ -10124,11 +10124,7 @@ async function runPostUpdate(config, options) {
     throw new Error("Provide --from-version and --to-version for post-update.");
   }
   const interactive = process.stdin.isTTY === true && process.stdout.isTTY === true;
-  await ensurePinnedOpenVikingInstalled(config, {
-    dryRun: options.dryRun === true,
-    interactive,
-    yes: options.yes === true
-  });
+  await ensurePinnedOpenVikingInstalled(config, { dryRun: options.dryRun === true });
   await runApplicablePostUpdateMigrations(config, {
     dryRun: options.dryRun === true,
     fromVersion: options.fromVersion,
@@ -10141,7 +10137,7 @@ async function runPostUpdate(config, options) {
 async function maybeRunPostUpdateAfterRepair(config, options) {
   const toVersion = await currentPackageVersion();
   const interactive = process.stdin.isTTY === true && process.stdout.isTTY === true;
-  await ensurePinnedOpenVikingInstalled(config, { dryRun: options.dryRun, interactive, yes: false });
+  await ensurePinnedOpenVikingInstalled(config, { dryRun: options.dryRun });
   const state = await readPostUpdateState(config);
   const migrations = await applicablePostUpdateMigrations(config, {
     fromVersion: "0.0.0",
@@ -10184,18 +10180,80 @@ async function ensurePinnedOpenVikingInstalled(config, options) {
     return;
   }
   console.log("");
-  console.log(`OpenViking ${installedVersion} is older than the pinned version ${pinned}.`);
-  console.log(
-    "Upgrading picks up upstream fixes for share-sync reliability (reindex lock acquisition, ov wait timeout)."
-  );
+  console.log(`Upgrading OpenViking ${installedVersion} -> ${pinned} (pinned by Threadnote).`);
+  console.log("Picks up upstream fixes for share-sync reliability (reindex lock acquisition, ov wait timeout).");
+  const wasRunning = await isOpenVikingHealthy(config);
+  const usingLaunchd = await isLaunchAgentInstalled();
   const threadnoteCommand = currentThreadnoteCommand() ?? await findExecutable([NPM_PACKAGE_NAME]) ?? NPM_PACKAGE_NAME;
-  const installArgs = ["install", "--force", "--no-start"];
-  const accepted = options.dryRun || options.yes || options.interactive && await confirmPostUpdateMigration(`Upgrade OpenViking to ${pinned} now? [Y/n] `, true);
-  if (!accepted) {
-    console.log(`Skipped. Run manually later: ${formatMigrationCommand(threadnoteCommand, installArgs)}`);
+  await maybeRun(options.dryRun, threadnoteCommand, ["install", "--force", "--no-start"]);
+  if (options.dryRun) {
+    if (wasRunning || usingLaunchd) {
+      console.log("Would restart OpenViking server so the new binary takes effect.");
+    }
+    return;
+  }
+  if (!wasRunning && !usingLaunchd) {
     return;
   }
-  await maybeRun(options.dryRun, threadnoteCommand, installArgs);
+  console.log("Restarting OpenViking server so the new binary takes effect.");
+  if (usingLaunchd) {
+    const launchAgentPath = launchAgentPlistPath();
+    await runCommand("launchctl", ["unload", launchAgentPath], { allowFailure: true });
+    await runCommand("launchctl", ["load", launchAgentPath], { allowFailure: true });
+  } else {
+    await maybeRun(false, threadnoteCommand, ["stop"]);
+    await maybeRun(false, threadnoteCommand, ["start"]);
+  }
+  const healthyAfter = await waitForOpenVikingHealthy(config, 1e4);
+  if (!healthyAfter) {
+    console.log(
+      `Warning: OpenViking did not return to healthy at ${openVikingHealthEndpoint(config)} within 10s after the restart.`
+    );
+    console.log("Check the server log or run: threadnote start");
+  }
+}
+function openVikingHealthEndpoint(config) {
+  return `http://${config.host}:${config.port}/health`;
+}
+async function isOpenVikingHealthy(config) {
+  const controller = new AbortController();
+  const timer = setTimeout(() => {
+    controller.abort();
+  }, 800);
+  try {
+    const response = await fetch(openVikingHealthEndpoint(config), { signal: controller.signal });
+    return response.ok;
+  } catch (_err) {
+    return false;
+  } finally {
+    clearTimeout(timer);
+  }
+}
+async function waitForOpenVikingHealthy(config, timeoutMs) {
+  const deadline = Date.now() + timeoutMs;
+  while (Date.now() < deadline) {
+    if (await isOpenVikingHealthy(config)) {
+      return true;
+    }
+    await new Promise((resolvePromise) => {
+      setTimeout(resolvePromise, 500);
+    });
+  }
+  return isOpenVikingHealthy(config);
+}
+function launchAgentPlistPath() {
+  return (0, import_node_path10.join)((0, import_node_os5.homedir)(), "Library", "LaunchAgents", "io.threadnote.openviking.plist");
+}
+async function isLaunchAgentInstalled() {
+  if (process.platform !== "darwin") {
+    return false;
+  }
+  try {
+    await (0, import_promises9.access)(launchAgentPlistPath(), import_node_fs5.constants.F_OK);
+    return true;
+  } catch (_err) {
+    return false;
+  }
 }
 async function readOpenVikingCliVersion(ov) {
   const result = await runCommand(ov, ["version"], { allowFailure: true });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "threadnote",
-  "version": "0.7.3",
+  "version": "0.7.4",
   "type": "commonjs",
   "main": "dist/threadnote.cjs",
   "description": "Shared local context and handoffs for development agents",