Merge branch 'main' into renovate/zod-3.x-lockfile

e2e/python/run.py

@@ -70,11 +70,20 @@ def gen_test_cases(n: int) -> list[MyTestCase]:
 
 if __name__ == "__main__":
   run_test_suite(
-      id="my-test-suite",
+      id="python-e2e-test-suite-1",
       fn=test_fn,
       test_cases=gen_test_cases(4),
       evaluators=[
           HasAllSubstrings(),
           IsFriendly(),
       ],
   )
+  run_test_suite(
+      id="python-e2e-test-suite-2",
+      fn=test_fn,
+      test_cases=gen_test_cases(10),
+      evaluators=[
+          HasAllSubstrings(),
+          IsFriendly(),
+      ],
+  )

e2e/typescript/run.ts

@@ -78,11 +78,20 @@ function genTestCases(n: number): MyTestCase[] {
 }
 
 (async () => {
-  await runTestSuite<MyTestCase, string>({
-    id: 'my-test-suite',
-    fn: testFn,
-    testCaseHash: ['input'],
-    testCases: genTestCases(4),
-    evaluators: [new HasAllSubstrings(), new IsFriendly()],
-  });
+  await Promise.allSettled([
+    runTestSuite<MyTestCase, string>({
+      id: 'typescript-e2e-test-suite-1',
+      fn: testFn,
+      testCaseHash: ['input'],
+      testCases: genTestCases(4),
+      evaluators: [new HasAllSubstrings(), new IsFriendly()],
+    }),
+    runTestSuite<MyTestCase, string>({
+      id: 'typescript-e2e-test-suite-2',
+      fn: testFn,
+      testCaseHash: ['input'],
+      testCases: genTestCases(10),
+      evaluators: [new HasAllSubstrings(), new IsFriendly()],
+    }),
+  ]);
 })();

src/handlers/testing/align/util/hono-app.ts

@@ -47,7 +47,7 @@ export function createHonoApp(sessionManager: SessionManager): Hono {
         event: z.object({
           message: z.string(),
           traceId: z.string(),
-          timestamp: z.string(),
+          timestamp: z.string().datetime({ offset: true }),
           properties: z.unknown(),
         }),
       }),

src/handlers/testing/exec/index.ts

@@ -160,17 +160,15 @@ export async function exec(args: {
         .finally(async () => {
           runManager.setEndTime();
 
-          if (args.slackWebhookUrl) {
-            // Post to Slack if a webhook URL has been set.
-            // Note: this doesn't happen in cleanup() because that
-            // function is also called during unexpected termination
-            // and makes a best effort to clean up resources, whereas
-            // we only want to post to Slack if the command has
-            // successfully executed.
-            await runManager.postSlackMessage({
-              webhookUrl: args.slackWebhookUrl,
-            });
-          }
+          // Post comments to Slack and GitHub (if set up)
+          // Note: this doesn't happen in cleanup() because that
+          // function is also called during unexpected termination
+          // and makes a best effort to clean up resources, whereas
+          // we only want to post comments if the command has
+          // successfully executed.
+          await runManager.postComments({
+            slackWebhookUrl: args.slackWebhookUrl,
+          });
 
           await cleanup();
         });