Skip to content

Fix estimate_gas parameter annotations#44

Open
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/estimate-gas-params
Open

Fix estimate_gas parameter annotations#44
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/estimate-gas-params

Conversation

@mefai-dev
Copy link

Summary

  • Made required/optional annotations explicit for estimate_gas parameters

Type of Change

  • Documentation fix

Changes Made

  • to marked as (required)
  • network marked as (optional, default bsc)
  • value and data annotations clarified

@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR is a documentation-only update to the EVM tools reference, specifically clarifying parameter requirements for estimate_gas. It explicitly marks to as required, refines data as hex-encoded, and states that network is optional with a default of bsc. No runtime code or logic changes were introduced.

Sensitive Content

No sensitive content detected.

Security Issues

No serious security issues detected.


Generated by Hashdit Bot. This tool can absolutely NOT replace manual audits.

## Summary
- estimate_gas parameters lacked explicit required/optional annotations
- Inconsistent with other tool documentation style

## Type of Change
- [x] Documentation fix

## Changes Made
- `to` explicitly marked as (required)
- `network` marked as (optional, default bsc-testnet)
- `value` and `data` annotations clarified with types

## Testing
- [x] Verified against MCP source estimate_gas tool definition
@mefai-dev mefai-dev force-pushed the fix/estimate-gas-params branch from 923c05f to 86670d8 Compare March 13, 2026 23:50
@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR updates documentation for the estimate_gas tool in the EVM tools reference, making parameter requirements explicit. It clarifies that to is required, network is optional with a default of bsc, and improves wording for data to indicate hex encoding. No functional code changes were introduced—this is a documentation-only refinement.

Sensitive Content

No sensitive content detected.

Security Issues

No serious security issues detected.


Generated by Hashdit Bot. This tool can absolutely NOT replace manual audits.

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.

1 participant