security-review

Detect shell injection, hardcoded secrets, and unsafe file access in code changes.

Updated Aug 27, 2026
One-click install
npx skills add https://github.com/mojwang/macbook-dev-setup --skill security-review-mojwang
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: security-review
Source: https://github.com/mojwang/macbook-dev-setup/tree/main/.claude/skills/security-review
Command: npx skills add https://github.com/mojwang/macbook-dev-setup --skill security-review-mojwang

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Security review checks for shell injection, secrets, file access vulnerabilities, and OWASP top 10. Use during code reviews to catch issues in changes, diffs, commits, or edits that handle user input, authentication, or file access.

Core Features & Use Cases

  • Shell Injection: Flag unquoted variables in command substitutions, flag dangerous eval usage, and detect unsafe use of xargs on untrusted input.
  • Secret/Credential Detection: Detect hardcoded API keys, tokens, passwords, and environment variables exposed in commits.
  • File Access Patterns: Highlight path traversal risks, unsafe temp file creation, and insecure permissions.
  • OWASP Top 10 (for code): Check for common web vulnerabilities such as SQL injection, XSS, CSRF, insecure deserialization, and broken authentication patterns.

Quick Start

Run the security-review skill on your diff or code changes to surface potential vulnerabilities.

Frequently Asked Questions about security-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I automate code security reviews for shell injection and hardcoded secrets?

Automate code security reviews by applying static checks and pattern matching to diffs and commits to identify shell injection, hardcoded secrets, and unsafe file access patterns. This process highlights risks and suggests remediation guidelines.

Can I detect OWASP top 10 vulnerabilities in my code changes?

Yes, you can detect OWASP top 10 vulnerabilities in code changes by checking for SQL injection, XSS, CSRF, insecure deserialization, and broken authentication patterns across multiple languages using static analysis.

How do I scan git diffs for path traversal and unsafe file access patterns?

Scan git diffs for path traversal and unsafe file access patterns by running static analysis on edits that handle user input or file operations. The review highlights path traversal risks, unsafe temp file creation, and insecure permissions.

Does static analysis catch hardcoded API keys and exposed environment variables in commits?

Yes, static analysis detects hardcoded API keys, tokens, passwords, and exposed environment variables in commits. It flags these secrets during code reviews to prevent credential leakage.

What is the best way to check for shell injection in unquoted command substitutions?

The best way to check for shell injection is using static analysis to flag unquoted variables in command substitutions, detect dangerous eval usage, and identify unsafe xargs execution on untrusted input.

How do I review code changes for broken authentication and insecure deserialization?

Review code changes for broken authentication and insecure deserialization by applying static checks to edits touching authentication or user input. The analysis identifies these OWASP top 10 risks and suggests remediation guidelines.