ha-mcp-dev 7.12.3.dev1914__tar.gz → 7.12.3.dev1917__tar.gz

{ha_mcp_dev-7.12.3.dev1914/src/ha_mcp_dev.egg-info → ha_mcp_dev-7.12.3.dev1917}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ha-mcp-dev
-Version: 7.12.3.dev1914
+Version: 7.12.3.dev1917
 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.12.3.dev1914 → ha_mcp_dev-7.12.3.dev1917}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ha-mcp-dev"
-version = "7.12.3.dev1914"
+version = "7.12.3.dev1917"
 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-7.12.3.dev1914 → ha_mcp_dev-7.12.3.dev1917}/src/ha_mcp/settings_ui/__init__.py

@@ -1982,12 +1982,39 @@ def build_settings_handlers(
                 ),
                 status_code=400,
             )
-        raw_flags = body.get("flags", {})
+        # Require the nested ``{"flags": {...}}`` shape. A flat body
+        # (e.g. ``{"enable_lite_docstrings": true}``) has no ``flags``
+        # key, so a lenient ``body.get("flags", {})`` default silently
+        # dropped every field and still returned
+        # ``success=True``/``restart_required=True`` — worse than a
+        # visible no-op, because the caller believed the flag changed
+        # and a restart was pending when nothing happened (#1840).
+        raw_flags = body.get("flags")
         if not isinstance(raw_flags, dict):
             return JSONResponse(
                 create_error_response(
                     ErrorCode.VALIDATION_INVALID_PARAMETER,
-                    "'flags' must be an object mapping field names to values",
+                    "Request body must contain a 'flags' object mapping "
+                    'field names to values, e.g. {"flags": '
+                    '{"enable_lite_docstrings": true}}. A flat body such '
+                    'as {"enable_lite_docstrings": true} is not accepted.',
+                    suggestions=[
+                        'Wrap the flags in a "flags" object, e.g. '
+                        '{"flags": {"enable_lite_docstrings": true}}.',
+                    ],
+                ),
+                status_code=400,
+            )
+        if not raw_flags:
+            return JSONResponse(
+                create_error_response(
+                    ErrorCode.VALIDATION_INVALID_PARAMETER,
+                    "'flags' object is empty; include at least one "
+                    "feature-flag field to update.",
+                    suggestions=[
+                        'Include at least one field inside "flags", e.g. '
+                        '{"flags": {"enable_lite_docstrings": true}}.',
+                    ],
                 ),
                 status_code=400,
             )

{ha_mcp_dev-7.12.3.dev1914 → ha_mcp_dev-7.12.3.dev1917/src/ha_mcp_dev.egg-info}/PKG-INFO

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