docs: rewrite README to house standard - #31
Conversation
|
Codex review: needs maintainer review before merge. Reviewed August 2, 2026, 9:48 PM ET / August 3, 2026, 01:48 UTC. ClawSweeper reviewWhat this changesRewrites the README as a concise product entry point and adds CLI and configuration reference pages for platform-specific details. Merge readinessThis owner-authored, documentation-only PR is coherent, factually aligned with the current implementation, and has no actionable patch defect. It should remain open for ordinary maintainer review rather than automated cleanup or repair. Priority: P3 Review scores
Verification
How this fits togetherNameplate is a cross-platform menu-bar or tray utility that derives a machine identity from local and fleet settings, then displays overlays, splashes, and attention prompts. The README and reference docs are the entry path from installation into platform-specific CLI and configuration behavior. flowchart LR
Install[Install channel] --> App[Nameplate app]
Fleet[Fleet and local settings] --> Identity[Machine identity]
Identity --> Overlays[Desktop overlays]
CLI[CLI commands] --> App
App --> Attention[Attention and splash output]
Docs[README and reference docs] --> Install
Before merge
Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Land the concise README and keep the detailed platform-specific CLI and configuration contract in the new reference pages, with future behavior changes updating the corresponding platform guide and reference page together. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes documentation rather than reporting a runtime failure. The moved CLI, fleet, and remote-viewing claims can be checked directly against current implementation and existing platform guides. Is this the best way to solve the issue? Yes. A short README with dedicated CLI and configuration references is the narrowest maintainable way to preserve the product front door while keeping platform-specific behavior discoverable. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 611446994d98. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
Summary
Rewrites the README as a 95-line front door with the house-standard title, dynamic badge row, verified install paths, a two-command quick start, one screenshot near the top, and progressive detail for fleet identity, automation, connection behavior, and development. The old README was 119 lines.
Moved and corrected
docs/cli.md.docs/configuration.md.locationis macOS-only rather than a shared fleet field, and--wait,settings, anddismissare macOS-only rather than common CLI behavior. No changelog entry was added because the changelog has no README-only documentation precedent.Verification
swift build— passed.swift test— 62 tests passed.APP_IDENTITY="-" ./Scripts/package_app.sh debug— built and packaged the app.open -a Nameplateandnameplate splash— exact quick start passed against the installed Developer ID-signed app.cargo test -p nameplate-core --manifest-path linux/Cargo.toml— 9 Linux core tests passed.nameplate --helpoutput and all three CLI implementations rather than presenting a live alert.invalidornot foundstate.