Improving method DX for getMessageProof

[mvares]

Currently, the code doesn't allow the use of both parameters at the same time. To use commitBlockHeight, I need to define commitBlockId as undefined, which doesn't seem to be a comprehensible approach for users.

fuels-ts/packages/account/src/providers/provider.ts

Lines 1605 to 1610 in 5b53581

	async getMessageProof(
	transactionId: string,
	nonce: string,
	commitBlockId?: string,
	commitBlockHeight?: BN
	): Promise<MessageProof | null> {

Therefore, I think it might be interesting if we could define this within an object:

async getMessageProof(
    transactionId: string,
    nonce: string,
    options: { 
      commitBlockId?: string,
      commitBlockHeight?: BN 
     }
  )

[petertonysmith94]

I think this has merit:

• Has better conformity with our approach to options.
• More explicit for the end user.

[mvares]

Makes sense. I think it's better to stick with the current DX.

Thanks for comment, @petertonysmith94!