Reduce interface scope to enable KMS

[connorwstein]

The full keystore (with Admin which has write capabilities) isn't needed, we can reduce scope to readonly and that allows us to plug in KMS impls which can't have an Admin implementation.

Note while technically a breaking change this won't break callers

[github-actions]

⚠️ API Diff Results - Breaking changes detected

📦 Module: github-com-smartcontractkit-chainlink-evm

🔴 Breaking Changes (2)

pkg/keys/v2 (2)

• GetTxKeys — Type changed:

func(
  context.Context, 
  - github.com/smartcontractkit/chainlink-common/keystore.Keystore, 
  + interface{github.com/smartcontractkit/chainlink-common/keystore.Reader; github.com/smartcontractkit/chainlink-common/keystore.Signer}, 
  []string
)
([]*TxKey, error)

• NewTxKeyCoreKeystore — Type changed:

func(
  - github.com/smartcontractkit/chainlink-common/keystore.Keystore
  + interface{github.com/smartcontractkit/chainlink-common/keystore.Reader; github.com/smartcontractkit/chainlink-common/keystore.Signer}, 
  + ...Option
)
*TxKeyCoreKeystore

---

📄 View full apidiff report