From 6d6ac89ab3773879406355f8e20361e7286c77ec Mon Sep 17 00:00:00 2001 From: Luke Curley Date: Sat, 31 Oct 2026 23:29:13 -0700 Subject: [PATCH 1/2] chore(quest): claim quest/m0/helpers/follow-up-plan.md Quest-Claim: 30da3116cb3d0191 From 56accd4591d29d78e0b29bc57fb1932051d03c3a Mon Sep 17 00:00:00 2001 From: Luke Curley Date: Sat, 31 Oct 2026 23:30:12 -0700 Subject: [PATCH 2/2] docs(guide): one rule for planning a delegated agent's follow-ups The guide's follow-up rule now says the interview lets the user drop a follow-up and that the caller plans a background agent's follow-ups the same way, so complete.md no longer repeats it. #292 already made merge, complete, and spawn plan follow-ups without offering; this finishes the quest. Co-Authored-By: Claude Opus 5.5 --- assets/AGENTS.md | 4 ++-- assets/skills/complete.md | 2 +- quest/m0/helpers/README.md | 1 - quest/m0/helpers/follow-up-plan.md | 26 -------------------------- 4 files changed, 3 insertions(+), 30 deletions(-) delete mode 100644 quest/m0/helpers/follow-up-plan.md diff --git a/assets/AGENTS.md b/assets/AGENTS.md index bcae98d6..53a39d56 100644 --- a/assets/AGENTS.md +++ b/assets/AGENTS.md @@ -95,8 +95,8 @@ Search the tree and git history first before making a new quest. Split independently completable work into separate quests. A suggested follow-up becomes a quest only through `/quest-plan`, never as an agent-session task or a GitHub issue. -Plan recommended follow-ups without offering first: run `/quest-plan` on each one in turn, one follow-up per run. -A background agent that cannot prompt reports them for its caller to plan. +Plan recommended follow-ups without offering first: run `/quest-plan` on each one in turn, one follow-up per run; its interview lets the user drop it. +A background agent that cannot prompt reports them, and its caller plans them the same way. Group related quests in an epic; its README holds the work no child owns. A release or pin bump that unblocks work is its own quest. diff --git a/assets/skills/complete.md b/assets/skills/complete.md index c5d9801e..d6928944 100644 --- a/assets/skills/complete.md +++ b/assets/skills/complete.md @@ -44,7 +44,7 @@ Keep going until all PRs have been decided, then wait for all spawned sub-agents Refresh the open PR list whenever a sub-agent reports, 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. -Collect each queued PR's suggested follow-ups, and once every sub-agent has finished, plan them as the guide describes; each interview lets the user drop its follow-up. +Collect each queued PR's suggested follow-ups, and once every sub-agent has finished, plan them as the guide describes. Dispatch already selected actions before asking the next question; background work can continue while the foreground waits. Once every sub-agent has finished, remove local worktrees and branches whose tip is the head of a merged PR; squash merges leave that tip off the base branch. diff --git a/quest/m0/helpers/README.md b/quest/m0/helpers/README.md index dbf75eed..c23ca0af 100644 --- a/quest/m0/helpers/README.md +++ b/quest/m0/helpers/README.md @@ -73,7 +73,6 @@ commands from Sep-Oct. Session counts in each child come from that window. - [quest merge --land gaps](/quest/m0/helpers/merge-land.md) - squash on squash-only repositories, PR URLs outside a checkout, labels released after direct merges, and a warning when auto-merge outlives its head - [Visible merge-driver fallback](/quest/m0/helpers/merge-driver.md) - a merge without `quest` on PATH says so, and this repository's dev shell provides one - [Review rules](/quest/m0/helpers/review-rules.md) - a head whose diff matches the reviewed diff keeps its review -- [Plan delegated follow-ups](/quest/m0/helpers/follow-up-plan.md) - a caller plans a background merge's follow-ups without asking - [Drop review-rounds](/quest/m0/helpers/review-rounds.md) - `quest status` stops counting GitHub reviews now that `quest review` numbers rounds - [Headless agents](/quest/m0/helpers/headless-agents.md) - `/quest-spawn api` launches, resumes, and stops billed agents through one command auto mode allows - [Sibling PRs through a server-side merge](/quest/m0/helpers/queue-siblings.md) - children of one epic land back to back through a merge queue without conflicting on its link list diff --git a/quest/m0/helpers/follow-up-plan.md b/quest/m0/helpers/follow-up-plan.md deleted file mode 100644 index 0c60647b..00000000 --- a/quest/m0/helpers/follow-up-plan.md +++ /dev/null @@ -1,26 +0,0 @@ -# [XS] Plan a delegated merge's follow-ups - -## Goal - -A caller that receives a merged PR's suggested follow-ups from a sub-agent, -such as a background `/quest-merge`, runs `/quest-plan` on them without -asking first, and its interview lets the user drop any. One rule in the -guide says this, replacing the copies in `complete.md` and `spawn.md`. - -## Plan - -Decided 2026-10-28 (user, /quest-plan of the follow-ups from [moq-dev/quest#271](https://github.com/moq-dev/quest/pull/271)). - -- [moq-dev/quest#266](https://github.com/moq-dev/quest/pull/266) made a - foreground `/quest-merge`, `/quest-complete`, and `/quest-spawn` plan - follow-ups without asking. A single `/quest-merge` run as a background - agent only lists them in its report, and no rule told the caller to plan - them: #271's follow-ups waited for the user to ask. -- The rule belongs in the guide, beside "A suggested follow-up becomes a quest - only through `/quest-plan`", so it covers any skill that delegates a merge. - `complete.md` and `spawn.md` keep only what is specific to them, such as - waiting until every sub-agent has finished. -- `/quest-afk` still holds follow-ups for the user, since it runs no - interview. -- Also update the guide's "`/quest-merge` decides which to plan", which - predates #266.