Skip to content

chore: replace local pr skill with rstackjs/agent-skills pr-creator#1255

Merged
fi3ework merged 1 commit into
mainfrom
chore-use-rstack-pr-skill
May 15, 2026
Merged

chore: replace local pr skill with rstackjs/agent-skills pr-creator#1255
fi3ework merged 1 commit into
mainfrom
chore-use-rstack-pr-skill

Conversation

@fi3ework

Copy link
Copy Markdown
Member

Summary

Drop the locally-maintained pr skill in .agents/skills/pr/ and install the shared pr-creator skill from rstackjs/agent-skills via the skills CLI (skills-lock.json). The two skills overlapped heavily, so this aligns rstest with the rest of the Rstack ecosystem instead of maintaining a parallel copy.

  • Replace .agents/skills/pr/SKILL.md with .agents/skills/pr-creator/ pulled from rstackjs/agent-skills
  • Refresh create-draft-release-notes to the upstream version (adds a Security Notes section)
  • Update AGENTS.md skills table to reference pr-creator

Note: the upstream pr-creator is intentionally more generic — it does not hard-code rstest validation commands (pnpm lint/typecheck/build/test, cd e2e && pnpm test) or the strict 3-section Summary format that the previous local skill enforced. If we want those rstest-specific guardrails back, we can either add a thin local skill on top or upstream them into rstackjs/agent-skills.

Related Links

Drop the locally-maintained `pr` skill and install the shared
`pr-creator` skill from rstackjs/agent-skills via the skills CLI, so
agent skills stay aligned across the Rstack ecosystem instead of
diverging per repo. Also refresh `create-draft-release-notes` from
upstream (adds Security Notes section).
@fi3ework fi3ework requested a review from chenjiahan May 14, 2026 08:09
@github-actions

Copy link
Copy Markdown

Rsdoctor Bundle Diff Analysis

⚠️ Note: The latest commit (e2dbcabc8d) does not have baseline artifacts. Using commit fc59ef660d for baseline comparison instead. If this seems incorrect, please wait a few minutes and try rerunning the workflow.

Found 12 projects in monorepo, 0 projects with changes.

📊 Quick Summary
Project Total Size Change
adapter-rsbuild 3.6 KB 0
adapter-rslib 24.6 KB 0
adapter-rspack 7.7 KB 0
browser 2.0 MB 0
browser-react 3.7 KB 0
browser-ui 803.5 KB 0
coverage-istanbul 9.6 KB 0
core/browser 970.0 KB 0
core/loaders 869.0 B 0
core/main 1.6 MB 0
vscode/extension 26.9 MB 0
vscode/worker 14.4 KB 0

Generated by Rsdoctor GitHub Action

@fi3ework fi3ework merged commit 8bdb786 into main May 15, 2026
12 checks passed
@fi3ework fi3ework deleted the chore-use-rstack-pr-skill branch May 15, 2026 02:56
@9aoy 9aoy mentioned this pull request May 18, 2026
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