From a19f53b2b45027f0b1a531d11527f8a45d38cc53 Mon Sep 17 00:00:00 2001
From: amosblomqvist <amosblomqvist@gmail.com>
Date: Mon, 26 Oct 2026 02:57:33 +0200
Subject: [PATCH] test: preserve PDF history guards alongside canonical result
 delivery

---
 tests/jobs.test.ts | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/tests/jobs.test.ts b/tests/jobs.test.ts
index d8b877a..be5478b 100644
--- a/tests/jobs.test.ts
+++ b/tests/jobs.test.ts
@@ -801,6 +801,7 @@ describe("pdf runner (sources v0)", () => {
 			const toolCallId = `call-pdf-relative-${mode}`;
 			const input = { ...PDF_INPUT, path: relativePath };
 			const messages = nativeToolMessages("read this pdf", "pdf", toolCallId, input);
+			const before = structuredClone(messages);
 			chatStore.saveMessages(chat.id, messages);
 			const job = jobStore.createJob({ kind: "pdf", chatId: chat.id, toolCallId });
 			trackJob(job.id);
@@ -813,15 +814,31 @@ describe("pdf runner (sources v0)", () => {
 				pdfModelOverride.current = undefined;
 			}
 
+			// Worker completion is not the delivery barrier: wait for canonical
+			// acceptance before asserting the history prefix and appended result.
+			await waitFor(() => jobStore.getJob(job.id)?.delivery?.status === "delivered", 8000, "canonical PDF delivery acknowledgement");
 			const done = jobStore.getJob(job.id)!;
 			assert.equal(done.status, "done");
+			assert.deepEqual(done.delivery, { kind: "result", status: "delivered" });
 			const result = done.result as PdfResult;
 			assert.equal(result.path, path.join(home, relativePath));
 			assert.match(result.brief, /integral rule/);
 			assert.deepEqual(result.pagesRead, [1, 2]);
 			assert.deepEqual(result.pagesConsulted, [1, 2]);
 			assert.equal(delivery.steers.length, 1);
-			assert.deepEqual(chatStore.getChat(chat.id)!.messages, messages, "tool-call arguments remain as supplied");
+			assert.equal(input.path, relativePath, "dispatch input remains relative");
+			const persisted = chatStore.getChat(chat.id)!.messages;
+			assert.deepEqual(persisted.slice(0, before.length), before, "original history and tool-call arguments remain as supplied");
+			const steer = delivery.steers[0]!;
+			assert.equal(steer.id, `steer_${job.id}`);
+			assert.equal(steer.role, "custom");
+			assert.equal(steer.customType, "subagent-steer");
+			const details = steer.details as { jobId: string; kind: string; toolCallId: string; result: PdfResult };
+			assert.equal(details.jobId, job.id);
+			assert.equal(details.kind, "pdf");
+			assert.equal(details.toolCallId, toolCallId);
+			assert.deepEqual(details.result, result, "the canonical steer carries the resolved PDF result");
+			assert.deepEqual(persisted.slice(before.length), [steer], "exactly one captured result is appended after the unchanged history");
 		});
 	}