feat: fast enum cache updates

python/composio/client/collections.py

@@ -1081,9 +1081,9 @@ class Actions(Collection[ActionModel]):
     # TODO: Overload
     def get(  # type: ignore
         self,
-        actions: t.Optional[t.Sequence[ActionType]] = None,
-        apps: t.Optional[t.Sequence[AppType]] = None,
-        tags: t.Optional[t.Sequence[TagType]] = None,
+        actions: t.Optional[t.Collection[ActionType]] = None,
+        apps: t.Optional[t.Collection[AppType]] = None,
+        tags: t.Optional[t.Collection[TagType]] = None,
         limit: t.Optional[int] = None,
         use_case: t.Optional[str] = None,
         allow_all: bool = False,
@@ -1392,6 +1392,14 @@ def search_for_a_task(
             for task in response.json().get("items", [])
         ]
 
+    def list_enums(self) -> list[str]:
+        response = self._raise_if_required(
+            response=self.client.http.get(
+                str(self.endpoint / "list" / "enums"),
+            )
+        )
+        return response.text.split("\n")
+
 
 class ExpectedFieldInput(BaseModel):
     name: str

python/composio/client/enums/action.py

@@ -1,7 +1,7 @@
 import typing as t
 import warnings
 
-from composio.client.enums.base import ActionData, replacement_action_name
+from composio.client.enums.base import ActionData, create_action
 from composio.client.enums.enum import Enum, EnumGenerator
 from composio.exceptions import ComposioSDKError
 
@@ -72,24 +72,7 @@ def fetch_and_cache(self) -> t.Optional[ActionData]:
         if "appName" not in response:
             return None
 
-        replaced_by = replacement_action_name(
-            response["description"], response["appName"]
-        )
-        return ActionData(  # type: ignore
-            name=response["name"],
-            app=response["appName"],
-            tags=response["tags"],
-            no_auth=(
-                client.http.get(url=str(client.apps.endpoint / response["appName"]))
-                .json()
-                .get("no_auth", False)
-            ),
-            is_local=False,
-            is_runtime=False,
-            shell=False,
-            path=self.storage_path,
-            replaced_by=replaced_by,
-        )
+        return create_action(client, response, self.storage_path)
 
     @property
     def name(self) -> str:

python/composio/client/enums/base.py

@@ -4,12 +4,16 @@
 
 import difflib
 import typing as t
+from pathlib import Path
 
 from composio.constants import LOCAL_CACHE_DIRECTORY
 from composio.exceptions import ComposioSDKError
 from composio.storage.base import LocalStorage
 
 
+if t.TYPE_CHECKING:
+    from composio.client import Composio
+
 _runtime_actions: t.Dict[str, "ActionData"] = {}
 
 EntityType = t.TypeVar("EntityType", bound=LocalStorage)
@@ -118,3 +122,28 @@ def replacement_action_name(description: str, app_name: str) -> t.Optional[str]:
         return (app_name + "_" + newact.replace(">>", "")).upper()
 
     return None
+
+
+def create_action(
+    client: "Composio",
+    response: dict[str, t.Any],
+    storage_path: Path,
+) -> ActionData:
+    replaced_by = replacement_action_name(response["description"], response["appName"])
+    return ActionData(  # type: ignore
+        name=response["name"],
+        app=response["appName"],
+        tags=response["tags"],
+        # TODO: Make no_auth part of the action response itself
+        # Ticket: ENG-3280
+        no_auth=(
+            client.http.get(url=str(client.apps.endpoint / response["appName"]))
+            .json()
+            .get("no_auth", False)
+        ),
+        is_local=False,
+        is_runtime=False,
+        shell=False,
+        path=storage_path,
+        replaced_by=replaced_by,
+    )

python/composio/client/enums/enum.py

@@ -90,24 +90,13 @@ def __eq__(self, other: object) -> bool:
         return False
 
     @classmethod
-    def iter(cls) -> t.Iterator[str]:
+    def iter(cls) -> t.Iterable[str]:
         """Yield the enum names as strings."""
-        # TODO: fetch trigger names from dedicated endpoint in the future
-        path = LOCAL_CACHE_DIRECTORY / cls.cache_folder
-        # If we try to fetch Actions.iter() with local caching disabled
-        # for example, we'd get here.
-        if not path.exists():
-            # pylint: disable=import-outside-toplevel
-            from composio.client import Composio
-
-            # pylint: disable=import-outside-toplevel
-            from composio.client.utils import check_cache_refresh
-
-            check_cache_refresh(Composio.get_latest())
-            if not path.exists():
-                return
-
-        yield from os.listdir(path)
+        # pylint: disable=import-outside-toplevel
+        from composio.client import Composio
+
+        client = Composio.get_latest()
+        return client.actions.list_enums()
 
     @classmethod
     def all(cls) -> t.Iterator[te.Self]:

python/composio/client/utils.py

@@ -1,10 +1,11 @@
 import json
 import os
+import time
 import typing as t
 
 from composio.client import Composio, enums
 from composio.client.collections import ActionModel, AppModel, TriggerModel
-from composio.client.enums.base import replacement_action_name
+from composio.client.enums.base import create_action, replacement_action_name
 from composio.tools.local import load_local_tools
 from composio.utils import get_enum_key
 from composio.utils.logging import get_logger
@@ -208,6 +209,9 @@ def _update_triggers(
             ).store()
 
 
+_cache_checked = False
+
+
 def check_cache_refresh(client: Composio) -> None:
     """
     Check if the actions have a 'replaced_by' field and refresh the cache if not.
@@ -218,18 +222,52 @@ def check_cache_refresh(client: Composio) -> None:
     SDK version, and didn't come from the API. We need to start storing the data
     from the API and invalidate the cache if the data is not already stored.
     """
+    global _cache_checked
+    if _cache_checked:
+        return
+
+    _cache_checked = True
+
+    t0 = time.monotonic()
     if NO_CACHE_REFRESH:
         return
 
+    local_actions = []
     if enums.base.ACTIONS_CACHE.exists():
-        first_file = next(enums.base.ACTIONS_CACHE.iterdir(), None)
-        if first_file is not None:
-            first_action = json.loads(first_file.read_text())
-            if "replaced_by" in first_action:
-                logger.debug("Actions cache is up-to-date")
-                return
-
-    logger.info("Actions cache is outdated, refreshing cache...")
-    apps = update_apps(client)
-    update_actions(client, apps)
-    update_triggers(client, apps)
+        actions = list(enums.base.ACTIONS_CACHE.iterdir())
+        for action in actions:
+            action_data = json.loads(action.read_text())
+            # The action file could be old. If it doesn't have a
+            # replaced_by field, we want to overwrite it.
+            if "replaced_by" not in action_data:
+                action.unlink()
+                continue
+
+            local_actions.append(action.stem)
+
+    api_actions = client.actions.list_enums()
+
+    actions_to_update = set(api_actions) - set(local_actions)
+    actions_to_delete = set(local_actions) - set(api_actions)
+    logger.debug("Actions to fetch: %s", actions_to_update)
+    logger.debug("Stale actions: %s", actions_to_delete)
+
+    for action_name in actions_to_delete:
+        (enums.base.ACTIONS_CACHE / action_name).unlink()
+
+    if actions_to_update:
+        # TODO: handle page size limit
+        actions_data = client.http.get(
+            str(
+                client.actions.endpoint(
+                    queries={"actions": ",".join(actions_to_update)}
+                )
+            )
+        ).json()
+        for action_data in actions_data["items"]:
+            storage_path = enums.base.ACTIONS_CACHE / action_data["name"]
+            create_action(
+                client, response=action_data, storage_path=storage_path
+            ).store()
+
+    logger.debug("Time taken to update cache: %.2f seconds", time.monotonic() - t0)