code-reviewer

Version: 1.0.0 Purpose: Comprehensive code review with quality scoring

Safety Notice

This listing is imported from skills.sh public index metadata. Review upstream SKILL.md and repository scripts before running.

Copy this and send it to your AI assistant to learn

Install skill "code-reviewer" with this command: npx skills add shunsukehayashi/miyabi/shunsukehayashi-miyabi-code-reviewer

Code Reviewer

Version: 1.0.0 Purpose: Comprehensive code review with quality scoring

Triggers

Trigger Examples

Review request "review this code", "check this PR", "コードレビューして"

Quality check "is this code good?", "品質チェック"

Bug hunt "find bugs", "バグを探して"

Review Checklist

  1. Correctness

// Check for off-by-one errors for (let i = 0; i < array.length; i++) { } // Correct for (let i = 0; i <= array.length; i++) { } // Bug

// Check null handling const value = obj?.property ?? defaultValue;

  • Logic is correct

  • Edge cases handled

  • Error handling present

  • Null/undefined checks

  1. Security

// SQL injection prevention const query = 'SELECT * FROM users WHERE id = $1'; await db.query(query, [userId]); // Parameterized

// XSS prevention const safe = DOMPurify.sanitize(userInput);

  • Input validation

  • No SQL injection

  • No XSS vulnerabilities

  • Secrets not hardcoded

  1. Performance

// Avoid N+1 queries const users = await db.query( SELECT u.*, array_agg(o.*) as orders FROM users u LEFT JOIN orders o ON u.id = o.user_id GROUP BY u.id);

  • No N+1 queries

  • Proper indexing

  • Efficient algorithms

  • Memory management

  1. Maintainability
  • Clear naming

  • Single responsibility

  • DRY principle

  • Proper types (TypeScript)

Quality Score (100 points)

Category Weight Criteria

Correctness 30 Logic, edge cases, error handling

Security 25 OWASP Top 10 compliance

Performance 20 Efficiency, no bottlenecks

Maintainability 15 Readability, structure

Testing 10 Coverage, quality

Pass threshold: 80+ points

Output Format

{ "score": 85, "issues": [ { "severity": "warning", "file": "src/api.ts", "line": 42, "message": "Consider adding error handling" } ], "suggestions": ["Add unit tests", "Extract helper function"] }

Source Transparency

This detail page is rendered from real SKILL.md content. Trust labels are metadata-based hints, not a safety guarantee.

Related Skills

Related by shared tags or category signals.

General

ccg

No summary provided by upstream source.

Repository SourceNeeds Review
General

test-generator

No summary provided by upstream source.

Repository SourceNeeds Review
General

doc-generator

No summary provided by upstream source.

Repository SourceNeeds Review
General

commit-helper

No summary provided by upstream source.

Repository SourceNeeds Review