Borrowing it
Nothing to install: this file belongs to hiepdnh/Agentic-Development-Lifecycle. Take a copy, put it at the same path in your own repository, and replace the rules that are about this project with yours.
curl -O https://raw.githubusercontent.com/hiepdnh/Agentic-Development-Lifecycle/main/.claude/skills/sec-review/SKILL.mdgit clone --depth 1 https://github.com/hiepdnh/Agentic-Development-LifecycleWrote this? Show the measurements
A badge with what this costs and how it scanned, read live from this page, so it follows the numbers instead of freezing them. Markdown for a README, HTML for a documentation site or a project page.
[](https://agentmods.dev/skills/hiepdnh/agentic-development-lifecycle/sec-review)<a href="https://agentmods.dev/skills/hiepdnh/agentic-development-lifecycle/sec-review"><img src="https://agentmods.dev/badge/skills/hiepdnh/agentic-development-lifecycle/sec-review.svg" alt="Measured on agentmods" height="20"></a>What it costs to keep this loaded
Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.
| Model | Per session | Once invoked |
|---|---|---|
| Fable 5.1 | $0.00076 | $0.01332 |
| Opus 5 | $0.00038 | $0.00666 |
| Sonnet 5 | $0.00015 | $0.00266 |
| Haiku 4.5 | $0.00008 | $0.00133 |
Grade A, and why
sec-review scanned grade A with 1 finding against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 7d ago.
A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.
Makes network callslowCapability
Not a fault in itself. Listed so you know the mod talks to something, and to what.
| A10 | SSRF | Grep fetch/axios với user-controlled URL | `fetch(req.body.url)` | How it starts
The opening of the file, as written. The whole thing — 146 lines — stays where its author put it; the contents beside it link to each section on GitHub.
Skill: /sec-review
Role: Developer / Tech Lead / Security
Mục đích: Security review có cấu trúc trước khi merge. Ba tier rõ ràng: Always / Ask First / Never.
Ba Tier Quyết định
✅ ALWAYS DO (tự động, không cần hỏi)
- Hash passwords với bcrypt/argon2 (không MD5/SHA1)
- Validate và sanitize tất cả input từ user
- Dùng parameterized queries (không string concatenation cho SQL)
- HTTPS cho tất cả external calls
- Không log sensitive data (password, token, PII)
- Dependency versions up-to-date (check CVE)
⚠️ ASK FIRST (dừng lại, hỏi human trước)
Danh sách đầy đủ + lý do:
assets/ask-first-gates.md
- Thay đổi authentication / authorization logic
- Thêm hoặc sửa permission / role checks
- Breaking changes trong public API
- Database migration có thể mất data
- Database schema thay đổi ảnh hưởng access control
- Thay đổi shared infrastructure hoặc config
- Lưu trữ sensitive data mới (PII, payment info, health data)
- Thay đổi CORS configuration
- Thay đổi cryptographic implementation
- Thêm new external API integration
❌ NEVER DO
- Implement custom encryption (dùng standard libraries)
- Hardcode credentials trong code
- Disable SSL verification
- Log full request/response với sensitive headers
- eval() với user input
- Expose stack traces cho user
Hướng dẫn thực hiện
Bước 1 — Scan changes
Spawn subagent đọc code diff:
"Đọc git diff và tìm: authentication logic, authorization checks, data storage, external API calls, input validation, cryptography usage. Trả về file:line cho mỗi điểm tìm thấy."
Bước 2 — Phân loại và Gate
Phân loại mỗi finding vào 3 tier:
## Security Review: [PR/Feature name]
### ⚠️ Ask First — Cần Senior Review
| Finding | File | Vấn đề / Câu hỏi |
|---------|------|------------------|
| [Auth change] | [file:line] | [Cần confirm: logic mới có cover case X không?] |
**Dừng lại — cần senior confirm những điểm trên trước khi proceed.**
### 🔴 Issues cần fix ngay
| Severity | File | Vấn đề | Fix |
|----------|------|--------|-----|
| Critical | [file:line] | [SQL injection risk] | [Parameterized query] |
| High | [file:line] | [...] | [...] |
### 🟡 Improvements nên làm
| File | Đề xuất |
|------|---------|
| [file:line] | [...] |
### ✅ Security checks OK
- [ ] Input validation: OK
- [ ] SQL injection: OK
- [ ] Auth logic: [Pending review / OK]
- [ ] Sensitive data logging: OK
- [ ] Dependencies: OK / [X CVEs cần update]
What ships with it
2 files beside SKILL.md in the same directory: the scripts, references and assets a skill reads on demand. Not counted in the per-session cost; read them before you install if any of them is executable.
What this file has done since we first saw it
Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.
- 7d ago First seen · 146 lines · 76 tokens per session scan A 292f8af0a768
sec-review is a skill published in the GitHub repository hiepdnh/Agentic-Development-Lifecycle (6 stars, last pushed 10d ago), licensed MIT. It adds 76 tokens to every session and 1,332 once invoked, about $0.0004 per session on Opus 5. A static security scan graded it A with 1 finding (makes network calls). No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-31.
Other skills, from other repositories
systematic-debugging
Use when encountering any bug, test failure, or unexpected behavior, before proposing fixes.
local-ai-agents
Build local-first AI agents that run entirely on a developer workstation with Microsoft Foundry Local and Qwen function-calling models. Covers Small Language Models (SLMs), the OpenAI-compatible local endpoint, sandboxed local tools, local RAG with Chroma, local MCP servers, hybrid cloud/local routing, and the…
next-cache-components-adoption
Turn on Cache Components in a Next.js app and resolve the blocking routes it surfaces. Use when the user wants to enable, adopt, or migrate to Cache Components, flip the cacheComponents flag, work through a flood of blocking-prerender / instant validation errors, run the cache-components-instant-false codemod, or…
next-partial-prefetching-adoption
Turn on Partial Prefetching in a Next.js app and work through the insights it surfaces. Use when the user wants to enable or adopt Partial Prefetching, flip the partialPrefetching flag, opt routes in with export const prefetch = 'partial', audit Link prefetch={true} behavior, preserve existing prefetched UI with…
chronicle
Analyze Copilot session history for standup reports, usage tips, session search, and session reindexing. Use when the user asks for a standup, daily summary, usage tips, workflow recommendations, wants to search or find past sessions by keyword/file/PR, wants to reindex their session store, or asks about deleting…
babysit-pr
Babysit a GitHub pull request after creation by continuously polling review comments, CI checks/workflow runs, and mergeability state until the PR is merged/closed or user help is required. Diagnose failures, retry likely flaky failures up to 3 times, auto-fix/push branch-related issues when appropriate, and keep…