docs: rewrite README to house standard - #206
Conversation
|
Codex review: needs changes before merge. Reviewed August 2, 2026, 8:46 PM ET / August 3, 2026, 00:46 UTC. ClawSweeper reviewWhat this changesThe branch turns the README into a concise install-and-workflow entry point, moves optional injected-bridge command reference material to Merge readinessThis focused documentation PR should remain open because it is authored by a repository collaborator. The README and bridge-reference split is coherent and aligns with current command surfaces, but the contributor-authored unreleased changelog entry should be removed before merge because release notes are release-owned. Priority: P3 Review scores
Verification
How this fits togetherThe README is the public entry point for the macOS Messages CLI. It guides users from installation and normal database/AppleScript workflows to canonical documentation, including the SIP-disabled IMCore bridge used for advanced Messages controls. flowchart LR
Visitor[CLI user] --> FrontDoor[README quick start]
FrontDoor --> Standard[Read and send workflows]
FrontDoor --> FullDocs[Canonical documentation]
FullDocs --> Bridge[Optional IMCore bridge reference]
Bridge --> Advanced[Advanced Messages controls]
Before merge
Findings
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Merge the concise README-to-canonical-docs split and bridge reference after removing the contributor-authored Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes documentation, so the relevant verification is CLI-help, documentation-link, and generated-docs validation rather than runtime bug reproduction. Is this the best way to solve the issue? No: the README and bridge-reference split is maintainable, but normal PRs should not add the release-owned Full review comments:
Overall correctness: patch is incorrect AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 4605e7ce16ea. LabelsLabel changes:
Label justifications:
EvidenceAcceptance criteria:
What I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
Summary
Rewrites the README to the shared house structure: a one-line pitch, dynamic flat-square badges, the smallest working install path, a short quick start, progressive workflow links, development, and license. The README is 103 lines, down from 445, while keeping the existing banner and the read/watch/send proof near the top.
Content movement
docs/bridge.md; the docs-site build publishes it asbridge.html.docs// imsg.sh.CHANGELOG.md.Verification
make build— passed; built the universal arm64/x86_64 CLI and arm64e/arm64/x86_64 helper.make lint— passed with 12 existing non-serious SwiftLint warnings.make test— passed, 510 tests across 5 suites.make docs-site— passed; generateddist/docs-site/bridge.html.IMsgCoreimport/sample — passedswiftc -typecheck, then removed.--version, generated help, andcompletions llm— passed; all README and bridge flags were checked against the generated command source of truth.authorization denied (code: 23). The exactchatsandhistoryflags were therefore validated against the built binary's help, as allowed for permission-gated commands. Live watch, send, RPC, and injected-bridge examples were likewise help-validated because running them would require a live service, mutate Messages, or disable SIP.invalidornot foundcard./Users/steipete/Projects/agent-skills/skills/autoreview/scripts/autoreview --mode local— clean, no accepted/actionable findings.