SuperAPI AI Directory

pm-product-review

General & other

View repository

Updated 10-16 06:51

About this skill

Product Review (PM Orchestrator) Skill

How this skill behaves (read first)

This is the front door for holistic PM-artifact review. It is the executable form of docs/orchestration-policy.md applied to PM work: it routes by artifact type to the relevant audits rather than firing all of them, auto-runs the one always-relevant lens (assumption rigor), and returns one prioritized report.

PM work has no "design system" suppression case, but it has the same two failures to avoid: a vague "looks good" review, and forcing every PM audit onto a document that only needs one.

When NOT to use this skill: if the user named a specific audit ("run spec-quality", "check my OKR metric validity"), skip this orchestrator and invoke that one directly. Explicit scope overrides orchestration.


Step 0 — Establish context (once)

  • Artifact type — spec/PRD, OKRs/KPIs, roadmap, prioritization, vision/strategy, GTM. This is the primary router. Infer it from the document; ask only if genuinely ambiguous.
  • Primary goal — what decision this artifact is meant to support (used for conflict resolution and prioritization).

Step 1 — Auto-run the always-relevant lens (Tier A)

  • pm-assumption-rigor-audit — runs whenever the artifact rests on unproven assumptions, which is nearly always true for PM work. It's cheap, high-leverage, and artifact-agnostic.

Each audit is named by its installable skill name. Invoke one only if that skill is installed; if one isn't, list it under "available but not run" in the report and don't block — this orchestrator degrades to whichever audits are present.


Step 2 — Route by artifact type (Tier B — run the matching one, offer adjacent ones)

Artifact Primary audit Adjacent lenses to offer
Spec / PRD pm-spec-quality-audit pm-assumption-rigor-audit (auto), pm-decision-quality-audit
OKRs / KPIs / metrics pm-okr-metric-validity-audit pm-assumption-rigor-audit (auto)
Roadmap / prioritization pm-prioritization-rigor-audit pm-decision-quality-audit, pm-assumption-rigor-audit (auto)
Vision / strategy pm-decision-quality-audit pm-assumption-rigor-audit (auto)
GTM plan pm-decision-quality-audit pm-assumption-rigor-audit (auto)

Run the primary audit for the artifact type. Offer the adjacent lenses rather than firing them: "This is a roadmap, so I ran prioritization-rigor and the assumption check. Want me to also run a decision-quality pass?"


Step 3 — Merge into one prioritized report

De-duplicate across lenses, resolve conflicts toward the primary goal from Step 0, and output a single severity-ordered list:

## Product Review — [artifact]
Context: [artifact type · primary goal]
Lenses run: [spec-quality, assumption-rigor]   Available but not run: [decision-quality]

### Critical
- [Issue] — what's wrong → fix. (spec-quality / assumption / metric-validity / …)
### Major
- …
### Minor
- …
### Working well ✓
- …

**Top 3 priorities:** the highest-impact fixes across all lenses.
Offered, not run: decision-quality — say the word and I'll add it.

Common do/don't patterns

❌ Don't ✅ Do
Run every PM audit on any document Route by artifact type; auto-run only the assumption check
Return one report per audit Merge into a single prioritized, de-duplicated list
Run this orchestrator when the user asked for one named audit Defer to that audit directly (explicit scope wins)
Treat strategy and a spec the same way Route: spec → spec-quality, OKRs → metric-validity, roadmap → prioritization-rigor

Relationship to the policy

This skill is docs/orchestration-policy.md in executable form for PM work. If the two ever diverge, the policy governs; update this skill to match it.

Source lessons (Uxcel)

This is an orchestrator — its substance lives in the audits it routes to. See the source lessons in each: pm-assumption-rigor-audit, pm-spec-quality-audit, pm-okr-metric-validity-audit, pm-prioritization-rigor-audit, pm-decision-quality-audit.