refactor: deduplicate _event_value across collector modules

Consolidate two copies of _event_value into the canonical
collector/intelligence/helpers.py definition. Scorer.py copy
left untouched (intentionally in SDK to avoid circular imports).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: acailic

collector/causal_analysis.py

@@ -6,13 +6,7 @@
 
 from agent_debugger_sdk.core.events import EventType, TraceEvent
 
-
-def _event_value(event: TraceEvent | None, key: str, default: Any = None) -> Any:
-    if event is None:
-        return default
-    if hasattr(event, key):
-        return getattr(event, key)
-    return event.data.get(key, default)
+from .intelligence.helpers import event_value as _event_value
 
 
 class CausalAnalyzer:

collector/ranking/event_ranker.py

@@ -7,14 +7,7 @@
 
 from agent_debugger_sdk.core.events import EventType, TraceEvent
 
-
-def _event_value(event: TraceEvent | None, key: str, default: Any = None) -> Any:
-    """Extract a value from an event's attributes or data dict."""
-    if event is None:
-        return default
-    if hasattr(event, key):
-        return getattr(event, key)
-    return event.data.get(key, default)
+from ..intelligence.helpers import event_value as _event_value
 
 
 class EventRankingService: