docs: rewrite README to house standard - #6
Conversation
|
Codex review: needs maintainer review before merge. Reviewed August 2, 2026, 9:18 PM ET / August 3, 2026, 01:18 UTC. ClawSweeper reviewWhat this changesThe PR replaces the long README with a concise installation and quick-start guide and adds Merge readiness✅ Ready for maintainer review Keep this owner-authored PR open for normal merge review. The focused README/usage-guide split matches the current command, configuration, release, and container surfaces, and no discrete documentation defect or security concern was found. Priority: P3 Review scores
Verification
How this fits togetherblucli is a Go command-line client that discovers and controls BluOS players over a local network. The README is its first-use entry point, while the usage guide expands into operational commands and the protocol specification explains implementation details. flowchart LR
User[User or script] --> Readme[README install and quick start]
Readme --> Usage[Usage guide]
Readme --> CLI[blu command]
Usage --> CLI
CLI --> Discovery[Player discovery and selection]
Discovery --> Players[BluOS players]
CLI --> Spec[Protocol specification]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Merge the focused README and usage-guide reorganization so the README remains a concise first-use path and Do we have a high-confidence way to reproduce the issue? Not applicable: this PR changes documentation rather than fixing a reported behavior defect. Its command and configuration statements were checked directly against current source. Is this the best way to solve the issue? Yes: separating a concise first-use README from a detailed usage guide is the narrowest maintainable solution, and the inspected examples align with current command and packaging contracts. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against e19ab32fa5d8. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
Fixes both pre-existing govulncheck CI failures. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Summary
docs/usage.mdand linking the existing protocol/implementation spec.Moved or dropped
docs/usage.md.docs/spec.md.ice1.somafm.comexample with an audio URL that returns HTTP 200.Verification
go build -o dist/blu ./cmd/blugo test ./...golangci-lint run --timeout=5mgo mod tidyfollowed by a cleangit diff --exit-code -- go.mod go.sumpnpm build,pnpm test, andpnpm lintbrew install steipete/tap/blucli; installedgo install github.com/steipete/blucli/cmd/blu@latestinto a temporaryGOBIN.devices,status, andnowhelp paths because no BluOS hardware was available to this worker./Users/steipete/Projects/agent-skills/skills/autoreview/scripts/autoreview --mode local: clean, no accepted/actionable findings.Docker was not installed on this worker, so the existing container commands were checked against
Dockerfileand the repository's Docker smoke-test workflow rather than run locally. Homebrew installation and the packaged binary worked; the tap formula's separatebrew testcurrently fails after invokingblu --versionbecause its Ruby assertion passes a HomebrewVersionobject toassert_match, which is outside this documentation-only change.CI
On exact head
219d8c85f991c874abdfc7199cf43f7faf4ebd05, both Docker jobs, both lint jobs, all six platform-test jobs, and GitGuardian passed. Both duplicategovulncheckjobs report pre-existing advisory GO-2026-5856 in the repository's pinned Go 1.25.11 standard library; the advisory is fixed in Go 1.25.12. This PR does not change the toolchain under its documentation-only scope.