feat(finalize-pr): add Phase 4 to auto-update PR metadata#65
feat(finalize-pr): add Phase 4 to auto-update PR metadata#65JacobPEvans merged 2 commits intomainfrom
Conversation
…rting ready Adds a new Phase 4 between pre-handoff verification and the ready report. Phase 4 delegates to a haiku subagent (token-efficient) to: - Update PR title (conventional commit format, <70 chars) and description (Summary, Changes, Test Plan) using compact git log/diff --stat output - Link related issues/PRs via keyword search with Closes #X / Related: #X Sub-steps 4.1 and 4.2 can run in parallel within the agent. The former Phase 4 (Report Ready Status) becomes Phase 5. /squash-merge-pr is now marked as optional since metadata is already updated by finalize-pr. (claude)
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
There was a problem hiding this comment.
Important
Looks good to me! 👍
Reviewed everything up to b6491de in 10 seconds. Click for details.
- Reviewed
108lines of code in1files - Skipped
0files when reviewing. - Skipped posting
0draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
Workflow ID: wflow_92K27giwQu98gL3e
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
There was a problem hiding this comment.
Pull request overview
This PR updates the finalize-pr skill documentation to introduce a new Phase 4 that automatically updates PR metadata (title/body and linked issues/PRs) before reporting readiness, shifting the prior “report ready” step to Phase 5.
Changes:
- Added Phase 4: Update PR Metadata (including parallelizable sub-steps 4.1/4.2 and a final apply step).
- Renumbered the former Phase 4 (ready report) to Phase 5 and updated the workflow diagram accordingly.
- Updated the “Critical Rules” to reflect automatic metadata updates and marked
/squash-merge-pras optional.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Greptile SummaryAdds Phase 4 to the
Confidence Score: 5/5
Important Files Changed
Last reviewed commit: b6491de |
… update - Number 'Apply Updates' as 4.3 to match Phase 2's sub-step convention - Add git fetch origin main before log/diff to avoid stale ref output - Switch gh pr edit to --body-file for multiline-safe markdown handling (claude)
Summary
git log --oneline+git diff --statinstead of full diff/squash-merge-pris now marked optional sincefinalize-prhandles metadata itselfChanges
github-workflows/skills/finalize-pr/SKILL.md: Added Phase 4, renumbered Phase 5, updated critical rule 9, updated workflow diagramTest Plan
/squash-merge-prstill referenced as optional in Phase 5🤖 Generated with Claude Code
Important
Adds Phase 4 to
finalize-prfor auto-updating PR metadata, making/squash-merge-proptional, and updatesSKILL.md.finalize-prto auto-update PR metadata (title, description, linked issues) using a haiku subagent./squash-merge-pris optional as metadata is handled in Phase 4.SKILL.mdto include new Phase 4, renumber Phase 5, and update critical rule 9.This description was created by
for b6491de. You can customize this summary. It will automatically update as commits are pushed.