Skip to content

fix(issue-auto-implement): iterate on PR when comment on PR; no .pr_* in commits; PR meta via outputs - #249

Merged
leggetter merged 1 commit into
mainfrom
feat/issue-auto-implement-setup
Mar 16, 2026
Merged

leggetter merged 1 commit into
mainfrom
feat/issue-auto-implement-setup

Conversation

@leggetter

Copy link
Copy Markdown
Collaborator

When you comment on the auto-implement PR (not the issue):

  • Resolve the real issue number from the PR (head ref or body) and run implement on that branch so the same PR is updated instead of creating a new branch/PR.

Workflow meta files (.pr_title, .pr_body):

  • Unstage before commit and add to the action's .gitignore (lives in the action dir; no consumer .gitignore changes).
  • Pass PR title/body to the Create PR step via step outputs instead of re-reading files.

Prompt: Clarify that .commit_msg/.pr_title/.pr_body are workflow-only and must not be committed.

Made with Cursor

… committing .pr_*; pass PR meta via outputs

- Treat issue_comment on a PR as iteration: resolve issue number from PR head/body, skip redirect, use existing branch
- Unstage .pr_title and .pr_body before commit; add to action .gitignore
- Pass PR title/body to Create PR step via step outputs instead of re-reading files
- Prompt: clarify that .commit_msg/.pr_title/.pr_body are workflow-only and must not be committed

Made-with: Cursor
@leggetter leggetter closed this Mar 16, 2026
@leggetter leggetter reopened this Mar 16, 2026
@leggetter
leggetter merged commit b403cd2 into main Mar 16, 2026
17 of 18 checks passed
@leggetter
leggetter deleted the feat/issue-auto-implement-setup branch March 16, 2026 22:50
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