Backend Code Review
Review the requested scope for concrete, reproducible defects. The nearest AGENTS.md owns package facts and commands; this skill owns the review workflow and routes to its bundled rule packs.
Evidence First
- Establish the requested review scope and inspect the relevant diff or files.
- Read the changed lines, their behavior owner, nearby tests, and local docstrings or comments that define contracts.
- Trace callers, persistence boundaries, authorization, generated schemas, or external I/O only when they decide correctness.
- Report only findings tied to an observable failure, violated contract, security boundary, data integrity risk, or demonstrated maintenance problem.
Rule Routing
Read only the packs matched by the diff:
- Models or migrations:
references/db-schema-rule.md
- Controller, service, core/domain, library, or model dependency direction:
references/architecture-rule.md
- Table access outside an established repository boundary:
references/repositories-rule.md
- SQLAlchemy sessions, queries, transactions, CRUD, concurrency, or raw SQL:
references/sqlalchemy-rule.md
When no pack applies, review correctness, security, behavior changes, and test evidence directly. Check current official documentation only when local code and contracts do not settle framework or library behavior.
Severity And Output
- P0: security or privacy exposure, data loss, or a production-wide outage.
- P1: user-visible regression, broken authorization or tenant isolation, invalid public contract, or failed primary workflow.
- P2: concrete correctness, performance, maintainability, or test defect likely to cause incorrect behavior.
- P3: minor actionable cleanup; omit unless the user requested a thorough audit.
Lead with findings ordered by severity. Include a tight file and line reference, the failing contract or reproduction path, impact, and a concrete fix direction. If there are no findings, say No issues found. and state any material verification gap. Do not add praise sections, speculative risks, or an unsolicited offer to implement fixes.
1---2name: backend-code-review3description: Use only when the user explicitly requests a review or audit of backend code under `api/`. Supports pending-change, file-focused, and pasted-diff reviews. Do not use for implementation-only requests, diagnosis without review intent, frontend code, or backend code outside `api/`.4---5
6# Backend Code Review
7
8Review the requested scope for concrete, reproducible defects. The nearest `AGENTS.md` owns package facts and commands; this skill owns the review workflow and routes to its bundled rule packs.
9
10## Evidence First
11
121. Establish the requested review scope and inspect the relevant diff or files.
132. Read the changed lines, their behavior owner, nearby tests, and local docstrings or comments that define contracts.
143. Trace callers, persistence boundaries, authorization, generated schemas, or external I/O only when they decide correctness.
154. Report only findings tied to an observable failure, violated contract, security boundary, data integrity risk, or demonstrated maintenance problem.
16
17## Rule Routing
18
19Read only the packs matched by the diff:
20
21- Models or migrations: [`references/db-schema-rule.md`][db-schema]
22- Controller, service, core/domain, library, or model dependency direction: [`references/architecture-rule.md`][architecture]
23- Table access outside an established repository boundary: [`references/repositories-rule.md`][repositories]
24- SQLAlchemy sessions, queries, transactions, CRUD, concurrency, or raw SQL: [`references/sqlalchemy-rule.md`][sqlalchemy]
25
26When no pack applies, review correctness, security, behavior changes, and test evidence directly. Check current official documentation only when local code and contracts do not settle framework or library behavior.
27
28## Severity And Output
29
30- **P0**: security or privacy exposure, data loss, or a production-wide outage.
31- **P1**: user-visible regression, broken authorization or tenant isolation, invalid public contract, or failed primary workflow.
32- **P2**: concrete correctness, performance, maintainability, or test defect likely to cause incorrect behavior.
33- **P3**: minor actionable cleanup; omit unless the user requested a thorough audit.
34
35Lead with findings ordered by severity. Include a tight file and line reference, the failing contract or reproduction path, impact, and a concrete fix direction. If there are no findings, say `No issues found.` and state any material verification gap. Do not add praise sections, speculative risks, or an unsolicited offer to implement fixes.
36
37[architecture]: references/architecture-rule.md
38[db-schema]: references/db-schema-rule.md
39[repositories]: references/repositories-rule.md
40[sqlalchemy]: references/sqlalchemy-rule.md
langgenius