Repository navigation
quest(m1): reconcile the outstanding quest tree #6268
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
9a7240f
chore(quest): claim quest/m1/audit-quests.md
kixelated b4974a5
quest(m1): rank timestamp provenance before passthrough
kixelated 1235529
quest(m1): correct replacement linger rationale
kixelated b6a9ea5
quest(m3): align binary plans with compression migration
kixelated 4f9ff2a
quest(m1): refresh the release size report prerequisite
kixelated c26c30b
quest(m3): refresh the unresolved WebKit condition
kixelated da7ee4f
quest(m1): retain only the remaining OBS load proof
kixelated 3b778bf
quest(m1): preserve the decided frozen C range adapter
kixelated 7cee253
quest(m1): distinguish discovery from connection linger
kixelated 0d8b414
quest(m2): reflect the decided NVIDIA queue gate
kixelated 0ca98d6
quest(m0): retain the moved failover quest ownership
kixelated 30924dd
quest(m2): carry the settled track compression migration forward
kixelated e8a3686
quest(m2): refresh recurring audit checkpoints from settled work
kixelated 83e729f
quest(m2): point performance plans at current measuring paths
kixelated a2ea0e6
quest(m2): encode two explicit implementation prerequisites
kixelated 377def5
quest(m2): retain bounded compression through front eviction
kixelated 9af9a6c
quest(m1): finish the tree audit and schedule its release successor
kixelated 5c87652
Merge origin/main into quest/m1/audit-quests
kixelated 495804f
Merge origin/main into quest/m1/audit-quests
kixelated 201f83c
Merge origin/main into quest/m1/audit-quests
kixelated File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,73 @@ | ||
| # [M] Quest tree audit | ||
|
|
||
| ## Goal | ||
|
|
||
| The quest tree agrees with itself and with `main`: done, stale, | ||
| conflicting, and misranked quests are fixed. | ||
|
|
||
| ## Plan | ||
|
|
||
| Recurs once per major release (decided 2026-10-19). The PR that finishes | ||
| this run also adds the next one with the same file name at the end of the next | ||
| milestone's `Required` (m0 to m1, m1 to m2). The successor keeps the Goal | ||
| and this paragraph; the rest of its Plan is rewritten with what this run | ||
| learned: baselines, results, and what to check first. A successor placed in m2 | ||
| also lists the [release cut](/quest/m2/release/cut.md) under `Required`, so it | ||
| waits for the release in flight to ship. Only `/quest-delete` ends the | ||
| recurrence. Fixes land in the run's PR, one commit each, rather than as follow-ups. | ||
| Only a change that needs a maintainer decision (public API, scope) or is | ||
| independently landable and too large to review alongside the rest becomes a | ||
| follow-up through `/quest-plan`. | ||
|
|
||
| Run `/quest-audit` after the release cut. Read every outstanding quest with | ||
| parallel auditors, then verify source claims, active PR ownership, Required | ||
| ordering, and the latest maintainer decisions before changing plans. | ||
|
|
||
| Baseline from the m1 run, 2026-10-22 UTC, against `main` at `bce217e063`: | ||
|
|
||
| - Read all 463 quest documents across m0-m3 and the root with parallel | ||
| read-only support. Checked 315 referenced GitHub items and the open-PR | ||
| inventory. Source checks were targeted to the factual claims; hardware | ||
| and deployment acceptance was not inferred from old inventory text. | ||
| - Fixed timestamp provenance's rank before its IETF passthrough consumer, | ||
| the generated-C exception for dynamic range fills, discovery's fixed 2s | ||
| linger, and replacement linger's existing-reader semantics. | ||
| - Carried m1's settled compression migration into deferred plans: one | ||
| compressed catalog/stats track, ordinary tracks for opaque bytes, no | ||
| moq-flate or `moq-mux::binary` helper, authenticated logical compression | ||
| in the e2ee wrapper, and bounded prefix reconstruction for front eviction. | ||
| - Refreshed partly landed benchmark/report baselines and explicit | ||
| prerequisites for the JS encrypted-wrapper mirror and fMP4 ambisonics. | ||
| - The JS allocation replacement passed all 2,493 js/net tests under load. | ||
| The OBS drain failure reproduced on unchanged main and passed with the | ||
| existing close-tail PR; its quest keeps the remaining merged-result proof. | ||
| - No m2 implementation quest was started, no external-organization report | ||
| was posted, and ambiguous remote claim branches were left intact. | ||
|
|
||
| Check first next run: | ||
|
|
||
| - A moved quest can still be owned by a PR under its old branch name. The | ||
| redundant failover quest moved to m0 while | ||
| [moq-dev/moq#6113](https://github.com/moq-dev/moq/pull/6113) retained its | ||
| m1 branch. Search PR titles, bodies, and source scope as well as exact | ||
| branch names before taking a claim. Do not infer abandonment from age. | ||
| - Refresh all linked open PRs before changing ownership or deleting a | ||
| quest. At this baseline, the TRACK_STATUS retirement, silent-peer | ||
| retirement, audio conformance, CI audit, and local gate each had owners; | ||
| an overlapping cleanup should finish through that PR. | ||
| - Confirm the first scheduled size report after the Linux release-link | ||
| fix, the required count of post-change audio-budget nights, shared FFI | ||
| publication, and release-gate availability. Partial evidence does not | ||
| clear an external-condition quest. Safari WebKit bug 319818 was still | ||
| NEW with no fixed shipping version established. | ||
| - Recheck local gate and binding-cost plans together: Bun `--no-orphans` | ||
| reaps newly spawned local descendants even after successful completion, | ||
| while the CI bypass preserves persistent queue daemons. A warm local | ||
| daemon policy needs an explicit choice; no dependency edge was invented. | ||
| - Re-run the transitive Required rank check, including prerequisites added | ||
| within an epic, and compare plans against replaced designs before looking | ||
| for new implementation work. | ||
|
|
||
| ## Required | ||
|
|
||
| - [Release cut](/quest/m2/release/cut.md) - the release in flight ships before this run starts | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This closes the m1 audit and defers its successor until after the release, but the recorded pass targets
bce217e063while this commit's parent is2d73eea, 26 commits later. Between those revisions the quest tree gained 20 documents, deleted 13, and changed numerous existing plans, so the stated 463-document pass does not cover the 470-document tree being declared reconciled. Re-run the audit on the actual parent before removing the current gate.AGENTS.md reference: AGENTS.md:L13-L16
Useful? React with 👍 / 👎.