@paw-workflow/cli 0.0.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +124 -0
- package/bin/paw.js +82 -0
- package/dist/agents/PAW-Review.agent.md +86 -0
- package/dist/agents/PAW.agent.md +171 -0
- package/dist/skills/paw-code-research/SKILL.md +209 -0
- package/dist/skills/paw-docs-guidance/SKILL.md +163 -0
- package/dist/skills/paw-git-operations/SKILL.md +196 -0
- package/dist/skills/paw-impl-review/SKILL.md +178 -0
- package/dist/skills/paw-implement/SKILL.md +153 -0
- package/dist/skills/paw-init/SKILL.md +118 -0
- package/dist/skills/paw-plan-review/SKILL.md +117 -0
- package/dist/skills/paw-planning/SKILL.md +217 -0
- package/dist/skills/paw-pr/SKILL.md +157 -0
- package/dist/skills/paw-review-baseline/SKILL.md +268 -0
- package/dist/skills/paw-review-correlation/SKILL.md +307 -0
- package/dist/skills/paw-review-critic/SKILL.md +373 -0
- package/dist/skills/paw-review-feedback/SKILL.md +437 -0
- package/dist/skills/paw-review-gap/SKILL.md +639 -0
- package/dist/skills/paw-review-github/SKILL.md +336 -0
- package/dist/skills/paw-review-impact/SKILL.md +569 -0
- package/dist/skills/paw-review-response/SKILL.md +118 -0
- package/dist/skills/paw-review-understanding/SKILL.md +372 -0
- package/dist/skills/paw-review-workflow/SKILL.md +239 -0
- package/dist/skills/paw-spec/SKILL.md +257 -0
- package/dist/skills/paw-spec-research/SKILL.md +138 -0
- package/dist/skills/paw-spec-review/SKILL.md +101 -0
- package/dist/skills/paw-status/SKILL.md +160 -0
- package/dist/skills/paw-transition/SKILL.md +134 -0
- package/dist/skills/paw-work-shaping/SKILL.md +99 -0
- package/dist/skills/paw-workflow/SKILL.md +142 -0
- package/lib/commands/install.js +103 -0
- package/lib/commands/list.js +18 -0
- package/lib/commands/uninstall.js +95 -0
- package/lib/commands/upgrade.js +119 -0
- package/lib/manifest.js +42 -0
- package/lib/paths.js +42 -0
- package/lib/registry.js +41 -0
- package/package.json +40 -0
|
@@ -0,0 +1,118 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: paw-init
|
|
3
|
+
description: Bootstrap skill for PAW workflow initialization. Creates WorkflowContext.md, directory structure, and git branch. Runs before workflow skill is loaded.
|
|
4
|
+
metadata:
|
|
5
|
+
version: "0.0.1"
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# PAW Initialization
|
|
9
|
+
|
|
10
|
+
> **Execution Context**: This skill runs **directly** in the PAW session (not a subagent), as bootstrap requires user input for workflow parameters.
|
|
11
|
+
|
|
12
|
+
Bootstrap skill that initializes the PAW workflow directory structure. This runs **before** the workflow skill is loaded—WorkflowContext.md must exist for the workflow to function.
|
|
13
|
+
|
|
14
|
+
## Capabilities
|
|
15
|
+
|
|
16
|
+
- Generate Work Title from issue URL, branch name, or user description
|
|
17
|
+
- Generate Work ID from Work Title (normalized, unique)
|
|
18
|
+
- Create `.paw/work/<work-id>/` directory structure
|
|
19
|
+
- Generate WorkflowContext.md with all configuration fields
|
|
20
|
+
- Create and checkout git branch (explicit or auto-derived)
|
|
21
|
+
- Commit initial artifacts if tracking is enabled
|
|
22
|
+
- Open WorkflowContext.md for review
|
|
23
|
+
|
|
24
|
+
## Input Parameters
|
|
25
|
+
|
|
26
|
+
| Parameter | Required | Default | Values |
|
|
27
|
+
|-----------|----------|---------|--------|
|
|
28
|
+
| `target_branch` | No | auto-derive from work ID | branch name |
|
|
29
|
+
| `workflow_mode` | No | `full` | `full`, `minimal`, `custom` |
|
|
30
|
+
| `review_strategy` | No | `prs` (`local` if minimal) | `prs`, `local` |
|
|
31
|
+
| `review_policy` | No | `milestones` | `always`, `milestones`, `planning-only`, `never` |
|
|
32
|
+
| `session_policy` | No | `per-stage` | `per-stage`, `continuous` |
|
|
33
|
+
| `track_artifacts` | No | `true` | boolean |
|
|
34
|
+
| `issue_url` | No | none | URL |
|
|
35
|
+
| `custom_instructions` | Conditional | — | text (required if `workflow_mode` is `custom`) |
|
|
36
|
+
| `work_description` | No | none | text |
|
|
37
|
+
|
|
38
|
+
### Handling Missing Parameters
|
|
39
|
+
|
|
40
|
+
When parameters are not provided:
|
|
41
|
+
1. Apply defaults from the table above
|
|
42
|
+
2. **Present configuration summary** and ask for confirmation before proceeding
|
|
43
|
+
3. If user requests changes, update values and re-confirm
|
|
44
|
+
|
|
45
|
+
This mirrors the VS Code command flow which prompts sequentially but allows skipping with defaults.
|
|
46
|
+
|
|
47
|
+
## Desired End States
|
|
48
|
+
|
|
49
|
+
### Work Title
|
|
50
|
+
- A 2-4 word human-readable name exists
|
|
51
|
+
- Sources (priority order): issue title → branch name → work description
|
|
52
|
+
- Capitalized appropriately (e.g., `User Auth System`)
|
|
53
|
+
|
|
54
|
+
### Work ID
|
|
55
|
+
- Unique within `.paw/work/`
|
|
56
|
+
- Format: lowercase letters, numbers, hyphens only; 1-100 chars
|
|
57
|
+
- No leading/trailing/consecutive hyphens
|
|
58
|
+
- Not reserved (`.`, `..`, `node_modules`, `.git`, `.paw`)
|
|
59
|
+
- If conflict: append `-2`, `-3`, etc.
|
|
60
|
+
|
|
61
|
+
### Configuration Validation
|
|
62
|
+
- If `workflow_mode` is `minimal`, `review_strategy` MUST be `local`
|
|
63
|
+
- If `review_policy` is `planning-only` or `never`, `review_strategy` MUST be `local`
|
|
64
|
+
- Invalid combinations: STOP and report error
|
|
65
|
+
|
|
66
|
+
### Directory Structure
|
|
67
|
+
```
|
|
68
|
+
.paw/work/<work-id>/
|
|
69
|
+
└── WorkflowContext.md
|
|
70
|
+
```
|
|
71
|
+
|
|
72
|
+
### WorkflowContext.md
|
|
73
|
+
Created at `.paw/work/<work-id>/WorkflowContext.md` with all input parameters:
|
|
74
|
+
|
|
75
|
+
```markdown
|
|
76
|
+
# WorkflowContext
|
|
77
|
+
|
|
78
|
+
Work Title: <generated_work_title>
|
|
79
|
+
Work ID: <generated_work_id>
|
|
80
|
+
Target Branch: <target_branch>
|
|
81
|
+
Workflow Mode: <workflow_mode>
|
|
82
|
+
Review Strategy: <review_strategy>
|
|
83
|
+
Review Policy: <review_policy>
|
|
84
|
+
Session Policy: <session_policy>
|
|
85
|
+
Custom Workflow Instructions: <custom_instructions or "none">
|
|
86
|
+
Initial Prompt: <work_description or "none">
|
|
87
|
+
Issue URL: <issue_url or "none">
|
|
88
|
+
Remote: origin
|
|
89
|
+
Artifact Paths: auto-derived
|
|
90
|
+
Additional Inputs: none
|
|
91
|
+
```
|
|
92
|
+
|
|
93
|
+
### Git Branch
|
|
94
|
+
> Branch creation and checkout follows `paw-git-operations` patterns.
|
|
95
|
+
|
|
96
|
+
- Target branch exists and is checked out
|
|
97
|
+
- If explicit branch provided: use as-is (prompt if exists)
|
|
98
|
+
- If auto-derive: `feature/<work-id>`
|
|
99
|
+
|
|
100
|
+
### Artifact Tracking
|
|
101
|
+
- **If tracking enabled**: WorkflowContext.md committed with message `Initialize PAW workflow for <Work Title>`
|
|
102
|
+
- **If tracking disabled**: `.gitignore` with `*` created in work directory
|
|
103
|
+
|
|
104
|
+
### User Review
|
|
105
|
+
- WorkflowContext.md presented for user review/confirmation
|
|
106
|
+
|
|
107
|
+
## Completion Response
|
|
108
|
+
|
|
109
|
+
Report initialization results to PAW agent including: work ID, workflow mode, target branch, and the recommended next step based on workflow mode (full → spec, minimal → code research, custom → per instructions).
|
|
110
|
+
|
|
111
|
+
## Validation Checklist
|
|
112
|
+
|
|
113
|
+
- [ ] Work ID is unique and valid format
|
|
114
|
+
- [ ] Review strategy valid for workflow mode
|
|
115
|
+
- [ ] WorkflowContext.md created with all fields
|
|
116
|
+
- [ ] Git branch created and checked out
|
|
117
|
+
- [ ] Artifacts committed (if tracking enabled)
|
|
118
|
+
- [ ] WorkflowContext.md opened for review
|
|
@@ -0,0 +1,117 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: paw-plan-review
|
|
3
|
+
description: Plan review activity skill for PAW workflow. Reviews implementation plans for feasibility, completeness, and spec alignment before implementation proceeds.
|
|
4
|
+
metadata:
|
|
5
|
+
version: "0.0.1"
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# Plan Review
|
|
9
|
+
|
|
10
|
+
> **Execution Context**: This skill runs in a **subagent** session, delegated by the PAW orchestrator. Return structured feedback (pass/fail + issues) to the orchestrator—do not make orchestration decisions.
|
|
11
|
+
|
|
12
|
+
Review implementation plans for quality, feasibility, and alignment with specification before implementation proceeds.
|
|
13
|
+
|
|
14
|
+
> **Reference**: Follow Core Implementation Principles from `paw-workflow` skill.
|
|
15
|
+
|
|
16
|
+
## Capabilities
|
|
17
|
+
|
|
18
|
+
- Review ImplementationPlan.md for feasibility, completeness, and spec alignment
|
|
19
|
+
- Validate against quality criteria
|
|
20
|
+
- Identify specific phases or sections needing revision
|
|
21
|
+
- Return structured feedback with pass/fail status
|
|
22
|
+
|
|
23
|
+
## Desired End State
|
|
24
|
+
|
|
25
|
+
- Plan reviewed against all quality criteria
|
|
26
|
+
- Specific issues identified with actionable feedback
|
|
27
|
+
- Structured PASS/FAIL response returned to PAW agent
|
|
28
|
+
|
|
29
|
+
## Quality Criteria
|
|
30
|
+
|
|
31
|
+
### Spec Coverage
|
|
32
|
+
|
|
33
|
+
- [ ] All spec requirements mapped to plan phases
|
|
34
|
+
- [ ] User stories have corresponding implementation work
|
|
35
|
+
- [ ] Success criteria from spec reflected in phase success criteria
|
|
36
|
+
- [ ] No spec requirements orphaned (missing from plan)
|
|
37
|
+
|
|
38
|
+
### Phase Feasibility
|
|
39
|
+
|
|
40
|
+
- [ ] Each phase has clear, testable success criteria
|
|
41
|
+
- [ ] Phase boundaries are logical (not too large, not too small)
|
|
42
|
+
- [ ] Dependencies between phases are explicit
|
|
43
|
+
- [ ] Phases build incrementally and can be reviewed independently
|
|
44
|
+
|
|
45
|
+
### Completeness
|
|
46
|
+
|
|
47
|
+
- [ ] No placeholder content or TBDs
|
|
48
|
+
- [ ] All file paths and components specified
|
|
49
|
+
- [ ] Unit tests specified alongside code they test
|
|
50
|
+
- [ ] "What We're NOT Doing" section exists
|
|
51
|
+
|
|
52
|
+
### Research Integration
|
|
53
|
+
|
|
54
|
+
- [ ] Code research findings incorporated (file:line references)
|
|
55
|
+
- [ ] Documentation System section from CodeResearch.md considered
|
|
56
|
+
- [ ] Existing patterns referenced where applicable
|
|
57
|
+
|
|
58
|
+
### Strategic Focus
|
|
59
|
+
|
|
60
|
+
- [ ] Changes describe WHAT not HOW (interfaces, responsibilities)
|
|
61
|
+
- [ ] Code blocks limited to architectural concepts (<10 lines)
|
|
62
|
+
- [ ] No implementation algorithms or pseudo-code
|
|
63
|
+
- [ ] Significant decisions include trade-off analysis
|
|
64
|
+
|
|
65
|
+
### Documentation Planning
|
|
66
|
+
|
|
67
|
+
- [ ] Documentation phase included (standard for all non-trivial changes)
|
|
68
|
+
- [ ] Docs.md specified for technical reference (always required)
|
|
69
|
+
- [ ] Project docs updates included if user-facing or API changes exist
|
|
70
|
+
- [ ] If project docs omitted: reason provided
|
|
71
|
+
|
|
72
|
+
## Review Process
|
|
73
|
+
|
|
74
|
+
1. Read ImplementationPlan.md completely
|
|
75
|
+
2. Cross-reference with Spec.md for coverage
|
|
76
|
+
3. Check CodeResearch.md integration
|
|
77
|
+
4. Evaluate each quality criteria category
|
|
78
|
+
5. Compile specific issues with locations
|
|
79
|
+
6. Determine overall PASS/FAIL status
|
|
80
|
+
|
|
81
|
+
## Feedback Categories
|
|
82
|
+
|
|
83
|
+
| Category | Description |
|
|
84
|
+
|----------|-------------|
|
|
85
|
+
| **BLOCKING** | Must fix before implementation (missing requirements, infeasible phases) |
|
|
86
|
+
| **IMPROVE** | Should address but not blocking (clarity, organization) |
|
|
87
|
+
| **NOTE** | Observation for awareness (minor suggestions, alternatives) |
|
|
88
|
+
|
|
89
|
+
## Completion Response
|
|
90
|
+
|
|
91
|
+
Return structured feedback to PAW agent:
|
|
92
|
+
|
|
93
|
+
**PASS**: Plan meets quality criteria, ready for implementation
|
|
94
|
+
- Confirm spec coverage complete
|
|
95
|
+
- Note any non-blocking observations
|
|
96
|
+
|
|
97
|
+
**FAIL**: Plan needs revision before implementation
|
|
98
|
+
- List blocking issues with specific locations
|
|
99
|
+
- Categorize issues (BLOCKING, IMPROVE)
|
|
100
|
+
- Suggest what needs to change (not how to change it)
|
|
101
|
+
|
|
102
|
+
### Response Content
|
|
103
|
+
|
|
104
|
+
Include in completion response:
|
|
105
|
+
- Overall status: PASS or FAIL
|
|
106
|
+
- Spec coverage assessment: which requirements are covered, any gaps
|
|
107
|
+
- Phase feasibility assessment: any phases too large, unclear, or missing dependencies
|
|
108
|
+
- Completeness check: any TBDs, missing tests, incomplete sections
|
|
109
|
+
- Specific issues: categorized with plan section references
|
|
110
|
+
- Recommendation: proceed to implementation or revise plan
|
|
111
|
+
|
|
112
|
+
## Non-Responsibilities
|
|
113
|
+
|
|
114
|
+
- Do NOT make orchestration decisions (PAW agent handles next steps)
|
|
115
|
+
- Do NOT modify the plan directly (return feedback for planning skill to address)
|
|
116
|
+
- Do NOT implement changes (review only)
|
|
117
|
+
- Do NOT handle handoffs (return status to PAW agent)
|
|
@@ -0,0 +1,217 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: paw-planning
|
|
3
|
+
description: Implementation planning activity skill for PAW workflow. Creates phased implementation plans with clear success criteria, documentation phase planning, and strategic architectural descriptions.
|
|
4
|
+
metadata:
|
|
5
|
+
version: "0.0.1"
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# Implementation Planning
|
|
9
|
+
|
|
10
|
+
> **Execution Context**: This skill runs **directly** in the PAW session (not a subagent), preserving user interactivity for phase decisions and blocker handling.
|
|
11
|
+
|
|
12
|
+
Create detailed implementation plans through interactive refinement. Plans describe WHAT to build (components, interfaces, behaviors) at the architectural level, delegating HOW to implement to the implementer.
|
|
13
|
+
|
|
14
|
+
> **Reference**: Follow Core Implementation Principles from `paw-workflow` skill.
|
|
15
|
+
|
|
16
|
+
## Capabilities
|
|
17
|
+
|
|
18
|
+
- Create implementation plan from spec and research
|
|
19
|
+
- Revise plan based on learnings or feedback
|
|
20
|
+
- Address PR review comments (load `paw-review-response` for mechanics)
|
|
21
|
+
- Plan documentation phases when documentation updates are warranted
|
|
22
|
+
|
|
23
|
+
## Be Skeptical
|
|
24
|
+
|
|
25
|
+
- Question vague requirements—ask "why" and "what about"
|
|
26
|
+
- Identify potential issues early
|
|
27
|
+
- Don't assume—verify with code research
|
|
28
|
+
- If user corrects a misunderstanding, verify the correction against code before accepting
|
|
29
|
+
|
|
30
|
+
## Strategic Planning Guidelines
|
|
31
|
+
|
|
32
|
+
Operate at the **C4 container/component abstraction level**:
|
|
33
|
+
|
|
34
|
+
**Do**:
|
|
35
|
+
- Describe component responsibilities and interfaces
|
|
36
|
+
- Reference file paths, module names, existing patterns
|
|
37
|
+
- Present 2-3 options with trade-offs for significant decisions
|
|
38
|
+
- Define success via observable outcomes (tests, SLOs, acceptance criteria)
|
|
39
|
+
- Limit code snippets to 3-10 lines for critical architectural concepts only
|
|
40
|
+
|
|
41
|
+
**Don't**:
|
|
42
|
+
- Include complete function implementations
|
|
43
|
+
- Provide pseudo-code walkthroughs
|
|
44
|
+
- Specify library choices without trade-off analysis
|
|
45
|
+
- Write tutorial-style code examples
|
|
46
|
+
|
|
47
|
+
**Example descriptions**:
|
|
48
|
+
- ✅ "Implement `UserRepository` interface with CRUD methods, following pattern in `models/repositories/`"
|
|
49
|
+
- ✅ "Add auth middleware validating JWT tokens, integrating with `services/auth/`"
|
|
50
|
+
- ❌ "Create function that loops through array and filters..."
|
|
51
|
+
|
|
52
|
+
## Documentation Phase Planning
|
|
53
|
+
|
|
54
|
+
Include documentation as the **final implementation phase**. The documentation phase is standard for all non-trivial changes.
|
|
55
|
+
|
|
56
|
+
### What the Documentation Phase Produces
|
|
57
|
+
|
|
58
|
+
1. **Docs.md** (always): Technical reference capturing implementation details, usage, and verification approach—serves as the "as-built" record even for internal changes
|
|
59
|
+
2. **Project documentation updates** (when warranted): README, CHANGELOG, guides following project conventions
|
|
60
|
+
3. **Documentation build verification**: Command from CodeResearch.md (if framework discovered)
|
|
61
|
+
|
|
62
|
+
### When to Include Project Docs
|
|
63
|
+
|
|
64
|
+
- Work creates user-facing features
|
|
65
|
+
- APIs are added or changed
|
|
66
|
+
- Existing behavior is modified in ways users should know about
|
|
67
|
+
|
|
68
|
+
### When to Omit Project Docs
|
|
69
|
+
|
|
70
|
+
- Purely internal changes (Docs.md still required)
|
|
71
|
+
- Refactors without behavior changes (Docs.md still required)
|
|
72
|
+
- User explicitly indicates no project docs needed
|
|
73
|
+
|
|
74
|
+
**Note**: Implementer loads `paw-docs-guidance` utility skill for templates and conventions during documentation phases.
|
|
75
|
+
|
|
76
|
+
## ImplementationPlan.md Template
|
|
77
|
+
|
|
78
|
+
Save to: `.paw/work/<work-id>/ImplementationPlan.md`
|
|
79
|
+
|
|
80
|
+
```markdown
|
|
81
|
+
# [Feature/Task Name] Implementation Plan
|
|
82
|
+
|
|
83
|
+
## Overview
|
|
84
|
+
[What we're implementing and why]
|
|
85
|
+
|
|
86
|
+
## Current State Analysis
|
|
87
|
+
[Existing state, gaps, key constraints from research]
|
|
88
|
+
|
|
89
|
+
## Desired End State
|
|
90
|
+
[Target state specification and verification approach]
|
|
91
|
+
|
|
92
|
+
## What We're NOT Doing
|
|
93
|
+
[Out-of-scope items]
|
|
94
|
+
|
|
95
|
+
## Phase Status
|
|
96
|
+
- [ ] **Phase 1: [Name]** - [Objective]
|
|
97
|
+
- [ ] **Phase 2: [Name]** - [Objective]
|
|
98
|
+
- [ ] **Phase N: Documentation** - [If warranted]
|
|
99
|
+
|
|
100
|
+
---
|
|
101
|
+
|
|
102
|
+
## Phase 1: [Name]
|
|
103
|
+
|
|
104
|
+
### Changes Required:
|
|
105
|
+
- **`path/to/file.ext`**: [Component changes, pattern references]
|
|
106
|
+
- **Tests**: [Test file, key scenarios]
|
|
107
|
+
|
|
108
|
+
### Success Criteria:
|
|
109
|
+
|
|
110
|
+
#### Automated Verification:
|
|
111
|
+
- [ ] Tests pass: `<command>`
|
|
112
|
+
- [ ] Lint/typecheck: `<command>`
|
|
113
|
+
|
|
114
|
+
#### Manual Verification:
|
|
115
|
+
- [ ] [User-observable behavior]
|
|
116
|
+
- [ ] [Edge cases requiring human judgment]
|
|
117
|
+
|
|
118
|
+
---
|
|
119
|
+
|
|
120
|
+
## Phase N: Documentation (if warranted)
|
|
121
|
+
|
|
122
|
+
### Changes Required:
|
|
123
|
+
- **`.paw/work/<work-id>/Docs.md`**: Technical reference (load `paw-docs-guidance`)
|
|
124
|
+
- **Project docs**: Per CodeResearch.md findings
|
|
125
|
+
|
|
126
|
+
### Success Criteria:
|
|
127
|
+
- [ ] Docs build: `<command>`
|
|
128
|
+
- [ ] Content accurate, style consistent
|
|
129
|
+
|
|
130
|
+
---
|
|
131
|
+
|
|
132
|
+
## References
|
|
133
|
+
- Issue: [link or 'none']
|
|
134
|
+
- Spec: `.paw/work/<work-id>/Spec.md`
|
|
135
|
+
- Research: `.paw/work/<work-id>/SpecResearch.md`, `.paw/work/<work-id>/CodeResearch.md`
|
|
136
|
+
```
|
|
137
|
+
|
|
138
|
+
**Phase Status format**: Use checkboxes `- [ ]` for pending, `- [x]` for complete. This provides at-a-glance status without heavy tables.
|
|
139
|
+
|
|
140
|
+
## Execution Contexts
|
|
141
|
+
|
|
142
|
+
### Initial Planning
|
|
143
|
+
|
|
144
|
+
**Desired end state**: Complete ImplementationPlan.md with all phases defined
|
|
145
|
+
|
|
146
|
+
1. Read all context: Issue, Spec.md, SpecResearch.md, CodeResearch.md
|
|
147
|
+
2. Analyze and verify requirements against actual code
|
|
148
|
+
3. Present understanding and resolve blocking questions
|
|
149
|
+
4. Research patterns and design options (if significant choices exist)
|
|
150
|
+
5. Write plan incrementally (outline, then phase by phase)
|
|
151
|
+
6. Handle branching per Review Strategy (see below)
|
|
152
|
+
|
|
153
|
+
### PR Review Response
|
|
154
|
+
|
|
155
|
+
When addressing Planning PR comments, load `paw-review-response` utility for mechanics.
|
|
156
|
+
|
|
157
|
+
**Desired end state**: All review comments addressed, artifacts consistent
|
|
158
|
+
|
|
159
|
+
1. Verify on planning branch (`<target>_plan`)
|
|
160
|
+
2. Read all unresolved PR comments
|
|
161
|
+
3. Create TODOs for comments (group related, separate complex)
|
|
162
|
+
4. For each TODO: make changes → commit → push → reply
|
|
163
|
+
5. Verify all comments addressed and artifacts consistent
|
|
164
|
+
|
|
165
|
+
### Plan Revision
|
|
166
|
+
|
|
167
|
+
When revising based on paw-plan-review feedback:
|
|
168
|
+
|
|
169
|
+
**Desired end state**: ImplementationPlan.md updated to address all BLOCKING issues
|
|
170
|
+
|
|
171
|
+
1. Read paw-plan-review feedback completely
|
|
172
|
+
2. Address BLOCKING issues first (these prevent implementation)
|
|
173
|
+
3. Address IMPROVE issues if scope permits
|
|
174
|
+
4. Acknowledge NOTE items in commit message if applicable
|
|
175
|
+
5. Re-run quality checklist to verify fix didn't introduce new issues
|
|
176
|
+
|
|
177
|
+
## Branching and Commits
|
|
178
|
+
|
|
179
|
+
> **Reference**: Load `paw-git-operations` skill for branch naming, commit mechanics, and PR descriptions.
|
|
180
|
+
|
|
181
|
+
## Quality Checklist
|
|
182
|
+
|
|
183
|
+
- [ ] All phases contain specific file paths, components, or interfaces
|
|
184
|
+
- [ ] Every phase has measurable automated and manual success criteria
|
|
185
|
+
- [ ] Phases build incrementally and can be reviewed independently
|
|
186
|
+
- [ ] Zero open questions or TBDs remain
|
|
187
|
+
- [ ] All references trace to Spec.md and CodeResearch.md
|
|
188
|
+
- [ ] Unit tests specified alongside code they test within each phase
|
|
189
|
+
- [ ] "What We're NOT Doing" section prevents scope creep
|
|
190
|
+
- [ ] Code blocks absent or limited to <10 lines for architectural concepts
|
|
191
|
+
- [ ] Documentation phase included (or explicitly omitted with reason)
|
|
192
|
+
- [ ] Phase Status exists with checkbox phases and one-sentence objectives
|
|
193
|
+
|
|
194
|
+
## Completion Response
|
|
195
|
+
|
|
196
|
+
Report to PAW agent:
|
|
197
|
+
- Artifact path: `.paw/work/<work-id>/ImplementationPlan.md`
|
|
198
|
+
- **Plan summary** for quick review:
|
|
199
|
+
- Architecture approach (1-2 sentences)
|
|
200
|
+
- Phase overview (numbered list with one-line objective each)
|
|
201
|
+
- What's explicitly NOT being done (scope boundaries)
|
|
202
|
+
- Review Strategy used (PR created or committed to target)
|
|
203
|
+
- Any items requiring user decision or observations worth noting
|
|
204
|
+
|
|
205
|
+
### Blocked on Open Questions
|
|
206
|
+
|
|
207
|
+
If planning encounters questions that cannot be answered from existing artifacts (Spec.md, CodeResearch.md):
|
|
208
|
+
|
|
209
|
+
1. **Do NOT write ImplementationPlan.md** — partial artifacts cause confusion on re-invocation
|
|
210
|
+
2. **Return to PAW agent** with:
|
|
211
|
+
- Status: `blocked`
|
|
212
|
+
- List of specific open questions
|
|
213
|
+
- What research or clarification would resolve each question
|
|
214
|
+
3. **PAW agent handles resolution** based on Review Policy:
|
|
215
|
+
- `never`: PAW agent conducts additional research to resolve questions autonomously
|
|
216
|
+
- `always`/`milestones`: PAW agent asks user for clarification
|
|
217
|
+
4. **Re-invocation**: PAW agent calls planning again with answers provided in the delegation prompt
|
|
@@ -0,0 +1,157 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: paw-pr
|
|
3
|
+
description: Final PR activity skill for PAW workflow. Creates comprehensive final PR to main with pre-flight validation, scaled descriptions, and merge guidance.
|
|
4
|
+
metadata:
|
|
5
|
+
version: "0.0.1"
|
|
6
|
+
---
|
|
7
|
+
|
|
8
|
+
# Final PR
|
|
9
|
+
|
|
10
|
+
> **Execution Context**: This skill runs **directly** in the PAW session (not a subagent), allowing user interaction for PR description and final checks.
|
|
11
|
+
|
|
12
|
+
Create the final PR merging all implementation work to main branch after pre-flight validation.
|
|
13
|
+
|
|
14
|
+
> **Reference**: Follow Core Implementation Principles from `paw-workflow` skill.
|
|
15
|
+
|
|
16
|
+
## Capabilities
|
|
17
|
+
|
|
18
|
+
- Run pre-flight validation checks
|
|
19
|
+
- Create comprehensive PR description (scaled to complexity)
|
|
20
|
+
- Open final PR from target branch to main
|
|
21
|
+
- Provide merge and deployment guidance
|
|
22
|
+
|
|
23
|
+
## Pre-flight Validation
|
|
24
|
+
|
|
25
|
+
Before creating the PR, verify and report status. Block on failures unless user explicitly confirms.
|
|
26
|
+
|
|
27
|
+
### Required Checks
|
|
28
|
+
|
|
29
|
+
**Phase Implementation**:
|
|
30
|
+
- All phases in ImplementationPlan.md marked complete
|
|
31
|
+
- All phase PRs merged (prs strategy) or commits pushed (local strategy)
|
|
32
|
+
- Target branch exists with implementation commits
|
|
33
|
+
|
|
34
|
+
**Artifacts Exist** (check existence per Workflow Mode):
|
|
35
|
+
- CodeResearch.md (required: all modes)
|
|
36
|
+
- ImplementationPlan.md (required: all modes)
|
|
37
|
+
- Spec.md (required: full mode; optional: minimal/custom)
|
|
38
|
+
- SpecResearch.md (optional: all modes)
|
|
39
|
+
- Docs.md (required: full mode; optional: minimal/custom)
|
|
40
|
+
|
|
41
|
+
**Branch Status**:
|
|
42
|
+
- Target branch up to date with base branch (main)
|
|
43
|
+
- No merge conflicts
|
|
44
|
+
|
|
45
|
+
**Build/Tests**:
|
|
46
|
+
- Latest build passes on target branch (if applicable)
|
|
47
|
+
- All tests passing
|
|
48
|
+
|
|
49
|
+
**Open Questions Resolved**:
|
|
50
|
+
- SpecResearch `## Open Unknowns` → resolved in Spec or clarified
|
|
51
|
+
- CodeResearch `## Open Questions` → resolved in Plan or code
|
|
52
|
+
- ImplementationPlan `## Open Questions` → empty
|
|
53
|
+
- Unresolved items → block and report with recommendation
|
|
54
|
+
|
|
55
|
+
## Workflow Mode Handling
|
|
56
|
+
|
|
57
|
+
### Full Mode
|
|
58
|
+
- Reference all artifacts: Spec.md, SpecResearch.md, CodeResearch.md, ImplementationPlan.md, Docs.md
|
|
59
|
+
- PRs strategy: Include links to intermediate PRs (Planning, Phase, Docs)
|
|
60
|
+
- Local strategy: Describe work directly from commits
|
|
61
|
+
|
|
62
|
+
### Minimal Mode
|
|
63
|
+
- Reference only core artifacts: CodeResearch.md, ImplementationPlan.md
|
|
64
|
+
- Check Spec.md and Docs.md existence before including
|
|
65
|
+
- Local strategy only (enforced)
|
|
66
|
+
|
|
67
|
+
### Custom Mode
|
|
68
|
+
- Dynamically check which artifacts exist
|
|
69
|
+
- Adapt references based on Custom Workflow Instructions
|
|
70
|
+
|
|
71
|
+
## Artifact Tracking Detection
|
|
72
|
+
|
|
73
|
+
Check if workflow artifacts are tracked in git:
|
|
74
|
+
1. Check for `.paw/work/<work-id>/.gitignore`
|
|
75
|
+
2. If contains `*` → artifacts **untracked**
|
|
76
|
+
3. If no `.gitignore` → artifacts **tracked**
|
|
77
|
+
|
|
78
|
+
**Impact on PR Description**:
|
|
79
|
+
- Tracked: Include Artifacts section with links
|
|
80
|
+
- Untracked: Omit Artifacts section; summarize key information in body
|
|
81
|
+
|
|
82
|
+
## PR Description Formats
|
|
83
|
+
|
|
84
|
+
**Scale description to change complexity.** Simple fixes need brief summaries; major features need comprehensive sections.
|
|
85
|
+
|
|
86
|
+
### Simple Changes
|
|
87
|
+
|
|
88
|
+
For bug fixes, small features: title with `[Work Title]` prefix, close issue, brief summary, changes list, testing status, PAW footer.
|
|
89
|
+
|
|
90
|
+
### Complex Changes
|
|
91
|
+
|
|
92
|
+
For large features, architectural changes, multi-phase implementations:
|
|
93
|
+
|
|
94
|
+
Include these elements as appropriate:
|
|
95
|
+
- Summary: Overview of what PR delivers and key design decisions
|
|
96
|
+
- Changes: Detailed changes with context and rationale
|
|
97
|
+
- Testing: Coverage notes, verification performed
|
|
98
|
+
- Breaking Changes: Migration guidance if applicable
|
|
99
|
+
- Deployment: Feature flags, rollout strategy, dependencies
|
|
100
|
+
- Artifacts: Links to workflow artifacts (if tracked)
|
|
101
|
+
|
|
102
|
+
Footer: `🐾 Generated with [PAW](https://github.com/lossyrob/phased-agent-workflow)`
|
|
103
|
+
|
|
104
|
+
### PRs Strategy Sections
|
|
105
|
+
|
|
106
|
+
When intermediate PRs exist, include:
|
|
107
|
+
- Planning section referencing Planning PR
|
|
108
|
+
- Implementation Phases listing Phase PRs
|
|
109
|
+
- Documentation section referencing Docs PR
|
|
110
|
+
|
|
111
|
+
### Local Strategy Sections
|
|
112
|
+
|
|
113
|
+
When no intermediate PRs:
|
|
114
|
+
- Implementation Summary describing work from commit history
|
|
115
|
+
- Focus on what was implemented, not which PRs
|
|
116
|
+
|
|
117
|
+
## PR Creation
|
|
118
|
+
|
|
119
|
+
**Final PR context requirements**:
|
|
120
|
+
- Source: `<target_branch>`
|
|
121
|
+
- Target: `main` (or specified base branch)
|
|
122
|
+
- Title format: `[<Work Title>] <description>`
|
|
123
|
+
- Body: Scaled description per formats above
|
|
124
|
+
- Issue linking: Include Issue URL from WorkflowContext.md
|
|
125
|
+
|
|
126
|
+
## Merge Guidance
|
|
127
|
+
|
|
128
|
+
After PR creation, provide:
|
|
129
|
+
- Summary of what reviewers should focus on
|
|
130
|
+
- Deployment considerations (if any)
|
|
131
|
+
- Next steps for completion
|
|
132
|
+
|
|
133
|
+
## Quality Checklist
|
|
134
|
+
|
|
135
|
+
- [ ] All pre-flight checks pass (or user confirmed proceed)
|
|
136
|
+
- [ ] PR description complete with all relevant sections
|
|
137
|
+
- [ ] All artifact links valid (if tracked)
|
|
138
|
+
- [ ] Intermediate PR links included (if prs strategy)
|
|
139
|
+
- [ ] Breaking changes documented (or stated "None")
|
|
140
|
+
- [ ] Issue URL linked in PR body
|
|
141
|
+
|
|
142
|
+
## Guardrails
|
|
143
|
+
|
|
144
|
+
- Do NOT modify code or documentation
|
|
145
|
+
- Do NOT approve or merge PRs
|
|
146
|
+
- Do NOT address review comments (Implementer handles this)
|
|
147
|
+
- Do NOT guess artifact locations—verify existence
|
|
148
|
+
|
|
149
|
+
## Completion Response
|
|
150
|
+
|
|
151
|
+
Report back:
|
|
152
|
+
- Final PR URL
|
|
153
|
+
- Pre-flight validation summary
|
|
154
|
+
- Items for reviewer attention
|
|
155
|
+
- Merge readiness assessment
|
|
156
|
+
|
|
157
|
+
Next steps: Address PR feedback via `address comments`, then merge to complete workflow.
|