[MISC] Bump tool versions for litellm 1.82.3 upgrade#1871
[MISC] Bump tool versions for litellm 1.82.3 upgrade#1871chandrasekharan-zipstack merged 1 commit intomainfrom
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
Summary by CodeRabbit
WalkthroughVersion bumps across multiple tool configurations. Updates include Structure Tool (0.0.98→0.0.99), Classifier Tool (0.0.77→0.0.78), and Text Extractor Tool (0.0.73→0.0.74) across environment variables, individual tool configurations, and the centralized tool registry. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Test ResultsSummary
Runner Tests - Full Report
SDK1 Tests - Full Report
|
|
Greptile SummaryThis PR bumps the tool versions for
Confidence Score: 5/5
Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Tools["Tool Version Sources"]
A["tools/classifier/src/config/properties.json\ntoolVersion: 0.0.78"]
B["tools/structure/src/config/properties.json\ntoolVersion: 0.0.99"]
C["tools/text_extractor/src/config/properties.json\ntoolVersion: 0.0.74"]
end
subgraph Registry["Tool Registry / Config"]
D["public_tools.json\nclassifier → image_tag: 0.0.78\ntext_extractor → image_tag: 0.0.74"]
E["backend/sample.env\nSTRUCTURE_TOOL_IMAGE_TAG=0.0.99\nSTRUCTURE_TOOL_IMAGE_URL=docker:unstract/tool-structure:0.0.99"]
end
subgraph Runtime["Runtime Resolution"]
F["Tool Registry\n(classifier, text_extractor)"]
G["Django Settings\n(STRUCTURE_TOOL_IMAGE_*)"]
end
A -- "classifier tool image" --> D
C -- "text_extractor tool image" --> D
B -- "structure tool image" --> E
D --> F
E --> G
F --> H["Workflow Execution"]
G --> H
Reviews (1): Last reviewed commit: "[MISC] Bump tool versions for litellm 1...." | Re-trigger Greptile |
chandrasekharan-zipstack
left a comment
There was a problem hiding this comment.
@pk-zipstack have the tools been built? Did you already update the sdk1 version for these?



What
Why
How
toolVersionin each tool'sproperties.jsonimage_urlandimage_taginpublic_tools.jsonSTRUCTURE_TOOL_IMAGE_URLandSTRUCTURE_TOOL_IMAGE_TAGinbackend/sample.envCan this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
Database Migrations
Env Config
STRUCTURE_TOOL_IMAGE_URLandSTRUCTURE_TOOL_IMAGE_TAGupdated to0.0.99inbackend/sample.envRelevant Docs
Related Issues or PRs
Dependencies Versions
Notes on Testing
Screenshots
N/A
Checklist
I have read and understood the Contribution Guidelines.