Back to skill

Security audit

Free Models for OpenClaw and other Agents

Security checks for vulnerabilities and agentic risk

Overview

The skill mostly matches its model-discovery purpose, but it unnecessarily asks for an OpenRouter API key that the code does not actually use.

Review before installing if you are uncomfortable with unnecessary credential handling. The model lookup behavior is narrow and non-destructive, but users should not paste or export a real OpenRouter API key for this CLI unless the package is changed to need it and provides clear secret-handling guidance.

Vulnerability Patterns
  • Skill Instruction HijackingAlters the agent's session goals or safety constraints when the skill loads
  • Agent Memory PoisoningWrites attacker-controlled rules into memory that affect later sessions
  • Remote Payload Retrieval and ExecutionFetches external code whose behavior can change after review
  • Embedded Malicious CodeShips malicious scripts inside the skill and executes them locally
  • Unauthorized Access and Privilege EscalationObtains permissions beyond the task's legitimate needs
Vulnerability Patterns
  • Data ExfiltrationExternal Transmission, Env Variable Harvesting, File System Enumeration
  • System Prompt LeakageDirect Leakage, Indirect Extraction, Tool-Based Exfiltration
  • MCP Tool PoisoningHidden Instructions, Unicode Deception, Parameter Description Injection
  • Prompt InjectionInstruction Override, Hidden Instructions, Exfiltration Commands
  • Privilege EscalationExcessive Permissions, Sudo/Root Execution, Credential Access
Findings (7)

Ae1

High
Category
analysis-evasion
Confidence
100% confidence
Finding

Referenced artifact was not completely inspected

Content

Scanner excerpt · SKILL.md (reported line 20)May include surrounding context.

md
import { discoverFreeModels } from './scripts/free-models.js';

Ae1

High
Category
analysis-evasion
Confidence
100% confidence
Finding

Referenced artifact was not completely inspected

Content

Scanner excerpt · SKILL.md (reported line 28)May include surrounding context.

md
import { discoverFreeModels } from './scripts/free-models.js';

Ae1

High
Category
analysis-evasion
Confidence
100% confidence
Finding

Referenced artifact was not completely inspected

Content

Scanner excerpt · SKILL.md (reported line 58)May include surrounding context.

md
import { discoverFreeModels } from './scripts/free-models.js';

Ae1

High
Category
analysis-evasion
Confidence
100% confidence
Finding

Referenced artifact was not completely inspected

Content

Scanner excerpt · SKILL.md (reported line 87)May include surrounding context.

md
import { discoverFreeModels } from './scripts/free-models.js';

Direct Prompt Extraction

High
Category
System Prompt Leakage
Confidence
85% confidence
Finding

Skill contains instructions that could directly expose system prompts, internal rules, or hidden instructions to users or external parties.

Content

Scanner excerpt · scripts/free-models.js (reported line 45)May include surrounding context.

js
const models = await fetchAllModels();
  return models.filter((m) => {
    const promptPrice = parseFloat(m.pricing.prompt);
    return promptPrice === 0 || promptPrice < 0.0001;
  });
}

Missing User Warnings

Low
Category
Not specified by scanner
Confidence
92% confidence
Finding

The skill instructs users to place a live OpenRouter API key directly into an environment variable and shows a realistic key prefix, but provides no warning about keeping the credential secret, avoiding shell history leakage, or using scoped/rotated keys. While environment variables are a common mechanism for API auth, documentation that normalizes pasting secrets into commands without safe-handling guidance increases the chance of credential exposure through shell history, logs, screenshots, or shared terminal sessions.

Content

No source excerpt is available for this finding.

Intent-Code Divergence

Low
Category
Not specified by scanner
Confidence
97% confidence
Finding

The header comment and CLI checks state that OPENROUTER_API_KEY is required, implying authenticated API use. However, fetchAllModels performs an unauthenticated fetch to the public models endpoint and never includes the API key, so the documentation overstates the code's dependency on credentials.

Content

No source excerpt is available for this finding.