claude-smart 0.2.41 → 0.2.43

package/plugin/vendor/reflexio/reflexio/server/services/extractor_config_utils.py

@@ -0,0 +1,91 @@
+"""
+Utility functions for filtering extractor configurations.
+"""
+
+import logging
+
+from reflexio.models.config_schema import (
+    SINGLETON_AGENT_SUCCESS_EVALUATION_NAME,
+    SINGLETON_PROFILE_EXTRACTOR_NAME,
+    SINGLETON_USER_PLAYBOOK_NAME,
+    AgentSuccessConfig,
+    ProfileExtractorConfig,
+    UserPlaybookExtractorConfig,
+)
+
+logger = logging.getLogger(__name__)
+
+
+def get_extractor_name(config: object) -> str:
+    """
+    Get the display name for an extractor config.
+
+    Checks extractor_name, playbook_name, and evaluation_name attributes in order.
+
+    Args:
+        config: Extractor configuration object (e.g., ProfileExtractorConfig, PlaybookConfig, AgentSuccessConfig)
+
+    Returns:
+        str: The extractor name, or "unknown" if none found
+    """
+    if isinstance(config, ProfileExtractorConfig):
+        return SINGLETON_PROFILE_EXTRACTOR_NAME
+    if isinstance(config, UserPlaybookExtractorConfig):
+        return SINGLETON_USER_PLAYBOOK_NAME
+    if isinstance(config, AgentSuccessConfig):
+        return SINGLETON_AGENT_SUCCESS_EVALUATION_NAME
+    return getattr(config, "extractor_name", None) or getattr(
+        config, "playbook_name", None
+    ) or getattr(config, "evaluation_name", "unknown") or "unknown"
+
+
+def filter_extractor_configs[TExtractorConfig](
+    extractor_configs: list[TExtractorConfig],
+    source: str | None = None,
+    allow_manual_trigger: bool = False,
+) -> list[TExtractorConfig]:
+    """
+    Filter extractor configs based on source and manual_trigger.
+
+    This is a standalone utility function that can be used by both BaseGenerationService
+    and GenerationService to filter extractor configurations consistently.
+
+    Args:
+        extractor_configs: List of extractor configuration objects (e.g., ProfileExtractorConfig,
+            PlaybookConfig, AgentSuccessConfig)
+        source: Request source for filtering by request_sources_enabled. If None, source
+            filtering is skipped.
+        allow_manual_trigger: Whether to allow extractors with manual_trigger=True.
+            If False, extractors with manual_trigger=True will be skipped.
+
+    Returns:
+        Filtered list of extractor configs that should run for the given parameters
+    """
+    filtered_configs = []
+
+    for config in extractor_configs:
+        # Check if config has request_sources_enabled attribute
+        if hasattr(config, "request_sources_enabled"):
+            sources_enabled = config.request_sources_enabled  # type: ignore[reportAttributeAccessIssue]
+            # Skip if source filtering applies and source is not in enabled list
+            if sources_enabled and source and source not in sources_enabled:
+                logger.debug(
+                    "Skipping extractor '%s' - source '%s' not in enabled sources %s",
+                    get_extractor_name(config),
+                    source,
+                    sources_enabled,
+                )
+                continue
+
+        # Check manual_trigger: skip if manual_trigger=True and allow_manual_trigger=False
+        manual_trigger = getattr(config, "manual_trigger", False)
+        if manual_trigger and not allow_manual_trigger:
+            logger.debug(
+                "Skipping extractor '%s' - manual_trigger=True and allow_manual_trigger=False",
+                get_extractor_name(config),
+            )
+            continue
+
+        filtered_configs.append(config)
+
+    return filtered_configs

package/plugin/vendor/reflexio/reflexio/server/services/extractor_interaction_utils.py

