archive228 Simplify
Over-engineering is the default failure mode. Cut it:
- Premature abstraction — a class/strategy/factory for one caller. Inline it. Abstract on the THIRD use, not the first.
- Speculative config — a parameter/flag/env var for something that never changes. Hardcode it until there's a real reason.
- Dead flexibility — an interface with one implementation, a generic with one type. Delete the indirection.
- Defensive noise — null checks on values that can't be null, try/except for errors that can't happen. Handle only real failure modes. Test: can you justify every changed line by a direct connection to what was asked? If a line is there "while I was in there", revert it.