Rewrite Merge Queue overview for skeptical CI engineers#63
Open
samgutentag wants to merge 4 commits into
Open
Conversation
Replaces marketing-style "Big Three problems" framing with a mechanics-first overview. Addresses Eli's feedback (2026-05-21): - "Big three items don't make sense or are outright lies" - "Calls out Nx/Bazel but not other build systems" - "Not selling - no clear audience, layout sucks" Page now: - Opens with a 2-paragraph hook that introduces predictive testing as the baseline every queue does, then names the three Trunk- specific layers (batching, parallel queues, anti-flake protection). - Gives each layer its own section with a plain-language mechanic explanation, the existing sub-page iframe, a one-line "what it costs you not to have it", and a link to the deep page. - Replaces "Native Bazel/Nx" with framing that includes any build system via the impacted-targets API. - Adds a "When does this make sense?" decision tree mapping symptom to mechanic in recognition voice. - Drops unsupported claims: "Batch up to 100 PRs", "cut CI costs up to 90%", "Automatic quarantine of flaky tests" (which was the wrong product entirely). - Strips marketing words per global style guide. Pattern captured for reuse on other product overviews at .claude/patterns/product-overview-for-skeptical-engineers.md (in main worktree, untracked). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replaces the three stacked mechanic sections with a 3-column Tiles layout (boxes-stacked / columns-3 / shield-check icons + title + short description) that links each tile to its deep page. - Tighter at-a-glance preview of batching / parallel queues / anti-flake protection - Adds Direct merge to main to "More to dial in" (the one architectural optimization in the settings UI that wasn't already represented at the top) The Vidyard / iframely embeds drop off this page; they still live on the deep optimization pages. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The tile title is "Anti-flake protection" so "Flakes don't evict PRs." was redundant. "CI gets a second chance." carries the same intuition without the duplicated noun. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds Predictive testing as a peer tile alongside batching, parallel queues, and anti-flake protection. Layout becomes 2x2. Intro tightened to call out the actual relationship between the mechanics: predictive testing is the correctness baseline; the other three are throughput and resilience optimizations on top of it. The old "three things on top of that baseline" wording overstated the hierarchy - parallel queues is a separate design axis, not strictly derived from predictive testing. Crystal-ball icon for the predictive tile (small wink at "looking ahead" without being too cute). Co-Authored-By: Claude Opus 4.7 (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
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
Ground-up rewrite of
merge-queue/merge-queue.mdx, triggered by Eli's feedback on 2026-05-21:What changed
Audience targeting
This rewrite uses the skeptical-engineer overview pattern (committed locally; not yet shared). The marketing site does the selling;
/getting-started/does onboarding; this page does the "is this worth ten more minutes?" mental model. No customer names, no numeric outcome claims — mechanics + visuals are the only evidence.Items flagged for review
descriptionentirely.boxes-stacked,columns-3,shield-check(FontAwesome / Lucide resolved by Mintlify). Easy to swap.Test plan
🤖 Generated with Claude Code