code-standards

name	code-standards
description	Code quality standards and style guide for reviewing pull requests
version	1.0.0
metadata	{"tags":["code-review","quality","style"]}

Code Standards Review

When reviewing code, follow this structured process:

Step 1: Critical Issues

Check for issues that MUST be fixed before merging:

• Logic bugs and incorrect behavior
• Missing error handling for failure cases
• Race conditions or concurrency issues
• Unhandled edge cases (null, undefined, empty arrays, boundary values)
• Breaking API changes without migration path

Step 2: Code Quality

Evaluate overall code quality:

• Functions should do one thing and be reasonably sized (< 50 lines preferred)
• Avoid code duplication — look for repeated patterns that should be abstracted
• Use descriptive, meaningful names for variables, functions, and types
• Prefer explicit types over any in TypeScript
• Ensure proper use of async/await (no floating promises, proper error propagation)

Step 3: Style Guide Conformance

Check against the style guide in references/style-guide.md:

• Naming conventions
• Code organization and import ordering
• Comment quality (explain "why", not "what")

Step 4: Linting Flags