T09 · Insecure Skill Coding Practices
- Location
trader.py:32- Finding
Declared Minimum-Volume Safeguard Is Not Enforced
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is transparently a trading bot, but several documented financial safeguards are not actually enforced strongly enough for live trading.
Review carefully before live use. Keep it in paper mode until the liquidity, position-count, and sizing controls are fixed; if enabling --live, use a limited, revocable API key and conservative account funding.
trader.py:32Declared Minimum-Volume Safeguard Is Not Enforced
trader.py:334Maximum Concurrent Position Limit Ignores Existing Open Positions
trader.py:260Minimum Trade Size Can Override the Configured Maximum Position Size
clawhub.json:6Privileged Trading SDK Dependency Is Not Version-Pinned
The skill references a high-value credential (SIMMER_API_KEY) and describes trading behavior, but it does not declare an explicit tool/permission scope such as allowed environment access. In an agent ecosystem, undeclared env capability weakens least-privilege controls and can allow broader secret access than operators expect, especially for a trading skill that could misuse credentials if execution content later expands.
This manifest defines an automated trading skill with a managed entrypoint and configurable position-sizing controls, but it does not provide any explicit user-facing warning that the skill may place real-money trades using configured funds. In a trading context, omission of this disclosure increases the risk of users enabling or deploying the skill without understanding that it can autonomously open positions and incur financial loss.