docs: reconcile README, design and harness docs with shipped behavior - #280
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
It is a documentation-only change and every reconciled claim was verified accurate against the current shipped code, with all cross-references resolving.
Review effort: Balanced
Findings: None
What changed in this PR
This PR updates documentation only (README, design docs, harness, and behavior references) to reconcile stale descriptions with behavior already shipped to main. It fixes nine out-of-date claims across five Markdown files without introducing new sections or code changes, following a series of recently merged feature PRs (#253, #259, #261, #266, #275).
Changes:
- README: documents TV autoplay-next, TV Search/History results playing media (else opening in Files), and mobile copy of shared-with-you items.
- design/README: corrects mobile fullscreen orientation (landscape only for landscape video) and the
hide_subtitlesCaptions behavior. - behavior/harness/design-v3 docs: clarifies TV shared-media Open in VLC, TV buffer-size rationale, and the TV file-row glyph size.
| File | Description |
|---|---|
| README.md | Updates mobile Files copy, TV autoplay-next, and TV Search/History open behavior to match shipped code. |
| docs/harness.md | Corrects the no-actions oracle wording and the TV buffer-size explanation, adding a resolving link to behavior.md#tv-playback. |
| docs/design-v3-discrepancies.md | Replaces the stale 21dp/42px glyph note with the Compose-for-TV ListItemDefaults.IconSize (32dp) description. |
| docs/behavior.md | Clarifies that a shared media file on TV still offers Open in VLC while other shared items open nothing. |
| design/README.md | Corrects the phone fullscreen orientation rule and the hide_subtitles Captions behavior. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Docs now match
mainat493cf18: nine stale claims fixed, no new sections.hide_subtitles(#261)TvFilesAction.kt); TV buffer size is tv-native's fixed default, not a missing server keyListItemDefaults.IconSize(32dp), not 21dpverifyWritten by an agent (Claude Code, Opus 5.5)