Skip to content

Add missing SSE port, LOG_LEVEL, and 8004scan documentation#36

Open
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/sse-documentation
Open

Add missing SSE port, LOG_LEVEL, and 8004scan documentation#36
mefai-dev wants to merge 1 commit intobnb-chain:mainfrom
mefai-dev:fix/sse-documentation

Conversation

@mefai-dev
Copy link

SSE mode documented --sse flag but not the PORT env variable or client connection URL. Added PORT documentation, LOG_LEVEL env variable, and clarified 8004scan references.

@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR updates documentation for the bnbchain-mcp skill, primarily clarifying SSE usage by explicitly documenting the client URL and the PORT environment variable with default 3001. It also adds LOG_LEVEL environment variable guidance and refines ERC-8004 registration verification wording by linking directly to 8004scan. Overall, this is a docs-only change with no runtime code modifications.

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
- SSE mode documented the --sse flag but not the PORT env variable or client connection URL
- LOG_LEVEL env variable was completely undocumented

## Type of Change
- [x] Documentation improvement

## Changes Made
- Added PORT env variable documentation (default: 3001) for SSE mode
- Added client connection URL: `http://localhost:PORT/sse`
- Added LOG_LEVEL env variable (DEBUG, INFO, WARN, ERROR)
- Clarified 8004scan explorer reference

## Testing
- [x] Verified PORT and LOG_LEVEL against MCP source
@mefai-dev mefai-dev force-pushed the fix/sse-documentation branch from 2886d1f to f544f6c Compare March 13, 2026 23:48
@hashdit-bot
Copy link

hashdit-bot bot commented Mar 13, 2026

Pull Request Review

This PR updates skills/bnbchain-mcp-skill/SKILL.md to improve connection and environment documentation for SSE mode and runtime logging. It clarifies that SSE clients should connect via http://localhost:3001/sse, documents use of the PORT environment variable (default 3001), and adds LOG_LEVEL options (DEBUG, INFO, WARN, ERROR). It also refines ERC-8004 registration guidance by replacing plain-text “8004scan” mentions with a direct explorer link.

Sensitive Content

Social Account:

  • https://8004scan.com (explorer/social-style external link) in skills/bnbchain-mcp-skill/SKILL.md — added as verification reference for ERC-8004 registration

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