@@ -0,0 +1,251 @@
+"""
+Utility functions for per-extractor interaction data collection.
+
+These functions are shared across ProfileExtractor, PlaybookExtractor, and AgentSuccessEvaluator
+to handle per-extractor stride checking, source filtering, and sliding window iteration.
+"""
+
+import logging
+
+from reflexio.models.api_schema.internal_schema import RequestInteractionDataModel
+from reflexio.models.config_schema import DEFAULT_STRIDE_SIZE, DEFAULT_WINDOW_SIZE
+from reflexio.server.services.extractor_config_utils import get_extractor_name
+
+logger = logging.getLogger(__name__)
+
+
+def get_extractor_window_params[TExtractorConfig](
+    extractor_config: TExtractorConfig,
+    global_window_size: int | None,
+    global_stride_size: int | None,
+) -> tuple[int, int]:
+    """
+    Get effective window_size and stride_size for a specific extractor.
+
+    Uses extractor's override values if set, otherwise falls back to global values,
+    then to defaults (window_size=10, stride_size=5).
+
+    Args:
+        extractor_config: Extractor configuration object
+        global_window_size: Global window_size from config
+        global_stride_size: Global stride_size from config
+
+    Returns:
+        Tuple of (window_size, stride_size) for this extractor
+    """
+    window_size_override = getattr(extractor_config, "window_size_override", None)
+    stride_size_override = getattr(extractor_config, "stride_size_override", None)
+
+    window_size = (
+        window_size_override
+        if window_size_override is not None
+        else (
+            global_window_size
+            if global_window_size is not None
+            else DEFAULT_WINDOW_SIZE
+        )
+    )
+    stride_size = (
+        stride_size_override
+        if stride_size_override is not None
+        else (
+            global_stride_size
+            if global_stride_size is not None
+            else DEFAULT_STRIDE_SIZE
+        )
+    )
+
+    return window_size, stride_size
+
+
+def get_effective_source_filter[TExtractorConfig](
+    extractor_config: TExtractorConfig,
+    triggering_source: str | None,
+) -> tuple[bool, list[str] | None]:
+    """
+    Get effective source filter for an extractor.
+
+    NOTE: By the time this function is called, the extractor has already been filtered
+    by filter_extractor_configs() in base_generation_service.py. That filter ensures:
+    - If sources_enabled is set and triggering_source is not in it, the extractor is skipped
+    - So if we reach here with sources_enabled set, triggering_source MUST be in the list
+      (unless triggering_source is None, which happens in rerun flows)
+
+    This function still includes a safety check for edge cases.
+
+    Args:
+        extractor_config: Extractor configuration with request_sources_enabled field
+        triggering_source: The source from the triggering request (None for rerun flows)
+
+    Returns:
+        Tuple of (should_skip, source_filter):
+        - (False, None): If request_sources_enabled = None or empty list → get ALL sources, no filtering
+        - (False, [triggering_source]): If sources_enabled is set AND triggering_source in list
+        - (False, sources_enabled): If sources_enabled is set AND triggering_source is None → return all enabled sources
+        - (True, None): Safety skip if triggering_source not in sources_enabled (shouldn't happen)
+    """
+    sources_enabled = getattr(extractor_config, "request_sources_enabled", None)
+
+    # If no sources_enabled configured or empty list, extractor wants ALL sources
+    if not sources_enabled:
+        return (False, None)  # No source filtering - get all
+
+    # If triggering_source is None (rerun without specific source),
+    # return all enabled sources so caller can filter by them
+    if triggering_source is None:
+        return (False, sources_enabled)  # Return list of enabled sources
+
+    # Safety check: triggering_source should be in sources_enabled
+    # (filter_extractor_configs should have already filtered this out)
+    if triggering_source not in sources_enabled:
+        logger.warning(
+            "Skipping extractor '%s' - triggering_source '%s' not in sources_enabled %s "
+            "(this should have been filtered earlier)",
+            get_extractor_name(extractor_config),
+            triggering_source,
+            sources_enabled,
+        )
+        return (True, None)  # Skip this extractor
+
+    # triggering_source is in sources_enabled, use it for filtering
+    return (False, [triggering_source])
+
+
+def should_extractor_run_by_stride(
+    new_interaction_count: int,
+    stride_size: int | None,
+) -> bool:
+    """
+    Determine if an extractor should run based on its stride_size configuration.
+
+    Args:
+        new_interaction_count: Number of new interactions since last run
+        stride_size: Configured stride_size for this extractor
+
+    Returns:
+        True if extractor should run, False otherwise
+    """
+    if new_interaction_count <= 0:
+        return False
+
+    if stride_size is None or stride_size <= 0:
+        return True  # No stride configured, always run
+
+    return new_interaction_count >= stride_size
+
+
+def filter_interactions_by_source(
+    request_interaction_data_models: list[RequestInteractionDataModel],
+    source_filter: str | list[str] | None,
+) -> list[RequestInteractionDataModel]:
+    """
+    Filter request interaction data models by source.
+
+    Args:
+        request_interaction_data_models: Data models to filter
+        source_filter:
+            - None: return all (no filtering)
+            - str: filter to only this source
+            - list[str]: filter to any of these sources
+
+    Returns:
+        Filtered list of RequestInteractionDataModel
+    """
+    if source_filter is None:
+        return request_interaction_data_models
+
+    if isinstance(source_filter, str):
+        allowed_sources = {source_filter}
+    else:
+        allowed_sources = set(source_filter)
+
+    return [
+        rim
+        for rim in request_interaction_data_models
+        if rim.request.source in allowed_sources
+    ]
+
+
+from collections.abc import Iterator
+
+
+def iter_sliding_windows(
+    request_interaction_data_models: list[RequestInteractionDataModel],
+    window_size: int,
+    stride_size: int | None,
+) -> Iterator[tuple[int, list[RequestInteractionDataModel]]]:
+    """
+    Yield sliding windows of RequestInteractionDataModel based on interaction count.
+
+    Windows are created based on total interaction count, not number of data models.
+    Each window contains complete RequestInteractionDataModel objects (never splits a model).
+
+    Args:
+        request_interaction_data_models: List of RequestInteractionDataModel to window over
+        window_size: Target number of interactions per window
+        stride_size: Step size in interactions between window starts (must be >= 1)
+
+    Yields:
+        Tuples of (window_index, list_of_request_interaction_data_models)
+
+    Example:
+        With 3 models having [10, 20, 15] interactions, window_size=25, stride_size=15:
+        - Window 0: models[0], models[1] (30 interactions, starts at 0)
+        - Window 1: models[1], models[2] (35 interactions, starts at 15)
+    """
+    if not request_interaction_data_models:
+        return
+
+    if window_size <= 0:
+        # Invalid window_size, yield single window with all data
+        yield (0, request_interaction_data_models)
+        return
+
+    # Default stride_size to window_size if not valid
+    effective_stride_size = (
+        stride_size if stride_size and stride_size > 0 else window_size
+    )
+
+    # Build cumulative interaction counts for each model
+    # cumulative[i] = total interactions from models[0] to models[i-1] (exclusive)
+    cumulative: list[int] = [0]
+    for rim in request_interaction_data_models:
+        cumulative.append(cumulative[-1] + len(rim.interactions))
+
+    total_interactions = cumulative[-1]
+
+    if total_interactions == 0:
+        return
+
+    # If all data fits in one window, yield single window
+    if total_interactions <= window_size:
+        yield (0, request_interaction_data_models)
+        return
+
+    window_idx = 0
+    window_start = 0  # Starting interaction index
+
+    while window_start < total_interactions:
+        window_end = window_start + window_size
+
+        # Find models that overlap with [window_start, window_end)
+        # A model overlaps if its start < window_end AND its end > window_start
+        selected_models: list[RequestInteractionDataModel] = []
+
+        for i, rim in enumerate(request_interaction_data_models):
+            model_start = cumulative[i]
+            model_end = cumulative[i + 1]
+
+            # Model overlaps with window if: model_start < window_end AND model_end > window_start
+            if model_start < window_end and model_end > window_start:
+                selected_models.append(rim)
+
+        if selected_models:
+            yield (window_idx, selected_models)
+
+        window_idx += 1
+        window_start += effective_stride_size
+
+        # Prevent infinite loop if stride_size is 0 (shouldn't happen with validation above)
+        if effective_stride_size <= 0:
+            break