Skip to content

chore(quest): delete MoQ on rind, finished in moq-dev/moq#8250 - #520

Merged
kixelated merged 3 commits into
mainfrom
quest/m1/moq/moq-on-rind
Nov 10, 2026
Merged

kixelated merged 3 commits into
mainfrom
quest/m1/moq/moq-on-rind

Conversation

@kixelated

@kixelated kixelated commented Nov 9, 2026 •

Copy link
Copy Markdown
Collaborator

Deletes the wait-only quest quest/m1/moq/moq-on-rind.md and its link in quest/m1/moq/README.md. The condition holds.

Verification

  • MoQ's quest/m1/rind/README.md does not exist on moq-dev/moq main. The only rind quest left there is quest/m5/rind-instant-acme.md.
  • It was finished, not moved. Its own work, removing aws-lc-rs and ring from the workspace, adding a CI guard against C crypto, and shipping an armv7 moq-cli, landed in moq-dev/moq#8250, the commit that removed the epic. Its children (transport.md, cpu-error.md, rtc.md) were removed by merged feature commits, and the workspace Cargo.toml declares no aws-lc-rs or ring.
  • rind-instant-acme is not the epic's remaining work. The epic's Plan removed relay ACME on 2026-11-03 and deferred its return to m5, so that quest tracks a new feature, not this epic.

Changes

  • Delete quest/m1/moq/moq-on-rind.md, and drop its Required entry from quest/m1/moq/README.md. The epic keeps quic-path, so its Required is not empty.
  • Replace the dead link to MoQ's migration epic in quest/m1/moq/README.md with a link to chore(rind): drop aws-lc-rs, guard against C crypto, and ship moq-cli for armv7 moq#8250, and scope its "no MoQ area waits on rind" sentence to the switch.
  • quest/m1/README.md needs no change: its rind covers MoQ entry is still open.

Decisions

  • Review finding on quest/m1/moq/README.md:14 (pullfrog, AI-attributed): accepted. moq's quest/m5/rind-instant-acme.md still waits on rind publishing rind-instant-acme for relay ACME, so the unscoped sentence overreached. ✅ scope it to the switch, with the reviewer's wording (rejected: keep the unscoped sentence).

Public API

None. Quest files only.

Checks

  • quest check --base origin/main: 147 documents ok.
  • just check: exit 0.

Models

Size: XS
Implementer: Claude, claude-haiku-5-5, high effort
Implementer: Claude Code, claude-haiku-5-5, high effort (iterate, XS)

🤖 Generated with Claude Code

(Written by Haiku 5.5 with high)

Review

Rewritten by quest review after each round.

  • Reviewer: Codex, gpt-6.1-sol, low effort
  • Session: 01a1c2b1-6171-7c92-a203-86dc5dcaf3c6, round 1
  • Head: 3bc00f3
  • Base: origin/main at 3d06e76
  • Score: 10/10, The scoped diff only updates migration tracking documentation and removes the completed quest, with no executable code changes or actionable defects identified.
  • Findings: none

kixelated and others added 2 commits November 8, 2026 23:00
MoQ's rind epic is gone from moq-dev/moq's quest tree: its last work, dropping
aws-lc-rs and ring and shipping an armv7 moq-cli, landed there, and every child
merged before it. Its waiting item is removed, and the epic's Plan links to
moq-dev/moq's migration epic, a file that no longer exists, so it names the PR
that finished the switch instead.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@kixelated kixelated changed the title MoQ runs on rind chore(quest): delete MoQ on rind, finished in moq-dev/moq#8250 Nov 9, 2026
@kixelated
kixelated marked this pull request as ready for review November 9, 2026 07:04
@coderabbitai

coderabbitai Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

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 15 seconds.

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: 81658f3c-2605-412b-83b4-6abed03bdfee
Commits

Reviewing files that changed from the base of the PR and between ccdda85 and 3bc00f3.

Files selected for processing: 2

  • quest/m1/moq/README.md
  • quest/m1/moq/moq-on-rind.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

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ No critical issues — one wording-precision suggestion inline.

Reviewed changes

  • Wait-only quest deleted: quest/m1/moq/moq-on-rind.md is gone along with its Required bullet in quest/m1/moq/README.md, which keeps quic-path (the file exists) so the section isn't empty.
  • Dead link replaced: the epic README's link to MoQ's quest/m1/rind/README.md, deleted upstream, now points at merged moq-dev/moq#8250 and records that the switch finished.
  • Upstream claims verified: moq-dev/moq#8250 is merged (2026-11-09) and its body confirms it was the epic's own work — aws-lc-rs and ring out of the workspace, a CI guard against C crypto, the armv7 moq-cli release — and that finishing it deletes the epic. moq's main tree (untruncated recursive listing) contains no quest/m1/rind/ and exactly one rind quest left, quest/m5/rind-instant-acme.md, which is new m5 feature work (instant-acme 0.9 / relay ACME), not this epic's remains.
  • Tree consistency checked by hand: quest check isn't runnable on this runner, so consistency was verified manually — no stale moq-on-rind references or links into moq's quest tree remain anywhere in this repo, quest/m1/README.md's still-open rind covers MoQ entry correctly needs no change, and the epic Goal prose was updated in step with the dropped bullet.

Pullfrog  | Fix all ➔ | Fix 👍s ➔ | View workflow run | Using glm-5.3 | 𝕏

Comment thread quest/m1/moq/README.md Resolved
rind-instant-acme's quest in moq-dev/moq (m5, relay ACME) still waits on
rind publishing that crate, so "no MoQ area waits on rind" overreached.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@kixelated
kixelated added this pull request to the merge queue Nov 9, 2026
Merged via the queue into main with commit 78353e6 Nov 10, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant