[kixelated]

Finishes inline-merge from the host-land epic. Picking a PR to merge no longer starts a sub-agent that fixes the PR, waits on CI, and lands it.

Behavior

• /quest-merge runs inline. It takes the hold, reads quest status once, and queues the head with quest merge --land when all of these hold:

  • a counted ## Review record names the head, or the guide's exemptions cover the changes since it;
  • no posted finding still needs a change, and no decision is open;
  • its required checks passed, other than the gate's own quest land check;
  • it has no conflict with its base;
  • quest check --base passes on the fetched head.

  Otherwise it releases the hold and recommends /quest-iterate, saying what's missing. It never fixes, pushes, or waits. While checks are only pending, it reports that the pick stands.

• /quest-complete: runs /quest-merge inline, and runs only /quest-review and /quest-iterate in sub-agents.

  • It recommends /quest-iterate for any PR whose head has no counted review.
  • For a picked PR, it dispatches /quest-iterate without asking, then merges inline when it reports. That includes after a merge_conflict or failed_checks ejection.
  • After checks_pending, it merges again on a later refresh.

• /quest-spawn: offers the same options and runs merges inline the same way.

• Follow-ups (user, 2026-10-30): /quest-merge plans a PR's suggested follow-ups as soon as it's queued, as if the PR lands, without waiting for the merge. /quest-complete and /quest-spawn batch them at the end. The guide now says to plan recommended follow-ups without offering first, one /quest-plan run per follow-up.

• The README skill table and getting-started describe the new /quest-merge.

Choices made without asking

• quest check --base stays in /quest-merge. The plan said the gate's check covers it, but CI runs quest check without --base, so the gate would miss a PR that leaves its quest stale. On a finding, merge now recommends /quest-iterate instead of fixing the PR itself.
• Pending checks don't queue. Today's gate ejects them as checks_pending, so /quest-merge reports that the pick stands, and /quest-complete retries on a later refresh. The host daemon can drop this requirement, since it runs the check itself.

Overlaps

#288 and #276 edit the same skills. Whichever lands second reconciles.

Checks

just check passed. just test passed: 626 + 426 tests. It was run with CLAUDE_CONFIG_DIR/QUEST_WORKTREE_HOME unset and GIT_CONFIG_GLOBAL=/dev/null. On this host, test/review.test.ts otherwise fails: its fixture commits under my global SSH commit signing without ssh-keygen on its PATH. That's an environment gap, not caused by this change.

Review

• Reviewer: Codex gpt-6.1-sol, medium, quest review --base origin/main, round 3 of session 01a18d98
• Head: 49f47d6
• Findings: none in round 3, which covers the follow-up planning change.
  • Round 1 found one (high, merge.md:21): requiring every required check would block on the gate's unreported quest land check. Fixed.
  • An earlier session on the pre-claim branch found two issues: the hold wasn't released on handoffs, and there was no fetch before quest check. Both fixed.

Suggested follow-ups

• test/review.test.ts (and agent.test.ts) should isolate themselves from the caller's global git and Claude config.

🤖 Generated with Claude Code

(written by Claude Opus 5.5)

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 1b03da8d-1ae9-43e4-9a6a-9badfb2c1b44

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}