@ricsam/r5d-worker 0.0.173 → 0.0.175

package/dist/cjs/package.json

@@ -1,5 +1,5 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.173",
+  "version": "0.0.175",
   "type": "commonjs"
 }

package/dist/mjs/internal-r5dctl.cjs

@@ -20605,7 +20605,7 @@ function resolveEntrypointPath(entrypoint) {
   }
 }
 function getR5dctlVersion() {
-  if (true) return "0.0.173";
+  if (true) return "0.0.175";
   const entrypoint = process.argv[1] ? resolveEntrypointPath(process.argv[1]) : null;
   let current = entrypoint ? import_node_path2.default.dirname(entrypoint) : process.cwd();
   for (let index = 0; index < 12; index += 1) {

package/dist/mjs/main.mjs

@@ -7,7 +7,7 @@ import { startManagerRpc } from "./runtime/releases/rpc-main.mjs";
 import { ManagerRpcConfig } from "./runtime/releases/rpc-protocol.mjs";
 const args = process.argv.slice(2);
 if (args.includes("--version")) {
-  console.log(`r5d-worker ${true ? "0.0.173" : "development"}`);
+  console.log(`r5d-worker ${true ? "0.0.175" : "development"}`);
 } else if (!args.length || args.includes("--help")) {
   console.log(
     "Usage: r5d-worker start --label <label> [--root <dir>] [--base-url <url>] [--token <worker-token>]\n       r5d-worker executor /absolute/private/config.json\n       r5d-worker manager /absolute/private/config.json\n\nRun independently supervised durable runtime services. Provision configuration with r5dinfra."
@@ -15,7 +15,7 @@ if (args.includes("--version")) {
 } else if (args[0] === "start") {
   const runtime = await startPersonalWorker(
     parsePersonalWorkerOptions(args.slice(1)),
-    true ? "0.0.173" : "development"
+    true ? "0.0.175" : "development"
   );
   console.log(`Worker connected: ${runtime.resourceId}`);
   let closing = false;

package/dist/mjs/package.json

@@ -1,5 +1,5 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.173",
+  "version": "0.0.175",
   "type": "module"
 }

package/dist/mjs/runtime/workspace/authority.mjs

@@ -164,6 +164,7 @@ class WorkspaceAuthority {
       for (const id of await fs.readdir(path.join(config.root, "state"))) {
         if (authority.benches.has(id)) continue;
         const directory = path.join(config.root, "state", id);
+        if (!(await fs.lstat(directory)).isDirectory()) continue;
         await noSymlinkAncestors(directory);
         const b = ApprovedWorkbench.parse(JSON.parse((await readRegular(path.join(directory, "workbench.json"), 128 * 1024)).toString()));
         if (b.id !== id || b.cwd !== authority.expectedCwd(b)) throw new WorkspaceError("wrong_binding", "Invalid durable dynamic workbench");
@@ -264,8 +265,13 @@ class WorkspaceAuthority {
     const result = [];
     for (const [operationId, metadata] of Object.entries(b.state.runs)) {
       if (metadata.state === "rejected_capacity" || (metadata.sessionId ?? b.config.sessionId) !== identity.sessionId) continue;
-      const poll = await this.poll(identity, { userId: identity.userId, sessionId: identity.sessionId, operationId, fence: route.workerFence }, 0, 0, 1);
-      result.push({ ...metadata, ...poll.receipt, workerLabel: route.workerFence.resourceId });
+      try {
+        const poll = await this.poll(identity, { userId: identity.userId, sessionId: identity.sessionId, operationId, fence: route.workerFence }, 0, 0, 1);
+        result.push({ ...metadata, ...poll.receipt, workerLabel: route.workerFence.resourceId });
+      } catch (error) {
+        if (!(error instanceof ExecutorError && error.code === "run_not_found")) throw error;
+        result.push({ ...metadata, operationId, userId: identity.userId, sessionId: identity.sessionId, state: "unknown", pid: null, pty: false, lane: "general", stdoutBytes: 0, stderrBytes: 0, stdinClosed: true, exitCode: null, signal: null, reason: "run_not_found", workerLabel: route.workerFence.resourceId });
+      }
     }
     return result;
   }
@@ -406,8 +412,20 @@ class WorkspaceAuthority {
       return { head, treeHash, files: changes, ...patch === void 0 ? {} : { patch } };
     });
   }
+  /** Bench state is one in-memory object with several writers on different
+   * queues (checkout work, run receipts). Writes of its file are chained so
+   * they never overlap and the write that lands last always serializes the
+   * newest state; the chain covers only the write itself, never the work
+   * around it, so a run receipt does not wait for a publication. */
   save(b) {
-    return durableJson(path.join(b.directory, "state.json"), b.state);
+    const write = (b.saving ?? Promise.resolve()).catch(() => {
+    }).then(() => durableJson(path.join(b.directory, "state.json"), b.state));
+    b.saving = write;
+    void write.finally(() => {
+      if (b.saving === write) b.saving = void 0;
+    }).catch(() => {
+    });
+    return write;
   }
   async owned(admitted = false) {
     if (this.closing && !admitted) throw new WorkspaceError("authority_closed", "Workspace authority is closing");
@@ -1846,7 +1864,12 @@ class WorkspaceAuthority {
       }
       const nonmutating = Object.keys(requestedEnv).length === 0 && payload.credentials.length === 0 && b.config.nonmutatingArgv.some((argv) => canonicalJson(argv) === canonicalJson(payload.argv));
       b.state.runs[operation.operationId] = { artifactEnvironment: true, sessionId: identity.sessionId, payloadHash: hash, mutating: !nonmutating, state: "unknown", argv: payload.argv, cwd: payload.cwd, startedAt: (/* @__PURE__ */ new Date()).toISOString(), updatedAt: (/* @__PURE__ */ new Date()).toISOString() };
-      await this.save(b);
+      try {
+        await this.save(b);
+      } catch (error) {
+        delete b.state.runs[operation.operationId];
+        throw error;
+      }
       for (let attempt = 0; ; attempt++) {
         try {
           const receipt = await route.client.start(operation);

package/dist/mjs/runtime/workspace/files.mjs

@@ -2,7 +2,7 @@ import fs from "node:fs/promises";
 import { constants } from "node:fs";
 import path from "node:path";
 import { spawn } from "node:child_process";
-import { createHash } from "node:crypto";
+import { createHash, randomBytes } from "node:crypto";
 import { safeTreePath, STORAGE_LIMITS } from "./storage-wire.mjs";
 import { WorkspaceError } from "./contracts.mjs";
 const sha256 = (bytes) => createHash("sha256").update(bytes).digest("hex");
@@ -84,14 +84,17 @@ async function ensureAuthorityGitRepositoryLayout(repo) {
   }
 }
 async function durableJson(file, value) {
-  const temporary = `${file}.next`;
+  const temporary = `${file}.${randomBytes(6).toString("hex")}.next`;
   const handle = await fs.open(temporary, constants.O_WRONLY | constants.O_CREAT | constants.O_EXCL | constants.O_NOFOLLOW, 384);
   try {
     await handle.writeFile(JSON.stringify(value));
     await handle.sync();
-  } finally {
+  } catch (error) {
     await handle.close();
+    await fs.rm(temporary, { force: true });
+    throw error;
   }
+  await handle.close();
   await fs.rename(temporary, file);
   const dir = await fs.open(path.dirname(file), constants.O_RDONLY);
   try {

package/dist/types/runtime/workspace/authority.d.ts

@@ -57,7 +57,7 @@ export declare class WorkspaceAuthority {
         content: string;
         bytes: number;
     }>;
-    processes(identity: WorkspaceIdentity): Promise<{
+    processes(identity: WorkspaceIdentity): Promise<({
         workerLabel: string;
         operationId: string;
         userId: string;
@@ -80,7 +80,30 @@ export declare class WorkspaceAuthority {
         startedAt?: string;
         updatedAt?: string;
         completedAt?: string;
-    }[]>;
+    } | {
+        operationId: string;
+        userId: string;
+        sessionId: string;
+        state: string;
+        pid: null;
+        pty: boolean;
+        lane: string;
+        stdoutBytes: number;
+        stderrBytes: number;
+        stdinClosed: boolean;
+        exitCode: null;
+        signal: null;
+        reason: string;
+        workerLabel: string;
+        artifactEnvironment?: boolean;
+        mutating: boolean;
+        payloadHash: string;
+        argv?: string[];
+        cwd?: string;
+        startedAt?: string;
+        updatedAt?: string;
+        completedAt?: string;
+    })[]>;
     fileHistoryDates(identity: WorkspaceIdentity, file: string): Promise<{
         createdAt: string | null;
         updatedAt: string;
@@ -194,6 +217,11 @@ export declare class WorkspaceAuthority {
         original?: undefined;
         modified?: undefined;
     }>;
+    /** Bench state is one in-memory object with several writers on different
+     * queues (checkout work, run receipts). Writes of its file are chained so
+     * they never overlap and the write that lands last always serializes the
+     * newest state; the chain covers only the write itself, never the work
+     * around it, so a run receipt does not wait for a publication. */
     private save;
     private owned;
     private serial;
@@ -223,11 +251,7 @@ export declare class WorkspaceAuthority {
             code: string;
             message: string;
             operationId?: string;
-            commit? /** Storage for the account's outer repository, bound to no session. Its
-             * presence selects the outer-repository model: the workspace root is one
-             * repository whose files synchronize as a unit and whose project checkouts
-             * are mirrored, not published, per branch. Without it linked checkouts keep
-             * publishing per project, which the server workspace service relies on. */: string;
+            commit?: string;
         };
         mirroredHead?: string | null;
         runs: Record<string, {

package/dist/types/runtime/workspace/files.d.ts

@@ -15,6 +15,10 @@ export declare function readHostRegular(file: string, limit?: number): Promise<B
  * Restore only that directory skeleton after confirming the authority-owned
  * repository markers; refs, objects, indexes and configuration are untouched. */
 export declare function ensureAuthorityGitRepositoryLayout(repo: string): Promise<void>;
+/** Writes `value` atomically: a private temporary file, fsync, rename, directory
+ * fsync. Concurrent writers of the same file each get their own temporary name,
+ * so neither fails on the other's; the rename that lands last wins, which is why
+ * callers that share in-memory state also serialize their writes. */
 export declare function durableJson(file: string, value: unknown): Promise<void>;
 /** All Git operates in authority-created repos; NEVER workbench .git/config/index.
  * hash-object --no-filters and update-index --index-info replace git add/checkout.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ricsam/r5d-worker",
-  "version": "0.0.173",
+  "version": "0.0.175",
   "type": "module",
   "main": "./dist/mjs/main.mjs",
   "module": "./dist/mjs/main.mjs",
@@ -21,7 +21,7 @@
     "r5d-worker": "dist/mjs/main.mjs"
   },
   "dependencies": {
-    "@ricsam/r5d-api": "^0.0.173",
+    "@ricsam/r5d-api": "^0.0.175",
     "node-pty": "1.1.0",
     "zod": "^4.1.13",
     "picomatch": "^4.0.3"