quest: dropped sources, video keyframe flag, interop flakes

[kixelated]

Problem

Follow-ups from the second spawn-quests round that the maintainer chose to keep.

Approach

Adds three m1 quests:

• dropped-sources.md [M]: consumers see the producer's real error, never Dropped. Covers the remaining paths feat(net): reset revoked lite streams with UNAUTHORIZED #4179 found, plus moq-transport PUBLISH_DONE Unauthorized arriving as Remote(1).
• video-keyframe-flag.md [S]: Encoded marks keyframes, so the Control::cut() throttle from feat(video,audio,publish)!: matching capture Control handles, with a keyframe request #4184 counts cadence keyframes.
• interop-flakes.md [S]: shared port reservations across Nix shells and a reliable pause control in the browser driver (seen in test(interop): assert each AUTH cell's grant and refuse a publish outside it #4181).

Impact

• None (quest docs only).

Alternatives

• None.

Follow-ups

• Line-specific follow-ups (ADTS refusals, C++ error messages, WebSocket upgrade polish) go to their lines in separate PRs.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

Next included review available in 11 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: ff1b0bfe-9ab5-450c-9d5f-f983b9e00aed

📥 Commits

Reviewing files that changed from the base of the PR and between 6d637a0 and fd07b1e.

📒 Files selected for processing (4)

• quest/m1/README.md
• quest/m1/dropped-sources.md
• quest/m1/interop-flakes.md
• quest/m1/video-keyframe-flag.md

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[chatgpt-codex-connector]

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	⚠️ Failed 2026-09-25T22:44:33.381193Z	fd07b1e	PR opened

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[kixelated]

MERGE

Quest-docs-only follow-ups from recent spawn work. Three small, well-scoped plans that match how m1 quests are written, and the README placements put each next to its nearest cousin.

Positive improvement? Yes. Each quest closes a concrete gap called out by a prior PR, without inventing new product surface.

• dropped-sources finishes what feat(net): reset revoked lite streams with UNAUTHORIZED #4179 started: consumers still see Dropped on source-close, route-leave, and withdrawn-broadcast paths, and Unauthorized PUBLISH_DONE lands as Remote(1). Distinct from session-death-error (session death / Cancel / clean-end races); this one is producer-side error propagation. Prefer fixing at the source over remapping at the consumer is the right bias.
• video-keyframe-flag completes the feat(video,audio,publish)!: matching capture Control handles, with a keyframe request #4184 Control::cut() throttle: cadence keyframes are invisible to it today, so a cut right after a GOP keyframe still forces another. Carrying the flag on Encoded and counting every keyframe matches what JS already does. The #[non_exhaustive] / dev note is the right API caution.
• interop-flakes makes just test interop --all trustworthy under shared machines. Port collisions from per-shell TMPDIR and canvas-blocked pause clicks are real test(interop): assert each AUTH cell's grant and refuse a publish outside it #4181 flakes; preferring OS-assigned ports (or one shared reservation root) and a non-retry pause path is the durable fix.

Worth the complexity? Yes. Docs only, [M]/[S]/[S], no wire or public-API surprises beyond one field on Encoded. Regression tests and a dual-harness proof are named up front.

Different approach? Not needed. Splitting these from the line-specific code follow-ups (called out in the PR body) keeps planning quests mergeable while implementation stays on its own lines. Optional polish: cross-link session-death-error from dropped-sources Related so the Dropped boundary stays obvious to whoever picks either up.

Recommendation: MERGE as planning quests. Implementation can follow per quest.

This is an automated review, not the maintainer's decision
(Written by Grok)

[kixelated]

Landing three m1 quest plans only: dropped-sources (consumers get the producer's real error instead of Dropped), video-keyframe-flag (Encoded carries a keyframe bit so the cut throttle counts cadence keyframes), and interop-flakes (shared port reservations and a reliable browser pause). Check and Test are green.

(Written by Grok 4.7)