ha-mcp-dev 7.6.0.dev629__tar.gz → 7.6.0.dev631__tar.gz

{ha_mcp_dev-7.6.0.dev629/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.6.0.dev631}/PKG-INFO

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

{ha_mcp_dev-7.6.0.dev629 → ha_mcp_dev-7.6.0.dev631}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.6.0.dev629"
+version = "7.6.0.dev631"
 description = "Home Assistant MCP Server - Complete control of Home Assistant through MCP"
 readme = "README.md"
 requires-python = ">=3.13,<3.14"

{ha_mcp_dev-7.6.0.dev629 → ha_mcp_dev-7.6.0.dev631}/src/ha_mcp/settings_ui.py

@@ -602,6 +602,10 @@ _SETTINGS_HTML = (
 <meta charset="UTF-8">
 <meta name="viewport" content="width=device-width, initial-scale=1.0">
 <title>HA-MCP Tool Settings</title>
+<!-- Empty data URI: tells the browser "no favicon" so it never requests
+     /favicon.ico (which would 404 and log a console error, since the
+     settings server serves no such asset in any deployment mode). -->
+<link rel="icon" href="data:,">
 <style>
   :root {
     --bg: #1c1c1e; --surface: #2c2c2e; --surface-hover: #3a3a3c;
@@ -2548,9 +2552,36 @@ function renderCodeModeSubRows(parentEl, masterOn, codeModeOn) {
 
     const info = document.createElement('div');
     info.className = 'feature-info';
-    const lockedNote = f.origin === 'env'
-      ? `<div class="feature-locked-note">Set via env var <code>${escapeHtml(f.env_var)}</code> — unset it to edit here.</div>`
-      : '';
+    // code_mode_saved_tools_path needs honest, field-specific copy:
+    //  - add-on mode: hardcoded by start.py (setdefault to /data); not
+    //    Supervisor-managed and absent from the addon schema, so it
+    //    genuinely can't be changed — don't imply a lever exists.
+    //  - standalone with the env var set: the "unset it" hint IS
+    //    actionable (the operator controls the env var), so keep it.
+    //  - standalone with no path: a blank path disables persistence —
+    //    warn that saved tools live in memory only.
+    // Other env-locked code-mode sub-fields keep the shared helper.
+    let lockedNote = '';
+    if (f.field === 'code_mode_saved_tools_path') {
+      if (IS_ADDON_MODE) {
+        lockedNote =
+          '<div class="feature-locked-note">Hardcoded to ' +
+          '<code>/data/saved_tools.json</code> in add-on mode and cannot ' +
+          'be changed (fixed here so saved tools survive add-on updates).' +
+          '</div>';
+      } else if (f.origin === 'env') {
+        lockedNote =
+          `<div class="feature-locked-note">${envLockedNoteHtml(f.env_var, f.field)}</div>`;
+      } else if (!f.value) {
+        lockedNote =
+          '<div class="feature-locked-note">If blank, custom tools are kept ' +
+          'in memory only and lost on restart. Set a path on persistent ' +
+          'storage to keep them.</div>';
+      }
+    } else if (f.origin === 'env') {
+      lockedNote =
+        `<div class="feature-locked-note">${envLockedNoteHtml(f.env_var, f.field)}</div>`;
+    }
     info.innerHTML =
       `<div class="feature-name">${escapeHtml(meta.label)}</div>` +
       `<div class="feature-help">${escapeHtml(meta.help)}</div>` +
@@ -5373,16 +5404,34 @@ def build_settings_handlers(
             if is_addon_synced:
                 addon_writes_present = True
             elif os.environ.get(env_name) is not None:
+                # Add-on mode has no env-var surface for non-schema keys
+                # (e.g. CODE_MODE_SAVED_TOOLS_PATH, set by start.py), so
+                # "unset it to edit here" is unactionable there — give
+                # add-on-aware copy instead of implying a lever exists.
+                if addon_mode:
+                    message = (
+                        f"{fname!r} is fixed by the add-on runtime and "
+                        "cannot be changed from the web UI."
+                    )
+                    suggestions = [
+                        "This value is baked into the add-on and is not "
+                        "exposed as an editable setting.",
+                    ]
+                else:
+                    message = (
+                        f"{fname!r} is set via {env_name} env var — "
+                        "unset it to edit here."
+                    )
+                    suggestions = [
+                        f"Unset the {env_name} environment variable (or "
+                        "remove it from your Docker config), then restart "
+                        "to edit this setting from the UI.",
+                    ]
                 return JSONResponse(
                     create_error_response(
                         ErrorCode.VALIDATION_INVALID_PARAMETER,
-                        f"{fname!r} is set via {env_name} env var — "
-                        "unset it to edit here.",
-                        suggestions=[
-                            f"Unset the {env_name} environment variable (or "
-                            "remove it from your addon/Docker config), then "
-                            "restart to edit this setting from the UI.",
-                        ],
+                        message,
+                        suggestions=suggestions,
                         context={"env_var": env_name},
                     ),
                     status_code=409,

{ha_mcp_dev-7.6.0.dev629 → ha_mcp_dev-7.6.0.dev631}/src/ha_mcp/tools/tools_code.py

@@ -1290,5 +1290,16 @@ def register_code_tools(mcp: Any, client: Any, **kwargs: Any) -> None:
                     "in-memory cache. Check operator logs for the "
                     "underlying I/O error."
                 )
+            elif not settings.code_mode_saved_tools_path:
+                # Blank path = persistence disabled: the in-memory save
+                # succeeded but is lost on restart. Surface a warning so
+                # the agent doesn't assume the entry is durable — mirrors
+                # the write-failure branch above.
+                response["data"]["save_warning"] = (
+                    f"save_as={save_as!r} is kept in memory only — "
+                    "code_mode_saved_tools_path is unset, so custom tools "
+                    "are not persisted and are lost on restart. Set a path "
+                    "on persistent storage to keep them."
+                )
 
         return response

{ha_mcp_dev-7.6.0.dev629 → ha_mcp_dev-7.6.0.dev631/src/ha_mcp_dev.egg-info}/PKG-INFO

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