Skip to content

docs(skills): plan follow-ups only through /quest-merge - #193

Merged
kixelated merged 2 commits into
mainfrom
claude/follow-ups-after-merge
Oct 22, 2026
Merged

kixelated merged 2 commits into
mainfrom
claude/follow-ups-after-merge

Conversation

@kixelated

@kixelated kixelated commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Problem

/quest-start asked about suggested follow-ups before running /quest-ready. A finished quest with no open decisions still stopped on that question, so its PR stayed a draft until the user answered. /quest-ready and /quest-iterate also offered follow-ups, even though they're decided once the PR merges.

Changes

  • start: as soon as the quest is complete and the agent sees no decision left for the maintainer, run /quest-ready without asking. Ask about open decisions only when there are some. The follow-up question is gone.
  • Guide: follow-ups stay suggested in the PR body, and /quest-merge decides which ones to plan.
  • ready, iterate: no longer offer /quest-plan for follow-ups.
  • merge: suggested follow-ups no longer abort the merge; /quest-plan is offered for them once the PR merges, and a background merge agent lists them in its report.
  • spawn, complete: offer /quest-plan for a PR's suggested follow-ups once it merges. A start agent's report now lists only its open decisions.
  • The ## Unattended sections of spawn and complete are untouched, because kixelated/quest#184 deletes them.

Validation

  • just check: passed
  • just test: passed

🤖 Generated with Claude Code

(written by Claude Opus 5.5)

/quest-start offered follow-ups before /quest-ready, so a quest with no
open decisions still stopped on a question. It now runs /quest-ready
right away when no decision is left, and /quest-merge alone decides
which suggested follow-ups to plan. ready and iterate stop offering
them; spawn and complete offer them only for PRs a merge sub-agent
landed.

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

chatgpt-codex-connector Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-22T06:59:13.897759Z bb3dde3 New commits
🔒 Security Review ✅ Completed 2026-10-22T07:00:22.110156Z bb3dde3 New commits
ℹ️ 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" or "@codex security review".

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 chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 44a9bfb1a5

ℹ️ 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".

Comment thread assets/skills/complete.md Resolved
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated

Copy link
Copy Markdown
Collaborator Author

Merge summary:

  • Codex found that merge.md still made the follow-up decision abort the merge, which left the new "offer /quest-plan once merged" gates in spawn and complete unreachable. Fixed in bb3dde3: suggested follow-ups no longer hold the merge; /quest-plan is offered for them once the PR merges, and a background merge agent lists them in its report. Codex then reacted with no findings on bb3dde3.
  • just check, just test, and CI pass. quest check --base origin/main, built from the base, finds nothing to delete; this PR touches no quest file.
  • kixelated/quest#184 edits neighboring lines of spawn.md, complete.md, and assets/AGENTS.md but none of the same ones.

Enabling auto-merge on bb3dde3.

(written by Claude Opus 5.5)

@kixelated
kixelated merged commit e220493 into main Oct 22, 2026
5 checks passed
@kixelated
kixelated deleted the claude/follow-ups-after-merge branch October 22, 2026 07:04
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