Claude-skill-registry commit-helper
Help create git commits and PRs with properly formatted messages and release notes following CockroachDB conventions. Use when committing changes or creating pull requests.
install
source · Clone the upstream repo
git clone https://github.com/majiayu000/claude-skill-registry
Claude Code · Install into ~/.claude/skills/
T=$(mktemp -d) && git clone --depth=1 https://github.com/majiayu000/claude-skill-registry "$T" && mkdir -p ~/.claude/skills && cp -r "$T/skills/data/commit-helper-cockroachdb-cockroach" ~/.claude/skills/majiayu000-claude-skill-registry-commit-helper && rm -rf "$T"
manifest:
skills/data/commit-helper-cockroachdb-cockroach/SKILL.mdsource content
CockroachDB Commit Helper
Help the user create properly formatted commit messages and release notes that follow CockroachDB conventions.
Workflow
- Analyze the changes: Run
orgit diff --staged
to understand what was modifiedgit diff - Determine the package prefix: Identify the primary package affected
- Ask the user for:
- Issue/epic number if not already known
- Whether release notes are needed and what category fits best
- Write the subject line: Imperative mood, no period, under 72 characters
- Write the body: Explain the before/after, why the change was needed
- Add issue references: Include Resolves/Epic as appropriate
- Write the release note: Clear, user-focused description of the change
- Create the commit using the properly formatted message
Commit Message Structure
Basic Format:
package: imperative title without period Detailed explanation of what changed, why it changed, and how it impacts users. Explain the problem that existed before and how this commit solves it. Include context about alternate approaches considered and any side effects or consequences. Resolves: #123 Epic: CRDB-357 Release note (category): Description of user-facing change in past or present tense explaining what changed, how users can see the change, and why it's important.
Key Requirements:
- Must include release note annotation (even if "Release note: None")
- Must include issue or epic reference
- Must separate subject from body with blank line
- Recommended prefix subject with affected package/area
- Recommended use imperative mood in subject (e.g. "fix bug" not "fixes bug")
- Recommended wrap body at 72-100 characters
Release Note Categories
When to include release notes:
- Changes to user interaction or experience
- Changes to product behavior (performance, command responses, architecture)
- Bug fixes affecting external users
When to exclude release notes:
- Internal refactors, testing, infrastructure work
- Code that's too immature for docs (private preview features)
- Internal settings beginning with
crdb_internal. - Functionality not accessible to external users
Valid Categories:
- Breaking changes to stable interfacesbackward-incompatible change
- Features requiring enterprise licenseenterprise change
- Commands/endpoints for operators (logging, metrics, CLI flags)ops change
- Commands for developers/contributors (SQL shells, debug tools)cli change
- SQL statements, functions, system catalogssql change
- DB Console changesui change
- Security feature changessecurity update
- Performance enhancementsperformance improvement
- Multi-tenancy infrastructurecluster virtualization
- Problem fixesbug fix
- Changes that don't fit elsewheregeneral change
- Source build requirementsbuild change
Release Note Best Practices
Description guidelines:
- Default to more information rather than less
- Explain what changed, how it changed, and why it's relevant
- Use past tense ("Added", "Fixed") or present tense ("CockroachDB now supports")
- For bug fixes: describe cause, symptoms, and affected versions
- Note if change is part of broader roadmap feature
Examples:
Good bug fix:
Release note (bug fix): Fixed a bug introduced in v19.2.3 that caused duplicate rows in CREATE TABLE ... AS results when multiple nodes attempt to populate the results.
Good feature:
Release note (enterprise change): Shortened the default interval for the kv.closed_timestamp.target_duration cluster setting from 30s to 3s. This allows follower reads at 4.8 seconds in the past, a much shorter window than the previous 48 seconds.
Issue References
- Auto-closes issue on PR mergeResolves: #123
- Cross-references issuesSee also: #456, #789
- Links to epicEpic: CRDB-357
How to Avoid Common Pitfalls
- Always include a release note annotation (even "Release note: None")
- Use only valid category names from the list above
- Keep release notes focused on user-facing information
- Write specific descriptions that explain the impact to users
- Use
category for any breaking changesbackward-incompatible change - End subject lines without punctuation
- Explain the "why" behind changes, not just the "what"
Pull Request Guidelines
- Create PRs from your personal fork, not directly on cockroachdb/cockroach
- Single-commit PRs: PR title should match commit title, PR body should match commit body