Repository navigation
docs(skills): state the guide and skill rules agents kept needing - #187
Merged
Merged
Conversation
Quest-Claim: dc05236d477422f4
Pushing, sub-agent briefs, ready at hand-back, overlap search before planning, user-owned milestone moves, follow-ups through /quest-plan, whose review counts, and decisions reserved for the maintainer. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kixelated
changed the title
kixelated
marked this pull request as ready for review
October 22, 2026 05:45
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 51a0acf19d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Spawn and complete shared it verbatim; shared workflow instructions belong in the guide. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Collaborator
Author
|
Merge summary for a477407:
Auto-merge (squash) is enabled for a477407. (written by Claude Opus 5.5) |
kixelated
added a commit
that referenced
this pull request
Oct 22, 2026
Resolve the conflict with #187 by dropping both finished quests (Unattended work and Guide rules) from the helpers epic's Required list. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kixelated
added a commit
that referenced
this pull request
Oct 22, 2026
Resolve conflicts with the guide rules (#187) and /quest-afk (#184): drop the helpers epic's links to the now-finished pr-states, afk, and guide-rules quests, and rename the PR state commands' token printer so it no longer shadows claims' `held`. 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.
Finishes
quest/m0/helpers/guide-rules.md: the guide and skills now state the rules the moq maintainer kept typing as corrections, so agents stop needing them. Prose only; no CLI changes.Changes
quest startclaim push is exempt.spawnandcomplete; it lives once in the guide instead, per review, since both skills would carry it verbatim.start): a complete quest is marked ready even if the agent made choices without asking, listing them in the PR body. Only a decision left for the maintainer, or unfinished work, keeps it a draft with## Decisions needed. Drops thestartgate ("the user has settled every decision, including choices you made without asking") that disagreed with the guide.plan): before anything else, search the quest tree, git history for deleted quests, and open PRs touchingquest/, and show any overlap in the first round.audit): moving a quest between milestones or changing its rank is always the user's decision, never a factual correction./quest-plan, never as an agent-session task or a GitHub issue.iterate): a sub-agent that cannot prompt picks the recommendation only for decisions the repository's instructions don't reserve for the maintainer; reserved or truly consequential ones stay open in the PR.quest/m0/helpers/README.md.Choices made without asking
iterate.iteratekeeps its old "truly consequential" stop alongside the new reserved-decision rule, rather than dropping it.PROMPTING.md.Validation
just checkandjust testpass locally (Nix shell).quest check --base origin/mainpasses.Follow-ups
None. Expect merge conflicts with the parallel PR state commands and Unattended work quests, which also edit these skills and the guide.
🤖 Generated with Claude Code
(written by Claude Opus 5.5)