Skip to content

quest: move tla-native-recheck to m0 - #8649

Merged
kixelated merged 1 commit into
mainfrom
quest/tla-native-recheck-m0
Nov 13, 2026
Merged

kixelated merged 1 commit into
mainfrom
quest/tla-native-recheck-m0

Conversation

@kixelated

@kixelated kixelated commented Nov 13, 2026 •

Copy link
Copy Markdown
Collaborator

Moves quest/m2/tla-native-recheck.md to m0 so a quest runner picks it up next, and records the decision in its Plan.

TLC is too slow for local checks. An exhaustive pass of route/Loop.cfg on #8622 ran 76 minutes on four cores and wrote 24 GB of state, and its queue was still growing when it was killed. #8628 removed SumOver, which this quest was waiting on, so it is ready to start.

quest check passes.

Review

Skipped on head 73912cc: a quest mv from m2 to m0 plus a paragraph recording the maintainer's decision; quest check passes.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5)

TLC is too slow for local checks; a faster native checker is now an
immediate priority.

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

@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 single commit moving the quest file.

  • Quest moved to m0: quest/m2/tla-native-recheck.md becomes quest/m0/tla-native-recheck.md, and no references to the old path remain.
  • Plan note: records the decision and the route/Loop.cfg timing evidence.

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

@kixelated
kixelated added this pull request to the merge queue Nov 13, 2026
@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The Plan records a decision dated 2026-11-13 to retain m0 and reject m2. It includes the reported 76-minute TLC run on four cores, the 24 GB state size, and the queue status before termination.

Merge Risk: Minimal · up to 73912

This is a documentation-only change that reprioritizes a quest and records the reasoning. It has no runtime impact and is ready to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title "quest: move tla-native-recheck to m0" matches the main change. The PR moves quest/m2/tla-native-recheck.md to m0 and records the decision in its Plan. The title is specific, concise, and na…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description states the PR moves quest/m2/tla-native-recheck.md to m0 and records the maintainer's decision in its Plan. It explains the reason: TLC is too slow, a 76-minute run of route/Loop.cfg, …
Full details: Title check

Explanation

The title "quest: move tla-native-recheck to m0" matches the main change. The PR moves quest/m2/tla-native-recheck.md to m0 and records the decision in its Plan. The title is specific, concise, and names the quest and the target stage.

Full details: Description check

Explanation

The description states the PR moves quest/m2/tla-native-recheck.md to m0 and records the maintainer's decision in its Plan. It explains the reason: TLC is too slow, a 76-minute run of route/Loop.cfg, and that #8628 removed SumOver, which the quest was waiting on. This matches the raw summary and the PR objectives, so the description is related to the changeset.

Finishing Touches

Simplify code

  • Commit to this branch
  • Create a new PR
  • 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

Recent review info

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: ce763d08-0afa-4025-84a5-9e6b625738d6
Commits

Reviewing files that changed from the base of the PR and between c748c0d and 73912cc.

Files selected for processing: 1

  • quest/m0/tla-native-recheck.md

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

Merged via the queue into main with commit 8206d7f Nov 13, 2026
8 checks passed
@kixelated
kixelated deleted the quest/tla-native-recheck-m0 branch November 13, 2026 17:09
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