Skip to content

Surface guide factory review on the issue - #9

Merged
walker-tx merged 2 commits into
mainfrom
fix/guide-factory-review-comments
Jul 23, 2026
Merged

Surface guide factory review on the issue#9
walker-tx merged 2 commits into
mainfrom
fix/guide-factory-review-comments

Conversation

@walker-tx

@walker-tx walker-tx commented Jul 23, 2026

Copy link
Copy Markdown
Member

Summary

  • After draft-guide, post a Pipeline review issue comment with numbered decisions (not fidelity/dossier jargon): what’s wrong, what would unblock it, optional quote of the guide section, and reply templates (Decision N: verified|drop|hedge).
  • Distill folds the issue comment thread into notes on retry.
  • Unconverged runs still open a draft PR (re-land of the fix lost in the Ensure guide factory labels exist #7 squash).
  • When setup.md exists, the comment embeds the relevant section under a <details> block.

Test plan

  • Merge; re-add guide:draft on an issue and confirm the Pipeline review comment is skimmable without opening the guide
  • Reply with Decision 1: drop this branch and confirm distill picks it up on retry

walker-tx and others added 2 commits July 23, 2026 14:54
Post a Pipeline review comment (and PR body section) with unresolved blockers and open questions; fold issue comments into distill on retry so clarifications live on the ticket. Also re-land opening draft PRs when draft-guide exits 2.

Co-authored-by: Cursor <cursoragent@cursor.com>
Each blocker is a numbered decision with plain-English why, what would unblock it, optional setup.md quote, and reply templates (verified / drop / hedge).

Co-authored-by: Cursor <cursoragent@cursor.com>
@walker-tx
walker-tx merged commit f6deaa0 into main Jul 23, 2026
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