T09 · Insecure Skill Coding Practices
- Location
SKILL.md:51- Finding
Potential Disclosure of Repository and CI Secrets to External AI Providers
- Content
View full analysis
Vulnerability Details
File Location:
SKILL.md:51-52,SKILL.md:88-90, andSKILL.md:115-125
Vulnerability Type: Sensitive information exposure through external provider submission
Risk Level: MediumVulnerable Code
text Both upstream and downstream workflows can send the same evidence package to one provider or a multi-agent tribunal.bash OPENAI_API_KEY=<key> npm run argus -- review owner/repo#123 --provider openai-api ANTHROPIC_API_KEY=<key> npm run argus -- review owner/repo#123 --provider anthropic-api GEMINI_API_KEY=<key> npm run argus -- review owner/repo#123 --provider gemini-apibash npm run argus -- ci-log logs/failure.txt npm run argus -- ci-log logs/failure.txt --provider codex-cli npm run argus -- ci-log logs/failure.txt --tribunal codex-cli,claude-cli,gemini-clibash GITHUB_TOKEN=<read-only-token> npm run argus -- ci-github owner/repo#123 GITHUB_TOKEN=<read-only-token> npm run argus -- ci-github owner/repo#123 --provider codex-cliTechnical Analysis
The Skill permits repository evidence and CI logs to be submitted to external AI APIs or provider CLIs. Pull-request diffs, source files, and build logs can contain private source code, access tokens, API keys, personal information, internal URLs, or environment values.
Although
SKILL.md:69instructs users not to print or write tokens, that safeguard does not address secrets already embedded in the evidence package. The Skill does not document automatic redaction, file or content allowlists, outbound-payload previews, destination restrictions, data classification checks, or explicit per-submission consent. Consequently, remote submission may disclose more information than is minimally necessary for code review.The behavior is related to the declared multi-provider review functionality, but its documented controls are insufficient to enforce least privilege o ...[truncated 1275 chars]
- Remediation
View remediation
Remediation Suggestions
- Keep deterministic local review as the default and require explicit, informed consent before every remote submission.
- Display the provider destination and a preview or manifest of the exact outbound payload.
- Apply automatic redaction for access tokens, private keys, passwords, connection strings, authorization headers, personal data, and common cloud credentials.
- Reject or quarantine evidence containing PEM blocks, high-confidence secret patterns, or values matching configured environment secrets.
- Introduce repository, file-type, path, and content allowlists; exclude environment files, credential stores, generated artifacts, and sensitive CI output by default.
- Minimize submitted context to the smallest diff or log fragment required for analysis.
- Document each provider's retention, training, residency, and deletion policies.
- Require separate confirmation before tribunal mode sends evidence to multiple destinations.
- Add tests demonstrating that credentials in diffs and logs are removed before provider invocation.
