Refresh the README app overview screenshot - #51
Conversation
Why: - The public README hero contradicted the Alpha 7 app by showing the obsolete Start brokerd CTA. Changed: - Replace docs/images/macos-overview.png with an unedited deterministic Alpha 7 Overview window capture showing Start service. Verification: - Focused stopped-service CTA XCTest passed 1/1. - Screenshot identity, hard gates 1-5, independent visual review, Vision OCR, dimensions, and public-safety scans passed. - Docs tests passed 51/51; public-surface verification and the spec-only agent profile passed. Affected: - Public README Overview image only; no runtime, release, tap, or service behavior changes. Refs: - #50 - c373a48 Session: - task-sessions/rr29-readme-screenshot-20260901
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
This meaningful task commit uses Summary, Verification, and Scope headings instead of the required Why:, Changed:, Verification:, Affected:, Refs:, and Session: sections. Merging it would leave unstructured durable history and contradict the claimed successful agent:complete closeout, so recreate the commit message using the mandated format.
AGENTS.md reference: AGENTS.md:L26-L26
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Connector P1 resolution: no history rewrite is needed. GitHub's exact commit payload for The Connector review created no inline review thread, so this evidence records the non-code resolution on the PR timeline; the valid commit and image remain unchanged. |
|
Bugbot reviewed the PR head ( Required local closeout already passed ( |
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
Post-merge proof:
|
Summary
Start brokerdCTAVerification
Start servicepresent;Start brokerdabsent)spec-onlyagent profile and clean closeout: passedScope
Docs-only correction. No runtime, release, Homebrew tap, or service behavior changes, and no new release is needed.
Closes #50