Collapsed the two-method split (build + _build_for_component) into a single method, eliminating the intermediate dict allocation and set() wrapping.

Author: toby-coleman

plugboard/connector/event_connector_spec_builder.py

@@ -23,23 +23,11 @@ def build(components: _t.Iterable[Component]) -> dict[str, ConnectorSpec]:
         """Returns mapping of connector specs for events handled by components."""
         evt_conn_map: dict[str, ConnectorSpec] = {}
         for component in components:
-            comp_evt_conn_map = EventConnectorSpecBuilder._build_for_component(
-                evt_conn_map, component
-            )
-            evt_conn_map.update(comp_evt_conn_map)
+            for evt in component.io.input_events + component.io.output_events:
+                if evt.type not in evt_conn_map:
+                    evt_conn_map[evt.type] = EventConnectorSpecBuilder._build_for_event(evt.type)
         return evt_conn_map
 
-    @staticmethod
-    def _build_for_component(
-        evt_conn_map: dict[str, ConnectorSpec], component: Component
-    ) -> dict[str, ConnectorSpec]:
-        component_evts = set(component.io.input_events + component.io.output_events)
-        return {
-            evt.type: EventConnectorSpecBuilder._build_for_event(evt.type)
-            for evt in component_evts
-            if evt.type not in evt_conn_map
-        }
-
     @staticmethod
     def _build_for_event(evt_type: str) -> ConnectorSpec:
         evt_type_safe = Event.safe_type(evt_type)