Repository navigation
quest: delete js-imquic-objects now that the runner ships @moq/net 0.4.7 - #7104
Conversation
Quest-Claim: 151ae109821b512f
…rage/js-imquic-objects
@moq/net 0.4.7 shipped from release with #6769, and englishm/moq-interop-runner#179 moves the runner's moq-dev-js client to it. Rewrite the quest to wait for that PR to merge and the client image to be rebuilt. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
englishm/moq-interop-runner#179 merged and the client image was rebuilt; data-subgroup-basic passes against imquic in all three cells of the 2026-10-31 nightly. 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 15 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. |
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. |
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed commit: ef20a7a
No actionable bugs found in the three-file documentation diff and surrounding quest context. Removing the completed subgroup-header quest and its Required link is a focused simplification; retaining the distinct announce-subscribe failure in the recurring audit preserves its ownership rather than treating all imquic interoperability as fixed.
Verification limits: static GitHub inspection only. I did not run quest checks, builds, or tests, or independently reproduce the cited upstream nightly results.
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
- Delete
quest/m1/interop-coverage/js-imquic-objects.md— removes the quest and its bullet in the interop coverage epic's## Requiredlist. The condition (@moq/net0.4.7 carrying#6769) has cleared. - Update
quest/m2/audit-interop.md— records thatmeetecho/imquic#43is closed, thatdata-subgroup-basicpasses in all three moq-dev-js → imquic cells of the 2026-10-31 nightly, and thatannounce-subscribestill fails there.
I verified the external claims: moq-dev/moq#6769 is merged to release (title "write IETF data stream headers in one write (backport #6504)"), @moq/net latest is 0.4.7, and meetecho/imquic#43 closed 2026-10-26. No dangling reference to js-imquic-objects remains in the repo, so quest check link validation is unaffected. Clean deletion, accurate prose.
deepseek-v4.1-flash (free via Pullfrog for OSS) | 𝕏
|
Landing summary: this PR deletes the finished
(Written by Claude Opus 5.5) |
|
quest land removed this pull request from the queue: the batch check failed (exit 100). Checking http-cache v1.0.0-alpha.8
Checking pipewire v0.10.1
Compiling proc-macro2-diagnostics v0.10.1
Checking moq-sock v0.2.0 (/work/rs/moq-sock)
Checking fs-err v3.3.1
Checking p384 v0.14.0
Checking p256 v0.14.0
Checking arboard v3.7.0
Checking reqwest-middleware v0.5.2
Checking web-transport-trait v0.5.2
Checking moq-auth v0.2.1 (/work/rs/moq-auth)
Checking axum v0.8.9
Checking axum-server v0.8.0
Checking egui-wgpu v0.36.2
Checking http-cache-reqwest v1.0.0-alpha.9
Checking egui-winit v0.36.2
Compiling duplicate v2.0.1
Checking eframe v0.36.2
Checking egui_dock v0.21.1
Checking cpal v0.18.2
Checking moq-json v0.5.8 (/work/rs/moq-json)
Checking moq-loc v0.2.19 (/work/rs/moq-loc)
Checking qmux v0.7.0 (/work/rs/qmux)
Checking hang v0.21.11 (/work/rs/hang)
Checking moq-stats v0.2.12 (/work/rs/moq-stats)
Checking moq-tokio v0.19.23 (/work/rs/moq-tokio)
Checking moq-mux v0.10.11 (/work/rs/moq-mux)
Checking moq-relay v0.17.2 (/work/rs/moq-relay)
Checking moq-audio v0.1.10 (/work/rs/moq-audio)
Checking moq-scene v0.1.0 (/work/rs/moq-scene)
Checking moq-studio v0.1.0 (/work/rs/moq-studio)
Finished `dev` profile [unoptimized + debuginfo] target(s) in 1m 07s
info: running `cargo clippy --locked --tests --all-features -- -D warnings` on moq-wasm (50/51)
Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.39s
info: running `cargo clippy --locked --tests --all-features -- -D warnings` on uniffi-bindgen (51/51)
Compiling getrandom v0.4.3
Compiling winnow v1.0.4
Compiling uniffi_internal_macros v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Compiling serde v1.0.229
Compiling libc v0.2.191
Checking indexmap v2.14.2
Compiling fs-err v3.3.1
Compiling askama_parser v0.16.1
Compiling basic-toml v0.1.10
Checking nom v7.1.3
Compiling scroll_derive v0.12.1
Checking tempfile v3.27.0
Checking smawk v0.3.3
Compiling askama_derive v0.16.1
Checking serde_json v1.0.154
Checking weedle2 v5.0.0 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Compiling toml_parser v1.1.3+spec-1.1.0
Checking textwrap v0.16.4
Checking uniffi_pipeline v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Checking scroll v0.12.0
Checking uniffi_meta v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Compiling askama_macros v0.16.1
Compiling toml_datetime v1.1.1+spec-1.1.0
Checking plain v0.2.3
Compiling toml v1.1.6+spec-1.1.0
Checking clap_builder v4.6.7
Checking uniffi_udl v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Checking goblin v0.8.2
Checking cargo_metadata v0.23.1
Checking askama v0.16.1
Checking uniffi_core v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Checking clap v4.6.7
Checking uniffi_bindgen v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Compiling uniffi_macros v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Checking uniffi v0.32.2 (https://github.com/kixelated/uniffi-rs?rev=88d835d#88d835d6)
Checking uniffi-bindgen v0.1.0 (/work/rs/uniffi-bindgen)
Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.79s
mbx[cache]: object cache: 1219 hits, 0 misses, 209 not looked up; 0 B downloaded, 0 B uploaded, 408.1 MiB stored locally
mbx[cache]: 10m 13s session; 17m 18s estimated compiler time avoided (summed across compilations)
mbx[cache]: 209 compilations had no usable prior inputs or matching prediction for a lookup
mbx[cache]: Cargo artifact reuse and CI cache archive transfers are not included above
mbx[savings]: 2.3 TiB in every checkout, 2.3 TiB on disk. arithmetic declined to comment.
ci: test rs failed
ci: seconds per module
614 check rs
307 test rs (failed)
1262 check rs_shapes
237 check js
46 test js
14 check markdown
3 check gh
48 test gh
66 check flake
0 check drill
5 check quest
9 check ts_paths
21 check shell
1 check large_files
0 check toml
1 check nix
0 check justfile
2183 wall time
error: recipe `ci` failed on line 104 with exit code 100
. |
|
Codex land fixer: moq-doc:check | (pass) tables render as tables > draft-lcurley-moq-timestamp [5.56ms] moq-doc:check | (pass) tables render as tables > draft-lcurley-moq-use-cases [10.60ms] moq-doc:check | (pass) tables render as tables > draft-lcurley-qmux-websocket [5.88ms] moq-doc:check | (pass) tables render as tables > row separators do not split a table apart [54.24ms] moq-doc:check | moq-doc:check | .vitepress/api/index.test.ts: moq-doc:check | (pass) a rendered workspace > each entrypoint gets a page named by its import specifier [6.76ms] moq-doc:check | (pass) a rendered workspace > a re-export from another workspace package links to its owner [6.54ms] moq-doc:check | (pass) a rendered workspace > an external re-export is documented under each public name [29.59ms] moq-doc:check | (pass) a rendered workspace > a namespace export keeps its public name [2.52ms] moq-doc:check | (pass) a rendered workspace > a hidden declaration is documented under the alias that re-exports it [2.59ms] moq-doc:check | �[93m[warning]�[0m Value, defined in @test/hidden/src/access.ts, is referenced by read but not included in the documentation moq-doc:check | �[93m[warning]�[0m Value, defined in @test/hidden/src/access.ts, is referenced by read but not included in the documentation moq-doc:check | (pass) an unexported declaration that a public one names fails the run [381.26ms] moq-doc:check | �[93m[warning]�[0m Failed to resolve link to "Missing" in comment for value moq-doc:check | (pass) a link that does not resolve fails the run [324.62ms] moq-doc:check | moq-doc:check | concept/audio-jitter/corpus.test.ts: moq-doc:check | (pass) audio jitter corpus > paced > matches the checked-in file [0.20ms] moq-doc:check | (pass) audio jitter corpus > paced > is a trace of equal-length clocks [0.03ms] moq-doc:check | (pass) audio jitter corpus > paced > arrives in a monotonic order [0.76ms] moq-doc:check | (pass) audio jitter corpus > paced > names a target for the first frame [0.02ms] moq-doc:check | (pass) audio jitter corpus > paced > reports every target as a change [0.08ms] moq-doc:check | (pass) audio jitter corpus > flush > matches the checked-in file [0.03ms] moq-doc:check | (pass) audio jitter corpus > flush > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > flush > arrives in a monotonic order [0.16ms] moq-doc:check | (pass) audio jitter corpus > flush > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > flush > reports every target as a change moq-doc:check | (pass) audio jitter corpus > advertised > matches the checked-in file [0.03ms] moq-doc:check | (pass) audio jitter corpus > advertised > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > advertised > arrives in a monotonic order [0.16ms] moq-doc:check | (pass) audio jitter corpus > advertised > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > advertised > reports every target as a change moq-doc:check | (pass) audio jitter corpus > buildup > matches the checked-in file [0.04ms] moq-doc:check | (pass) audio jitter corpus > buildup > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > buildup > arrives in a monotonic order [0.24ms] moq-doc:check | (pass) audio jitter corpus > buildup > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > buildup > reports every target as a change [0.01ms] moq-doc:check | (pass) audio jitter corpus > reorder > matches the checked-in file [0.03ms] moq-doc:check | (pass) audio jitter corpus > reorder > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > reorder > arrives in a monotonic order [0.13ms] moq-doc:check | (pass) audio jitter corpus > reorder > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > reorder > reports every target as a change moq-doc:check | (pass) audio jitter corpus > backlog > matches the checked-in file [0.05ms] moq-doc:check | (pass) audio jitter corpus > backlog > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > backlog > arrives in a monotonic order [0.29ms] moq-doc:check | (pass) audio jitter corpus > backlog > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > backlog > reports every target as a change moq-doc:check | (pass) audio jitter corpus > idle > matches the checked-in file [0.03ms] moq-doc:check | (pass) audio jitter corpus > idle > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > idle > arrives in a monotonic order [0.13ms] moq-doc:check | (pass) audio jitter corpus > idle > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > idle > reports every target as a change moq-doc:check | (pass) audio jitter corpus > tunein > matches the checked-in file [0.04ms] moq-doc:check | (pass) audio jitter corpus > tunein > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > tunein > arrives in a monotonic order [0.19ms] moq-doc:check | (pass) audio jitter corpus > tunein > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > tunein > reports every target as a change moq-doc:check | (pass) audio jitter corpus > spike > matches the checked-in file [0.07ms] moq-doc:check | (pass) audio jitter corpus > spike > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > spike > arrives in a monotonic order [0.33ms] moq-doc:check | (pass) audio jitter corpus > spike > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > spike > reports every target as a change moq-doc:check | (pass) audio jitter corpus > straggler > matches the checked-in file [0.04ms] moq-doc:check | (pass) audio jitter corpus > straggler > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > straggler > arrives in a monotonic order [0.17ms] moq-doc:check | (pass) audio jitter corpus > straggler > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > straggler > reports every target as a change moq-doc:check | (pass) audio jitter corpus > fixed > matches the checked-in file [0.02ms] moq-doc:check | (pass) audio jitter corpus > fixed > is a trace of equal-length clocks moq-doc:check | (pass) audio jitter corpus > fixed > arrives in a monotonic order [0.09ms] moq-doc:check | (pass) audio jitter corpus > fixed > names a target for the first frame moq-doc:check | (pass) audio jitter corpus > fixed > reports every target as a change moq-doc:check | moq-doc:check | 130 pass moq-doc:check | 0 fail moq-doc:check | 15955 expect() calls moq-doc:check | Ran 130 tests across 3 files. [5.08s] moq-doc:check | API reference: 17 packages in 59.0s moq-doc:check | moq-doc:check | �[32m�[1mvitepress�[22m v1.6.4�[39m moq-doc:check | moq-doc:check | - building client + server bundles... moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m �[1mvitepress-plugin-llms�[22m initialized �[2m(client build)�[22m with workDir: �[36m/work/doc�[39m moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m Build started, file collection cleared moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m Processing �[1m123�[22m markdown files from �[36m/work/doc�[39m moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m Generating �[36mllms.txt�[39m... moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m Generating full documentation bundle (�[36mllms-full.txt�[39m)... moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed concept/audio-jitter.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed lib/c.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed draft/moq-flate.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed bin/demo.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed lib/rs/hang.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed concept/use-case/ai.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed bin/relay/http.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed lib/js/api.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Generated �[36mllms.txt�[39m (~�[1m4.6K�[22m tokens, �[1m14.2 kB�[22m) with �[1m123�[22m documentation links moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed setup/agent.md moq-doc:check | �[34mllmstxt�[39m�[2m » �[22m�[32m✓�[39m Processed bin.md moq-doc:chequest: fixer agent exited 101 (written by Codex) |
|
quest land removed this pull request from the queue: the batch check failed (exit 1). Checking egui v0.36.2
Checking moq-congestion v0.1.0 (/work/rs/moq-congestion)
Checking ecdsa v0.17.0
Checking tungstenite v0.30.0
Checking calloop-wayland-source v0.4.1
Checking wgpu v30.0.1
Checking scuffle-h265 v0.2.2
Checking primeorder v0.14.0
Checking zbus_names v4.3.4
Checking smithay-client-toolkit v0.19.2
Checking hang v0.21.11 (/work/rs/hang)
Compiling moq-vaapi v0.1.0
Checking webm-iterable v0.7.1
Checking scuffle-av1 v0.1.4
Checking moq-loc v0.2.19 (/work/rs/moq-loc)
Checking wayland-protocols-experimental v20250721.0.1
Checking wayland-protocols-plasma v0.3.12
Checking wayland-protocols-misc v0.3.12
Checking sha2 v0.11.1
Checking http-serde v2.1.1
Checking moq-msf v0.5.1 (/work/rs/moq-msf)
Checking toml v1.1.6+spec-1.1.0
Checking winit v0.30.13
Checking humantime-serde v1.1.1
Checking moq-quic v0.1.0 (/work/rs/moq-quic)
Checking moka v0.12.16
Checking http-cache-semantics v3.0.0
Checking moq-mux v0.10.11 (/work/rs/moq-mux)
Checking zbus v5.19.0
Checking postcard v1.1.3
Checking smithay-client-toolkit v0.20.0
Checking tokio-tungstenite v0.30.0
Checking sfv v0.15.0
Checking spki v0.7.3
Compiling moq-video v0.1.10 (/work/rs/moq-video)
Checking jsonwebtoken v11.1.0
Checking axum-core v0.5.6
Checking x11rb v0.14.0
Checking serde_urlencoded v0.7.1
Checking pkcs1 v0.7.5
Checking qmux v0.7.0 (/work/rs/qmux)
Checking smithay-clipboard v0.7.3
Checking web-transport-proto v0.6.3
Checking rcgen v0.14.10
Checking ashpd v0.13.13
Checking http-cache v1.0.0-alpha.8
Compiling proc-macro2-diagnostics v0.10.1
Checking moq-sock v0.2.0 (/work/rs/moq-sock)
Checking fs-err v3.3.1
Checking p384 v0.14.0
Checking p256 v0.14.0
Checking arboard v3.7.0
Checking reqwest-middleware v0.5.2
Checking web-transport-trait v0.5.2
Checking egui-wgpu v0.36.2
Checking egui-winit v0.36.2
Checking axum v0.8.9
Checking axum-server v0.8.0
Checking moq-auth v0.2.1 (/work/rs/moq-auth)
Checking moq-tokio v0.19.23 (/work/rs/moq-tokio)
Compiling duplicate v2.0.1
Checking moq-stats v0.2.12 (/work/rs/moq-stats)
Checking http-cache-reqwest v1.0.0-alpha.9
Checking eframe v0.36.2
Checking egui_dock v0.21.1
Checking moq-relay v0.17.2 (/work/rs/moq-relay)
Checking pipewire v0.10.1
Checking cpal v0.18.2
Checking moq-audio v0.1.10 (/work/rs/moq-audio)
Checking moq-scene v0.1.0 (/work/rs/moq-scene)
Checking moq-studio v0.1.0 (/work/rs/moq-studio)
Finished `dev` profile [unoptimized + debuginfo] target(s) in 7.20s
info: running `cargo clippy --locked --tests --all-features -- -D warnings` on moq-wasm (33/33)
Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.24s
mbx[cache]: object cache: 1301 hits, 3 misses, 40 not looked up, 3 bypassed; 0 B downloaded, 0 B uploaded, 86.4 KiB stored locally
mbx[cache]: 1m 12s session; 24m 44s estimated compiler time avoided (summed across compilations)
mbx[cache]: 40 compilations had no usable prior inputs or matching prediction for a lookup
mbx[cache]: bypass reasons: 3 build-script-declarations-pending
mbx[cache]: Cargo artifact reuse and CI cache archive transfers are not included above
mbx[savings]: 9.0 TiB logical of build debris binned so far. cargo clean remains unemployed.
ci: check quest failed
ci: seconds per module
301 check rs
113 test rs
134 check rs_shapes
187 check js
13 test js
24 check markdown
6 check gh
91 test gh
0 check drill
3 check quest (failed)
22 check shell
0 check large_files
0 check toml
1 check nix
0 check justfile
551 wall time
error: recipe `ci` failed on line 106 with exit code 1
. |
|
codex / Codex land fixer: (written by codex / Codex) |

Deletes quest/m1/interop-coverage/js-imquic-objects.md. Its condition has cleared.
Why it's done
@moq/net0.4.7, which writes data stream headers in one write.main8dab46a, which contains WIP: draft-05 updates #179, builtmoq-dev-js-clientsuccessfully at 2026-10-28 18:11 UTC. That run's overall failure comes only from themoq-rs-draft-18-*jobs.shard-imquic) shows the client banner@moq/net 0.4.7, andok 8 - data-subgroup-basicin all three moq-dev-js → imquic cells: docker, remote-webtransport, and remote-quic.@moq/net 0.4.4, the docker cell hadnot ok 8 - data-subgroup-basic.Changes
quest rm quest/m1/interop-coverage/js-imquic-objects.mddropped the quest and its entry in the interop coverage epic.quest rmleaves for hand-editing. It now records that imquic#43 is closed and thatdata-subgroup-basicpasses. It also notes that moq-dev-jsannounce-subscribestill fails in all three imquic cells of the 2026-10-31 nightly. The audit already owns classifying that failure.origin/main. The conflict was in the epic'sRequiredlink list, and the quest merge driver resolved it with no hand edits.Checks
quest checkandquest check --base origin/main: ok.just check origin/main: passes (markdown, gh, quest, shell, toml, nix, justfile).Review
Skipped: a trivial quest deletion. The diff against
maindeletes one quest file and its link, plus a two-sentence prose update in the interop audit. Skip recorded for head ef20a7a by Claude Code (Claude Opus 5.5).Suggested follow-ups
subscribe-before-announcein both remote imquic cells (lminiero.it), andpublish-namespace-donein the remote-quic cell. All three pass in the docker cell. No quest tracks these yet. They belong to the next interop audit run.🤖 Generated with Claude Code
(Written by Claude Opus 5.5)