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.
In this repository20 other files it gives its agents.
- CLAUDE.mdCLAUDE.md
- component-checkSkill
- component-selectSkill
- debug-assistSkill
- design-assistSkill
- eng-calcSkill
- git-workflowSkill
- hardware-checklistSkill
- hardware-specificationSkill
- hw-git-changelogSkill
- hw-git-readmeSkill
- hw-git-session-logSkill
- hw-phase-recordSkill
- part-spec-tableSkill
- pcb-inspectSkill
- pcb-reviewSkill
- price-compare-tableSkill
- progress-updateSkill
- sch-inspectSkill
- stock-selectSkill
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
- Very specific: line 27
Keep schematic evidence, calculations, datasheet requirements, inferences and unknowns apart.
- Distinctive: line 32
Check part-specific claims against the actual MPN and current documentation.
- Very specific: line 44
Never open or read a PDF. Datasheet values come from the user, official manufacturer web pages, or NotebookLM answers with cited text, used …
- Very specific: line 50
If the project has .claude/rules/engineering.md, follow it.
- Distinctive: line 76
Challenge assumptions: Test at least one adverse condition where it applies: input extremes, transients, reverse current, reset, an …
- Distinctive: line 81
Stop at the requested scope.
- Distinctive: line 135
If no supported issue is found, say "No issue found within the inspected scope", not "schematic verified" or "ready for fabrication".
≈ “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.
Files
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) orNOT_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
- 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.
- Collect targeted evidence. Query the saved
.kicad_sch(withsch-inspectif 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. - 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.
- 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.
- 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.
- 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.
- 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) orINFO(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. Itsproject-netresults 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-selectandstock-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:
- Target and scope: block or sheet, operating assumptions, evidence reviewed.
- Findings: one row per finding, with severity, reference/pin/net, status, evidence label, issue, evidence source, impact and recommended action.
- NOT_VERIFIED: missing requirements, ambiguous connectivity, unchecked datasheet data, or untested behavior.
- 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 thresholdIf no supported issue is found, say "No issue found within the inspected scope", not "schematic verified" or "ready for fabrication".
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
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.

