From 92adb87d5b68e93e3e661a14dc7b98badfbaba68 Mon Sep 17 00:00:00 2001 From: nickyleach <331803+nickyleach@users.noreply.github.com> Date: Thu, 10 Sep 2026 18:39:31 +0000 Subject: [PATCH] test(server): add the two new actor fields to six queued-comment test literals The QueuedCommentActor type now declares five fields: actorType, actorId, agentId, runId, and agentApiKeyId. Six actor literals in this test file still declared only three fields, so the type check failed. Add runId: null and agentApiKeyId: null to each of the six literals. This matches the shape that the other literals in this file already use. No production file changes. Co-authored-by: Paperclip --- .../adapters/queued-comment-postgres.test.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/server/src/modules/wake-queue/adapters/queued-comment-postgres.test.ts b/server/src/modules/wake-queue/adapters/queued-comment-postgres.test.ts index 7f3fa19214..f29482b826 100644 --- a/server/src/modules/wake-queue/adapters/queued-comment-postgres.test.ts +++ b/server/src/modules/wake-queue/adapters/queued-comment-postgres.test.ts @@ -389,7 +389,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { // context; that single value binds every read and write for the // whole transaction, so it alone must decide what is visible. issue: { id: issueId, companyId: otherCompanyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, commentId, queueId: wakeId, targetRunId, @@ -416,7 +416,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { const queue = await issueLock.withLockedQueue( { issue: { id: issueId, companyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, queueId: wakeId, }, async (locked, transaction) => { @@ -425,7 +425,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { // below. return transaction.buildQueueSnapshot({ issue: { id: issueId, companyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, wake: locked.wake, state: locked.state, queueRun: locked.queueRun, @@ -462,7 +462,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { const peeked = await issueLock.withLockedQueue( { issue: { id: issueId, companyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, queueId: wakeId, }, async (locked) => locked.queue, @@ -470,7 +470,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { const result = await issueLock.steerQueuedWakeComment({ issue: { id: issueId, companyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, commentId: firstCommentId, queueId: wakeId, targetRunId, @@ -492,7 +492,7 @@ describeEmbeddedPostgres("queued-comment postgres adapter", () => { steerNativeSessionMock.mockResolvedValueOnce({ turnId: "turn-2" }); const second = await issueLock.steerQueuedWakeComment({ issue: { id: issueId, companyId, assigneeAgentId: agentId, executionRunId: null }, - actor: { actorType: "user", actorId: "user-1", agentId: null }, + actor: { actorType: "user", actorId: "user-1", agentId: null, runId: null, agentApiKeyId: null }, commentId: secondCommentId, queueId: wakeId, targetRunId,