code-review-gate
Skill1 starsChanged 45 days agoMITView on GitHub
Chạy đủ 4 lớp Validation Gate (Automated Build & Lint / Spec Compliance / Constitution Check / Acceptance Criteria) trước khi mở PR.
3 other projects use a version of this.
Adopted by
Most recently taken up by
- kyy-logs/code-review-skilltheir own version, first seen Nov 8, 2026
- ultimatile/development-skillstheir own version, first seen Oct 20, 2026
- wangheng19901021/skillstheir own version, first seen Jul 18, 2026
Other versions3 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 3
- kyy-logs/code-review-skill★ 400same name
- wangheng19901021/skills★ 16same name
- ultimatile/development-skills★ 0same name
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository83 other files it gives its agents.
- AGENTS.mdAGENTS.md
- CLAUDE.mdCLAUDE.md
- adopt-repoSkill
- agent-strict-guardSkill
- analyze-featureSkill
- backend-api-design-flowSkill
- clean-code-naming-conventionsSkill
- code-review-gateSkill
- dangerous-action-blockerSkill
- database-transaction-managementSkill
- db-interface-procedure-flowSkill
- db-migrationSkill
- defensive-troubleshooting-guideSkill
- git-workflowSkill
- human-authority-overrideSkill
- human-master-mapSkill
- implement-featureSkill
- jpa-n-plus-one-optimizationSkill
- knowledge-graphSkill
- multi-language-error-handlingSkill
- post-code-verificationSkill
- project-doctorSkill
- project-governance-dashboardSkill
- redis-cache-patternsSkill
- release-notesSkill
- safe-git-rebaseSkill
- security-secret-scannerSkill
- speckit-analyzeSkill
- speckit-auditSkill
- speckit-checklistSkill
- speckit-clarifySkill
- speckit-constitutionSkill
- speckit-helpSkill
- speckit-implementSkill
- speckit-ingestSkill
- speckit-planSkill
- speckit-specifySkill
- speckit-tasksSkill
- spec-writerSkill
- spring-middleware-pipelineSkill
- sql-repository-patternSkill
- system-dna-managerSkill
- test-featureSkill
- adopt-repoSkill
- agent-strict-guardSkill
- analyze-featureSkill
- backend-api-design-flowSkill
- clean-code-naming-conventionsSkill
- dangerous-action-blockerSkill
- database-transaction-managementSkill
- db-interface-procedure-flowSkill
- db-migrationSkill
- defensive-troubleshooting-guideSkill
- git-workflowSkill
- human-authority-overrideSkill
- human-master-mapSkill
- implement-featureSkill
- jpa-n-plus-one-optimizationSkill
- knowledge-graphSkill
- multi-language-error-handlingSkill
- The first 60 shown.
The file
Files
<!-- GENERATED FROM registry/skills/code-review-gate/SKILL.md — DO NOT EDIT DIRECTLY --><!-- GENERATED FROM .shared/skills/03-hooks/code-review-gate/SKILL.md — DO NOT EDIT DIRECTLY -->Skill — Code Review Gate (Backend Microservice)
Chạy 4 lớp kiểm tra chất lượng nghiêm ngặt trước khi code được merge vào branch chính (kết hợp quy trình từ /post-code-verification và /defensive-troubleshooting-guide).
4 LỚP VALIDATION GATE:
Gate 1: Automated Checks (Build, Lint, Unit Tests)
- Chạy build/linter của stack (Spotless / Gradle / Maven / golangci-lint / ESLint): 0 warning.
- Toàn bộ test suite chạy pass: 100% tests green.
- Coverage đạt ngưỡng
CONSTITUTION.md §5(>= 80% line coverage).
Gate 2: Spec Compliance
- API Request/Response DTO khớp đúng contract trong
SPEC.md. - Event message schema khớp đúng topic và trường dữ liệu.
- Mọi task trong
TASKS.mdđã được hoàn thành.
Gate 3: Constitution & Architecture Check
- Zero Entity Leakage: DTO Pattern được áp dụng 100%.
- Soft delete: Không có câu lệnh
DELETE FROMtrực tiếp. - Không hardcode secret hoặc credentials (Zero Secret Leak).
- Không có query
SELECT *không kiểm soát, không bị bẫy N+1 Query (xem/jpa-n-plus-one-optimization). @TransactionalcórollbackFor = [Exception.class], không bọc lệnh gọi mạng bên ngoài trong DB transaction.- Giới hạn method <= 40 dòng, file <= 300 dòng, PR <= 400 dòng.
Gate 4: Acceptance Criteria & Defensive Verification
- Kiểm tra happy path & error paths hoạt động đúng mong đợi.
- Rà soát 10 bẫy lỗi theo
/defensive-troubleshooting-guide(Zero Silent Catch, PII Masking, HTTP Timeouts). - Idempotency key và Resiliency fallback hoạt động tốt.
QUY TẮC: Bất kỳ Gate nào FAIL -> DỪNG LẠI và sửa ngay, KHÔNG tạo PR.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.