Add agentic skills for commits, PRs, and issues - #27
Merged
Merged
Conversation
Port the commit, pr, and issue skills from elastic/docs-builder. The skills encode plain-language writing rules, full-URL linking, and a PR lifecycle check that refreshes stale descriptions after follow-up commits. Co-Authored-By: Claude <noreply@anthropic.com>
settings.local.json holds personal Claude Code overrides and should not be shared. The rest of .claude/ (skills, settings.json, hooks) is team-shared and stays committed. Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Four Claude Code skills land in
.claude/skills/: a commit flow, a PR flow, an issue flow, and a shared writing-style guide. Together they make Claude Code agents produce consistent, reviewable artifacts across repos.Prompt summary: The ask was to port a set of agentic skills from elastic/docs-builder to a wider set of repos. The skills enforce plain-language writing, full-URL issue/PR linking, and a PR lifecycle check that keeps descriptions current after follow-up commits.
Why
Without shared conventions, Claude Code agents in different repos produce commit messages and PR bodies with inconsistent structure, bare
#numlinks that break in cross-repo contexts, and PR descriptions that silently drift after new commits land on the branch.What
Commit skill
A guided commit flow: check for hooks, stage deliberately, write an imperative-mood subject line with a
Co-Authored-By:trailer, and refresh the PR description if one already exists.PR skill
A guided PR flow: front-loaded lead sentence, a
**Prompt summary:**field capturing the originating ask, a grounded## Why, behaviour-led## Whatsections, and an update path that rebuilds the body from the cumulative diff rather than appending to the old one.Issue skill
A guided issue flow: duplicate check first, consistent bug-report and feature-request structure, labels per repo conventions.
Writing style
ISO 24495-1 / ASD-STE100 sentence mechanics, liberal backtick use, and a full-URL linking rule — always
[#42](https://github.com/org/repo/pull/42), never a bare#42.