Skip to content

Commit 0e3e61e

Browse files
sktbrdclaude
andcommitted
docs: replace PR protocol with Vlad's direct-to-main process
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 4c5dc3c commit 0e3e61e

1 file changed

Lines changed: 8 additions & 43 deletions

File tree

CLAUDE.md

Lines changed: 8 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -154,50 +154,15 @@ Secrets never go in client code. `NEXT_PUBLIC_*` is public by definition.
154154
- **Address displays link to the internal profile** `/members/[address]`, never Basescan.
155155
- **Never invent user-facing status copy** ("coming soon", "beta", "unstable") from code state — confirm the feature's real status with the user first.
156156
- **Format + lint before every commit**, not after CI flags it: `pnpm format` on touched files, then `pnpm lint`. Pre-existing lint errors outside your diff: report, don't fix.
157-
- **End every change report with explicit git state**: branch, commit hash, pushed or not, PR URL. Never commit or push without explicit instruction.
157+
- **End every change with the one-line report**: `<projeto> · <sha curto> · <o que mudou> · deploy <success|building|failed>`. A change is NOT done until its deploy is `success` (verify via GitHub deployments API, full sha).
158158
- **Continuing an existing PR**: `git fetch origin pull/<N>/head:pr-<N>` and commit on that branch — don't create a new one. Fresh worktrees need `pnpm install` first (or invoke binaries from the main repo's `node_modules/.bin/`).
159159

160-
## Pull Request Protocol
160+
## Direct-to-Main Protocol (Vlad, 2026-08-24 — supersedes the old PR protocol)
161161

162-
All medium and large tasks MUST be delivered via Pull Request. Do not commit directly to `main`.
162+
Everything lands directly on `main` and in production. Do not open a PR and wait for review — the goal is visibility, not speed: work sitting in unreviewed PRs made it impossible to know what was live.
163163

164-
| Size | Criteria | PR Required? |
165-
| ---------- | ---------------------------------------------- | ------------ |
166-
| **Small** | Single-file fix, typo, config tweak, <20 lines | Optional |
167-
| **Medium** | Multi-file, new component, 20–100 lines | **Yes** |
168-
| **Large** | Cross-cutting, refactor, 100+ lines | **Yes** |
169-
170-
When in doubt, create a PR.
171-
172-
### Workflow
173-
174-
1. Branch from `main` (or stack from prior feature branch). Format: `feat/*`, `fix/*`, `update/*`. Use a git worktree when the main repo has unrelated work in progress.
175-
2. Small atomic commits with clear messages.
176-
3. `gh pr create` with title <70 chars and the body template below. Target `main` unless stacking.
177-
4. Report the PR URL to the user.
178-
179-
### PR Body Template
180-
181-
```markdown
182-
## Summary
183-
184-
- [1-3 bullets: what changed and why]
185-
186-
## Changes
187-
188-
- [key files/areas modified]
189-
190-
## Test plan
191-
192-
- [ ] [how to verify]
193-
194-
Generated with [Claude Code](https://claude.com/claude-code)
195-
```
196-
197-
### Worktrees
198-
199-
Use when: current dir has uncommitted work on another branch; implementing a plan that needs isolation; user asks for isolation.
200-
201-
### Stacking
202-
203-
First PR → `main`. Subsequent PRs target the previous feature branch. Merge in order; rebase as needed.
164+
- After EVERY change, report in ONE line: `<projeto> · <sha curto> · <o que mudou> · deploy <success|building|failed>`.
165+
- A change is not done until the deploy is `success`. A commit on `main` without a confirmed deploy is the old blindness, only faster.
166+
- Gates that still apply before pushing: `tsc` and the test suite green; PT-BR capture for any layout change; a failed read renders as a failure, never as zero.
167+
- Do NOT wait for approval on routine changes. Ask first ONLY for: changes touching power or money (governance, funds, permissions), hard-to-reverse actions, and anything touching credentials.
168+
- PRs are the exception, used only when explicitly requested. Worktrees still apply when the main repo has unrelated work in progress.

0 commit comments

Comments
 (0)