From 93c9def0eff0c2d9055b87ddc7dbca1d8b7f536e Mon Sep 17 00:00:00 2001 From: Ethan Hurst Date: Fri, 27 Feb 2026 18:58:42 +0000 Subject: [PATCH] fix: load nyquist_validation from config (#740) loadConfig() didn't include nyquist_validation in its return object, so cmdInitPlanPhase always set nyquist_validation_enabled to undefined. The plan-phase workflow could never detect whether Nyquist validation was enabled or disabled via config. Co-authored-by: Ethan Hurst --- get-shit-done/bin/lib/core.cjs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/get-shit-done/bin/lib/core.cjs b/get-shit-done/bin/lib/core.cjs index daf673ca5..6ef6ccb2a 100644 --- a/get-shit-done/bin/lib/core.cjs +++ b/get-shit-done/bin/lib/core.cjs @@ -76,6 +76,7 @@ function loadConfig(cwd) { research: true, plan_checker: true, verifier: true, + nyquist_validation: false, parallelization: true, brave_search: false, }; @@ -109,6 +110,7 @@ function loadConfig(cwd) { research: get('research', { section: 'workflow', field: 'research' }) ?? defaults.research, plan_checker: get('plan_checker', { section: 'workflow', field: 'plan_check' }) ?? defaults.plan_checker, verifier: get('verifier', { section: 'workflow', field: 'verifier' }) ?? defaults.verifier, + nyquist_validation: get('nyquist_validation', { section: 'workflow', field: 'nyquist_validation' }) ?? defaults.nyquist_validation, parallelization, brave_search: get('brave_search') ?? defaults.brave_search, model_overrides: parsed.model_overrides || null,