cleanups

esphome/config_validation.py

@@ -354,8 +354,7 @@ def sub_device_id(value: str | None) -> core.ID:
     # Lazy import to avoid circular imports
     from esphome.core.config import Device
 
-    validator = use_id(Device)
-    return validator(value)
+    return use_id(Device)(value)
 
 
 def boolean(value):

esphome/dashboard/util/text.py

@@ -3,7 +3,7 @@ from __future__ import annotations
 from esphome.helpers import slugify
 
 
-def friendly_name_slugify(value):
+def friendly_name_slugify(value: str) -> str:
     """Convert a friendly name to a slug with dashes instead of underscores."""
     # First use the standard slugify, then convert underscores to dashes
     return slugify(value).replace("_", "-")

tests/unit_tests/fixtures/core/config/area_id_hash_collision.yaml

@@ -1 +1,10 @@
+esphome:
+  name: test
+  areas:
+    - id: test_2258
+      name: "Area 1"
+    - id: d6ka
+      name: "Area 2"
 
+esp32:
+  board: esp32dev

tests/unit_tests/fixtures/core/config/device_duplicate_id.yaml

@@ -1 +1,10 @@
+esphome:
+  name: test
+  devices:
+    - id: duplicate_device
+      name: "Device 1"
+    - id: duplicate_device
+      name: "Device 2"
 
+esp32:
+  board: esp32dev

tests/unit_tests/fixtures/core/config/device_id_collision.yaml

@@ -7,4 +7,4 @@ esphome:
       name: "Device 2"
 
 esp32:
-  board: esp32dev
+  board: esp32dev

tests/unit_tests/fixtures/core/config/device_invalid_area.yaml

@@ -1 +1,12 @@
+esphome:
+  name: test
+  areas:
+    - id: valid_area
+      name: "Valid Area"
+  devices:
+    - id: test_device
+      name: "Test Device"
+      area_id: nonexistent_area
 
+esp32:
+  board: esp32dev