Aktualisieren von CHANGELOG.md

Fixed Home Assistant service UI integration and improved service robustness in multi-host environments. This release is a bugfix release for v0.7.0.

CHANGELOG.md

@@ -1,4 +1,19 @@
-# Changelog
+# Changelog
+
+## 0.7.2
+- Fixed service validation for device targets:
+  - Home Assistant may pass `device_id` as a list (target/data wrapper)
+  - Services now accept both `str` and `list[str]` for `device_id`
+  - Improved device target parsing for UI and script wrappers
+
+## 0.7.1
+- Fixed Home Assistant service UI integration:
+  - Services now properly expose the **Device selector** in the visual automation editor
+  - Implemented correct handling of `call.target.device_id`
+  - Services are now fully compatible with Home Assistant’s target system
+- Fixed issue where only a YAML data field was shown instead of a device selection field
+- Improved service robustness when used in multi-host environments
+- This release is a bugfix release for v0.7.0
 
 ## 0.7.0
 - Added full service support for automations and scripts:
@@ -112,3 +127,4 @@
 - First experimental entities
 
 
+

custom_components/proxmox_pve/manifest.json

@@ -1,7 +1,7 @@
 {
   "domain": "proxmox_pve",
   "name": "Easy Proxmox (by René Bachmann)",
-  "version": "0.6.0-alpha",
+  "version": "0.7.1",
   "documentation": "https://example.invalid",
   "requirements": ["aiohttp>=3.9.0"],
   "codeowners": ["@BAHMCLOUD"],
@@ -9,4 +9,3 @@
   "iot_class": "local_polling",
   "integration_type": "service"
 }
-

custom_components/proxmox_pve/services.py

@@ -24,9 +24,11 @@ ATTR_TYPE = "type"
 
 VALID_TYPES = ("qemu", "lxc")
 
