gsd-pi 2.65.0-dev.5c8557b → 2.65.0-dev.d0517ff

package/dist/resources/extensions/gsd/preferences-validation.js

@@ -542,6 +542,14 @@ export function validatePreferences(preferences) {
                 errors.push("parallel.auto_merge must be one of: auto, confirm, manual");
             }
         }
+        if (p.worker_model !== undefined) {
+            if (typeof p.worker_model === "string" && p.worker_model.length > 0) {
+                parallel.worker_model = p.worker_model;
+            }
+            else {
+                errors.push("parallel.worker_model must be a non-empty string");
+            }
+        }
         if (Object.keys(parallel).length > 0) {
             validated.parallel = parallel;
         }
@@ -574,7 +582,15 @@ export function validatePreferences(preferences) {
                     errors.push('reactive_execution.isolation_mode must be "same-tree"');
                 }
             }
-            const knownReKeys = new Set(["enabled", "max_parallel", "isolation_mode"]);
+            if (re.subagent_model !== undefined) {
+                if (typeof re.subagent_model === "string" && re.subagent_model.length > 0) {
+                    validRe.subagent_model = re.subagent_model;
+                }
+                else {
+                    errors.push("reactive_execution.subagent_model must be a non-empty string");
+                }
+            }
+            const knownReKeys = new Set(["enabled", "max_parallel", "isolation_mode", "subagent_model"]);
             for (const key of Object.keys(re)) {
                 if (!knownReKeys.has(key)) {
                     warnings.push(`unknown reactive_execution key "${key}" — ignored`);
@@ -972,5 +988,33 @@ export function validatePreferences(preferences) {
             errors.push("enhanced_verification_strict must be a boolean");
         }
     }
+    // ─── Discuss Preparation ────────────────────────────────────────────
+    if (preferences.discuss_preparation !== undefined) {
+        if (typeof preferences.discuss_preparation === "boolean") {
+            validated.discuss_preparation = preferences.discuss_preparation;
+        }
+        else {
+            errors.push("discuss_preparation must be a boolean");
+        }
+    }
+    // ─── Discuss Web Research ───────────────────────────────────────────
+    if (preferences.discuss_web_research !== undefined) {
+        if (typeof preferences.discuss_web_research === "boolean") {
+            validated.discuss_web_research = preferences.discuss_web_research;
+        }
+        else {
+            errors.push("discuss_web_research must be a boolean");
+        }
+    }
+    // ─── Discuss Depth ──────────────────────────────────────────────────
+    if (preferences.discuss_depth !== undefined) {
+        const validDepths = new Set(["quick", "standard", "thorough"]);
+        if (typeof preferences.discuss_depth === "string" && validDepths.has(preferences.discuss_depth)) {
+            validated.discuss_depth = preferences.discuss_depth;
+        }
+        else {
+            errors.push(`discuss_depth must be one of: quick, standard, thorough`);
+        }
+    }
     return { preferences: validated, errors, warnings };
 }

package/dist/resources/extensions/gsd/preferences.js

@@ -127,9 +127,10 @@ function loadPreferencesFile(path, scope) {
     };
 }
 let _warnedUnrecognizedFormat = false;
-/** @internal Reset the warn-once flag — exported for testing only. */
+/** @internal Reset the warn-once flags — exported for testing only. */
 export function _resetParseWarningFlag() {
     _warnedUnrecognizedFormat = false;
+    _warnedFrontmatterParse = false;
 }
 /** @internal Exported for testing only */
 export function parsePreferencesMarkdown(content) {
@@ -156,6 +157,7 @@ export function parsePreferencesMarkdown(content) {
     }
     return null;
 }
+let _warnedFrontmatterParse = false;
 function parseFrontmatterBlock(frontmatter) {
     try {
         const parsed = parseYaml(frontmatter);
@@ -165,7 +167,11 @@ function parseFrontmatterBlock(frontmatter) {
         return parsed;
     }
     catch (e) {
-        logWarning("guided", `YAML parse error in frontmatter block: ${e.message}`);
+        // Warn at most once per session to avoid flooding TUI (#3376)
+        if (!_warnedFrontmatterParse) {
+            _warnedFrontmatterParse = true;
+            logWarning("guided", `YAML parse error in preferences frontmatter (suppressing further): ${e.message}`);
+        }
         return {};
     }
 }
@@ -449,5 +455,6 @@ export function resolveParallelConfig(prefs) {
         budget_ceiling: prefs?.parallel?.budget_ceiling,
         merge_strategy: prefs?.parallel?.merge_strategy ?? "per-milestone",
         auto_merge: prefs?.parallel?.auto_merge ?? "confirm",
+        worker_model: prefs?.parallel?.worker_model,
     };
 }