chisle-review

1# Chisle review
2 
3Review the diff/file for **what could be deleted or simplified**, not for style nits.
4 
5## What to flag
6 
7- Abstraction with one implementation (interface/factory/wrapper for a single case)
8- Reinvented stdlib (hand-rolled debounce, deep-clone, groupBy, date math)
9- New dependency for what a few lines or an installed dep already does
10- Config/option that never varies
11- Speculative "for later" scaffolding with no current caller
12- Verbose code where a native platform feature (CSS, DB constraint, `<input type>`) does it
13 
14## What NOT to flag
15 
16- Input validation at trust boundaries
17- Error handling that prevents data loss
18- Security / accessibility code
19- Anything the task explicitly required
20 
21## Output format
22 
23One finding per line. No preamble, no praise.
24 
25```
26path:line: <what's over-built>. <the lazier replacement>.
27```
28 
29Example:
30 
31```
32cache.js:12: ApiCacheManager class for one call site. Replace with lru_cache / a Map.
33search.js:40: hand-rolled debounce util. setTimeout+clearTimeout inline is enough.
34```
35 
36End with a one-line verdict: `N findings. Est. <X> lines removable.`