diff --git a/plugins/aidd-vcs/skills/02-pull-request/actions/01-collect.md b/plugins/aidd-vcs/skills/02-pull-request/actions/01-collect.md index ef5042d99..d7b0b6e0c 100644 --- a/plugins/aidd-vcs/skills/02-pull-request/actions/01-collect.md +++ b/plugins/aidd-vcs/skills/02-pull-request/actions/01-collect.md @@ -14,7 +14,7 @@ The VCS tool, the head and base branches, and the commits and changed files sinc 1. **Tool.** Use the VCS tool from project memory, else infer it from the remote URL. 2. **Base.** Use a provided base, else resolve it per the project's branch convention, else the repo's default branch. Surface the base and why. -3. **Gather.** Summarize the commits and changed files since the base. +3. **Gather.** Summarize the change from the diff against the resolved base. ## Test diff --git a/plugins/aidd-vcs/skills/02-pull-request/actions/02-draft.md b/plugins/aidd-vcs/skills/02-pull-request/actions/02-draft.md index be7170123..496dbd502 100644 --- a/plugins/aidd-vcs/skills/02-pull-request/actions/02-draft.md +++ b/plugins/aidd-vcs/skills/02-pull-request/actions/02-draft.md @@ -14,6 +14,10 @@ The proposed title, body, and base, approved by the user. 1. **Template.** Load the request template, the project's own when set, else the bundled [pull_request.md](../assets/pull_request.md). 2. **Write.** Draft a concise title and a body following the template from the change summary. Link every changed `**/qa/*.webm` under the template's testing or verification section. + - Give the fewest concrete verification steps needed for the changed behavior. + - State the expected result for each step. + - Omit generic CI summaries. + - Label scenarios that were not run. 3. **Confirm.** Show the title, body, and base, apply any overrides, and wait for approval. ## Test