From 7ac99b89a0ddf4f8f0d6f0d08b4fac66b7e2c1a2 Mon Sep 17 00:00:00 2001
From: CREDO23 <bakerathierry@gmail.com>
Date: Wed, 27 May 2026 13:29:30 +0200
Subject: [PATCH] refactor(automation): drop Capability registry

---
 .../app/automations/registries/__init__.py    | 14 +----------
 .../automations/registries/actions/types.py   |  2 +-
 .../registries/capabilities/__init__.py       | 14 -----------
 .../registries/capabilities/store.py          | 23 -------------------
 .../registries/capabilities/types.py          | 18 ---------------
 .../automations/registries/triggers/types.py  |  2 +-
 6 files changed, 3 insertions(+), 70 deletions(-)
 delete mode 100644 surfsense_backend/app/automations/registries/capabilities/__init__.py
 delete mode 100644 surfsense_backend/app/automations/registries/capabilities/store.py
 delete mode 100644 surfsense_backend/app/automations/registries/capabilities/types.py

diff --git a/surfsense_backend/app/automations/registries/__init__.py b/surfsense_backend/app/automations/registries/__init__.py
index a97595ced..f497caf59 100644
--- a/surfsense_backend/app/automations/registries/__init__.py
+++ b/surfsense_backend/app/automations/registries/__init__.py
@@ -1,4 +1,4 @@
-"""Capability, action, and trigger registries — populated at process startup."""
+"""Action and trigger registries — populated at process startup."""
 
 from __future__ import annotations
 
@@ -9,13 +9,6 @@ from .actions import (
     get_action,
     register_action,
 )
-from .capabilities import (
-    Capability,
-    CapabilityHandler,
-    all_capabilities,
-    get_capability,
-    register_capability,
-)
 from .triggers import (
     TriggerDefinition,
     all_triggers,
@@ -26,16 +19,11 @@ from .triggers import (
 __all__ = [
     "ActionDefinition",
     "ActionHandler",
-    "Capability",
-    "CapabilityHandler",
     "TriggerDefinition",
     "all_actions",
-    "all_capabilities",
     "all_triggers",
     "get_action",
-    "get_capability",
     "get_trigger",
     "register_action",
-    "register_capability",
     "register_trigger",
 ]
diff --git a/surfsense_backend/app/automations/registries/actions/types.py b/surfsense_backend/app/automations/registries/actions/types.py
index 13c826c66..99f94ae7c 100644
--- a/surfsense_backend/app/automations/registries/actions/types.py
+++ b/surfsense_backend/app/automations/registries/actions/types.py
@@ -14,5 +14,5 @@ class ActionDefinition:
     type: str
     name: str
     description: str
-    config_schema: dict[str, Any]
+    params_schema: dict[str, Any]
     handler: ActionHandler
diff --git a/surfsense_backend/app/automations/registries/capabilities/__init__.py b/surfsense_backend/app/automations/registries/capabilities/__init__.py
deleted file mode 100644
index 213303fc0..000000000
--- a/surfsense_backend/app/automations/registries/capabilities/__init__.py
+++ /dev/null
@@ -1,14 +0,0 @@
-"""Capability registry."""
-
-from __future__ import annotations
-
-from .store import all_capabilities, get_capability, register_capability
-from .types import Capability, CapabilityHandler
-
-__all__ = [
-    "Capability",
-    "CapabilityHandler",
-    "all_capabilities",
-    "get_capability",
-    "register_capability",
-]
diff --git a/surfsense_backend/app/automations/registries/capabilities/store.py b/surfsense_backend/app/automations/registries/capabilities/store.py
deleted file mode 100644
index 4d87abe47..000000000
--- a/surfsense_backend/app/automations/registries/capabilities/store.py
+++ /dev/null
@@ -1,23 +0,0 @@
-"""In-memory capability registry. Populated once at process startup."""
-
-from __future__ import annotations
-
-from .types import Capability
-
-_REGISTRY: dict[str, Capability] = {}
-
-
-def register_capability(capability: Capability) -> None:
-    """Register a capability. Raises on duplicate id."""
-    if capability.id in _REGISTRY:
-        raise ValueError(f"Capability already registered: {capability.id!r}")
-    _REGISTRY[capability.id] = capability
-
-
-def get_capability(capability_id: str) -> Capability | None:
-    return _REGISTRY.get(capability_id)
-
-
-def all_capabilities() -> dict[str, Capability]:
-    """Defensive snapshot of the registry."""
-    return dict(_REGISTRY)
diff --git a/surfsense_backend/app/automations/registries/capabilities/types.py b/surfsense_backend/app/automations/registries/capabilities/types.py
deleted file mode 100644
index 2759bc809..000000000
--- a/surfsense_backend/app/automations/registries/capabilities/types.py
+++ /dev/null
@@ -1,18 +0,0 @@
-"""``Capability`` dataclass and handler signature. Locked at five fields for v1."""
-
-from __future__ import annotations
-
-from collections.abc import Awaitable, Callable
-from dataclasses import dataclass
-from typing import Any
-
-CapabilityHandler = Callable[[dict[str, Any]], Awaitable[Any]]
-
-
-@dataclass(frozen=True, slots=True)
-class Capability:
-    id: str
-    description: str
-    input_schema: dict[str, Any]
-    output_schema: dict[str, Any]
-    handler: CapabilityHandler
diff --git a/surfsense_backend/app/automations/registries/triggers/types.py b/surfsense_backend/app/automations/registries/triggers/types.py
index 5da081343..783bd7842 100644
--- a/surfsense_backend/app/automations/registries/triggers/types.py
+++ b/surfsense_backend/app/automations/registries/triggers/types.py
@@ -10,5 +10,5 @@ from typing import Any
 class TriggerDefinition:
     type: str
     description: str
-    config_schema: dict[str, Any]
+    params_schema: dict[str, Any]
     payload_schema: dict[str, Any]