@ddtcorex/dsh-maestro-config 0.3.0 → 0.3.1

package/lib/client.js

@@ -3304,10 +3304,6 @@ function MaestroSettingsTab({ rpcCall, configRpcCall }) {
     call(MAESTRO_ENDPOINTS.getConfig, {}).then((saved) => setConfig((prev) => ({ ...prev, ...saved }))).catch(() => {
     });
     if (configRpcCall) {
-      configRpcCall("get", { domain: "supervisor" }).then((res) => {
-        if (res?.ok && res.value?.model) setConfig((prev) => ({ ...prev, supervisorModel: res.value.model }));
-      }).catch(() => {
-      });
       Promise.all([cfgGet("guard").catch(() => ({})), cfgGet("guardBlacklist").catch(() => ({ patterns: [], placeholders: {} })), cfgGet("supervisor").catch(() => ({})), cfgGet("notifier").catch(() => ({}))]).then(([g, bl, sup, not]) => {
         setGuard(g ?? {});
         const pats = Array.isArray(bl?.patterns) ? bl.patterns : [];
@@ -3416,13 +3412,6 @@ function MaestroSettingsTab({ rpcCall, configRpcCall }) {
   const saveField = async (field, value) => {
     setError(null);
     setConfig((prev) => ({ ...prev, [field]: value }));
-    if (field === "supervisorModel" && configRpcCall) {
-      try {
-        const res = await configRpcCall("set", { domain: "supervisor", patch: { model: value } });
-        if (res?.ok) return;
-      } catch {
-      }
-    }
     try {
       await call(MAESTRO_ENDPOINTS.saveConfig, { [field]: value });
     } catch (err) {
@@ -3474,7 +3463,6 @@ function MaestroSettingsTab({ rpcCall, configRpcCall }) {
       { style: { display: "flex", flexDirection: "column" } },
       (0, import_react.createElement)(ToggleRow, { title: "Re-review on push", description: "When new commits are pushed, trigger an automatic re-review.", checked: config.autoRereviewOnPush === true, onChange: (v) => saveField("autoRereviewOnPush", v) }),
       (0, import_react.createElement)(SettingRow, { title: "Global review model", description: "Model for automated reviews. Empty = DSH default.", control: (0, import_react.createElement)(ReviewModelSelector, { value: config.reviewModel ?? null, catalog, fallbackValue: catalog?.current ?? null, fallbackLabel: "Use DSH default", onChange: (v) => saveField("reviewModel", v), label: null }) }),
-      (0, import_react.createElement)(SettingRow, { title: "Supervisor model", description: "Model for auto-fixing DSH Web crashes.", control: (0, import_react.createElement)(ReviewModelSelector, { value: config.supervisorModel ?? null, catalog, fallbackValue: catalog?.current ?? null, fallbackLabel: "Use DSH default", onChange: (v) => saveField("supervisorModel", v), label: null }) }),
       (0, import_react.createElement)(ProjectMappingsEditor, { mappings: config.projectMappings ?? [], onChange: (mappings) => saveField("projectMappings", mappings), catalog, globalReviewModel: config.reviewModel ?? null })
     ),
     guard: (0, import_react.createElement)(

package/lib/types/client/MaestroSettings.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"MaestroSettings.d.ts","sourceRoot":"","sources":["../../../src/client/MaestroSettings.tsx"],"names":[],"mappings":"AACA;;;;;;;;GAQG;AAisCH,wBAAgB,kBAAkB,CAAC,EAAE,OAAO,EAAE,aAAa,EAAE,EAAE;IAAE,OAAO,EAAE,GAAG,CAAC;IAAC,aAAa,CAAC,EAAE,GAAG,CAAA;CAAE;;;;;;;;;mBAkZmE,GAAG;;gBAoBzK"}
+{"version":3,"file":"MaestroSettings.d.ts","sourceRoot":"","sources":["../../../src/client/MaestroSettings.tsx"],"names":[],"mappings":"AACA;;;;;;;;GAQG;AAisCH,wBAAgB,kBAAkB,CAAC,EAAE,OAAO,EAAE,aAAa,EAAE,EAAE;IAAE,OAAO,EAAE,GAAG,CAAC;IAAC,aAAa,CAAC,EAAE,GAAG,CAAA;CAAE;;;;;;;;;mBAsYmE,GAAG;;gBAoBzK"}

package/lib/types/client/maestro-card.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"maestro-card.d.ts","sourceRoot":"","sources":["../../../src/client/maestro-card.jsx"],"names":[],"mappings":"AAweA;;;;;;;;gBA0dC"}
+{"version":3,"file":"maestro-card.d.ts","sourceRoot":"","sources":["../../../src/client/maestro-card.jsx"],"names":[],"mappings":"AAweA;;;;;;;;gBA2bC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ddtcorex/dsh-maestro-config",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "private": false,
   "description": "Maestro Config — shared settings service for the dsh-maestro-* suite over the single namespaced store (~/.dsh/maestro/settings.json)",
   "type": "module",

package/src/client/MaestroSettings.tsx

@@ -1400,11 +1400,6 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }: { rpcCall: any; c
       .then((saved) => setConfig((prev: any) => ({ ...prev, ...saved })))
       .catch(() => {})
     if (configRpcCall) {
-      configRpcCall('get', { domain: 'supervisor' })
-        .then((res: any) => {
-          if (res?.ok && res.value?.model) setConfig((prev: any) => ({ ...prev, supervisorModel: res.value.model }))
-        })
-        .catch(() => {})
       Promise.all([cfgGet('guard').catch(() => ({})), cfgGet('guardBlacklist').catch(() => ({ patterns: [], placeholders: {} })), cfgGet('supervisor').catch(() => ({})), cfgGet('notifier').catch(() => ({}))])
         .then(([g, bl, sup, not]) => {
           setGuard(g ?? {})
@@ -1516,12 +1511,6 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }: { rpcCall: any; c
   const saveField = async (field: string, value: unknown) => {
     setError(null)
     setConfig((prev: any) => ({ ...prev, [field]: value }))
-    if (field === 'supervisorModel' && configRpcCall) {
-      try {
-        const res = await configRpcCall('set', { domain: 'supervisor', patch: { model: value } })
-        if (res?.ok) return
-      } catch {}
-    }
     try {
       await call(MAESTRO_ENDPOINTS.saveConfig, { [field]: value })
     } catch (err: any) {
@@ -1574,7 +1563,6 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }: { rpcCall: any; c
         { style: { display: 'flex', flexDirection: 'column' } },
         h(ToggleRow as any, { title: 'Re-review on push', description: 'When new commits are pushed, trigger an automatic re-review.', checked: config.autoRereviewOnPush === true, onChange: (v: boolean) => saveField('autoRereviewOnPush', v) }),
         h(SettingRow as any, { title: 'Global review model', description: 'Model for automated reviews. Empty = DSH default.', control: h(ReviewModelSelector as any, { value: config.reviewModel ?? null, catalog, fallbackValue: catalog?.current ?? null, fallbackLabel: 'Use DSH default', onChange: (v: any) => saveField('reviewModel', v), label: null }) }),
-        h(SettingRow as any, { title: 'Supervisor model', description: 'Model for auto-fixing DSH Web crashes.', control: h(ReviewModelSelector as any, { value: config.supervisorModel ?? null, catalog, fallbackValue: catalog?.current ?? null, fallbackLabel: 'Use DSH default', onChange: (v: any) => saveField('supervisorModel', v), label: null }) }),
         h(ProjectMappingsEditor as any, { mappings: config.projectMappings ?? [], onChange: (mappings: any) => saveField('projectMappings', mappings), catalog, globalReviewModel: config.reviewModel ?? null }),
       ),
     guard: h(

package/src/client/maestro-card.jsx

@@ -578,15 +578,7 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }) {
     call(MAESTRO_ENDPOINTS.getConfig, {})
       .then(saved => setConfig(prev => ({ ...prev, ...saved })))
       .catch(() => { /* first run, no config saved yet — keep defaults */ })
-    // Supervisor model may be stored via generic config service when review not installed — try fallback
     if (configRpcCall) {
-      configRpcCall('get', { domain: 'supervisor' })
-        .then(res => {
-          if (res?.ok && res.value?.model) {
-            setConfig(prev => ({ ...prev, supervisorModel: res.value.model }))
-          }
-        })
-        .catch(() => { /* supervisor domain not yet set or config service unavailable */ })
       // Task 3: load Guard/Blacklist/Supervisor/Notifier domains
       Promise.all([
         cfgGet('guard').catch(() => ({})),
@@ -688,16 +680,6 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }) {
   const saveField = async (field, value) => {
     setError(null)
     setConfig(prev => ({ ...prev, [field]: value }))
-    // Supervisor model can be saved via generic config service when review not installed (independent install)
-    if (field === 'supervisorModel' && configRpcCall) {
-      try {
-        const res = await configRpcCall('set', { domain: 'supervisor', patch: { model: value } })
-        if (res?.ok) return
-        // Fall through to review RPC if generic set fails
-      } catch (e) {
-        // Fall through to review RPC
-      }
-    }
     try {
       await call(MAESTRO_ENDPOINTS.saveConfig, { [field]: value })
     } catch (err) {
@@ -817,19 +799,6 @@ export function MaestroSettingsTab({ rpcCall, configRpcCall }) {
       }),
     ),
 
-    h('div', { style: sectionStyle },
-      h('h4', { style: headingStyle }, 'Supervisor LLM'),
-      h('p', { style: captionStyle }, 'Model used by the supervisor debug-agent to auto-fix DSH Web crashes. Empty = DSH default (or Review model if set). Uses the same provider catalog as Review.'),
-      h(ReviewModelSelector, {
-        value: config.supervisorModel ?? null,
-        catalog,
-        fallbackValue: catalog?.current ?? null,
-        fallbackLabel: 'Use DSH default',
-        onChange: v => saveField('supervisorModel', v),
-        label: 'Supervisor model',
-      }),
-    ),
-
     h('div', { style: sectionStyle },
       h('h4', { style: headingStyle }, 'Projects'),
       h(ProjectMappingsEditor, { mappings: config.projectMappings ?? [], onChange: mappings => saveField('projectMappings', mappings), catalog, globalReviewModel: config.reviewModel ?? null }),