Skip to content

graphql: implement estimateGas, gasPrice, storage#21379

Merged
AskAlexSharov merged 2 commits into
mainfrom
lupin012/graphql_impl_estimateGas_gasPrice
May 24, 2026
Merged

graphql: implement estimateGas, gasPrice, storage#21379
AskAlexSharov merged 2 commits into
mainfrom
lupin012/graphql_impl_estimateGas_gasPrice

Conversation

@lupin012

Copy link
Copy Markdown
Contributor
  • Wire Pending resolver into gqlgen (account/call/estimateGas dispatch)
  • Implement blockResolver.EstimateGas and queryResolver.GasPrice
  • Implement accountResolver.Storage (was panic)
  • Move callDataToArgs and helpers to resolver_helpers.go
  • Add nil guard on GasPrice return value
  • Validate IsHexAddress on Storage account address
  • Enforce 0x-prefixed 32-byte hex on Storage slot (Bytes32 schema contract)
  • Add unit tests for EstimateGas, GasPrice, Storage resolvers

lupin012 and others added 2 commits May 23, 2026 20:28
…ve fixes

- Wire Pending resolver into gqlgen (account/call/estimateGas dispatch)
- Implement blockResolver.EstimateGas and queryResolver.GasPrice
- Implement accountResolver.Storage (was panic)
- Move callDataToArgs and helpers to resolver_helpers.go
- Add nil guard on GasPrice return value
- Validate IsHexAddress on Storage account address
- Enforce 0x-prefixed 32-byte hex on Storage slot (Bytes32 schema contract)
- Add unit tests for EstimateGas, GasPrice, Storage resolvers

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@lupin012 lupin012 marked this pull request as ready for review May 23, 2026 20:17
@AskAlexSharov AskAlexSharov added this pull request to the merge queue May 24, 2026
Merged via the queue into main with commit 871ee56 May 24, 2026
284 of 307 checks passed
@AskAlexSharov AskAlexSharov deleted the lupin012/graphql_impl_estimateGas_gasPrice branch May 24, 2026 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants