chore(deps): bump the vendored tinyhumans-sdk pin - #5632
Conversation
`vendor/tinyhumans-sdk` sat at 1cd5dee (the optional Socket.IO transport, tinyhumansai/sdk#13). Move it to 83ab7b1, picking up: - tinyhumansai/sdk#14, tinyhumansai#15 — the spend-cap routes and their spec sync - tinyhumansai/sdk#16 — `/agent-integrations/*` split one module per provider, a verified pure move with `api::agent_integration_types` kept as a re-export shim This crate imports `tinyhumans_sdk::api::types` and `tinyhumans_sdk::jwt` only, neither of which the split touches, and the SDK is taken with `default-features = false`, so the socket feature stays off. Gitlink only — Cargo.lock is unchanged, since the crate version and its dependency set did not move. Co-authored-by: Medulla <medulla@tinyhumans.ai>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe ChangesSDK Submodule Update
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This PR only updates the vendored SDK pin and does not change repository source or intended runtime behavior; no actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
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 |
|
The two red checks are pre-existing breakage on
The latest The failures are memory-module family/capability mismatches: The bound driver advertises 13 families where the tests expect 18. Nothing in that path is reachable from the SDK: this PR's entire diff is one gitlink line, and this crate imports only Locally against the new pin, with
I have not tried to fix the family mismatch here — it is unrelated to this change and wants its own PR against the memory-module pin rather than being smuggled in behind a dependency bump. Flagging it because it is currently blocking every PR into |
PR Babysitter statusHead: Root cause confirmed: the two failing checks (
Fix pushed as a separate PR: #5639 — Next action: #5639 needs to go green and be merged to |
Summary
vendor/tinyhumans-sdk:1cd5dee→83ab7b1. Gitlink only — one line, no source and no lockfile change./agent-integrations/*surface split one module per provider).Problem
The vendored SDK pin sat at tinyhumansai/sdk#13 (the optional Socket.IO transport) while the SDK's
mainmoved three commits ahead. The largest of those, #16, restructures the agent-integration surface from two flat files intosrc/api/agent_integrations/with one module per provider — the kind of change that looks alarming on a vendored dependency, so it is worth stating exactly what it does and does not do.Solution
Move the gitlink and nothing else.
The restructure is a verified pure move: the SDK's 102 public types and 62 public methods are unchanged, no source line was altered, and
api::agent_integration_typesremains as a re-export shim precisely so vendored consumers like this one keep resolving their existing imports.This crate's exposure is narrower still. Its only SDK imports are:
tinyhumans_sdk::api::typestinyhumans_sdk::jwt/jwt::decode_jwt_exp_unixNeither is in the moved surface. The dependency is also declared
default-features = false, so the socket feature stays off.Submission Checklist
83ab7b1carries its own suite — 145 tests passing, plus a newagent_integration_module_layouttest pinning both the per-provider paths and the compat shim.Cargo.lockneeded no update.Impact
Desktop/CLI only, and no runtime behavior change. The pin also carries the SDK's
post_multipartContent-Typefix (tinyhumansai/sdk#11, already in at1cd5dee's ancestry) — relevant because SentryTAURI-RUST-QBNstayed open client-side pending exactly this kind of re-pin.No migration, no compatibility break: the SDK's public API is identical before and after.
Related
ComposioConnectionsResponsesilently decodes an unrecognized object shape to zero connections. Not reachable from this crate today.AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
chore/bump-sdk-gitlink5c5a4d5b94555f4c04bc8f1eddae78fd5e064de9Validation Run
pnpm --filter openhuman-app format:check— no app/TS files changedpnpm typecheck— no TS files changedcargo check --locked --libclean against the new pin (Checking openhuman v0.63.14…Finished), andcargo check -p tinyhumans-sdk --lockedclean.Cargo.lockconfirmed untouched after both.83ab7b1:cargo fmt --all -- --check,cargo clippy --all-targets -- -D warnings,cargo test(145 passed) andcargo packageall green.Validation Blocked
command:N/Aerror:N/Aimpact:N/ABehavior Changes
Parity Contract
agent_integration_typesre-export shim preserves the pre-split import path.Duplicate / Superseded PR Handling
Summary by CodeRabbit