feat(chain): introduce tron KMS signer (#272)

To introduce kms signer for TRON, i have to perform a interface refactor
so the existing generator does not return a keystore but instead a sign
hash function as KMS does not expose the keystore.

Will get some people from tron side to review this.

Author: graham-chainlink

.changeset/common-bats-film.md

@@ -0,0 +1,5 @@
+---
+"chainlink-deployments-framework": minor
+---
+
+Introdce KMS signer for TRON

chain/tron/provider/account_generator.go

@@ -30,10 +30,10 @@ import (
 
 // CTFChainProviderConfig holds the configuration to initialize the CTFChainProvider.
 type CTFChainProviderConfig struct {
-	// Required: A generator for the deployer account. Use AccountGenCTFDefault to
-	// create a deployer account from the default CTF account. Alternatively, you can use
-	// AccountRandom to create a new random account.
-	DeployerAccountGen AccountGenerator
+	// Required: A generator for the deployer signer. Use SignerGenCTFDefault to
+	// create a deployer signer from the default CTF account. Alternatively, you can use
+	// SignerRandom to create a new random signer.
+	DeployerSignerGen SignerGenerator
 
 	// Required: A sync.Once instance to ensure that the CTF framework only sets up the new
 	// DefaultNetwork once
@@ -42,8 +42,8 @@ type CTFChainProviderConfig struct {
 
 // validate checks whether the configuration contains all required values.
 func (c CTFChainProviderConfig) validate() error {
-	if c.DeployerAccountGen == nil {
-		return errors.New("deployer account generator is required")
+	if c.DeployerSignerGen == nil {
+		return errors.New("deployer signer generator is required")
 	}
 	if c.Once == nil {
 		return errors.New("sync.Once instance is required")
@@ -121,23 +121,23 @@ func (p *CTFChainProvider) Initialize(_ context.Context) (chain.BlockChain, erro
 		return nil, fmt.Errorf("failed to create combined client: %w", err)
 	}
 
-	// Generate deployer keystore and address
-	deployerKeystore, deployerAddr, err := p.config.DeployerAccountGen.Generate()
+	// Get deployer address from the signer generator
+	deployerAddr, err := p.config.DeployerSignerGen.GetAddress()
 	if err != nil {
-		return nil, fmt.Errorf("failed to generate deployer account: %w", err)
+		return nil, fmt.Errorf("failed to get deployer address: %w", err)
 	}
 
-	// Initialize local RPC client wrapper that uses the keystore for signing
-	client := rpcclient.New(combinedClient, deployerKeystore, deployerAddr)
+	// Initialize local RPC client wrapper that uses the signer generator's signing function
+	client := rpcclient.New(combinedClient, p.config.DeployerSignerGen.Sign)
 
 	// Construct and cache the Tron chain instance with helper methods for deploying and interacting with contracts
 	p.chain = &tron.Chain{
 		ChainMetadata: tron.ChainMetadata{
 			Selector: p.selector,
 		},
-		Client:   combinedClient,   // Underlying client for Tron node communication
-		Keystore: deployerKeystore, // Keystore for signing transactions
-		Address:  deployerAddr,     // Default "from" address for transactions
+		Client:   combinedClient,                  // Underlying client for Tron node communication
+		SignHash: p.config.DeployerSignerGen.Sign, // Function for signing transactions
+		Address:  deployerAddr,                    // Default "from" address for transactions
 		URL:      fullNodeURL,
 		// Helper for sending and confirming transactions
 		SendAndConfirm: func(ctx context.Context, tx *common.Transaction, opts *tron.ConfirmRetryOptions) (*soliditynode.TransactionInfo, error) {

chain/tron/provider/account_generator_test.go

@@ -25,20 +25,20 @@ func TestCTFChainProviderConfig_validate(t *testing.T) {
 		{
 			name:        "empty config",
 			config:      CTFChainProviderConfig{},
-			expectedErr: "deployer account generator is required",
+			expectedErr: "deployer signer generator is required",
 		},
 		{
 			name: "missing sync.Once",
 			config: CTFChainProviderConfig{
-				DeployerAccountGen: AccountGenCTFDefault(),
+				DeployerSignerGen: SignerGenCTFDefault(),
 			},
 			expectedErr: "sync.Once instance is required",
 		},
 		{
 			name: "valid config",
 			config: CTFChainProviderConfig{
-				DeployerAccountGen: AccountGenCTFDefault(),
-				Once:               &sync.Once{},
+				DeployerSignerGen: SignerGenCTFDefault(),
+				Once:              &sync.Once{},
 			},
 			expectedErr: "",
 		},
@@ -62,8 +62,8 @@ func TestNewCTFChainProvider(t *testing.T) {
 	t.Parallel()
 
 	config := CTFChainProviderConfig{
-		DeployerAccountGen: AccountGenCTFDefault(),
-		Once:               &sync.Once{},
+		DeployerSignerGen: SignerGenCTFDefault(),
+		Once:              &sync.Once{},
 	}
 
 	provider := NewCTFChainProvider(t, 123456, config)
@@ -87,8 +87,8 @@ func TestCTFChainProvider_Initialize(t *testing.T) {
 			name:         "valid initialization",
 			giveSelector: chain_selectors.TRON_TESTNET_NILE.Selector,
 			giveConfig: CTFChainProviderConfig{
-				DeployerAccountGen: AccountGenCTFDefault(),
-				Once:               &sync.Once{},
+				DeployerSignerGen: SignerGenCTFDefault(),
+				Once:              &sync.Once{},
 			},
 		},
 		{
@@ -97,22 +97,22 @@ func TestCTFChainProvider_Initialize(t *testing.T) {
 			giveConfig: CTFChainProviderConfig{
 				Once: &sync.Once{},
 			},
-			wantErr: "deployer account generator is required",
+			wantErr: "deployer signer generator is required",
 		},
 		{
 			name:         "missing sync.Once",
 			giveSelector: chain_selectors.TRON_TESTNET_NILE.Selector,
 			giveConfig: CTFChainProviderConfig{
-				DeployerAccountGen: AccountGenCTFDefault(),
+				DeployerSignerGen: SignerGenCTFDefault(),
 			},
 			wantErr: "sync.Once instance is required",
 		},
 		{
 			name:         "chain id not found for selector",
 			giveSelector: 999999, // Invalid selector
 			giveConfig: CTFChainProviderConfig{
-				DeployerAccountGen: AccountGenCTFDefault(),
-				Once:               &sync.Once{},
+				DeployerSignerGen: SignerGenCTFDefault(),
+				Once:              &sync.Once{},
 			},
 			wantErr: "failed to get chain ID from selector 999999",
 		},
@@ -136,7 +136,7 @@ func TestCTFChainProvider_Initialize(t *testing.T) {
 				require.True(t, ok, "expected got to be of type tron.Chain")
 				require.Equal(t, tt.giveSelector, gotChain.Selector)
 				require.NotEmpty(t, gotChain.Client)
-				require.NotEmpty(t, gotChain.Keystore)
+				require.NotEmpty(t, gotChain.SignHash)
 				require.NotEmpty(t, gotChain.Address)
 				require.NotEmpty(t, gotChain.URL)
 				require.NotEmpty(t, gotChain.SendAndConfirm)
@@ -150,8 +150,8 @@ func TestCTFChainProvider_Initialize(t *testing.T) {
 func TestCTFChainProvider_ContainerStartup(t *testing.T) {
 	t.Parallel()
 	config := CTFChainProviderConfig{
-		DeployerAccountGen: AccountGenCTFDefault(),
-		Once:               &sync.Once{},
+		DeployerSignerGen: SignerGenCTFDefault(),
+		Once:              &sync.Once{},
 	}
 
 	provider := NewCTFChainProvider(t, chain_selectors.TRON_TESTNET_NILE.Selector, config)
@@ -169,8 +169,8 @@ func TestCTFProvider_SendAndConfirmTx_And_CheckContractDeployed(t *testing.T) {
 	t.Parallel()
 
 	config := CTFChainProviderConfig{
-		DeployerAccountGen: AccountGenCTFDefault(),
-		Once:               &sync.Once{},
+		DeployerSignerGen: SignerGenCTFDefault(),
+		Once:              &sync.Once{},
 	}
 
 	chainSelector := chain_selectors.TRON_TESTNET_NILE.Selector