Skill flagged — suspicious patterns detected

ClawHub Security flagged this skill as suspicious. Review the scan results before using.

Binance P2p

v1.0.0

Binance P2P trading assistant. Use when asked about P2P prices, ads, payment methods, or P2P order history. NOT for spot/futures prices or exchange trading.

0· 79·0 current·0 all-time
Security Scan
VirusTotalVirusTotal
Benign
View report →
OpenClawOpenClaw
Suspicious
medium confidence
!
Purpose & Capability
The SKILL.md clearly supports both public (no-auth) queries and authenticated personal-order endpoints that require BINANCE_API_KEY and BINANCE_SECRET_KEY. However, the registry metadata lists no required environment variables or primary credential. This is an internal inconsistency: a Binance P2P order-history feature legitimately needs API credentials, so the registry should declare them. The lack of a homepage/owner context increases uncertainty.
Instruction Scope
The instructions themselves are focused on P2P queries and SAPI-authenticated order history and provide detailed signing guidance (HMAC SHA256, percent-encoding, SAPI parameter ordering). They instruct the agent to use env vars for credentials, to mask secrets in outputs, and to prefer not writing secrets to disk unless the user consents. They also include concrete shell examples (openssl, curl) which are expected for HMAC signing but give the agent explicit shell-level operations — normal for this task but worth noting. The unusual SAPI requirement to preserve insertion order for parameters is highlighted; while this may be valid for Binance P2P SAPI, it is nonstandard and should be validated against official docs.
Install Mechanism
This is an instruction-only skill with no install spec and no code files. Nothing is written to disk or downloaded by default, which minimizes installer risk.
!
Credentials
The SKILL.md requires two sensitive environment variables (BINANCE_API_KEY and BINANCE_SECRET_KEY) for personal-order features, which is proportionate to the claimed Phase 2 capability. However, the registry metadata declares no required env vars or primary credential — an inconsistency that could hide the fact the skill needs secrets. Requesting both API key and secret is expected for signed requests, but the skill should declare these formally and document required minimum permissions (it does recommend read-only).
Persistence & Privilege
The skill is not always-on and does not request system-wide config paths or other skills' configs. The SKILL.md explicitly discourages writing secrets to disk without explicit user consent and suggests .env with .gitignore. Autonomous invocation is allowed by platform default, but the skill does not request elevated persistence privileges.
What to consider before installing
This skill appears to implement what it claims (Binance P2P public queries plus optional authenticated order-history), but there is a clear mismatch: the bundled SKILL.md expects BINANCE_API_KEY and BINANCE_SECRET_KEY while the registry metadata lists no credentials. Before installing or providing secrets: - Ask the publisher/owner to correct the registry metadata so required env vars and primary credential are explicit. A trustworthy skill should declare its need for secrets in the registry. - If you must enable personal-order features, create a dedicated Binance API key with the minimum permissions (reading only), enable IP whitelist, and avoid using a key tied to large balances. - Prefer injecting keys at runtime/session (not writing to disk). If you must store them, only save to disk with your explicit consent and ensure the file is in .gitignore and protected. - Verify the SAPI signing instructions against Binance's official docs (the 'do not sort parameters' claim is unusual and should be confirmed). - Because the skill is instruction-only, it will run curl/openssl-style commands to sign and call endpoints; restrict execution environment and do not supply secrets to untrusted agents. If you cannot confirm the publisher or do not want to share secrets, use only the public (no-auth) features or consider declining installation.

Like a lobster shell, security has layers — review code before you run it.

latestvk974xd7ygmhzdxmyff19e5d9fd841dw1
79downloads
0stars
1versions
Updated 2w ago
v1.0.0
MIT-0

Binance P2P Trading Skill

Help users interact with Binance P2P (C2C) via natural-language queries.

When to Use / When NOT to Use

Use this skill when the user wants to:

  • Check P2P buy/sell quotes for a crypto/fiat pair (e.g., USDT/CNY).
  • Search P2P advertisements and filter by payment method(s), limits, merchant quality.
  • Compare prices across payment methods (e.g., Alipay vs bank transfer).
  • View their own P2P order history / summary (requires API key).

