[AI Reviewer] Rule improvement: PERF-6

[kacper-mikolajczak]

Long-lived tracking issue for AI reviewer rule PERF-6.

Each comment below is an automated analysis run. Fix PRs should reference this issue.

---

Auto-managed by analyzeNegativeFeedback workflow

[kacper-mikolajczak]

Analysis run — 2026-02-16

Classification (30 instances sampled, 420 total)

Category	Count	Instances
Wrong context	28	Rule flags removal of useMemo/useCallback in components migrated to React Compiler — manual memoization is intentionally removed
Wrong context	1	State initialization pattern misidentified as missing memoization
Too strict	1	Trivial computation flagged unnecessarily

Root cause

React Compiler unawareness — the rule does not account for React Compiler, which automatically handles memoization. Developers are intentionally removing manual useMemo/useCallback as part of compiler adoption. The rule fights against this deliberate architectural decision.

Developer replies consistently state: "It's unnecessary, since the component gets compiled by React Compiler"

Proposed changes

Add React Compiler exclusion to PERF-6:

• Skip components compiled by React Compiler (check for compiler directives, 'use no memo', or project-level config)
• If PR description mentions React Compiler migration, suppress PERF-6 entirely
• When in doubt, add caveat: "If this component uses React Compiler, this suggestion may not apply"

Narrow trigger for trivial computations:

• Don't flag simple property access, basic conditionals, or single-method chains
• Only flag genuinely expensive operations (filtering/sorting/mapping large arrays, complex multi-step transformations)

---

Auto-generated from tracking issue #596749