T09 · Insecure Skill Coding Practices
- Location
scripts/search_skillscout_official.mjs:173- Finding
Unvalidated Catalog Metadata Is Converted into Execution-Ready Shell Commands
- Content
View full analysis
- Remediation
View remediation
Security audit
Security checks for vulnerabilities and agentic risk
This skill is a catalog lookup helper that runs a bundled search script and shows user-directed install commands, with no evidence of hidden execution, persistence, or data theft.
Install only if you are comfortable using Skillscout as the source of truth for official skill listings. Before running any displayed command, check that it has the expected simple form `npx skills add owner/repo@skill-name` and does not contain extra shell syntax, unexpected repositories, or unrelated commands.
scripts/search_skillscout_official.mjs:173Unvalidated Catalog Metadata Is Converted into Execution-Ready Shell Commands
Referenced artifact was not completely inspected
node scripts/search_skillscout_official.mjs "query" --json
Referenced artifact was not completely inspected
node scripts/search_skillscout_official.mjs "query" --json
Referenced artifact was not completely inspected
node scripts/search_skillscout_official.mjs "query" --json
Referenced artifact was not completely inspected
node scripts/search_skillscout_official.mjs "query" --json
The skill instructs the agent to run a bundled Node.js script that performs external lookups, implying network and execution capabilities, but the manifest does not declare any tool scope such as permissions or allowed-tools. That mismatch weakens policy enforcement and reviewability: an agent may execute code or make network requests without an explicit, auditable capability declaration.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
npx commands without a version suffix (e.g. @1.0.0) create a rug-pull risk if the upstream server is compromised and publishes a malicious update.
The formatCount function hard-codes toLocaleString("en-US"), which imposes a specific language/locale policy on all numeric output. The file does not offer the user a locale choice or document this as a justified region-specific constraint.