Maximum instructions count is too limiting

[lordsteva]

Hey team, is there any specific reason why you stop parsing a transactions if there are more than 4 instructions?

Link: https://github.com/LedgerHQ/app-solana/blob/develop/libsol/message.c#L23

I feel this thing wasn’t changed once you added support for priority fees, and now every transaction effectively has 2 instructions more with prio fee instructions.

We find this too limiting (it was introduced 5 years ago) and suggest bumping this to 6. It would make staking operation signing without blind signing turned on, which would be huge for our users 🙏

Here's an example transactions we'd like supported: https://solscan.io/tx/2GfJeu51Mo5qu8RQ9Jbyd6V3LjBte7zX7mZESSVLA2yScCYXQuTAjw2614RGeo1TBMHaqpGK5ZCja1a6BcdhUANF

List of instructions:
1 - Compute Budget: setComputeUnitPrice
2 - Compute Budget: setComputeUnitLimit
3 - System Program: createAccountWithSeed
4 - Stake Program: initialize
5 - Stake Program: delegate