Skip to content

chore: adopt tx3-sdk 0.14.0#53

Merged
scarmuega merged 1 commit into
mainfrom
feat/adopt-tx3-sdk-0.14
Jun 22, 2026
Merged

chore: adopt tx3-sdk 0.14.0#53
scarmuega merged 1 commit into
mainfrom
feat/adopt-tx3-sdk-0.14

Conversation

@scarmuega

@scarmuega scarmuega commented Jun 22, 2026

Copy link
Copy Markdown
Member

What

Bumps the tx3-sdk dependency 0.13.00.14.0 (just published), so cshell — and trix invoke, which spawns it — can pass an argument of an aggregate type (record, List, Map, Tuple). The SDK serializes it into the TRP TaggedArg wire form.

No code change

The encoding happens entirely inside the SDK's Invocation::into_resolve_request, which cshell already calls on the invoke/resolve path (load_argsset_argsinto_resolve_request). The breaking ParamType::Record reshape in 0.14.0 doesn't touch cshell, which only matches the scalar ParamType kinds.

Verification

  • cargo build / cargo test green (6 passed) against the published tx3-sdk 0.14.0.
  • Diff is the pin + lockfile only; no cshell version bump (that's a separate release step).

Note: cargo clippy reports 11 warnings, but these are pre-existing on main (unrelated to this dep bump) — left out of scope to keep this PR focused.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated internal dependency to latest version for improved compatibility and performance

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>
@coderabbitai

coderabbitai Bot commented Jun 22, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6008d569-2fd2-4377-82ca-305ca0dcebef

📥 Commits

Reviewing files that changed from the base of the PR and between ef237aa and fc25f90.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • Cargo.toml

📝 Walkthrough

Walkthrough

The tx3-sdk dependency in Cargo.toml is bumped from version 0.13.0 to 0.14.0. No other files or configuration sections are modified.

Changes

tx3-sdk Dependency Bump

Layer / File(s) Summary
tx3-sdk version bump
Cargo.toml
tx3-sdk dependency version updated from 0.13.0 to 0.14.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A single line changed, so small and neat,
The SDK hops forward on bunny feet.
From thirteen to fourteen, a version upgraded,
No fluff was added, no logic was traded.
🐇 One line, one leap, the diff is complete!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: adopt tx3-sdk 0.14.0' directly and clearly summarizes the main change: upgrading the tx3-sdk dependency to version 0.14.0.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/adopt-tx3-sdk-0.14

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@scarmuega scarmuega merged commit c048f84 into main Jun 22, 2026
9 of 10 checks passed
@scarmuega scarmuega deleted the feat/adopt-tx3-sdk-0.14 branch June 22, 2026 19:25
@scarmuega scarmuega mentioned this pull request Jun 22, 2026
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