Skip to content

Add /pr agent skill and refresh PR template - #779

Merged
ktehranchi merged 1 commit into
developfrom
pr-skill
Aug 24, 2026
Merged

Add /pr agent skill and refresh PR template#779
ktehranchi merged 1 commit into
developfrom
pr-skill

Conversation

@ktehranchi

Copy link
Copy Markdown
Collaborator

Changes proposed in this Pull Request

Adds a repo-committed Claude Code skill, .claude/skills/pr/SKILL.md, invoked as /pr, that guides agents through opening a PR to this repository:

  • Base branch: develop always (never master), with an exception for branches cut from an active integration branch (e.g. v1-epic), detected via merge-bases.
  • Push remote: upstream when the invoking user has push permission, otherwise their fork with a cross-repo PR — so the same skill serves maintainers and external contributors.
  • Pre-PR gauntlet: clean tree, up-to-date with base, pre-commit, and Tier A (pytest -m fast, mirroring CI's fast-tests job). For changes that can alter network artifacts, the agent prompts the user to run Tier B (pytest -m integration) — encouraged, not required.
  • PR body: fills the template, adds a Testing section with tier evidence, and for larger changes an Equivalence section citing harness results / deltas-ledger rows — or an explicit statement that no run exists.
  • Draft when evidence is missing on a larger change; watches the fast-tests check after opening.

Also refreshes .github/pull_request_template.md: checklist paths had gone stale (envs/environment.yaml, config.default.yaml, doc/configtables/*.csvworkflow/envs/environment.yaml, workflow/repo_data/config/config.default.yaml, docs/source/configtables/*.csv) and adds a Testing section the skill fills in.

Note: the skill references the tiered pytest markers (fast/integration) that land on develop with #775; until then Tier A resolves against the existing unit suite once that merges.

Testing

Checklist

  • I tested my contribution locally and it seems to work fine.
  • pre-commit and pytest -m fast pass locally.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to workflow/envs/environment.yaml. (n/a)
  • Changes in configuration options are added in workflow/repo_data/config/config.default.yaml. (n/a)
  • Changes in configuration options are also documented in docs/source/configtables/*.csv. (n/a)

🤖 Generated with Claude Code

.claude/skills/pr/SKILL.md guides agents through opening a PR to
PyPSA/pypsa-usa: base develop (never master, integration branches
excepted), push-permission-aware remote choice, pre-commit + Tier A
gauntlet, Tier B prompt for larger changes, equivalence/deltas-ledger
evidence in the body, draft-when-evidence-missing, and fast-tests CI
watch.

Refresh .github/pull_request_template.md paths that had gone stale
(envs/, config.default.yaml, doc/configtables -> workflow/envs/,
workflow/repo_data/config/, docs/source/configtables/) and add a
Testing section for tier evidence.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ktehranchi
ktehranchi merged commit d93763f into develop Aug 24, 2026
1 check passed
@ktehranchi ktehranchi mentioned this pull request Aug 28, 2026
5 tasks
@ktehranchi
ktehranchi deleted the pr-skill branch August 28, 2026 20:07
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