Skip to content

ci(performance): split deterministic and lab gates - #34

Merged
furedea merged 1 commit into
mainfrom
ci/split-performance-gates
Aug 15, 2026
Merged

ci(performance): split deterministic and lab gates#34
furedea merged 1 commit into
mainfrom
ci/split-performance-gates

Conversation

@furedea

@furedea furedea commented Aug 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • enforce image, JavaScript, font, total transfer-size, and request-count budgets in the required E2E job
  • discover localized built pages automatically for both asset and lab checks
  • run sampled FCP, LCP, CLS, and blocking-time measurements in an independent non-required workflow
  • supersede ADR-0012 with ADR-0013

Verification

  • unit tests: 98 passed
  • E2E: 60 passed, 2 skipped
  • asset budgets: 6 passed in about 5 seconds
  • lab performance: 6 passed in about 1.9 minutes
  • Astro typecheck, oxlint, oxfmt, knip, build verification, and actionlint passed

Merge behavior

The repository ruleset requires only all-green. Deterministic asset budgets run inside its required E2E dependency, while Lab Performance is reported separately and does not block merging.

Copilot AI lite review requested due to automatic review settings August 14, 2026 18:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying furedea-com with  Cloudflare Pages  Cloudflare Pages

Latest commit: f8fb694
Status: ✅  Deploy successful!
Preview URL: https://672f748a.furedea-com.pages.dev
Branch Preview URL: https://ci-split-performance-gates.furedea-com.pages.dev

View logs

@furedea
furedea merged commit 47e5c49 into main Aug 15, 2026
18 checks passed
@furedea
furedea deleted the ci/split-performance-gates branch August 15, 2026 02:43
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.

2 participants