Mobile brand refresh + CI fail-fast hardening#174
Open
brianorwhatever wants to merge 16 commits intomainfrom
Open
Mobile brand refresh + CI fail-fast hardening#174brianorwhatever wants to merge 16 commits intomainfrom
brianorwhatever wants to merge 16 commits intomainfrom
Conversation
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Replace "Poo App" → "boop", trypoo.app → ad.boop.app, pooapp scheme → boop scheme across all prose, docs, and user-facing source strings. Rewrite resources/README.md with brand-refresh pipeline description. Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Correct three issues introduced by the Task 11 copy audit (2ad6553): - Replace ad.boop.app URLs with boop.ad in sitemap, robots.txt, README, docs, src pages, webvh.ts, deep links docs, and convex comments - Fix email sender in convex/feedback.ts from [email protected] to [email protected] - Fix all contact/support emails in Terms, Privacy, Landing pages to @boop.ad - Rename all pooapp:* localStorage keys to boop:* across storage.ts, streaks.ts, OnboardingFlow.tsx, useSettings.tsx, Home.tsx, and e2e tests - Simplify index.html dark mode bootstrap to read only boop:darkMode
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Previously CI would run 113 tests × 1 worker × retries=2 × default 30s timeout and get cancelled at the 30-minute workflow cap when tests started failing, producing no useful signal. With these changes the job aborts after 10 failures with the list reporter printing the first specs that broke, so regressions surface in ~3 minutes instead of 30.
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
Superset of #173 — same 14-commit mobile brand refresh plus one CI hardening commit on top.
The one extra commit (
378ca2f) fixes a pain point uncovered by #173:maxFailures: 10so Playwright aborts once the first 10 specs fail instead of burning the full 30-minute workflow budgetretriesfrom 2 → 1 in CI (retries triple the wall clock when every test is broken)workersfrom 1 → 2 in CItimeout: 30_000,actionTimeout: 10_000,navigationTimeout: 15_000listreporter in CI so broken specs print in the run log as they failResult: when tests are broken, CI surfaces the actual failures in ~3 minutes instead of timing out at 30 minutes with no log output.
Test plan
🤖 Generated with Claude Code