[claude] Add Commit Discipline section to CLAUDE.md (#429) #436

claude · 2026-03-24T18:13:22Z

claude commented

2026-03-24 18:13:22 +00:00

Fixes #429

Summary

Adds a dedicated Commit Discipline section to CLAUDE.md making PR size limits and concern-separation rules explicit and prominent.

Changes

New ## Commit Discipline section with:
- <150 line target, >200 line hard split requirement
- One-concern-per-PR rule (data, logic, OR visuals — never mixed)
- Commit-by-function table with example messages per concern type
- Decomposition rules for multi-concern features (data first, visual second)
- Modularization epic exception (e.g. #409 phases)

Data Integrity Audit

Element	Category	Data Source
CLAUDE.md (docs only)	N/A — no visual element	N/A

No visual elements added or modified.

Test Plan

Open CLAUDE.md — verify ## Commit Discipline section appears between ## Conventions and ## PR Rules
Confirm size limits (<150 target, >200 split) are stated
Confirm one-concern-per-PR rule is explicit
node --check app.js — passes ✅

Verification

Docs-only change. No screenshots required. node --check app.js passes.

Syntax Check

node --check app.js  →  OK

Fixes #429 ## Summary Adds a dedicated **Commit Discipline** section to `CLAUDE.md` making PR size limits and concern-separation rules explicit and prominent. ### Changes - New `## Commit Discipline` section with: - `<150` line target, `>200` line hard split requirement - One-concern-per-PR rule (data, logic, OR visuals — never mixed) - Commit-by-function table with example messages per concern type - Decomposition rules for multi-concern features (data first, visual second) - Modularization epic exception (e.g. #409 phases) ## Data Integrity Audit | Element | Category | Data Source | |---------|----------|-------------| | CLAUDE.md (docs only) | N/A — no visual element | N/A | No visual elements added or modified. ## Test Plan - [ ] Open CLAUDE.md — verify `## Commit Discipline` section appears between `## Conventions` and `## PR Rules` - [ ] Confirm size limits (<150 target, >200 split) are stated - [ ] Confirm one-concern-per-PR rule is explicit - [ ] `node --check app.js` — passes ✅ ## Verification Docs-only change. No screenshots required. `node --check app.js` passes. ## Syntax Check ``` node --check app.js → OK ```

claude added 1 commit 2026-03-24 18:13:23 +00:00

docs: add Commit Discipline section to CLAUDE.md

CI / validate (pull_request) Successful in 4s

Details

CI / auto-merge (pull_request) Successful in 1s

Details

a3bcc7b0ee

Adds a dedicated "Commit Discipline" section with explicit PR size
limits (<150 lines target, >200 lines must split), one-concern-per-PR
rule, commit-by-function table, decomposition rules, and the
modularization epic exception.

Fixes #429

claude referenced this pull request

2026-03-24 18:13:32 +00:00

[process] Atomic Commits — PR Size Discipline in CLAUDE.md #429

claude merged commit ee9d5b0108 into main

2026-03-24 18:13:33 +00:00

claude referenced this issue from a commit

2026-03-24 18:13:35 +00:00

[claude] Add Commit Discipline section to CLAUDE.md (#429) (#436)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Timmy_Foundation/the-nexus#436