Skip to content

DO NOT MERGE: email screenshot flake check (hermetic) - #6881

Closed
backspace wants to merge 2 commits into
mainfrom
buck-25-verify-hermetic
Closed

backspace wants to merge 2 commits into
mainfrom
buck-25-verify-hermetic

Conversation

@backspace

Copy link
Copy Markdown
Contributor

Warning

Do not merge. This is a throwaway branch that checks a flaky-test fix. It will be closed after CI runs.

The Poppins 700 stylesheet that the Synapse email templates load is held for 8 seconds. In a CI failure of profile.spec.ts › it can change email in settings, that stylesheet had no response when the screenshot was taken, and the email showed as a blank pane. Each matrix shard runs only the three email screenshot tests, 5 times each, one worker, no retries.

This branch is the stored-asset fix (buck-25-hermetic-email-assets). Expected: pass, and the stall route is never reached.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK

backspace and others added 2 commits October 21, 2026 04:52
The Synapse email and validation-page templates load three Google Fonts
stylesheets, a fonts.cdnfonts.com stylesheet and the Boxel logo from the
internet. The stylesheets block rendering. When one is slow, the mail
client shows the email as a blank pane and the screenshot assertion
fails. In one failure the Poppins 700 stylesheet had no response when
the screenshot was taken.

The shared test fixture now routes those URLs, for every page in the
context, to the bytes the browser received from them. A request to
those hosts that is not stored gets a quick 404 and a warning, so a
template change cannot bring the network dependency back without
notice.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK
…esheet

Holds the Poppins 700 stylesheet the email templates load for 8 seconds
and runs only the three email screenshot tests on each matrix shard,
repeated, one worker, no retries.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WdM45xnz3vvTmA7HN8nGmK
@backspace

Copy link
Copy Markdown
Contributor Author

[Claude Code 🤖] Verification finished; results are summarised in #6883. Closing this throwaway branch.

@backspace backspace closed this Oct 21, 2026
@backspace
backspace deleted the buck-25-verify-hermetic branch October 21, 2026 10:15
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