Repository navigation
chore(release): tinywallet 0.29.0 - #129
Conversation
Update the version of the tinywallet crate and all its workspace members from 0.28.0 to 0.29.0, refreshing the lockfiles to match. Auto-committed-on: macbook Co-authored-by: Medulla <medulla@tinyhumans.ai>
Tiny Sweeper reviewReviewed head: Verdict: Ready for maintainer review · Priority: none Diagnostics and supporting detailsFull summaryPull request purpose (title): chore(release): tinywallet 0.29.0. This pull request is a release version bump: every TinyWallet workspace crate moves from 0.28.0 to 0.29.0 so the merged #128 work (the new `NativeWalletTokenBalances` bus member) can be released as a minor version. The description lane noted that the body claims Cargo.lock and proof-workspaces/storage-contract/Cargo.lock changes that are not present in the reviewed diff, which is a description overstating the change rather than a code defect. The latest-push delta is unavailable, so no push-level comparison is asserted. Unreviewed paths
Review snapshot
Completeness: Complete What changedThe root manifest and the bus, client, crypto, module, web3 and x402 crate manifests each change their `version` field from "0.28.0" to "0.29.0", with no other edits in the reviewed diff. The description states the path dependencies are unversioned, so no inter-crate requirements need to move, and that no other 0.28.0 remains in the repo. The description also claims lockfile version-line updates in Cargo.lock and proof-workspaces/storage-contract/Cargo.lock; those files are listed in the pull request's changed files but are not part of the reviewed diff, and the description lane flagged this as the body overstating the change. Features
TestsNo supported feature-to-test mapping was produced. Test execution is not inferred. FindingsNo active actionable findings. Before mergeNone. Agent review detailscritique
security
tests
commits
description
Evidence and run details
|
Codex Review Summary: ✅ CompletedThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
No actionable comments were generated in the recent review. Walkthrough
Merge Risk:
|
Bumps every TinyWallet workspace crate from 0.28.0 to 0.29.0 so the merged #128 work can be released. v0.28.0 is already released, and the release workflow only tags a version that is already committed on main.
Why a minor bump: since v0.28.0, main adds a new bus member,
NativeWalletTokenBalances(SPL token balances across bus, web3 and module, bus contract 1.15, from #128). That is new public surface, so it goes out as a minor release. The only other commit on main since then is a CI action bump (#109).Changes (9 files, +15/−15):
version = "0.29.0"in the root Cargo.toml and in the bus, client, crypto, module, web3 and x402 crates.The path dependencies are unversioned on purpose, so there are no inter-crate requirements to move, and no other
0.28.0is left in the repo. 0.29.0 isn't tagged yet.Validation (local, macOS):
cargo build --locked --workspacepasses, so the lock is unchanged apart from the version lines.cargo test --locked --workspace: 909 passed, 0 failed.cargo metadata --lockedon proof-workspaces/storage-contract passes with the bumped bus line.Known red check:
module-ci / supply-chainfails because of a pre-existing libsqlite3-sys advisory. It is already red on main at 04a059d and at the #128 merge (de4fe10). This PR doesn't touch dependencies.After merge: tag v0.29.0 at the merge commit.
Summary by CodeRabbit