opencode-swarm-plugin 0.12.27 → 0.12.28
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 +7 -14
- package/bin/swarm.ts +100 -50
- package/package.json +1 -1
- package/global-skills/agent-patterns/SKILL.md +0 -682
- package/global-skills/code-review/SKILL.md +0 -166
- package/global-skills/debugging/SKILL.md +0 -150
- package/global-skills/resilience-patterns/SKILL.md +0 -648
- package/global-skills/tacit-knowledge-extraction/SKILL.md +0 -387
- package/global-skills/testing-strategies/SKILL.md +0 -558
- package/global-skills/zod-validation/SKILL.md +0 -763
|
@@ -1,166 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: code-review
|
|
3
|
-
description: Code review patterns for agents. Use when reviewing changes before committing, evaluating pull requests, or assessing code quality. Provides systematic review checklists and feedback patterns.
|
|
4
|
-
tags:
|
|
5
|
-
- review
|
|
6
|
-
- quality
|
|
7
|
-
- best-practices
|
|
8
|
-
tools:
|
|
9
|
-
- swarm_evaluation_prompt
|
|
10
|
-
---
|
|
11
|
-
|
|
12
|
-
# Code Review Skill
|
|
13
|
-
|
|
14
|
-
Systematic code review patterns for ensuring quality.
|
|
15
|
-
|
|
16
|
-
## When to Use
|
|
17
|
-
|
|
18
|
-
- Before committing changes
|
|
19
|
-
- Reviewing pull requests
|
|
20
|
-
- Evaluating code quality
|
|
21
|
-
- Self-review before completing subtasks
|
|
22
|
-
- Assessing unfamiliar code
|
|
23
|
-
|
|
24
|
-
## Review Protocol
|
|
25
|
-
|
|
26
|
-
### 1. Understand Context
|
|
27
|
-
|
|
28
|
-
Before reviewing code:
|
|
29
|
-
- What is this change trying to accomplish?
|
|
30
|
-
- What files/modules are affected?
|
|
31
|
-
- Are there related tests?
|
|
32
|
-
- Is this a bugfix, feature, or refactor?
|
|
33
|
-
|
|
34
|
-
### 2. Quick Scan
|
|
35
|
-
|
|
36
|
-
First pass - high level issues:
|
|
37
|
-
- Does the change make sense overall?
|
|
38
|
-
- Is the approach reasonable?
|
|
39
|
-
- Any obvious red flags?
|
|
40
|
-
|
|
41
|
-
### 3. Detailed Review
|
|
42
|
-
|
|
43
|
-
For each file changed:
|
|
44
|
-
|
|
45
|
-
**Correctness**
|
|
46
|
-
- Does the logic do what it's supposed to?
|
|
47
|
-
- Are edge cases handled?
|
|
48
|
-
- Are there potential bugs?
|
|
49
|
-
|
|
50
|
-
**Security**
|
|
51
|
-
- Input validation?
|
|
52
|
-
- Authentication/authorization checks?
|
|
53
|
-
- Sensitive data exposure?
|
|
54
|
-
- Injection vulnerabilities?
|
|
55
|
-
|
|
56
|
-
**Performance**
|
|
57
|
-
- Unnecessary loops or database calls?
|
|
58
|
-
- Memory leaks potential?
|
|
59
|
-
- Appropriate data structures?
|
|
60
|
-
|
|
61
|
-
**Maintainability**
|
|
62
|
-
- Clear naming?
|
|
63
|
-
- Reasonable complexity?
|
|
64
|
-
- Follows existing patterns?
|
|
65
|
-
|
|
66
|
-
**Testing**
|
|
67
|
-
- Tests for new functionality?
|
|
68
|
-
- Edge cases tested?
|
|
69
|
-
- Tests actually test the right things?
|
|
70
|
-
|
|
71
|
-
### 4. Provide Feedback
|
|
72
|
-
|
|
73
|
-
Structure feedback clearly:
|
|
74
|
-
|
|
75
|
-
```markdown
|
|
76
|
-
## Summary
|
|
77
|
-
[Overall assessment - approve, needs changes, questions]
|
|
78
|
-
|
|
79
|
-
## Issues
|
|
80
|
-
- [Critical] Issue description and suggestion
|
|
81
|
-
- [Minor] Style issue
|
|
82
|
-
|
|
83
|
-
## Questions
|
|
84
|
-
- Why was X approached this way?
|
|
85
|
-
|
|
86
|
-
## Suggestions (optional)
|
|
87
|
-
- Consider using Y pattern here
|
|
88
|
-
```
|
|
89
|
-
|
|
90
|
-
## Review Checklists
|
|
91
|
-
|
|
92
|
-
### For New Features
|
|
93
|
-
|
|
94
|
-
- [ ] Feature works as described
|
|
95
|
-
- [ ] Tests cover happy path and edge cases
|
|
96
|
-
- [ ] Error handling is appropriate
|
|
97
|
-
- [ ] No hardcoded values that should be configurable
|
|
98
|
-
- [ ] Documentation updated if needed
|
|
99
|
-
- [ ] No TODO comments for critical functionality
|
|
100
|
-
|
|
101
|
-
### For Bug Fixes
|
|
102
|
-
|
|
103
|
-
- [ ] Bug is actually fixed
|
|
104
|
-
- [ ] Regression test added
|
|
105
|
-
- [ ] Root cause addressed (not just symptoms)
|
|
106
|
-
- [ ] Similar issues elsewhere checked
|
|
107
|
-
- [ ] No new bugs introduced
|
|
108
|
-
|
|
109
|
-
### For Refactoring
|
|
110
|
-
|
|
111
|
-
- [ ] Behavior unchanged (all tests pass)
|
|
112
|
-
- [ ] Code actually improved
|
|
113
|
-
- [ ] No unnecessary changes
|
|
114
|
-
- [ ] No feature changes mixed in
|
|
115
|
-
|
|
116
|
-
## Self-Review Before Commit
|
|
117
|
-
|
|
118
|
-
Use `swarm_evaluation_prompt` for structured self-evaluation:
|
|
119
|
-
|
|
120
|
-
```
|
|
121
|
-
swarm_evaluation_prompt(
|
|
122
|
-
bead_id: "bd-xxx",
|
|
123
|
-
subtask_title: "What I implemented",
|
|
124
|
-
files_touched: ["file1.ts", "file2.ts"]
|
|
125
|
-
)
|
|
126
|
-
```
|
|
127
|
-
|
|
128
|
-
Criteria:
|
|
129
|
-
- **type_safe**: Does it compile without errors?
|
|
130
|
-
- **no_bugs**: Any obvious bugs or edge cases?
|
|
131
|
-
- **patterns**: Follows existing code style?
|
|
132
|
-
- **readable**: Would another developer understand it?
|
|
133
|
-
|
|
134
|
-
## Common Issues to Catch
|
|
135
|
-
|
|
136
|
-
### Logic Errors
|
|
137
|
-
- Off-by-one errors
|
|
138
|
-
- Incorrect boolean logic
|
|
139
|
-
- Missing null checks
|
|
140
|
-
- Race conditions
|
|
141
|
-
|
|
142
|
-
### Style Issues
|
|
143
|
-
- Inconsistent naming
|
|
144
|
-
- Magic numbers
|
|
145
|
-
- Deep nesting
|
|
146
|
-
- Large functions
|
|
147
|
-
|
|
148
|
-
### Security Issues
|
|
149
|
-
- SQL injection potential
|
|
150
|
-
- XSS vulnerabilities
|
|
151
|
-
- Hardcoded credentials
|
|
152
|
-
- Missing auth checks
|
|
153
|
-
|
|
154
|
-
### Performance Issues
|
|
155
|
-
- N+1 queries
|
|
156
|
-
- Unnecessary re-renders
|
|
157
|
-
- Memory leaks
|
|
158
|
-
- Blocking operations
|
|
159
|
-
|
|
160
|
-
## Swarm Integration
|
|
161
|
-
|
|
162
|
-
When reviewing in a swarm:
|
|
163
|
-
1. **Coordinate reviews** - Don't duplicate effort
|
|
164
|
-
2. **Share patterns** - If you catch something, broadcast it
|
|
165
|
-
3. **Track issues** - Create beads for non-blocking issues
|
|
166
|
-
4. **Learn together** - Good review findings become skills
|
|
@@ -1,150 +0,0 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: debugging
|
|
3
|
-
description: Systematic debugging patterns for agents. Use when encountering errors, unexpected behavior, or when tests fail. Provides structured approaches for root cause analysis and error resolution.
|
|
4
|
-
tags:
|
|
5
|
-
- debugging
|
|
6
|
-
- errors
|
|
7
|
-
- troubleshooting
|
|
8
|
-
tools:
|
|
9
|
-
- swarm_accumulate_error
|
|
10
|
-
- swarm_get_error_context
|
|
11
|
-
- beads_create
|
|
12
|
-
- semantic-memory_store
|
|
13
|
-
---
|
|
14
|
-
|
|
15
|
-
# Debugging Skill
|
|
16
|
-
|
|
17
|
-
Systematic approaches for diagnosing and resolving issues.
|
|
18
|
-
|
|
19
|
-
## When to Use
|
|
20
|
-
|
|
21
|
-
- Encountering error messages or exceptions
|
|
22
|
-
- Tests failing unexpectedly
|
|
23
|
-
- Behavior differs from expectations
|
|
24
|
-
- Performance issues or timeouts
|
|
25
|
-
- Build or compilation errors
|
|
26
|
-
|
|
27
|
-
## Debugging Protocol
|
|
28
|
-
|
|
29
|
-
### 1. Reproduce the Issue
|
|
30
|
-
|
|
31
|
-
Before fixing, ensure you can reproduce:
|
|
32
|
-
|
|
33
|
-
```bash
|
|
34
|
-
# Run the failing command/test again
|
|
35
|
-
# Note exact error output
|
|
36
|
-
# Capture any stack traces
|
|
37
|
-
```
|
|
38
|
-
|
|
39
|
-
**Record what you find** - Use `swarm_accumulate_error` to track errors for retry context.
|
|
40
|
-
|
|
41
|
-
### 2. Gather Context
|
|
42
|
-
|
|
43
|
-
Read surrounding code and error context:
|
|
44
|
-
|
|
45
|
-
- Error message and stack trace
|
|
46
|
-
- Recent changes to affected files
|
|
47
|
-
- Related configuration files
|
|
48
|
-
- Test fixtures and setup
|
|
49
|
-
|
|
50
|
-
### 3. Form Hypotheses
|
|
51
|
-
|
|
52
|
-
Based on the error, list possible causes:
|
|
53
|
-
|
|
54
|
-
1. Most likely cause based on error message
|
|
55
|
-
2. Recent changes that could cause this
|
|
56
|
-
3. Environmental/configuration issues
|
|
57
|
-
4. Edge cases or missing validations
|
|
58
|
-
|
|
59
|
-
### 4. Test Hypotheses Systematically
|
|
60
|
-
|
|
61
|
-
For each hypothesis:
|
|
62
|
-
|
|
63
|
-
1. Add targeted logging or assertions
|
|
64
|
-
2. Modify one variable at a time
|
|
65
|
-
3. Verify if behavior changes
|
|
66
|
-
4. Document findings
|
|
67
|
-
|
|
68
|
-
### 5. Implement Fix
|
|
69
|
-
|
|
70
|
-
Once root cause is identified:
|
|
71
|
-
|
|
72
|
-
1. Write the minimal fix
|
|
73
|
-
2. Add regression test
|
|
74
|
-
3. Verify original error is gone
|
|
75
|
-
4. Check for similar issues elsewhere
|
|
76
|
-
|
|
77
|
-
### 6. Document Learning
|
|
78
|
-
|
|
79
|
-
Use `semantic-memory_store` if you discovered a pattern worth preserving for future sessions.
|
|
80
|
-
|
|
81
|
-
## Common Error Patterns
|
|
82
|
-
|
|
83
|
-
### Type Errors
|
|
84
|
-
|
|
85
|
-
```
|
|
86
|
-
Error: Cannot read property 'x' of undefined
|
|
87
|
-
```
|
|
88
|
-
|
|
89
|
-
**Check**: Variable initialization, null/undefined checks, async timing
|
|
90
|
-
|
|
91
|
-
### Import Errors
|
|
92
|
-
|
|
93
|
-
```
|
|
94
|
-
Error: Module not found
|
|
95
|
-
```
|
|
96
|
-
|
|
97
|
-
**Check**: Path correctness, package installation, export statements
|
|
98
|
-
|
|
99
|
-
### Timeout Errors
|
|
100
|
-
|
|
101
|
-
```
|
|
102
|
-
Error: Timeout of 5000ms exceeded
|
|
103
|
-
```
|
|
104
|
-
|
|
105
|
-
**Check**: Async operations, network calls, infinite loops
|
|
106
|
-
|
|
107
|
-
### Validation Errors
|
|
108
|
-
|
|
109
|
-
```
|
|
110
|
-
Error: Validation failed for field 'x'
|
|
111
|
-
```
|
|
112
|
-
|
|
113
|
-
**Check**: Input data, schema definitions, required fields
|
|
114
|
-
|
|
115
|
-
## Debugging Tools
|
|
116
|
-
|
|
117
|
-
### Console Logging
|
|
118
|
-
|
|
119
|
-
```typescript
|
|
120
|
-
console.log("[DEBUG]", { variable, context });
|
|
121
|
-
```
|
|
122
|
-
|
|
123
|
-
### Breakpoints (if interactive)
|
|
124
|
-
|
|
125
|
-
- Add `debugger;` statement
|
|
126
|
-
- Use IDE breakpoints
|
|
127
|
-
|
|
128
|
-
### Binary Search
|
|
129
|
-
|
|
130
|
-
For "it was working before":
|
|
131
|
-
|
|
132
|
-
1. Find last known good commit
|
|
133
|
-
2. Binary search through commits
|
|
134
|
-
3. Identify the breaking change
|
|
135
|
-
|
|
136
|
-
## Swarm Integration
|
|
137
|
-
|
|
138
|
-
When debugging in a swarm:
|
|
139
|
-
|
|
140
|
-
1. **Report blocker immediately** - Don't spin alone
|
|
141
|
-
2. **Share context** - Use `swarm_broadcast` with findings
|
|
142
|
-
3. **Create beads** - Track discovered issues as bugs
|
|
143
|
-
4. **Check other agents** - They might have relevant context
|
|
144
|
-
|
|
145
|
-
## Anti-Patterns
|
|
146
|
-
|
|
147
|
-
- **Shotgun debugging** - Random changes hoping something works
|
|
148
|
-
- **Silent fixes** - Fixing without understanding root cause
|
|
149
|
-
- **Ignoring warnings** - Warnings often predict errors
|
|
150
|
-
- **Assuming environment** - Works on my machine != works everywhere
|