+# IMPORTANT:
+# HA may pass device_id via target (list) OR data (list) depending on UI/script wrapper.
 SERVICE_SCHEMA = vol.Schema(
     {
-        vol.Optional(ATTR_DEVICE_ID): str,
+        vol.Optional(ATTR_DEVICE_ID): vol.Any(str, [str]),
         vol.Optional(ATTR_CONFIG_ENTRY_ID): str,
         vol.Optional(ATTR_HOST): str,
         vol.Optional(ATTR_NODE): str,
@@ -36,6 +38,31 @@ SERVICE_SCHEMA = vol.Schema(
 )
 
 
+def _first_str(value: Any) -> str | None:
+    """Return first string from str or list[str], else None."""
+    if isinstance(value, str) and value.strip():
+        return value.strip()
+    if isinstance(value, list) and value:
+        v0 = value[0]
+        if isinstance(v0, str) and v0.strip():
+            return v0.strip()
+    return None
+
+
+def _get_target_device_id(call: ServiceCall) -> str | None:
+    """
+    Prefer call.target.device_id (UI target).
+    Fallback to call.data.device_id (some wrappers convert target to data).
+    """
+    if call.target and isinstance(call.target, dict):
+        dev_id = _first_str(call.target.get("device_id"))
+        if dev_id:
+            return dev_id
+
+    # fallback: data.device_id can be str or list[str]
+    return _first_str(call.data.get(ATTR_DEVICE_ID))
+
+
 def _parse_guest_identifier(identifier: str) -> Tuple[str, str, int]:
     """
     Guest device identifier format: "node:type:vmid"
@@ -52,8 +79,8 @@ def _parse_guest_identifier(identifier: str) -> Tuple[str, str, int]:
 
 
 def _resolve_target(hass: HomeAssistant, call: ServiceCall) -> Tuple[str, str, int]:
-    """Resolve node/type/vmid from device_id OR node+vmid (+ optional type)."""
+    """Resolve node/type/vmid from device target OR node+vmid (+ optional type)."""
-    device_id = call.data.get(ATTR_DEVICE_ID)
+    device_id = _get_target_device_id(call)
     node = call.data.get(ATTR_NODE)
     vmid = call.data.get(ATTR_VMID)
     vmtype = call.data.get(ATTR_TYPE, "qemu")
@@ -64,20 +91,17 @@ def _resolve_target(hass: HomeAssistant, call: ServiceCall) -> Tuple[str, str, i
         if not device:
             raise ValueError(f"Device not found: {device_id}")
 
-        # Find our guest identifier in device.identifiers
         for ident_domain, ident_value in device.identifiers:
             if ident_domain != DOMAIN:
                 continue
-            # Node devices are "node:<name>" — ignore those
             if ident_value.startswith("node:"):
                 continue
             return _parse_guest_identifier(ident_value)
 
         raise ValueError(f"Selected device has no Easy Proxmox guest identifier: {device_id}")
 
-    # manual mode
     if not node or vmid is None:
-        raise ValueError("Provide either device_id OR node + vmid (+ optional type/host/config_entry_id).")
+        raise ValueError("Provide a Device target OR node + vmid (+ optional type/host/config_entry_id).")
 
     if vmtype not in VALID_TYPES:
         raise ValueError(f"Invalid type: {vmtype} (allowed: {VALID_TYPES})")
@@ -93,7 +117,6 @@ def _get_domain_entries(hass: HomeAssistant) -> dict[str, Any]:
 
 
 def _pick_entry_id_for_device(hass: HomeAssistant, device_id: str) -> str:
-    """Pick correct config_entry_id by using device.config_entries."""
     dev_reg = dr.async_get(hass)
     device = dev_reg.async_get(device_id)
     if not device:
@@ -104,7 +127,6 @@ def _pick_entry_id_for_device(hass: HomeAssistant, device_id: str) -> str:
     if not candidates:
         raise ValueError("Device is not linked to any loaded Easy Proxmox config entry.")
     if len(candidates) > 1:
-        # Very unlikely, but handle it
         _LOGGER.warning("Device %s belongs to multiple Easy Proxmox entries, using first.", device_id)
     return candidates[0]
 
@@ -115,7 +137,6 @@ def _pick_entry_id_by_host(hass: HomeAssistant, host: str) -> str:
     for entry_id, data in domain_entries.items():
         if not isinstance(data, dict):
             continue
-        # host is stored in entry.data, but we keep it accessible here via "client.host" too
         client = data.get("client")
         if client and getattr(client, "host", None) == host:
             matches.append(entry_id)
@@ -128,10 +149,6 @@ def _pick_entry_id_by_host(hass: HomeAssistant, host: str) -> str:
 
 
 def _pick_entry_id_by_guest_lookup(hass: HomeAssistant, node: str, vmtype: str, vmid: int) -> str:
-    """
-    If user provides only node/vmid/type, try to find the correct entry by
-    scanning each entry's cluster resources list.
-    """
     domain_entries = _get_domain_entries(hass)
     matches = []
 
@@ -154,44 +171,38 @@ def _pick_entry_id_by_guest_lookup(hass: HomeAssistant, node: str, vmtype: str,
     if not matches:
         raise ValueError(
             f"Could not find guest {node}/{vmtype}/{vmid} in any configured Proxmox host. "
-            "Provide host or config_entry_id."
+            "Provide host or config_entry_id, or use device target."
         )
     if len(matches) > 1:
         raise ValueError(
             f"Guest {node}/{vmtype}/{vmid} exists on multiple configured hosts (ambiguous). "
-            "Please provide host or config_entry_id, or use device_id."
+            "Please provide host or config_entry_id, or use device target."
         )
     return matches[0]
 
 
 def _resolve_entry_id(hass: HomeAssistant, call: ServiceCall, target: Tuple[str, str, int]) -> str:
-    """Resolve which config entry should execute this service call."""
     domain_entries = _get_domain_entries(hass)
 
-    # 1) explicit config_entry_id
     config_entry_id = call.data.get(ATTR_CONFIG_ENTRY_ID)
     if config_entry_id:
         if config_entry_id not in domain_entries:
             raise ValueError(f"config_entry_id '{config_entry_id}' not found or not loaded.")
         return config_entry_id
 
-    # 2) by device_id (best + unambiguous)
+    device_id = _get_target_device_id(call)
-    device_id = call.data.get(ATTR_DEVICE_ID)
     if device_id:
         return _pick_entry_id_for_device(hass, device_id)
 
-    # 3) by host
     host = call.data.get(ATTR_HOST)
     if host:
         return _pick_entry_id_by_host(hass, host)
 
-    # 4) last resort: guest lookup in resources list
     node, vmtype, vmid = target
     return _pick_entry_id_by_guest_lookup(hass, node, vmtype, vmid)
 
 
 async def async_register_services(hass: HomeAssistant) -> None:
-    """Register services once per HA instance."""
     if hass.services.has_service(DOMAIN, SERVICE_START):
         return
 
@@ -205,7 +216,6 @@ async def async_register_services(hass: HomeAssistant) -> None:
             raise ValueError(f"Selected config entry '{entry_id}' has no client (not loaded).")
 
         client = entry_data["client"]
-
         _LOGGER.debug("Service action=%s entry=%s target=%s/%s/%s", action, entry_id, node, vmtype, vmid)
         await client.guest_action(node=node, vmid=vmid, vmtype=vmtype, action=action)
 
@@ -228,7 +238,6 @@ async def async_register_services(hass: HomeAssistant) -> None:
 
 
 async def async_unregister_services(hass: HomeAssistant) -> None:
-    """Unregister services (optional cleanup)."""
     for svc in (SERVICE_START, SERVICE_SHUTDOWN, SERVICE_STOP_HARD, SERVICE_REBOOT):
         if hass.services.has_service(DOMAIN, svc):
-            hass.services.async_remove(DOMAIN, svc)
+            hass.services.async_remove(DOMAIN, svc)

custom_components/proxmox_pve/services.yaml

@@ -1,99 +1,111 @@
 start:
   name: Start guest
   description: Start a VM or container.
+  target:
+    device:
+      integration: proxmox_pve
   fields:
-    device_id:
-      name: Device
-      description: Select the VM/CT device (recommended, works with multi-host).
-      example: "a1b2c3d4e5f6..."
     config_entry_id:
       name: Config entry id
       description: Optional. Force a specific Proxmox host configuration entry.
+      example: "8d9f2e7b1c3d4a5f..."
     host:
       name: Host
-      description: Optional. Proxmox host/IP of the configured entry (only used when device_id is not provided).
+      description: Optional. Proxmox host/IP of the configured entry (only used when device is not selected).
       example: "192.168.178.101"
     node:
       name: Node
-      description: Proxmox node name (only used when device_id is not provided).
+      description: Proxmox node name (only used when device is not selected).
       example: "pve1"
     vmid:
       name: VMID
-      description: Guest VMID (only used when device_id is not provided).
+      description: Guest VMID (only used when device is not selected).
       example: 100
     type:
       name: Type
-      description: Guest type.
+      description: Guest type (only used when device is not selected).
       example: "qemu"
 
 shutdown:
   name: Shutdown guest
-  description: Soft shutdown a VM or container.
+  description: Gracefully shutdown a VM or container.
+  target:
+    device:
+      integration: proxmox_pve
   fields:
-    device_id:
-      name: Device
-      description: Select the VM/CT device (recommended, works with multi-host).
     config_entry_id:
       name: Config entry id
       description: Optional. Force a specific Proxmox host configuration entry.
+      example: "8d9f2e7b1c3d4a5f..."
     host:
       name: Host
-      description: Optional. Proxmox host/IP of the configured entry (only used when device_id is not provided).
+      description: Optional. Proxmox host/IP of the configured entry (only used when device is not selected).
+      example: "192.168.178.101"
     node:
       name: Node
-      description: Proxmox node name (only used when device_id is not provided).
+      description: Proxmox node name (only used when device is not selected).
+      example: "pve1"
     vmid:
       name: VMID
-      description: Guest VMID (only used when device_id is not provided).
+      description: Guest VMID (only used when device is not selected).
+      example: 100
     type:
       name: Type
-      description: Guest type.
+      description: Guest type (only used when device is not selected).
       example: "qemu"
 
 stop_hard:
   name: Stop guest (hard)
   description: Hard stop a VM or container (equivalent to Stop).
+  target:
+    device:
+      integration: proxmox_pve
   fields:
-    device_id:
-      name: Device
-      description: Select the VM/CT device (recommended, works with multi-host).
     config_entry_id:
       name: Config entry id
       description: Optional. Force a specific Proxmox host configuration entry.
+      example: "8d9f2e7b1c3d4a5f..."
     host:
       name: Host
-      description: Optional. Proxmox host/IP of the configured entry (only used when device_id is not provided).
+      description: Optional. Proxmox host/IP of the configured entry (only used when device is not selected).
+      example: "192.168.178.101"
     node:
       name: Node
-      description: Proxmox node name (only used when device_id is not provided).
+      description: Proxmox node name (only used when device is not selected).
+      example: "pve1"
     vmid:
       name: VMID
-      description: Guest VMID (only used when device_id is not provided).
+      description: Guest VMID (only used when device is not selected).
+      example: 100
     type:
       name: Type
-      description: Guest type.
+      description: Guest type (only used when device is not selected).
       example: "qemu"
 
 reboot:
   name: Reboot guest
   description: Reboot a VM or container.
+  target:
+    device:
+      integration: proxmox_pve
   fields:
-    device_id:
-      name: Device
-      description: Select the VM/CT device (recommended, works with multi-host).
     config_entry_id:
       name: Config entry id
       description: Optional. Force a specific Proxmox host configuration entry.
+      example: "8d9f2e7b1c3d4a5f..."
     host:
       name: Host
-      description: Optional. Proxmox host/IP of the configured entry (only used when device_id is not provided).
+      description: Optional. Proxmox host/IP of the configured entry (only used when device is not selected).
+      example: "192.168.178.101"
     node:
       name: Node
-      description: Proxmox node name (only used when device_id is not provided).
+      description: Proxmox node name (only used when device is not selected).
+      example: "pve1"
     vmid:
       name: VMID
-      description: Guest VMID (only used when device_id is not provided).
+      description: Guest VMID (only used when device is not selected).
+      example: 100
     type:
       name: Type
-      description: Guest type.
+      description: Guest type (only used when device is not selected).
       example: "qemu"