docs: add PR size limits to CLAUDE.md
docs: add PR size limits to CLAUDE.md
This commit is contained in:
commit
aa1353e6a9
2 changed files with 18 additions and 4 deletions
13
CLAUDE.md
13
CLAUDE.md
|
|
@ -125,6 +125,19 @@ The authenticated email comes from `securityIdentity.getPrincipal().getName()` a
|
||||||
|
|
||||||
This ensures code review, clear history, and traceability.
|
This ensures code review, clear history, and traceability.
|
||||||
|
|
||||||
|
### PR Size Limits
|
||||||
|
|
||||||
|
To keep PRs focused and reviewable, enforce these limits **per PR**:
|
||||||
|
- **Maximum 5 files changed**
|
||||||
|
- **Maximum 30 lines changed** (added, modified, or deleted combined)
|
||||||
|
|
||||||
|
**If exceeding these limits:** Split into multiple PRs. **Prefer splitting by layer:**
|
||||||
|
- One PR for backend changes (`backend/`)
|
||||||
|
- One PR for frontend changes (`frontend/`)
|
||||||
|
- One PR for config/infra (`docker-compose.yml`, `.env`, etc.)
|
||||||
|
|
||||||
|
This keeps code review focused, reduces merge conflicts, and makes history easier to bisect.
|
||||||
|
|
||||||
## Skills
|
## Skills
|
||||||
|
|
||||||
Skills do projeto em `.claude/skills/`:
|
Skills do projeto em `.claude/skills/`:
|
||||||
|
|
|
||||||
|
|
@ -21,16 +21,17 @@ export async function apiFetch<T>(path: string, options: RequestOptions = {}) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
const isFormData = body instanceof FormData
|
||||||
|
const contentTypeHeader = isFormData ? {} : { 'Content-Type': 'application/json' }
|
||||||
|
|
||||||
const response = await fetch(buildUrl(path), {
|
const response = await fetch(buildUrl(path), {
|
||||||
...rest,
|
...rest,
|
||||||
headers: {
|
headers: {
|
||||||
'Content-Type': 'application/json',
|
...contentTypeHeader,
|
||||||
...authHeader,
|
...authHeader,
|
||||||
...headers,
|
...headers,
|
||||||
},
|
},
|
||||||
body: body !== undefined && body !== null && !(body instanceof FormData)
|
body: isFormData ? body : (body !== undefined && body !== null ? JSON.stringify(body) : undefined),
|
||||||
? JSON.stringify(body)
|
|
||||||
: body,
|
|
||||||
})
|
})
|
||||||
|
|
||||||
const text = await response.text()
|
const text = await response.text()
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue