Repository navigation
Clarify HDR display diagnostics and add bounded playback evidence - #317
Merged
Merged
Conversation
jlipworth
force-pushed
the
codex/parked-hdr-and-host
branch
from
September 14, 2026 22:14
5da3122 to
6d35ae4
Compare
added 7 commits
September 14, 2026 18:15
Preserve player-screen EDR diagnostics, eligibility observations, tests and documentation separately from playback PRs. Physical mixed-display and hot-plug acceptance remains open; not a release-readiness claim.
Keep local host network-permission workaround and tooling assertions separate from paused HDR changes.
jlipworth
force-pushed
the
codex/parked-hdr-and-host
branch
from
September 14, 2026 22:15
6d35ae4 to
53bd104
Compare
This was referenced Sep 14, 2026
Closed
jlipworth
added a commit
that referenced
this pull request
Sep 14, 2026
…318) ## Summary - Investigates #316 with exact-source native decoder and cross-backend delivery evidence. Depends on HDR #317, already merged; rebased onto its main commit without duplicating HDR changes. - Adds a default-off, DEBUG-only macOS P7 HDR10-base candidate with strict initialization/playlist admission and unchanged media/audio bytes. This is not full Dolby Vision decoding or a production enablement. - Retains automatic waiting for verified Jellyfin video-copy reopens, independently fixing bounded P7 and ordinary HDR10 paused-seek reproductions. - Captures final selected media-browser initialization after copy-child rewrites; preserves same-origin, no-redirect and bounded diagnostic limits. - Adds exact-ID disambiguation for duplicate-title debug probes, failure diagnostics, and the user-requested cross-backend validation guidance. - Prepares coordinated app/PMSKit 1.7.1 build 1. No release tag, upload or distribution. ## Evidence and limits Plex P7 candidate multi-segment, seek/reopen and both tested audio transitions have inspected PQ/BT.2020 frames. Jellyfin/Emby deliver the exact P7 source without a retained DV configuration atom in their final selected copy initialization; this proves a packaging distinction, not enhancement-layer/RPU processing. Plex P8.1 has a separate incomplete HEVC configuration failure and is not admitted by the P7 candidate. P5 consent/guard cases remain blocked, not passes. Same-source SDR controls distinguish Plex copy from media-browser direct-file delivery. Jellyfin/Emby direct seek holds have inspected BT.709 frames. A narrow DEBUG classifier correction now records explicit directPlay as video/audio copy with serverDecision provenance; fresh Original reopen runs pass with inspected BT.709 frames. Older decisionUnknown reports remain historical blocked evidence, not retroactive passes. No implicit video-encode approval was used. Private source bindings, frames and wire evidence are retained outside Git. Public research documentation records boundaries and full test failures without private media identifiers. ## Validation All four products build as 1.7.1 build 1; hermetic PMSKit 1,707 Swift Testing + 119 XCTest pass. After replacing the Offline test's independent wall-clock sleep with a DEBUG-only deterministic gate, full macOS 637 passes twice, iOS 621 passes, and tvOS 339 passes with normal runner exit in 24 seconds. Production 750ms preparation delay and cancellation guards are unchanged. Historical full-suite failures and tvOS runner hangs are retained; current non-reproduction does not establish their root cause. Native exact-source SDR controls have inspected frames on all three backends. Hygiene/docs/link/Mermaid checks pass. A UI-fixture freshness audit found older runs had cleared non-UI DerivedData instead of the UI-specific paths. Older fixture passes are not standalone clean-revision evidence; replacement fixtures explicitly clear the correct UI build paths. ## Open acceptance gates - Historical intermittent deadline/ordering failures and runner-hang attribution; new full-suite passes do not prove these cannot recur. - Authorized quality/subtitle video-encoding transitions (not authorized in this task). - Broader source/profile coverage and cross-backend scenarios not explicitly tested. - Physical Dolby Vision processing, HDR luminance, platform release acceptance. - visionOS install/run: missing canonical golden-simulator pointer; no other task's simulator substituted. This is not a request to merge, close #316, enable the candidate in production, or publish a release. Replacement iPhone detail and tvOS home semantic fixtures both pass from the correct clean UI build paths, with app-view attachments inspected. All owned simulators are shut down. --------- Co-authored-by: Jonathan Lipworth <yoyoyopop40@gmail.com>
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
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.
Summary
Validation
Final merge revision
The cross-backend CLAUDE guidance is preserved separately on the DV branch and is not included here. App, test and script bytes match the validated review revision; the final change only makes existing Plex-only evidence scope explicit.
Explicit limits / remaining gates
All temporary display settings were restored; our simulator runs were shut down and isolated Mac staged apps removed. The separate DV task now owns the simulator lease. Existing sessions and production app preserved. No version/tag changes. User approved integration after final-revision checks.