T09 · Insecure Skill Coding Practices
- Location
SKILL.md:49- Finding
Database Credentials Exposed Through Command-Line Arguments
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This database skill is mostly coherent, but it includes high-impact restore, migration, credential, and export examples that need careful review before use.
Review this skill before installing. Use read-only, least-privilege database accounts; avoid putting passwords directly in commands; avoid running restore or migration examples against production unless you have a verified plan and explicit confirmation; and replace shared /tmp export paths with private, access-controlled temporary directories.
SKILL.md:49Database Credentials Exposed Through Command-Line Arguments
SKILL.md:121Predictable Shared Temporary Files Used for Database Exports
SKILL.md:256Predictable Shared Temporary Files Used for Database Backups
The safety section says write operations require explicit confirmation, but later restore and migration commands are provided as ready-to-run examples with no confirmation, dry-run, or rollback guidance. This inconsistency can cause an agent or user to treat dangerous commands as routine operational steps, leading to accidental data overwrite, corruption, or unauthorized imports.
The activation phrases are broad enough to match common troubleshooting or data-related requests, which can cause the skill to activate in contexts where the user did not intend database access. Because this skill can expose schemas, row contents, credentials usage patterns, and even administrative operations, overbroad triggering increases the risk of unnecessary sensitive-data access or escalation into higher-risk actions.
The skill is framed as a database exploration/querying tool, but it also embeds restore, backup, and migration procedures that can materially change database state or move sensitive data. In an agent setting, this scope expansion increases the chance the skill is invoked for benign inspection tasks but then enables destructive or high-impact operations without clear separation of privileges or intent.
Restore and migration commands are documented without prominent user-facing warnings even though they can overwrite data, import untrusted content, or alter production systems. In a terminal-oriented skill, examples are often copied verbatim, so missing warnings materially increases the likelihood of accidental destructive execution.
No suspicious patterns detected.