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

test(#2152): sandbox HOME in readGsdEffectiveModelOverrides subtest

Browse Source 
The subtest asserted project-only model_overrides but called
readGsdEffectiveModelOverrides without isolating HOME, so the real
~/.gsd/defaults.json global overrides bled into the deepEqual on any dev box or
non-hermetic runner that has one. Pass a sandboxed homedir (mirroring the sibling
warnIfStaleBake subtests that already inject homedir).
This commit is contained in:
Tom Boucher
2026-07-12 15:49:16 -04:00
parent e3231717d0
commit 5bce5c46a6
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/stale-bake-guard.test.cjs
View File

@@ -468,16 +468,21 @@ describe('stale-bake-guard parity with bin/install.js bake paths', () => {
test('readGsdEffectiveModelOverrides resolves codex + opencode overrides from .planning/config.json', () => {
const tmp = fs.mkdtempSync(path.join(os.tmpdir(), 'gsd-parity-'));
// #2152: sandbox HOME so the real ~/.gsd/defaults.json cannot bleed its global
// model_overrides into this project-only assertion (hermeticity). Mirrors the
// sibling subtests that pass a homedir option to warnIfStaleBake.
const sandboxHome = fs.mkdtempSync(path.join(os.tmpdir(), 'gsd-parity-home-'));
try {
fs.mkdirSync(path.join(tmp, '.planning'), { recursive: true });
fs.writeFileSync(
path.join(tmp, '.planning', 'config.json'),
JSON.stringify({ model_overrides: { 'gsd-executor': 'opencode-go/flash', 'gsd-planner': 'openai/gpt-5' } }),
);
const resolved = install.readGsdEffectiveModelOverrides(tmp);
const resolved = install.readGsdEffectiveModelOverrides(tmp, { homedir: () => sandboxHome });
assert.deepEqual(resolved, { 'gsd-executor': 'opencode-go/flash', 'gsd-planner': 'openai/gpt-5' });
} finally {
cleanup(tmp);
cleanup(sandboxHome);
}
});
});