Skip to content

docs: reconcile README, design and harness docs with shipped behavior - #280

Merged
altaywtf merged 1 commit into
mainfrom
docs/reconcile-2026-10-01
Oct 1, 2026
Merged

altaywtf merged 1 commit into
mainfrom
docs/reconcile-2026-10-01

Conversation

@altaywtf

@altaywtf altaywtf commented Oct 1, 2026

Copy link
Copy Markdown
Member

Docs now match main at 493cf18: nine stale claims fixed, no new sections.

  • README: TV Search and History results play media and open everything else in Files (#253); TV autoplay next (#259); mobile copies items shared with you (#275)
  • design/README: phones turn to landscape only for a landscape video (#266); Captions hides under hide_subtitles (#261)
  • behavior and harness: a shared media file on TV still offers Open in VLC (TvFilesAction.kt); TV buffer size is tv-native's fixed default, not a missing server key
  • design-v3-discrepancies: TV file-row glyphs render ListItemDefaults.IconSize (32dp), not 21dp
  • Checked unchanged: test and proof names, instrumentation args, constants, build and CI facts; relative links and anchors resolve. No doc check exists in verify

Written by an agent (Claude Code, Opus 5.5)

Copilot AI balanced review requested due to automatic review settings October 1, 2026 12:41
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_subtitles Captions 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.

@altaywtf
altaywtf merged commit fa3d1c3 into main Oct 1, 2026
2 checks passed
@altaywtf
altaywtf deleted the docs/reconcile-2026-10-01 branch October 1, 2026 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants