ha-mcp-dev 8.3.0.dev2457__tar.gz → 8.3.0.dev2461__tar.gz

{ha_mcp_dev-8.3.0.dev2457/src/ha_mcp_dev.egg-info → ha_mcp_dev-8.3.0.dev2461}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 8.3.0.dev2457
+Version: 8.3.0.dev2461
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT

{ha_mcp_dev-8.3.0.dev2457 → ha_mcp_dev-8.3.0.dev2461}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "8.3.0.dev2457"
+version = "8.3.0.dev2461"
 description = "Home Assistant MCP Server - Complete control of Home Assistant through MCP"
 readme = "README.md"
 # Upper bound widened to <3.15 so the package installs on Home Assistant's

{ha_mcp_dev-8.3.0.dev2457 → ha_mcp_dev-8.3.0.dev2461}/src/ha_mcp/settings_ui/settings.js

@@ -489,7 +489,11 @@ async function loadTools() {
   updateStatus(t('status.loaded', {}, 'Loaded'));
 }
 
-async function applyInfoChrome() {
+// Restart-chrome half of applyInfoChrome: which restart control is shown
+// and what the restart notice tells the user to do. Split out (with the
+// footer below) so applyInfoChrome stays orchestration — the branch ladder
+// here alone sits near the complexity limit.
+function _applyRestartChrome(info) {
   // Show restart button if running as add-on; show Stop Sidecar
   // button only when this page is served by the stdio sidecar
   // (HTTP modes serve the same HTML but is_sidecar=false there, so
@@ -499,84 +503,91 @@ async function applyInfoChrome() {
   // reopen Claude Desktop" vs "click Restart Add-on" vs "restart
   // your Docker container") instead of a generic "restart the add-on"
   // that only matches one of three real deployment surfaces.
-  try {
-    const infoResp = await fetch('./api/settings/info');
-    const info = await infoResp.json();
-    const noticeEl = document.getElementById('restartNoticeText');
-    if (info.is_addon) {
-      document.getElementById('restartBtn').style.display = '';
-      if (noticeEl) {
-        noticeEl.textContent = t(
-          'notice.restart.addon',
-          {},
-          '⚠ Changes saved. Click "Restart App (add-on)" for them to take effect. Then refresh the MCP tool list in your AI client.'
-        );
-      }
-    } else if (info.deployment_mode === 'embedded') {
-      // In-process (custom component) server: the restart endpoint reloads
-      // the server config entry, which reinstalls-if-newer and swaps the
-      // worker onto the freshly installed code.
-      const rbtn = document.getElementById('restartBtn');
-      rbtn.style.display = '';
-      restartTargetEmbedded = true;
-      rbtn.textContent = t('actions.restart_server', {}, 'Restart HA-MCP Server');
-      if (noticeEl) {
-        noticeEl.textContent = t(
-          'notice.restart.embedded',
-          {},
-          '⚠ Changes saved. Click "Restart HA-MCP Server" for them to take effect. Then refresh the MCP tool list in your AI client.'
-        );
-      }
-    } else if (info.is_sidecar) {
-      if (noticeEl) {
-        noticeEl.textContent = t(
-          'notice.restart.sidecar',
-          {},
-          '⚠ Changes saved. Fully quit and reopen your MCP client for them to take effect.'
-        );
-      }
-      document.getElementById('sidecarStopRow').style.display = '';
-    } else if (noticeEl) {
-      // HTTP / Docker / standalone — no button we can wire to a restart,
-      // so describe the action in process terms, then the client-refresh
-      // step (remote connectors cache the tool list, same as add-on mode).
+  const noticeEl = document.getElementById('restartNoticeText');
+  if (info.is_addon) {
+    document.getElementById('restartBtn').style.display = '';
+    if (noticeEl) {
       noticeEl.textContent = t(
-        'notice.restart.standalone',
+        'notice.restart.addon',
         {},
-        '⚠ Changes saved. Restart the ha-mcp process, then refresh the MCP tool list in your AI client.'
+        '⚠ Changes saved. Click "Restart App (add-on)" for them to take effect. Then refresh the MCP tool list in your AI client.'
       );
     }
-    // Footer — show the running build and the same deployment classification
-    // used by ha_report_issue. The backend keeps embedded and sidecar distinct
-    // because they need different restart behavior; preserve those concise
-    // names here and clarify the less obvious packaging-oriented values.
-    if (info.version) {
-      const fEl = document.getElementById('versionFooterText');
-      const deploymentMode = typeof info.deployment_mode === 'string'
-        ? info.deployment_mode
-        : '';
-      const deploymentLabels = {
-        embedded: 'embedded',
-        sidecar: 'sidecar',
-        addon: t('footer.deployment.addon', {}, 'app/add-on'),
-        docker: t('footer.deployment.docker', {}, 'container/docker'),
-        pyinstaller: t('footer.deployment.pyinstaller', {}, 'standalone binary'),
-        git: t('footer.deployment.git', {}, 'source checkout'),
-        pypi: t('footer.deployment.pypi', {}, 'python package'),
-        unknown: t('footer.deployment.unknown', {}, 'unknown'),
-      };
-      const deploymentLabel = Object.prototype.hasOwnProperty.call(
-        deploymentLabels, deploymentMode
-      ) ? deploymentLabels[deploymentMode] : deploymentMode;
-      const installation = deploymentLabel
-        ? ' · ' + t(
-          'footer.installation',
-          {method: deploymentLabel},
-          'installation: {method}'
-        )
-        : '';
-      if (fEl) fEl.textContent = 'ha-mcp ' + info.version + installation;
+  } else if (info.deployment_mode === 'embedded') {
+    // In-process (custom component) server: the restart endpoint reloads
+    // the server config entry, which reinstalls-if-newer and swaps the
+    // worker onto the freshly installed code.
+    const rbtn = document.getElementById('restartBtn');
+    rbtn.style.display = '';
+    restartTargetEmbedded = true;
+    rbtn.textContent = t('actions.restart_server', {}, 'Restart HA-MCP Server');
+    if (noticeEl) {
+      noticeEl.textContent = t(
+        'notice.restart.embedded',
+        {},
+        '⚠ Changes saved. Click "Restart HA-MCP Server" for them to take effect. Then refresh the MCP tool list in your AI client.'
+      );
+    }
+  } else if (info.is_sidecar) {
+    if (noticeEl) {
+      noticeEl.textContent = t(
+        'notice.restart.sidecar',
+        {},
+        '⚠ Changes saved. Fully quit and reopen your MCP client for them to take effect.'
+      );
     }
+    document.getElementById('sidecarStopRow').style.display = '';
+  } else if (noticeEl) {
+    // HTTP / Docker / standalone — no button we can wire to a restart,
+    // so describe the action in process terms, then the client-refresh
+    // step (remote connectors cache the tool list, same as add-on mode).
+    noticeEl.textContent = t(
+      'notice.restart.standalone',
+      {},
+      '⚠ Changes saved. Restart the ha-mcp process, then refresh the MCP tool list in your AI client.'
+    );
+  }
+}
+
+// Footer — show the running build and the same deployment classification
+// used by ha_report_issue. The backend keeps embedded and sidecar distinct
+// because they need different restart behavior; preserve those concise
+// names here and clarify the less obvious packaging-oriented values.
+function _applyVersionFooter(info) {
+  if (!info.version) return;
+  const fEl = document.getElementById('versionFooterText');
+  const deploymentMode = typeof info.deployment_mode === 'string'
+    ? info.deployment_mode
+    : '';
+  const deploymentLabels = {
+    embedded: 'embedded',
+    sidecar: 'sidecar',
+    addon: t('footer.deployment.addon', {}, 'app/add-on'),
+    docker: t('footer.deployment.docker', {}, 'container/docker'),
+    pyinstaller: t('footer.deployment.pyinstaller', {}, 'standalone binary'),
+    git: t('footer.deployment.git', {}, 'source checkout'),
+    pypi: t('footer.deployment.pypi', {}, 'python package'),
+    unknown: t('footer.deployment.unknown', {}, 'unknown'),
+  };
+  const deploymentLabel = Object.prototype.hasOwnProperty.call(
+    deploymentLabels, deploymentMode
+  ) ? deploymentLabels[deploymentMode] : deploymentMode;
+  const installation = deploymentLabel
+    ? ' · ' + t(
+      'footer.installation',
+      {method: deploymentLabel},
+      'installation: {method}'
+    )
+    : '';
+  if (fEl) fEl.textContent = 'ha-mcp ' + info.version + installation;
+}
+
+async function applyInfoChrome() {
+  try {
+    const infoResp = await fetch('./api/settings/info');
+    const info = await infoResp.json();
+    _applyRestartChrome(info);
+    _applyVersionFooter(info);
   } catch (e) {
     // A transient /api/settings/info failure must not leave the restart
     // button hidden / the restart notice unset silently — log it so the
@@ -3627,14 +3638,28 @@ async function policyDecide(token, action) {
 // have landed with only its response lost — so re-read first and revert only
 // when the readback actually shows the old value.
 //
-// `spec` is {readState, repaint, revertKey, revertText}: readState() returns
-// {value, known} for this toggle's slice of policyState/readOnlyState, and
-// repaint() is the toggle's own painter.
+// `spec` is {readState, revertKey, revertText}: readState() returns
+// {value, known} for this toggle's slice of policyState/readOnlyState.
+//
+// Repaints are NOT per-toggle. The re-read below is loadPolicyState(),
+// which refreshes — or, through _clearFlagSwitchState(), clears — the
+// state slices of ALL THREE switches. Repainting only the toggle being
+// saved left the other two painted from state that no longer existed:
+// a failed re-read after a Read Only Mode save cleared both policy
+// slices while their switches stayed editable at the stale value with
+// #policyUnknownNotice hidden, and vice versa for #roUnknownNotice.
+function _repaintFlagSwitches() {
+  paintPolicyGlobalToggles();
+  syncReadOnlyToggle();
+  render();
+}
+
 async function handleFailedFlagSave(checkbox, previous, saved, spec) {
   if (saved === false) {
+    // No re-read happened, so the other switches' state is untouched and
+    // repainting them is a no-op — one shape for both branches.
     checkbox.checked = previous;
-    spec.repaint();
-    render();
+    _repaintFlagSwitches();
     updateStatus(t(spec.revertKey, {}, spec.revertText), false, true);
     return;
   }
@@ -3661,8 +3686,7 @@ async function handleFailedFlagSave(checkbox, previous, saved, spec) {
       'reload to see what the server has.');
   }
   // Paint before the status line so the repaint can't clobber it.
-  spec.repaint();
-  render();
+  _repaintFlagSwitches();
   updateStatus(msg, ok, !ok);
 }
 
@@ -3677,7 +3701,6 @@ document.getElementById('policy-master-toggle').addEventListener('change', async
   if (!saved) {
     await handleFailedFlagSave(e.target, previous, saved, {
       readState: () => ({value: policyState.enabled, known: policyState.enabledKnown}),
-      repaint: paintPolicyGlobalToggles,
       revertKey: 'policies.errors.master_save',
       revertText:
         'Tool Security Policies change did not save. The server still has the previous value',
@@ -3713,7 +3736,6 @@ document.getElementById('policy-manage-tool-toggle').addEventListener('change',
   if (!saved) {
     await handleFailedFlagSave(e.target, previous, saved, {
       readState: () => ({value: policyState.manageToolEnabled, known: policyState.manageToolKnown}),
-      repaint: paintPolicyGlobalToggles,
       revertKey: 'policies.errors.manage_tool_save',
       revertText:
         'Policy-editing tool change did not save. The server still has the previous value',
@@ -3744,7 +3766,6 @@ document.getElementById('read-only-mode-toggle').addEventListener('change', asyn
   if (!saved) {
     await handleFailedFlagSave(e.target, previous, saved, {
       readState: () => ({value: readOnlyState.enabled, known: readOnlyState.enabledKnown}),
-      repaint: syncReadOnlyToggle,
       revertKey: 'tools.read_only.save_failed',
       revertText:
         'Read Only Mode change did not save. The server still has the previous value',

{ha_mcp_dev-8.3.0.dev2457 → ha_mcp_dev-8.3.0.dev2461/src/ha_mcp_dev.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 8.3.0.dev2457
+Version: 8.3.0.dev2461
 Summary: Home Assistant MCP Server - Complete control of Home Assistant through MCP
 Author-email: Julien <github@qc-h.net>
 License: MIT