T09 · Insecure Skill Coding Practices
- Location
trader.py:38- Finding
Minimum Market Volume Safeguard Is Declared but Never Enforced
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This is a disclosed paper-by-default trading skill, but live mode can place real-money trades while important documented risk limits are not actually enforced.
Review before installing for any live account. Keep it in paper mode unless you are prepared for real USDC loss, restrict the API key and account limits, monitor positions externally, and do not rely on SIMMER_MIN_VOLUME or SIMMER_MAX_POSITIONS as documented until those controls are fixed. Pin and review simmer-sdk before production use.
trader.py:38Minimum Market Volume Safeguard Is Declared but Never Enforced
trader.py:504Maximum Concurrent Position Limit Resets on Every Execution
clawhub.json:3Unpinned Third-Party SDK Receives a High-Value Trading Credential
The skill declares access to a high-value credential (SIMMER_API_KEY) and describes trading execution, but it does not define any explicit tool scope such as permissions or allowed-tools. In practice, that means the runtime may grant broader environment or tool access than is necessary, violating least privilege and increasing the blast radius if the skill is modified, misused, or composed with other agent behaviors.
The manifest describes an automated trading skill with autostart disabled but explicit position-sizing and trade-threshold controls, which indicates it can place financial trades once run. There is no user-facing disclosure in the manifest about autonomous execution, market risk, loss potential, or the fact that configuration values can materially affect exposure, increasing the chance of uninformed or unsafe deployment.