Skip to content

Commit 263aac9

Browse files
committed
chore: update pull request template and skills
1 parent c0ab876 commit 263aac9

4 files changed

Lines changed: 23 additions & 21 deletions

File tree

.agents/skills/create-draft-release-notes/SKILL.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ metadata:
99

1010
## Overview
1111

12-
Create a GitHub draft release when possible, organize the generated notes by conventional commit type, and save the organized body back to the draft. If `gh` cannot create or edit the draft, return the organized Markdown in the conversation with manual creation steps. Preserve each release note item exactly; only split accidentally joined bullets, move bullets into sections, and adjust headings. Add a top `## Highlights` section only when the user explicitly asks for highlights.
12+
Create a GitHub draft release when possible, organize the generated notes by conventional commit type, and save the organized body back to the draft. If `gh` cannot create or edit the draft, return the organized Markdown in the conversation with manual creation steps. Preserve each release note item exactly except stale release PRs; otherwise only split accidentally joined bullets, move bullets into sections, and adjust headings. Add a top `## Highlights` section only when the user explicitly asks for highlights.
1313

1414
## Security Notes
1515

@@ -138,7 +138,7 @@ Use this when the user provides generated release note Markdown and only wants i
138138
node .agents/skills/create-draft-release-notes/scripts/create-draft-release-notes.mjs release-notes.md
139139
```
140140

141-
Omit the file path to read from stdin. Review that every original item still appears once and non-item sections remain.
141+
Omit the file path to read from stdin. Review that every retained item appears once and non-item sections remain.
142142

143143
## Optional Highlights Workflow
144144

@@ -154,7 +154,7 @@ Write highlights before `## What's Changed`:
154154
- Use one `###` heading per highlight.
155155
- Keep each highlight to a short paragraph plus an optional fenced code example.
156156
- Include examples only when the API/configuration is clear.
157-
- Do not rewrite or reorder changelog items below `## What's Changed`.
157+
- Do not rewrite or reorder retained changelog items below `## What's Changed`.
158158
- Replace an existing top `## Highlights` block instead of adding another one.
159159

160160
Example shape:
@@ -203,6 +203,8 @@ Keep each category in generated top-to-bottom order.
203203

204204
## Preservation Rules
205205

206+
- Remove bullets that clearly identify a release PR for `$previous_tag` or an older published version, such as `release: v1.0.0` in `v1.0.1` notes; keep current-version and ambiguous items.
207+
- If anything is removed, list the original bullets verbatim for the user outside the release note body.
206208
- Do not rewrite bullet text, authors, URLs, PR numbers, package names, scopes, punctuation, or casing.
207209
- Do not drop comments, `**Full Changelog**`, or other non-item sections.
208210
- Do not add commentary to the release note itself, except for a requested `## Highlights` section.

.agents/skills/pr-creator/SKILL.md

Lines changed: 9 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
---
22
name: pr-creator
3-
description: Use when asked to create a pull request for this repository. It helps the PR follow the repository's branch safety rules, title convention, pull request template, and concise English writing style.
3+
description: Create a pull request using repository branch rules, title conventions, templates, and concise English descriptions.
44
metadata:
55
internal: true
66
---
@@ -17,7 +17,7 @@ metadata:
1717
Do not revert unrelated user changes.
1818
Before creating the PR, ensure the intended changes are committed and never commit directly on the default branch.
1919

20-
3. If `.github/PULL_REQUEST_TEMPLATE.md` exists, read it and follow its structure.
20+
3. Read the repository's PR template when available and follow its current headings and guidance.
2121

2222
4. Draft the PR title in the repository's standard format. If the repository uses Conventional Commits, common patterns include:
2323
- `feat(core): add ...`
@@ -29,20 +29,13 @@ metadata:
2929
- `release: v1.2.0`
3030

3131
5. Write the PR body in concise, clear English.
32-
- In `Summary`, explain the change context first: the user-facing problem, maintenance goal, or compatibility constraint that makes the change necessary.
33-
- Prioritize high-signal information: public API changes, behavior changes, breaking changes, migration notes, and important compatibility implications.
34-
- Then describe the main implementation change only as much as needed to understand the review.
35-
- Keep the PR body concise and review-oriented: use 1-4 short standalone sentences for typical changes, covering why it matters, what changed, and any reviewer-important impact.
36-
- Avoid low-signal sections such as `Test plan` or `Validation`, routine verification commands, generated file lists, or obvious implementation details unless the repository template explicitly requires them or the change has unusual validation risk.
37-
- Good background examples:
38-
- `This PR adds support for custom logger injection so CLI output can be isolated per instance.`
39-
- `This PR fixes incorrect padding in URL labels to keep terminal output aligned across different label lengths.`
40-
- `This PR updates the English docs to clarify how the extraction option works and when to enable it.`
41-
42-
6. Fill `Related Links` with issue links, design docs, related PRs, or discussion pages.
43-
If the PR upgrades an npm dependency, add a link to the upgraded version's release notes or tag page when available.
44-
Example: `https://github.com/web-infra-dev/rspack/releases/tag/v1.0.0`
45-
If there is no relevant link, omit the entire `Related Links` section from the PR body.
32+
- Explain the problem or motivation and why it matters, then describe the approach and resulting behavior.
33+
- Include API, compatibility, or migration details when they help reviewers assess the change.
34+
- Keep typical descriptions to a few short sentences. Focus on the key changes rather than a file-by-file summary.
35+
- Mention tests, documentation, and validation only when required by the template, central to the change, or relevant to review risk.
36+
37+
6. Include relevant issue, discussion, or design links alongside the context they support, following the template's guidance.
38+
For dependency upgrades, link to the target version's release notes or tag when available.
4639

4740
7. Push the branch only after re-checking the branch name. Never push the default branch directly.
4841

.github/PULL_REQUEST_TEMPLATE.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
## Motivation
2+
3+
<!-- Explain the problem or motivation and why it matters. Include relevant context and links to issues or discussions. -->
4+
5+
## Changes
6+
7+
<!-- Describe how this PR addresses the motivation and what behavior changes. Focus on the key approach rather than a file-by-file summary. -->

skills-lock.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@
55
"source": "rstackjs/agent-skills",
66
"sourceType": "github",
77
"skillPath": ".agents/skills/create-draft-release-notes/SKILL.md",
8-
"computedHash": "9fa7807d607c7fb7e02795ddee2560cc190d4af791320537a66ca8b2e429d858"
8+
"computedHash": "1c07601e93b35361c92cd6f067ddba3c710c27e98fd42f7b11e74b446590d1a9"
99
},
1010
"pr-creator": {
1111
"source": "rstackjs/agent-skills",
1212
"sourceType": "github",
1313
"skillPath": ".agents/skills/pr-creator/SKILL.md",
14-
"computedHash": "c49fcbba72894d08a34d054a00720610d19a2e87e25942aee05da09e87b268df"
14+
"computedHash": "ed4ca0b83b380c9a4a350047ef5f2616c0446b2830d59e3a06839cb827fd39b4"
1515
}
1616
}
1717
}

0 commit comments

Comments
 (0)