Support updated upstream submodule APIs - #179
Conversation
Updated the pinned commits for the tinyinference, tinytools, and wiki submodules to incorporate upstream changes. Auto-committed-on: dragonfly Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper review
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (6)
🚧 Files skipped from review as they are similar to previous changes (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe changes make tool dialect selection model-aware, filter custom Claude Code records before coalescing, retain provider-extension stream blocks, update rendering and integration tests, adjust CI coverage setup, and advance the tinyinference reference. ChangesHarness behavior updates
Integration and infrastructure alignment
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~25 minutes Change: Feature 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
A rabbit checks the dialect gate Comment |
There was a problem hiding this comment.
Requesting changes: 2 lane(s) blocking, worst finding is high.
Fix or reply to the findings below and push. The next review clears this automatically once they are gone — you should not need to dismiss anything by hand.
$0.1848 · 2,857,288 in / 57,594 out · 138,771 cached (5%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 829 embedded
critique: $0.0941 · 1,480,299 in / 24,041 out · 72,955 cached (5%) · gpt-5.6-luna, deepseek/deepseek-v4-flash
security: $0.0834 · 1,313,443 in / 17,124 out · 46,616 cached (4%) · gpt-5.6-luna
tests: $0.0034 · 28,913 in / 4,796 out · 1,280 cached (4%) · deepseek/deepseek-v4-flash
description: $0.0026 · 16,495 in / 6,294 out · 0 cached (0%) · deepseek/deepseek-v4-flash
# Conflicts: # vendor/tinyinference
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
There was a problem hiding this comment.
The previously-blocking findings are resolved. Clearing the changes request.
$0.0400 · 522,406 in / 39,856 out · 41,473 cached (8%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 991 embedded
critique: $0.0189 · 266,007 in / 11,174 out · 12,433 cached (5%) · gpt-5.6-luna, deepseek/deepseek-v4-flash
security: $0.0145 · 206,948 in / 7,067 out · 5,488 cached (3%) · gpt-5.6-luna
tests: $0.0038 · 35,594 in / 10,096 out · 17,152 cached (48%) · deepseek/deepseek-v4-flash
description: $0.0016 · 7,246 in / 5,481 out · 0 cached (0%) · deepseek/deepseek-v4-flash
Co-authored-by: Medulla <medulla@tinyhumans.ai>
There was a problem hiding this comment.
Requesting changes: 1 lane(s) blocking, worst finding is high.
Fix or reply to the findings below and push. The next review clears this automatically once they are gone — you should not need to dismiss anything by hand.
$0.0058 · 70,909 in / 13,525 out · 29,762 cached (42%) · ladder/vectors, gpt-5.6-luna, deepseek/deepseek-v4-flash · 1,002 embedded
critique: $0.0017 · 26,997 in / 845 out · 4,132 cached (15%) · gpt-5.6-luna
security: $0.0008 · 12,670 in / 531 out · 1,822 cached (14%) · gpt-5.6-luna
tests: $0.0014 · 17,302 in / 6,035 out · 17,152 cached (99%) · deepseek/deepseek-v4-flash
description: $0.0010 · 7,279 in / 2,155 out · 0 cached (0%) · deepseek/deepseek-v4-flash
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Co-authored-by: Medulla <medulla@tinyhumans.ai>
Update the tinyinference and tinytools submodule pointers to their upstream main heads (and the wiki to its upstream default branch), then adapt TinyAgents to the expanded message and tool-content APIs. Preserve media tool output, filter host-only custom messages before provider serialization, and update tests/examples for new required fields and variants.\n\nValidation:\n- cargo fmt --check\n- git diff --check\n- cargo build --workspace --all-targets\n- cargo test --workspace
Summary by CodeRabbit
New Features
Bug Fixes
Tests