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 <ethan.hurst@outlook.com.au>
This commit is contained in:
@@ -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,
|
||||
|
||||
Reference in New Issue
Block a user