aboutsummaryrefslogtreecommitdiff
path: root/review-code
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-04-22 08:02:27 -0500
committerCraig Jennings <c@cjennings.net>2026-04-22 08:02:27 -0500
commit5302673fab3ade9a348dd096aafe12fe21310ff6 (patch)
tree3d819f87227de733623b4aa88963f76f99f86ca2 /review-code
parent36bf98003a794cd8c780c32a6246cb3eaea04148 (diff)
downloadrulesets-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