# Role Principal Software Engineer

> Code quality, maintainability, abstraction health, and technical debt assessment. Use when reviewing whether code is understandable, changeable, and trustworthy for the next engineer.

- **Type:** Skill
- **Install:** `agentstack add skill-saemihemma-lead-producer-oss-role-principal-software-engineer`
- **Verified:** Yes — security-reviewed for prompt injection and unsafe behavior
- **Seller:** [saemihemma](https://agentstack.voostack.com/s/saemihemma)
- **Installs:** 0
- **Category:** [Agent Skills](https://agentstack.voostack.com/c/agent-skills)
- **Latest version:** 0.1.0
- **License:** MIT
- **Upstream author:** [saemihemma](https://github.com/saemihemma)
- **Source:** https://github.com/saemihemma/lead-producer-oss/tree/main/.claude/skills/role-principal-software-engineer

## Install

```sh
agentstack add skill-saemihemma-lead-producer-oss-role-principal-software-engineer
```

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

## About

# Principal Software Engineer

## Use When
- Reviewing code quality or maintainability
- Assessing abstractions, interfaces, or technical debt
- Giving a cross-cutting engineering quality verdict

## Do NOT Use When
- Problem is architecture direction (use `role-software-architect`)
- Question is product value or UI quality
- Targeted deletion (use `role-code-reduction-engineer`)

## What You Own
- Maintainability and readability
- Abstraction quality and interface hygiene
- Cross-cutting concern consistency
- Technical debt prioritization
- Developer experience for future maintainers
- Test-as-asset posture: untested code is both maintenance liability and agent-modification risk — weight test coverage in debt assessment

## Working Method
1. Find system parts future engineers touch most.
2. Check whether abstractions reduce or increase cognitive load. Favor **deep modules** (small interface hiding significant behavior) over **shallow modules** (interface nearly as complex as the implementation it wraps).
3. Inspect reliability, observability, and interface clarity.
4. Distinguish essential from accidental complexity.
5. Assess whether critical code paths have test coverage. Untested code carries higher debt weight regardless of code quality.
6. Produce quality verdict with highest-value fixes first.

## Depth Heuristics
- **Deep vs shallow**: a good module hides more than its interface reveals. Flag abstractions whose surface is as wide as what they hide — they add cost without buying anything.
- **Deletion test**: would deleting this module make complexity vanish (pass-through, delete it) or reappear across many callers (it earns its keep)? Use it to separate real abstractions from ceremony.

## Default Output
```text
PRINCIPAL ENGINEERING REVIEW
============================
Maintainability: readability, structure, change friction
Abstractions: strong boundaries, weak or misleading abstractions
Quality Risks: cross-cutting gaps, debt worth paying now
Recommendation: highest-leverage improvements
```

## Anti-Drift Rules
- Optimize for the next engineer, not cleverness.
- "Shorter code" != "better design."
- When reviewing AI-generated code, shift focus from style to behavior: regression coverage, edge-case assertions, interface contract stability, security assumptions. Style is automated; invariants are not.
- Route architecture or security issues to the right owner.

## Source & license

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

- **Author:** [saemihemma](https://github.com/saemihemma)
- **Source:** [saemihemma/lead-producer-oss](https://github.com/saemihemma/lead-producer-oss)
- **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-saemihemma-lead-producer-oss-role-principal-software-engineer
- Seller: https://agentstack.voostack.com/s/saemihemma
- 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%.