Addressed PR comments

Author: kidambisrinivas

transact/transact.go

@@ -62,9 +62,10 @@ func NewClient(opts *ClientOptions) (*Client, error) {
 
 // HashOperation computes the EIP-712 digest of the given operation.
 //   - op: The operation to hash.
-//   - chainId: The chain ID of the blockchain network in which the operation is being executed.
+//   - chainSelector: chainSelector of the blockchain network in which the operation is being executed.
+//
+// Fetches chainID corresponding to the chain selector from smartcontractkit/chain-selectors package.
 func (t *Client) HashOperation(op *types.Operation, chainSelector string) (common.Hash, error) {
-	// Fetches chainID corresponding to the chain selector from smartcontractkit/chain-selectors package.
 	chainSelectorUint, err := strconv.ParseUint(chainSelector, 10, 64)
 	if err != nil {
 		return common.Hash{}, fmt.Errorf("failed to parse chain selector: %w", err)
@@ -98,6 +99,7 @@ func (t *Client) HashOperation(op *types.Operation, chainSelector string) (commo
 //   - ctx: The context for the request.
 //   - op: The operation to sign.
 //   - signer: The signer to use for signing the operation. See signer.Signer for details.
+//   - chainSelector: chainSelector of the blockchain network in which the operation is being executed.
 //
 // Fetches chainID corresponding to the chain selector from smartcontractkit/chain-selectors package.
 func (t *Client) SignOperation(

transact/types/types.go

@@ -83,15 +83,6 @@ func (op *Operation) EIP712Message() apitypes.TypedDataMessage {
 // ChainId is parsed as int64 because go-ethereum's apitypes.TypedDataDomain
 // uses math.HexOrDecimal256 type for ChainID, whose constructor accepts
 // only int64.
-//
-// -------------------------------
-// Non-EVM Chains Compatibility
-// -------------------------------
-//
-// ChainId being int64 instead of string, makes this EIP-712 domain
-// incompatible with non-EVM chains and we'll need to explore other methods
-// to encode Operations, generate hash and sign them for Solana, Cosmos
-// Aptos and Sui chain families.
 func (op *Operation) TypedData(chainId string) (*apitypes.TypedData, error) {
 	chainIdInt, err := strconv.ParseInt(chainId, 10, 64)
 	if err != nil {
@@ -126,15 +117,6 @@ func (op *Operation) TypedData(chainId string) (*apitypes.TypedData, error) {
 //
 // ChainId is int64 because go-ethereum's apitypes.TypedDataDomain uses uses
 // math.HexOrDecimal256 type for ChainID, whose constructor accepts only int64.
-//
-// -------------------------------
-// Non-EVM Chains Compatibility
-// -------------------------------
-//
-// ChainId being int64 instead of string makes this EIP-712 domain
-// incompatible with non-EVM chains, and we'll need to explore other methods
-// to encode Operations, generate hashes, and sign them for Solana, Cosmos,
-// Aptos, and Sui chain families.
 type EIP712Domain struct {
 	Name              string         `json:"name"`
 	Version           string         `json:"version"`