T09 · Insecure Skill Coding Practices
- Location
scripts/timesfm_adapter.py:134- Finding
Caller-Controlled Values Are Embedded into Executable Python Source
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill is a coherent trading-prediction tool, but it has risky runtime and install behavior that users should review before installing.
Review before installing. Use an isolated virtual environment, do not run post_install.py until the package-rewrite approach is removed or justified, avoid passing untrusted symbol/timeframe values, pin Hugging Face models to reviewed revisions, and explicitly control any output path instead of allowing default writes into a personal Obsidian vault.
scripts/timesfm_adapter.py:134Caller-Controlled Values Are Embedded into Executable Python Source
scripts/post_install.py:80Post-Install Script Rewrites an Installed Third-Party Package
scripts/kronos_adapter.py:35Remote Model Artifacts Are Loaded Without Immutable Revision Pins
scripts/run_team.py:266Orchestration Script Automatically Writes Outside the Project Directory
Pinning transformers==4.57.6 is highly dangerous if it is affected by multiple RCE and arbitrary code execution advisories, because transformer ecosystems often load remote or serialized model assets and execute complex initialization paths. This skill explicitly uses model libraries for prediction and optional neural sentiment analysis, which increases the practical attack surface if any model loading, Trainer usage, or untrusted configuration is involved.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
The finding says the code is focused on backtesting/validation rather than prediction generation and lacks the declared integrations and network behavior. Although this is less severe than overtly dangerous side effects, it is still a trust and transparency failure that can mask what code will actually inspect, produce, or store.
If click==8.3.2 is affected by a command injection flaw, any CLI entry points built on top of it could become dangerous when attacker-controlled arguments, environment variables, or shell-like inputs are passed through unsafe code paths. This project includes a CLI-oriented dependency set, so the vulnerability is relevant even if no exploit path is visible from requirements.txt alone.
Package name closely resembles a popular package, suggesting possible typosquatting. Attackers publish malicious packages with similar names to trick developers into installing them.
No suspicious patterns detected.