unisat-wallet / dev-support

35 stars 18 forks source link

got issue in send-inscription.test.ts #29

Open cloud6605 opened 6 months ago

cloud6605 commented 6 months ago

image

I'm checking unisat-wallet-sdk, especially send-inscription.test.ts. It is working well. But I just changed the fromBtcWallet & fromAssetWallet address Type. Then I have issue, could you check what is wrong?

send one inscription with lower outputValue: Error: Input #1 has too much or too little data to clean at Object.inputCheckUncleanFinalized (node_modules\bip174\src\lib\utils.js:51:11) at Psbt.clearFinalizedInput (node_modules\bip174\src\lib\psbt.js:120:13) at Psbt._finalizeTaprootInput (node_modules\bitcoinjs-lib\src\psbt.js:356:15) at Psbt.finalizeInput (node_modules\bitcoinjs-lib\src\psbt.js:281:19) at C:\Users\Administrator\Downloads\wallet-sdk\src\wallet\estimate-wallet.ts:176:14 at Array.forEach () at EstimateWallet. (src\wallet\estimate-wallet.ts:174:15) at Generator.next () at fulfilled (src\wallet\estimate-wallet.ts:5:58)

cloud6605 commented 6 months ago

Yes, but in this test, fromBTCWallet and fromAssetWallet addressType is always same. But if 2 wallet addresstypes are different, then unisat-sdk script has issue. Especially, fromBTCWallet(P2SH_P2WKH) and fromAssetWallet(P2TR). Can you please check in this case?

cloud6605 commented 6 months ago

image

ponypick commented 3 months ago

The send-inscription.ts doesn't work for me and shows the same error, when the wallet sign psbt: image

Here is the code when the error occur: await wallet.signPsbt(psbt, { autoFinalized: true, toSignInputs, });