fix(langchain): add run_id check on_llm_error

[hassiebp]

Important

Add run_id existence check in on_llm_error and adjust sleep timing in tests for async operations.

• Behavior:
  • Add run_id existence check in on_llm_error in langchain.py to prevent errors when run_id is not found.
• Tests:
  • Add sleep(0.1) in test_core_sdk.py and sleep(1) in test_datasets.py to ensure proper timing for asynchronous operations.

^{This description was created by for f05dcec. It will automatically update as commits are pushed.}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

Enhanced error handling in the Langchain callback handler by adding a safety check for run_id existence before processing LLM errors.

• Added defensive check if run_id in self.runs in on_llm_error method within langfuse/callback/langchain.py to prevent KeyErrors
• Aligns error handling with existing patterns in on_chain_error and on_tool_error methods

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

Disclaimer: Experimental PR review

PR Summary

(updates since last review)

This PR adds timing-related fixes to improve test stability in the test suite.

• Added time.sleep(1) in tests/test_datasets.py before trace retrieval to ensure data availability
• Added time.sleep(0.1) in tests/test_core_sdk.py between trace creation and update to prevent race conditions
• Both changes address potential flakiness in tests by adding small delays at critical points

_{2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}