T09 · Insecure Skill Coding Practices
- Location
SKILL.md:21- Finding
Unsafe and Incomplete Mainnet Arbitrage Transaction Guidance
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is a coherent SaucerSwap arbitrage guide, but it gives unsafe and incomplete mainnet trading instructions that could cause financial loss if followed.
Review this carefully before installing. Do not let an agent use it to trade real funds unless you add explicit confirmation, testnet or dry-run defaults, full route validation, quote freshness checks, fee and slippage accounting, and transaction parameters that enforce minimum output and deadlines.
SKILL.md:21Unsafe and Incomplete Mainnet Arbitrage Transaction Guidance
The skill explicitly advertises executing atomic swaps and arbitrage on mainnet but does not prominently warn that transactions are irreversible and may result in immediate financial loss from slippage, fees, stale quotes, or logic errors. In the context of an agent skill that could be followed operationally, missing risk disclosures makes unsafe execution more likely.
The skill claims to support triangular arbitrage, but the quote function only fetches a single route from tokenA to tokenB and ignores the remaining hop(s). In a trading skill, this can cause users or downstream agents to calculate profitability on incomplete data and execute loss-making swaps under the false assumption that the full arbitrage path was evaluated.
The examples are presented as triangular arbitrage paths, but they are only two-token round trips such as USDC → HBAR → USDC, not three-asset cycles. This mislabels the trading strategy and can mislead operators into using incorrect logic, reducing strategy validity and increasing the chance of accidental losses when real funds are used.
No suspicious patterns detected.