# Review Engineering Fundamentals

Teams often need consistent feedback on readability, naming, and code organization. This skill gives Claude, Codex, and Claude Code a practical checklist for everyday code quality review.

## Install

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

## Metadata

- Status: approved
- Slug: danielpodolsky-engineering-fundamentals
- Skillstore revision: r1
- Version status: missing
- Tree hash: 40548e826ce50e1f4cdc1b79995d847def9ab0c38b5850e25416b53b954877df
- Author: DanielPodolsky
- GitHub username: DanielPodolsky
- License: MIT
- Repository: https://github.com/DanielPodolsky/mentor-spec/tree/main/.claude/skills/fundamentals/engineering
- 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-engineering-fundamentals
- Manifest: https://skillstore.pages.dev/api/skills/danielpodolsky-engineering-fundamentals/manifest

## Capabilities

- Guides reviews for naming, function design, organization, comments, and documentation.
- Flags common issues such as magic numbers, unclear names, deep nesting, and large functions.
- Applies SOLID review questions to classes, interfaces, dependencies, and extension points.
- Provides Socratic questions for coaching junior developers during review.
- Supplies naming conventions for variables, constants, functions, classes, and files.

## Use Cases

- Review a Pull Request: Apply a consistent checklist to names, functions, duplication, comments, and organization before merge.
- Coach Junior Developers: Use guided questions to help teammates identify readability and design improvements themselves.
- Plan Refactoring Work: Identify large functions, deep nesting, repeated logic, and premature abstractions before changing code.

## Prompt Templates

### Run a Basic Review

```
Review this code using engineering fundamentals. Focus on naming, function size, duplication, organization, and comments.
```

### Improve Naming and Structure

```
Check this change for unclear names, generic variables, boolean naming, function verbs, and related code organization.
```

### Find Refactoring Targets

```
Identify refactoring opportunities in this code. Prioritize large functions, deep nesting, magic values, repeated logic, and unjustified abstractions.
```

### Lead a Socratic Review

```
Review this code as a mentor. Ask targeted questions about readability, responsibility boundaries, duplication, abstraction value, and SOLID tradeoffs.
```

## Limitations

- It does not execute static analysis tools or inspect code automatically.
- It focuses on general code quality, not security testing or performance profiling.
- It gives review guidance, not framework-specific architecture rules.
- It references a standards document that may not exist in every workspace.

## Best Practices

- Use the checklist before merge, when changes are still easy to adjust.
- Ask review questions that teach reasoning instead of only listing fixes.
- Prioritize changes that improve readability, cohesion, and future maintenance.

## Anti Patterns

- Do not apply every rule mechanically when the local codebase has stronger conventions.
- Do not create abstractions before repeated use or clear responsibility boundaries exist.
- Do not use the skill as a substitute for tests, security review, or performance measurement.

## Security Audit

- Audited at: 2026-07-06T10:45:30.328\+00:00
- Summary: All static external command detections are false positives caused by Markdown inline code or fenced examples in SKILL.md. The skill is a prose checklist for code review and does not define scripts, shell commands, network calls, or prompt-injection text.

## Stats

- Views: 259
- Downloads: 12
- Favorites: 0
- Popularity score: 0