Skip to content

feat: add Movement#2

Open
ctoyan wants to merge 17 commits intomainfrom
feat/impl-movement
Open

feat: add Movement#2
ctoyan wants to merge 17 commits intomainfrom
feat/impl-movement

Conversation

@ctoyan
Copy link
Member

@ctoyan ctoyan commented Nov 20, 2025

No description provided.

@ctoyan ctoyan marked this pull request as draft November 20, 2025 16:25
"@wdk/wallet-tron": "https://github.com/tetherto/wdk-wallet-tron#ec076473a5b8340599e005ab5ece8c8d246b3965",
"@wdk/wallet-tron-gasfree": "https://github.com/tetherto/wdk-wallet-tron-gasfree#fd57ed375801ae281e98c6882ea0f2ad4c2b30f5",
"bare-ipc": "1.1.1",
"bufferutil": "^4.0.9",
Copy link
Member Author

Choose a reason for hiding this comment

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

bare requiresbufferutil and utf-8-validate in order to generate the bundle. Otherwise we get this error:

$ npm run gen:mobile-bundle

> @wdk/bare@0.1.1 gen:mobile-bundle
> npx bare-pack --target ios-arm64 --target ios-arm64-simulator --target ios-x64-simulator --target android-arm --target android-arm64 --target android-ia32 --target android-x64 --linked --imports ./pack.imports.json --out bundle/wdk-worklet.mobile.bundle.js src/wdk-worklet.js

/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/paparam/index.js:518
    throw new Bail(bail.reason)
          ^

Bail: ModuleTraverseError: MODULE_NOT_FOUND: Cannot find module 'bufferutil' imported from 'file:///Users/stoyankirov/projects/work/wdk/pear-wrk-wdk/node_modules/ethers/node_modules/ws/lib/buffer-util.js'
    at exports.imports (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/bare-module-traverse/index.js:460:24)
    at exports.imports.next (<anonymous>)
    at exports.module (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/bare-module-traverse/index.js:236:20)
    at exports.module.next (<anonymous>)
    at process (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/bare-pack/index.js:51:26)
    at async Promise.all (index 10)
    at async process (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/bare-pack/index.js:84:5)
    at async Promise.all (index 2)
    at async process (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/bare-pack/index.js:84:5)
    at async Promise.all (index 1)
    at Command._bail (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/paparam/index.js:518:11)
    at Command.bail (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/paparam/index.js:127:36)
    at runAsync (/Users/stoyankirov/.npm/_npx/8cbcd94b89e309a2/node_modules/paparam/index.js:806:7)

Node.js v25.1.0

@ctoyan ctoyan self-assigned this Nov 20, 2025
@ctoyan ctoyan marked this pull request as ready for review November 20, 2025 16:28
package.json Outdated
"@wdk/wallet-btc": "https://github.com/tetherto/wdk-wallet-btc#ed35de0b49344bb07736fc296a28b8465a8e8a6e",
"@wdk/wallet-evm": "https://github.com/tetherto/wdk-wallet-evm#74e281ea63f7df6e2590580bba988daeda021df6",
"@wdk/wallet-evm-erc-4337": "https://github.com/tetherto/wdk-wallet-evm-erc-4337#cc4431332b86d9c0609eaf833daa7295c13d8f86",
"@wdk/wallet-movement": "github:eigerco/wdk-wallet-movement#49a3d85f63e8a7f064b07666e5c58d3a41b0f5fe",
Copy link
Member Author

@ctoyan ctoyan Nov 20, 2025

Choose a reason for hiding this comment

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

This commit should be changed to point to the latest commit in the repo before mergingRepo might change as well.

"@wdk/wallet-btc": "https://github.com/tetherto/wdk-wallet-btc#ed35de0b49344bb07736fc296a28b8465a8e8a6e",
"@wdk/wallet-evm": "https://github.com/tetherto/wdk-wallet-evm#74e281ea63f7df6e2590580bba988daeda021df6",
"@wdk/wallet-evm-erc-4337": "https://github.com/tetherto/wdk-wallet-evm-erc-4337#cc4431332b86d9c0609eaf833daa7295c13d8f86",
"@wdk/wallet-movement": "github:eigerco/wdk-wallet-movement#38ae6c78c18bf180d36cc14fddb66cfdcf6eec92",
Copy link
Member Author

Choose a reason for hiding this comment

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

Should be pinned to latest main commit and re-tested before merge.

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