thermo-nuclear-code-quality-review

agentic-workflows-blueprint.workflow.thermo-nuclear-code-quality-review

Perform an unusually strict review focused on implementation quality, maintainability, abstraction quality, and codebase health.

Applies to: Any code change or pull request diff across the codebase.
Does not cover: Direct code modification or executing fixes (use thermo-fix).

Be ambitious about structural simplification: Search for "code judo" moves that make implementation simpler, smaller, and cleaner.
1000 line limit: Do not let a PR push a file past 1k lines without strong justification.
No random spaghetti growth: Reject ad-hoc conditionals or one-off branches inserted into unrelated flows.
Clean design over working code: Do not approve merely because tests pass if the structure is messier.
No silent fallbacks: Ensure boundary cleanliness and explicit error/state contracts.

Identify all changed files in diffScope.
Evaluate against the Non-Negotiable Additional Standards:
- Structural simplification & Code Judo opportunities.
- Decomposition of files approaching 1k lines.
- Elimination of ad-hoc conditionals or leaky abstractions.
- Type contract and boundary cleanliness.
Produce a Severity-Ranked Finding Table:
- Blocker: Structural regressions, 1k+ line file growth, severe spaghetti.
- High: Major maintainability or boundary issues.
- Medium: Moderate cleanup or abstraction improvements.
- Low: Informational nits.
Output specific, actionable remedies for every Blocker and High finding.

Severity-ranked finding table appended to walkthrough.md or conversation output.
Final verdict: APPROVED or FINDINGS REMAIN.