Do NOT use this skill when the user asks about:

  • Spot/Convert prices, futures/derivatives, margin, trading bots.
  • Deposits/withdrawals, wallet transfers, on-chain transactions.
  • Creating/cancelling orders, appeals, releasing coins (trading operations).

Ask clarifying questions (do not guess) if any key inputs are missing:

  • fiat (e.g., CNY)
  • asset (e.g., USDT)
  • user intent: buy crypto or sell crypto
  • preferred payment method(s)
  • target amount (optional but recommended for ad filtering)

Core Concepts

tradeType mapping (avoid ambiguity)

  • User wants to buy crypto (pay fiat, receive USDT/BTC) → tradeType=BUY
  • User wants to sell crypto (receive fiat, pay USDT/BTC) → tradeType=SELL

Always reflect this mapping in responses when the user’s wording is ambiguous.

Capabilities

Phase 1 — Public Market (No Auth)

  • Quote P2P prices
  • Search ads
  • Compare payment methods
  • Filter/Rank ads by limits and merchant indicators

Phase 2 — Personal Orders (Requires API Key)

  • List P2P order history
  • Filter by trade type / time range
  • Provide summary statistics

Security & Privacy Rules

Credentials

  • Required env vars:
    • BINANCE_API_KEY (sent as header)
    • BINANCE_SECRET_KEY (used for signing)

Never display full secrets

  • API Key: show first 5 + last 4 characters: abc12...z789
  • Secret Key: always mask; show only last 5: ***...c123

Permission minimization

  • Binance API permissions: Enable Reading only.
  • Do NOT request/encourage trading, withdrawal, or modification permissions.

Storage guidance

  • Prefer environment injection (session/runtime env vars) over writing to disk.
  • Only write to .env if the user explicitly agrees.
  • Ensure .env is in .gitignore before saving.

⚠️ CRITICAL: SAPI Signing (Different from Standard Binance API)

Parameter ordering

  • DO NOT sort parameters for SAPI requests.
  • Keep original insertion order when building the query string.

Example:

# ✅ Correct for SAPI: keep insertion order
params = {"page": 1, "rows": 20, "timestamp": 1710460800000}
query_string = urlencode(params)  # NO sorting

# ❌ Wrong (standard Binance API only): sorted
query_string = urlencode(sorted(params.items()))

Signing details

See: references/authentication.md for:

  • RFC 3986 percent-encoding
  • HMAC SHA256 signing process
  • Required headers (incl. User-Agent)
  • SAPI-specific parameter ordering

API Overview

Public Queries (MGS C2C Agent API — No Auth)

Base URL: https://www.binance.com

EndpointMethodParamsUsage
/bapi/c2c/v1/public/c2c/agent/quote-priceGETfiat, asset, tradeTypeQuick price quote
/bapi/c2c/v1/public/c2c/agent/ad-listGETfiat, asset, tradeType, limit, order, tradeMethodIdentifiersSearch ads
/bapi/c2c/v1/public/c2c/agent/trade-methodsGETfiatPayment methods

Parameter notes:

  • tradeType: BUY or SELL (treat as case-insensitive)
  • limit: 1–20 (default 10)
  • tradeMethodIdentifiers: pass as a plain string (not JSON array) — e.g. tradeMethodIdentifiers=BANK or tradeMethodIdentifiers=WECHAT. Values must use the identifier field returned by the trade-methods endpoint (see workflow below). ⚠️ Do NOT use JSON array syntax like ["BANK"] — it will return empty results.

Workflow: Compare Prices by Payment Method

When the user wants to compare prices across payment methods (e.g., "Alipay vs WeChat"), follow this two-step flow:

Step 1 — Call trade-methods to get the correct identifiers for the target fiat:

GET /bapi/c2c/v1/public/c2c/agent/trade-methods?fiat=CNY
→ [{"identifier":"ALIPAY",...}, {"identifier":"WECHAT",...}, {"identifier":"BANK",...}]

Step 2 — Pass the identifier as a plain string into ad-list via tradeMethodIdentifiers, one payment method per request, then compare:

GET /bapi/c2c/v1/public/c2c/agent/ad-list?fiat=CNY&asset=USDT&tradeType=BUY&limit=5&tradeMethodIdentifiers=ALIPAY&tradeMethodIdentifiers=WECHAT

