-
Notifications
You must be signed in to change notification settings - Fork 1
Extend implement skill to address unresolved PR review comments #86
Description
Description
Update the implement skill so that the agent fetches unresolved PR review comments in addition to the PR description. The completion check (step 5) should consider both the implementation plan checkboxes and any outstanding review comments — the plan is not complete until all checkboxes are checked and all review comments are resolved.
Motivation
Currently the implement skill only reads the PR description's implementation plan checkboxes to determine whether work remains. When a reviewer leaves inline comments requesting changes, the skill has no awareness of them. Running implement on a PR with all checkboxes checked but unresolved review comments incorrectly reports "implementation plan appears complete" and stops, forcing the user to work around the skill or invoke it with ad-hoc arguments.
Fetching review comments and treating them as additional work items closes this gap and makes the skill useful for the full review-implement-review cycle.
Expected outcome
When implement is invoked on a PR whose implementation plan checkboxes are all checked but has unresolved review comments, the agent fetches those comments, includes them in the planning phase as actionable work items, and proceeds to address them. The "implementation plan appears complete" early exit only fires when both the checkboxes are checked and no unresolved comments remain.