[OPIK-2787] [P SDK] GeneratorExit Error with LangGraph Integration

[yaricom]

Details

Allows to skip specific errors with LangChain/LangGraph integration

• Introduced skip_error_callback to allow selective skipping of error outputs during trace handling.
• Updated OpikTracer to incorporate conditional error handling based on the callback's result.
• Enhanced error processing logic to handle skipped errors with placeholder outputs.
• Added integration test test_langchain_callback__skip_error_callback__error_output_skipped.

Example of usage

from opik.integrations.langchain.opik_tracer import OpikTracer

def _should_skip_error(error: str) -> bool:
    if error is not None and error.startswith("FakeListLLMError"):
        # skip processing - we are sure that this is OK
        return True
    else:
        return False

callback = OpikTracer(
    skip_error_callback=_should_skip_error,
)

Result in UI

Change checklist

• User facing
• Documentation update

Issues

• OPIK-2787

Testing

Implemented related integration test.

Documentation

Nothing added

[Copilot]

Pull Request Overview

This PR introduces selective error skipping functionality for the LangChain/LangGraph integration to prevent GeneratorExit errors from being logged as failures. A new skip_error_callback parameter allows users to define custom logic for determining which errors should be skipped during trace handling, with skipped errors resulting in a placeholder output instead of error information.

Key Changes:

• Added skip_error_callback parameter to OpikTracer for conditional error handling
• Implemented _should_skip_error method to evaluate error skipping logic
• Introduced ERROR_SKIPPED_OUTPUTS constant as placeholder for skipped errors
• Updated error processing in _persist_run and _process_end_span_with_error methods

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File	Description
sdks/python/src/opik/integrations/langchain/opik_tracer.py	Added skip_error_callback parameter, implemented error skipping logic in error handling methods
sdks/python/tests/library_integration/langchain/test_langchain.py	Added integration test verifying error skipping functionality with FakeStreamingListLLM

[github-actions]

SDK E2E Tests Results

0 tests  ±0   0 ✅ ±0   0s ⏱️ ±0s
0 suites ±0   0 💤 ±0 
0 files   ±0   0 ❌ ±0 

Results for commit ba0ce46. ± Comparison against base commit f303a9b.

♻️ This comment has been updated with latest results.