docs: doc-vs-software audit round 2 (zero2w source, hw claims, diagnostic path, changelog anchors) - #656
Merged
Merged
Conversation
The manual diagnostic-bundle command documented only /opt/snapmulti/scripts/diagnostic.sh. On a player-only (Audio Player) install the script is staged to /opt/snapclient/scripts/diagnostic.sh (staging-manifest.sh STAGING_CLIENT_OPTIONAL, dest "scripts"), so the documented path does not exist there and the command fails. Both call-sites (the "diagnostic bundle" section and the "symptom" inline mention) now point client users at the /opt/snapclient path. Mirrored to the Italian doc. CHANGELOG updated. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011xbiRoWQLVmKDxNkBqLsvk
Ready to merge — no issues foundSmall, accurate documentation fix. Verified the underlying claim: Checklist
|
…changelog anchors Doc-vs-software audit round 2 (extends #656): - HARDWARE.{md,it.md}: Pi Zero 2 W client-native bullet said snapclient is installed "from the upstream badaix .deb". Actual chain (setup-zero2w.sh:152): prefer the snapMULTI-built .deb (snapclient-deb/v0.35.0-snapmulti1), fall back to distro apt (Trixie 0.31 / Bookworm 0.27) — never badaix. Corrected. - HARDWARE.{md,it.md}: "Recommended Setups" listed boards not in the narrower "Tested Combinations" matrix, implying launch validation. Added a note: the picks come from the validated PCM5122 DAC family and are expected-to-work; only "Tested Combinations" units are physically verified end-to-end. - CHANGELOG.md: fixed two stale IPv6 anchors in the ADR-007 entry after the target headings were renamed (disabled -> enabled). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011xbiRoWQLVmKDxNkBqLsvk
lollonet
changed the title
Ready to merge — no issues foundReviewed the new commits in this round ( Verified each new claim against the actual code/docs:
Checklist
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Second doc-vs-software audit pass, verified against code:
HARDWARE.{md,it.md}.deb" —setup-zero2w.sh:152prefers the snapMULTI-built.deb(v0.35.0-snapmulti1), falls back to distro apt. Never badaix. Corrected.HARDWARE.{md,it.md}TROUBLESHOOTING.{md,it.md}/opt/snapmulti/...; player-only installs stage it to/opt/snapclient/...(staging-manifest.sh). Both call-sites now note the client path.CHANGELOG.md#ipv6-disabled-*→#ipv6-enabled-*) after the target headings were renamed.Doc-only, no code paths touched. Extends the #652 doc-vs-software audit.
🤖 Generated with Claude Code