T09 · Insecure Skill Coding Practices
- Location
scripts/add-liquidity.mjs:46- Finding
Liquidity Transactions Use Zero Slippage Protection and Excessive Token Allowances
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This Sepolia DeFi skill is not malicious, but it can sign blockchain transactions and handle private keys with weaker safeguards than users are likely to expect.
Install only if you intend to interact with this specific Sepolia deployment. Use a dedicated low-value Sepolia key that is not reused elsewhere, review transaction details before running write commands, and be aware that the scripts currently require a private key even for read-only queries. Treat the approvals and zero-slippage liquidity defaults as review items before using the pattern outside testnet.
scripts/add-liquidity.mjs:46Liquidity Transactions Use Zero Slippage Protection and Excessive Token Allowances
scripts/config.mjs:21Read-Only Commands Unnecessarily Require and Materialize a Private Key
scripts/package.json:5Unlocked Dependency Resolution Makes Installations Non-Reproducible
scripts/post-signal.mjs:12State-Changing Commands Forward Unvalidated User-Controlled Parameters
The skill advertises itself for broad topics like DeFi liquidity provision, impermanent loss, and automated market making, which can cause it to activate for general informational conversations beyond its safe transactional scope. That over-broad invocation surface is dangerous because users may be funneled into scripts that interact with wallets or contracts when they only asked for education or analysis.
The command section describes liquidity-management operations and wallet-backed scripts without a clear, prominent warning that these actions may use a built-in demo wallet, approve tokens, mint tokens, and submit blockchain transactions. Even on testnet, this can normalize unsafe behavior and cause users or agents to execute state-changing commands without informed consent or understanding of key material and transaction side effects.
The architecture section includes unrelated shell commands embedded in the documented execution flow, which can mislead an agent or operator into running unintended local commands. In a skill that is supposed to bridge chat actions to blockchain transactions, contradictory command text increases the risk of prompt/command confusion and accidental execution of actions outside the intended workflow.
The dependency uses a caret range (^2.20.0), which permits automatic installation of newer minor and patch releases rather than a single immutable version. In a security-sensitive DeFi automation skill, this increases supply-chain risk because a compromised upstream release or unexpected behavioral change in the viem library could alter transaction-building, signing, or on-chain interaction logic without an explicit review.
"type": "module",
"private": true,
"dependencies": {
"viem": "^2.20.0"
}
}
No suspicious patterns detected.