fix: handle ValueError and RecursionError in _safe_json_serialize

[voidborne-d]

Summary

Both _safe_json_serialize implementations only catch TypeError and OverflowError from json.dumps, but json.dumps can also raise:

• ValueError: circular references (e.g. obj.ref = obj)
• RecursionError: deeply nested structures exceeding Python's recursion limit

Impact

telemetry/tracing.py (critical): _safe_json_serialize is called inside a finally block during trace_tool_call. An unhandled exception here can suppress the original tool execution result — telemetry failures should never affect runtime correctness.

models/lite_llm.py: An unhandled exception during message serialization causes hard LLM request construction failures where a graceful fallback is expected.

Root Cause

The except clause catches (TypeError, OverflowError) but not ValueError or RecursionError. Both are documented json.dumps failure modes:

• ValueError for circular references (per Python docs)
• RecursionError for structures exceeding sys.getrecursionlimit()

Note: in tracing.py, the default=lambda o: ... callback handles TypeError for individual non-serializable objects, but circular references and deep recursion bypass the default callback entirely.

Fix

• Add ValueError and RecursionError to the exception tuple in both implementations.
• In lite_llm.py, wrap the str() fallback in a secondary try/except RecursionError since str() → __repr__() can also fail on deeply recursive objects.

Reproduction

class Node:
    def __init__(self):
        self.ref = self

obj = Node()
# Before fix: raises ValueError
# After fix: returns fallback string
_safe_json_serialize(obj)

Tests

8 regression tests added across both modules:

• test_safe_json_serialize.py (telemetry): circular ref, deep nesting, normal dict, non-serializable object
• test_litellm_safe_serialize.py (models): circular ref, deep nesting, normal dict, non-serializable object

All 8 tests pass.

Fixes #5411
Fixes #5412

[rohityan]

Hi @voidborne-d , Thank you for your contribution! We appreciate you taking the time to submit this pull request. Please fix formatting errors by running autoformat.sh

[voidborne-d]

Hi @rohityan, thanks for the review! I've run autoformat.sh and pushed the formatting fix in ddff034. The changes were isort reordering + pyink reformatting across lite_llm.py and both test files. All 8 tests pass.