T08 · Insecure Dependencies
- Location
SKILL.md:55- Finding
Unpinned and Unnecessary Third-Party Dependencies
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill is a local financial-analysis tool, but its financial health scoring has under-disclosed assumptions and its install command adds unnecessary supply-chain risk.
Install only after reviewing the financial-method limitations. Treat generated scores as rough calculations, not investment advice, and remove or pin the unused dependency install command before use in a sensitive environment.
SKILL.md:55Unpinned and Unnecessary Third-Party Dependencies
The Piotroski F-Score implementation claims to evaluate all 9 criteria but unconditionally marks three criteria as passed by assumption. This produces misleadingly optimistic health scores and can bias downstream risk and investment conclusions, which is especially dangerous in a financial analysis skill because users may treat these outputs as evidence-based assessments.
The risk assessment function calls nonexistent methods on the FinancialStatements object (fs.debt_to_equity(...) and fs.interest_coverage(...)) instead of evaluating computed values. In practice this will raise an exception during analysis, causing denial of service for any caller that reaches this path and preventing delivery of risk results in a finance-focused tool where users may rely on the output for decisions.
The analyzer initializes with a hard-coded default currency of "CNY", which imposes a specific locale convention unless the caller overrides it. The file does not indicate that this China-specific default is intentional or region-scoped, so it can conflict with language/locale policy expectations for neutral defaults or user choice.