fix(async-jobs): stop await-tool timers from hanging tests

src/resources/extensions/async-jobs/await-tool.test.ts

@@ -20,6 +20,8 @@ test("await_job returns immediately when no running jobs exist", async () => {
 	const result = await tool.execute("tc1", {}, noopSignal, () => {}, undefined as never);
 	const text = getTextFromResult(result);
 	assert.match(text, /No running background jobs/);
+
+	manager.shutdown();
 });
 
 test("await_job returns immediately when all watched jobs are already completed", async () => {
@@ -36,6 +38,8 @@ test("await_job returns immediately when all watched jobs are already completed"
 	const text = getTextFromResult(result);
 	assert.match(text, /fast-job/);
 	assert.match(text, /completed/);
+
+	manager.shutdown();
 });
 
 test("await_job returns on timeout when jobs are still running", async () => {
@@ -165,3 +169,23 @@ test("unawaited jobs still get follow-up delivery (#2248)", async () => {
 
 	manager.shutdown();
 });
+
+test("completed jobs use unref'd eviction timers so await-tool tests can exit cleanly", async () => {
+	const manager = new AsyncJobManager();
+
+	const jobId = manager.register("bash", "completed-job", async () => "done");
+	const job = manager.getJob(jobId)!;
+	await job.promise;
+
+	const timers = (manager as unknown as {
+		evictionTimers: Map<string, ReturnType<typeof setTimeout>>;
+	}).evictionTimers;
+	const timer = timers.get(jobId);
+
+	assert.ok(timer, "Expected eviction timer for completed job");
+	if (typeof timer === "object" && "hasRef" in timer) {
+		assert.equal(timer.hasRef(), false, "Eviction timer should not keep the process alive");
+	}
+
+	manager.shutdown();
+});

src/resources/extensions/async-jobs/job-manager.ts

@@ -186,6 +186,7 @@ export class AsyncJobManager {
 			this.evictionTimers.delete(id);
 			this.jobs.delete(id);
 		}, this.evictionMs);
+		if (typeof timer === "object" && "unref" in timer) timer.unref();
 
 		this.evictionTimers.set(id, timer);
 	}