Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions assets/skills/complete.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,11 @@ Ask synchronously with one question per PR, never grouping PRs into one question

Include a brief summary of each PR.
Act on a PR only after the user picks its action in this session, including PRs found on a later refresh.
A blanket answer, such as "accept all" or "use your recommendations", never picks `/quest-merge`; ask again for each PR it would merge.
A skip or hold stands until the user lifts it: do not ask about that PR again on a refresh, and state the hold in every sub-agent brief for it.
GitHub auto-merge is that pick: enabling it takes write access, and GitHub disables it when anyone without write access pushes.
Do not ask about a PR with auto-merge enabled; watch it like an enqueued PR, and dispatch `/quest-merge` only when a conflict, failing check, or review finding stalls it.
A pick covers the PR until it lands or needs the user: when its push draws new review findings, send the agent back without asking, and apply the merge skill's design checkpoint across those rounds.
A pick covers the PR until it lands or needs the user: when its push draws new review findings, send the agent back without asking; the merge skill's design checkpoint counts rounds from the PR, so it fires across those re-dispatches.
Review comments are input, not approval, even a merge verdict on the maintainer's account.
Consider ordering, queuing any merges if they would result in conflicts.

Expand All @@ -41,7 +43,7 @@ dispatch `/quest-merge` again. Any other reason, such as a person removing it, i
user's call, so ask about the PR again.

Keep going until all PRs have been decided, then wait for all spawned sub-agents and enqueued PRs to finish.
Before finishing, refresh the open PR list and process any new PRs in scope.
Refresh the open PR list whenever a sub-agent reports or a merge wait wakes, and once more before finishing; ask about each new PR in scope the same way.

As each sub-agent reports, include a summary and interactively prompt the user for any outstanding decisions, offering `/quest-plan` for a PR's suggested follow-ups once it merges.
Dispatch already selected actions before asking the next question; background work can continue while the foreground waits.
Expand Down
4 changes: 4 additions & 0 deletions assets/skills/iterate.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,10 @@ If you cannot push to the PR's branch, stop and report.
Read the PR, its linked issues and quest, the reviews and comments posted on it, and the surrounding code.
`quest status <pr>` reports its checks, reviews by head, and unresolved threads in one call.
Judge the approach before touching anything: would you take a different one?
Checkpoint the design across review rounds, as the merge skill does.
A review round is one pushed head that drew reviews, counted from the PR's full history (`gh pr view <pr> --json reviews,comments`) since its last comment that starts with `Design change:`.
On every fourth round, report your judgment of the approach and what remains before fixing more findings, recommending to continue, rethink the approach, or drop the PR; that choice is the user's.
Record a design change you make, including a rethink the user picks, in a PR comment that starts with `Design change:`.

Scope the changes this PR needs with the plan skill's interview, starting from the user's feedback, your judgment of the approach, and any review findings that need a decision.
Skip its quest and PR steps: record the settled decisions in this PR's description as the paper trail, then implement them immediately.
Expand Down
5 changes: 4 additions & 1 deletion assets/skills/merge.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,11 @@ Read the PR's head, checks, merge state, reviews by head, and unresolved threads
Fix any mechanical issues with the PR, such as merge conflicts and failing CI checks.
Address any review findings, fixing those you agree with and leaving a comment on any you disagree with.
Automated reviewers often find something new on every push, so keep fixing their findings without asking, but checkpoint the design.
Every fourth review round since the PR's last design change (one that reshapes the approach; fixing a finding is not one), stop before the next fix and reconsider the design: is something fundamentally wrong with the approach?
A review round is one pushed head that drew reviews; count rounds from the PR, not your session, since its last design change (one that reshapes the approach; fixing a finding is not one).
`quest status` shows only each reviewer's latest review, so read the full history with `gh pr view <pr> --json reviews,comments`.
Every fourth round, stop before the next fix and reconsider the design: is something fundamentally wrong with the approach?
Report that judgment and what remains unresolved, recommending to continue, rethink the approach, merge as is, or drop the PR, and let the user decide.
Record each design change, including a rethink the user picks, in a PR comment that starts with `Design change:`; the next agent counts rounds from it.
Never request a review, including by toggling a ready PR to draft and back.

Abort the merge if there is a significant decision to be made.
Expand Down
1 change: 0 additions & 1 deletion quest/m0/helpers/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ commands from Sep-Oct. Session counts in each child come from that window.

## Required

- [Complete loop](/quest/m0/helpers/complete-loop.md) - `/quest-complete` re-scans PRs, keeps picks and holds, and counts rounds per PR
- [Status holds](/quest/m0/helpers/status-holds.md) - `quest status` shows a PR's `quest:*` hold
- [Headless agents](/quest/m0/helpers/headless-agents.md) - `/quest-spawn api` launches, resumes, and stops billed agents through one command auto mode allows
- [Resolve](/quest/m0/helpers/resolve.md) - a local merge resolves concurrent link-list edits in quest files, set up by `quest init`
Expand Down
26 changes: 0 additions & 26 deletions quest/m0/helpers/complete-loop.md

This file was deleted.

2 changes: 0 additions & 2 deletions quest/m0/helpers/status-holds.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,3 @@ added the holds:
GitHub client, and report a draft its starter still holds the same way.
- Update the `complete` skill to read holds from `quest status`, and the
README's status example.
- Ranked after the [complete loop](/quest/m0/helpers/complete-loop.md), which
also edits `complete`.
Loading