mirror of
https://github.com/EveryInc/compound-engineering-plugin.git
synced 2026-09-19 01:09:55 +08:00
Trevin Chow
dc5f80a838
The quality tail chose between a branch-wide diff and the fix-owned files. That choice was never worth making: when the branch holds only the fix the branch diff equals the fix scope, so branch-wide buys nothing, and when it does not the branch diff is exactly the WIP-swallowing case the condition was added to guard. A conditional whose safe arm is redundant and whose other arm is the hazard should not exist. Simplify now always receives the fix-owned files that were clean before Phase 3 as an explicit scope. `ce-simplify-code` treats a named scope as authoritative and will not widen it, so passing one is the guardrail rather than a preference, and it matters most for that pass because it rewrites what it is handed. Review keeps a single narrower distinction on its own merits: it reads rather than writes, and `base:<pre-fix-HEAD>` is a better input than a file list when the pre-fix tree was provably clean. Claude-Session: https://claude.ai/code/session_015eproCfjCz2WZdynWFwGiT