ReviewOS

also looking at this

stacks/drone

test: retain execution lifetime failure observations

#62
Draft chris wants to merge codex/execution-lifetime-observations into main
10 files +174 -64

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

tests/unit/public-safety-execution-lifetime.test.tsmodified+1-0
Changes to tests/unit/public-safety-execution-lifetime.test.ts
@@ -25,6 +25,7 @@ for (const mode of ['completed', 'canceled', 'quarantine']) test(`native executi
2525 console.error('LIFETIME_FIXTURE_DEADLINE', JSON.stringify({ mode, pid: running.pid, timeoutMs: 90000, elapsedMs: performance.now() - began, requestedSignal: 'SIGKILL' }))
2626 }, 90000)
2727 try {
28 console.log(JSON.stringify({ oracle: 'lifetime-fixture-start', mode, pid: running.pid, timeoutMs: 90000 }))
2829 const [code, stdout, stderr] = await Promise.all([running.exited, new Response(running.stdout).text(), new Response(running.stderr).text()])
2930 // eslint-disable-next-line no-console
3031 for (const line of stdout.split('\n')) if (line.startsWith('{"oracle":')) console.log(line)