CtrlK
BlogDocsLog inGet started
Tessl Logo

security-review

認証の追加、ユーザー入力の処理、シークレットの操作、APIエンドポイントの作成、支払い/機密機能の実装時にこのスキルを使用します。包括的なセキュリティチェックリストとパターンを提供します。

56

Quality

63%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./docs/ja-JP/skills/security-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

57%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The content is a well-organized, largely actionable security checklist with concrete code examples and per-topic verification steps — genuinely useful as a review aid. Its weaknesses are length and redundancy (much of it re-teaches security basics Claude already knows), the absence of any ordered review workflow, and a fully monolithic structure with no progressive disclosure despite being far over the 50-line guideline.

Suggestions

Move topic deep-dives (e.g., Solana/blockchain security, full RLS SQL, dependency management) into references/ files and keep only the checklist summaries plus pointers in SKILL.md, cutting the body well under 200 lines.

Add an explicit ordered review workflow at the top — e.g., 1. identify the changed surface (auth, input, queries, secrets), 2. apply the matching topic checklist, 3. verify findings against the pre-deploy checklist — so the sections compose into a process.

Fix or remove non-executable examples: the fabricated "verify" export from @solana/web3.js, the undefined "@/lib/csrf" module, and the unimplemented getBalance call, so all code is copy-paste ready.

Trim material Claude already knows (DOMPurify usage, express-rate-limit config, npm audit commands) down to one-line checklist items, and drop the motivational closing paragraph.

DimensionReasoningScore

Conciseness

The body is mostly tight checklists and code with little prose padding, but at ~490 lines it teaches security fundamentals Claude already knows (DOMPurify sanitization, express-rate-limit configuration, npm audit commands, httpOnly cookie basics), plus a motivational closing section ("セキュリティはオプションではありません…") that adds no actionable value. This fits anchor 3 (mostly efficient but includes unnecessary material that could be tightened) — above anchor 2 because the structure is not padded explanation, below anchor 4 because the volume of already-known material is more than minor.

3 / 5

Actionability

Most guidance is concrete and executable: complete TypeScript/SQL/bash examples for each topic (zod schemas, Supabase RLS policies, CSP headers, rate limiter configs) plus specific per-topic checklists ("検証ステップ") and a pre-deploy checklist. It is not anchor 5 because several examples are non-executable pseudocode — "import { verify } from '@solana/web3.js'" (no such export), the undefined "@/lib/csrf" module, and the unimplemented "getBalance" — and not anchor 3 because the large majority of the code is genuinely copy-paste-adaptable.

4 / 5

Workflow Clarity

Topics are well organized and each has its own verification checklist, but there is no sequenced process for actually conducting a security review (e.g., scan the diff → apply each topic checklist → report findings), and the checks are static checkbox lists rather than validate-and-retry feedback loops. Anchor 3 (steps/checks present per section but no overall sequence or explicit checkpoints) fits — above anchor 2 because each topic's steps and checks are well defined, below anchor 4 because the sections never compose into an ordered workflow.

3 / 5

Progressive Disclosure

There are no bundle files at all — everything, including deep-dive material (Solana security, full RLS policy SQL, dependency management), is inlined in a single ~490-line SKILL.md. The file is well-sectioned with clear headers (above anchor 2's no-structure example), and the "リソース" section points to external docs, but bulk reference content that belongs in references/ files is inline with no one-level-deep pointers — anchor 3, not 4 because the content is not appropriately split across files.

3 / 5

Total

13

/

20

Passed

Description

70%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description has a strong, explicit multi-trigger "use when" clause covering the right security-sensitive scenarios, and it is clearly distinct from other skills. Its main weakness is a generic "what" — "comprehensive security checklist and patterns" — that never names the concrete capabilities the skill provides, and it misses common trigger vocabulary like "security review" or "vulnerability".

Suggestions

Make the "what" concrete: instead of "包括的なセキュリティチェックリストとパターンを提供します", enumerate 3-4 specific capabilities such as identifying hardcoded secrets, verifying parameterized queries, checking auth/authorization patterns, and validating file uploads.

Add missing natural trigger terms users would say: "セキュリティレビュー" (security review), "脆弱性" (vulnerability), "監査/audit", and common vulnerability names (XSS, SQLインジェクション, CSRF).

Consider covering remaining scenarios from the body's trigger list (file uploads, sensitive-data storage, third-party API integration) so the description's "when" matches the skill's full scope.

DimensionReasoningScore

Specificity

The description names the security domain and a single generic deliverable — "包括的なセキュリティチェックリストとパターンを提供します" (provides a comprehensive security checklist and patterns) — but lists no concrete actions such as identifying hardcoded secrets or checking queries for injection. This matches anchor 3 (domain plus 1-2 deliverables, not comprehensive); it is above anchor 2 because a checklist/patterns deliverable is more than minimal, and below anchor 4 because no specific capabilities are enumerated.

3 / 5

Completeness

Both parts are explicitly present: an explicit "when" clause ("…実装時にこのスキルを使用します" with five concrete trigger scenarios) and a "what" ("包括的なセキュリティチェックリストとパターンを提供します"). The "when" is strong and multi-trigger, but the "what" is generic rather than concrete, so it fits anchor 4 — not 5 (the "what" is not crisply specific) and not 3 (the "when" is explicit, not weakly implied).

4 / 5

Trigger Term Quality

Natural trigger phrases a user would say are present: "認証の追加" (adding authentication), "ユーザー入力の処理" (processing user input), "シークレットの操作" (handling secrets), "APIエンドポイントの作成", "支払い/機密機能の実装" (payments/sensitive features). Good coverage but missing common synonyms and vocabulary users would use, such as "security review", "脆弱性" (vulnerability), "audit", or named vulnerability classes — anchor 4, not 5 (synonym coverage incomplete), not 3 (coverage is genuinely good rather than partial).

4 / 5

Distinctiveness Conflict Risk

Security review is a clear niche with distinct triggers (secrets handling, payment implementation, API endpoint creation, authentication), making it mostly distinguishable from other skills. Minor overlap risk remains with general code-review skills, so anchor 4 fits better than 5; it is clearly above anchor 3 because the trigger scenarios would not naturally invoke an unrelated skill.

4 / 5

Total

15

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
affaan-m/ECC
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.