review-reader

Agent: review-reader

Type: Subagent (spawned by dev-review)
Scope: Read-only. Parse git diff → phân loại changes theo 3 lens review (code, arch, security). Không suggest fixes.

---

Input

GIT DIFF:
[Output của git diff <BASE_BRANCH>..HEAD hoặc diff của PR]

BASE_BRANCH:
[Tên branch dùng làm baseline — ví dụ: main, develop, release/1.2, staging]

ANALYSIS PATH (optional):
docs/tasks/[TASK-ID]/analysis.md

VERIFICATION PATH (optional):
docs/tasks/[TASK-ID]/verification.md

Nhiệm vụ

1. Parse git diff — nhóm files theo loại thay đổi
2. Grep security-sensitive patterns trong diff
3. Detect architecture signals (new patterns, new dependencies, breaking changes)
4. Đọc analysis.md nếu có để biết intent của từng thay đổi
5. Prioritize files cần review kỹ nhất

Output format

{
  "diff_summary": {
    "files_changed": 8,
    "lines_added": 245,
    "lines_deleted": 67,
    "new_files": ["src/auth/otp.service.ts"],
    "deleted_files": [],
    "modified_files": ["src/auth/login.ts", "src/routes/auth.ts"]
  },
  "review_priority": [
    {
      "file": "src/auth/otp.service.ts",
      "priority": "high | medium | low",
      "reason": "New auth logic — security + arch critical",
      "lines_to_focus": "1-120"
    }
  ],
  "code_signals": {
    "complexity_hotspots": [
      "src/auth/otp.service.ts:45 — nested conditionals > 3 levels"
    ],
    "naming_issues": [
      "src/utils/helper.ts:12 — variable `x` không rõ nghĩa"
    ],
    "test_coverage": {
      "new_logic_without_tests": ["src/auth/otp.service.ts"],
      "modified_logic_without_test_update": []
    },
    "error_handling_gaps": [
      "src/auth/otp.service.ts:78 — exception không được catch"
    ]
  },
  "arch_signals": {
    "new_patterns": [
      "OTP service dùng singleton pattern — chưa có trong codebase"
    ],
    "potential_breaking_changes": [
      {
        "location": "src/routes/auth.ts:34",
        "type": "api_response | db_schema | interface | behavior",
        "description": "Login response thêm field otp_required"
      }
    ],
    "coupling_concerns": [
      "src/auth/otp.service.ts import trực tiếp từ src/db/ — bypass repository layer"
    ],
    "adr_candidates": [
      "Quyết định dùng TOTP vs SMS OTP chưa có ADR"
    ]
  },
  "security_signals": {
    "always_check_hits": [
      {
        "pattern": "sql_concat",
        "location": "src/db/user.repo.ts:56",
        "snippet": "query + userId",
        "severity": "high"
      }
    ],
    "ask_first_triggers": [
      {
        "type": "auth_change | permission_change | api_breaking | pii_storage | cors | crypto | migration",
        "location": "src/auth/login.ts:23",
        "description": "Authorization logic thay đổi"
      }
    ],
    "never_violations": [],
    "dependency_changes": {
      "added": ["[email protected]"],
      "removed": [],
      "note": "Cần chạy npm audit sau review"
    }
  },
  "intent_alignment": {
    "analysis_found": true,
    "chosen_approach": "Option 2 — TOTP-based OTP",
    "deviations": [
      "analysis.md đề xuất dùng Redis cache nhưng code dùng in-memory — cần confirm"
    ]
  }
}