Skip to content
Back to skills

Code Review 56

ASecurity

Validates TypeScript/React code against dev_instruction_v3.md standards. Checks for anti-patterns like export default, onClick usage, wrapper components, and any types. Use before committing or during code review.

  • 2 stars
  • 0 votes
  • 0 copies
  • 0 views
  • Added September 27, 2026
developmenttypescriptbashreactcode-review

Works with

  • cli

Security analysis

A100/100

Pro scans all 2 files and shows the line behind each finding

Scanned September 27, 2026

npx -y skills add David-Li0406/meta-skill-evloving --skill code-review-56 --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Code Review 56?

Add the live security badge to your README. It updates with every re-scan.

Security grade badge for Code Review 56
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/david-li0406-code-review-56/badge)](https://www.skillsdirectory.com/skills/david-li0406-code-review-56)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

Download with Pro
SKILL.md
---
name: code-review
description: Validates TypeScript/React code against dev_instruction_v3.md standards. Checks for anti-patterns like export default, onClick usage, wrapper components, and any types. Use before committing or during code review.
---

# Code Review Skill

Validates code against project standards from `dev_instruction_v3.md`.

## Anti-Patterns to Detect

| Pattern | Violation | Correct |
|---------|-----------|---------|
| `export default` | ❌ Forbidden | `export function` |
| `onClick` on HeroUI | ❌ Use onPress | `onPress={handler}` |
| Wrapper imports | ❌ No wrappers | Import from `@heroui/react` |
| `any` type | ❌ Strict typing | Define proper types |
| `"use client"` | ❌ Not for Vite SPA | Remove directive |

## How to Check

Run the standards check script:

```bash
.agent/skills/code-review/scripts/check-standards.sh src/
```

## Manual Review Checklist

- [ ] Named exports only (`export function`)
- [ ] No `any` types
- [ ] No wrapper components
- [ ] `onPress` for HeroUI interactions
- [ ] Mock data in `src/data/`
- [ ] Logic extracted to `src/hooks/`
- [ ] Data fetching uses `useQuery`
- [ ] Tested in both light and dark themes

## Reference

See `dev_instruction_v3.md` for complete coding standards.

Files in this skill

  • SKILL.md1.2 KB
  • scripts/check-standards.sh1.8 KB

Attribution

Is this your skill, or is something wrong with this listing? Request removal or report an issue. Author removals are honored within 72 hours.

Comments

Loading comments…