fix: ensure launched servers in local executor are killed upon timeout

Whenever an eval times out, the abort signal fires but we weren't
using it for the serve execution— so it was never properly killed.

Author: devversion

examples/environments/remote_env/fake-executor.ts

@@ -64,6 +64,7 @@ export class FakeRemoteExecutor implements Executor {
     appDirectoryPath: string,
     rootPromptDef: RootPromptDefinition,
     progress: ProgressLogger,
+    abortSignal: AbortSignal,
     logicWhileServing: (serveUrl: string) => Promise<T>,
   ): Promise<T> {
     // Start serving of the app.

runner/orchestration/executors/executor.ts

@@ -65,6 +65,9 @@ export const executorSchema = z.object({
         z.string().describe('Path to the application directory'),
         z.custom<RootPromptDefinition>().describe('Root prompt definition'),
         z.custom<ProgressLogger>().describe('Progress logger'),
+        z
+          .custom<AbortSignal>()
+          .describe('Abort Signal to fire when the server should be canceled.'),
         z
           .function(
             z.tuple([z.string().describe('URL of the running server')]),

runner/orchestration/executors/local-executor.ts

@@ -178,6 +178,7 @@ export class LocalExecutor implements Executor {
     appDirectoryPath: string,
     rootPromptDef: RootPromptDefinition,
     progress: ProgressLogger,
+    abortSignal: AbortSignal,
     logicWhileServing: (serveUrl: string) => Promise<ServeTestingResult>,
   ): Promise<ServeTestingResult | null> {
     // Serve testing is explicitly disabled.
@@ -190,6 +191,7 @@ export class LocalExecutor implements Executor {
       rootPromptDef,
       appDirectoryPath,
       progress,
+      abortSignal,
       logicWhileServing,
     );
   }

runner/orchestration/serve-testing-worker.ts

@@ -37,6 +37,7 @@ export async function serveAndTestApp(
       appDirectoryPath,
       rootPromptDef,
       progress,
+      abortSignal,
       async serveUrl => {
         progress.log(
           rootPromptDef,

runner/run-cli.ts

@@ -96,6 +96,7 @@ async function runApp(options: Options) {
       rootPromptDef,
       directory,
       new NoopProgressLogger(),
+      new AbortSignal(),
       async url => {
         console.log();
         console.log(formatTitleCard(`🎉 App is up and running at ${url}`));

runner/workers/serve-testing/serve-app.ts

@@ -9,12 +9,13 @@ export async function serveApp<T>(
   rootPromptDef: RootPromptDefinition,
   appDirectoryPath: string,
   progress: ProgressLogger,
+  abortSignal: AbortSignal,
   logicWhileServing: (serveUrl: string) => Promise<T>,
 ): Promise<T> {
   let serveProcess: ChildProcess | null = null;
 
   try {
-    serveProcess = exec(serveCommand, {cwd: appDirectoryPath});
+    serveProcess = exec(serveCommand, {cwd: appDirectoryPath, signal: abortSignal});
     progress.log(
       rootPromptDef,
       'eval',