review

<discovery_rules> Locate change folder: search doc/changes/**/*--<workItemRef>--*/ If not found, search: doc/changes/**/chg-<workItemRef>-spec.md Spec file: chg-<workItemRef>-spec.md; derive slug & change.type from frontmatter. Plan file: chg-<workItemRef>-plan.md Folder pattern: doc/changes/YYYY-MM/YYYY-MM-DD--<workItemRef>--<slug>/ Abort with clear error if spec OR plan missing. </discovery_rules>

<branch_resolution> changeBranch = <change.type>/<workItemRef>/<slug> headRef: directives override, else try changeBranch, fallback: current HEAD. baseBranch: directives override, else main, fallback: master. Compute merge-base and gather symmetric diff baseBranch...headRef. </branch_resolution>

<directive_parsing> Directives (case-insensitive):

• Base branch: base=<branch> | base branch <branch> | compare vs <branch>
• Head ref: head=<ref> | head ref <ref> | branch <ref>
• Suppress commit trigger: no commit | commit=false
• Dry run: dry run | preview only Unrecognized tokens ignored. </directive_parsing>

<pre_flight>

1. Validate workItemRef format (uppercase prefix + hyphen + digits).
2. Resolve spec; extract change.type & slug.
3. Resolve plan; parse phases & existing remediation.
4. Load repository rules (AGENTS.md, .ai/rules/**).
5. Resolve branches & compute diff metadata.
6. If dryRun: prepare preview but do NOT write. </pre_flight>

<review_method> The reviewer agent applies its full review framework:

Spec/plan compliance (local mode):

• Scope compliance: changed files align with spec capabilities
• Plan alignment: all tasks done, acceptance criteria have evidence
• Plan task audit: OPEN_TASKS, DONE_BUT_UNCHECKED, CHECKED_BUT_MISSING
• Out-of-scope detection: changes to files not in plan

Code quality heuristics (both modes):

• Full built-in heuristic framework: correctness, security, performance, reliability, API compat, testing gaps, documentation, dependencies
• Repository-local rules from .ai/agent/code-review-instructions.md and .ai/rules/

Ticket context (when available):

• Acceptance criteria verification against implementation
• Linked issue traversal for additional constraints and decisions

The reviewer loads all available context — spec, plan, ticket, repo rules, heuristics — and applies them holistically. Do not duplicate heuristic definitions here; they live in the reviewer agent prompt. </review_method>

<findings_format> [severity: major|minor|nit] <file>[:line] — <description>; fix: <action> </findings_format>

<remediation_phase> If findings exist, append new phase to plan:

### Phase X: Code Review Remediation

- Goal: Address code review findings.
- Tasks:
  - [ ] <precise fix per finding>
- Acceptance criteria:
  - Must: All fixes implemented and validated.
  - Must: Updated tests pass.
- Files and modules: <paths>
- Completion signal: docs(plan): remediate review findings for <workItemRef>

Rules:

• X = max existing phase + 1.
• Do NOT modify earlier phases.
• Append revision log entry. </remediation_phase>

<commit_behavior>