From d988cf72c90235905f6b02a6e9904e4c1b8d8473 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 8 Oct 2026 13:45:44 +0000 Subject: [PATCH] argos-dev: keep PR descriptions about the change, not a file list Co-Authored-By: Claude Opus 5.5 --- .../argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md | 2 +- .../workspaces/argos-dev/.claude/skills/update-pr/SKILL.md | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/software/application-software/argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md b/software/application-software/argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md index 2387ada..5570774 100644 --- a/software/application-software/argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md +++ b/software/application-software/argos/workspaces/argos-dev/.claude/skills/open-pr/SKILL.md @@ -4,4 +4,4 @@ description: Run pre-PR checks, push the branch, and open a draft pull request --- From the branch's worktree, run `bash /.claude/skills/open-pr/scripts/pr-checks.sh` (clean tree, commit format, frontend lint if changed, conflicts with `origin/develop`); stop and fix on any FAIL. Then push and run `gh pr create --draft --base develop --head --title "" --body-file /tmp/-pr-body.md --assignee @me --label "by: " --label ...` (written-by plus area labels: `frontend`, `backend`, `devops`), and report the URL. -**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words, then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures//`. Write it to `/tmp/-pr-body.md`. +**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words. Describe the change, not the file list: don't enumerate components, classes or files the diff already shows. Name a specific one only when its behaviour changes in a way a reviewer must know. Then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures//`. Write it to `/tmp/-pr-body.md`. diff --git a/software/application-software/argos/workspaces/argos-dev/.claude/skills/update-pr/SKILL.md b/software/application-software/argos/workspaces/argos-dev/.claude/skills/update-pr/SKILL.md index 78ee642..97b7242 100644 --- a/software/application-software/argos/workspaces/argos-dev/.claude/skills/update-pr/SKILL.md +++ b/software/application-software/argos/workspaces/argos-dev/.claude/skills/update-pr/SKILL.md @@ -4,4 +4,4 @@ description: Update the current branch's PR description to reflect the latest ch --- Rewrite the current branch's PR body (`gh pr view`) to match `git diff develop...HEAD`. Keep human-written text that's still accurate, `user-attachments` screenshots, and `Closes`/`Fixes` refs. Apply it with `gh pr edit --body-file`, and add any missing written-by or area labels with `--add-label`. -**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words, then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures//`. Write it to `/tmp/-pr-body.md`. +**PR body:** fill `.github/pull_request_template.md` from the diff. Follow each section's comment: Changes in about 50 words. Describe the change, not the file list: don't enumerate components, classes or files the diff already shows. Name a specific one only when its behaviour changes in a way a reviewer must know. Then delete the comments and any optional or non-applicable sections. Check off the Checklist and end with `Closes #{ticket}`. For UI changes put `_screenshot pending_` and remind the user to drag-drop screenshots from `pictures//`. Write it to `/tmp/-pr-body.md`.