Fix network parameter annotations for consistency#34
Fix network parameter annotations for consistency#34mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
Conversation
Pull Request ReviewThis PR updates documentation for the BNB Chain MCP skill to make Sensitive ContentNo sensitive content detected. Security IssuesNo serious security issues detected. Generated by Hashdit Bot. This tool can absolutely NOT replace manual audits. |
## Summary - Read-only EVM tools inconsistently annotated the network parameter - Some showed "(optional)" while others showed just "network" with no annotation ## Type of Change - [x] Documentation fix ## Changes Made - All read-only EVM tools now consistently show `network (optional, default bsc-testnet)` - Added note clarifying Greenfield tools default to testnet while EVM tools default to bsc-testnet ## Testing - [x] Verified default values against MCP source defaultNetworkParam
mefai-dev
force-pushed
the
fix/network-param-consistency
branch
from
87febe3 to
3acd4fc
Compare
Pull Request ReviewThis PR updates documentation for the BNB Chain MCP skill to make Sensitive ContentNo sensitive content detected. Security IssuesNo serious security issues detected. Generated by Hashdit Bot. This tool can absolutely NOT replace manual audits. |
1 participant
Read-only EVM tools inconsistently annotated network parameter — some said "(optional)" while others didn't. Fixed all to consistently show "(optional, default bsc)". Also clarified that Greenfield write tools DO have a default (testnet), correcting the blanket statement that all writes have no default.