ci(tests): upload Vitest and Playwright JUnit reports to Trunk Flaky Tests (#2941)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: gantoine

.github/workflows/test.yml

@@ -71,6 +71,20 @@ jobs:
       - name: Run tests
         run: pnpm test
 
+      - name: Upload test results to Trunk
+        # Run even when tests fail so flaky/failed results are still reported,
+        # but never let an upload problem fail the job.
+        if: ${{ !cancelled() }}
+        continue-on-error: true
+        uses: trunk-io/analytics-uploader@385f1ccdf345b4532dc4b6c665dd432b702b8e28 # v2.1.2
+        with:
+          # Scope to each package root. A bare **/junit.xml glob descends into
+          # node_modules, where pnpm symlinks workspace packages, and uploads
+          # every report many times over.
+          junit-paths: "apps/*/junit.xml,packages/*/junit.xml"
+          org-slug: posthog-inc
+          token: ${{ secrets.TRUNK_API_TOKEN }}
+
   integration-test:
     needs: changes
     # Fail closed: if change detection itself failed, run instead of skipping.
@@ -140,6 +154,17 @@ jobs:
         env:
           CI: true
 
+      - name: Upload test results to Trunk
+        # Run even when E2E tests fail so flaky/failed results are still
+        # reported, but never let an upload problem fail the job.
+        if: ${{ !cancelled() }}
+        continue-on-error: true
+        uses: trunk-io/analytics-uploader@385f1ccdf345b4532dc4b6c665dd432b702b8e28 # v2.1.2
+        with:
+          junit-paths: "apps/code/tests/e2e/junit.xml"
+          org-slug: posthog-inc
+          token: ${{ secrets.TRUNK_API_TOKEN }}
+
       - name: Upload Playwright report
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7.0.1
         if: failure()

.gitignore

@@ -57,6 +57,8 @@ test-results/
 *storybook.log
 .session-store.json
 .playwright-mcp
+# Trunk Flaky Tests JUnit reports (one per package, uploaded from CI)
+junit.xml
 
 # Downloaded binaries
 apps/code/resources/codex-acp/

apps/code/tests/e2e/playwright.config.ts

@@ -9,10 +9,18 @@ export default defineConfig({
   // configs (playwright.update*.config.ts), never in the general suite.
   testIgnore: "**/update*.spec.ts",
   timeout: 60000,
-  retries: isCI ? 2 : 0,
+  // No retries: Trunk Flaky Tests needs raw pass/fail results to detect flakes.
+  retries: 0,
   // Must run serially - Electron app has single instance lock
   workers: 1,
-  reporter: isCI ? [["github"], ["html", { open: "never" }]] : [["list"]],
+  // junit.xml (resolved next to this config) is uploaded to Trunk in CI.
+  reporter: isCI
+    ? [
+        ["junit", { outputFile: "junit.xml" }],
+        ["github"],
+        ["html", { open: "never" }],
+      ]
+    : [["junit", { outputFile: "junit.xml" }], ["list"]],
   outputDir: "../playwright-results",
   use: {
     trace: "retain-on-failure",

apps/code/vitest.config.ts

@@ -1,6 +1,7 @@
 import path from "node:path";
 import react from "@vitejs/plugin-react";
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 import { rendererAliases } from "./vite.shared.mjs";
 
 export default defineConfig({
@@ -10,6 +11,7 @@ export default defineConfig({
   },
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "jsdom",
     setupFiles: ["./src/shared/test/setup.ts"],
     exclude: ["**/node_modules/**", "**/dist/**", "tests/e2e/**"],

apps/mobile/vitest.config.ts

@@ -1,11 +1,13 @@
 import path from "node:path";
 import react from "@vitejs/plugin-react";
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 export default defineConfig({
   plugins: [react()],
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "node",
     setupFiles: ["./src/test/setup.ts"],
     exclude: ["**/node_modules/**", "**/dist/**"],

packages/agent/vitest.config.ts

@@ -1,5 +1,6 @@
 import { resolve } from "node:path";
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 export default defineConfig({
   resolve: {
@@ -9,6 +10,7 @@ export default defineConfig({
   },
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "node",
     include: ["src/**/*.test.ts"],
     exclude: ["**/node_modules/**", "**/dist/**"],

packages/electron-trpc/vitest.config.ts

@@ -2,11 +2,13 @@
 import path from "node:path";
 import { fileURLToPath } from "node:url";
 import { defineConfig } from "vite";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
 export default defineConfig({
   test: {
+    ...trunkTestOptions,
     coverage: {
       all: true,
       include: ["src/**/*"],

packages/git/vitest.config.ts

@@ -1,8 +1,10 @@
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 export default defineConfig({
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "node",
     include: ["src/**/*.test.ts"],
     exclude: ["**/node_modules/**", "**/.git/**"],

packages/shared/vitest.config.ts

@@ -1,8 +1,10 @@
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 export default defineConfig({
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "node",
     include: ["src/**/*.test.ts"],
     exclude: ["**/node_modules/**", "**/dist/**"],

packages/ui/vitest.config.ts

@@ -1,6 +1,7 @@
 import { fileURLToPath } from "node:url";
 import react from "@vitejs/plugin-react";
 import { defineConfig } from "vitest/config";
+import { trunkTestOptions } from "../../vitest.config.base";
 
 export default defineConfig({
   plugins: [react()],
@@ -20,6 +21,7 @@ export default defineConfig({
   },
   test: {
     globals: true,
+    ...trunkTestOptions,
     environment: "jsdom",
     setupFiles: ["./src/test/setup.ts"],
     include: ["src/**/*.test.ts", "src/**/*.test.tsx"],