[kixelated]

Problem

moq-dev/studio#123 pins pin-sync exactly at moq 644d48996. moq's main (103052ed0 on 2026-11-09) is 117 first-parent commits past that, 27 of them breaking, including moq-dev/moq#8220 (kio's Instant loses +/- with a Duration, used throughout playout.rs and source.rs), moq-dev/moq#8270, and moq-dev/moq#8372. Without a quest for them, the first wait-gated XS pin to clear absorbs all of them.

Approach

• Adds quest/m0/pin-main.md [L]: the submodule moves to moq's main on the day the quest starts, after pin-sync, with every break fixed in one PR.
• Every pin past pin-sync now requires it instead of pin-sync: pin-aac-encode, pin-audio-filters, pin-audio-tap, pin-egress-dial, pin-portal-token, m2/compose/hang-layers, m2/pin-browser-audio, m2/pin-browser-video.

Decisions

Maintainer, 2026-11-09 (/quest-plan on #123's follow-up):

• Goal: ✅ moq main on the day it starts, via just upgrade-moq (rejected: a commit fixed now, which goes stale; dropping it and letting the first small pin absorb the catch-up).
• Ordering: ✅ later pins require it (rejected: leaving them on pin-sync).
• Milestone: ✅ m0 (rejected: m1).

Choices made without asking

• pin-audio-tap, pin-portal-token, and the two m2 browser pins also moved onto pin-main. The prompt named the AAC, filters, egress, and hang-layers pins, but these follow the same rule from quest: audit corrections, and the maintainer's decisions on what the audit raised #123 ("every pin whose upstream work lands later").

Impact

Quest text only. Public API and wire: none.

Alternatives

Listed with each decision above.

Follow-ups

None.

Models

Size: XS
Implementer: Claude Code, claude-opus-5-5

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

Review

Rewritten by quest review after each round.

• Reviewer: Codex, gpt-6.1-sol, low effort
• Session: 01a1c453-c50f-7641-9c92-2e91e4a609aa, round 1
• Head: b6a3210
• Base: origin/main at b038b8f
• Score: 10/10, The planning-only change consistently routes later pin quests through the new catch-up quest, with no actionable defects found.
• Findings: none

[pullfrog]

  ^{｜ Review this ➔}

[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 12 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review 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: 62752fd2-8af3-40f5-b087-698826b828d6

Commits

Reviewing files that changed from the base of the PR and between b038b8f and b6a3210.

Files selected for processing: 11

• quest/m0/README.md
• quest/m0/pin-aac-encode.md
• quest/m0/pin-audio-filters.md
• quest/m0/pin-audio-tap.md
• quest/m0/pin-egress-dial.md
• quest/m0/pin-main.md
• quest/m0/pin-portal-token.md
• quest/m0/pin-sync.md
• quest/m2/compose/hang-layers.md
• quest/m2/pin-browser-audio.md
• quest/m2/pin-browser-video.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

[kixelated]

Automated review by review (OpenAI)

Reviewed commit: b6a3210

No actionable findings in the 11-file planning-only diff. The eight downstream quests consistently require pin-main, which retains pin-sync as its prerequisite; the index and pin-sync handoff agree. The documented no-argument upgrade command matches the current recipe and script.

Overall direction: the dedicated catch-up quest is coherent; no changes requested from this review.

Verification limits: GitHub-only static inspection of the diff and relevant repository files. Did not run quest context, quest check, just ci, builds, or runtime tests, or independently validate the historical upstream commit counts. This does not verify the future dependency migration.