# Design Review

> >-

- **Type:** Skill
- **Install:** `agentstack add skill-nickkraakman-skidl-skills-design-review`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [nickkraakman](https://agentstack.voostack.com/s/nickkraakman)
- **Installs:** 0
- **Category:** [Agent Skills](https://agentstack.voostack.com/c/agent-skills)
- **Latest version:** 0.1.0
- **License:** MIT
- **Upstream author:** [nickkraakman](https://github.com/nickkraakman)
- **Source:** https://github.com/nickkraakman/skidl-skills/tree/master/skills/design-review

## Install

```sh
agentstack add skill-nickkraakman-skidl-skills-design-review
```

Requires the [AgentStack CLI](https://agentstack.voostack.com/docs/cli). Works with Claude Code, Cursor, and any MCP-compatible agent.

## About

# Design Review

## When to use
- Before exporting a netlist or generating a BOM
- After skidl-coder finishes a circuit
- User says "review the circuit", "design review", or "check the design"
- Invoked via `/skidl-ee:design-review [circuit_name]`

## Inputs
- `circuits/.py` — SKiDL source
- `SPEC.md` — captured design requirements
- `architecture/` folder (if present) — block diagram and net plan
- `pipeline_state.json` — current pipeline status

## Procedure

Run these three checks (in parallel where possible):

### 1. Electrical Rules (ERC)
- Spawn the `erc-reviewer` agent (it applies the erc-rules skill automatically)
- Receive `erc_report.md` with pass/fail verdict

### 2. Architecture Sanity
- Re-read `SPEC.md` and `architecture/` folder
- Check:
  - All functional blocks present in the circuit?
  - Power budget realistic?
  - Net plan complete and consistent with code?
  - Known design risks (decoupling, protection, EMI) addressed?

### 3. SKiDL Code Quality
- Read current `circuits/*.py` file(s)
- Verify using the `.claude/rules/skidl-syntax.md` rules:
  - Every part has `ref`, `value`, and `footprint`?
  - `@subcircuit` used for repeated blocks?
  - Decoupling caps named `C_DECOUP_*`?
  - `NC` on all intentionally unused pins?
  - Net naming: UPPERCASE power rails, camelCase signals?
  - `ERC()` call present in `__main__`?

### 4. Footprint Validation
- Run: `python3 .claude/scripts/validate-footprints.py circuits//` (or `.py` for monolithic)
- Every missing footprint is a **HIGH** severity finding — it will cause KiCad import errors
- For each missing footprint in the report:
  - If the script suggests close matches → recommend the corrected string
  - If no standard library match exists → generate a custom footprint:
    1. Read the datasheet mechanical drawing from `datasheets/_SUMMARY.md` or PDF
    2. Run `.claude/scripts/generate-footprint.py` with pad positions and sizes
    3. Update the circuit code to reference `ProjectLocal:FootprintName`
- Do not mark review complete until footprint validation exits 0

## Output

Write `outputs/design_review.md` using this structure:

```
# Design Review — 

## Summary
ERC: PASS/FAIL | Architecture: OK/Issues | Code: OK/Issues
Severity:  HIGH,  MEDIUM,  LOW issues

## Findings
(sorted HIGH → MEDIUM → LOW)

| # | Severity | Area | Issue | Suggested Fix |
|---|----------|------|-------|---------------|
| 1 | HIGH | ERC | Two OUT pins on net VCC | Add series resistor |
...
```

## Validation

- Every HIGH-severity finding must include a specific SKiDL fix snippet
- Update `pipeline_state.json` with `review_status` after writing the report
- If HIGH severity issues found → offer to auto-fix by spawning `skidl-coder` with the report
- If no issues → declare circuit ready for BOM/netlist export
- Do not mark review complete until all three checks have been run

## Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

- **Author:** [nickkraakman](https://github.com/nickkraakman)
- **Source:** [nickkraakman/skidl-skills](https://github.com/nickkraakman/skidl-skills)
- **License:** MIT

Install and usage instructions live in the source repository linked above.

## Pricing

- **Free** — Free

## Security capabilities

Automated source analysis of v0.1.0 — what this tool can access:

- **Network access:** no
- **Filesystem access:** no
- **Shell / process execution:** no
- **Environment & secrets:** no
- **Dynamic code execution:** no

*"Yes" means the capability is present in the source — more access means more to trust, not that it is unsafe.*


## Versions

- **0.1.0** — security scan: passed — Imported from the upstream source.

## Links

- Listing page: https://agentstack.voostack.com/l/skill-nickkraakman-skidl-skills-design-review
- Seller: https://agentstack.voostack.com/s/nickkraakman
- Browse the marketplace: https://agentstack.voostack.com/browse

---
Listed on AgentStack — the marketplace for AI agent skills and MCP servers. Every listing is security-reviewed. Creators keep 70%.