Skip to content

Commit ffc52b4

Browse files
committed
fix(sdk,webapp): drop stale lastOutTimestamp from snapshot test fixtures
vitest was happy with the extra property at runtime so the local suite passed, but tsc on the typecheck job (strict on object literals) caught three test fixtures still setting the field after it was removed from ChatSnapshotV1 in the resume-cursor commit.
1 parent 20fb74a commit ffc52b4

3 files changed

Lines changed: 0 additions & 3 deletions

File tree

apps/webapp/test/chat-snapshot-integration.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ function makeSnapshot(opts: { messages?: UIMessage[]; lastOutEventId?: string }
4949
},
5050
],
5151
lastOutEventId: opts.lastOutEventId ?? "evt-42",
52-
lastOutTimestamp: 1_700_000_000_500,
5352
};
5453
}
5554

apps/webapp/test/replay-after-crash.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,6 @@ describe("replay after crash (MinIO + SDK helpers)", () => {
267267
{ id: "a-1", role: "assistant", parts: [{ type: "text", text: "stale-assistant" }] },
268268
],
269269
lastOutEventId: "evt-prev",
270-
lastOutTimestamp: 1_700_000_000_500,
271270
};
272271

273272
// Use the SDK's own writer to lay the snapshot down, then swap

packages/trigger-sdk/test/chat-snapshot.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ function buildSnapshot(count = 1): ChatSnapshotV1 {
2929
parts: [{ type: "text" as const, text: `hello ${i}` }],
3030
})),
3131
lastOutEventId: "evt-42",
32-
lastOutTimestamp: 2_000_000,
3332
};
3433
}
3534

0 commit comments

Comments
 (0)