Remove repo-specific git author/workflow rules from CLAUDE.md

The house rules pinned a fixed commit author identity and personal git/CI workflow notes, which is arbitrary and incorrect for anyone else using Claude Code on this repo. Keep only the code-quality conventions (layout, tests, lint).
This commit is contained in:
medievalshell
2026-06-03 13:12:18 +02:00
committed by medievalshell
parent 62062cec9e
commit 8c12d8f7b0
+6 -14
View File
@@ -387,22 +387,14 @@ See `docs/ARCHITECTURE.md` "Recently fixed" for fix shapes.
## House rules ## House rules
- **Commit author**: `simoleo89 <simoleo89@users.noreply.github.com>`.
When committing, pass these via per-command overrides
(`git -c user.name=simoleo89 -c user.email=...`) — do NOT modify the
global git config.
- **No `claude/...` branch names** — auto-generated names should be
renamed before pushing. Prefer `feat/<description>`.
- **Never merge a branch that violates the layout convention** above. - **Never merge a branch that violates the layout convention** above.
The `feat/react19-hooks-adapter` branch (deleted) put hooks under The hooks-adapter approach that put hooks under `src/components/...` is
`src/components/...`; that's wrong and a recurring temptation. wrong and a recurring temptation.
- **Skip-motivated god-hook splits are fine** — when a hook's actions - **Skip-motivated god-hook splits are fine** — when a hook's actions
mutate internal state, document the reason in the commit message and mutate internal state, document the reason and move on rather than
move on rather than forcing a bad split. forcing a bad split.
- **`yarn test` must stay green** on every commit. Currently 193/193. - **`yarn test` must stay green**. `yarn typecheck` + `yarn test --run`
The GitHub Actions workflow at `.github/workflows/ci.yml` runs must both pass.
`yarn typecheck` + `yarn test --run` on every push to `main` /
`feat/**` and on every PR — both must pass.
- **Lint baseline**: don't regress. Some pre-existing errors (`FC<{}>`, - **Lint baseline**: don't regress. Some pre-existing errors (`FC<{}>`,
`IMessageEvent | undefined` redundant union in the local sandbox where `IMessageEvent | undefined` redundant union in the local sandbox where
the renderer SDK isn't installed) are out of scope here. the renderer SDK isn't installed) are out of scope here.