Compare the best price from each result set.

Important: Do not hardcode identifier values like "Alipay" or "BANK". Always call trade-methods first to get the exact identifier strings for the given fiat currency.

Personal Orders (Binance SAPI — Requires Auth)

Base URL: https://api.binance.com

EndpointMethodAuthUsage
/sapi/v1/c2c/orderMatch/listUserOrderHistoryGETYesOrder history
/sapi/v1/c2c/orderMatch/getUserOrderSummaryGETYesUser statistics

Authentication requirements:

  • Header: X-MBX-APIKEY
  • Query: timestamp + signature
  • Header: User-Agent: binance-wallet/1.0.0 (Skill)

Output Format Guidelines

Price quote

  • Show both sides when available (best buy / best sell).
  • Use fiat symbol and 2-decimal formatting.

Example:

USDT/CNY (P2P)
- Buy USDT (you buy crypto): ¥7.20
- Sell USDT (you sell crypto): ¥7.18

Ad list

Return Top N items with a stable schema:

  1. adNo (ad number / identifier)
  2. price (fiat)
  3. merchant name
  4. completion rate
  5. limits
  6. payment methods (identifiers)

Avoid generating parameterized external URLs unless the API returns them.

Placing orders (when user requests):

  • This skill does NOT support automated order placement.

  • When user wants to place an order, provide a direct link to the specific ad using the adNo:

    https://c2c.binance.com/en/adv?code={adNo}
    
    • {adNo}: the ad number/identifier from the ad list result

    Example: https://c2c.binance.com/en/adv?code=123

  • This opens the specific ad detail page where user can place order directly with the selected advertisement.

Personal orders

  • Time format: YYYY-MM-DD HH:mm (UTC+0) — always display in UTC timezone
  • Include: type, asset/fiat, amount, total, status
  • Provide a brief summary line (count + totals) when filtering

Time field conversion (for createTime in listUserOrderHistory):

  • The createTime field returns a Unix timestamp in milliseconds (13 digits).
  • Convert to human-readable format in UTC+0 timezone:
    # Python example
    from datetime import datetime, timezone
    readable_time = datetime.fromtimestamp(createTime / 1000, tz=timezone.utc).strftime('%Y-%m-%d %H:%M (UTC+0)')
    
    # JavaScript example
    const readableTime = new Date(createTime).toISOString().replace('T', ' ').slice(0, 16) + ' (UTC+0)';
    // Or more explicitly:
    const date = new Date(createTime);
    const readableTime = date.getUTCFullYear() + '-' +
      String(date.getUTCMonth() + 1).padStart(2, '0') + '-' +
      String(date.getUTCDate()).padStart(2, '0') + ' ' +
      String(date.getUTCHours()).padStart(2, '0') + ':' +
      String(date.getUTCMinutes()).padStart(2, '0') + ' (UTC+0)';
    
  • Always display the converted time to users with timezone info, not the raw timestamp.

Error Handling (User-Facing)

  • Invalid API key (-2015): prompt to verify .env / API Management.
  • Signature failed (-1022): warn about wrong secret, sorted params, or stale timestamp.
  • Timestamp invalid (-1021): advise time sync / regenerate timestamp.
  • Rate limit: ask to retry later.

Limitations (By Design)

This skill does NOT:

  • Place/cancel orders
  • Mark as paid / release coins
  • Submit appeals
  • Post/modify advertisements
  • Expose sensitive order-detail endpoints beyond what’s needed for history/summary

For in-app actions, guide users to the official P2P orders page (only as a general entry point).

Developer Notes

Version Check (First Invocation per Conversation)

On the first invocation of this skill per conversation, call:

  • GET /bapi/c2c/v1/public/c2c/agent/check-version?version=1.0.1 (Base: https://www.binance.com)

Behavior:

  • If needUpdate=true: show: New version of P2P Skill is available (current: {clientVersion}, latest: {latestVersion}), update recommended.
  • Else / on failure: proceed silently.

Client-side operations

  • Asset filtering: if API doesn’t support it, fetch then filter locally.
  • Aggregations: compute totals client-side when summary endpoint is insufficient.

Comments

Loading comments...