fraim-framework 2.0.44 → 2.0.45
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/bin/fraim.js +1 -1
- package/dist/registry/ai-manager-rules/design-phases/design-completeness-review.md +73 -0
- package/dist/registry/ai-manager-rules/design-phases/design-design.md +145 -0
- package/dist/registry/ai-manager-rules/design-phases/design.md +108 -0
- package/dist/registry/ai-manager-rules/design-phases/finalize.md +60 -0
- package/dist/registry/ai-manager-rules/design-phases/validate.md +125 -0
- package/dist/registry/ai-manager-rules/implement-phases/code.md +323 -0
- package/dist/registry/ai-manager-rules/implement-phases/completeness-review.md +94 -0
- package/dist/registry/ai-manager-rules/implement-phases/finalize.md +177 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-code.md +286 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-completeness-review.md +120 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-regression.md +173 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-repro.md +104 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-scoping.md +100 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-smoke.md +230 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-spike.md +121 -0
- package/dist/registry/ai-manager-rules/implement-phases/implement-validate.md +371 -0
- package/dist/registry/ai-manager-rules/implement-phases/quality-review.md +304 -0
- package/dist/registry/ai-manager-rules/implement-phases/regression.md +159 -0
- package/dist/registry/ai-manager-rules/implement-phases/repro.md +101 -0
- package/dist/registry/ai-manager-rules/implement-phases/scoping.md +93 -0
- package/dist/registry/ai-manager-rules/implement-phases/smoke.md +225 -0
- package/dist/registry/ai-manager-rules/implement-phases/spike.md +118 -0
- package/dist/registry/ai-manager-rules/implement-phases/validate.md +347 -0
- package/dist/registry/ai-manager-rules/shared-phases/finalize.md +169 -0
- package/dist/registry/ai-manager-rules/shared-phases/submit-pr.md +202 -0
- package/dist/registry/ai-manager-rules/shared-phases/wait-for-pr-review.md +170 -0
- package/dist/registry/ai-manager-rules/spec-phases/finalize.md +60 -0
- package/dist/registry/ai-manager-rules/spec-phases/spec-completeness-review.md +66 -0
- package/dist/registry/ai-manager-rules/spec-phases/spec-spec.md +139 -0
- package/dist/registry/ai-manager-rules/spec-phases/spec.md +102 -0
- package/dist/registry/ai-manager-rules/spec-phases/validate.md +118 -0
- package/dist/src/ai-manager/ai-manager.js +380 -119
- package/dist/src/ai-manager/evidence-validator.js +309 -0
- package/dist/src/ai-manager/phase-flow.js +244 -0
- package/dist/src/ai-manager/types.js +5 -0
- package/dist/src/fraim-mcp-server.js +45 -153
- package/dist/src/static-website-middleware.js +75 -0
- package/dist/tests/test-ai-coach-edge-cases.js +415 -0
- package/dist/tests/test-ai-coach-mcp-integration.js +432 -0
- package/dist/tests/test-ai-coach-performance.js +328 -0
- package/dist/tests/test-ai-coach-phase-content.js +264 -0
- package/dist/tests/test-ai-coach-workflows.js +487 -0
- package/dist/tests/test-ai-manager-phase-protocol.js +147 -0
- package/dist/tests/test-ai-manager.js +60 -71
- package/dist/tests/test-evidence-validation.js +221 -0
- package/dist/tests/test-mcp-lifecycle-methods.js +18 -23
- package/dist/tests/test-pr-review-integration.js +1 -0
- package/dist/tests/test-pr-review-workflow.js +299 -0
- package/dist/website/.nojekyll +0 -0
- package/dist/website/404.html +101 -0
- package/dist/website/CNAME +1 -0
- package/dist/website/README.md +22 -0
- package/dist/website/demo.html +604 -0
- package/dist/website/images/.gitkeep +1 -0
- package/dist/website/images/fraim-logo.png +0 -0
- package/dist/website/index.html +290 -0
- package/dist/website/pricing.html +414 -0
- package/dist/website/script.js +55 -0
- package/dist/website/styles.css +2647 -0
- package/package.json +2 -1
- package/registry/agent-guardrails.md +1 -1
- package/registry/stubs/workflows/brainstorming/blue-sky-brainstorming.md +11 -0
- package/registry/stubs/workflows/brainstorming/codebase-brainstorming.md +11 -0
- package/registry/stubs/workflows/compliance/detect-compliance-requirements.md +11 -0
- package/registry/stubs/workflows/compliance/generate-audit-evidence.md +11 -0
- package/registry/stubs/workflows/learning/synthesize-learnings.md +11 -0
- package/registry/stubs/workflows/legal/nda.md +11 -0
- package/registry/stubs/workflows/legal/patent-filing.md +11 -0
- package/registry/stubs/workflows/legal/trademark-filing.md +11 -0
- package/registry/stubs/workflows/product-building/design.md +1 -1
- package/registry/stubs/workflows/product-building/implement.md +1 -2
package/bin/fraim.js
CHANGED
|
@@ -0,0 +1,73 @@
|
|
|
1
|
+
# Phase: Design-Completeness-Review
|
|
2
|
+
|
|
3
|
+
## INTENT
|
|
4
|
+
To verify that the technical design addresses all aspects of the functional specification, including the validation plan and testing strategy.
|
|
5
|
+
|
|
6
|
+
## OUTCOME
|
|
7
|
+
Confirmation that:
|
|
8
|
+
- Design addresses all functional spec requirements
|
|
9
|
+
- Validation plan from spec is reflected in design
|
|
10
|
+
- Testing strategy covers all specified scenarios
|
|
11
|
+
- Technical approach enables spec validation
|
|
12
|
+
- Design is ready for implementation
|
|
13
|
+
|
|
14
|
+
## WORKFLOW
|
|
15
|
+
|
|
16
|
+
### Step 1: Load Functional Specification
|
|
17
|
+
- Read the functional specification document from `docs/feature specs/{issue_number}-*.md`
|
|
18
|
+
- Extract all requirements, user workflows, and validation criteria
|
|
19
|
+
- Note any UI mockups and interaction patterns
|
|
20
|
+
|
|
21
|
+
### Step 2: Verify Spec Requirements Coverage
|
|
22
|
+
- Check that design addresses each functional requirement
|
|
23
|
+
- Confirm technical approach supports all user workflows from spec
|
|
24
|
+
- Verify API design enables the specified user experience
|
|
25
|
+
- Ensure data models support all spec requirements
|
|
26
|
+
|
|
27
|
+
### Step 3: Validation Plan Alignment
|
|
28
|
+
**Critical**: The spec's validation plan must be implementable with this design
|
|
29
|
+
- Review validation scenarios from functional spec
|
|
30
|
+
- Confirm design provides necessary APIs/interfaces for validation
|
|
31
|
+
- Check that testing strategy covers all validation scenarios
|
|
32
|
+
- Verify design enables browser testing (if UI changes)
|
|
33
|
+
- Ensure backend validation points are accessible
|
|
34
|
+
|
|
35
|
+
### Step 4: Technical Feasibility Check
|
|
36
|
+
- Confirm design is technically sound and implementable
|
|
37
|
+
- Check that all spec requirements can be built with this approach
|
|
38
|
+
- Verify integration points are well-defined
|
|
39
|
+
- Assess if design enables the spec's success criteria
|
|
40
|
+
|
|
41
|
+
## VALIDATION
|
|
42
|
+
|
|
43
|
+
### Phase Complete When:
|
|
44
|
+
- ✅ All functional spec requirements addressed in design
|
|
45
|
+
- ✅ Validation plan from spec is implementable with this design
|
|
46
|
+
- ✅ Testing strategy covers all spec scenarios
|
|
47
|
+
- ✅ Technical approach is sound and feasible
|
|
48
|
+
- ✅ Design enables spec success criteria
|
|
49
|
+
- ✅ Ready for implementation phase
|
|
50
|
+
|
|
51
|
+
### Phase Incomplete If:
|
|
52
|
+
- ❌ Functional spec requirements not addressed
|
|
53
|
+
- ❌ Validation plan not implementable with current design
|
|
54
|
+
- ❌ Testing strategy insufficient for spec scenarios
|
|
55
|
+
- ❌ Technical approach infeasible or unclear
|
|
56
|
+
- ❌ Design doesn't enable spec success criteria
|
|
57
|
+
|
|
58
|
+
### Report Back:
|
|
59
|
+
```javascript
|
|
60
|
+
seekCoachingOnNextStep({
|
|
61
|
+
workflowType: "design",
|
|
62
|
+
issueNumber: "{issue_number}",
|
|
63
|
+
currentPhase: "design-completeness-review",
|
|
64
|
+
status: "complete",
|
|
65
|
+
evidence: {
|
|
66
|
+
specRequirementsCovered: true,
|
|
67
|
+
validationPlanImplementable: true,
|
|
68
|
+
testingStrategySufficient: true,
|
|
69
|
+
technicallyFeasible: true,
|
|
70
|
+
functionalSpecPath: "docs/feature specs/{issue_number}-*.md"
|
|
71
|
+
}
|
|
72
|
+
})
|
|
73
|
+
```
|
|
@@ -0,0 +1,145 @@
|
|
|
1
|
+
# Phase: Design-Design
|
|
2
|
+
|
|
3
|
+
## INTENT
|
|
4
|
+
To create a comprehensive technical design that translates the feature specification into implementable architecture, APIs, and technical solutions.
|
|
5
|
+
|
|
6
|
+
## OUTCOME
|
|
7
|
+
A complete technical design document that:
|
|
8
|
+
- Translates spec requirements into technical architecture
|
|
9
|
+
- Defines APIs, data models, and system interactions
|
|
10
|
+
- Identifies technical risks and mitigation strategies
|
|
11
|
+
- Provides clear implementation guidance
|
|
12
|
+
- Includes technical validation criteria
|
|
13
|
+
|
|
14
|
+
## PRINCIPLES
|
|
15
|
+
- **Spec-Driven**: Design directly addresses specification requirements
|
|
16
|
+
- **Implementation-Ready**: Provides clear technical guidance
|
|
17
|
+
- **Risk-Aware**: Identifies and mitigates technical risks
|
|
18
|
+
- **Validation-Focused**: Includes technical validation criteria
|
|
19
|
+
- **Template-Compliant**: Follows RFC template structure
|
|
20
|
+
|
|
21
|
+
## WORKFLOW
|
|
22
|
+
|
|
23
|
+
### Step 1: Get Repository Context and Issue Details
|
|
24
|
+
|
|
25
|
+
**GitHub Repository Context**: Before using GitHub MCP tools, read the local `.fraim/config.json` file to get the repository context:
|
|
26
|
+
- Use `git.repoOwner` for the GitHub repository owner
|
|
27
|
+
- Use `git.repoName` for the GitHub repository name
|
|
28
|
+
- These values are required for GitHub MCP API calls (owner/repo parameters)
|
|
29
|
+
|
|
30
|
+
**Read Issue Details**: Use GitHub MCP tools to get the complete issue information:
|
|
31
|
+
- Read the full issue description, title, and labels
|
|
32
|
+
- Check for linked documents (specs, RFCs, design docs)
|
|
33
|
+
- Review any comments or discussion threads
|
|
34
|
+
- Note acceptance criteria and requirements
|
|
35
|
+
- Identify if there's an existing specification document
|
|
36
|
+
|
|
37
|
+
### Step 2: Load Template and Specification
|
|
38
|
+
- Call `get_fraim_file({ path: "templates/specs/TECHSPEC-TEMPLATE.md" })` (for features)
|
|
39
|
+
- Call `get_fraim_file({ path: "templates/specs/BUGSPEC-TEMPLATE.md" })` (for bugs)
|
|
40
|
+
- Read the completed specification document from previous phase
|
|
41
|
+
- Call `get_fraim_file({ path: "rules/spike-first-development.md" })`
|
|
42
|
+
|
|
43
|
+
### Step 3: Check for PR Feedback
|
|
44
|
+
|
|
45
|
+
**CRITICAL**: Before creating design, check for feedback:
|
|
46
|
+
- **PR Review Feedback**: `docs/evidence/{issue_number}-design-feedback.md`
|
|
47
|
+
|
|
48
|
+
**If PR feedback exists:**
|
|
49
|
+
- **MUST address all UNADDRESSED items** before proceeding
|
|
50
|
+
- Update feedback file after addressing each item
|
|
51
|
+
- Change status from UNADDRESSED to ADDRESSED with resolution details
|
|
52
|
+
- Cannot skip or defer feedback items
|
|
53
|
+
- Phase cannot complete until all feedback is addressed
|
|
54
|
+
|
|
55
|
+
**PR Feedback File Format** (if exists):
|
|
56
|
+
```markdown
|
|
57
|
+
### Comment X - ADDRESSED
|
|
58
|
+
- **Author**: reviewer_name
|
|
59
|
+
- **Comment**: Original feedback
|
|
60
|
+
- **Status**: ADDRESSED
|
|
61
|
+
- **Resolution**: How you addressed this feedback
|
|
62
|
+
```
|
|
63
|
+
|
|
64
|
+
### Step 4: Analyze Specification Requirements
|
|
65
|
+
- Extract all technical requirements from specification
|
|
66
|
+
- Identify system components that need modification
|
|
67
|
+
- Determine API changes or additions needed
|
|
68
|
+
- Note any performance or scalability considerations
|
|
69
|
+
- List integration points with existing systems
|
|
70
|
+
|
|
71
|
+
### Step 5: Create Technical Design Document
|
|
72
|
+
- Create RFC file at `docs/rfcs/{issue_number}-{kebab-case-title}.md`
|
|
73
|
+
- Follow appropriate template structure (TECHSPEC or BUGSPEC)
|
|
74
|
+
- Include all required sections:
|
|
75
|
+
- Problem statement (from spec)
|
|
76
|
+
- Technical requirements
|
|
77
|
+
- Proposed solution architecture
|
|
78
|
+
- API design and data models
|
|
79
|
+
- Implementation plan
|
|
80
|
+
- Testing strategy
|
|
81
|
+
- Risk assessment and mitigation
|
|
82
|
+
- Alternative approaches considered
|
|
83
|
+
|
|
84
|
+
### Step 6: Design System Architecture
|
|
85
|
+
- Define component interactions and data flow
|
|
86
|
+
- Specify API endpoints and request/response formats
|
|
87
|
+
- Design database schema changes (if needed)
|
|
88
|
+
- Plan integration with existing systems
|
|
89
|
+
- Consider scalability and performance implications
|
|
90
|
+
|
|
91
|
+
### Step 7: Identify Technical Risks
|
|
92
|
+
- List unfamiliar technologies or patterns
|
|
93
|
+
- Note complex integrations or dependencies
|
|
94
|
+
- Identify performance bottlenecks
|
|
95
|
+
- Consider security implications
|
|
96
|
+
- Plan risk mitigation strategies
|
|
97
|
+
|
|
98
|
+
### Step 8: Apply Labels
|
|
99
|
+
- Label the GitHub issue with `phase:design`
|
|
100
|
+
- This will trigger automatic PR creation/update
|
|
101
|
+
|
|
102
|
+
## VALIDATION
|
|
103
|
+
|
|
104
|
+
### Phase Complete When:
|
|
105
|
+
- ✅ Technical design document created using appropriate template
|
|
106
|
+
- ✅ All specification requirements addressed technically
|
|
107
|
+
- ✅ System architecture clearly defined
|
|
108
|
+
- ✅ API design and data models specified
|
|
109
|
+
- ✅ Technical risks identified and mitigation planned
|
|
110
|
+
- ✅ Implementation plan provides clear guidance
|
|
111
|
+
- ✅ Testing strategy defined
|
|
112
|
+
- ✅ **PR feedback addressed** (if `docs/evidence/{issue_number}-design-feedback.md` exists)
|
|
113
|
+
- ✅ Issue labeled with `phase:design`
|
|
114
|
+
|
|
115
|
+
### Phase Incomplete If:
|
|
116
|
+
- ❌ Design document missing or incomplete
|
|
117
|
+
- ❌ Template structure not followed
|
|
118
|
+
- ❌ Specification requirements not addressed
|
|
119
|
+
- ❌ Architecture unclear or missing
|
|
120
|
+
- ❌ Technical risks not identified
|
|
121
|
+
- ❌ Implementation guidance insufficient
|
|
122
|
+
- ❌ **PR feedback not addressed** (UNADDRESSED items remain in feedback file)
|
|
123
|
+
|
|
124
|
+
### Report Back:
|
|
125
|
+
When you complete this phase, call:
|
|
126
|
+
|
|
127
|
+
```javascript
|
|
128
|
+
seekCoachingOnNextStep({
|
|
129
|
+
workflowType: "design",
|
|
130
|
+
issueNumber: "{issue_number}",
|
|
131
|
+
currentPhase: "design-design",
|
|
132
|
+
status: "complete",
|
|
133
|
+
evidence: {
|
|
134
|
+
designDocument: "docs/rfcs/{issue_number}-{title}.md",
|
|
135
|
+
templateUsed: "TECHSPEC-TEMPLATE.md",
|
|
136
|
+
architectureComponents: ["API layer", "Database schema", "UI components"],
|
|
137
|
+
technicalRisks: ["Integration with legacy system", "Performance at scale"],
|
|
138
|
+
issueLabeled: "phase:design"
|
|
139
|
+
},
|
|
140
|
+
findings: {
|
|
141
|
+
keyDecisions: ["Use REST API", "PostgreSQL for data storage", "React components"],
|
|
142
|
+
riskMitigation: ["Spike legacy integration", "Load testing plan"]
|
|
143
|
+
}
|
|
144
|
+
})
|
|
145
|
+
```
|
|
@@ -0,0 +1,108 @@
|
|
|
1
|
+
# Phase: Design
|
|
2
|
+
|
|
3
|
+
## INTENT
|
|
4
|
+
To create a comprehensive technical design that translates the feature specification into implementable architecture, APIs, and technical solutions.
|
|
5
|
+
|
|
6
|
+
## OUTCOME
|
|
7
|
+
A complete technical design document that:
|
|
8
|
+
- Translates spec requirements into technical architecture
|
|
9
|
+
- Defines APIs, data models, and system interactions
|
|
10
|
+
- Identifies technical risks and mitigation strategies
|
|
11
|
+
- Provides clear implementation guidance
|
|
12
|
+
- Includes technical validation criteria
|
|
13
|
+
|
|
14
|
+
## PRINCIPLES
|
|
15
|
+
- **Spec-Driven**: Design directly addresses specification requirements
|
|
16
|
+
- **Implementation-Ready**: Provides clear technical guidance
|
|
17
|
+
- **Risk-Aware**: Identifies and mitigates technical risks
|
|
18
|
+
- **Validation-Focused**: Includes technical validation criteria
|
|
19
|
+
- **Template-Compliant**: Follows RFC template structure
|
|
20
|
+
|
|
21
|
+
## WORKFLOW
|
|
22
|
+
|
|
23
|
+
### Step 1: Load Template and Specification
|
|
24
|
+
- Call `get_fraim_file({ path: "templates/specs/TECHSPEC-TEMPLATE.md" })` (for features)
|
|
25
|
+
- Call `get_fraim_file({ path: "templates/specs/BUGSPEC-TEMPLATE.md" })` (for bugs)
|
|
26
|
+
- Read the completed specification document from previous phase
|
|
27
|
+
- Call `get_fraim_file({ path: "rules/spike-first-development.md" })`
|
|
28
|
+
|
|
29
|
+
### Step 2: Analyze Specification Requirements
|
|
30
|
+
- Extract all technical requirements from specification
|
|
31
|
+
- Identify system components that need modification
|
|
32
|
+
- Determine API changes or additions needed
|
|
33
|
+
- Note any performance or scalability considerations
|
|
34
|
+
- List integration points with existing systems
|
|
35
|
+
|
|
36
|
+
### Step 3: Create Technical Design Document
|
|
37
|
+
- Create RFC file at `docs/rfcs/{issue_number}-{kebab-case-title}.md`
|
|
38
|
+
- Follow appropriate template structure (TECHSPEC or BUGSPEC)
|
|
39
|
+
- Include all required sections:
|
|
40
|
+
- Problem statement (from spec)
|
|
41
|
+
- Technical requirements
|
|
42
|
+
- Proposed solution architecture
|
|
43
|
+
- API design and data models
|
|
44
|
+
- Implementation plan
|
|
45
|
+
- Testing strategy
|
|
46
|
+
- Risk assessment and mitigation
|
|
47
|
+
- Alternative approaches considered
|
|
48
|
+
|
|
49
|
+
### Step 4: Design System Architecture
|
|
50
|
+
- Define component interactions and data flow
|
|
51
|
+
- Specify API endpoints and request/response formats
|
|
52
|
+
- Design database schema changes (if needed)
|
|
53
|
+
- Plan integration with existing systems
|
|
54
|
+
- Consider scalability and performance implications
|
|
55
|
+
|
|
56
|
+
### Step 5: Identify Technical Risks
|
|
57
|
+
- List unfamiliar technologies or patterns
|
|
58
|
+
- Note complex integrations or dependencies
|
|
59
|
+
- Identify performance bottlenecks
|
|
60
|
+
- Consider security implications
|
|
61
|
+
- Plan risk mitigation strategies
|
|
62
|
+
|
|
63
|
+
### Step 6: Apply Labels
|
|
64
|
+
- Label the GitHub issue with `phase:design`
|
|
65
|
+
- This will trigger automatic PR creation/update
|
|
66
|
+
|
|
67
|
+
## VALIDATION
|
|
68
|
+
|
|
69
|
+
### Phase Complete When:
|
|
70
|
+
- ✅ Technical design document created using appropriate template
|
|
71
|
+
- ✅ All specification requirements addressed technically
|
|
72
|
+
- ✅ System architecture clearly defined
|
|
73
|
+
- ✅ API design and data models specified
|
|
74
|
+
- ✅ Technical risks identified and mitigation planned
|
|
75
|
+
- ✅ Implementation plan provides clear guidance
|
|
76
|
+
- ✅ Testing strategy defined
|
|
77
|
+
- ✅ Issue labeled with `phase:design`
|
|
78
|
+
|
|
79
|
+
### Phase Incomplete If:
|
|
80
|
+
- ❌ Design document missing or incomplete
|
|
81
|
+
- ❌ Template structure not followed
|
|
82
|
+
- ❌ Specification requirements not addressed
|
|
83
|
+
- ❌ Architecture unclear or missing
|
|
84
|
+
- ❌ Technical risks not identified
|
|
85
|
+
- ❌ Implementation guidance insufficient
|
|
86
|
+
|
|
87
|
+
### Report Back:
|
|
88
|
+
When you complete this phase, call:
|
|
89
|
+
|
|
90
|
+
```javascript
|
|
91
|
+
seekCoachingOnNextStep({
|
|
92
|
+
workflowType: "design",
|
|
93
|
+
issueNumber: "{issue_number}",
|
|
94
|
+
currentPhase: "design",
|
|
95
|
+
status: "complete",
|
|
96
|
+
evidence: {
|
|
97
|
+
designDocument: "docs/rfcs/{issue_number}-{title}.md",
|
|
98
|
+
templateUsed: "TECHSPEC-TEMPLATE.md",
|
|
99
|
+
architectureComponents: ["API layer", "Database schema", "UI components"],
|
|
100
|
+
technicalRisks: ["Integration with legacy system", "Performance at scale"],
|
|
101
|
+
issueLabeled: "phase:design"
|
|
102
|
+
},
|
|
103
|
+
findings: {
|
|
104
|
+
keyDecisions: ["Use REST API", "PostgreSQL for data storage", "React components"],
|
|
105
|
+
riskMitigation: ["Spike legacy integration", "Load testing plan"]
|
|
106
|
+
}
|
|
107
|
+
})
|
|
108
|
+
```
|
|
@@ -0,0 +1,60 @@
|
|
|
1
|
+
# Phase: Finalize
|
|
2
|
+
|
|
3
|
+
## INTENT
|
|
4
|
+
To prepare completed work for human review by setting proper labels, creating/updating PR, and documenting evidence.
|
|
5
|
+
|
|
6
|
+
## OUTCOME
|
|
7
|
+
Work is ready for human review with:
|
|
8
|
+
- Proper issue labels applied
|
|
9
|
+
- PR created/updated with work
|
|
10
|
+
- Evidence documented
|
|
11
|
+
- Clean handoff to next phase or review
|
|
12
|
+
|
|
13
|
+
## WORKFLOW
|
|
14
|
+
|
|
15
|
+
### Step 1: Apply Issue Labels
|
|
16
|
+
- Remove current phase label (e.g., `phase:spec`, `phase:design`, `phase:impl`)
|
|
17
|
+
- Add `status:needs-review` label
|
|
18
|
+
- Keep any other relevant labels
|
|
19
|
+
|
|
20
|
+
### Step 2: Create/Update Pull Request
|
|
21
|
+
- Ensure PR exists for the feature branch
|
|
22
|
+
- Update PR title to reflect completed work
|
|
23
|
+
- Update PR description with summary of work completed
|
|
24
|
+
- Link to relevant documents (specs, designs, evidence)
|
|
25
|
+
|
|
26
|
+
### Step 3: Document Evidence
|
|
27
|
+
- Create evidence file at `docs/evidence/{issue_number}-{workflow}-evidence.md`
|
|
28
|
+
- Document key decisions made
|
|
29
|
+
- List deliverables created
|
|
30
|
+
- Note any recommendations for next phase
|
|
31
|
+
|
|
32
|
+
### Step 4: Prepare Handoff
|
|
33
|
+
- Ensure all work is committed to git
|
|
34
|
+
- Verify documentation is complete
|
|
35
|
+
- Confirm work is ready for human review
|
|
36
|
+
|
|
37
|
+
## VALIDATION
|
|
38
|
+
|
|
39
|
+
### Phase Complete When:
|
|
40
|
+
- ✅ Issue labels updated (`status:needs-review`)
|
|
41
|
+
- ✅ PR created/updated with work summary
|
|
42
|
+
- ✅ Evidence documented
|
|
43
|
+
- ✅ All work committed to git
|
|
44
|
+
- ✅ Ready for human review
|
|
45
|
+
|
|
46
|
+
### Report Back:
|
|
47
|
+
```javascript
|
|
48
|
+
seekCoachingOnNextStep({
|
|
49
|
+
workflowType: "{workflow_type}",
|
|
50
|
+
issueNumber: "{issue_number}",
|
|
51
|
+
currentPhase: "finalize",
|
|
52
|
+
status: "complete",
|
|
53
|
+
evidence: {
|
|
54
|
+
labelsApplied: ["status:needs-review"],
|
|
55
|
+
prUpdated: true,
|
|
56
|
+
evidenceDocumented: "docs/evidence/{issue_number}-{workflow}-evidence.md",
|
|
57
|
+
workCommitted: true
|
|
58
|
+
}
|
|
59
|
+
})
|
|
60
|
+
```
|
|
@@ -0,0 +1,125 @@
|
|
|
1
|
+
# Phase: Validate (Design)
|
|
2
|
+
|
|
3
|
+
## INTENT
|
|
4
|
+
To systematically validate that the technical design is complete, addresses all specification requirements, and provides clear implementation guidance.
|
|
5
|
+
|
|
6
|
+
## OUTCOME
|
|
7
|
+
Confirmation that:
|
|
8
|
+
- Design addresses all specification requirements
|
|
9
|
+
- Technical architecture is sound and implementable
|
|
10
|
+
- APIs and data models are well-defined
|
|
11
|
+
- Technical risks are identified and mitigated
|
|
12
|
+
- Implementation guidance is clear and actionable
|
|
13
|
+
|
|
14
|
+
## PRINCIPLES
|
|
15
|
+
- **Spec Alignment**: Design must address all specification requirements
|
|
16
|
+
- **Technical Soundness**: Architecture must be feasible and scalable
|
|
17
|
+
- **Implementation Ready**: Clear guidance for development phase
|
|
18
|
+
- **Risk Awareness**: Technical risks identified and mitigated
|
|
19
|
+
- **Quality Standards**: Professional-grade design ready for implementation
|
|
20
|
+
|
|
21
|
+
## WORKFLOW
|
|
22
|
+
|
|
23
|
+
### Step 1: Load Specification and Design
|
|
24
|
+
- Read the original specification document
|
|
25
|
+
- Read the completed design document
|
|
26
|
+
- Call `get_fraim_workflow({ workflow: "review-implementation-vs-design-spec" })`
|
|
27
|
+
|
|
28
|
+
### Step 2: Execute Design Validation Checklist
|
|
29
|
+
|
|
30
|
+
**Specification Alignment:**
|
|
31
|
+
1. **Requirements Coverage**: All spec requirements addressed in design
|
|
32
|
+
2. **Customer Needs**: Design supports all customer desired outcomes
|
|
33
|
+
3. **User Experience**: Technical design enables specified user workflows
|
|
34
|
+
4. **Validation Plan**: Design supports spec validation criteria
|
|
35
|
+
|
|
36
|
+
**Technical Soundness:**
|
|
37
|
+
5. **Architecture Clarity**: System components and interactions clearly defined
|
|
38
|
+
6. **API Design**: Endpoints, request/response formats well-specified
|
|
39
|
+
7. **Data Models**: Database schema and data structures defined
|
|
40
|
+
8. **Integration Points**: External system interactions documented
|
|
41
|
+
9. **Scalability**: Performance and scaling considerations addressed
|
|
42
|
+
|
|
43
|
+
**Implementation Readiness:**
|
|
44
|
+
10. **Implementation Plan**: Clear step-by-step development guidance
|
|
45
|
+
11. **Testing Strategy**: Comprehensive testing approach defined
|
|
46
|
+
12. **Technical Risks**: Risks identified with mitigation strategies
|
|
47
|
+
13. **Alternative Analysis**: Other approaches considered and rejected
|
|
48
|
+
|
|
49
|
+
**Quality Standards:**
|
|
50
|
+
14. **Template Compliance**: Follows TECHSPEC/BUGSPEC template structure
|
|
51
|
+
15. **Professional Quality**: Clear writing, proper formatting, complete sections
|
|
52
|
+
|
|
53
|
+
### Step 3: Risk Assessment Review
|
|
54
|
+
- Verify all technical risks are realistic and significant
|
|
55
|
+
- Confirm mitigation strategies are actionable
|
|
56
|
+
- Check if any risks require spikes or prototypes
|
|
57
|
+
- Assess overall project feasibility
|
|
58
|
+
|
|
59
|
+
### Step 4: Implementation Guidance Review
|
|
60
|
+
- Ensure implementation plan is detailed enough
|
|
61
|
+
- Verify testing strategy covers all requirements
|
|
62
|
+
- Check that guidance addresses potential challenges
|
|
63
|
+
- Confirm design enables spec validation plan
|
|
64
|
+
|
|
65
|
+
### Step 5: Generate Validation Report
|
|
66
|
+
Document findings for each validation area:
|
|
67
|
+
- Specification alignment assessment
|
|
68
|
+
- Technical soundness evaluation
|
|
69
|
+
- Implementation readiness review
|
|
70
|
+
- Quality standards compliance
|
|
71
|
+
- Overall recommendation
|
|
72
|
+
|
|
73
|
+
## VALIDATION
|
|
74
|
+
|
|
75
|
+
### Phase Complete When:
|
|
76
|
+
- ✅ All specification requirements addressed in design
|
|
77
|
+
- ✅ Technical architecture is sound and implementable
|
|
78
|
+
- ✅ APIs and data models clearly defined
|
|
79
|
+
- ✅ Technical risks identified with mitigation plans
|
|
80
|
+
- ✅ Implementation guidance is clear and actionable
|
|
81
|
+
- ✅ Quality standards met consistently
|
|
82
|
+
- ✅ Validation report generated with evidence
|
|
83
|
+
|
|
84
|
+
### Phase Incomplete If:
|
|
85
|
+
- ❌ Specification requirements not fully addressed
|
|
86
|
+
- ❌ Technical architecture unclear or infeasible
|
|
87
|
+
- ❌ API design incomplete or poorly defined
|
|
88
|
+
- ❌ Technical risks not identified or mitigated
|
|
89
|
+
- ❌ Implementation guidance insufficient
|
|
90
|
+
- ❌ Quality issues identified
|
|
91
|
+
|
|
92
|
+
### Report Back:
|
|
93
|
+
When you complete this phase, call:
|
|
94
|
+
|
|
95
|
+
```javascript
|
|
96
|
+
seekCoachingOnNextStep({
|
|
97
|
+
workflowType: "design",
|
|
98
|
+
issueNumber: "{issue_number}",
|
|
99
|
+
currentPhase: "validate",
|
|
100
|
+
status: "complete",
|
|
101
|
+
evidence: {
|
|
102
|
+
validationReport: "All specification requirements addressed, architecture sound",
|
|
103
|
+
specAlignmentScore: "100% - all requirements covered",
|
|
104
|
+
technicalSoundness: "PASS - architecture feasible and scalable",
|
|
105
|
+
implementationReadiness: "PASS - clear guidance provided",
|
|
106
|
+
qualityAssessment: "Professional grade, ready for implementation",
|
|
107
|
+
validationDecision: "PASS"
|
|
108
|
+
}
|
|
109
|
+
})
|
|
110
|
+
```
|
|
111
|
+
|
|
112
|
+
If validation fails:
|
|
113
|
+
```javascript
|
|
114
|
+
seekCoachingOnNextStep({
|
|
115
|
+
workflowType: "design",
|
|
116
|
+
issueNumber: "{issue_number}",
|
|
117
|
+
currentPhase: "validate",
|
|
118
|
+
status: "incomplete",
|
|
119
|
+
findings: {
|
|
120
|
+
failedAreas: ["API design incomplete", "Technical risks not mitigated"],
|
|
121
|
+
validationDecision: "FAIL",
|
|
122
|
+
nextAction: "Return to design phase to address validation failures"
|
|
123
|
+
}
|
|
124
|
+
})
|
|
125
|
+
```
|