Repository navigation
quest: import ten open issues - #5969
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f022892f9c
ℹ️ 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".
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
SummaryImports nine open issues as quests and declines #5912. Every decision is in the PR body with ✅. After review:
Codex 👍 on 3bf0ddf; CI green. Enabling auto-merge. (Written by Claude Opus 5.5) |
Summary
Imports every open issue without the
questlabel. Each one was checked against the code first, and the reporters' proposed fixes were weighed against alternatives.New quests
quest/m1/stateless-reset-key.md[S]: land feat(quic)!: derive the stateless reset key from listen.reset.secret #5910 (a filelisten.reset_key, HKDF into the reset and connection ID keys). For relay: a restarted relay can't stateless-reset its predecessor's connections, so clients wait out the idle timeout #5861.quest/m1/track-status-race.md[S]: a subscribing reader's subscription lands with its request, so it never asks TRACK_STATUS. Deletes the NOT_SUPPORTED fallback, files upstream issues for relays that refuse TRACK_STATUS. For ietf: a TRACK_STATUS refused with any code but NOT_SUPPORTED fails the reader's subscription #5954.quest/m1/ietf-max-age.md[S]: a new SUBSCRIBE/REQUEST_UPDATE parameter in draft-lcurley-moq-timestamp carrying lite's max age. The IETF delivery timeouts stay dropped. For IETF subscribers cannot bound delivery lateness: SUBSCRIBE delivery timeouts are dropped and no max_delay maps from the wire #5881.quest/m1/ts-reliability/aud-first.md[XS]: the AUD stays ahead of re-injected SPS/PPS. For export ts: keyframe parameter sets are written ahead of the access unit delimiter #5951.quest/m1/ts-reliability/source-front-end.md[XS]: cli.md says to pipe a TS source's bytes, and the first PMT logs each stream's outcome. For Docs: piping a broadcast TS through ffmpeg -c copy drops most of its PIDs before import ts #5950.quest/m2/quic-outage-probe.md[S]: re-arm the probe timer when the peer is heard again. For QUIC settings for provisioned contribution paths: hold the rate through random loss, relax reorder thresholds, cap the probe backoff #5953.quest/m2/ts-hitless-h265.md[S]: the H.265 half of the HRD anchor.quest/m2/publish-announce-never.md[XS]: dropannounce="never". For JS moq-publish: Announce is required to be subscribed by moq-watch #5703.Updated quests
quest/m2/ts-hitless.md: records the HRD buffering-period anchor (feat(moq-mux): render identical TS legs withexport ts --sync#5949) and closesexport ts --sync: a group that began behind the one before it lays apart on a leg that lost that earlier group #5864.quest/m1/public-model/ingest.md: moq-net: no way to accept a track::Request with content, so a lazily served relay track can answer SUBSCRIBE_OK without Largest Location #5862 becomes a concrete case for the pre-accept shape; closure-styleaccept_withrejected.quest/m1/js-fetch.md: its deferred "delete the fallback" step moves to the TRACK_STATUS quest.Declined: #5912 (MMTP container in Hang).
Issues are mentioned without closing keywords; each quest's PR closes its issue.
Decisions
#5861: how to track the stateless-reset fix?
#5953: which provisioned-path QUIC settings?
#5951: where does the AUD fix go?
#5950: doc sentence and PMT log?
#5864: which placement rule? (#5963 checked: it does not change this; identity is scoped to legs of one broadcast)
export ts --sync#5949, H.265 as a follow-up#5954: TRACK_STATUS refusals
#5881: where does an IETF subscriber's budget come from?
#5862: where does the pre-accept shape go?
accept_withclosure shape)#5912: MMTP container
#5703: should
@moq/publishkeep announce="never"? (maintainer: a broadcast that never announces only works when client and server pre-share the intent to publish).-prefixed paths cover unlisted broadcasts)Checks
quest checkpasses (441 documents). Quests only, no code.(Written by Claude Opus 5.5)
🤖 Generated with Claude Code