Skip to content

Revert "fix(gemini): dereference $defs/$ref in tool response content"#19173

Merged
uc4w6c merged 1 commit intomainfrom
revert-19062-gemini-defsrefs
Jan 16, 2026
Merged

Revert "fix(gemini): dereference $defs/$ref in tool response content"#19173
uc4w6c merged 1 commit intomainfrom
revert-19062-gemini-defsrefs

Conversation

@uc4w6c
Copy link
Contributor

@uc4w6c uc4w6c commented Jan 16, 2026

Reverts #19062

I reverted this change because the following command, which runs in CI, failed:

python ./tests/code_coverage_tests/recursive_detector.py

Error detail:

🚨 Unignored recursive functions found in ./litellm/litellm_core_utils/prompt_templates/factory.py: ['_clean_refs_for_gemini']. THIS IS REALLY BAD, it has caused CPU Usage spikes in the past. Only keep this if it's ABSOLUTELY necessary.
Traceback (most recent call last):
  File "/home/circleci/project/litellm/./****s/code_coverage_****s/recursive_detector.py", line 123, in <module>
    raise Exception(
Exception: 🚨 Unignored recursive functions found include ./litellm/litellm_core_utils/prompt_templates/factory.py: ['_clean_refs_for_gemini']. THIS IS REALLY BAD, it has caused CPU Usage spikes in the past. Only keep this if it's ABSOLUTELY necessary.

@vercel
Copy link

vercel bot commented Jan 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
litellm Building Building Preview, Comment Jan 16, 2026 1:58am

Review with Vercel Agent

@uc4w6c uc4w6c merged commit 1d16029 into main Jan 16, 2026
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant