agentleFS
NatchpaiNatchpai/Hardware-Copilot/.claude/skills/sch-review/SKILL.md

sch-review

Skill0 starsChanged 19 days agoMITView on GitHub

Reviews an electronic schematic or a focused circuit block for engineering correctness, risks, missing protection, component suitability and design assumptions. Use when the user asks for a schematic review, a circuit design check, a power or signal-path review, or an assessment of a KiCad schematic before PCB layout. Not for looking up schematic data only.

The file

We split this skill into the rules it gives an agent

7 rules. About 0 are common, 4 are distinctive, and 3 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈4

    Distinctive · estimate

    This author's own choices. Worth a look.

  • 3

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  1. Very specific:

    Keep schematic evidence, calculations, datasheet requirements, inferences and unknowns apart.

    line 27
  2. Distinctive:

    Check part-specific claims against the actual MPN and current documentation.

    line 32
  3. Very specific:

    Never open or read a PDF. Datasheet values come from the user, official manufacturer web pages, or NotebookLM answers with cited text, used …

    line 44
  4. Very specific:

    If the project has .claude/rules/engineering.md, follow it.

    line 50
  5. Distinctive:

    Challenge assumptions: Test at least one adverse condition where it applies: input extremes, transients, reverse current, reset, an …

    line 76
  6. Distinctive:

    Stop at the requested scope.

    line 81
  7. Distinctive:

    If no supported issue is found, say "No issue found within the inspected scope", not "schematic verified" or "ready for fabrication".

    line 135

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Schematic Engineering Review

Review the engineering design, not just the schematic's structure. Pick the checks from the circuit's purpose and constraints, not from a fixed topology, manufacturer or device checklist.

Boundaries

  • KiCad schematic, project, library, BOM and review records are read-only. Recommend changes to symbols, connectivity, review status or approvals; do not apply them.
  • A targeted review rates the inspected scope only. It is not a project-wide PASS, a gate closure, or part of a formal review framework.
  • Query the requested block or concern only. Use targeted queries, not full schematic dumps or a full netlist export.
  • Keep schematic evidence, calculations, datasheet requirements, inferences and unknowns apart. An unsupported or ambiguous connection is NOT_VERIFIED.
  • Matching net labels, a parser's proposed connection, ERC success, or no visible errors do not show electrical correctness.
  • Check part-specific claims against the actual MPN and current documentation.
  • Changes, purchases, commits, reports and logs happen only when the user asks and the repository rules allow them.
  • ERC, simulation, lab measurement, PCB checks and formal release are separate activities. Recommend them where needed and say they were not performed.

Evidence baseline

  • Label each significant claim VERIFIED (named evidence: CAD data, a calculation or an official source), ASSUMED (assumption stated), ESTIMATED (method and margin stated) or NOT_VERIFIED (evidence missing or check not run).
  • Never open or read a PDF. Datasheet values come from the user, official manufacturer web pages, or NotebookLM answers with cited text, used only after the user approves uploading the file. With none of these, the value is NOT_VERIFIED.
  • KiCad design and library files are read-only. Propose changes; do not apply them.
  • If the project has .claude/rules/engineering.md, follow it; it is the full policy.

Workflow

  1. Scope the review. The block's intended function, its known operating range, and the review goal. Prefer the user's specific concern over a project-wide sweep. Ask for a missing constraint only if going on without it would mislead; otherwise state the assumption.
  2. Collect targeted evidence. Query the saved .kicad_sch (with sch-inspect if installed) for the relevant symbols, pins, local, hierarchical and project connectivity, and instance paths. Record the scope it actually covered and any parser limits. Ask for a human visual check where native data cannot settle a question.
  3. Rebuild the intended behavior. Signal and power path, operating modes, startup and shutdown, interfaces, expected loading and fault cases for the block. Being wired is not the same as working.
  4. Choose risk-based checks. Operating limits, component stress, power paths, decoupling, biasing, pull-ups and pull-downs, floating pins, protection, feedback stability, timing, signal integrity, grounding, sequencing and interface compatibility, as they apply. Report only the checks you actually ran.
  5. Check calculations and components. Calculate stress, tolerance, thermal, timing or worst-case numbers; check pin mapping, MPN, package and datasheet requirements. Look at alternatives or sourcing only when the user asks for them, or a finding cannot be resolved without them.
  6. Challenge assumptions. Test at least one adverse condition where it applies: input extremes, transients, reverse current, reset, an unpowered interface, startup or a short circuit. Keep a demonstrated defect apart from a plausible risk that needs more evidence.
  7. Report and stop. Lead with actionable findings and targeted next checks. Stop at the requested scope. Where evidence is missing, report the gap instead of a verification result.

Findings

Give each finding a severity and a status:

  • Severity: CRITICAL (credible hazard or severe defect with clear evidence), MAJOR (likely functional or reliability issue), MINOR (bounded margin or documentation issue) or INFO (observation, not a defect). Severity rates the individual finding, not the whole schematic.
  • Status: Confirmed defect (direct CAD, datasheet or calculation evidence shows a specific mismatch), Potential risk (a plausible concern that lacks the constraint, model or inspection to confirm it), or NOT_VERIFIED (the evidence is not enough to decide; not a PASS or FAIL).

State uncertainties plainly, and keep hypothetical worst cases as risks, not proven faults.

Optional companions

  • sch-inspect: scoped structural evidence. Its project-net results are not full KiCad netlist or ERC parity. Respect its limits on buses, inherited symbols, hidden connectivity and ambiguous instances.
  • eng-calc: auditable equations, units, tolerances and worst cases.
  • component-check: datasheet and physical mapping of a chosen device.
  • component-select and stock-select: alternatives and sourcing.
  • design-assist: architecture alternatives when the review raises a basic design-choice question.

These help if installed. If one is missing, do the step directly under the same evidence rules, or mark the dependent check NOT_VERIFIED.

Output

Match the size of the output to the requested review. For a full review:

  1. Target and scope: block or sheet, operating assumptions, evidence reviewed.
  2. Findings: one row per finding, with severity, reference/pin/net, status, evidence label, issue, evidence source, impact and recommended action.
  3. NOT_VERIFIED: missing requirements, ambiguous connectivity, unchecked datasheet data, or untested behavior.
  4. Next checks: a short, ordered list of the visual checks, calculations, simulations, ERC runs or measurements that would close the gaps.

Example finding row:

MAJOR | U3 pin 4 / EN | Potential risk | VERIFIED | EN is tied to VIN through R12 only; no divider for UVLO | local-net query, datasheet table 6 (via NotebookLM) | Startup below the intended input voltage | Add a divider or confirm the intended threshold

If no supported issue is found, say "No issue found within the inspected scope", not "schematic verified" or "ready for fabrication".

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.