review-verification-protocol

SKILL.md

This protocol MUST be followed before reporting any code review finding. Skipping these steps leads to false positives that waste developer time and erode trust in reviews.

| map.get(key) \|\| [] | Map.get() returns T \| undefined, fallback is correct | | Class exports without separate type export | Classes work as both value and type | | as const on literal arrays | Creates readonly tuple types | | Type annotation on variable declaration | Not a type assertion | | satisfies instead of as | Type checking without assertion |

| Array index as key (static list) | Valid when: items don't reorder, list is static, no item identity needed | | Inline arrow in onClick | Valid for non-performance-critical handlers (runs once per click) | | State that appears unused | May be set via refs, external callbacks, or triggers re-renders |

View raw