Prompt file imported from wghglory/ngx-lift-workspace (
.cursor/commands/review.md). Copyright stays with the author.
Code Review
Review changed files against tech standards and automatically apply fixes.
Usage
/review # Review local staged/unstaged changes
/review 123 # Review PR #123
How It Works
Local Changes (no PR number)
- Find changed files via
git statusandgit diff - Read files from local workspace
- Apply fixes automatically
- Generate report with before/after code
Pull Request (with PR number)
- Detect repository:
wghglory/ngx-lift-workspacefromgit remote -v - Fetch PR via GitHub MCP tools (
pull_request_read,get_files,get_diff) - Read file content from PR branch using
get_file_contents - Report issues (read-only, no auto-fix)
- Generate review report
Review Checklist
Review standards are defined in .cursor/rules/ with checklists in each file:
general.mdc- TypeScript, code quality, library development, JSDoc documentationangular.mdc- Angular patterns, signals, OnPush, control flowclarity.mdc- Clarity components, SASS variables, styling (clr-lift only)testing.mdc- Test coverage, stable selectorsnx.mdc- Module boundaries, workspace structure, library publishingrxjs.mdc- RxJS patterns, operators, async state management
Priority Levels
Critical: Missing JSDoc for public APIs, TypeScript/ESLint errors, missing OnPush, any types, hard-coded colors
(clr-lift), accessibility issues
Warnings: Old Angular patterns, missing track functions, missing test coverage
Suggestions: Code style, performance optimizations
Key Rules
- Review ONLY changed files, not entire codebase
- PR reviews: Read from PR branch (not local workspace)
- Local reviews: Apply fixes automatically
- PR reviews: Report issues only (read-only)
- Check JSDoc documentation for all new public APIs
- Respect library context:
ngx-lift/(no UI) vsclr-lift/(Clarity UI) vsdemo/(showcase app)
