golem15/msd-core
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(#3149): bind planningPaths once in cmdStateLoad

Browse Source 
The debug_dir change introduced a second planningPaths(cwd) call in the
same function. Bind the struct once and read both .planning and .debug
from it; emitted output is byte-identical.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
sim
2026-08-07 09:33:47 -04:00
parent 2bead6ca1d
commit 654b2cc10c
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/state.cts
View File

@@ -335,7 +335,8 @@ const STOP_H2_ONLY = (lv: number): boolean => lv === 2;
function cmdStateLoad(cwd: string, raw: boolean): void { function cmdStateLoad(cwd: string, raw: boolean): void {
const config = loadConfig(cwd); const config = loadConfig(cwd);
const planDir = planningPaths(cwd).planning; const paths = planningPaths(cwd);
const planDir = paths.planning;
const stateRaw = platformReadSync(path.join(planDir, 'STATE.md')) || ''; const stateRaw = platformReadSync(path.join(planDir, 'STATE.md')) || '';
@@ -356,7 +357,7 @@ function cmdStateLoad(cwd: string, raw: boolean): void {
// bundle regardless: it is a shipped query surface with its own test anchor // bundle regardless: it is a shipped query surface with its own test anchor
// (tests/state.test.cjs), so narrowing it would break unseen consumers for // (tests/state.test.cjs), so narrowing it would break unseen consumers for
// no gain (Hyrum's Law). Both emit the SAME `planningPaths(cwd).debug`. // no gain (Hyrum's Law). Both emit the SAME `planningPaths(cwd).debug`.
debug_dir: toPosixPath(planningPaths(cwd).debug), debug_dir: toPosixPath(paths.debug),
}; };
// For --raw, output a condensed key=value format // For --raw, output a condensed key=value format