orkestrator 2.34.4-beta → 2.34.5-beta

package/README.md

@@ -57,6 +57,10 @@ bunx orkestrator \
   --allow-non-tailscale-bind
 ```
 
+Only the default data directory's backend defaults to port 34121. With another
+`--data-dir` and no `--port`, the backend takes a free port that changes on each
+start, so pass `--port` if remote clients need a stable address.
+
 Or install Bun when needed and start the service in one command:
 
 ```bash
@@ -94,6 +98,14 @@ points at the installed set and the service uses it by default for that data
 directory — restart a running service to pick up a new one. Pass `--data-dir`
 when the service does, and see `orkestrator toolchain install --help`.
 
+## Cursor and Pi
+
+The Cursor and Pi bridges ship without their SDKs, which are too large or too
+platform-specific to bundle. The first time one starts, the backend installs
+the SDK it pins into `<data-dir>/bridge-runtimes/` with
+`bun install --frozen-lockfile`, so that first start needs access to the npm
+registry. Later starts, and restarts, reuse it.
+
 ## Client commands
 
 Select the backend explicitly. A running backend publishes a private

package/bin/orkestrator-launcher.js

@@ -9,7 +9,10 @@
  * - A release that has not reached readiness (a backend-instance.json naming
  *   its process and version) gets two boots. After that the launcher points
  *   `current` back at `previous` and records the rollback, which the restored
- *   backend reports as the outcome of its update.
+ *   backend reports as the outcome of its update. A release restored this way
+ *   runs with the serve arguments it had before the install that replaced it.
+ * - Each boot that exits without reaching readiness is recorded, so
+ *   `service install` can stop waiting and report it.
  * - When the backend exits after writing `restart-requested` (its self-update
  *   just switched `current`), the launcher starts the new release at once.
  * - SIGTERM and SIGINT are forwarded; the launcher then exits with the
@@ -69,10 +72,16 @@ function readState() {
   const value = readJson(layout.stateFile) ?? {};
   return {
     version: 1,
+    ...(service.installGeneration ? { installGeneration: service.installGeneration } : {}),
     lastReadyVersion: typeof value.lastReadyVersion === "string" ? value.lastReadyVersion : null,
     bootAttempts:
-      value.bootAttempts && typeof value.bootAttempts === "object" ? value.bootAttempts : {},
+      (!service.installGeneration || value.installGeneration === service.installGeneration) &&
+      value.bootAttempts &&
+      typeof value.bootAttempts === "object"
+        ? value.bootAttempts
+        : {},
     rollback: value.rollback ?? null,
+    bootFailure: value.bootFailure ?? null,
   };
 }
 
@@ -109,8 +118,21 @@ if (!service || typeof service.bun !== "string" || !Array.isArray(service.args))
   process.exit(1);
 }
 
+/**
+ * After a rollback `current` and `previous` name the same release. Only then do
+ * the arguments recorded for it apply: the same version reached by a later
+ * self-update runs with the service's own.
+ */
+function serveArguments(version, previousVersion) {
+  const target = service.rollbackTarget;
+  return target?.version === version && previousVersion === version && Array.isArray(target.args)
+    ? target.args
+    : service.args;
+}
+
 let child = null;
 let stopping = false;
+let failedBoots = { version: null, exits: 0 };
 for (const signal of ["SIGTERM", "SIGINT"]) {
   process.on(signal, () => {
     stopping = true;
@@ -150,6 +172,7 @@ while (!stopping) {
       pointLink(layout.current, previousTarget);
       state.bootAttempts[version] = 0;
       state.rollback = {
+        ...(service.installGeneration ? { installGeneration: service.installGeneration } : {}),
         failedVersion: version,
         restoredVersion: previousVersion,
         at: new Date().toISOString(),
@@ -172,9 +195,10 @@ while (!stopping) {
   rmSync(layout.restartMarker, { force: true });
   log(`starting ${version}`);
   let exited = false;
+  const serveArgs = serveArguments(version, path.basename(linked(layout.previous) ?? ""));
   child = spawn(
     service.bun,
-    [entry, "serve", ...service.args, "--data-dir", dataDir, "--standalone-updater", root],
+    [entry, "serve", ...serveArgs, "--data-dir", dataDir, "--standalone-updater", root],
     { stdio: "inherit", env: process.env },
   );
   const exit = new Promise((resolve) =>
@@ -188,7 +212,9 @@ while (!stopping) {
     const next = readState();
     next.lastReadyVersion = version;
     next.bootAttempts[version] = 0;
+    next.bootFailure = null;
     writeState(next);
+    failedBoots = { version: null, exits: 0 };
     log(`${version} is ready`);
   }
   if (!ready && !exited) {
@@ -212,5 +238,18 @@ while (!stopping) {
     continue;
   }
   log(`${version} exited (code ${code ?? "none"}, signal ${signal ?? "none"})`);
+  if (!ready) {
+    failedBoots = {
+      version,
+      exits: failedBoots.version === version ? failedBoots.exits + 1 : 1,
+    };
+    const next = readState();
+    next.bootFailure = {
+      ...failedBoots,
+      ...(service.installGeneration ? { installGeneration: service.installGeneration } : {}),
+      at: new Date().toISOString(),
+    };
+    writeState(next);
+  }
   await sleep(ready ? CRASH_DELAY_MS : RETRY_DELAY_MS);
 }