Skip to content

quest: move quests that required a later milestone - #8645

Closed
kixelated wants to merge 1 commit into
mainfrom
quest-plan/milestone-order
Closed

kixelated wants to merge 1 commit into
mainfrom
quest-plan/milestone-order

Conversation

@kixelated

@kixelated kixelated commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Moves the quests whose Required pointed to a later milestone. quest unblocked ranks a blocker by its own milestone, so each of these stalled behind all of the later milestone's work. moq-dev/quest is planning a quest check rule that rejects such links (moq-dev/quest#698), and this clears moq's tree before that rule ships.

  • m1/buffered-ascending, m1/export-order (both require m2/lite07-finalize), m1/receive-windows-evicted (requires m2/group-drops), m1/route-trace-validation (requires m2/cluster-routes/scope), and m1/routing-guide (requires m2/cluster-routing/federation) move to m3.
  • m5/bun-lint-release moves to m4, beside m4/formatter-defaults, which requires it.

Links are rewritten with quest mv, and each moved quest's Plan records the move. buffered-ascending, route-trace-validation, and routing-guide mark their earlier m1 placement as superseded.

Decisions

  • Where do the five m1 quests go?
    • ✅ m3 (rejected: m2, which m3/release/cut.md requires whole, so the cut would wait on additive work, and buffered-ascending had already rejected m2; pulling the blockers into m1, which drags 3 to 7 more breaking m2 quests along)
  • bun-lint-release vs formatter-defaults?
    • ✅ Pull bun-lint-release to m4 (rejected: push formatter-defaults to m5). m4's Goal already covers outside conditions that only later work needs.

Validation

quest check passes (500 documents). Re-scanning every Required entry finds no link to a later milestone.

Models

Size: S
Implementer: Claude Code, claude-opus-5-5, unknown effort

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

Review

Rewritten by quest review after each round.

  • Head: ae3ce6c

  • Base: origin/main at 7df2ebf

  • Reviewer: Codex, gpt-6.1-sol, medium effort

  • Session: 01a1d56d-5806-7b30-9b1a-b6276beb20a9, round 1

  • Score: 10/10, The planning changes preserve quest prerequisites and consistently update references to their new milestone locations, with no actionable regressions.

  • Rounds: 1; findings 0: fixed 0, refuted 0, open 0

  • Findings: none

Five m1 quests waited on m2 work and move to m3, and m5's bun-lint wait
moves to m4 beside the quest that requires it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

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 48 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews 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: 2c0528b3-dec4-402f-bce8-9e868367bb9d
Commits

Reviewing files that changed from the base of the PR and between 7df2ebf and ae3ce6c.

Files selected for processing: 19

  • quest/m0/nightly/README.md
  • quest/m1/route-inductive.md
  • quest/m2/cluster-routes/route-exhaustive-sim.md
  • quest/m2/cluster-routing/README.md
  • quest/m2/cluster-routing/federation.md
  • quest/m2/group-drops.md
  • quest/m3/buffered-ascending.md
  • quest/m3/cache-routing/README.md
  • quest/m3/export-order.md
  • quest/m3/quic-path-return.md
  • quest/m3/receive-windows-evicted.md
  • quest/m3/route-trace-validation.md
  • quest/m3/routing-guide.md
  • quest/m4/bun-lint-release.md
  • quest/m4/formatter-defaults.md
  • quest/m4/quic-deadline.md
  • quest/m4/reconnect-trace-validation.md
  • quest/m4/resume-trace-validation.md
  • quest/m4/subscribe-trace-validation.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
Contributor

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes: the full diff, which only touches quest/ planning docs.

  • Five m1 quests moved to m3: buffered-ascending, export-order, receive-windows-evicted, route-trace-validation and routing-guide. Each Plan records the move and the rejected alternatives. The earlier m1 placement is marked superseded where it was a decision.
  • bun-lint-release moved from m5 to m4: it now sits beside formatter-defaults, which requires it.
  • Inbound links rewritten: the 11 other files that link to a moved quest now use the new path, including the Required entries in the m4 trace-validation quests.

I found no stale references to the old paths in the diff.

Pullfrog  | View workflow run | Using claude-sonnet-5-5 | 𝕏

@kixelated

Copy link
Copy Markdown
Collaborator Author

Superseded: these moves fold into #8637, the pre-cut audit, which reshuffles the same quests (and adds more inverted links to fix). The maintainer picked folding them there.

(Written by Claude Opus 5.5)

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