chore: Solana skip preflight only for CCM

[albert-llimos]

Pull Request

Checklist

Please conduct a thorough self-review before opening the PR.

• I am confident that the code works.
• I have written sufficient tests.
• I have written and tested required migrations.
• I have updated documentation where appropriate.

Summary

Following the approach of EVM, where the transaction specifies some of the broadcasting parameters (gas_limit, max_fee_per_gas ...), I've added the skip_preflight to the Solana transactions.
For CCM transactions can revert so we do want to force broadcast it. For non-CCM transactions we expect them to succeed so we'd rather broadcast them only if they succeed (aka not skip preflight).

[codecov]

Codecov Report

Attention: Patch coverage is 17.02128% with 39 lines in your changes missing coverage. Please review.

Project coverage is 72%. Comparing base (c70d72c) to head (48193e6).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
...rc/migrations/solana_transaction_data_migration.rs	0%	25 Missing ⚠️
engine/src/sol/retry_rpc.rs	8%	12 Missing ⚠️
engine/src/state_chain_observer/sc_observer.rs	0%	1 Missing ⚠️
state-chain/chains/src/sol/benchmarking.rs	0%	1 Missing ⚠️

Additional details and impacted files

@@          Coverage Diff           @@
##            main   #5480    +/-   ##
======================================
- Coverage     72%     72%    -0%     
======================================
  Files        494     495     +1     
  Lines      88073   87991    -82     
  Branches   88073   87991    -82     
======================================
- Hits       63191   63017   -174     
- Misses     22305   22395    +90     
- Partials    2577    2579     +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[dandanlen]

Nice - I made a couple of small changes (see commits).