@sudobility/testomniac_runner 0.0.172 → 0.0.174

package/bun.lock

@@ -8,8 +8,8 @@
         "@noble/curves": "^1.0.0",
         "@noble/hashes": "^1.0.0",
         "@sudobility/signic_sdk": "^0.1.7",
-        "@sudobility/testomniac_runner_service": "^0.1.166",
-        "@sudobility/testomniac_types": "^0.0.80",
+        "@sudobility/testomniac_runner_service": "^0.1.167",
+        "@sudobility/testomniac_types": "^0.0.81",
         "hono": "^4.7.0",
         "jose": "^6.1.2",
         "openai": "^6.7.0",
@@ -172,9 +172,9 @@
 
     "@sudobility/signic_sdk": ["@sudobility/signic_sdk@0.1.7", "", {}, "sha512-5XSgHSVsmyrMQ/ui1nDywwzt9dbRCsaeJ5tX6mKw2ZXbTZ82OsMr+dqDyV9XV3pfy7IHRIZq73af5KBamx72Fw=="],
 
-    "@sudobility/testomniac_runner_service": ["@sudobility/testomniac_runner_service@0.1.166", "", { "peerDependencies": { "@sudobility/testomniac_types": "^0.0.80" } }, "sha512-FPSuTOSAQN77SAnhxw/FAXk7HZp6yshmcmDjo8DTt3bRzx2xMzt0cmoftjoIUtfapmFLXr68GpqJs6+H6KbgHQ=="],
+    "@sudobility/testomniac_runner_service": ["@sudobility/testomniac_runner_service@0.1.167", "", { "peerDependencies": { "@sudobility/testomniac_types": "^0.0.81" } }, "sha512-6g1yTH1gQ3DJvIjlivDtgD0nSDcnV98o+6x1x1hvn2BwkkNgEulhbO/G3D4UVdB8jjIQ3hQYQEypgYsfHIh0DA=="],
 
-    "@sudobility/testomniac_types": ["@sudobility/testomniac_types@0.0.80", "", { "peerDependencies": { "@sudobility/types": "^1.9.62" } }, "sha512-BugxR1yC5pSN97lN+bMz+mD87or+7dNHX+Qsof2QQHrYszPpuKjWNTYwQW2wma7ekUqWgykOU3iNDq+wC0KNQw=="],
+    "@sudobility/testomniac_types": ["@sudobility/testomniac_types@0.0.81", "", { "peerDependencies": { "@sudobility/types": "^1.9.62" } }, "sha512-jYeE1TwSFNhY1j+ZdVSGno61+5uAgMg0AVeyUkKF3UAXyUa5eABK46mPZb2fa3MvgoAyDWOWPvn8WsmoInfFcA=="],
 
     "@sudobility/types": ["@sudobility/types@1.9.61", "", {}, "sha512-SODGpstB/iKfK3H/4BvJx/FBcc1h3gutUjGotyxN19VnOfWyzaDoEmW7eyoxOAYhZyXMXagSiii+NIEZvuxKog=="],
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sudobility/testomniac_runner",
-  "version": "0.0.172",
+  "version": "0.0.174",
   "publishConfig": {
     "access": "public"
   },
@@ -24,8 +24,8 @@
     "@noble/curves": "^1.0.0",
     "@noble/hashes": "^1.0.0",
     "@sudobility/signic_sdk": "^0.1.7",
-    "@sudobility/testomniac_runner_service": "^0.1.166",
-    "@sudobility/testomniac_types": "^0.0.80",
+    "@sudobility/testomniac_runner_service": "^0.1.167",
+    "@sudobility/testomniac_types": "^0.0.81",
     "hono": "^4.7.0",
     "jose": "^6.1.2",
     "openai": "^6.7.0",

package/src/runner-manager.ts

@@ -112,7 +112,14 @@ export class RunnerManager {
             "runner not found for pending run"
           );
           try {
-            await api.completeTestRun(pendingRun.id, { status: "failed" });
+            // Use /scan/end for closeout (not the legacy completeTestRun). The
+            // run was never claimed/started, so there is no bundle work to
+            // cancel — just mark the test run failed.
+            await api.scanEnd({
+              testRunId: pendingRun.id,
+              status: "failed",
+              runDetection: false,
+            });
           } catch (err) {
             logger.error(
               { err, runId: pendingRun.id },
@@ -220,9 +227,28 @@ export class RunnerManager {
       logger.info({ runId: params.runId }, "run completed successfully");
     } catch (error) {
       logger.error({ err: error, runId: params.runId }, "claimed run failed");
+      // runTestRun() already closes a failed run out through /scan/end on its
+      // own error path. Use scanEnd here too (idempotent) rather than the legacy
+      // completeTestRun, which would reintroduce the old completion path and
+      // clobber the server-side closeout. This stays purely as a backstop for
+      // the rare case the inner service threw before owning closeout.
       try {
-        await api.completeTestRun(params.runId, {
+        // Best-effort: resolve the bundle run so the backstop closeout can also
+        // cancel pending interaction runs and close surface/bundle runs — not
+        // just the test_run. Falls back to test-run-only closeout if the lookup
+        // fails (still better than leaving it open).
+        let bundleRunId: number | undefined;
+        try {
+          const testRun = await api.getTestRun(params.runId);
+          bundleRunId = testRun?.testSurfaceBundleRunId ?? undefined;
+        } catch {
+          // ignore — fall back to test-run-only closeout
+        }
+        await api.scanEnd({
+          testRunId: params.runId,
+          bundleRunId,
           status: "failed",
+          runDetection: false,
         });
       } catch (completeError) {
         logger.error(