Split planning.mjs by effect domain #166

New issue

Closed

opened 2026-08-15 14:33:58 +00:00 by john · 0 comments

john commented

2026-08-15 14:33:58 +00:00

Owner

Source: deep dive, broader design B4; general scan recommendation 9
Verdict: DISPUTE - preference, closing

Claim as reviewed

planning.mjs owns state, UAT, audits, overlaps, leases, detection, tracing, renumbering, capture and milestone pruning. Module size increases coupling and makes security properties harder to audit; split into domain modules behind the current CLI facade.

At HEAD `f354864` (v3.3.0)

The size is accurate - cadence-core/bin/planning.mjs is 3,705 lines at HEAD.

Note

No defect in this batch is caused by the file's size, and none of the fixes needs the split to land. A 3,700-line move touching every seam is new review surface bought with no failing behaviour to point at.

Closed as a preference. Reopen with a specific defect the layout caused, and it becomes plannable.

**Source:** deep dive, broader design B4; general scan recommendation 9 **Verdict:** DISPUTE - preference, closing ### Claim as reviewed `planning.mjs` owns state, UAT, audits, overlaps, leases, detection, tracing, renumbering, capture and milestone pruning. Module size increases coupling and makes security properties harder to audit; split into domain modules behind the current CLI facade. ### At HEAD f354864 (v3.3.0) The size is accurate - `cadence-core/bin/planning.mjs` is 3,705 lines at HEAD. ### Note No defect in this batch is caused by the file's size, and none of the fixes needs the split to land. A 3,700-line move touching every seam is new review surface bought with no failing behaviour to point at. Closed as a preference. Reopen with a specific defect the layout caused, and it becomes plannable.