Skip to content

Add open-pr skill and link from CLAUDE.md#17

Merged
VPDPersonal merged 1 commit into
Developfrom
chore/document-pr-conventions
May 10, 2026
Merged

Add open-pr skill and link from CLAUDE.md#17
VPDPersonal merged 1 commit into
Developfrom
chore/document-pr-conventions

Conversation

@VPDPersonal
Copy link
Copy Markdown
Owner

@VPDPersonal VPDPersonal commented May 10, 2026

Summary

  • Adds .claude/skills/open-pr/SKILL.md — a project skill that captures Aspid.FastTools PR conventions: title format, .github/PULL_REQUEST_TEMPLATE.md body shape (Summary / Notes for review / Linked issues), label policy, commit-message rules (no Co-Authored-By: Claude … trailers), scope hygiene, and a recipe checklist.
  • Replaces the long ad-hoc section I first wrote in CLAUDE.md with a 5-line Pull request conventions pointer that quotes the must-know rules and points at the skill for the full procedure.
  • Registers the new skill in the Local Claude Code automation list next to build-generator and sync-readmes.

Notes for review

Triggered by #16, where @claude opened a PR titled Claude/add onenable idregistry i bk jl with an empty body — exactly the failure mode this skill is meant to prevent. Skill-form has two advantages over a passive CLAUDE.md section:

  • The agent can be explicitly directed at the procedure (open-pr).
  • The procedure can be detailed (recipe, common failure modes, gh-cli HEREDOC pitfall) without bloating CLAUDE.md.

The CLAUDE.md pointer is kept because Claude Code Action reads CLAUDE.md as passive context — without it the skill might never get invoked. The pointer summarises the five rules that must apply even if the skill is missed.

Documentation-only — no code, no runtime impact.

Linked issues

(none)

@VPDPersonal VPDPersonal added type: documentation Documentation only type: chore Maintenance, version bumps, tooling and removed type: chore Maintenance, version bumps, tooling labels May 10, 2026
@VPDPersonal VPDPersonal force-pushed the chore/document-pr-conventions branch from 3674ca3 to f72f108 Compare May 10, 2026 20:17
@VPDPersonal VPDPersonal changed the title Document PR conventions in CLAUDE.md Add open-pr skill and link from CLAUDE.md May 10, 2026
@VPDPersonal VPDPersonal merged commit 0fa3755 into Develop May 10, 2026
1 check passed
@VPDPersonal VPDPersonal deleted the chore/document-pr-conventions branch May 10, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: documentation Documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant