Skip to content

Add electrical-engineer persona, schematic-compliance-audit protocol, and review-schematic template#97

Merged
Alan-Jowett merged 5 commits intomicrosoft:mainfrom
Alan-Jowett:add-electrical-engineer-phase1
Mar 29, 2026
Merged

Add electrical-engineer persona, schematic-compliance-audit protocol, and review-schematic template#97
Alan-Jowett merged 5 commits intomicrosoft:mainfrom
Alan-Jowett:add-electrical-engineer-phase1

Conversation

@Alan-Jowett
Copy link
Copy Markdown
Member

Summary

Phase 1 of #91 — PromptKit's first non-software engineering domain. Adds electrical engineering schematic review capability.

New Components

Type Name Path Description
Persona electrical-engineer personas/electrical-engineer.md Power delivery, signal integrity, voltage domains, component selection
Protocol schematic-compliance-audit protocols/analysis/schematic-compliance-audit.md 7-phase systematic schematic review
Template review-schematic templates/review-schematic.md Audit netlist/schematic against requirements + datasheets

Design Decisions

  • Protocol type: analysis (like memory-safety-c), not reasoning — this is a domain-specific checklist methodology applied to a concrete artifact type.
  • 7 phases cover the standard EE schematic review checklist: power architecture, pin-level audit, bus integrity (I2C/SPI/UART/USB), protection circuits, power sequencing, passive component verification, and completeness.
  • Format: Reuses investigation-report — findings with severity, affected components (reference designators), and remediation.
  • Persona behavioral constraints are EE-specific: thinks in voltage domains, traces current paths, conservative about datasheet margins, audits every IC pin.
  • Template placed under code-analysis for now — may warrant a hardware-analysis category as more hardware templates land.

Remaining phases (#91)

  • Phase 2: simulation-validation protocol + validate-simulation template
  • Phase 3: BOM and layout protocols/templates
  • Phase 4: hardware-lifecycle pipeline definition

Checklist

  • All files have SPDX license headers
  • YAML frontmatter is valid and complete
  • Component names match file names (kebab-case)
  • manifest.yaml updated with all new components
  • No vague instructions in protocols or templates
  • Protocols have numbered, ordered phases
  • Templates have a quality checklist section
  • New components do not conflict with existing ones
  • python tests/validate-manifest.py passes

… and review-schematic template

Phase 1 of microsoft#91 — PromptKit's first non-software engineering domain.

- Persona: electrical-engineer — expertise in power delivery, signal
  integrity, voltage domain crossings, ESD protection, component
  selection, and standard interfaces. Conservative about datasheet
  margins. Thinks in voltage domains and current paths.

- Protocol: schematic-compliance-audit (analysis) — 7-phase systematic
  schematic review: power architecture, pin-level audit, bus integrity,
  protection circuits, power sequencing/reset, passive component
  verification, and completeness check.

- Template: review-schematic — audits a netlist or schematic against
  requirements and datasheets. Produces investigation-report with
  findings categorized by protocol phase.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 28, 2026 16:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds PromptKit’s first electrical-engineering domain components to enable schematic/netlist audits against requirements and datasheets, using the existing investigation-report output format.

Changes:

  • Introduces a new electrical-engineer persona for EE-focused review behavior and constraints.
  • Adds schematic-compliance-audit analysis protocol with a 7-phase schematic review methodology.
  • Adds review-schematic template and registers all new components in manifest.yaml.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
templates/review-schematic.md New template that orchestrates schematic/netlist review using the new persona + protocol and investigation-report format.
protocols/analysis/schematic-compliance-audit.md New 7-phase EE schematic audit protocol (power, pins, buses, protection, sequencing, passives, completeness).
personas/electrical-engineer.md New EE persona defining expertise areas and behavioral constraints for schematic reviews.
manifest.yaml Registers the new persona, protocol, and template under the existing manifest structure.

- Align epistemic label to [ASSUMPTION] per anti-hallucination protocol
- Use global severity ordering instead of grouping by category
- Remove references to not-yet-existing templates in Non-Goals

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

- Protocol: reframe USB differential pair check as schematic-verifiable
  connectivity (not layout routing). Add layout carry-forward note.
- Protocol: reframe ESD placement check as schematic topology
  verification. Flag physical placement as layout carry-forward.
- Template: soften layout non-goal to allow noting layout-critical
  constraints as carry-forward items.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@Alan-Jowett Alan-Jowett merged commit eff0cbd into microsoft:main Mar 29, 2026
6 checks passed
@Alan-Jowett Alan-Jowett deleted the add-electrical-engineer-phase1 branch March 29, 2026 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants