parol6: fix _ToolsCollection membership + by_type for StrEnum ToolType

waldoctl's ToolType is now a StrEnum (category members are also str) and the
ToolsSpec.by_type ABC widened to `str | ToolType`:

- __contains__: test ToolType before the plain-str branch — otherwise
  `ToolType.GRIPPER in tools` misrouted to the by-key lookup (keyed by tool
  key, not category) and returned False. Safe under plain Enum and StrEnum.
- by_type: widen the parameter to `str | ToolType` to match the widened ABC
  (LSP-safe against both the old and new base signatures).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: Jepson2k

parol6/robot.py

@@ -404,13 +404,16 @@ def __getitem__(self, key: str) -> ToolSpec:
         return self._by_key[key]
 
     def __contains__(self, item: object) -> bool:
-        if isinstance(item, str):
-            return item in self._by_key
+        # ToolType is a StrEnum, so test it before the plain-str branch —
+        # otherwise a category like ToolType.GRIPPER would misroute to the
+        # by-key lookup (keyed by tool key, not category) and return False.
         if isinstance(item, ToolType):
             return any(t.tool_type == item for t in self._tools)
+        if isinstance(item, str):
+            return item in self._by_key
         return False
 
-    def by_type(self, tool_type: ToolType) -> tuple[ToolSpec, ...]:
+    def by_type(self, tool_type: str | ToolType) -> tuple[ToolSpec, ...]:
         return tuple(t for t in self._tools if t.tool_type == tool_type)