chore: rename to remove sol

[Arrowana]

The field like in the system processor lamports was self explanatory, using transferSol and amount is not

[joncinque]

Makes sense to me! Lamports are the actual amount being transferred, and not SOL. We'll see what @lorisleiva thinks

[lorisleiva]

The idea was that you would use amount: lamports(1_500_000_000n) and could use amount: sol("1.5") in the near future (when I implement @solana/fixed-points) but I suppose lamports: sol("1.5") isn't the end of the world and works better when you provide direct numbers.

The issue is this is a pretty big breaking change and some of our documentation / examples refer to this instruction a lot. I personally think transferSol/amount is not a bad API versus transfer/lamports but I'm happy to merge this and deal with the breaking changes if we think it's worth it.

[joncinque]

Ah I see, that sounds pretty slick actually -- I'll let @Arrowana give his opinion though.

Since the program defines it as transfer and not transferSol I would lean towards the consistency myself.

To avoid a lot of breaking, is it possible to have a transferSol function that just forwards to transfer?

[lorisleiva]

Yes good shout, we could add some manually written aliases with a @deprecated tag on the JS side.