T09 · Insecure Skill Coding Practices
- Location
references/anti-amnesia-checklist.md:13- Finding
Plaintext Persistence of Sensitive Configuration and Credential Metadata
- Content
View full analysis
Vulnerability Details
File Location:
references/anti-amnesia-checklist.md:13-18
Related Locations:SKILL.md:59-64,references/anti-amnesia-checklist.md:104-106,references/memory-lifecycle.md:161-170
Vulnerability Type: Plaintext sensitive-data persistence
Risk Level: MediumVulnerable Snippet:
markdown - [ ] A critical file path or config location is discovered - [ ] A dependency version is pinned for a specific reason - [ ] A blocking issue is identified (and its current status) - [ ] A commitment is made to a user or another agent - [ ] A config value is changed for a non-obvious reason - [ ] A security-relevant finding or credential location is notedAdditional relevant instruction from
SKILL.md:59-64:markdown 1. **Decisions made** — anything that affects future work 2. **Key discoveries** — bugs found, root causes, unexpected behaviors 3. **Pending tasks** — unfinished items that must survive session restart 4. **Context anchors** — file paths, API endpoints, config valuesAdditional routing instruction from
references/anti-amnesia-checklist.md:100-106:markdown | Journal item mentions... | Route to | |---|---| | "we chose", "decided to", "architecture" | `decisions.md` | | "bug", "root cause", "workaround", "踩坑" | `lessons.md` | | "tech stack", "deploy", "schema", "config path" | `project-context.md` | | "Simon prefers", "team role", "communication style" | `people.md` | | "command", "proxy", "env variable", "tool setup" | `tools-config.md` |Technical Analysis
The Skill instructs the agent to persist security findings, credential locations, configuration values, environment-variable information, API endpoints, proxy settings, and deployment information in ordinary Markdown files. It does not require secret detection or redaction, restrict file permissions, prevent the memory directory from being committed to source control ...[truncated 1835 chars]
- Remediation
View remediation
Remediation Suggestions
- Explicitly prohibit storing passwords, access tokens, API keys, private keys, cookies, session identifiers, connection strings, recovery codes, and complete credential contents.
- Add mandatory redaction rules for configuration values, such as retaining only variable names or masked forms like
TOKEN=***. - Store only non-sensitive references to credential-management systems rather than credential values or unnecessarily precise credential locations.
- Require user confirmation before retaining security-sensitive environment or infrastructure information.
- Apply restrictive permissions to the memory directory and files, allowing access only to the intended account.
- Add
memory/and generated search indexes to.gitignoreand equivalent backup or synchronization exclusion rules unless protected retention is explicitly required. - Define retention and secure-deletion rules for sensitive metadata rather than allowing ordinary promotion and archival.
- Run secret scanning before journal promotion, indexing, archival, synchronization, or source-control operations.
- Document how users can inspect, redact, export, and delete stored memory.
