From 82ba704957533c8f3e2f83657fe77e26cffc77bb Mon Sep 17 00:00:00 2001 From: christian-westerkam Date: Wed, 21 Oct 2026 00:12:37 +0200 Subject: [PATCH] Document branch-and-PR workflow; never write to main Co-Authored-By: Claude Opus 5.5 --- CLAUDE.md | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/CLAUDE.md b/CLAUDE.md index 2117ff2..e3dbd86 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -12,5 +12,14 @@ A tiny zero-dependency Node.js todo CLI, used as a sandbox for trying Claude Cod - Every behavior change gets a test in `test/`. - Commit messages: short imperative subject line. +## Workflow (required) +Never commit, merge, or push to `main`. Only a human (Christian) merges into `main`. +1. Branch off an up-to-date `main` before any change: `git switch -c /` (e.g. `fix/duplicate-ids`, `docs/readme`). +2. Commit on that branch; run `npm test` before pushing. +3. Push the branch and open a PR against `main`. +4. Run `/code-review` on the PR branch. +5. Post the result as a PR comment that starts with either **Approved** or **Needs fixes**, followed by the findings. If it needs fixes, fix them on the same branch, push, and repeat steps 4–5. +6. Stop there. Do not merge the PR, even if approved — tell the human it's ready. + ## Remote Gitea at code.westerkam.org — `origin` = `git@code.westerkam.org:christian/claude-playground.git`.