docs(quest): plan follow-ups from the next-16 quest run

[kixelated]

Problem

The spawn-quests run over the next 16 ready quests turned up follow-ups that need durable scope.

Approach

Seven new quests, planned with the maintainer:

• m1 lite-07 stream count [M]: moq-lite-07 (unpublished) drops SUBSCRIBE_DROP, and SUBSCRIBE_END carries a group-stream count, sent once every stream below the end is opened, like PUBLISH_DONE. Neither publisher ever sent DROP. Rust track tail now requires it.
• m1 Signal.race cleanup [S]: Signal.race returns the disposable awaitable that race()/effect.race from fix(js): stop retaining a listener, reaction, or task per frame #4085 already understand, so a lost race releases its listeners.
• m1 Reconnect test ports [S]: tcp_local_addr() on moq-tokio's Server and Listener, so the reconnect and worker tests bind :0 instead of pick-and-rebind.
• m1 NVENC teardown [S]: a rejected NVENC encode (P7 with HQ tuning) hangs shutdown. Found in feat(video,audio): encoder latency presets with applied-control reporting #4099.
• m2 noq socket close [M]: upstream the socket release so moq-tokio drops the wrapper from fix(moq-tokio): release the QUIC socket before Listener::close returns #4087.
• m2 NVENC buffer pool [M]: stop allocating per frame. Benchmark-gated.
• m2 AAC encode refusal [S]: the encode half of fix(mux): parse the AAC program config element instead of guessing stereo #4093, a dev break.

These follow-up edits ride on the open PRs that caused them instead of landing here: the publisher-clock split (#4082), the TS export PCE quest and PCE-to-Layout note (#4093), C++ shutdown (#4100), and keeping #4073's delta_ratio (#4089).

Impact

None (quest text only). The lite-07 quest plans a wire change to an unpublished draft version.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

Next included review available in 47 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: cc2be809-dce4-46ec-b248-cb3c072c9f05

📥 Commits

Reviewing files that changed from the base of the PR and between f591b9c and cc20663.

📒 Files selected for processing (10)

• quest/m1/README.md
• quest/m1/lite-stream-count.md
• quest/m1/nvenc-teardown.md
• quest/m1/rust-track-tail.md
• quest/m1/signal-race.md
• quest/m1/tokio-reconnect-ports.md
• quest/m2/README.md
• quest/m2/aac-encode-refusal.md
• quest/m2/noq-socket-close.md
• quest/m2/nvenc-pool.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	✅ Completed 2026-09-25T05:29:48.129587Z	cc20663	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

Positive improvement: yes. This turns concrete follow-ups from the next-16 quest run into durable m1/m2 quests instead of leaving them buried in PR threads. Impact is quest text only.

Worth the complexity: yes. Seven sized quests ([S]/[M]), README index lines, and one Required link on rust-track-tail. No production code. The heavier items are scoped sensibly: lite-07 is unpublished so the SUBSCRIBE_DROP → stream-count change can land in place; NVENC pool is abandon-if-benchmark-loses; AAC encode refusal correctly targets dev for the published API break; noq socket close waits on an upstream release in m2.

Different approach: keeping the open-PR follow-up edits on #4082 / #4093 / #4100 / #4089 (rather than restating them here) is the right split. Issues would be weaker than quests in this repo’s planning model.

Small nits only (non-blocking):

• signal-race correctly gates on fix(js): stop retaining a listener, reaction, or task per frame #4085; worth confirming that PR is merged or still Required before anyone dispatches it.
• nvenc-teardown names good suspects and rejects a join-timeout “fix”; keep that bar in the eventual PR.

Recommendation: merge as planning hygiene.

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

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cc2066312b

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

Add the required concept-doc sync

This quest changes the rs/moq-net wire format but only schedules the draft update, so it can be completed while leaving the user-facing doc/concept description out of sync. Include the applicable concept documentation in the quest scope alongside the Rust, JavaScript, and draft changes.

AGENTS.md reference: AGENTS.md:L94-L98

Useful? React with 👍 / 👎.

[chatgpt-codex-connector]

Require the NVENC regression test to run in CI

When no nightly GPU lane exists, the otherwise clause permits completing this quest by merely documenting where the hardware test runs, which can leave the regression test manual and unable to catch future teardown hangs. Require creating or using an automated CI lane, at least nightly, or define a CI-runnable alternative rather than accepting non-CI coverage.

AGENTS.md reference: AGENTS.md:L18-L23

Useful? React with 👍 / 👎.