# Review Code Fundamentals

Small readability problems make code reviews slower and harder to maintain. This skill reviews naming, structure, reuse, and comments so Claude, Codex, or Claude Code can return clear pass or suggestion guidance.

## Install

```bash
npx skillstore add danielpodolsky/fundamentals-gate
```

## Metadata

- Status: approved
- Slug: danielpodolsky-fundamentals-gate
- Skillstore revision: r1
- Version status: missing
- Tree hash: f890794f5bfa5a51a7611943a5f9d98c1920f6127389c2f23fce5c19bd8f76fb
- Author: DanielPodolsky
- GitHub username: DanielPodolsky
- License: MIT
- Repository: https://github.com/DanielPodolsky/mentor-spec/tree/main/.claude/skills/gates/fundamentals
- Ref: 34f316ba14ef36c7a620fc09f2676d2429997a77
- Supported tools: Claude, Codex, Claude Code
- Audit status: complete
- Agent install advisory: allowed
- Manual install advisory: allowed
- Artifact signature: available
- Audit attestation: unavailable
- Human verification: not\_verified
- Risk factors: external\_commands
- Quality score: 79
- Quality tier: bronze
- Public page: https://skillstore.pages.dev/skills/danielpodolsky-fundamentals-gate
- Manifest: https://skillstore.pages.dev/api/skills/danielpodolsky-fundamentals-gate/manifest

## Capabilities

- Checks variable, boolean, function, constant, and class naming against readable naming patterns.
- Reviews function focus, approximate size, parameter count, nesting, and hidden side effects.
- Identifies repeated patterns that may benefit from extraction after enough duplication appears.
- Evaluates comments for useful intent and flags obvious or commented-out code.
- Returns either a pass status or concise suggestions with practical reasons.

## Use Cases

- Prepare Pull Requests: Check naming, function focus, comments, and duplication before requesting a full review.
- Coach Junior Engineers: Turn unclear code into learning suggestions that explain readability and maintainability tradeoffs.
- Standardize Review Feedback: Use a consistent fundamentals checklist across teams without making style polish a hard blocker.

## Prompt Templates

### Basic Fundamentals Check

```
Review this change with the fundamentals gate. Check naming, function focus, structure, comments, and reuse. Return PASS or SUGGESTIONS.
```

### Pull Request Polish Review

```
Apply the fundamentals checklist to this pull request. Separate must-fix concerns from polish suggestions. Keep each suggestion brief and actionable.
```

### Mentoring Review

```
Review this code as a mentor. Ask Socratic fundamentals questions for unclear naming, large functions, duplication, comments, and magic values.
```

### Advanced Consistency Audit

```
Compare this module against nearby conventions. Identify naming drift, repeated patterns, function scope issues, comment quality, and structure concerns. Prioritize suggestions by impact.
```

## Limitations

- It gives suggestions and does not act as a mandatory release blocker.
- It does not run tests, linters, type checkers, or security scanners.
- Referenced standards paths are not included in this skill package.
- It is not a substitute for domain-specific architecture or performance review.

## Best Practices

- Provide the surrounding files or conventions so suggestions match the existing codebase.
- Ask for pass or suggestions before a full code review to keep feedback focused.
- Treat each recommendation as a maintainability tradeoff, not an automatic rewrite request.

## Anti Patterns

- Do not use this skill as the only security, correctness, or production readiness review.
- Do not apply every suggestion when local conventions intentionally differ.
- Do not ask it to rewrite large modules without naming the review priorities.

## Security Audit

- Audited at: 2026-07-06T09:29:38.955\+00:00
- Summary: The static external-command detections are false positives caused by Markdown inline code and fenced examples in SKILL.md. The low blocker finding is also a false positive because PASS is a review status label, not reconnaissance. No prompt injection, data exfiltration, command execution, or malicious intent was found.

## Stats

- Views: 186
- Downloads: 13
- Favorites: 0
- Popularity score: 0