Back to skill

Security audit

Temp Skills

Security checks for vulnerabilities and agentic risk

Overview

This skill is for OKX trading, but it publishes account details and a trading API key while giving instructions for real asset-affecting actions without clear safeguards.

Review this carefully before installing. The skill appears intended for real OKX trading and includes published account and API-key material; that key should be treated as exposed and revoked or rotated. Do not use it for live trading unless credentials are moved to secure runtime storage and state-changing actions require explicit user approval with clear testnet/mainnet and logging safeguards.

Vulnerability Patterns
  • Insecure Skill Coding PracticesFinds exploitable flaws such as hardcoded secrets or command injection
  • Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
  • Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
  • Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
  • Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally
Findings (1)

T09 · Insecure Skill Coding Practices

Error
Location
SKILL.md:7
Finding

Hardcoded OKX Trading API Credential

Content
View full analysis

Vulnerability Details

File Location: SKILL.md, line 7
Vulnerability Type: Plaintext hardcoded API credential
Risk Level: High

Vulnerable Code

markdown
- **API Key**: 418d477d-7247-4df9-9270-66055550c1cc

Technical Analysis

The skill documentation embeds an OKX API key directly in a repository-tracked plaintext file. Nearby configuration states that the associated credential has read and trading permissions, although it does not have withdrawal permission. Anyone who can read the skill package, repository history, build artifacts, backups, or logs containing this file can recover the exposed key.

OKX authentication commonly requires additional authentication material, such as a secret and passphrase. Those values are not present in the audited file, so the API key alone cannot be assumed to provide immediate authenticated access. Nevertheless, exposing one component of a privileged credential reduces authentication security and enables account targeting, credential correlation, and abuse if the remaining components are leaked, reused, weakly protected, or otherwise obtained.

Attack Path

  1. An attacker obtains read access to the skill package, repository, published artifact, backup, or copied documentation.
  2. The attacker extracts the API key from SKILL.md.
  3. The attacker identifies the associated OKX account using the account metadata present in the same file.
  4. The attacker attempts to obtain or correlate the required companion secret and passphrase through another disclosure, reused configuration, compromised environment, logs, or social engineering.
  5. If all required authentication material is recovered and the key remains active, the attacker submits authenticated OKX API requests.
  6. Based on the documented permissions, the attacker may query account information or place unauthorized trades. Withdrawal is reportedly disabled, but malicious trading could still cause financial lo ...[truncated 740 chars]
Remediation
View remediation

Remediation Suggestions

  1. Revoke the exposed API key immediately and create a replacement; removal from the current file does not eliminate copies in repository history, caches, backups, or distributed artifacts.
  2. Remove API credentials and sensitive account identifiers from SKILL.md and all other tracked files.
  3. Store the replacement key, secret, and passphrase in a managed secret store or protected environment variables injected only at runtime.
  4. Ensure secrets are never printed in application logs, error messages, diagnostic output, or audit artifacts.
  5. Apply least privilege. Use a read-only credential unless trading access is strictly required, and separate market-data, account-reading, and trading credentials where practical.
  6. Retain withdrawal restrictions and configure an IP allowlist so the key is accepted only from approved hosts or egress addresses.
  7. Add repository secret scanning and pre-commit checks to prevent future credential commits.
  8. Review OKX access and trading logs for use of the exposed key, including unknown source addresses, unexpected account queries, and unauthorized orders.
  9. Rotate any companion secret or passphrase that may have been stored or distributed with this key elsewhere.
Vulnerability Patterns
  • Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
  • Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
  • Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
  • Supply ChainUnpinned Dependencies, External Script Fetching, Obfuscated Code
  • Excessive AgencyUnrestricted Tool Access, Autonomous Decision Making, Scope Creep
Findings (4)

Missing User Warnings

High
Category
Not specified by scanner
Confidence
99% confidence
Finding

The skill embeds a live-looking OKX API key alongside an account identifier and explicitly states trading permissions. Exposed exchange credentials can enable unauthorized account access and automated trading, causing direct financial loss even if withdrawals are disabled, and the presence of localhost proxy details may further aid operational misuse.

Content

No source excerpt is available for this finding.

Missing User Warnings

High
Category
Not specified by scanner
Confidence
93% confidence
Finding

The documented functions include balance checks, fund transfers, and order placement but provide no prominent warning, confirmation requirements, or authorization boundaries for operations affecting assets and account state. In a trading skill, this omission increases the chance of accidental or unauthorized execution of real financial actions with immediate monetary consequences.

Content

No source excerpt is available for this finding.

Ssd 3

Medium
Category
Not specified by scanner
Confidence
80% confidence
Finding

A blanket requirement to log all trading activity can cause sensitive financial data, account identifiers, order details, and potentially secrets to be retained or forwarded without clear minimization rules. In the context of an exchange trading skill, overbroad logging increases exposure through log stores, support channels, and incident notifications.

Content

No source excerpt is available for this finding.

Natural-Language Policy Violations

Low
Category
Not specified by scanner
Confidence
71% confidence
Finding

All user-facing instructions in the file are presented only in Chinese, which may implicitly force a specific language for users who did not opt into that locale. The document does not mention that the skill is intended only for Chinese-speaking users or provide an alternative language option.

Content

No source excerpt is available for this finding.