Skip to content

Avoid frequent eth_chainId calls#2380

Closed
Adamantios wants to merge 4 commits intomainfrom
feat/cache-chain-id
Closed

Avoid frequent eth_chainId calls#2380
Adamantios wants to merge 4 commits intomainfrom
feat/cache-chain-id

Conversation

@Adamantios
Copy link
Collaborator

Proposed changes

Caches the chain ID to reduce repeated eth_chainId requests, preventing unnecessary RPC calls and associated costs.

Fixes

n/a

Types of changes

What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

  • Non-breaking fix (non-breaking change which fixes an issue)
  • Breaking fix (breaking change which fixes an issue)
  • Non-breaking feature (non-breaking change which adds functionality)
  • Breaking feature (breaking change which adds functionality)
  • Refactor (non-breaking change which changes implementation)
  • Messy (mixture of the above - requires explanation!)

Checklist

Put an x in the boxes that apply.

  • I have read the CONTRIBUTING doc
  • I am making a pull request against the main branch (left side). Also you should start your branch off our main.
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have locally run AI agents that could be impacted and they do not present failures derived from my changes
  • Public-facing documentation has been updated with the changes affected by this PR. Even if the provided contents are not in their final form, all significant information must be included.
  • Any backwards-incompatible/breaking change has been clearly documented in the upgrading document.

Further comments

n/a

@Adamantios Adamantios added the enhancement New feature or request label Jan 8, 2026
@Adamantios Adamantios force-pushed the feat/cache-chain-id branch 2 times, most recently from c5c068c to 2914f25 Compare January 8, 2026 18:37
@Adamantios Adamantios requested a review from OjusWiZard January 8, 2026 18:49
@Adamantios
Copy link
Collaborator Author

Closing as the chain id is now cached in open-aea@v2.0.8.

@Adamantios Adamantios closed this Jan 20, 2026
@Adamantios Adamantios deleted the feat/cache-chain-id branch January 20, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant