perf(ci): right-size local E2E browser coverage - #951
Merged
Conversation
Run Chromium on the ordinary PR path while preserving full cross-browser coverage on push, dispatch, and full-e2e label events. Make aggregation mode-aware and report queue versus execution timing. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
TortoiseWolfe
added a commit
that referenced
this pull request
Aug 26, 2026
… clipping (#954) The deck's argument is "check my numbers", so a wrong one costs more here than in ordinary docs. Reconciles the two drafts in docs/talks/ into one and corrects every claim I could verify against the repo, the API and CI logs. The reproducibility defects, which is what prompted this: - The lineage slide printed `git log --reverse --format='%h %ad %s'` above output that command cannot produce — bare `%ad` renders "Sat Sep 27 11:16:44 2025", not "2025-09-27", and no log.date is configured. It needs --date=short. That was on the one slide inviting the room to run it. - Slide 1 carried the whole twelve-command script with no `$` prompts, so the copy button offered a plain "Copy" and pasted the comment line with it. It now shows the five-step shape; the exact paste-safe form lives on the step slides, which is also what stops it overflowing its printed page by 439px. - Adopts the `data-copy="commands"` + `.cmd` span mechanism, so an output line like "Created repository" can never reach somebody's shell. Corrections, each verified rather than assumed: - 4,750 tests / 448 files -> 4,787 / 452 (CI job log 97228328942) - "10 wks to live on its own domain" -> 31 days. Jun 9 split, raisedpaws.com CNAME landed Jul 10 in c2b564d4. Ten weeks was the project's age at the measurement date, not its time to a domain. - 212 commits -> 222 unique to the fork - "Four good first issues" -> eight - "140 pages built" -> 137 routes; 140 counted two copied files, and Next prints a route count rather than "pages built" - "35 themes, WCAG-tested" -> 35 themes, 31 AAA and 4 AA. The whole-page AAA sweep covers only the three house themes. - "RLS conformance against a live database" -> conformance.yml brings up a local ephemeral stack; Auth Config Drift is the check that talks to a live project - The assertion gate is #861; #396 is the catalogue of gates that could not fail. The reporter prints #396, so the screenshot stays and the note explains it. - E2E is Chromium on an ordinary PR since #951, all three browsers on merge - The rebrand figures are re-measured against current main: 16s, and 926 paths by git's count. The summary line says 1002 because it counts a file once per substitution pass, and the note says so. - #859 was open and fixed inside two hours, and it was surfaced by the zero-assertion work on the same slide — the two panels are one story Also fixes the print box. scripts/render-talk.mjs reported four slides clipping, which the page count cannot reveal; the print type scale is now measured against it rather than guessed. 21 slides, 21 pages, fonts verified embedded. CLAUDE.md said "35 themes: 2 house + 32 DaisyUI", which is 34. There are three house themes. Co-authored-by: TurtleWolfe <jonpohlner@gmail.com> Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
full-e2eExpected job shape
full-e2ePR, push to main, or dispatch: changes + 24 shards + aggregate = 26 jobsVerification
pnpm test:scripts— 580 passedCloses #950.