Pre-merge review checklist based on recurring AI reviewer feedback patterns
Before merging, verify these items. They represent the most common issues caught by automated code reviewers (Copilot, Gemini) on IronClaw PRs.
redact_params() before logging or SSE/WebSocket broadcastrequires_approval() returning Always or UnlessAutoApproved&s[..n]) on external/user strings -- use is_char_boundary() or char_indices().to_ascii_lowercase() before matching)LlmProvider trait methods are delegated in ALL wrapper types (grep impl LlmProvider for)tempfile crate, no hardcoded /tmp/ pathsserial_test)