Skip to content

Add missing privateKey parameters to Greenfield tool documentation#33

Open
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/greenfield-missing-privatekey
Open

Add missing privateKey parameters to Greenfield tool documentation#33
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/greenfield-missing-privatekey

Conversation

@mefai-dev
Copy link

Several Greenfield tools were missing the required privateKey parameter. In the MCP source, privateKey is a required parameter that defaults to env PRIVATE_KEY. Without documenting it, agents won't include it in tool calls.

@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR updates a Greenfield tools reference document in an MCP skill to clarify that privateKey is required for specific tools (gnfd_list_buckets, gnfd_get_bucket_full_info, and gnfd_download_object), while noting it can default to env PRIVATE_KEY. The changes are documentation-only and align tool parameter descriptions with actual MCP source behavior. No code logic or runtime behavior is modified.

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
- Several Greenfield tools require privateKey but it was not documented
- Without this parameter, tool calls will fail or use unexpected defaults

## Type of Change
- [x] Documentation fix

## Changes Made
- Added `privateKey (required, defaults to env PRIVATE_KEY)` to gnfd_list_buckets, gnfd_get_bucket_full_info, gnfd_download_object
- Consistent annotation across all Greenfield write and read tools

## Testing
- [x] Verified against MCP source privateKeyParam definitions
@mefai-dev mefai-dev force-pushed the fix/greenfield-missing-privatekey branch from 5b5f2ec to 056d2c0 Compare March 13, 2026 23:47
@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR updates the Greenfield tools reference documentation to explicitly mark privateKey as required (while defaulting to PRIVATE_KEY in environment) for several tools that previously appeared optional or unspecified. The changes are limited to markdown docs and improve parameter accuracy for gnfd_list_buckets, gnfd_get_bucket_full_info, and gnfd_download_object. Overall, it aligns the skill docs with MCP implementation behavior so agents include required auth material in tool calls.

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