T08 · Insecure Dependencies
- Location
requirements.txt:1- Finding
Unnecessary Unpinned Third-Party Dependency
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill’s reference-formatting purpose is coherent, but it needs review because its script can overwrite any writable output path and its CSV export does not neutralize spreadsheet formulas.
Install only if you are comfortable reviewing output paths before every run. Use a dedicated output directory, avoid overwriting existing files unless intended, treat processed bibliography metadata as untrusted, and be cautious opening exported CSV files in spreadsheet software. The package should remove the unnecessary dependency and add path, overwrite, extension, and CSV formula protections before broad use.
requirements.txt:1Unnecessary Unpinned Third-Party Dependency
scripts/main.py:609Spreadsheet Formula Injection in CSV Export
scripts/main.py:780Unrestricted Output Path Allows Arbitrary File Overwrite
The skill advertises and documents file-reading and file-writing behavior via scripts/main.py and CLI parameters, but it does not declare an explicit tool scope such as permissions or allowed-tools. That creates an authorization ambiguity where an agent or runtime may permit broader filesystem actions than reviewers expect, increasing the risk of unintended file access or overwrite if the skill is invoked with attacker-controlled paths.
The 'When to Use' section broadens invocation beyond reference synchronization into generic academic writing tasks, which can cause the skill to be selected for requests outside its validated scope. In agentic systems, overbroad routing language is dangerous because it increases the chance that a file-processing skill with write capability is invoked in inappropriate contexts, leading to unsafe assumptions, misuse of inputs, or execution against unrelated documents.
This code performs a direct file write using write mode, which can overwrite existing user data at the destination path. Although the script prints a message after exporting, there is no pre-write confirmation, warning comment/docstring about overwrite behavior, or other user disclosure near the destructive operation.
Dependencies lack version pinning, allowing potential malicious package updates. Consider pinning versions.
dataclasses
No suspicious patterns detected.