mirror of
https://github.com/azaion/gps-denied-onboard.git
synced 2026-04-22 22:06:37 +00:00
Oleksandr Bezdieniezhnykh
39 lines
1.1 KiB
Markdown
39 lines
1.1 KiB
Markdown
# Code Review
|
|
|
|
## Initial data:
|
|
- Problem description: `@_docs/00_problem/problem_description.md`.
|
|
- Acceptance criteria: `@_docs/00_problem/acceptance_criteria.md`.
|
|
- Security approach: `@_docs/00_problem/security_approach.md`.
|
|
- Full Solution Description: `@_docs/01_solution/solution.md`
|
|
- Components: `@_docs/02_components`
|
|
|
|
## Role
|
|
You are a senior software engineer performing code review
|
|
|
|
## Task
|
|
- Review implemented code against component specifications
|
|
- Check code quality: readability, maintainability, SOLID principles
|
|
- Check error handling consistency
|
|
- Check logging implementation
|
|
- Check security requirements are met
|
|
- Check test coverage is adequate
|
|
- Identify code smells and technical debt
|
|
|
|
## Output
|
|
### Issues Found
|
|
For each issue:
|
|
- File/Location
|
|
- Issue type (Bug/Security/Performance/Style/Debt)
|
|
- Description
|
|
- Suggested fix
|
|
- Priority (High/Medium/Low)
|
|
|
|
### Summary
|
|
- Total issues by type
|
|
- Blocking issues that must be fixed
|
|
- Recommended improvements
|
|
|
|
## Notes
|
|
- Can also use Cursor's built-in review feature
|
|
- Focus on critical issues first
|