Query token audit Skill from Binance#8
Conversation
58d4e48 to
6ce35a7
Compare
Greptile SummaryThis PR adds a new Key findings:
Confidence Score: 2/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant Agent
participant BinanceAPI as Binance Web3 API
User->>Agent: "Check if this BSC token is safe before I buy it"
Agent->>Agent: Extract contractAddress + chainId from user input
Agent->>Agent: Generate UUID v4 for requestId
Agent->>BinanceAPI: POST /bapi/defi/v1/public/wallet-direct/security/token/audit<br/>{ binanceChainId, contractAddress, requestId }
BinanceAPI-->>Agent: { code, data: { hasResult, isSupported, riskLevelEnum, riskLevel, extraInfo, riskItems[] } }
alt hasResult=false OR isSupported=false
Agent->>User: "Audit data not available for this token"
else hasResult=true AND isSupported=true
Agent->>Agent: Parse riskLevel (0-5) and riskLevelEnum
Agent->>Agent: Filter riskItems where isHit=true
Agent->>User: Summary of risk level + flagged risk items<br/>+ disclaimer
end
Last reviewed commit: "add Query Token Audi..." |
| id: query-token-audit | ||
| name: Query Token Audit | ||
| description: Query token security audit details to detect scams, honeypots, malicious contract risks, and dangerous trading conditions before swapping. | ||
| category: Blockchain |
There was a problem hiding this comment.
The category field is set to Blockchain, which is not one of the valid values defined in CONTRIBUTING.md. The allowed values are: productivity, development, communication, writing, research, or other.
Despite the PR checklist marking this box as ✅, the value used does not conform to the schema. The PR checklist review system also flags this automatically per the repository's CI description.
The correct value for this skill would be other (or development, given it's an API integration skill).
| category: Blockchain | |
| category: other |
| **Request Headers**: | ||
| ```text | ||
| Content-Type: application/json | ||
| Accept-Encoding: identity | ||
| ``` |
There was a problem hiding this comment.
source: agent header missing from documented headers
The "Request Headers" section (lines 56–60) only documents Content-Type and Accept-Encoding, but the example curl command on line 66 also sends a third header:
--header 'source: agent'
If source: agent is required or meaningful for the API to behave correctly, omitting it from the headers table is a documentation gap that will cause agents following only the table to send requests without it. Conversely, if it is optional or has no effect, it should be removed from the example to avoid confusion.
Please either add it to the headers table with a description of what it does, or remove it from the example request.
| | hasResult | boolean | Whether audit data is available | | ||
| | isSupported | boolean | Whether the token is supported for audit | | ||
| | riskLevelEnum | string | Risk level: `LOW`, `MEDIUM`, `HIGH` | | ||
| | riskLevel | number | Risk level number (1-5) | |
There was a problem hiding this comment.
riskLevel range description is inconsistent
The response field table documents riskLevel as number (1-5), but the Risk Level Reference table directly below it shows 0-1 as a valid range, implying riskLevel can be 0. An agent interpreting the field description literally would not handle a riskLevel: 0 response correctly (e.g., might treat it as an out-of-range/unexpected value).
| | riskLevel | number | Risk level number (1-5) | | |
| | riskLevel | number | Risk level number (0-5) | |
| --- | ||
| id: query-token-audit | ||
| name: Query Token Audit | ||
| description: Query token security audit details to detect scams, honeypots, malicious contract risks, and dangerous trading conditions before swapping. | ||
| category: Blockchain | ||
| author: binance-web3-team | ||
| version: 1.4.0 | ||
| requires: [] | ||
| examples: | ||
| - Check if this BSC token is safe before I buy it. | ||
| - Audit this Base token contract for honeypot and scam risk. | ||
| - Run a token security check on this Solana contract and summarize the risk items. | ||
| --- |
There was a problem hiding this comment.
External URLs in instructions (unchecked checklist item)
The PR checklist item "No external URLs referenced in instructions" is explicitly unchecked, and the skill does embed the live Binance endpoint URL directly in the instructions:
https://web3.binance.com/bapi/defi/v1/public/wallet-direct/security/token/audit
Per CONTRIBUTING.md guideline #3: "Don't reference external URLs — Skills should be self-contained."
For an API-integration skill this is somewhat unavoidable, but it's worth noting this is a deliberate deviation from the guidelines that may need explicit maintainer approval.
New Skill Contribution
Skill Info
Checklist
Testing
Describe how you tested this skill:
Notes
Any additional context or notes for reviewers.