T09 · Insecure Skill Coding Practices
- Location
SKILL.md:35- Finding
Unsafe SQL Interpolation in Database Operations
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This flashcard review skill is purpose-aligned, but its raw SQLite instructions can let unescaped user or AI-generated text alter the local study database in unintended ways.
Review this skill before installing if your OpenAlgernon study database contains important or sensitive material. It should be revised to use parameterized SQLite operations, validate slugs and IDs, escape generated text safely, and document or make optional the conversation-log append.
SKILL.md:35Unsafe SQL Interpolation in Database Operations
The skill persists review-session details to disk without any notice or consent mechanism, which is a privacy and transparency failure. Because review sessions may reflect user interests, study content, timestamps, and performance metrics, silent storage can surprise users and create compliance or confidentiality issues, especially in shared or synced environments.
The skill writes session metadata into a persistent conversation log even though its stated purpose is flashcard review and scheduling. This creates unnecessary secondary storage of user activity data, expanding the privacy surface and potentially exposing study habits or sensitive topics if the memory directory is later accessed or reused by other components.
No suspicious patterns detected.