fix(workflows-shared): reduce test flakiness on Windows CI

The workflows-shared vitest config was not extending the monorepo-wide
vitest.shared.ts, so tests ran with vitest's default 5s test timeout
(vs the shared 50s) and zero retries. Combined with tight vi.waitUntil
polling timeouts (500-1000ms), this caused consistent failures on slow
Windows CI runners.

- Merge vitest.shared.ts into workflows-shared vitest config (adds
  testTimeout: 50s, retry: 2, restoreMocks: true)
- Raise waitUntilLogEvent helper default timeout from 1s to 5s
- Fix bare-number vi.waitUntil(cb, 500) calls to use { timeout: 5000 }
- Bump all vi.waitUntil polling timeouts from 500-3000ms to 5000ms
  (preserving deliberate step.waitForEvent timeouts that test workflow
  timeout semantics)

Author: petebacondarwin

packages/workflows-shared/tests/binding.test.ts

@@ -24,7 +24,7 @@ function createBinding(): WorkflowBinding {
 async function waitUntilLogEvent(
 	engineStub: DurableObjectStub<Engine>,
 	event: InstanceEvent,
-	timeout = 1000
+	timeout = 5000
 ): Promise<void> {
 	await vi.waitUntil(
 		async () => {
@@ -77,7 +77,7 @@ describe("WorkflowBinding", () => {
 					const s = await instance.status();
 					return s.status === "complete";
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 		});
 
@@ -120,7 +120,7 @@ describe("WorkflowBinding", () => {
 					const s = await instance.status();
 					return s.status === "complete";
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 		});
 	});
@@ -150,7 +150,7 @@ describe("WorkflowBinding", () => {
 						const s = await instance.status();
 						return s.status === "complete";
 					},
-					{ timeout: 1000 }
+					{ timeout: 5000 }
 				);
 			}
 		});
@@ -209,7 +209,7 @@ describe("WorkflowBinding", () => {
 				const status = await instance.status();
 				return status.status === "complete";
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		expect(disposeSpy).not.toHaveBeenCalled();
@@ -386,7 +386,7 @@ describe("WorkflowHandle", () => {
 					const status = await instance.status();
 					return status.status === "complete";
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 
 			const status = await instance.status();
@@ -431,7 +431,7 @@ describe("WorkflowHandle", () => {
 					);
 					return waitStarts.length === 2;
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 
 			await instance.sendEvent({
@@ -444,7 +444,7 @@ describe("WorkflowHandle", () => {
 					const status = await instance.status();
 					return status.status === "complete";
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 
 			const status = await instance.status();
@@ -531,7 +531,7 @@ describe("WorkflowHandle", () => {
 					const s = await instance.status();
 					return s.status === "complete";
 				},
-				{ timeout: 1000 }
+				{ timeout: 5000 }
 			);
 
 			// Verify second run completed
@@ -570,7 +570,7 @@ describe("WorkflowHandle", () => {
 					const s = await instance.status();
 					return s.status === "paused";
 				},
-				{ timeout: 2000 }
+				{ timeout: 5000 }
 			);
 
 			const finalStatus = await instance.status();
@@ -608,7 +608,7 @@ describe("WorkflowHandle", () => {
 					const s = await instance.status();
 					return s.status === "paused";
 				},
-				{ timeout: 2000 }
+				{ timeout: 5000 }
 			);
 
 			await instance.resume();
@@ -618,7 +618,7 @@ describe("WorkflowHandle", () => {
 					const s = await instance.status();
 					return s.status === "complete";
 				},
-				{ timeout: 3000 }
+				{ timeout: 5000 }
 			);
 
 			const finalStatus = await instance.status();
@@ -665,7 +665,7 @@ describe("WorkflowHandle", () => {
 					const s = await instance.status();
 					return s.status === "complete";
 				},
-				{ timeout: 3000 }
+				{ timeout: 5000 }
 			);
 
 			const finalStatus = await instance.status();

packages/workflows-shared/tests/context.test.ts

@@ -50,7 +50,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_SUCCESS
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		expect(receivedAttempt).toBe(1);
@@ -86,7 +86,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_FAILURE
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		// Should have received attempts 1, 2, and 3
@@ -192,7 +192,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_SUCCESS
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		expect(receivedCtx).toMatchObject({
@@ -231,7 +231,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_SUCCESS
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		expect(receivedContexts[0]).toMatchObject({
@@ -269,7 +269,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_SUCCESS
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		expect(receivedCtx).toMatchObject({
@@ -317,7 +317,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_SUCCESS
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		// User overrides should be merged with defaults
@@ -369,7 +369,7 @@ describe("Context", () => {
 					(val) => val.event === InstanceEvent.WORKFLOW_FAILURE
 				);
 			},
-			{ timeout: 1000 }
+			{ timeout: 5000 }
 		);
 
 		// With limit: 1, the engine should execute attempt 1 + 1 retry = 2 attempts total.