Repository navigation
quest: delete content protection - #8332
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 11 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
- Content protection epic deleted — removes
quest/m3/drm/(README.md,import.md,bindings.md,mp4-atom.md) andquest/m5/drm-playback.md, plus the two milestone bullets that listed them. - No survivors left behind — a repo-wide search finds no link,
## Requirededge,sh/gh/test-inventory.allowentry, or prose gate referencing the deleted quests, and every/quest/...link in the edited milestone READMEs still resolves. - Bookkeeping matches the decision — #7766 is closed with its
questlabel removed, and both cited implementing PRs (#8285, #7916) are closed unmerged, so nothing is left pointing at work that no longer exists.quest check --base origin/main --branch pr-8332reports596 documents ok.
deepseek-v4.1-flash-1017 (free via Pullfrog for OSS) | 𝕏
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed head 7d65bab. No actionable findings in this planning-document removal.
The content-protection epic, its three child plans and deferred browser-playback plan are removed together with both milestone index entries. This retires planned work without changing runtime code, public APIs or wire behavior; the related implementation PRs #7916 and #8285 are closed.
Verification limits: GitHub-only static review of the full seven-file diff and related PR state. No builds, tests or repository-wide link checker run.

Deletes the content protection epic
quest/m3/drm/(README, import, mp4-atom, bindings) andquest/m5/drm-playback.md, plus their links from the m3 and m5 milestone READMEs.Why: no in-tree player can play protected content (WebCodecs can't decrypt, EME needs MSE which
<moq-watch>removed, native has no CDM), so the hangdrmsection, MSFcontentProtections, import, and bindings would be surface with no consumer. Revisit when someone needs it. The implementing PRs #8285 and #7916 were already closed; thequestlabel was removed from #7766.Public API / wire impact: none (quest docs only). Review skipped: deletion-only quest diff.
quest checkpasses.🤖 Generated with Claude Code
(Written by Claude Opus 5.5)
Review
Skipped at 7d65bab: the diff only deletes finished-by-decision quest files and their index links;
quest checkpasses.