Merge pull request #1326 from paperclipai/ci/pr-e2e-tests

ci: run e2e tests on PRs
fix(ci): use --frozen-lockfile in e2e workflow
2026-03-20 15:47:21 -07:00 · 2026-03-20 14:20:46 -07:00 · 2026-03-20 13:49:03 -07:00 · 2026-03-19 19:19:32 -07:00
2 changed files with 70 additions and 1 deletions
--- a/.github/workflows/pr-e2e.yml
+++ b/.github/workflows/pr-e2e.yml
@@ -0,0 +1,69 @@
+name: PR E2E
+
+on:
+  pull_request:
+    branches:
+      - master
+
+concurrency:
+  group: pr-e2e-${{ github.event.pull_request.number }}
+  cancel-in-progress: true
+
+jobs:
+  e2e:
+    runs-on: ubuntu-latest
+    timeout-minutes: 30
+
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Setup pnpm
+        uses: pnpm/action-setup@v4
+        with:
+          version: 9.15.4
+
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: pnpm
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Build
+        run: pnpm build
+
+      - name: Install Playwright
+        run: npx playwright install --with-deps chromium
+
+      - name: Generate Paperclip config
+        run: |
+          mkdir -p ~/.paperclip/instances/default
+          cat > ~/.paperclip/instances/default/config.json << 'CONF'
+          {
+            "$meta": { "version": 1, "updatedAt": "2026-01-01T00:00:00.000Z", "source": "onboard" },
+            "database": { "mode": "embedded-postgres" },
+            "logging": { "mode": "file" },
+            "server": { "deploymentMode": "local_trusted", "host": "127.0.0.1", "port": 3100 },
+            "auth": { "baseUrlMode": "auto" },
+            "storage": { "provider": "local_disk" },
+            "secrets": { "provider": "local_encrypted", "strictMode": false }
+          }
+          CONF
+
+      - name: Run e2e tests
+        env:
+          PAPERCLIP_E2E_SKIP_LLM: "true"
+        run: pnpm run test:e2e
+
+      - name: Upload Playwright report
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: playwright-report
+          path: |
+            tests/e2e/playwright-report/
+            tests/e2e/test-results/
+          retention-days: 14
--- a/tests/e2e/playwright.config.ts
+++ b/tests/e2e/playwright.config.ts
@@ -25,7 +25,7 @@ export default defineConfig({
  webServer: {
    command: `pnpm paperclipai run`,
    url: `${BASE_URL}/api/health`,
-    reuseExistingServer: !!process.env.CI,
+    reuseExistingServer: !process.env.CI,
    timeout: 120_000,
    stdout: "pipe",
    stderr: "pipe",
Author	SHA1	Message	Date
Devin Foley	036e2b52db	Merge pull request #1326 from paperclipai/ci/pr-e2e-tests ci: run e2e tests on PRs	2026-03-20 15:47:21 -07:00
Devin Foley	da9b31e393	fix(ci): use --frozen-lockfile in e2e workflow Align with e2e.yml and ensure CI tests exactly the committed dependency tree. The pr-policy job already blocks lockfile changes in PRs, so frozen-lockfile is safe here. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-20 14:20:46 -07:00
Devin Foley	652fa8223e	fix: invert reuseExistingServer and remove CI="" workaround The playwright.config.ts had `reuseExistingServer: !!process.env.CI` which meant CI would reuse (expect) an existing server while local dev would start one. This is backwards — in CI Playwright should manage the server, and in local dev you likely already have one running. Flip to `!process.env.CI` and remove the `CI: ""` env override from the workflow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-20 13:49:03 -07:00
Devin Foley	df8cc8136f	ci: add e2e tests to PR checks Add a PR E2E workflow that runs the Playwright onboarding test on every PR targeting master. Generates a minimal config file and lets Playwright manage the server lifecycle. Runs in skip_llm mode so no secrets are required. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-03-19 19:19:32 -07:00