[kixelated]

Plans the follow-ups held from this session's merges.

Quests

• quest/m1/eager-encode/cli-modes.md [S]:
  • moq import capture takes --video <on|demand|off> and --audio <on|demand|off>, replacing --no-video, --no-audio and --on-demand (from moq-dev/moq#7188).
  • export and play take --video/--audio <on|off> in place of --no-*.
  • Raw audio later adds buffered to --audio.
• quest/m2/release/smoke-bindings-rename.md [XS]: a waiting quest. Once a binding release ships request/follow, the native smoke clients move to them.
• quest/m2/hls-follow-once.md [S]: Following reports synced absence, so moq-hls resolves with one follow (from moq-dev/moq#7192).
• quest/m2/bench-epoch-pin.md [XS]: moq-bench pins pooled requests to their announced epoch.
• Plan notes on existing quests:
  • migrate-audit.md: correct overturned Unreleased entries, starting with js/net's announced note.
  • raw-audio.md: the CLI's audio mode gains buffered as its default.
  • land.md: the gate must test each PR on the latest main. moq-dev/moq#7162 broke in the queue on renames its own CI never saw.

Decisions

• CLI flags:
  • ✅ --video/--audio with on|demand|off in capture and on|off in export and play
  • --on-demand kept
  • --video-encode / --audio-encode
  • --video-enabled / --audio-enabled
  • always, never or none as values
• Stale js/net/MIGRATE.md:
  • ✅ Fold into migrate-audit
  • A separate [XS] fix
• CLI help after raw audio:
  • ✅ Superseded by the CLI modes quest, plus a raw-audio Plan line
• Lite publisher yield after SUBSCRIBE_UPDATE (from moq-dev/moq#6988):
  • ✅ Dropped, since nothing user-visible depends on it
  • An [S] m2 quest
• Land gate note:
  • ✅ Add to land.md
  • Skip
• Smoke clients:
  • ✅ An [XS] waiting quest in the m2 release epic
  • Fold into cut.md
• C route epochs:
  • ✅ Dropped at merge: moq-dev/moq#7151, already on main, fills moq_route's epoch for every received announcement
  • A separate [S] m1/c quest (the original pick, made before feat(c)!: announce through moq_announce_route #7151's receiving side was noticed)
  • Fold into announce-route
  • Drop
• moq-hls double resolve:
  • ✅ An [S] m2 quest
  • m1 in the resolve epic
  • Drop
• moq-bench epoch pin:
  • ✅ An [XS] m2 quest
  • Drop

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

[coderabbitai]

Warning

Review limit reached

You'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 24 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5882df31-d533-4ff9-ad27-89d52b90c738

Commits

Reviewing files that changed from the base of the PR and between 66bdbb9 and fef89a6.

Files selected for processing: 10

• quest/m0/ci/land.md
• quest/m1/eager-encode/README.md
• quest/m1/eager-encode/cli-modes.md
• quest/m1/eager-encode/raw-audio.md
• quest/m2/README.md
• quest/m2/bench-epoch-pin.md
• quest/m2/hls-follow-once.md
• quest/m2/release/README.md
• quest/m2/release/migrate-audit.md
• quest/m2/release/smoke-bindings-rename.md

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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	✅ Completed 2026-10-29T15:26:13.548396Z	fef89a6	New commits

ℹ️ 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]

Automated review by review (OpenAI)

Reviewed commit: 50913ca

No actionable planning bugs found. The CLI modes retain the distinction between capture demand and export/play selection, and the raw-audio coordination handles either landing order. The route-epoch and HLS follow-once goals match the current receiving-route and discovery gaps; pooled benchmark requests currently discard announcement epochs. Smoke-client migration is appropriately conditional on published packages, while migration-guide corrections remain with the release audit.

Direction: these are focused follow-ups with existing consumers. Keeping absent-path reporting nonterminal for Following avoids turning a reusable follower into a one-shot request. No extra implementation detail is needed in these plans.

Verification: static review of the plans, repository review guidance, and relevant CLI, origin, HLS, benchmark, C/FFI route, and smoke-policy code/docs. No tests or builds run; future implementations and release availability were not verified.

(Written by OpenAI)

[kixelated]

Merge summary: the plans settled through /quest-plan land unchanged at 50913ca.

• Reviews of this head: Codex 👍 (no findings), and the automated OpenAI review found no actionable issues.
• quest check (against origin/main) and sh/markdown.sh check pass locally.
• No findings to fix and no follow-up decisions.

(Written by Claude Opus 5.5)

[kixelated]

Merge summary: landing fef89a6, after it left the queue on merge_conflict.

• Merged main in. The quest merge driver resolved the link-list conflicts.
• Dropped quest/m1/c/route-epoch.md with quest rm. moq-dev/moq#7151 landed on main and removed announce-route.md, which this quest linked to. Its receiving side already fills moq_route.epoch for every announcement it receives, which was this quest's whole goal. The PR body's decision list records the drop.
• Review of this head: Codex 👍 (no findings).
• quest check against the latest origin/main and sh/markdown.sh check pass locally.

Suggested follow-up (not planned): a moq-c test that pins a request to a received route's epoch and checks that it refuses another instance. Today's tests pin only Rust-minted epochs.

(Written by Claude Opus 5.5)