Cleanup: review feedback fixes and missing CI provider assets

[samgutentag]

Summary

Cleanup pass addressing Claude review feedback from the batch docs run and missing assets.

Review feedback fixes

• PR [TRUNK-17578] Add Atlassian Bamboo CI provider documentation #479 (Bamboo): Fix "XCode" → "Xcode" casing
• PR [TRUNK-16666] Improve Merge Queue API reference with summary table and examples #484 (API ref): Clarify "Cancel or merge" → "Cancel all enqueued PRs or wait for them to be merged first"
• PR Document broken state detection in threshold monitors #490 (Broken detection): Link threshold monitors to dashboard docs, soften "distinct series" wording
• PR Update graph view docs for new merge queue graph UI #494 (Graph UI): Split long paragraph into two sentences
• PR Document Slack App Home merge queue dashboard #495 (Slack Home): Fix heading case — sentence case → Title Case for consistency

Missing assets

• Added placeholder bamboo.png for Atlassian Bamboo CI provider card
• Added placeholder google-cloud-build.png for Google Cloud Build CI provider card

Not addressed (need eng input)

• PR [TRUNK-17757] Document Test History tab on Flaky Tests test detail page #481 (Test history tab): Color legend inconsistency (gray=skipped in chart but not in row borders), quarantine tab wording — needs @mmatheson to confirm UI behavior
• PR [TRUNK-17578] Add Atlassian Bamboo CI provider documentation #479 (Bamboo): "XCode" typo also exists in other CI provider docs (azure-devops, etc.) — fixing only in bamboo.md to avoid scope creep, can do a follow-up

Note on placeholder images

The bamboo.png and google-cloud-build.png are simple colored placeholders. They should be replaced with proper product logos before merging.

Files changed

• flaky-tests/get-started/ci-providers/atlassian-bamboo.md
• flaky-tests/detection.md
• merge-queue/using-the-queue/monitor-queue-status.md
• merge-queue/integration-for-slack.md
• merge-queue/reference/merge.md
• .gitbook/assets/bamboo.png (new)
• .gitbook/assets/google-cloud-build.png (new)

Test plan

• Check GitBook preview for all changed pages
• Replace placeholder images with proper logos
• Confirm threshold monitors link resolves correctly

🤖 Generated with Claude Code

[trunk-staging-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

[claude]

Claude finished @samgutentag's task in 1m 1s —— View job

---

PR Review

• Gather context and read changed files
• Review changes
• Provide feedback

---

The changes are clean and accurate. All fixes address the intended review feedback correctly. Here are my observations:

Changes look good:

• atlassian-bamboo.md: "XCode" → "Xcode" ✓ (correct Apple product casing)
• detection.md: Added [Threshold monitors](dashboard.md#monitors) link ✓ and softened "distinct series" wording ✓
• monitor-queue-status.md: Split long paragraph for readability ✓
• merge.md: Clarified "Cancel or merge all enqueued PRs first" → "Cancel all enqueued PRs or wait for them to be merged first" ✓
• integration-for-slack.md: Heading case fixes to Title Case ✓

Minor observation — inconsistent heading not addressed:

In merge-queue/integration-for-slack.md at line 47, the heading ### Setting up Personal Notifications is still in sentence case while all surrounding headings (including the three fixed in this PR) use Title Case. This is a pre-existing issue not introduced here, but worth a follow-up since it's in the same section.

Blocking issue — placeholder images:

As noted in the PR description, bamboo.png and google-cloud-build.png are colored placeholders. If these docs pages are publicly visible, users will see placeholder images instead of actual CI provider logos. This should be resolved before merging unless the pages already show a missing image state (placeholder is an improvement) or there's a plan to replace them promptly.

---

Summary: All text changes are accurate and improve the documentation. The placeholder images are the main concern for merging.

[claude]

Claude finished @samgutentag's task in 1m 27s —— View job

---

PR Review

• Gather context and read changed files
• Review changes
• Provide feedback

---

All five text changes are accurate and well-executed. Here are the details:

Changes look good:

• atlassian-bamboo.md line 131: XCode → Xcode ✓ (correct Apple product casing)
• detection.md line 110: Added [Threshold monitors](dashboard.md#monitors) link ✓
• detection.md line 117: "distinct series" → "broken and flaky events are shown as separate series" ✓ (improved clarity)
• monitor-queue-status.md lines 44–46: Long paragraph split into two sentences ✓
• merge.md line 183: "Cancel or merge all enqueued PRs first" → "Cancel all enqueued PRs or wait for them to be merged first" ✓ (clearer user action)
• integration-for-slack.md: Three headings converted to Title Case ("What You'll See", "Linking Your Account", "Managing Notification Preferences") ✓

Pre-existing issues (not introduced by this PR, worth tracking):

• integration-for-slack.md line 46: ### Setting up Personal Notifications — "up" is lowercase, inconsistent with the Title Case headings in the same section and the three fixed by this PR. Fix this →
• monitor-queue-status.md line 87: "merge conflics" → should be "merge conflicts" (typo)
• monitor-queue-status.md line 71: "statical trends" → should be "statistical trends"
• integration-for-slack.md line 158: "The PR will be merged when it reached the top of the queue" → should be "reaches" (tense mismatch)

Blocking concern — placeholder images:

As noted in the PR description, bamboo.png and google-cloud-build.png are colored placeholders. The PR description itself says these "should be replaced with proper product logos before merging." This remains unresolved.

---

Summary: All text changes are correct and improve the docs. The placeholder images are the blocking concern for merging, as acknowledged in the PR description.