T09 · Insecure Skill Coding Practices
- Location
scripts/migrate_timestamps.py:21- Finding
Predictable Temporary File Enables Symlink-Based File Overwrite During Timestamp Migration
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
The skill mostly matches its token-tracking purpose, but review is warranted because its log-maintenance scripts can unsafely rewrite files in some local deployments.
Install only after reviewing the scripts and setting the log directory to a private, trusted workspace path. Do not run the migration or dedupe utilities as root or from a directory writable by other users unless the temporary-file handling is hardened. Treat the systemd file as a template and change the user and paths before enabling it.
scripts/migrate_timestamps.py:21Predictable Temporary File Enables Symlink-Based File Overwrite During Timestamp Migration
scripts/dedupe_log.py:11Predictable Temporary File Enables Symlink-Based File Overwrite During Log Deduplication
The skill advertises file-reading and file-writing functionality through its described scripts and log handling, but it does not declare any tool scope such as permissions or allowed-tools. This creates a trust and review gap: consumers cannot easily determine the intended file-system access boundaries, increasing the risk of overbroad access, unsafe deployment, or misuse if the examples are wired into an agent pipeline without restrictions.