code-review-typescript

Code Review — TypeScript

Apply TypeScript-specific linting using the checklist in this skill directory.

Step 0 — Load Project Map

Check for .agentic/project-map.md:

• If present: read it. Use the layer structure, key modules table, and non-obvious conventions it defines to orient all findings. Skip redundant filesystem exploration.
• If absent: run lightweight auto-discovery:
  • Read package.json to identify dependencies and scripts
  • Inspect tsconfig.json for strict mode settings
  • Detect framework (Next.js, Hono, Vue, React, etc.) from dependencies
  • List top-level src/ directories
  • Suggest running the project-map skill after this review to avoid this overhead next time

Step 1 — Load Checklist

Read checklist.md in this skill directory and apply every item to the codebase.

Step 2 — Determine Scope

• If the user specifies files, review those.
• Otherwise review the current diff: git diff HEAD or staged changes.
• Do not review files outside stated scope.

Step 3 — Analyze with TypeScript-specific Lenses

Work through the checklist systematically. For each issue found, note:

• File path and line number
• Risk level: critical / high / medium / low
• Description of the issue
• Why it matters
• Verifiable source reference when the finding is non-obvious

Focus particularly on:

• Type safety gaps (any, implicit any, missing strict mode)
• ESM/CJS interop issues
• Async correctness (floating promises, Promise.all misuse)
• Money/precision issues
• Domain modeling boundary violations

Step 4 — Write the Review

Output the review in this exact format:

## Code Review — TypeScript

### What Works Well
- [At least one specific positive observation with file reference]

### Findings

#### Critical
- `path/to/file.ts:42` [critical] Description. Why it must change. *Source: [TypeScript Handbook — unknown](https://www.typescriptlang.org/docs/handbook/2/functions.html#unknown)*

#### High
- `path/to/file.ts:18` [high] Description. Why it matters. *Source: ...*

#### Medium
- `path/to/file.ts:7` [medium] Description.

#### Low
- `path/to/file.ts:5` [low] Minor note.

### Suggested Improvements
[Concrete alternatives and solutions for the most impactful findings]

### Summary
[One paragraph: overall quality, main risks, merge recommendation]