Optimize tool call reactor handler ordering

src/core/services/tool_call_reactor_service.py

@@ -39,6 +39,7 @@ def __init__(self, history_tracker: IToolCallHistoryTracker | None = None) -> No
             history_tracker: Optional history tracker for tracking tool calls.
         """
         self._handlers: dict[str, IToolCallHandler] = {}
+        self._sorted_handlers: tuple[IToolCallHandler, ...] = ()
         self._history_tracker = history_tracker
         self._lock = asyncio.Lock()
 
@@ -61,6 +62,7 @@ def register_handler_sync(self, handler: IToolCallHandler) -> None:
             )
 
         self._handlers[handler.name] = handler
+        self._refresh_sorted_handlers()
         logger.info(f"Registered tool call handler synchronously: {handler.name}")
 
     async def register_handler(self, handler: IToolCallHandler) -> None:
@@ -79,6 +81,7 @@ async def register_handler(self, handler: IToolCallHandler) -> None:
                 )
 
             self._handlers[handler.name] = handler
+            self._refresh_sorted_handlers()
             logger.info(f"Registered tool call handler: {handler.name}")
 
     async def unregister_handler(self, handler_name: str) -> None:
@@ -97,6 +100,7 @@ async def unregister_handler(self, handler_name: str) -> None:
                 )
 
             del self._handlers[handler_name]
+            self._refresh_sorted_handlers()
             logger.info(f"Unregistered tool call handler: {handler_name}")
 
     async def process_tool_call(
@@ -138,12 +142,8 @@ async def process_tool_call(
                 history_context,
             )
 
-        # Get handlers sorted by priority (highest first)
-        handlers = sorted(
-            self._handlers.values(),
-            key=lambda h: h.priority,
-            reverse=True,
-        )
+        # Handlers are maintained in priority order (highest first)
+        handlers = self._sorted_handlers
 
         # Process through handlers
         for handler in handlers:
@@ -183,6 +183,17 @@ def get_registered_handlers(self) -> list[str]:
         """
         return list(self._handlers.keys())
 
+    def _refresh_sorted_handlers(self) -> None:
+        """Recompute cached handler ordering by priority."""
+
+        self._sorted_handlers = tuple(
+            sorted(
+                self._handlers.values(),
+                key=lambda handler: handler.priority,
+                reverse=True,
+            )
+        )
+
 
 class InMemoryToolCallHistoryTracker(IToolCallHistoryTracker):
     """In-memory implementation of tool call history tracking."""