Skip to content

feat(web): compare deploy suite runs#2399

Draft
james-elicx wants to merge 4 commits into
mainfrom
codex/compatibility-deploy-suite-compare
Draft

feat(web): compare deploy suite runs#2399
james-elicx wants to merge 4 commits into
mainfrom
codex/compatibility-deploy-suite-compare

Conversation

@james-elicx

@james-elicx james-elicx commented Jun 29, 2026

Copy link
Copy Markdown
Member

Summary

  • add a compatibility-page panel for comparing two Next.js deploy-suite workflow run IDs
  • add /api/compatibility/compare to download deploy-suite-report artifacts and group assertion-level changes
  • default the baseline input from the latest ingested main deploy-suite run
  • require a Worker-side GITHUB_TOKEN because GitHub artifact archive downloads return 401 without authentication
  • list public run artifacts without auth so an under-scoped token cannot break the public artifact-listing step
  • polish the comparison panel with a clearer control layout, visible Compare runs button, reset-baseline affordance, and clearer result cards

Tests

  • vp check apps/web/app/api/compatibility/compare/route.ts apps/web/app/compatibility/deploy-suite-compare.tsx apps/web/app/compatibility/page.tsx apps/web/app/lib/db/client.ts apps/web/worker/index.ts apps/web/wrangler.jsonc tests/web-compatibility-compare.test.ts tests/web-compatibility-failures.test.ts
  • vp check apps/web/app/compatibility/deploy-suite-compare.tsx apps/web/app/compatibility/page.tsx
  • vp check apps/web/app/api/compatibility/compare/route.ts tests/web-compatibility-compare.test.ts
  • vp test run tests/web-compatibility-failures.test.ts tests/web-compatibility-compare.test.ts
  • vp run @apps/web#build:vinext
  • local browser smoke of /compatibility comparison panel

Note: local web build warned that COMPAT_INGEST_SECRET and GITHUB_TOKEN are not configured, which is expected in this worktree.

@pkg-pr-new

pkg-pr-new Bot commented Jun 29, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@vinext/cloudflare@2399
npm i https://pkg.pr.new/vinext@2399

commit: 37e69d8

@github-actions

Copy link
Copy Markdown
Contributor
Example Preview Production Original
app-router-cloudflare preview production
pages-router-cloudflare preview production
app-router-playground preview production original
realworld-api-rest preview production
nextra-docs-template preview production
benchmarks preview production
hackernews preview production original
workers-cache preview production
web preview production

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Performance benchmarks

Compared 37e69d8 against base 4a88a9c using alternating same-runner rounds. Next.js was unchanged and skipped.

0 improved · 0 regressed · 6 within ±1.5%

Scenario Framework Baseline Current Change
Client bundle size (gzip) vinext 124.0 KB 124.0 KB ⚫ +0.0%
Client entry size (gzip) vinext 118.8 KB 118.8 KB ⚫ +0.0%
Dev server cold start vinext 2.52 s 2.52 s ⚫ -0.0%
Production build time vinext 3.03 s 3.02 s ⚫ -0.5%
RSC entry closure size (gzip) vinext 95.2 KB 95.2 KB ⚫ +0.0%
Server bundle size (gzip) vinext 161.4 KB 161.4 KB ⚫ +0.0%

View detailed results and traces

🟢 improvement · 🔴 regression · ⚫ change below 1.5% · paired base/head

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant