Fix transaction serialize/deserialize bugs in ghost wallet.

[yrliou]

Contribute what we have fixed in https://github.com/brave/wallet-standard-brave back to the reference wallet implementation. Most codes are actually from wallet-adapter.

• Always deserialize transactions using VersionedTransaction.deserialize()
• In #signTransaction, specify requireAllSignatures and verifySignatures to false when serializing legacy transactions signed by wallet, because transactions could be partial signed.

[jordaaash]

Looks good, thank you!

Are the type casts for signedTransaction as Transaction required? It looks like isVersionedTransaction should narrow the types properly.

[yrliou]

@jordaaash Seems to be, I was seeing below errors:

[4:09:02 PM] Building project '/Users/yrliou/solana-wallet-standard/packages/wallets/ghost/tsconfig.cjs.json'...

src/wallet.ts:242:41 - error TS2339: Property 'serialize' does not exist on type 'never'.

242                       signedTransaction.serialize({
                                            ~~~~~~~~~

[4:09:04 PM] Project 'tsconfig.esm.json' is out of date because output file 'lib/esm/account.js' does not exist

[4:09:04 PM] Building project '/Users/yrliou/solana-wallet-standard/packages/wallets/ghost/tsconfig.esm.json'...

src/wallet.ts:242:41 - error TS2339: Property 'serialize' does not exist on type 'never'.

242                       signedTransaction.serialize({
                                            ~~~~~~~~~