@workos/quickstudy 0.0.1 → 0.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@workos/quickstudy",
-  "version": "0.0.1",
+  "version": "0.1.0",
   "description": "An open-source, company-agnostic eval harness for measuring whether coding agents can complete real integration tasks, across an agent x feature x framework x surface matrix.",
   "license": "MIT",
   "type": "module",

package/src/build-info.generated.ts

@@ -5,7 +5,11 @@ export const EMBEDDED_BUILD_INFO: {
   dirty: boolean | null;
   sourceHash: string | null;
 } = {
-  version: "0.1.0",
+  // release-please's generic updater bumps this semver on the release PR
+  // (the `x-release-please-version` marker anchors the replacement). It is
+  // restored as-is after a binary build (scripts/build-binary.ts), so the
+  // committed source `--version` stays in lockstep with package.json.
+  version: "0.1.0", // x-release-please-version
   revision: null,
   dirty: null,
   sourceHash: null,

package/src/isolation/docker.ts

@@ -93,6 +93,21 @@ async function mustDocker(args: string[], opts: { timeoutMs?: number; stdin?: st
   return result;
 }
 
+/**
+ * `docker rm -f` stderr that still means "the container is gone": it was
+ * already removed, or another caller's removal is in flight (a deadline kill
+ * racing the attempt's final teardown) and the daemon will finish that one.
+ */
+export const CONTAINER_GONE_STDERR = /no such container|already in progress/i;
+
+/** `docker rm -f <id>`, idempotent per CONTAINER_GONE_STDERR. */
+async function removeContainer(id: string): Promise<void> {
+  const result = await runDocker(["rm", "-f", id]);
+  if (result.exitCode !== 0 && !CONTAINER_GONE_STDERR.test(result.stderr)) {
+    throw new DockerError(["rm", "-f", id], result.exitCode, result.stderr);
+  }
+}
+
 /** Preflight: is a Docker daemon reachable at all? */
 export async function dockerDaemonReachable(): Promise<boolean> {
   const result = await runDocker(["version", "--format", "{{.Server.Version}}"], { timeoutMs: 10_000 });
@@ -277,13 +292,7 @@ export async function createAttemptContainer(opts: CreateAttemptContainerOptions
         await rm(scratch, { recursive: true, force: true });
       }
     },
-    teardown: async () => {
-      const result = await runDocker(["rm", "-f", id]);
-      // Idempotent: an already-removed container is success, not an error.
-      if (result.exitCode !== 0 && !/no such container/i.test(result.stderr)) {
-        throw new DockerError(["rm", "-f", id], result.exitCode, result.stderr);
-      }
-    },
+    teardown: () => removeContainer(id),
   };
 
   try {
@@ -510,12 +519,7 @@ export async function startEgressSidecar(opts: StartEgressSidecarOptions): Promi
       const result = await mustDocker(["logs", id]);
       return result.stdout;
     },
-    async teardown() {
-      const result = await runDocker(["rm", "-f", id]);
-      if (result.exitCode !== 0 && !/no such container/i.test(result.stderr)) {
-        throw new DockerError(["rm", "-f", id], result.exitCode, result.stderr);
-      }
-    },
+    teardown: () => removeContainer(id),
   };
   return sidecar;
 }
@@ -620,12 +624,7 @@ export async function startMcpSidecar(opts: StartMcpSidecarOptions): Promise<Mcp
       const state = await runDocker(["inspect", "--format", "{{.State.Running}}", id]);
       return state.exitCode === 0 && state.stdout.trim() === "true";
     },
-    async teardown() {
-      const result = await runDocker(["rm", "-f", id]);
-      if (result.exitCode !== 0 && !/no such container/i.test(result.stderr)) {
-        throw new DockerError(["rm", "-f", id], result.exitCode, result.stderr);
-      }
-    },
+    teardown: () => removeContainer(id),
   };
 }