| name | pm-product-review |
| description | Orchestrated review of a PM artifact (spec, PRD, OKRs, roadmap, prioritization, strategy). Routes by artifact type to the right audits, auto-runs the always-relevant assumption check, and returns one prioritized report. Trigger when the user asks for a full review of a PM document, "review my spec/roadmap/OKRs", "is this PRD solid", or general PM feedback — but NOT when they name a single specific audit. |
| license | MIT |
| metadata | {"author":"uxcel","version":"1.0.0"} |
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.
