Repository navigation
fix(land): the fixer hands back only injections aimed at it - #328
Merged
Merged
Conversation
The fixer handed back moq-dev/moq#6634 because the PR's quest plan says "agents do not push tags", and moq-dev/moq#7426 for a new AGENTS.md rule. Guidance a PR adds for contributors or agents in general is ordinary content; a handback is for content that targets the fixer, its result, or its environment. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Warning Review limit reachedYou'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 2 minutes. View limit details
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. |
|
quest land removed this pull request from the queue: required checks had not passed on its head when the gate ran: check (from app 15368); land it again once they pass. |
kixelated
enabled auto-merge (squash)
October 31, 2026 19:31
This was referenced Oct 31, 2026
moq-bot Bot
pushed a commit
that referenced
this pull request
Oct 31, 2026
Plans what moq.pro's and rust2's cutover to the host gate and the fixer repairs (#320, #325, #327, #328) left: collateral ejections keep their fix attempt, a restarted daemon keeps its check, GitHub retries, current conflict reports, land checks at gate priority, a fast kix runner test, and deleting the legacy hold labels. The gate host records what shipped. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The land fixer now hands back a PR for prompt injection only when the content targets the fixer itself.
Change
After #325, the fixer still handed back PRs whose added docs tell contributors or agents how to work:
AGENTS.mdrule.In repositories run by agents, that's ordinary content.
assets/land-fix.md: guidance a PR adds or changes for contributors or coding agents in general is project content, merged like any other file.Checks
just checkpasses.just testpasses (726 CLI and 426 Worker tests), run withCLAUDE_CONFIG_DIRunset.Review
quest review --base origin/main --deep(Codex), round 1 on head 1723aff: no findings.🤖 Generated with Claude Code
(Written by Claude Opus 5.5)