now turn around and critique this plan. What are the gaps? unstated assumptions? brittle design elements? code smells? And anything else you don't like?
🚫 Anti-Patterns to Flag
- Gold-Plating: Building features or abstractions "just in case" they are needed in the future (YAGNI).
- Abstractions for One: Creating generic interfaces or wrappers for a single implementation without clear justification.
- The "Inner Platform" Effect: Re-implementing features that already exist in the language, framework, or OS.
- Locked File Modification: Any changes to files designated as protected in
.lockedfiles. - Vague Errors: "Catch-all" blocks without specific logging/recovery.
- Bloat: Violations of DRY (Don't Repeat Yourself) or SOLID principles—but also warning against "Over-DRYing" (creating complex abstractions to avoid three lines of repetition).