mirror of
https://github.com/BerriAI/litellm.git
synced 2026-10-14 04:09:09 +00:00
* feat(model-catalog): add Rust registry validation (#43136) * doc * feat(model-catalog): add Rust registry validation * test(model-catalog): ignore integration tests * docs(model-catalog): fix validation note grammar --------- Co-authored-by: Yujong Lee <yujong@berri.ai> (cherry picked from commitc289d5d6fb) * test(rust): reorganize core crate tests and split cache and OCR suites (#43177) * test(rust): group cache tests under cache/ and fold test_ocr.py into ocr/ The two failure cases in test_ocr.py duplicated the upstream-500 and timeout rows of PUBLIC_FAILURES, so only the file-input encoding case moves to ocr/test_requests.py Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(rust): split the response cache suite into one file per backend test_response_cache.py grew to 2400 lines. Each backend now has its own file, shared fixtures live in cache/conftest.py and shared helpers in support/cache.py. The helpers alias the private native test handles once, dropping the per-call reportPrivateUsage hits Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * split tokenizer test * test(core): consolidate route integration tests under tests/ with rstest and wiremock Moves the public-API OCR route tests out of src/ocr/route.rs and document.rs into tests/ocr/, split per provider plus lifecycle, machine, and document tests, merging the duplicated pairs. Messages, audio transcription, and chat completions share one wiremock-based upstream and recording secret source in tests/support, and gain table-driven cases for auth, routing, upstream errors, streaming, and declines. Tests of litellm-llms items move to that crate. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(messages): keep the stream relay test independent of the stream head contents The stream head carries no headers on main, so the relay test asserts the open-then-deliver order and the relayed body instead of header hand-off. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit10413796c6) * feat(rust): hand upstream response headers to the native Messages stream (#43178) * ci: drop the ocr_testing job now that tests/ocr_tests is gone Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(ocr): restore the live OCR matrix and the ocr_testing job The public litellm.ocr / aocr / Router interface is unchanged by the Rust migration, so the live provider matrix still applies. Drops the stale VCR skip list for the deleted test_rust_bridge.py. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * test(messages): show streamed upstream headers never reach the native stream The Python handler puts the upstream response headers on the stream's _hidden_params before the first chunk so the proxy can forward them as llm_provider-* headers. The native route drops them, and this test fails on the Rust path while passing on Python. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * feat(messages): hand upstream response headers to the native stream before its first chunk The Messages route fills MessagesStreamHead from the upstream response and yields it on Open. The Python driver converts it through the protocol host and hands it to Stream and SyncStream as their _hidden_params, so a streamed native call carries additional_headers the same way the Python handler does and the proxy can forward them as llm_provider-* headers. The relay contract lives in the core crate test, the hand-off in the host-python driver test, and the header projection in the route host test, so the recording-server test that showed the gap is dropped. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * wip --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> (cherry picked from commit081f73f021) * feat(testkit): agent clients for Claude Code, Codex and opencode (#43181) * feat(testkit): install and configure Claude Code, Codex and opencode against a gateway Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(testkit): derive targets from target-lexicon and split agents behind a trait Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * refactor(testkit): group sources into agent and install folders Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * feat(testkit): split agents into install, configure and drive with semver-aware launch Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * style(testkit): drop a needless borrow Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com> (cherry picked from commit9c10e0f985) * ci: run migrated unit selections on every event in legacy GHA shards (#43182) * ci: run the unit_selection.sh shard files on every event instead of only fork pull requests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * ci: rename fork-flag to unit-flag now that it applies on every event --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit694783ebbe) * test: move tests/test_litellm root and small trees into tests/unit (#43186) * ci: run the unit_selection.sh shard files on every event instead of only fork pull requests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * ci: rename fork-flag to unit-flag now that it applies on every event * test: move tests/test_litellm root and small trees into tests/unit Pure renames, no content changes. Follow-up commits in this PR fix references, merge the three files that already existed in tests/unit, keep live-provider tests in tests/test_litellm and wire CI. * test: carry tests/test_litellm conftest isolation into tests/unit Callback lists, routing fallbacks, cached HTTP clients, logger state, AWS, proxy-URL and keychain env, and session-end client cleanup now reset for unit tests too. The environment isolation owns its MonkeyPatch so a test's own monkeypatch is undone before the model-cost teardown runs. * test: merge, split and prune the moved root and small-tree tests Merge batches/test_batch_utils.py and the chat_completions and messages dispatch tests into the files that already existed in tests/unit. Keep the live Gemini interactions tests, the async image-fetch format test and the OpenAI embedding scorer test in tests/test_litellm since they need real network or keys. Put test_router.py under tests/unit/test_router so the existing package no longer shadows it. Delete eight tests the audit found superseded by stronger ones kept in this move. * ci: run the moved root and small-tree tests under their legacy flags Add the misc and responses-caching-types flags to unit_selection.sh and CircleCI, extend enterprise-routing and mcp-integration, and point the legacy GHA shards, Makefile, redis-compat workflow, merge smoke manifest and change classifier at the new paths. * test: make the new tests/unit directories packages tests/unit/test_package_layout.py requires every directory to carry an __init__.py, and without one the moved and retained test_litellm_responses_bridge.py modules collide on import. * test: scope the unit socket block to tests/unit in shared sessions The GHA shards collect the legacy test-path and the unit selection in one pytest session. The unit conftest's loopback-only block leaked into legacy modules that reach the network at import. The legacy conftest now lifts the restriction at collect and setup time, and the unit conftest re-applies it when collecting its own modules. * test: give the shard-script tests their own GITHUB_OUTPUT They only passed where the runner set it. The CircleCI unit job's env allowlist drops it, so the script's redirect failed there. * test: point the router and module-deletion checks at tests/unit router_code_coverage and code_qa_check_tests only searched tests/test_litellm, so the moved router tests no longer counted. The two silent-experiment tests the audit deleted were the only direct callers of those methods; they are replaced with tests that assert the forwarded shadow request and the recursion guard. --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commitf6882246d4) * refactor(framer): replace Framer trait with tokio-util codecs (#43193) * refactor(framer): replace Framer trait with tokio-util codecs Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * refactor(framer): port SSE and AWS event stream framing to codecs Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * fix(rust): drop clone on Copy capabilities in messages request test Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * fix(rust): use field init shorthand in messages request test Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit3c93ea1697) * test: move tests/test_litellm/llms into tests/unit/llms (#43191) * ci: run the unit_selection.sh shard files on every event instead of only fork pull requests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * ci: rename fork-flag to unit-flag now that it applies on every event * test: move tests/test_litellm root and small trees into tests/unit Pure renames, no content changes. Follow-up commits in this PR fix references, merge the three files that already existed in tests/unit, keep live-provider tests in tests/test_litellm and wire CI. * test: carry tests/test_litellm conftest isolation into tests/unit Callback lists, routing fallbacks, cached HTTP clients, logger state, AWS, proxy-URL and keychain env, and session-end client cleanup now reset for unit tests too. The environment isolation owns its MonkeyPatch so a test's own monkeypatch is undone before the model-cost teardown runs. * test: merge, split and prune the moved root and small-tree tests Merge batches/test_batch_utils.py and the chat_completions and messages dispatch tests into the files that already existed in tests/unit. Keep the live Gemini interactions tests, the async image-fetch format test and the OpenAI embedding scorer test in tests/test_litellm since they need real network or keys. Put test_router.py under tests/unit/test_router so the existing package no longer shadows it. Delete eight tests the audit found superseded by stronger ones kept in this move. * ci: run the moved root and small-tree tests under their legacy flags Add the misc and responses-caching-types flags to unit_selection.sh and CircleCI, extend enterprise-routing and mcp-integration, and point the legacy GHA shards, Makefile, redis-compat workflow, merge smoke manifest and change classifier at the new paths. * test: make the new tests/unit directories packages tests/unit/test_package_layout.py requires every directory to carry an __init__.py, and without one the moved and retained test_litellm_responses_bridge.py modules collide on import. * test: scope the unit socket block to tests/unit in shared sessions The GHA shards collect the legacy test-path and the unit selection in one pytest session. The unit conftest's loopback-only block leaked into legacy modules that reach the network at import. The legacy conftest now lifts the restriction at collect and setup time, and the unit conftest re-applies it when collecting its own modules. * test: move tests/test_litellm/llms into tests/unit/llms Rename-only. Moves the provider tests and the fine-tuning fixtures they load, mirroring the old paths. Follow-up commits merge, split and wire them. * test: merge, split and prune the moved llms tests Merges the Databricks chat transformation tests into the existing unit file, keeps the tests that need real keys or the network in tests/test_litellm, deletes the audited tests a stronger unit test already covers, and points imports at tests.unit.llms. * ci: run the moved llms tests under their legacy flags The Vertex AI and All Other Providers shards keep their legacy test-path for the retained files and add the llm-vertex-ai and llm-other-providers unit selections. CircleCI gets matching unit jobs. * test: make the tests/unit/llms directories packages Adds __init__.py to the moved dirs and drops the legacy ones whose directories no longer hold tests. * test: drop script runners and path hacks the llms split left dangling The __main__ runners in the split openai_like files and the Databricks e2e runner called tests that now live in the other half of the split or were deleted. The retained legacy halves also no longer need sys.path edits. * test: give the shard-script tests their own GITHUB_OUTPUT They only passed where the runner set it. The CircleCI unit job's env allowlist drops it, so the script's redirect failed there. * test: point the router and module-deletion checks at tests/unit router_code_coverage and code_qa_check_tests only searched tests/test_litellm, so the moved router tests no longer counted. The two silent-experiment tests the audit deleted were the only direct callers of those methods; they are replaced with tests that assert the forwarded shadow request and the recursion guard. * test: keep the Databricks manual e2e runner and fix the SageMaker Nova run path The Databricks e2e file is a manual script whose main() calls the tests that were pruned, so pruning them broke the documented run. It is back to its main version. The SageMaker Nova docstring now points at the file's real location in tests/local_testing. * test: keep the job's UNIT_FLAG out of the shard-script tests --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit5e6dc89ba1) * test: move tests/test_litellm integrations and secret_managers into tests/unit (#43194) * ci: run the unit_selection.sh shard files on every event instead of only fork pull requests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * ci: rename fork-flag to unit-flag now that it applies on every event * test: move tests/test_litellm root and small trees into tests/unit Pure renames, no content changes. Follow-up commits in this PR fix references, merge the three files that already existed in tests/unit, keep live-provider tests in tests/test_litellm and wire CI. * test: carry tests/test_litellm conftest isolation into tests/unit Callback lists, routing fallbacks, cached HTTP clients, logger state, AWS, proxy-URL and keychain env, and session-end client cleanup now reset for unit tests too. The environment isolation owns its MonkeyPatch so a test's own monkeypatch is undone before the model-cost teardown runs. * test: merge, split and prune the moved root and small-tree tests Merge batches/test_batch_utils.py and the chat_completions and messages dispatch tests into the files that already existed in tests/unit. Keep the live Gemini interactions tests, the async image-fetch format test and the OpenAI embedding scorer test in tests/test_litellm since they need real network or keys. Put test_router.py under tests/unit/test_router so the existing package no longer shadows it. Delete eight tests the audit found superseded by stronger ones kept in this move. * ci: run the moved root and small-tree tests under their legacy flags Add the misc and responses-caching-types flags to unit_selection.sh and CircleCI, extend enterprise-routing and mcp-integration, and point the legacy GHA shards, Makefile, redis-compat workflow, merge smoke manifest and change classifier at the new paths. * test: make the new tests/unit directories packages tests/unit/test_package_layout.py requires every directory to carry an __init__.py, and without one the moved and retained test_litellm_responses_bridge.py modules collide on import. * test: scope the unit socket block to tests/unit in shared sessions The GHA shards collect the legacy test-path and the unit selection in one pytest session. The unit conftest's loopback-only block leaked into legacy modules that reach the network at import. The legacy conftest now lifts the restriction at collect and setup time, and the unit conftest re-applies it when collecting its own modules. * test: move tests/test_litellm/llms into tests/unit/llms Rename-only. Moves the provider tests and the fine-tuning fixtures they load, mirroring the old paths. Follow-up commits merge, split and wire them. * test: merge, split and prune the moved llms tests Merges the Databricks chat transformation tests into the existing unit file, keeps the tests that need real keys or the network in tests/test_litellm, deletes the audited tests a stronger unit test already covers, and points imports at tests.unit.llms. * ci: run the moved llms tests under their legacy flags The Vertex AI and All Other Providers shards keep their legacy test-path for the retained files and add the llm-vertex-ai and llm-other-providers unit selections. CircleCI gets matching unit jobs. * test: make the tests/unit/llms directories packages Adds __init__.py to the moved dirs and drops the legacy ones whose directories no longer hold tests. * test: drop script runners and path hacks the llms split left dangling The __main__ runners in the split openai_like files and the Databricks e2e runner called tests that now live in the other half of the split or were deleted. The retained legacy halves also no longer need sys.path edits. * test: give the shard-script tests their own GITHUB_OUTPUT They only passed where the runner set it. The CircleCI unit job's env allowlist drops it, so the script's redirect failed there. * test: point the router and module-deletion checks at tests/unit router_code_coverage and code_qa_check_tests only searched tests/test_litellm, so the moved router tests no longer counted. The two silent-experiment tests the audit deleted were the only direct callers of those methods; they are replaced with tests that assert the forwarded shadow request and the recursion guard. * test: move tests/test_litellm integrations and secret_managers into tests/unit Rename-only. Mirrors the old paths, including the directory conftests and the prompt and JSON fixtures. Follow-up commits prune and wire them. * test: prune and repoint the moved integrations tests Deletes the 7 audited tests a stronger test in the same tree already covers, imports the TLS sink helpers from their new conftest path, and restores os.environ after each integrations test. Some presets write OTEL_EXPORTER_OTLP_HEADERS straight into os.environ, and without the legacy tree's test ordering that header leaked into the AgentOps tests. * ci: run the moved integrations tests under their legacy flag The integrations GHA shard and a new CircleCI job run the integrations unit selection. secret_managers joins the misc selection. * docs: point integrations and secret_managers references at tests/unit * test: make the moved integrations directories packages * test: keep the Databricks manual e2e runner and fix the SageMaker Nova run path The Databricks e2e file is a manual script whose main() calls the tests that were pruned, so pruning them broke the documented run. It is back to its main version. The SageMaker Nova docstring now points at the file's real location in tests/local_testing. * test: keep the job's UNIT_FLAG out of the shard-script tests --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commitcf491d1df9) * fix(tests): drop the repeated UNIT_FLAG key in test_unit_shard_missing_paths (#43212) Co-authored-by: mateo-berri <277851410+mateo-berri@users.noreply.github.com> (cherry picked from commit25de1ab2b0) * fix(jwt): let x-litellm-team-id select DB membership teams when the token also carries a team claim (#43206) * fix(jwt): let x-litellm-team-id select DB membership teams when the token also carries a team claim Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * docs(jwt): describe header team selection under fallback_to_db_teams Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: yassin <yassin@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit7b4fd47c6e) * test: move tests/test_litellm core utils, routing, responses, caching and rust_bridge into tests/unit (#43199) * ci: run the unit_selection.sh shard files on every event instead of only fork pull requests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * ci: rename fork-flag to unit-flag now that it applies on every event * test: move tests/test_litellm root and small trees into tests/unit Pure renames, no content changes. Follow-up commits in this PR fix references, merge the three files that already existed in tests/unit, keep live-provider tests in tests/test_litellm and wire CI. * test: carry tests/test_litellm conftest isolation into tests/unit Callback lists, routing fallbacks, cached HTTP clients, logger state, AWS, proxy-URL and keychain env, and session-end client cleanup now reset for unit tests too. The environment isolation owns its MonkeyPatch so a test's own monkeypatch is undone before the model-cost teardown runs. * test: merge, split and prune the moved root and small-tree tests Merge batches/test_batch_utils.py and the chat_completions and messages dispatch tests into the files that already existed in tests/unit. Keep the live Gemini interactions tests, the async image-fetch format test and the OpenAI embedding scorer test in tests/test_litellm since they need real network or keys. Put test_router.py under tests/unit/test_router so the existing package no longer shadows it. Delete eight tests the audit found superseded by stronger ones kept in this move. * ci: run the moved root and small-tree tests under their legacy flags Add the misc and responses-caching-types flags to unit_selection.sh and CircleCI, extend enterprise-routing and mcp-integration, and point the legacy GHA shards, Makefile, redis-compat workflow, merge smoke manifest and change classifier at the new paths. * test: make the new tests/unit directories packages tests/unit/test_package_layout.py requires every directory to carry an __init__.py, and without one the moved and retained test_litellm_responses_bridge.py modules collide on import. * test: scope the unit socket block to tests/unit in shared sessions The GHA shards collect the legacy test-path and the unit selection in one pytest session. The unit conftest's loopback-only block leaked into legacy modules that reach the network at import. The legacy conftest now lifts the restriction at collect and setup time, and the unit conftest re-applies it when collecting its own modules. * test: move tests/test_litellm/llms into tests/unit/llms Rename-only. Moves the provider tests and the fine-tuning fixtures they load, mirroring the old paths. Follow-up commits merge, split and wire them. * test: merge, split and prune the moved llms tests Merges the Databricks chat transformation tests into the existing unit file, keeps the tests that need real keys or the network in tests/test_litellm, deletes the audited tests a stronger unit test already covers, and points imports at tests.unit.llms. * ci: run the moved llms tests under their legacy flags The Vertex AI and All Other Providers shards keep their legacy test-path for the retained files and add the llm-vertex-ai and llm-other-providers unit selections. CircleCI gets matching unit jobs. * test: make the tests/unit/llms directories packages Adds __init__.py to the moved dirs and drops the legacy ones whose directories no longer hold tests. * test: drop script runners and path hacks the llms split left dangling The __main__ runners in the split openai_like files and the Databricks e2e runner called tests that now live in the other half of the split or were deleted. The retained legacy halves also no longer need sys.path edits. * test: give the shard-script tests their own GITHUB_OUTPUT They only passed where the runner set it. The CircleCI unit job's env allowlist drops it, so the script's redirect failed there. * test: point the router and module-deletion checks at tests/unit router_code_coverage and code_qa_check_tests only searched tests/test_litellm, so the moved router tests no longer counted. The two silent-experiment tests the audit deleted were the only direct callers of those methods; they are replaced with tests that assert the forwarded shadow request and the recursion guard. * test: move tests/test_litellm integrations and secret_managers into tests/unit Rename-only. Mirrors the old paths, including the directory conftests and the prompt and JSON fixtures. Follow-up commits prune and wire them. * test: prune and repoint the moved integrations tests Deletes the 7 audited tests a stronger test in the same tree already covers, imports the TLS sink helpers from their new conftest path, and restores os.environ after each integrations test. Some presets write OTEL_EXPORTER_OTLP_HEADERS straight into os.environ, and without the legacy tree's test ordering that header leaked into the AgentOps tests. * ci: run the moved integrations tests under their legacy flag The integrations GHA shard and a new CircleCI job run the integrations unit selection. secret_managers joins the misc selection. * docs: point integrations and secret_managers references at tests/unit * test: make the moved integrations directories packages * test: keep the Databricks manual e2e runner and fix the SageMaker Nova run path The Databricks e2e file is a manual script whose main() calls the tests that were pruned, so pruning them broke the documented run. It is back to its main version. The SageMaker Nova docstring now points at the file's real location in tests/local_testing. * test: move tests/test_litellm core utils, routing, responses, caching and rust_bridge into tests/unit Rename-only. Mirrors the old paths, including fixtures, the stubtest config and the native-route wheel script. Two files that collide with existing unit files are merged in a follow-up commit. * test: merge, prune and repoint the moved core, routing, responses, caching and rust_bridge tests Merges the two files that collided with existing unit files, folding the legacy extra case into test_is_chat_completion_cached_dict, and deletes the 9 audited tests a stronger test in the same file already covers. Keeps what needs the network in tests/test_litellm: test_tokenizers pulls a tokenizer from the Hugging Face hub, and the gpt2 and r50k_base tokenizer cases download their BPE files. The unit core_utils conftest points TIKTOKEN_CACHE_DIR at litellm's bundled encodings so the rest never depend on import order to stay offline, and FakeSecretVault moves to a shared module so both trees can build it. * ci: run the moved core, routing, responses, caching and rust_bridge tests under their flags core_utils gets a core-utils flag and CircleCI job, and its GHA shard keeps the legacy path for the retained network tests. router_utils and router_strategy join enterprise-routing, responses joins responses-caching-types (minus responses/mcp, which mcp-integration owns), caching joins caching-local and rust_bridge joins misc. The redis-compat, test-rust, stubtest and merge-smoke paths follow the move. * docs: point the Rust crate references at tests/unit * test: make the moved core, routing and rust_bridge directories packages * test: keep the no-loop DualCache batch_get_cache regression test It runs the sync path outside any event loop, which the inside-loop test cannot, so a change that picks the Redis client by loop state would only show up there. * test: keep the job's UNIT_FLAG out of the shard-script tests * fix(url_utils): block 192.0.0.0/24 on every Python patch release * test: move the new budget limiter tests into tests/unit/router_strategy * test: move the new sentry scrubbing tests into tests/unit/litellm_core_utils * test: move the new zerobus tests into tests/unit/integrations * test: make tests/unit/integrations/zerobus a package * test: load litellm's own tiktoken cache setup once instead of resetting it per test --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commita11a93f44a) * refactor(rust): move credential inheritance and the SDK limits out of the legacy callback crate into a driver preflight (#43259) The legacy callback crate carried two rewrites that have nothing to do with the Logging contract: litellm_credential_name inheritance and the max_budget and num_retries_per_request checks. Any later callback host would need them unchanged, which is the smell the crate's AGENTS.md now names. They are now a Preflight the driver in litellm-host-python runs on the keyword view begin returned, before the host projects from it, supplied by python-bridge and passed through run_legacy_call. The call order is unchanged (setup, deployment hook, credentials, limits) and a rejection still fails the call as a host failure, so the failure callbacks run as before. The preflight rewrites the adapter's own copy in place, so no extra dict copy and no new lifecycle method Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> (cherry picked from commite9491d31b5) * refactor(http): hand out an owned Client and route all providers through the pool (#43245) * refactor(messages): take the provider client from the injected HTTP pool The messages route kept its own process-wide reqwest client, so it ignored ssl_verify, CA bundles, client certs, proxies and every other setting that litellm-http resolves. The machine now takes the HttpClientPool and the call's HttpClientConfig, as OCR does, and the bridge passes its shared pool. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * refactor(http): hand out an owned Client and move chat, audio and OIDC onto the pool HttpClientPool now returns litellm_http::Client, a newtype only crates/http can build, so every provider client carries the resolved TLS, proxy and timeout settings. Chat completions and audio transcription drop their process-wide reqwest clients and take the pool and call config like messages; their 600s ceiling moves to the request. OidcResolver takes its client instead of building one, and the bridge hands it the pooled one. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * refactor(secrets): build Google, Azure and CyberArk manager clients from the pool The native secret managers built bare reqwest clients, so they ignored the host's TLS and proxy settings. load_native_manager now takes the pool and the host config and hands each manager a pooled client. CyberArk's CYBERARK_SSL_VERIFY and CYBERARK_CLIENT_CERT/KEY become an override on the host config instead of a hand-built client. To express a certificate and key in separate files, HttpClientConfig::client_certificate is now a ClientIdentity that is either one PEM or a split pair. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * chore(clippy): only crates/http may build a reqwest client Fence reqwest::Client, ClientBuilder and the TLS builder methods with disallowed-types and disallowed-methods so new code takes a litellm_http::Client from the pool. crates/http is exempt as the one place clients are built, and testkit as a dev-only installer. Tests move to litellm_http::Client::plain_for_test or a pooled client. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(secrets-cyberark): keep verifying certificates when the host disables it Python hands CyberArk its own ssl_verify, which wins over the global setting, so CYBERARK_SSL_VERIFY unset or true still verifies even when the host sets ssl_verify false. The pooled client copied the host's Disabled and would send the API key unverified; fall back to the built-in roots instead. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * fix(python-bridge): treat a missing litellm package as no host HTTP settings Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit1a58162630) * refactor(rust): promote anthropic messages out of experimental_pass_through (#43269) Co-authored-by: Yujong Lee <yujong@berri.ai> (cherry picked from commit2ef3250ec3) * test: stop CI tests from downloading tokenizer files and images (#43257) * test: load the embedding base image from a committed 100x100 PNG instead of downloading it * test: move the volcengine embedding test into tests/unit * test: check gpt2 and r50k_base tokenizer parity against committed tiktoken reference files * test: check hub tokenizer selection against an in-memory Hugging Face hub * test: serve image URLs from respx in the gemini tool-result and format-param tests * ci: drop the emptied legacy core-utils test path * test: cover the cohere and anthropic tokenizer paths in the hub tokenizer test * test: fetch every format-param image through respx and check its bytes reach the request * test: drop the gpt2 and r50k_base parity tests, which no litellm path uses * test: drop comments that restate assertions in the format-param test (cherry picked from commit2530255624) * fix(rust): preserve nested optional import failures (#43265) * fix(rust): preserve nested optional import failures Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * test(rust): restore Python modules after settings tests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commite1a9378093) * fix(callbacks-legacy-python): traverse and release the retained headers dict (#43274) * fix(callbacks-legacy-python): traverse and release the retained headers dict LegacyLogging keeps the headers dict it hands to pre_call and post_call, but its traverse never reported that edge to the collector and close never dropped it. A cycle a callback builds through that dict could not be collected, and a closed call kept the dict alive until the driver dropped the whole adapter. Visit and clear headers like body, with regression tests for both * refactor(callbacks-legacy-python): move the test support module into its own file --------- Co-authored-by: Yujong Lee <yujong@berri.ai> (cherry picked from commit4eb340abe0) * test(integration): port langfuse callbacks-in-db coverage to the local harness (#43282) * test(integration): port langfuse callbacks-in-db coverage to the local harness Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * test(integration): assert the langfuse db rows after the exported span Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: yuneng <yuneng@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commite11c3f5815) * test: finish the non-proxy half of tests/test_litellm (#43281) * test: move key-gated tests/test_litellm SDK tests into tests/llm_translation and drop empty folders * test: make token counter and health check unit tests run offline * ci: point unit shards, rust path filter, Makefile and docs at tests/unit * docs: fix stale test_litellm run paths in moved llm_translation tests * fix: correct databricks e2e sys.path depth and contributing example path --------- Co-authored-by: yuneng <yuneng@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit99655b6f86) * refactor(rust): prepare inference and auth foundations for the gateway (#43287) * refactor(rust): prepare inference and auth foundations * fix(rust): keep textract operations parsing from kebab-case model names Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit7ae721bf79) * feat(rust): add config router and gateway crates (#43289) Co-authored-by: Yujong Lee <yujong@berri.ai> (cherry picked from commitaffb547525) * test(integration): run the Langfuse DB-callback test on its own scratch database (#43288) * test(integration): run the Langfuse DB-callback test on its own scratch database The test from #43282 wrote success_callback=langfuse and the LANGFUSE_* env into the shared integration LiteLLM_Config. The suite's long-running gateway reloads that table and only ever adds callbacks, so it kept exporting to the test's closed Langfuse fake for the rest of the shard even after the rows were restored. The owned proxy now gets a scratch database, which also removes the snapshot/restore code. scratch_database moves into _support/database.py so test_cache_and_quota and this test share one copy, and the stock-config guard now checks the callback settings instead of the raw YAML text. * test(integration): include failure_callback in the stock-config Langfuse guard (cherry picked from commitdd63637322) * refactor(rust): expand logging and test coverage across gateway and Anthropic messages (#43295) * refactor(rust): prepare inference and auth foundations * fix(rust): keep textract operations parsing from kebab-case model names Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * done * refactor(types): derive Anthropic beta string conversions with Strum * fix(anthropic): report missing max_tokens as a missing field * refactor(rust): type Anthropic messages headers and auth after the Python layout Delete anthropic/messages/headers.rs. Its OAuth handling, credential ladder and beta merging move to anthropic/common_utils.rs where Python keeps them (optionally_handle_anthropic_oauth, get_auth_header, _merge_beta_headers), and the feature beta injection becomes update_headers_with_anthropic_beta on the messages config, as in Python. The BaseAnthropicMessagesConfig impl is unchanged apart from the bodies of validate_environment and request_headers Beta values are now the AnthropicBeta enum and BetaSet, which sort, dedupe and comma-join by construction. Request params gain typed speed, tools and context_management through Recognized, so the beta logic matches on enums instead of string-comparing JSON. OauthToken parses the sk-ant-oat token once and the chat config shares that detection instead of its own copy Case-insensitive header helpers move next to has_header in litellm-http. One deliberate divergence: a Bearer-prefixed OAuth key configured through api_key or ANTHROPIC_API_KEY is sent with a single Bearer scheme, where Python would emit "Bearer Bearer" Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * done * fix(rust): repair test compilation and clippy failures resolve auth before building the outbound request in prepare tests, give the host hook tests their own error type, and drop the disallowed reqwest client and err().expect() from core tests Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> (cherry picked from commit90873c46de) * test(proxy_behavior): scope the management proxy fixture to its package so its spend monitor cannot race the spend tests (#43302) Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit115668f43e) * fix(ci): stop stale CI reds, keep unit tests off the host env, retry CyberArk policy conflicts (#43294) * fix(ci): stop five stale or flaky CI reds and retry CyberArk policy-load conflicts The Langfuse redaction unit test exports to a local OTLP capture instead of polling Langfuse Cloud through a recorded lookup. The passthrough worker-kill test only requires spend rows for requests the surviving worker served. The spend-routes sweep treats the intentional /spend/capture_rate 503 as expected. CyberArk retries a 409 policy load in Python, Rust and the e2e Conjur helper instead of reading it as "variable exists". The integration egress guard now matches the script's own cgroup, so it no longer blocks the CircleCI agent, which runs as the same user. * fix(ci): keep the policy-load backoff typed as float * fix(ci): retry CyberArk policy loads without blocking the event loop and tighten the worker-kill and Langfuse tests * fix(secrets): load CyberArk policy one request at a time per manager * test(secrets): pin that non-conflict CyberArk policy failures are not retried * test(unit): run tests/unit with only an allowlisted host environment CircleCI's unit job inherits every project env var, so real provider keys, REDIS_HOST, DATABASE_URL and AWS or Azure credentials reached tests that assume none are set. Locally, litellm's import-time load_dotenv did the same from any .env up the tree. The unit conftest now drops every variable outside a small allowlist and disables dotenv before litellm is imported. * test(e2e): name a failed search and the stuck batch status instead of misattributing them The websearch session test read an empty web_search_tool_result_error block as a successful search, so a failing search tool surfaced as a session billing bug. The batch cancellation timeout now reports the last status the proxy returned. * fix(ci): scrub the host environment per unit test instead of for the whole pytest process GHA shards run tests/unit next to other suites in one process, so the import-time scrub deleted MCP_TEST_PEER_PYTHON before tests/mcp_tests read it and the MCP upstream fell back to the SDK2 interpreter. The two websearch tests that called OpenAI and Perplexity live are removed: tests/unit no longer sees their keys. * fix(ci): scrub only the host variables present before litellm is imported The per-test scrub also deleted TIKTOKEN_CACHE_DIR, which litellm sets at import to its bundled encodings, so tokenizer paths tried to download them and hit the socket guard. The prisma setup test now passes its own database URL instead of reading one another test leaked into the process environment. * fix(ci): stop the order-dependent unit reds and settle logging tasks on their own queue LoggingWorker marked a task done on whichever queue was current when the callback finished, so a callback that outlived an event-loop change raised "task_done() called too many times" or undercounted the new loop's queue. It now settles the queue the task came from. The rest are test isolation fixes for failures that only appeared when another file ran first on the same xdist worker: a replaced user_api_key_cache, breaker metrics unregistered by prometheus tests, semantic_router's health-check filter on uvicorn.access, logging tasks carried over from bedrock tests, a Router-written model_cost entry, and a stray post captured by the langflow test. The token counter check now asserts bounded chunking instead of wall-clock time. * test(e2e/ui): wait for the logout redirect before visiting a protected page Logout revokes the session server-side before clearing cookies and navigating, so an immediate page.goto either ran with the cookie still set or was aborted by the logout redirect (net::ERR_ABORTED). * test(unit): restore the prometheus metrics config per test and settle logs carried from earlier tests in the a2a cost tests * test(router): pin the router clock in the usage counter tests so a minute rollover cannot empty the read * test(e2e/ui): wait for logout to clear the token cookie instead of for a login redirect * test(integration/mcp): answer the model-info probe another test's proxy sends to the model double (cherry picked from commit14f4c34c61) * test(e2e): assert only litellm-owned batch behavior and move the blank S3 env pin to an integration test (#43321) (cherry picked from commite53e67ede5) * ci: drop main and litellm_* branch filters from the CircleCI litellm-main workflows (#43272) Co-authored-by: yuneng <yuneng@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commitc822c7fffa) * test(e2e): accept the otel cost write as a linked root trace (#42931) * test(e2e): accept the otel cost write as a linked root trace Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * test(e2e): window and fail-closed the linked otel trace read-back Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> * test(e2e): poll the otel read-back without recursion Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> --------- Co-authored-by: yuneng <yuneng@berri.ai> Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> (cherry picked from commit3ebf6a1fd8) * refactor(rust): exclude gateway and config crates from rc backport * refactor(rust): exclude router crate from rc backport --------- Co-authored-by: devin-ai-integration[bot] <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: Yujong Lee <yujong@berri.ai> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com> Co-authored-by: mateo-berri <277851410+mateo-berri@users.noreply.github.com> Co-authored-by: yassin <yassin@berri.ai>
1515 lines
53 KiB
Python
1515 lines
53 KiB
Python
import json
|
|
from typing import Optional
|
|
|
|
# Adds the grandparent directory to sys.path to allow importing project modules
|
|
|
|
import asyncio
|
|
|
|
import pytest
|
|
|
|
import litellm
|
|
from litellm.integrations._types.open_inference import (
|
|
MessageAttributes,
|
|
SpanAttributes,
|
|
ToolCallAttributes,
|
|
)
|
|
from litellm.integrations.arize.arize import ArizeLogger
|
|
from litellm.integrations.custom_logger import CustomLogger
|
|
from litellm.types.utils import Choices, StandardCallbackDynamicParams
|
|
|
|
|
|
def test_arize_set_attributes():
|
|
"""
|
|
Test setting attributes for Arize, including all custom LLM attributes.
|
|
Ensures that the correct span attributes are being added during a request.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import ModelResponse
|
|
|
|
span = MagicMock() # Mocked tracing span to test attribute setting
|
|
|
|
# Construct kwargs to simulate a real LLM request scenario
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "Basic Request Content"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {"user": "test_user"},
|
|
"metadata": {"key_1": "value_1", "key_2": None},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {
|
|
"max_tokens": "100",
|
|
"temperature": "1",
|
|
"top_p": "5",
|
|
"stream": False,
|
|
"user": "test_user",
|
|
"tools": [
|
|
{
|
|
"function": {
|
|
"name": "get_weather",
|
|
"description": "Fetches weather details.",
|
|
"parameters": {
|
|
"type": "object",
|
|
"properties": {
|
|
"location": {
|
|
"type": "string",
|
|
"description": "City name",
|
|
}
|
|
},
|
|
"required": ["location"],
|
|
},
|
|
}
|
|
}
|
|
],
|
|
"functions": [{"name": "get_weather"}, {"name": "get_stock_price"}],
|
|
},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
|
|
# Simulated LLM response object
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 100, "completion_tokens": 60, "prompt_tokens": 40},
|
|
choices=[
|
|
Choices(message={"role": "assistant", "content": "Basic Response Content"})
|
|
],
|
|
model="gpt-4o",
|
|
id="chatcmpl-ID",
|
|
)
|
|
|
|
# Apply attribute setting via ArizeLogger
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
|
|
# Validate that the expected number of attributes were set.
|
|
# OPENINFERENCE_SPAN_KIND is written exactly once (defensively, before
|
|
# the main attribute pipeline) so a partial failure cannot blank it.
|
|
# Per the OpenInference spec, a chat completion that passes `tools=[...]`
|
|
# is still an LLM span — not TOOL (TOOL is reserved for actual tool
|
|
# execution by application code).
|
|
assert span.set_attribute.call_count == 26
|
|
|
|
# Metadata attached to the span
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.METADATA, json.dumps({"key_1": "value_1", "key_2": None})
|
|
)
|
|
|
|
# Basic LLM information
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_MODEL_NAME, "gpt-4o")
|
|
span.set_attribute.assert_any_call("llm.request.type", "completion")
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_PROVIDER, "openai")
|
|
|
|
# LLM generation parameters
|
|
span.set_attribute.assert_any_call("llm.request.max_tokens", "100")
|
|
span.set_attribute.assert_any_call("llm.request.temperature", "1")
|
|
span.set_attribute.assert_any_call("llm.request.top_p", "5")
|
|
|
|
# Streaming and user info
|
|
span.set_attribute.assert_any_call("llm.is_streaming", "False")
|
|
span.set_attribute.assert_any_call("llm.user", "test_user")
|
|
|
|
# Response metadata
|
|
span.set_attribute.assert_any_call("llm.response.id", "chatcmpl-ID")
|
|
span.set_attribute.assert_any_call("llm.response.model", "gpt-4o")
|
|
# Span kind stays LLM even when tools are passed (OpenInference spec).
|
|
span.set_attribute.assert_any_call(SpanAttributes.OPENINFERENCE_SPAN_KIND, "LLM")
|
|
# And TOOL must never be written for an LLM chat completion call.
|
|
span_kind_writes = [
|
|
c.args[1]
|
|
for c in span.set_attribute.call_args_list
|
|
if c.args[0] == SpanAttributes.OPENINFERENCE_SPAN_KIND
|
|
]
|
|
assert "TOOL" not in span_kind_writes
|
|
|
|
# Request message content and metadata
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.INPUT_VALUE, "Basic Request Content"
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_INPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_ROLE}",
|
|
"user",
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_INPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_CONTENT}",
|
|
"Basic Request Content",
|
|
)
|
|
|
|
# Tool call definitions and function names
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_TOOLS}.0.name", "get_weather"
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_TOOLS}.0.description",
|
|
"Fetches weather details.",
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_TOOLS}.0.parameters",
|
|
json.dumps(
|
|
{
|
|
"type": "object",
|
|
"properties": {
|
|
"location": {"type": "string", "description": "City name"}
|
|
},
|
|
"required": ["location"],
|
|
}
|
|
),
|
|
)
|
|
|
|
# Invocation parameters
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.LLM_INVOCATION_PARAMETERS, '{"user": "test_user"}'
|
|
)
|
|
|
|
# User ID
|
|
span.set_attribute.assert_any_call(SpanAttributes.USER_ID, "test_user")
|
|
|
|
# Output message content
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.OUTPUT_VALUE, "Basic Response Content"
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_ROLE}",
|
|
"assistant",
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_CONTENT}",
|
|
"Basic Response Content",
|
|
)
|
|
|
|
# Token counts
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_TOTAL, 100)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_COMPLETION, 60)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_PROMPT, 40)
|
|
|
|
|
|
def test_arize_set_attributes_responses_api():
|
|
"""
|
|
Test setting attributes for Responses API with mixed output (reasoning + message).
|
|
Verifies that multiple output types are correctly handled.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
from litellm.types.llms.openai import (
|
|
ResponsesAPIResponse,
|
|
ResponseAPIUsage,
|
|
OutputTokensDetails,
|
|
)
|
|
from openai.types.responses import (
|
|
ResponseReasoningItem,
|
|
ResponseOutputMessage,
|
|
ResponseOutputText,
|
|
)
|
|
from openai.types.responses.response_reasoning_item import Summary
|
|
|
|
span = MagicMock() # Mocked tracing span to test attribute setting
|
|
|
|
# Construct kwargs to simulate a real LLM request scenario
|
|
kwargs = {
|
|
"model": "o3-mini",
|
|
"messages": [{"role": "user", "content": "What is the answer?"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {"user": "test_user", "stream": True},
|
|
"metadata": {"key_1": "value_1", "key_2": None},
|
|
"call_type": "responses",
|
|
},
|
|
"optional_params": {
|
|
"max_tokens": "100",
|
|
"temperature": "1",
|
|
"top_p": "5",
|
|
"stream": True,
|
|
"user": "test_user",
|
|
},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
|
|
# Simulate Responses API response with mixed output
|
|
response_obj = ResponsesAPIResponse(
|
|
id="response-123",
|
|
created_at=1625247600,
|
|
output=[
|
|
ResponseReasoningItem(
|
|
id="reasoning-001",
|
|
type="reasoning",
|
|
summary=[
|
|
Summary(text="First, I need to analyze...", type="summary_text")
|
|
],
|
|
),
|
|
ResponseOutputMessage(
|
|
id="msg-001",
|
|
type="message",
|
|
role="assistant",
|
|
status="completed",
|
|
content=[
|
|
ResponseOutputText(
|
|
annotations=[],
|
|
text="The answer is 42",
|
|
type="output_text",
|
|
)
|
|
],
|
|
),
|
|
],
|
|
usage=ResponseAPIUsage(
|
|
input_tokens=120,
|
|
output_tokens=250,
|
|
total_tokens=370,
|
|
output_tokens_details=OutputTokensDetails(reasoning_tokens=180),
|
|
),
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
|
|
# Verify reasoning summary was set (index 0)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_REASONING_SUMMARY}",
|
|
"First, I need to analyze...",
|
|
)
|
|
|
|
# Verify message content was set (index 1)
|
|
span.set_attribute.assert_any_call(SpanAttributes.OUTPUT_VALUE, "The answer is 42")
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.1.{MessageAttributes.MESSAGE_CONTENT}",
|
|
"The answer is 42",
|
|
)
|
|
span.set_attribute.assert_any_call(
|
|
f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.1.{MessageAttributes.MESSAGE_ROLE}",
|
|
"assistant",
|
|
)
|
|
|
|
# Verify token counts including reasoning tokens
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_TOTAL, 370)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_COMPLETION, 250)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_PROMPT, 120)
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.LLM_TOKEN_COUNT_COMPLETION_DETAILS_REASONING, 180
|
|
)
|
|
|
|
|
|
def test_set_usage_outputs_pydantic_completion_usage():
|
|
"""
|
|
Regression test for https://github.com/BerriAI/litellm/issues/13672
|
|
|
|
`_set_usage_outputs` previously called `usage.get(...)` which crashes when
|
|
`usage` is a plain Pydantic model (e.g. openai.types.completion_usage.CompletionUsage)
|
|
that does not implement dict-style `.get()`. Same crash for nested
|
|
`output_tokens_details` / `completion_tokens_details`.
|
|
|
|
The function must:
|
|
1. Read total/prompt/completion tokens from a Pydantic usage without `.get`.
|
|
2. Read reasoning_tokens from `completion_tokens_details` (chat completions API)
|
|
OR `output_tokens_details` (responses API), even when those nested objects
|
|
are Pydantic models without `.get`.
|
|
3. Not raise AttributeError; not call span.record_exception.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from openai.types.completion_usage import (
|
|
CompletionTokensDetails,
|
|
CompletionUsage,
|
|
)
|
|
|
|
from litellm.integrations.arize._utils import _set_usage_outputs
|
|
|
|
span = MagicMock()
|
|
|
|
# Plain OpenAI Pydantic model — has no `.get()`
|
|
usage = CompletionUsage(
|
|
completion_tokens=60,
|
|
prompt_tokens=40,
|
|
total_tokens=100,
|
|
completion_tokens_details=CompletionTokensDetails(reasoning_tokens=25),
|
|
)
|
|
assert not hasattr(usage, "get"), "precondition: CompletionUsage must lack .get"
|
|
|
|
response_obj = {"usage": usage}
|
|
|
|
# Must not raise
|
|
_set_usage_outputs(span, response_obj, SpanAttributes)
|
|
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_TOTAL, 100)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_PROMPT, 40)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_COMPLETION, 60)
|
|
# reasoning_tokens for chat completions live in completion_tokens_details
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.LLM_TOKEN_COUNT_COMPLETION_DETAILS_REASONING, 25
|
|
)
|
|
|
|
|
|
def test_set_usage_outputs_pydantic_response_api_usage():
|
|
"""
|
|
Same crash also affects Responses API with `output_tokens_details` as a
|
|
Pydantic model that lacks `.get()`. Verifies the responses-API path.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _set_usage_outputs
|
|
from litellm.types.llms.openai import OutputTokensDetails
|
|
|
|
# Build an object that mimics openai ResponsesAPI usage but lacks `.get`
|
|
# (uses a plain class — not BaseLiteLLMOpenAIResponseObject)
|
|
class PlainResponsesUsage:
|
|
def __init__(self):
|
|
self.total_tokens = 370
|
|
self.input_tokens = 120
|
|
self.output_tokens = 250
|
|
self.output_tokens_details = OutputTokensDetails(reasoning_tokens=180)
|
|
|
|
usage = PlainResponsesUsage()
|
|
assert not hasattr(usage, "get")
|
|
|
|
span = MagicMock()
|
|
response_obj = {"usage": usage}
|
|
|
|
_set_usage_outputs(span, response_obj, SpanAttributes)
|
|
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_TOTAL, 370)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_PROMPT, 120)
|
|
span.set_attribute.assert_any_call(SpanAttributes.LLM_TOKEN_COUNT_COMPLETION, 250)
|
|
span.set_attribute.assert_any_call(
|
|
SpanAttributes.LLM_TOKEN_COUNT_COMPLETION_DETAILS_REASONING, 180
|
|
)
|
|
|
|
|
|
class TestArizeLogger(CustomLogger):
|
|
"""
|
|
Custom logger implementation to capture standard_callback_dynamic_params.
|
|
Used to verify that dynamic config keys are being passed to callbacks.
|
|
"""
|
|
|
|
def __init__(self, *args, **kwargs):
|
|
super().__init__(*args, **kwargs)
|
|
self.standard_callback_dynamic_params: Optional[
|
|
StandardCallbackDynamicParams
|
|
] = None
|
|
|
|
async def async_log_success_event(self, kwargs, response_obj, start_time, end_time):
|
|
# Capture dynamic params and print them for verification
|
|
print("logged kwargs", json.dumps(kwargs, indent=4, default=str))
|
|
self.standard_callback_dynamic_params = kwargs.get(
|
|
"standard_callback_dynamic_params"
|
|
)
|
|
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_arize_dynamic_params():
|
|
"""
|
|
Test to ensure that dynamic Arize keys (API key and space key)
|
|
are received inside the callback logger at runtime.
|
|
"""
|
|
test_arize_logger = TestArizeLogger()
|
|
litellm.callbacks = [test_arize_logger]
|
|
|
|
# Perform a mocked async completion call to trigger logging
|
|
await litellm.acompletion(
|
|
model="gpt-4o",
|
|
messages=[{"role": "user", "content": "Basic Request Content"}],
|
|
mock_response="test",
|
|
arize_api_key="test_api_key_dynamic",
|
|
arize_space_key="test_space_key_dynamic",
|
|
)
|
|
|
|
# Allow for async propagation
|
|
await asyncio.sleep(2)
|
|
|
|
# Assert dynamic parameters were received in the callback
|
|
assert test_arize_logger.standard_callback_dynamic_params is not None
|
|
assert (
|
|
test_arize_logger.standard_callback_dynamic_params.get("arize_api_key")
|
|
== "test_api_key_dynamic"
|
|
)
|
|
assert (
|
|
test_arize_logger.standard_callback_dynamic_params.get("arize_space_key")
|
|
== "test_space_key_dynamic"
|
|
)
|
|
|
|
|
|
def test_construct_dynamic_arize_headers():
|
|
"""
|
|
Test the construct_dynamic_arize_headers method with various input scenarios.
|
|
Ensures that dynamic Arize headers are properly constructed from callback parameters.
|
|
"""
|
|
from litellm.types.utils import StandardCallbackDynamicParams
|
|
|
|
# Test with all parameters present
|
|
dynamic_params_full = StandardCallbackDynamicParams(
|
|
arize_api_key="test_api_key", arize_space_id="test_space_id"
|
|
)
|
|
arize_logger = ArizeLogger()
|
|
|
|
headers = arize_logger.construct_dynamic_otel_headers(dynamic_params_full)
|
|
expected_headers = {"api_key": "test_api_key", "arize-space-id": "test_space_id"}
|
|
assert headers == expected_headers
|
|
|
|
# Test with only space_id
|
|
dynamic_params_space_id_only = StandardCallbackDynamicParams(
|
|
arize_space_id="test_space_id"
|
|
)
|
|
|
|
headers = arize_logger.construct_dynamic_otel_headers(dynamic_params_space_id_only)
|
|
expected_headers = {"arize-space-id": "test_space_id"}
|
|
assert headers == expected_headers
|
|
|
|
# Test with empty parameters dict
|
|
dynamic_params_empty = StandardCallbackDynamicParams()
|
|
|
|
headers = arize_logger.construct_dynamic_otel_headers(dynamic_params_empty)
|
|
assert headers == {}
|
|
|
|
# test with space key and api key
|
|
dynamic_params_space_key_and_api_key = StandardCallbackDynamicParams(
|
|
arize_space_key="test_space_key", arize_api_key="test_api_key"
|
|
)
|
|
headers = arize_logger.construct_dynamic_otel_headers(
|
|
dynamic_params_space_key_and_api_key
|
|
)
|
|
expected_headers = {"arize-space-id": "test_space_key", "api_key": "test_api_key"}
|
|
|
|
|
|
# ---------------------------------------------------------------------------
|
|
# Additive rendering-enhancement tests. None of these assert that previously
|
|
# emitted attributes were removed or changed — they only assert that the new
|
|
# attributes appear in their respective scenarios.
|
|
# ---------------------------------------------------------------------------
|
|
|
|
|
|
def _collect_calls(span):
|
|
"""Helper: return dict[attr_name] = value of all set_attribute calls."""
|
|
out = {}
|
|
for call in span.set_attribute.call_args_list:
|
|
args = call.args
|
|
if len(args) >= 2:
|
|
out[args[0]] = args[1]
|
|
return out
|
|
|
|
|
|
def test_arize_emits_cache_tokens_openai_style():
|
|
"""OpenAI prompt_tokens_details.cached_tokens → cache_read attr."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _set_usage_outputs
|
|
|
|
span = MagicMock()
|
|
response_obj = {
|
|
"usage": {
|
|
"total_tokens": 100,
|
|
"completion_tokens": 60,
|
|
"prompt_tokens": 40,
|
|
"prompt_tokens_details": {"cached_tokens": 32, "audio_tokens": 8},
|
|
}
|
|
}
|
|
_set_usage_outputs(span, response_obj, SpanAttributes)
|
|
attrs = _collect_calls(span)
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_CACHE_READ] == 32
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_AUDIO] == 8
|
|
|
|
|
|
def test_arize_emits_cache_tokens_anthropic_style():
|
|
"""Anthropic/Bedrock cache_read_input_tokens / cache_creation_input_tokens."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _set_usage_outputs
|
|
|
|
span = MagicMock()
|
|
response_obj = {
|
|
"usage": {
|
|
"input_tokens": 100,
|
|
"output_tokens": 50,
|
|
"cache_read_input_tokens": 80,
|
|
"cache_creation_input_tokens": 20,
|
|
}
|
|
}
|
|
_set_usage_outputs(span, response_obj, SpanAttributes)
|
|
attrs = _collect_calls(span)
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_CACHE_READ] == 80
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_CACHE_WRITE] == 20
|
|
|
|
|
|
def test_arize_emits_no_cache_tokens_when_absent():
|
|
"""Regression guard: when no cache fields exist, no cache attrs emitted."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _set_usage_outputs
|
|
|
|
span = MagicMock()
|
|
response_obj = {
|
|
"usage": {"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6}
|
|
}
|
|
_set_usage_outputs(span, response_obj, SpanAttributes)
|
|
attrs = _collect_calls(span)
|
|
assert SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_CACHE_READ not in attrs
|
|
assert SpanAttributes.LLM_TOKEN_COUNT_PROMPT_DETAILS_CACHE_WRITE not in attrs
|
|
|
|
|
|
def test_passthrough_call_type_resolves_to_llm_span_kind():
|
|
"""`allm_passthrough_route` should map to LLM (was UNKNOWN before fix)."""
|
|
from litellm.integrations._types.open_inference import OpenInferenceSpanKindValues
|
|
from litellm.integrations.arize._utils import _infer_open_inference_span_kind
|
|
|
|
assert (
|
|
_infer_open_inference_span_kind("allm_passthrough_route")
|
|
== OpenInferenceSpanKindValues.LLM.value
|
|
)
|
|
assert (
|
|
_infer_open_inference_span_kind("llm_passthrough_route")
|
|
== OpenInferenceSpanKindValues.LLM.value
|
|
)
|
|
|
|
|
|
def test_arize_chat_completion_with_tools_stays_llm_span_kind():
|
|
"""Regression guard against the old `TOOL` override: a chat completion
|
|
that passes `tools=[...]` AND returns `tool_calls` must remain LLM."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "weather?"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {
|
|
"tools": [
|
|
{
|
|
"type": "function",
|
|
"function": {
|
|
"name": "get_weather",
|
|
"description": "weather",
|
|
"parameters": {"type": "object", "properties": {}},
|
|
},
|
|
}
|
|
]
|
|
},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6},
|
|
choices=[
|
|
Choices(
|
|
message={
|
|
"role": "assistant",
|
|
"content": "",
|
|
"tool_calls": [
|
|
{
|
|
"id": "call_x",
|
|
"type": "function",
|
|
"function": {"name": "get_weather", "arguments": "{}"},
|
|
}
|
|
],
|
|
}
|
|
)
|
|
],
|
|
model="gpt-4o",
|
|
id="r-toolkind",
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
span_kind_writes = [
|
|
c.args[1]
|
|
for c in span.set_attribute.call_args_list
|
|
if c.args[0] == SpanAttributes.OPENINFERENCE_SPAN_KIND
|
|
]
|
|
assert span_kind_writes, "span.kind must be written"
|
|
assert all(v == "LLM" for v in span_kind_writes)
|
|
assert "TOOL" not in span_kind_writes
|
|
|
|
|
|
def test_arize_emits_assistant_tool_calls_on_output_message():
|
|
"""Assistant tool_calls should surface as MESSAGE_TOOL_CALLS.* attrs."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "weather?"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6},
|
|
choices=[
|
|
Choices(
|
|
message={
|
|
"role": "assistant",
|
|
"content": "",
|
|
"tool_calls": [
|
|
{
|
|
"id": "call_abc",
|
|
"type": "function",
|
|
"function": {
|
|
"name": "get_weather",
|
|
"arguments": '{"location": "SF"}',
|
|
},
|
|
}
|
|
],
|
|
}
|
|
)
|
|
],
|
|
model="gpt-4o",
|
|
id="chatcmpl-1",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
base = f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_TOOL_CALLS}.0"
|
|
assert attrs[f"{base}.{ToolCallAttributes.TOOL_CALL_ID}"] == "call_abc"
|
|
assert (
|
|
attrs[f"{base}.{ToolCallAttributes.TOOL_CALL_FUNCTION_NAME}"] == "get_weather"
|
|
)
|
|
assert (
|
|
attrs[f"{base}.{ToolCallAttributes.TOOL_CALL_FUNCTION_ARGUMENTS_JSON}"]
|
|
== '{"location": "SF"}'
|
|
)
|
|
|
|
|
|
def test_arize_output_value_falls_back_to_tool_calls_summary():
|
|
"""When the assistant returns no text content but did request tool
|
|
calls, OUTPUT_VALUE should contain a JSON summary so Arize's Output
|
|
pane shows something."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "weather?"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6},
|
|
choices=[
|
|
Choices(
|
|
message={
|
|
"role": "assistant",
|
|
"content": "",
|
|
"tool_calls": [
|
|
{
|
|
"id": "call_abc",
|
|
"type": "function",
|
|
"function": {
|
|
"name": "get_weather",
|
|
"arguments": '{"location": "SF"}',
|
|
},
|
|
}
|
|
],
|
|
}
|
|
)
|
|
],
|
|
model="gpt-4o",
|
|
id="r-tc-out",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
|
|
# OUTPUT_VALUE should contain the tool_call name + arguments JSON
|
|
out = attrs[SpanAttributes.OUTPUT_VALUE]
|
|
assert "tool_calls" in out
|
|
assert "get_weather" in out
|
|
assert "SF" in out
|
|
|
|
|
|
def test_arize_output_value_unchanged_when_content_present():
|
|
"""Regression guard: when content is non-empty, OUTPUT_VALUE must be
|
|
exactly that content (no summary written)."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 4, "completion_tokens": 2, "prompt_tokens": 2},
|
|
choices=[
|
|
Choices(
|
|
message={
|
|
"role": "assistant",
|
|
"content": "hello world",
|
|
"tool_calls": [
|
|
{
|
|
"id": "call_x",
|
|
"type": "function",
|
|
"function": {"name": "n", "arguments": "{}"},
|
|
}
|
|
],
|
|
}
|
|
)
|
|
],
|
|
model="gpt-4o",
|
|
id="r-content",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
assert attrs[SpanAttributes.OUTPUT_VALUE] == "hello world"
|
|
|
|
|
|
def test_arize_emits_tool_call_id_and_name_on_input_tool_message():
|
|
"""A tool-result input message should expose tool_call_id + name."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [
|
|
{"role": "user", "content": "weather?"},
|
|
{
|
|
"role": "assistant",
|
|
"content": "",
|
|
"tool_calls": [
|
|
{
|
|
"id": "call_abc",
|
|
"type": "function",
|
|
"function": {
|
|
"name": "get_weather",
|
|
"arguments": '{"location": "SF"}',
|
|
},
|
|
}
|
|
],
|
|
},
|
|
{
|
|
"role": "tool",
|
|
"tool_call_id": "call_abc",
|
|
"name": "get_weather",
|
|
"content": "sunny, 72F",
|
|
},
|
|
],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6},
|
|
choices=[Choices(message={"role": "assistant", "content": "It's sunny."})],
|
|
model="gpt-4o",
|
|
id="chatcmpl-2",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
# Assistant tool_call surfaces on input msg index 1
|
|
assistant_base = f"{SpanAttributes.LLM_INPUT_MESSAGES}.1.{MessageAttributes.MESSAGE_TOOL_CALLS}.0"
|
|
assert attrs[f"{assistant_base}.{ToolCallAttributes.TOOL_CALL_ID}"] == "call_abc"
|
|
# Tool message at index 2
|
|
tool_prefix = f"{SpanAttributes.LLM_INPUT_MESSAGES}.2"
|
|
assert (
|
|
attrs[f"{tool_prefix}.{MessageAttributes.MESSAGE_TOOL_CALL_ID}"] == "call_abc"
|
|
)
|
|
assert attrs[f"{tool_prefix}.{MessageAttributes.MESSAGE_NAME}"] == "get_weather"
|
|
|
|
|
|
def test_arize_emits_multimodal_input_contents():
|
|
"""List-shaped content should populate MESSAGE_CONTENTS.* alongside the
|
|
legacy MESSAGE_CONTENT (which stays for back-compat)."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [
|
|
{
|
|
"role": "user",
|
|
"content": [
|
|
{"type": "text", "text": "What is in this image?"},
|
|
{
|
|
"type": "image_url",
|
|
"image_url": {"url": "https://example.com/cat.png"},
|
|
},
|
|
],
|
|
}
|
|
],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 10, "completion_tokens": 4, "prompt_tokens": 6},
|
|
choices=[Choices(message={"role": "assistant", "content": "A cat."})],
|
|
model="gpt-4o",
|
|
id="chatcmpl-img",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
base = f"{SpanAttributes.LLM_INPUT_MESSAGES}.0.{MessageAttributes.MESSAGE_CONTENTS}"
|
|
assert attrs[f"{base}.0.message_content.type"] == "text"
|
|
assert attrs[f"{base}.0.message_content.text"] == "What is in this image?"
|
|
assert attrs[f"{base}.1.message_content.type"] == "image"
|
|
assert (
|
|
attrs[f"{base}.1.message_content.image.image.url"]
|
|
== "https://example.com/cat.png"
|
|
)
|
|
|
|
|
|
def test_arize_emits_session_and_user_attrs_from_metadata():
|
|
"""end_user_id → SESSION_ID; user_api_key_user_id → USER_ID (only when
|
|
optional_params.user/model_params.user absent)."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {
|
|
"user_api_key_user_id": "user_42",
|
|
"user_api_key_end_user_id": "session_99",
|
|
"user_api_key_team_id": "team_7",
|
|
"user_api_key_team_alias": "alpha",
|
|
"user_api_key_alias": "key_alpha",
|
|
},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 4, "completion_tokens": 2, "prompt_tokens": 2},
|
|
choices=[Choices(message={"role": "assistant", "content": "hello"})],
|
|
model="gpt-4o",
|
|
id="r1",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
assert attrs[SpanAttributes.SESSION_ID] == "session_99"
|
|
assert attrs[SpanAttributes.USER_ID] == "user_42"
|
|
assert attrs["litellm.team_id"] == "team_7"
|
|
assert attrs["litellm.team_alias"] == "alpha"
|
|
assert attrs["litellm.key_alias"] == "key_alpha"
|
|
|
|
|
|
def test_arize_does_not_use_trace_id_as_session_id_fallback():
|
|
"""SESSION_ID must NOT fall back to trace_id (one session-per-request
|
|
would distort Arize Session analytics). trace_id is emitted under its
|
|
own `litellm.trace_id` key instead.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
"trace_id": "trace-xyz-123",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 4, "completion_tokens": 2, "prompt_tokens": 2},
|
|
choices=[Choices(message={"role": "assistant", "content": "hi"})],
|
|
model="gpt-4o",
|
|
id="r-trace",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
|
|
# SESSION_ID must NOT be derived from trace_id.
|
|
assert SpanAttributes.SESSION_ID not in attrs
|
|
# trace_id surfaces under its own key.
|
|
assert attrs["litellm.trace_id"] == "trace-xyz-123"
|
|
|
|
|
|
def test_arize_does_not_overwrite_user_id_from_optional_params():
|
|
"""If optional_params.user is set, metadata USER_ID must NOT overwrite."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {"user": "from_model_params"},
|
|
"metadata": {"user_api_key_user_id": "from_metadata"},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {"user": "from_optional_params"},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 4, "completion_tokens": 2, "prompt_tokens": 2},
|
|
choices=[Choices(message={"role": "assistant", "content": "hello"})],
|
|
model="gpt-4o",
|
|
id="r2",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
user_id_writes = [
|
|
c.args[1]
|
|
for c in span.set_attribute.call_args_list
|
|
if c.args[0] == SpanAttributes.USER_ID
|
|
]
|
|
assert "from_metadata" not in user_id_writes
|
|
|
|
|
|
def test_arize_emits_response_cost():
|
|
"""StandardLoggingPayload.response_cost → llm.cost.total (+ legacy llm.response.cost)."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
"response_cost": 0.0012345,
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
usage={"total_tokens": 4, "completion_tokens": 2, "prompt_tokens": 2},
|
|
choices=[Choices(message={"role": "assistant", "content": "hello"})],
|
|
model="gpt-4o",
|
|
id="r3",
|
|
)
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
assert attrs["llm.cost.total"] == 0.0012345
|
|
assert attrs["llm.response.cost"] == 0.0012345 # legacy key still emitted
|
|
|
|
|
|
def test_arize_passthrough_bedrock_anthropic_normalization():
|
|
"""Bedrock-Anthropic passthrough: input/output text must be set so the
|
|
span renders something other than raw provider attrs."""
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
bedrock_response_body = {
|
|
"id": "msg_01",
|
|
"type": "message",
|
|
"role": "assistant",
|
|
"content": [{"type": "text", "text": "The capital of France is Paris."}],
|
|
"model": "anthropic.claude-sonnet-4-v1:0",
|
|
"stop_reason": "end_turn",
|
|
"usage": {"input_tokens": 18, "output_tokens": 12},
|
|
}
|
|
|
|
class FakeHttpxResponse:
|
|
"""Minimal httpx.Response stand-in: has `.text` and no `.get`."""
|
|
|
|
def __init__(self, body):
|
|
self.text = json.dumps(body)
|
|
|
|
response_obj = FakeHttpxResponse(bedrock_response_body)
|
|
kwargs = {
|
|
"model": "anthropic.claude-sonnet-4-v1:0",
|
|
"messages": [
|
|
{
|
|
"role": "user",
|
|
"content": json.dumps({"messages": [{"role": "user", "content": "?"}]}),
|
|
}
|
|
],
|
|
"additional_args": {
|
|
"complete_input_dict": {
|
|
"anthropic_version": "bedrock-2023-05-31",
|
|
"max_tokens": 64,
|
|
"messages": [
|
|
{"role": "user", "content": "What is the capital of France?"}
|
|
],
|
|
}
|
|
},
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "allm_passthrough_route",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "bedrock"},
|
|
}
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
attrs = _collect_calls(span)
|
|
|
|
# Input rendering
|
|
assert attrs[SpanAttributes.INPUT_VALUE] == "What is the capital of France?"
|
|
msg0 = f"{SpanAttributes.LLM_INPUT_MESSAGES}.0"
|
|
assert attrs[f"{msg0}.{MessageAttributes.MESSAGE_ROLE}"] == "user"
|
|
assert (
|
|
attrs[f"{msg0}.{MessageAttributes.MESSAGE_CONTENT}"]
|
|
== "What is the capital of France?"
|
|
)
|
|
|
|
# Output rendering (Anthropic content[].text)
|
|
assert attrs[SpanAttributes.OUTPUT_VALUE] == "The capital of France is Paris."
|
|
out0 = f"{SpanAttributes.LLM_OUTPUT_MESSAGES}.0"
|
|
assert attrs[f"{out0}.{MessageAttributes.MESSAGE_ROLE}"] == "assistant"
|
|
assert (
|
|
attrs[f"{out0}.{MessageAttributes.MESSAGE_CONTENT}"]
|
|
== "The capital of France is Paris."
|
|
)
|
|
|
|
# Token counts (Bedrock input_tokens/output_tokens) — extracted via
|
|
# coercion of the non-dict response.
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_PROMPT] == 18
|
|
assert attrs[SpanAttributes.LLM_TOKEN_COUNT_COMPLETION] == 12
|
|
|
|
# Span kind defended even though the call_type is a passthrough variant.
|
|
span_kind_writes = [
|
|
c.args[1]
|
|
for c in span.set_attribute.call_args_list
|
|
if c.args[0] == SpanAttributes.OPENINFERENCE_SPAN_KIND
|
|
]
|
|
assert span_kind_writes # at least one
|
|
assert all(v == "LLM" for v in span_kind_writes)
|
|
|
|
|
|
def test_arize_passthrough_call_type_does_not_run_on_chat_completion():
|
|
"""Guard: passthrough normalizer must not fire for normal chat calls.
|
|
|
|
If it did, it could double-write input/output for ordinary completions.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _maybe_normalize_passthrough
|
|
|
|
span = MagicMock()
|
|
_maybe_normalize_passthrough(
|
|
span,
|
|
{
|
|
"additional_args": {
|
|
"complete_input_dict": {"messages": [{"role": "user", "content": "x"}]}
|
|
}
|
|
},
|
|
{"choices": [{"message": {"role": "assistant", "content": "y"}}]},
|
|
{"choices": [{"message": {"role": "assistant", "content": "y"}}]},
|
|
{"call_type": "completion"},
|
|
)
|
|
assert span.set_attribute.call_count == 0
|
|
|
|
|
|
def test_arize_passthrough_skipped_when_message_redaction_enabled():
|
|
"""Security guard: when message-logging redaction is enabled, the
|
|
passthrough normalizer must NOT export the raw prompt (read from
|
|
`complete_input_dict`, which bypasses central redaction) to the span.
|
|
"""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.integrations.arize._utils import _maybe_normalize_passthrough
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"additional_args": {
|
|
"complete_input_dict": {
|
|
"messages": [
|
|
{"role": "user", "content": "Patient John Doe, SSN 123-45-6789"}
|
|
]
|
|
}
|
|
},
|
|
# Enables redaction via the dynamic-param path inside
|
|
# should_redact_message_logging(), without touching globals.
|
|
"standard_callback_dynamic_params": {"turn_off_message_logging": True},
|
|
}
|
|
_maybe_normalize_passthrough(
|
|
span,
|
|
kwargs,
|
|
{"content": [{"type": "text", "text": "secret response"}]},
|
|
{"content": [{"type": "text", "text": "secret response"}]},
|
|
{"call_type": "allm_passthrough_route"},
|
|
)
|
|
# Nothing — neither input nor output — should be written to the span.
|
|
assert span.set_attribute.call_count == 0
|
|
|
|
|
|
def test_arize_coerce_response_obj_passes_dicts_through_untouched():
|
|
"""Regression guard for the BaseModel/dict path."""
|
|
from litellm.integrations.arize._utils import _coerce_response_obj_for_attrs
|
|
|
|
d = {"id": "x", "model": "m"}
|
|
assert _coerce_response_obj_for_attrs(d) is d
|
|
|
|
class HasGet:
|
|
def get(self, *a, **k): # noqa: D401
|
|
return None
|
|
|
|
obj = HasGet()
|
|
assert _coerce_response_obj_for_attrs(obj) is obj
|
|
|
|
assert _coerce_response_obj_for_attrs(None) is None
|
|
|
|
|
|
def test_arize_coerce_response_obj_parses_httpx_like():
|
|
"""httpx.Response-like objects without `.get` should JSON-decode."""
|
|
from litellm.integrations.arize._utils import _coerce_response_obj_for_attrs
|
|
|
|
class FakeHttpxResponse:
|
|
text = '{"id": "msg_1", "model": "claude"}'
|
|
|
|
parsed = _coerce_response_obj_for_attrs(FakeHttpxResponse())
|
|
assert parsed == {"id": "msg_1", "model": "claude"}
|
|
|
|
|
|
def test_arize_coerce_response_obj_returns_original_on_bad_json():
|
|
from litellm.integrations.arize._utils import _coerce_response_obj_for_attrs
|
|
|
|
class BadJson:
|
|
text = "not-json"
|
|
|
|
obj = BadJson()
|
|
assert _coerce_response_obj_for_attrs(obj) is obj
|
|
|
|
|
|
def test_arize_mcp_call_tool_result_does_not_break_attribute_setting():
|
|
"""`call_mcp_tool` logs the MCP SDK's `CallToolResult`, a Pydantic model
|
|
with no `.get`. It used to raise inside `_set_request_attributes`, aborting
|
|
the whole attribute block (input messages, invocation params, outputs)."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, TextContent
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "MCP: get_weather",
|
|
"standard_logging_object": {
|
|
"model_parameters": {"user": "u-1"},
|
|
"metadata": {},
|
|
"call_type": "call_mcp_tool",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "mcp"},
|
|
}
|
|
response_obj = CallToolResult(
|
|
content=[TextContent(type="text", text="sunny, 21C")], isError=False
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
|
|
span.record_exception.assert_not_called()
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.OPENINFERENCE_SPAN_KIND] == "TOOL"
|
|
assert written["llm.request.type"] == "call_mcp_tool"
|
|
# Emitted after the old crash point, so absent before the fix.
|
|
assert written[SpanAttributes.LLM_INVOCATION_PARAMETERS] == '{"user": "u-1"}'
|
|
assert written[SpanAttributes.USER_ID] == "u-1"
|
|
|
|
|
|
def test_arize_coerce_response_obj_dumps_pydantic_without_get():
|
|
from mcp.types import CallToolResult, TextContent
|
|
|
|
from litellm.integrations.arize._utils import _coerce_response_obj_for_attrs
|
|
|
|
result = CallToolResult(content=[TextContent(type="text", text="hi")], isError=False)
|
|
coerced = _coerce_response_obj_for_attrs(result)
|
|
|
|
assert isinstance(coerced, dict)
|
|
assert coerced["is_error"] is False
|
|
assert coerced["content"][0]["text"] == "hi"
|
|
|
|
|
|
def test_arize_request_attributes_survive_uncoercible_response_obj():
|
|
"""A response object that is neither dict-like nor coercible (binary
|
|
passthrough body, SDK object) must not abort attribute setting."""
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
|
|
class Opaque:
|
|
pass
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, Opaque())
|
|
|
|
span.record_exception.assert_not_called()
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written["llm.provider"] == "openai"
|
|
|
|
|
|
def _mcp_kwargs(mcp_tool_call_metadata=None, **overrides):
|
|
return {
|
|
"model": "MCP: get_weather",
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {
|
|
"mcp_tool_call_metadata": mcp_tool_call_metadata
|
|
or {
|
|
"name": "get_weather",
|
|
"arguments": {"city": "Seoul"},
|
|
"namespaced_tool_name": "weather-mcp/get_weather",
|
|
}
|
|
},
|
|
"call_type": "call_mcp_tool",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "mcp"},
|
|
**overrides,
|
|
}
|
|
|
|
|
|
def test_arize_mcp_tool_span_renders_name_input_and_output():
|
|
"""`call_mcp_tool` spans have no messages/choices, so Input and Output came
|
|
out blank. Render them from mcp_tool_call_metadata + CallToolResult."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, TextContent
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(
|
|
content=[TextContent(type="text", text="sunny, 21C")], isError=False
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.TOOL_NAME] == "get_weather"
|
|
assert written[SpanAttributes.INPUT_VALUE] == '{"city": "Seoul"}'
|
|
assert written[SpanAttributes.INPUT_MIME_TYPE] == "application/json"
|
|
assert written[SpanAttributes.OUTPUT_VALUE] == "sunny, 21C"
|
|
assert written[SpanAttributes.OUTPUT_MIME_TYPE] == "text/plain"
|
|
|
|
|
|
def test_arize_mcp_tool_span_serializes_non_text_content():
|
|
"""Image/resource results have no text part, so fall back to JSON."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, ImageContent
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(
|
|
content=[ImageContent(type="image", data="Zm9v", mimeType="image/png")],
|
|
isError=False,
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.OUTPUT_MIME_TYPE] == "application/json"
|
|
assert "image/png" in written[SpanAttributes.OUTPUT_VALUE]
|
|
|
|
|
|
def test_arize_mcp_tool_span_respects_message_redaction():
|
|
"""Tool arguments and results are user content. With redaction on, only the
|
|
tool name may reach the span."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, TextContent
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(
|
|
content=[TextContent(type="text", text="SSN 123-45-6789")], isError=False
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(
|
|
span,
|
|
_mcp_kwargs(standard_callback_dynamic_params={"turn_off_message_logging": True}),
|
|
response_obj,
|
|
)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.TOOL_NAME] == "get_weather"
|
|
assert SpanAttributes.INPUT_VALUE not in written
|
|
assert SpanAttributes.OUTPUT_VALUE not in written
|
|
|
|
|
|
def test_arize_non_mcp_span_gets_no_tool_name():
|
|
"""The MCP emitter must not fire on ordinary completions."""
|
|
from unittest.mock import MagicMock
|
|
|
|
from litellm.types.utils import Choices, ModelResponse
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "gpt-4o",
|
|
"messages": [{"role": "user", "content": "hi"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {"mcp_tool_call_metadata": {"name": "get_weather"}},
|
|
"call_type": "completion",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "openai"},
|
|
}
|
|
response_obj = ModelResponse(
|
|
choices=[Choices(message={"role": "assistant", "content": "hello"})],
|
|
model="gpt-4o",
|
|
id="r-1",
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert SpanAttributes.TOOL_NAME not in written
|
|
assert written[SpanAttributes.OUTPUT_VALUE] == "hello"
|
|
|
|
|
|
def test_arize_mcp_tool_span_renders_empty_arguments():
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, TextContent
|
|
|
|
span = MagicMock()
|
|
kwargs = _mcp_kwargs(mcp_tool_call_metadata={"name": "ping", "arguments": {}})
|
|
response_obj = CallToolResult(content=[TextContent(type="text", text="pong")], isError=False)
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.INPUT_VALUE] == "{}"
|
|
assert written[SpanAttributes.INPUT_MIME_TYPE] == "application/json"
|
|
|
|
|
|
def test_arize_mcp_tool_span_renders_empty_content():
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(content=[], isError=False)
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.OUTPUT_VALUE] == "[]"
|
|
assert written[SpanAttributes.OUTPUT_MIME_TYPE] == "application/json"
|
|
|
|
|
|
def test_arize_mcp_tool_span_falls_back_to_structured_content():
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(content=[], structuredContent={"temp_c": 21}, isError=False)
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.OUTPUT_VALUE] == '{"temp_c": 21}'
|
|
assert written[SpanAttributes.OUTPUT_MIME_TYPE] == "application/json"
|
|
|
|
|
|
def test_arize_list_mcp_tools_response_does_not_break_attribute_setting():
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "MCP: list_tools",
|
|
"messages": [{"role": "user", "content": "list"}],
|
|
"standard_logging_object": {
|
|
"model_parameters": {},
|
|
"metadata": {},
|
|
"call_type": "list_mcp_tools",
|
|
},
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "mcp"},
|
|
}
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, [{"name": "get_weather"}])
|
|
|
|
span.record_exception.assert_not_called()
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written["llm.input_messages.0.message.content"] == "list"
|
|
|
|
|
|
def test_arize_mcp_tool_span_serializes_mixed_text_and_media():
|
|
from unittest.mock import MagicMock
|
|
|
|
from mcp.types import CallToolResult, ImageContent, TextContent
|
|
|
|
span = MagicMock()
|
|
response_obj = CallToolResult(
|
|
content=[
|
|
TextContent(type="text", text="see image"),
|
|
ImageContent(type="image", data="Zm9v", mimeType="image/png"),
|
|
],
|
|
isError=False,
|
|
)
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), response_obj)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.OUTPUT_MIME_TYPE] == "application/json"
|
|
assert "see image" in written[SpanAttributes.OUTPUT_VALUE]
|
|
assert "image/png" in written[SpanAttributes.OUTPUT_VALUE]
|
|
|
|
|
|
def test_arize_mcp_tool_span_without_response_object_keeps_name_and_input():
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), None)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.TOOL_NAME] == "get_weather"
|
|
assert written[SpanAttributes.INPUT_VALUE] == '{"city": "Seoul"}'
|
|
assert SpanAttributes.OUTPUT_VALUE not in written
|
|
|
|
|
|
def test_arize_mcp_tool_span_without_content_emits_no_output():
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
|
|
ArizeLogger.set_arize_attributes(span, _mcp_kwargs(), {"isError": False})
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert written[SpanAttributes.TOOL_NAME] == "get_weather"
|
|
assert SpanAttributes.OUTPUT_VALUE not in written
|
|
|
|
|
|
def test_arize_mcp_emitter_is_inert_without_a_standard_logging_object():
|
|
from unittest.mock import MagicMock
|
|
|
|
span = MagicMock()
|
|
kwargs = {
|
|
"model": "MCP: get_weather",
|
|
"optional_params": {},
|
|
"litellm_params": {"custom_llm_provider": "mcp"},
|
|
}
|
|
|
|
ArizeLogger.set_arize_attributes(span, kwargs, None)
|
|
|
|
written = {c.args[0]: c.args[1] for c in span.set_attribute.call_args_list}
|
|
assert SpanAttributes.TOOL_NAME not in written
|