TDT-129 – Always run postrun, even on failure (#368)

* Vibe-code a fix

https://www.notion.so/postrun-lifecycle-script-doesn-t-run-test-failure-23a6adb97c888026a58bc62804a1bc1b?v=1f46adb97c88807ca025000c05b7b897&source=copy_link

* Minimize changes

Author: ericclemmons

agent/index.js

@@ -196,7 +196,8 @@ class TestDriverAgent extends EventEmitter2 {
     }
 
     await this.summarize(error.message);
-    return await this.exit(true);
+    // Always run postrun lifecycle script, even for fatal errors
+    return await this.exit(true, false, true);
   }
 
   // creates a new "thread" in which the AI is given an error

interfaces/cli/lib/base.js

@@ -101,9 +101,14 @@ class BaseCommand extends Command {
     logger.createMarkdownLogger(this.agent.emitter);
 
     // Handle exit events
-    this.agent.emitter.on("error:fatal", (error) => {
+    this.agent.emitter.on("error:fatal", async (error) => {
       console.error("Fatal error:", error);
-      process.exit(1);
+      // Call the agent's exit method to ensure postrun lifecycle script is executed
+      if (this.agent) {
+        await this.agent.exit(true, false, true);
+      } else {
+        process.exit(1);
+      }
     });
 
     this.agent.emitter.on("exit", (exitCode) => {

testdriver/behavior/lifecycle/postrun.yaml

@@ -0,0 +1,8 @@
+version: 6.0.1-canary.4f31816.0
+session: 68767e31eb9cc3d8f639b8e0
+steps:
+  - prompt: "run"
+    commands:
+      - command: exec
+        lang: pwsh
+        code: Write-Output "postrun.yaml"

testdriver/behavior/lifecycle/prerun.yaml

@@ -0,0 +1,8 @@
+version: 6.0.1-canary.4f31816.0
+session: 68767e31eb9cc3d8f639b8e0
+steps:
+  - prompt: "run"
+    commands:
+      - command: exec
+        lang: pwsh
+        code: Write-Output "prerun.yaml"

testdriver/behavior/lifecycle/provision.yaml

@@ -0,0 +1,8 @@
+version: 6.0.1-canary.4f31816.0
+session: 68767e31eb9cc3d8f639b8e0
+steps:
+  - prompt: "run"
+    commands:
+      - command: exec
+        lang: pwsh
+        code: Write-Output "provision.yaml"