diff options
| author | Craig Jennings <c@cjennings.net> | 2026-04-22 08:02:27 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-04-22 08:02:27 -0500 |
| commit | 5302673fab3ade9a348dd096aafe12fe21310ff6 (patch) | |
| tree | 3d819f87227de733623b4aa88963f76f99f86ca2 /review-code | |
| parent | 36bf98003a794cd8c780c32a6246cb3eaea04148 (diff) | |
| download | rulesets-5302673fab3ade9a348dd096aafe12fe21310ff6.tar.gz rulesets-5302673fab3ade9a348dd096aafe12fe21310ff6.zip | |
docs(commits): add review-and-publish flow, tighten message conventions
Three additions to claude-rules/commits.md.
A new conventions section adds two rules. First, do not hard-wrap commit body lines. Let the renderer soft-wrap instead. Hard wraps shrink the visible width in web UIs and cause awkward mid-sentence breaks. Second, do not reference how the change was produced (code review, linting, test runs) in the message. The body is for the what and the why. The how belongs in the PR discussion.
A new Review and Publish section defines a two-step gate for every commit and PR. Step 1 runs /review-code against the change and blocks on Critical or Important findings unless the user overrides. Step 2 drafts the message to /tmp, opens it in the user's editor, runs the humanizer skill, and waits for explicit approval before committing or pushing. PRs also require a Linear cross-link line, a Linear comment with the PR URL after creation, and a status transition to Dev Review.
A short hook-level authorization note clarifies that the review-plus-approval flow is the authorization gate. Tools like git-commit-confirm should not add a second independent prompt once Step 2 has been approved.
Diffstat (limited to 'review-code')
0 files changed, 0 insertions, 0 deletions
