sdd-mcp-server 1.8.0 → 2.0.0
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 +97 -41
- package/dist/cli/install-skills.d.ts +53 -0
- package/dist/cli/install-skills.js +171 -0
- package/dist/cli/install-skills.js.map +1 -0
- package/dist/index.js +140 -122
- package/dist/index.js.map +1 -1
- package/dist/skills/SkillManager.d.ts +69 -0
- package/dist/skills/SkillManager.js +138 -0
- package/dist/skills/SkillManager.js.map +1 -0
- package/package.json +6 -3
- package/skills/sdd-commit/SKILL.md +285 -0
- package/skills/sdd-design/SKILL.md +262 -0
- package/skills/sdd-implement/SKILL.md +281 -0
- package/skills/sdd-requirements/SKILL.md +140 -0
- package/skills/sdd-steering/SKILL.md +225 -0
- package/skills/sdd-steering-custom/SKILL.md +211 -0
- package/skills/sdd-tasks/SKILL.md +233 -0
- package/skills/simple-task/SKILL.md +148 -0
|
@@ -0,0 +1,148 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: simple-task
|
|
3
|
+
description: Implement simple features with best practices. Use when adding small features, bug fixes, or quick enhancements without the full SDD workflow. Invoked via /simple-task <description>.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Simple Task Implementation
|
|
7
|
+
|
|
8
|
+
Implement small features, bug fixes, or quick enhancements while following best practices from the project's steering documents.
|
|
9
|
+
|
|
10
|
+
## When to Use
|
|
11
|
+
|
|
12
|
+
Use `/simple-task` for:
|
|
13
|
+
- Small feature additions (e.g., "add logout button")
|
|
14
|
+
- Bug fixes
|
|
15
|
+
- Minor enhancements
|
|
16
|
+
- Quick refactoring
|
|
17
|
+
|
|
18
|
+
Use **full SDD workflow** for:
|
|
19
|
+
- Complex features requiring multiple components
|
|
20
|
+
- New modules or subsystems
|
|
21
|
+
- Features needing formal requirements/design review
|
|
22
|
+
|
|
23
|
+
## Workflow
|
|
24
|
+
|
|
25
|
+
### Step 1: Understand the Task
|
|
26
|
+
|
|
27
|
+
1. Clarify what needs to be done
|
|
28
|
+
2. Identify affected files/components
|
|
29
|
+
3. Estimate scope (if larger than expected, suggest full SDD workflow)
|
|
30
|
+
|
|
31
|
+
### Step 2: Apply TDD (Test-Driven Development)
|
|
32
|
+
|
|
33
|
+
**Reference:** `.spec/steering/tdd-guideline.md`
|
|
34
|
+
|
|
35
|
+
Follow the Red-Green-Refactor cycle:
|
|
36
|
+
|
|
37
|
+
```
|
|
38
|
+
1. RED → Write a failing test first
|
|
39
|
+
2. GREEN → Write minimal code to pass
|
|
40
|
+
3. REFACTOR → Clean up while tests pass
|
|
41
|
+
```
|
|
42
|
+
|
|
43
|
+
**Quick TDD Checklist:**
|
|
44
|
+
- [ ] Write test before implementation
|
|
45
|
+
- [ ] Test describes expected behavior
|
|
46
|
+
- [ ] Minimal code to make test pass
|
|
47
|
+
- [ ] Refactor without breaking tests
|
|
48
|
+
|
|
49
|
+
### Step 3: Apply Design Principles
|
|
50
|
+
|
|
51
|
+
**Reference:** `.spec/steering/principles.md`
|
|
52
|
+
|
|
53
|
+
**SOLID Quick Reference:**
|
|
54
|
+
- **S**ingle Responsibility: Each function/class does one thing
|
|
55
|
+
- **O**pen/Closed: Extend behavior without modifying existing code
|
|
56
|
+
- **L**iskov Substitution: Subtypes must be substitutable
|
|
57
|
+
- **I**nterface Segregation: Small, focused interfaces
|
|
58
|
+
- **D**ependency Inversion: Depend on abstractions
|
|
59
|
+
|
|
60
|
+
**Other Principles:**
|
|
61
|
+
- **DRY**: Don't repeat yourself - extract common logic
|
|
62
|
+
- **KISS**: Keep it simple - avoid unnecessary complexity
|
|
63
|
+
- **YAGNI**: You aren't gonna need it - only implement what's required
|
|
64
|
+
|
|
65
|
+
### Step 4: Code Quality Review
|
|
66
|
+
|
|
67
|
+
**Reference:** `.spec/steering/linus-review.md`
|
|
68
|
+
|
|
69
|
+
Before finalizing, ask:
|
|
70
|
+
1. **Taste**: Is the solution elegant? Can special cases be eliminated?
|
|
71
|
+
2. **Simplicity**: Can it be simpler? Fewer lines? Less nesting?
|
|
72
|
+
3. **Data Structures**: Is the right data structure used?
|
|
73
|
+
4. **Breaking Changes**: Does this break existing functionality?
|
|
74
|
+
|
|
75
|
+
**Quality Checklist:**
|
|
76
|
+
- [ ] Functions are short and focused
|
|
77
|
+
- [ ] No more than 3 levels of indentation
|
|
78
|
+
- [ ] Clear, descriptive naming
|
|
79
|
+
- [ ] No unnecessary complexity
|
|
80
|
+
|
|
81
|
+
### Step 5: Security Check
|
|
82
|
+
|
|
83
|
+
**Reference:** `.spec/steering/owasp-top10-check.md`
|
|
84
|
+
|
|
85
|
+
**Quick Security Checklist:**
|
|
86
|
+
- [ ] Input validation (sanitize user inputs)
|
|
87
|
+
- [ ] No SQL/command injection (use parameterized queries)
|
|
88
|
+
- [ ] Access control enforced
|
|
89
|
+
- [ ] No secrets in code (use env vars)
|
|
90
|
+
- [ ] Proper error handling (no stack traces to users)
|
|
91
|
+
|
|
92
|
+
### Step 6: Implement and Test
|
|
93
|
+
|
|
94
|
+
1. Write the failing test (RED)
|
|
95
|
+
2. Implement minimal solution (GREEN)
|
|
96
|
+
3. Run tests to confirm pass
|
|
97
|
+
4. Refactor if needed
|
|
98
|
+
5. Run full test suite
|
|
99
|
+
6. Verify no lint/type errors
|
|
100
|
+
|
|
101
|
+
## Reference Documents
|
|
102
|
+
|
|
103
|
+
These steering documents provide detailed guidance:
|
|
104
|
+
|
|
105
|
+
| Document | Content |
|
|
106
|
+
|----------|---------|
|
|
107
|
+
| `tdd-guideline.md` | TDD methodology, test pyramid, Red-Green-Refactor |
|
|
108
|
+
| `principles.md` | SOLID, DRY, KISS, YAGNI, Separation of Concerns |
|
|
109
|
+
| `linus-review.md` | Code quality, "good taste", simplicity standards |
|
|
110
|
+
| `owasp-top10-check.md` | Security checklist (OWASP Top 10) |
|
|
111
|
+
|
|
112
|
+
## Output
|
|
113
|
+
|
|
114
|
+
After implementing, provide:
|
|
115
|
+
|
|
116
|
+
```markdown
|
|
117
|
+
## Implementation Summary
|
|
118
|
+
|
|
119
|
+
**Task:** {what was implemented}
|
|
120
|
+
|
|
121
|
+
**Changes:**
|
|
122
|
+
- {file1}: {what changed}
|
|
123
|
+
- {file2}: {what changed}
|
|
124
|
+
|
|
125
|
+
**Tests Added:**
|
|
126
|
+
- {test description}
|
|
127
|
+
|
|
128
|
+
**Principles Applied:**
|
|
129
|
+
- TDD: {how TDD was followed}
|
|
130
|
+
- Design: {which principles were applied}
|
|
131
|
+
- Security: {security considerations}
|
|
132
|
+
|
|
133
|
+
**Ready for:** {commit / further review / testing}
|
|
134
|
+
```
|
|
135
|
+
|
|
136
|
+
## Example
|
|
137
|
+
|
|
138
|
+
```
|
|
139
|
+
User: /simple-task add a logout button to the navbar
|
|
140
|
+
|
|
141
|
+
Claude:
|
|
142
|
+
1. Understand: Add logout button that clears session and redirects to login
|
|
143
|
+
2. TDD: Write test for logout functionality first
|
|
144
|
+
3. Principles: Single responsibility - logout logic in AuthService
|
|
145
|
+
4. Security: Ensure session is properly invalidated
|
|
146
|
+
5. Implement: Button component + logout handler
|
|
147
|
+
6. Test: Verify all tests pass
|
|
148
|
+
```
|