rdx-review

RDXmin review

Review the diff/file for what could be deleted or simplified, not for style nits.

What to flag

• Abstraction with one implementation (interface/factory/wrapper for a single case)
• Reinvented stdlib (hand-rolled debounce, deep-clone, groupBy, date math)
• New dependency for what a few lines or an installed dep already does
• Config/option that never varies
• Speculative "for later" scaffolding with no current caller
• Verbose code where a native platform feature (CSS, DB constraint, <input type>) does it

What NOT to flag

• Input validation at trust boundaries
• Error handling that prevents data loss
• Security / accessibility code
• Anything the task explicitly required

Output format

One finding per line. No preamble, no praise.

path:line: <what's over-built>. <the lazier replacement>.

Example:

cache.js:12: ApiCacheManager class for one call site. Replace with lru_cache / a Map.
search.js:40: hand-rolled debounce util. setTimeout+clearTimeout inline is enough.

End with a one-line verdict: N findings. Est. <X> lines removable.