Skip to content

fix(vendor): restore tinywallet 0.29.0 pin reverted by #8655 - #8656

Merged
senamakel merged 2 commits into
tinyhumansai:mainfrom
senamakel:tinywallet-pin-restore
Nov 11, 2026
Merged

senamakel merged 2 commits into
tinyhumansai:mainfrom
senamakel:tinywallet-pin-restore

Conversation

@senamakel

Copy link
Copy Markdown
Member

Summary

#8655's merge brought in two stray commits that undid #8650's tinywallet re-pin:

  • eddf17b15b "chore(vendor): bump tinywallet submodule" moved vendor/tinywallet from 522121a (v0.29.0) back to 30e92e6 (v0.28.0).
  • e3b24c6271 "chore(deps): downgrade tinywallet crates to 0.28.0" changed the six tinywallet crates in Cargo.lock to 0.28.0.

0.28.0 doesn't have TokenBalanceInfo, NATIVE_WALLET_TOKEN_BALANCES or into_cli_compatible_json, which #8650 uses. As a result, main no longer compiles with the web3 feature:

error[E0432]: unresolved import `tinywallet_bus::wire::TokenBalanceInfo`
error[E0425]: cannot find value `NATIVE_WALLET_TOKEN_BALANCES` in module `methods`
error[E0599]: no method named `into_cli_compatible_json` found for type `_` in the current scope

This PR puts the gitlink back to v0.29.0 (522121a, tag v0.29.0) and restores Cargo.lock to the state #8650 merged with (c369124). The two files differ only in those 12 lockfile lines and the gitlink.

Regression test

The existing test modules::registry::tests::tinywallet_registry_matches_the_verified_v0290_release_manifest pins the version and CONTRACT_VERSION == (1, 15). On main the crate doesn't compile, so the test fails. With this change it passes:

cargo test --locked -p openhuman --lib --features web3 tinywallet_registry_matches
test ...tinywallet_registry_matches_the_verified_v0290_release_manifest ... ok

cargo check --locked --lib --features web3,tinywallet passes.

Update the vendored tinywallet submodule to 0.29.0 and refresh the lockfile so the tinywallet crates resolve to the new version.

Auto-committed-on: macbook
Revert the vendored tinywallet submodule and its lockfile entries from 0.29.0 back to 0.28.0, affecting the core crate and its bus, crypto, module, web3, and x402 companions.

Auto-committed-on: macbook
Copilot AI balanced review requested due to automatic review settings November 11, 2026 08:12
@tinysweeper

tinysweeper Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Tiny Sweeper review

⚠️ Review of 1fa92dae2a27 stopped because the pull request closed or changed.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-11T08:14:09.000373Z 1fa92da PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@senamakel
senamakel merged commit d5a336c into tinyhumansai:main Nov 11, 2026
7 of 9 checks passed
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.

2 participants