chore: adopt tx3-sdk 0.14.0#53
Conversation
Bump the `tx3-sdk` pin 0.13.0 -> 0.14.0 to pick up the type-directed argument encoder, so `cshell` (and `trix invoke` through it) serializes an argument of an aggregate type into the TaggedArg wire form. No cshell code change: the encoding happens inside the SDK's `into_resolve_request` on the existing invoke/resolve path. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Changestx3-sdk Dependency Bump
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
What
Bumps the
tx3-sdkdependency0.13.0→0.14.0(just published), socshell— andtrix invoke, which spawns it — can pass an argument of an aggregate type (record,List,Map,Tuple). The SDK serializes it into the TRPTaggedArgwire form.No code change
The encoding happens entirely inside the SDK's
Invocation::into_resolve_request, whichcshellalready calls on the invoke/resolve path (load_args→set_args→into_resolve_request). The breakingParamType::Recordreshape in 0.14.0 doesn't touchcshell, which only matches the scalarParamTypekinds.Verification
cargo build/cargo testgreen (6 passed) against the publishedtx3-sdk 0.14.0.cshellversion bump (that's a separate release step).🤖 Generated with Claude Code
Summary by CodeRabbit