@sudobility/testomniac_runner 0.0.173 → 0.0.174

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sudobility/testomniac_runner",
-  "version": "0.0.173",
+  "version": "0.0.174",
   "publishConfig": {
     "access": "public"
   },

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 },
@@ -226,8 +233,20 @@ export class RunnerManager {
       // clobber the server-side closeout. This stays purely as a backstop for
       // the rare case the inner service threw before owning closeout.
       try {
+        // 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,
         });