[IMPORTANT] Use TaskCreate to break ALL work into small tasks BEFORE starting.
Evidence Gate: MANDATORY IMPORTANT MUST — every claim requires file:line proof or traced evidence with confidence percentage (>80% to act).
Quick Summary
Goal: Help Dev BA PIC review BA drafters' PBI drafts by generating specific, actionable challenge prompts. AI provides analysis; human makes the decision.
Key distinction: Collaborative review tool (drafter → reviewer flow), NOT self-review (use /refine-review for AI self-review).
Be skeptical. Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence percentages (Idea should be more than 80%).
Frontend/UI Context (if applicable)
When this task involves frontend or UI changes, MUST READ .claude/skills/shared/ui-system-context.md and the following docs:
- Component patterns:
docs/project-reference/frontend-patterns-reference.md
- Styling/BEM guide:
docs/project-reference/scss-styling-guide.md
- Design system tokens:
docs/project-reference/design-system/README.md
Workflow
- Locate PBI draft — Find BA drafters' draft PBI in
team-artifacts/pbis/ or path provided by user
- Load protocols — Read these 3 protocols:
.claude/skills/shared/ba-team-decision-model-protocol.md (decision model, veto scope)
.claude/skills/shared/refinement-dor-checklist-protocol.md (DoR criteria)
.claude/skills/shared/cross-cutting-quality-concerns-protocol.md (authorization, seed data, migration)
- Load domain context — Auto-detect module from PBI content, load:
docs/project-reference/domain-entities-reference.md (entity definitions)
- Relevant feature docs from
docs/business-features/{App}/
- Existing business rules (BR-{MOD}-XXX) from feature docs
- Technical Feasibility Analysis:
- Can described features be built with the project's architecture?
- Any domain entity conflicts? (cross-reference entity definitions)
- Any cross-service implications? (message bus events, shared data between services)
- Estimated complexity alignment (does scope match story points?)
- AC Quality Analysis:
- Vagueness detector: flag "should", "might", "TBD", "etc.", "various", "appropriate"
- Coverage check: happy path + edge case + error case + authorization scenario
- Missing scenarios: suggest specific additions based on feature type
- Cross-Cutting Concerns Check:
- Authorization section present and complete? (roles × CRUD matrix)
- Seed data requirements addressed? (or explicit "N/A")
- Data migration implications? (schema changes)
- Performance considerations? (list/grid/export features)
- UI Layout section present? If PBI involves UI: must have
## UI Layout per ui-wireframe-protocol.md with wireframe + components (with tiers) + states + design tokens. If backend-only: explicit "N/A". Flag missing UI visualization as a gap.
- Generate Challenge Prompts — Output specific, actionable questions:
- NOT vague: "needs work" or "improve AC"
- SPECIFIC: "AC #2 says 'user can filter results' — which filters exactly? Suggest: status, date range, priority"
- Provide AI Verdict — APPROVE / REQUEST_REVISION / ESCALATE_TO_LEAD
- AskUserQuestion — Dev BA PIC reviews AI analysis and makes final human decision
Output
## PBI Challenge Review
**PBI:** {PBI filename}
**Reviewer:** Dev BA PIC
**Date:** {date}
**Module:** {detected module code}
### Technical Feasibility
**Status:** FEASIBLE | CONCERNS | INFEASIBLE
{Analysis with evidence — cite domain entities, service boundaries, architecture constraints}
### AC Quality
**Status:** GOOD | NEEDS_REVISION | POOR
| AC # | Issue | Suggested Fix |
| ---- | ---------------- | ------------------------- |
| {#} | {specific issue} | {specific fix suggestion} |
### Cross-Cutting Concerns
| Concern | Status | Issue |
| -------------- | --------- | -------- |
| Authorization | ✅/❌ | {detail} |
| Seed Data | ✅/❌/N/A | {detail} |
| Data Migration | ✅/❌/N/A | {detail} |
| Performance | ✅/❌/N/A | {detail} |
### Challenge Prompts for BA Drafters
1. {Specific actionable question with suggested answer}
2. {Specific actionable question with suggested answer}
3. {Specific actionable question with suggested answer}
### AI Verdict
**{APPROVE | REQUEST_REVISION | ESCALATE_TO_LEAD}**
**Reason:** {evidence-based justification}
**Confidence:** {X%} — {what was verified vs. what needs more investigation}
### Decision Record
**Dev BA PIC Decision:** {filled after human review via AskUserQuestion}
**Vote:** {approve / request-revision / escalate}
**Conditions:** {if any}
Key Rules
- AI provides ANALYSIS, human makes DECISION — Never auto-approve or auto-reject
- Challenge prompts must be specific — Include suggested answers, not just questions
- Domain context required — Always load entity reference + feature docs before analysis
- Technical veto scope — Dev BA PIC CAN veto: architecture feasibility, dependency correctness, cross-service impact, performance, security. CANNOT veto: UI/UX design, visual design, business value (see
ba-team-decision-model-protocol.md §2)
- Evidence-based — Every concern raised must cite source (protocol section, entity definition, feature doc)
- Constructive tone — Focus on improving the PBI, not criticizing the drafters
Next Steps
MANDATORY IMPORTANT MUST after completing this skill, use AskUserQuestion to recommend:
- "/dor-gate (Recommended)" — If APPROVE: validate DoR before grooming
- "/refine" — If REQUEST_REVISION: BA drafters revise, then re-run
/pbi-challenge
- "Escalate to Engineering Manager" — If ESCALATE_TO_LEAD: document concern for technical consultation
- "Skip, continue manually" — user decides
Closing Reminders
MANDATORY IMPORTANT MUST break work into small todo tasks using TaskCreate BEFORE starting.
MANDATORY IMPORTANT MUST validate decisions with user via AskUserQuestion — never auto-decide.
MANDATORY IMPORTANT MUST add a final review todo task to verify work quality.
1---2name: pbi-challenge3description: [Code Quality] AI-assisted Dev BA PIC review of PBI drafts. Generates challenge prompts, flags gaps, provides actionable feedback for BA drafter revision.4---5
6> **[IMPORTANT]** Use `TaskCreate` to break ALL work into small tasks BEFORE starting.
7
8> **Evidence Gate:** MANDATORY IMPORTANT MUST — every claim requires `file:line` proof or traced evidence with confidence percentage (>80% to act).
9
10## Quick Summary
11
12**Goal:** Help Dev BA PIC review BA drafters' PBI drafts by generating specific, actionable challenge prompts. AI provides analysis; human makes the decision.
13
14**Key distinction:** Collaborative review tool (drafter → reviewer flow), NOT self-review (use `/refine-review` for AI self-review).
15
16**Be skeptical. Apply critical thinking, sequential thinking. Every claim needs traced proof, confidence percentages (Idea should be more than 80%).**
17
18### Frontend/UI Context (if applicable)
19
20When this task involves frontend or UI changes, **MUST READ** `.claude/skills/shared/ui-system-context.md` and the following docs:
21
22- Component patterns: `docs/project-reference/frontend-patterns-reference.md`
23- Styling/BEM guide: `docs/project-reference/scss-styling-guide.md`
24- Design system tokens: `docs/project-reference/design-system/README.md`
25
26## Workflow
27
281. **Locate PBI draft** — Find BA drafters' draft PBI in `team-artifacts/pbis/` or path provided by user
292. **Load protocols** — Read these 3 protocols:
30 - `.claude/skills/shared/ba-team-decision-model-protocol.md` (decision model, veto scope)
31 - `.claude/skills/shared/refinement-dor-checklist-protocol.md` (DoR criteria)
32 - `.claude/skills/shared/cross-cutting-quality-concerns-protocol.md` (authorization, seed data, migration)
333. **Load domain context** — Auto-detect module from PBI content, load:
34 - `docs/project-reference/domain-entities-reference.md` (entity definitions)
35 - Relevant feature docs from `docs/business-features/{App}/`
36 - Existing business rules (BR-{MOD}-XXX) from feature docs
374. **Technical Feasibility Analysis:**
38 - Can described features be built with the project's architecture?
39 - Any domain entity conflicts? (cross-reference entity definitions)
40 - Any cross-service implications? (message bus events, shared data between services)
41 - Estimated complexity alignment (does scope match story points?)
425. **AC Quality Analysis:**
43 - Vagueness detector: flag "should", "might", "TBD", "etc.", "various", "appropriate"
44 - Coverage check: happy path + edge case + error case + authorization scenario
45 - Missing scenarios: suggest specific additions based on feature type
466. **Cross-Cutting Concerns Check:**
47 - Authorization section present and complete? (roles × CRUD matrix)
48 - Seed data requirements addressed? (or explicit "N/A")
49 - Data migration implications? (schema changes)
50 - Performance considerations? (list/grid/export features)
51 - **UI Layout section present?** If PBI involves UI: must have `## UI Layout` per `ui-wireframe-protocol.md` with wireframe + components (with tiers) + states + design tokens. If backend-only: explicit "N/A". Flag missing UI visualization as a gap.
527. **Generate Challenge Prompts** — Output specific, actionable questions:
53 - NOT vague: "needs work" or "improve AC"
54 - SPECIFIC: "AC #2 says 'user can filter results' — which filters exactly? Suggest: status, date range, priority"
558. **Provide AI Verdict** — APPROVE / REQUEST_REVISION / ESCALATE_TO_LEAD
569. **AskUserQuestion** — Dev BA PIC reviews AI analysis and makes final human decision
57
58## Output
59
60```markdown
61## PBI Challenge Review
62
63**PBI:** {PBI filename}
64**Reviewer:** Dev BA PIC
65**Date:** {date}
66**Module:** {detected module code}
67
68### Technical Feasibility
69
70**Status:** FEASIBLE | CONCERNS | INFEASIBLE
71{Analysis with evidence — cite domain entities, service boundaries, architecture constraints}
72
73### AC Quality
74
75**Status:** GOOD | NEEDS_REVISION | POOR
76
77| AC # | Issue | Suggested Fix |
78| ---- | ---------------- | ------------------------- |
79| {#} | {specific issue} | {specific fix suggestion} |
80
81### Cross-Cutting Concerns
82
83| Concern | Status | Issue |
84| -------------- | --------- | -------- |
85| Authorization | ✅/❌ | {detail} |
86| Seed Data | ✅/❌/N/A | {detail} |
87| Data Migration | ✅/❌/N/A | {detail} |
88| Performance | ✅/❌/N/A | {detail} |
89
90### Challenge Prompts for BA Drafters
91
921. {Specific actionable question with suggested answer}
932. {Specific actionable question with suggested answer}
943. {Specific actionable question with suggested answer}
95
96### AI Verdict
97
98**{APPROVE | REQUEST_REVISION | ESCALATE_TO_LEAD}**
99**Reason:** {evidence-based justification}
100**Confidence:** {X%} — {what was verified vs. what needs more investigation}
101
102### Decision Record
103
104**Dev BA PIC Decision:** {filled after human review via AskUserQuestion}
105**Vote:** {approve / request-revision / escalate}
106**Conditions:** {if any}
107```
108
109## Key Rules
110
111- **AI provides ANALYSIS, human makes DECISION** — Never auto-approve or auto-reject
112- **Challenge prompts must be specific** — Include suggested answers, not just questions
113- **Domain context required** — Always load entity reference + feature docs before analysis
114- **Technical veto scope** — Dev BA PIC CAN veto: architecture feasibility, dependency correctness, cross-service impact, performance, security. CANNOT veto: UI/UX design, visual design, business value (see `ba-team-decision-model-protocol.md` §2)
115- **Evidence-based** — Every concern raised must cite source (protocol section, entity definition, feature doc)
116- **Constructive tone** — Focus on improving the PBI, not criticizing the drafters
117
118---
119
120## Next Steps
121
122**MANDATORY IMPORTANT MUST** after completing this skill, use `AskUserQuestion` to recommend:
123
124- **"/dor-gate (Recommended)"** — If APPROVE: validate DoR before grooming
125- **"/refine"** — If REQUEST_REVISION: BA drafters revise, then re-run `/pbi-challenge`
126- **"Escalate to Engineering Manager"** — If ESCALATE_TO_LEAD: document concern for technical consultation
127- **"Skip, continue manually"** — user decides
128
129## Closing Reminders
130
131**MANDATORY IMPORTANT MUST** break work into small todo tasks using `TaskCreate` BEFORE starting.
132**MANDATORY IMPORTANT MUST** validate decisions with user via `AskUserQuestion` — never auto-decide.
133**MANDATORY IMPORTANT MUST** add a final review todo task to verify work quality.