@ddtcorex/dsh-maestro-supervisor 0.6.7 → 0.6.8

package/lib/debug-agent.js

@@ -268,6 +268,7 @@ async function defaultFetchLLM(prompt) {
             body: JSON.stringify({
                 model: cfg.model,
                 temperature: 0.2,
+                ...(cfg.reasoningEffort ? { reasoning_effort: cfg.reasoningEffort, reasoningEffort: cfg.reasoningEffort } : {}),
                 messages: [
                     { role: 'system', content: 'You are a systematic-debugging agent for DSH Web resilience. Follow the 4 phases: root cause, pattern analysis, hypothesis, implementation. Always propose minimal single-file fix.' },
                     { role: 'user', content: prompt },
@@ -312,7 +313,7 @@ async function defaultFetchLLM(prompt) {
                 { role: 'user', content: prompt },
             ],
             max_output_tokens: 4000,
-            reasoning: { effort: 'low' },
+            ...(cfg.reasoningEffort ? { reasoning: { effort: cfg.reasoningEffort } } : { reasoning: { effort: 'low' } }),
         }),
     });
     if (!res2.ok)
@@ -339,7 +340,9 @@ async function resolveLLMConfig() {
     // Model: AI_MODEL / DEEPSEEK_MODEL / settings.json domains.supervisor.model -> domains.review.model / default
     // Supervisor has its own picker; falls back to review model for backward compat, then DSH default.
     let model = process.env.AI_MODEL ?? process.env.DEEPSEEK_MODEL ?? process.env.OPENAI_MODEL ?? null;
-    if (!model) {
+    // reasoningEffort: env first, then supervisor -> review, then settings.yaml fallback
+    let reasoningEffort = (process.env.AI_REASONING_EFFORT ?? process.env.REASONING_EFFORT ?? null) ?? undefined;
+    if (!model || !reasoningEffort) {
         try {
             const { readFileSync } = await import('node:fs');
             const { homedir } = await import('node:os');
@@ -347,24 +350,57 @@ async function resolveLLMConfig() {
             const raw = readFileSync(settingsPath, 'utf-8');
             const j = JSON.parse(raw);
             // Prefer supervisor model, fall back to review model (so old installs keep working)
-            const sup = j?.domains?.supervisor?.model?.model ?? j?.domains?.supervisor?.model;
-            const rev = j?.domains?.review?.model?.model ?? j?.domains?.review?.model;
-            let m = null;
-            if (typeof sup === 'string')
-                m = sup;
-            else if (sup?.model && typeof sup.model === 'string')
-                m = sup.model;
-            else if (typeof rev === 'string')
-                m = rev;
-            else if (rev?.model && typeof rev.model === 'string')
-                m = rev.model;
-            if (typeof m === 'string' && m.trim() !== '')
-                model = m;
+            if (!model) {
+                const sup = j?.domains?.supervisor?.model?.model ?? j?.domains?.supervisor?.model;
+                const rev = j?.domains?.review?.model?.model ?? j?.domains?.review?.model;
+                let m = null;
+                if (typeof sup === 'string')
+                    m = sup;
+                else if (sup?.model && typeof sup.model === 'string')
+                    m = sup.model;
+                else if (typeof rev === 'string')
+                    m = rev;
+                else if (rev?.model && typeof rev.model === 'string')
+                    m = rev.model;
+                if (typeof m === 'string' && m.trim() !== '')
+                    model = m;
+            }
+            if (!reasoningEffort) {
+                // config-lib validator: model is {provider, model, reasoningEffort?} inside domains.supervisor.model / domains.review.model
+                // Support both nested object and flat legacy fallback for future compat
+                const supObj = j?.domains?.supervisor?.model;
+                const revObj = j?.domains?.review?.model;
+                const supEff = typeof supObj === 'object' && supObj !== null ? supObj.reasoningEffort : undefined;
+                const supFlat = j?.domains?.supervisor?.reasoningEffort;
+                const revEff = typeof revObj === 'object' && revObj !== null ? revObj.reasoningEffort : undefined;
+                const revFlat = j?.domains?.review?.reasoningEffort;
+                const candidate = supEff ?? supFlat ?? revEff ?? revFlat;
+                if (typeof candidate === 'string' && candidate.trim() !== '')
+                    reasoningEffort = candidate.trim();
+            }
         }
         catch { }
     }
     if (!model)
         model = 'deepseek-chat';
+    // Also try settings.yaml for reasoningEffort if still missing (e.g., llm-pi-ai default reasoning)
+    if (!reasoningEffort) {
+        try {
+            const { readFileSync } = await import('node:fs');
+            const { homedir } = await import('node:os');
+            const yamlPath = `${homedir()}/.dsh/settings.yaml`;
+            const yaml = readFileSync(yamlPath, 'utf-8');
+            // Look for reasoningEffort near the model id in provider blocks (future-proof; currently not in yaml)
+            const re = new RegExp(`id:\\s*${model}[\\s\\S]{0,200}reasoningEffort:\\s*(\\S+)`, 'm');
+            const m = yaml.match(re);
+            if (m) {
+                const v = m[1].trim().replace(/^["']|["']$/g, '');
+                if (v)
+                    reasoningEffort = v;
+            }
+        }
+        catch { }
+    }
     // If url not set via env, try to resolve from ~/.dsh/settings.yaml llm-pi-ai providers (DeepSeek suggested setup)
     if (!url) {
         try {
@@ -432,7 +468,7 @@ async function resolveLLMConfig() {
     if (!url.includes('/v1/') && !url.includes('/chat/completions')) {
         finalUrl = url.replace(/\/$/, '') + '/v1/chat/completions';
     }
-    return { key, url: finalUrl, model };
+    return { key, url: finalUrl, model, ...(reasoningEffort ? { reasoningEffort } : {}) };
 }
 async function resolveApiKey() {
     const cfg = await resolveLLMConfig();

package/lib/health-poller.js

@@ -50,6 +50,10 @@ function isRecentlyStarted(opts, wallMs) {
         return true;
     return false;
 }
+// Specific parse/boot-failure markers only. Bare 'JSON'/'YAML' were removed
+// (2026-08-31): they matched any line whose payload merely *contained* those
+// substrings — e.g. maestro-sync's status JSON listing session.jsonl.zstd /
+// settings.json paths — turning a healthy 401 into a rollback + restart.
 const ERROR_PATTERNS = [
     'ERR_MODULE_NOT_FOUND',
     'ERR_PNPM',
@@ -58,8 +62,6 @@ const ERROR_PATTERNS = [
     'SyntaxError',
     'YAMLParseError',
     'ParseError',
-    'YAML',
-    'JSON',
     'corrupted',
     'allowBuilds',
     'Cannot find module',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ddtcorex/dsh-maestro-supervisor",
-  "version": "0.6.7",
+  "version": "0.6.8",
   "description": "Supervisor daemon for DSH Web resilience — auto-detect crashes, rollback to LKG, report",
   "type": "module",
   "main": "./lib/index.js",