| Functionality |
20 |
Doesn’t function. Cannot demo core flow. |
Partially works. Several acceptance criteria unmet. Crashes on edge cases. |
Core happy path works. Some acceptance criteria missing. Limited edge case handling. |
Core acceptance criteria met. Most edge cases handled. Minor gaps in error handling. |
All acceptance criteria met. Edge cases handled. Graceful error handling. Works in all specified scenarios. |
| Code Quality |
20 |
Unreadable. No conventions followed. Security issues. |
Hard to follow. Multiple convention violations. Poor structure. |
Readable but inconsistent. Some convention violations. Acceptable structure. |
Follows most conventions. Minor style inconsistencies. Good structure overall. |
Follows all .dev/conventions/. Clean separation of concerns. Type-safe. No dead code. No OWASP top-10 issues. |
| Testing |
20 |
No tests, or tests don’t pass. |
Minimal tests. Only trivial cases. Coverage <40%. |
Basic unit tests exist. Happy path covered. Limited edge case tests. Coverage >40%. |
Good unit test coverage. Some integration tests. Core paths tested. Coverage >60%. |
Unit tests for all public functions. Integration tests for API/DB flows. Edge case tests. Readable + maintainable. Coverage >80%. |
| Integration & Architecture |
20 |
Incompatible with the platform. Cannot merge. |
Doesn’t follow platform architecture. Would require significant rework to merge. |
Works in isolation but integration story is incomplete. Architectural shortcuts. |
Good architectural fit. Uses most existing patterns. Minor integration gaps. |
Follows .dev/conventions/03-architecture.md. Uses existing patterns (MCP, bridge protocol, config cascade). Clean API boundaries. No circular imports. Backward compatible. |
| Demo & Documentation |
20 |
Cannot demo. No PR. |
No real demo. PR description is a title only. No documentation. |
Demo exists but incomplete. PR description is minimal. Limited documentation. |
Good demo covering core flow. Adequate PR description. Some documentation. |
Clear 10-minute demo with end-to-end flow. PR description: what / why / how. Inline comments where logic is non-obvious. CLAUDE.md updated. |