clikit-plugin 0.1.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/AGENTS.md +46 -0
- package/README.md +282 -0
- package/command/commit.md +137 -0
- package/command/create.md +150 -0
- package/command/debug.md +185 -0
- package/command/design.md +82 -0
- package/command/handoff.md +95 -0
- package/command/import-plan.md +175 -0
- package/command/init.md +112 -0
- package/command/issue.md +110 -0
- package/command/plan.md +205 -0
- package/command/pr.md +174 -0
- package/command/research.md +103 -0
- package/command/resume.md +80 -0
- package/command/review-codebase.md +228 -0
- package/command/review.md +135 -0
- package/command/ship.md +109 -0
- package/command/start.md +77 -0
- package/command/status.md +123 -0
- package/command/verify.md +79 -0
- package/command/vision.md +142 -0
- package/dist/agents/index.d.ts +4 -0
- package/dist/agents/index.d.ts.map +1 -0
- package/dist/cli.d.ts +3 -0
- package/dist/cli.d.ts.map +1 -0
- package/dist/cli.js +103 -0
- package/dist/clikit.schema.json +423 -0
- package/dist/commands/index.d.ts +4 -0
- package/dist/commands/index.d.ts.map +1 -0
- package/dist/config.d.ts +118 -0
- package/dist/config.d.ts.map +1 -0
- package/dist/hooks/auto-format.d.ts +30 -0
- package/dist/hooks/auto-format.d.ts.map +1 -0
- package/dist/hooks/comment-checker.d.ts +17 -0
- package/dist/hooks/comment-checker.d.ts.map +1 -0
- package/dist/hooks/compaction.d.ts +60 -0
- package/dist/hooks/compaction.d.ts.map +1 -0
- package/dist/hooks/empty-message-sanitizer.d.ts +9 -0
- package/dist/hooks/empty-message-sanitizer.d.ts.map +1 -0
- package/dist/hooks/env-context.d.ts +43 -0
- package/dist/hooks/env-context.d.ts.map +1 -0
- package/dist/hooks/git-guard.d.ts +14 -0
- package/dist/hooks/git-guard.d.ts.map +1 -0
- package/dist/hooks/index.d.ts +35 -0
- package/dist/hooks/index.d.ts.map +1 -0
- package/dist/hooks/ritual-enforcer.d.ts +29 -0
- package/dist/hooks/ritual-enforcer.d.ts.map +1 -0
- package/dist/hooks/security-check.d.ts +20 -0
- package/dist/hooks/security-check.d.ts.map +1 -0
- package/dist/hooks/session-notification.d.ts +23 -0
- package/dist/hooks/session-notification.d.ts.map +1 -0
- package/dist/hooks/subagent-question-blocker.d.ts +11 -0
- package/dist/hooks/subagent-question-blocker.d.ts.map +1 -0
- package/dist/hooks/swarm-enforcer.d.ts +31 -0
- package/dist/hooks/swarm-enforcer.d.ts.map +1 -0
- package/dist/hooks/todo-enforcer.d.ts +19 -0
- package/dist/hooks/todo-enforcer.d.ts.map +1 -0
- package/dist/hooks/truncator.d.ts +28 -0
- package/dist/hooks/truncator.d.ts.map +1 -0
- package/dist/hooks/typecheck-gate.d.ts +31 -0
- package/dist/hooks/typecheck-gate.d.ts.map +1 -0
- package/dist/index.d.ts +13 -0
- package/dist/index.d.ts.map +1 -0
- package/dist/index.js +5785 -0
- package/dist/skills/index.d.ts +9 -0
- package/dist/skills/index.d.ts.map +1 -0
- package/dist/tools/beads-memory-sync.d.ts +17 -0
- package/dist/tools/beads-memory-sync.d.ts.map +1 -0
- package/dist/tools/context-summary.d.ts +21 -0
- package/dist/tools/context-summary.d.ts.map +1 -0
- package/dist/tools/index.d.ts +7 -0
- package/dist/tools/index.d.ts.map +1 -0
- package/dist/tools/memory.d.ts +56 -0
- package/dist/tools/memory.d.ts.map +1 -0
- package/dist/tools/observation.d.ts +28 -0
- package/dist/tools/observation.d.ts.map +1 -0
- package/dist/tools/quick-research.d.ts +16 -0
- package/dist/tools/quick-research.d.ts.map +1 -0
- package/dist/tools/swarm.d.ts +57 -0
- package/dist/tools/swarm.d.ts.map +1 -0
- package/dist/types.d.ts +32 -0
- package/dist/types.d.ts.map +1 -0
- package/memory/_templates/handoff.md +114 -0
- package/memory/_templates/plan.md +135 -0
- package/memory/_templates/prd.md +147 -0
- package/memory/_templates/research.md +108 -0
- package/memory/_templates/review.md +117 -0
- package/memory/_templates/spec.md +116 -0
- package/memory/beads/.gitkeep +0 -0
- package/memory/handoffs/.gitkeep +0 -0
- package/memory/plans/.gitkeep +0 -0
- package/memory/prds/.gitkeep +0 -0
- package/memory/research/.gitkeep +0 -0
- package/memory/reviews/.gitkeep +0 -0
- package/memory/specs/.gitkeep +0 -0
- package/package.json +64 -0
- package/skill/accessibility-audit/SKILL.md +115 -0
- package/skill/beads/SKILL.md +56 -0
- package/skill/beads-bridge/SKILL.md +45 -0
- package/skill/brainstorming/SKILL.md +41 -0
- package/skill/chrome-devtools/SKILL.md +36 -0
- package/skill/cloudflare/SKILL.md +96 -0
- package/skill/condition-based-waiting/SKILL.md +92 -0
- package/skill/deep-research/SKILL.md +130 -0
- package/skill/defense-in-depth/SKILL.md +89 -0
- package/skill/design-system-audit/SKILL.md +136 -0
- package/skill/development-lifecycle/SKILL.md +58 -0
- package/skill/dispatching-parallel-agents/SKILL.md +94 -0
- package/skill/executing-plans/SKILL.md +54 -0
- package/skill/figma/SKILL.md +34 -0
- package/skill/finishing-a-development-branch/SKILL.md +127 -0
- package/skill/frontend-aesthetics/SKILL.md +63 -0
- package/skill/gemini-large-context/SKILL.md +80 -0
- package/skill/mockup-to-code/SKILL.md +98 -0
- package/skill/mqdh/SKILL.md +54 -0
- package/skill/notebooklm/SKILL.md +71 -0
- package/skill/playwright/SKILL.md +37 -0
- package/skill/playwriter/SKILL.md +56 -0
- package/skill/polar/SKILL.md +39 -0
- package/skill/receiving-code-review/SKILL.md +47 -0
- package/skill/requesting-code-review/SKILL.md +47 -0
- package/skill/resend/SKILL.md +85 -0
- package/skill/ritual-workflow/SKILL.md +101 -0
- package/skill/root-cause-tracing/SKILL.md +73 -0
- package/skill/session-management/SKILL.md +53 -0
- package/skill/sharing-skills/SKILL.md +50 -0
- package/skill/source-code-research/SKILL.md +138 -0
- package/skill/subagent-driven-development/SKILL.md +69 -0
- package/skill/supabase/SKILL.md +80 -0
- package/skill/supabase-postgres-best-practices/SKILL.md +168 -0
- package/skill/swarm-coordination/SKILL.md +54 -0
- package/skill/systematic-debugging/SKILL.md +58 -0
- package/skill/test-driven-development/SKILL.md +53 -0
- package/skill/testing-anti-patterns/SKILL.md +100 -0
- package/skill/testing-skills-with-subagents/SKILL.md +102 -0
- package/skill/ui-ux-research/SKILL.md +93 -0
- package/skill/using-git-worktrees/SKILL.md +129 -0
- package/skill/v0/SKILL.md +67 -0
- package/skill/v1-run/SKILL.md +85 -0
- package/skill/vercel-react-best-practices/SKILL.md +174 -0
- package/skill/verification-before-completion/SKILL.md +55 -0
- package/skill/visual-analysis/SKILL.md +113 -0
- package/skill/writing-plans/SKILL.md +55 -0
- package/skill/writing-skills/SKILL.md +68 -0
- package/src/agents/AGENTS.md +46 -0
- package/src/agents/build.md +170 -0
- package/src/agents/explore.md +113 -0
- package/src/agents/general.md +92 -0
- package/src/agents/index.ts +64 -0
- package/src/agents/librarian.md +116 -0
- package/src/agents/looker.md +107 -0
- package/src/agents/oracle.md +138 -0
- package/src/agents/plan.md +78 -0
- package/src/agents/review.md +98 -0
- package/src/agents/scout.md +84 -0
- package/src/agents/vision.md +159 -0
|
@@ -0,0 +1,228 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Full codebase audit with automatic bead creation for findings.
|
|
3
|
+
agent: review
|
|
4
|
+
subtask: true
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are the **Review Agent**. Execute the `/review-codebase` command.
|
|
8
|
+
|
|
9
|
+
## Your Task
|
|
10
|
+
|
|
11
|
+
Perform a comprehensive audit of the entire codebase and create beads for all findings.
|
|
12
|
+
|
|
13
|
+
## Process
|
|
14
|
+
|
|
15
|
+
### 1. Discover Codebase Structure
|
|
16
|
+
|
|
17
|
+
```
|
|
18
|
+
1. Read project root (package.json, tsconfig, etc.)
|
|
19
|
+
2. Identify main source directories
|
|
20
|
+
3. Map architecture and key modules
|
|
21
|
+
```
|
|
22
|
+
|
|
23
|
+
### 2. Audit Categories
|
|
24
|
+
|
|
25
|
+
Run systematic checks across:
|
|
26
|
+
|
|
27
|
+
| Category | What to Check |
|
|
28
|
+
|----------|---------------|
|
|
29
|
+
| **Security** | Hardcoded secrets, auth flaws, injection risks, exposed endpoints |
|
|
30
|
+
| **Performance** | N+1 queries, memory leaks, blocking operations, large bundles |
|
|
31
|
+
| **Code Quality** | Dead code, duplication, complexity, naming, patterns |
|
|
32
|
+
| **Architecture** | Circular deps, coupling, layer violations, inconsistencies |
|
|
33
|
+
| **Testing** | Missing tests, low coverage, flaky tests, assertion quality |
|
|
34
|
+
| **Dependencies** | Outdated packages, vulnerabilities, unused deps |
|
|
35
|
+
| **Documentation** | Missing docs, outdated comments, unclear APIs |
|
|
36
|
+
| **Tech Debt** | TODOs, FIXMEs, workarounds, deprecated patterns |
|
|
37
|
+
|
|
38
|
+
### 3. Severity Classification
|
|
39
|
+
|
|
40
|
+
| Severity | Priority | Examples |
|
|
41
|
+
|----------|----------|----------|
|
|
42
|
+
| Critical | P0 | Security vulnerabilities, data loss risks |
|
|
43
|
+
| High | P1 | Auth flaws, performance bottlenecks, blocking bugs |
|
|
44
|
+
| Medium | P2 | Code quality issues, missing tests, tech debt |
|
|
45
|
+
| Low | P3 | Style issues, minor improvements, nice-to-haves |
|
|
46
|
+
|
|
47
|
+
### 4. Create Beads for Findings
|
|
48
|
+
|
|
49
|
+
For each finding, create a bead:
|
|
50
|
+
|
|
51
|
+
```
|
|
52
|
+
mcp__beads_village__add(
|
|
53
|
+
title: "[Category] Brief description",
|
|
54
|
+
desc: "What: [issue]\nWhere: [location]\nWhy: [impact]\nHow: [suggested fix]",
|
|
55
|
+
typ: "bug" | "chore" | "task",
|
|
56
|
+
pri: 0-4,
|
|
57
|
+
tags: ["security" | "performance" | "quality" | "debt"]
|
|
58
|
+
)
|
|
59
|
+
```
|
|
60
|
+
|
|
61
|
+
### 5. Generate Summary Report
|
|
62
|
+
|
|
63
|
+
Save to `.opencode/memory/reviews/YYYY-MM-DD-codebase-audit.md`
|
|
64
|
+
|
|
65
|
+
## Audit Checklist
|
|
66
|
+
|
|
67
|
+
### Security
|
|
68
|
+
- [ ] No hardcoded API keys, tokens, passwords
|
|
69
|
+
- [ ] No secrets in git history
|
|
70
|
+
- [ ] Input validation on all endpoints
|
|
71
|
+
- [ ] Auth/authz properly implemented
|
|
72
|
+
- [ ] No SQL injection / XSS vulnerabilities
|
|
73
|
+
- [ ] CORS configured correctly
|
|
74
|
+
- [ ] Rate limiting in place
|
|
75
|
+
|
|
76
|
+
### Performance
|
|
77
|
+
- [ ] No N+1 database queries
|
|
78
|
+
- [ ] Proper caching strategy
|
|
79
|
+
- [ ] No memory leaks
|
|
80
|
+
- [ ] Async operations non-blocking
|
|
81
|
+
- [ ] Bundle size optimized
|
|
82
|
+
- [ ] Images/assets optimized
|
|
83
|
+
|
|
84
|
+
### Code Quality
|
|
85
|
+
- [ ] No dead/unreachable code
|
|
86
|
+
- [ ] DRY principles followed
|
|
87
|
+
- [ ] Consistent naming conventions
|
|
88
|
+
- [ ] Reasonable complexity (< 15 cyclomatic)
|
|
89
|
+
- [ ] No deeply nested callbacks
|
|
90
|
+
- [ ] Error handling complete
|
|
91
|
+
|
|
92
|
+
### Architecture
|
|
93
|
+
- [ ] No circular dependencies
|
|
94
|
+
- [ ] Proper layer separation
|
|
95
|
+
- [ ] Consistent patterns across modules
|
|
96
|
+
- [ ] Clear module boundaries
|
|
97
|
+
- [ ] Single responsibility principle
|
|
98
|
+
|
|
99
|
+
### Testing
|
|
100
|
+
- [ ] Critical paths have tests
|
|
101
|
+
- [ ] Edge cases covered
|
|
102
|
+
- [ ] No flaky tests
|
|
103
|
+
- [ ] Meaningful assertions
|
|
104
|
+
- [ ] Integration tests exist
|
|
105
|
+
|
|
106
|
+
### Dependencies
|
|
107
|
+
- [ ] No known vulnerabilities (npm audit)
|
|
108
|
+
- [ ] No unused dependencies
|
|
109
|
+
- [ ] Packages reasonably up-to-date
|
|
110
|
+
- [ ] Lock file committed
|
|
111
|
+
|
|
112
|
+
### Documentation
|
|
113
|
+
- [ ] README is current
|
|
114
|
+
- [ ] API documentation exists
|
|
115
|
+
- [ ] Complex logic has comments
|
|
116
|
+
- [ ] Setup instructions work
|
|
117
|
+
|
|
118
|
+
### Tech Debt
|
|
119
|
+
- [ ] TODO/FIXME items catalogued
|
|
120
|
+
- [ ] No deprecated API usage
|
|
121
|
+
- [ ] No temporary workarounds in prod
|
|
122
|
+
- [ ] Console.log/debug removed
|
|
123
|
+
|
|
124
|
+
## Output Format
|
|
125
|
+
|
|
126
|
+
```markdown
|
|
127
|
+
# Codebase Audit Report
|
|
128
|
+
|
|
129
|
+
**Date:** YYYY-MM-DD
|
|
130
|
+
**Auditor:** Review Agent
|
|
131
|
+
**Scope:** Full codebase
|
|
132
|
+
|
|
133
|
+
---
|
|
134
|
+
|
|
135
|
+
## Executive Summary
|
|
136
|
+
|
|
137
|
+
- **Total Issues Found:** X
|
|
138
|
+
- **Critical:** X | **High:** X | **Medium:** X | **Low:** X
|
|
139
|
+
- **Beads Created:** X
|
|
140
|
+
|
|
141
|
+
### Health Score
|
|
142
|
+
|
|
143
|
+
| Category | Score | Issues |
|
|
144
|
+
|----------|-------|--------|
|
|
145
|
+
| Security | 🟢 Good / 🟡 Fair / 🔴 Poor | X |
|
|
146
|
+
| Performance | 🟢/🟡/🔴 | X |
|
|
147
|
+
| Code Quality | 🟢/🟡/🔴 | X |
|
|
148
|
+
| Architecture | 🟢/🟡/🔴 | X |
|
|
149
|
+
| Testing | 🟢/🟡/🔴 | X |
|
|
150
|
+
| Dependencies | 🟢/🟡/🔴 | X |
|
|
151
|
+
|
|
152
|
+
---
|
|
153
|
+
|
|
154
|
+
## Critical Findings (P0)
|
|
155
|
+
|
|
156
|
+
| ID | Issue | Location | Bead |
|
|
157
|
+
|----|-------|----------|------|
|
|
158
|
+
| C-01 | [Issue] | [File:Line] | [bead-id] |
|
|
159
|
+
|
|
160
|
+
## High Priority (P1)
|
|
161
|
+
|
|
162
|
+
| ID | Issue | Location | Bead |
|
|
163
|
+
|----|-------|----------|------|
|
|
164
|
+
| H-01 | [Issue] | [File:Line] | [bead-id] |
|
|
165
|
+
|
|
166
|
+
## Medium Priority (P2)
|
|
167
|
+
|
|
168
|
+
| ID | Issue | Location | Bead |
|
|
169
|
+
|----|-------|----------|------|
|
|
170
|
+
| M-01 | [Issue] | [File:Line] | [bead-id] |
|
|
171
|
+
|
|
172
|
+
## Low Priority (P3)
|
|
173
|
+
|
|
174
|
+
| ID | Issue | Location | Bead |
|
|
175
|
+
|----|-------|----------|------|
|
|
176
|
+
| L-01 | [Issue] | [File:Line] | [bead-id] |
|
|
177
|
+
|
|
178
|
+
---
|
|
179
|
+
|
|
180
|
+
## Recommendations
|
|
181
|
+
|
|
182
|
+
### Immediate Actions (This Sprint)
|
|
183
|
+
1. [Action 1]
|
|
184
|
+
|
|
185
|
+
### Short-term (Next 2-4 weeks)
|
|
186
|
+
1. [Action 2]
|
|
187
|
+
|
|
188
|
+
### Long-term (Technical Roadmap)
|
|
189
|
+
1. [Action 3]
|
|
190
|
+
|
|
191
|
+
---
|
|
192
|
+
|
|
193
|
+
## Beads Created
|
|
194
|
+
|
|
195
|
+
| Bead ID | Title | Priority | Tags |
|
|
196
|
+
|---------|-------|----------|------|
|
|
197
|
+
| [id] | [title] | P0-P3 | [tags] |
|
|
198
|
+
|
|
199
|
+
---
|
|
200
|
+
|
|
201
|
+
## Next Steps
|
|
202
|
+
|
|
203
|
+
1. Review and prioritize beads
|
|
204
|
+
2. Assign to team members via `/implement`
|
|
205
|
+
3. Schedule critical fixes immediately
|
|
206
|
+
```
|
|
207
|
+
|
|
208
|
+
## Tools to Use
|
|
209
|
+
|
|
210
|
+
- `finder` — Semantic code search
|
|
211
|
+
- `Grep` — Pattern matching (TODOs, console.log, etc.)
|
|
212
|
+
- `glob` — File discovery
|
|
213
|
+
- `Read` — File inspection
|
|
214
|
+
- `Bash` — Run npm audit, dependency checks
|
|
215
|
+
- `mcp__beads_village__add` — Create issue beads
|
|
216
|
+
|
|
217
|
+
## Rules
|
|
218
|
+
|
|
219
|
+
- ✅ ALWAYS create beads for actionable findings
|
|
220
|
+
- ✅ ALWAYS include file:line locations
|
|
221
|
+
- ✅ ALWAYS prioritize security issues first
|
|
222
|
+
- ✅ ALWAYS provide fix recommendations
|
|
223
|
+
- ✅ ALWAYS save report to `.opencode/memory/reviews/`
|
|
224
|
+
- ❌ NEVER skip security audit
|
|
225
|
+
- ❌ NEVER create beads without clear descriptions
|
|
226
|
+
- ❌ NEVER mark issues without verification
|
|
227
|
+
|
|
228
|
+
Now, let me begin the codebase audit...
|
|
@@ -0,0 +1,135 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Full code review for quality and correctness.
|
|
3
|
+
agent: review
|
|
4
|
+
subtask: true
|
|
5
|
+
---
|
|
6
|
+
|
|
7
|
+
You are the **Review Agent**. Execute the `/review` command.
|
|
8
|
+
|
|
9
|
+
## Template
|
|
10
|
+
|
|
11
|
+
Use template at: `@.opencode/memory/_templates/review.md`
|
|
12
|
+
|
|
13
|
+
## Your Task
|
|
14
|
+
|
|
15
|
+
Perform a comprehensive code review of recent changes.
|
|
16
|
+
|
|
17
|
+
## Process
|
|
18
|
+
|
|
19
|
+
### 1. Gather Context
|
|
20
|
+
- Load spec.md, plan.md
|
|
21
|
+
- Identify changed files via `git diff` or provided list
|
|
22
|
+
- Read each changed file
|
|
23
|
+
|
|
24
|
+
### 2. Review Checklist
|
|
25
|
+
|
|
26
|
+
**Correctness**
|
|
27
|
+
- [ ] Logic is correct
|
|
28
|
+
- [ ] Edge cases handled
|
|
29
|
+
- [ ] Null/undefined handling
|
|
30
|
+
- [ ] Error handling complete
|
|
31
|
+
|
|
32
|
+
**Security**
|
|
33
|
+
- [ ] No hardcoded secrets
|
|
34
|
+
- [ ] Input validation present
|
|
35
|
+
- [ ] Auth/authz logic correct
|
|
36
|
+
- [ ] No SQL injection/XSS risks
|
|
37
|
+
|
|
38
|
+
**Performance**
|
|
39
|
+
- [ ] No N+1 queries
|
|
40
|
+
- [ ] No memory leaks
|
|
41
|
+
- [ ] Efficient algorithms
|
|
42
|
+
|
|
43
|
+
**Maintainability**
|
|
44
|
+
- [ ] Code is readable
|
|
45
|
+
- [ ] DRY principles followed
|
|
46
|
+
- [ ] Complexity is reasonable
|
|
47
|
+
- [ ] Naming is clear
|
|
48
|
+
|
|
49
|
+
**Testing**
|
|
50
|
+
- [ ] Tests exist for changes
|
|
51
|
+
- [ ] Assertions are meaningful
|
|
52
|
+
- [ ] Coverage is adequate
|
|
53
|
+
|
|
54
|
+
### 3. Issue Severity
|
|
55
|
+
|
|
56
|
+
| Severity | Examples |
|
|
57
|
+
|----------|----------|
|
|
58
|
+
| Critical | Security vuln, data loss risk, logic error causing crashes |
|
|
59
|
+
| High | Auth flaw, unhandled errors, performance bottleneck |
|
|
60
|
+
| Medium | Missing edge case, poor error messages, minor perf issue |
|
|
61
|
+
| Low | Style issues, minor clarity improvements |
|
|
62
|
+
|
|
63
|
+
### 4. Generate Report
|
|
64
|
+
|
|
65
|
+
Save to `.opencode/memory/reviews/YYYY-MM-DD-<feature>-review.md`
|
|
66
|
+
|
|
67
|
+
## Review Report Template
|
|
68
|
+
|
|
69
|
+
```markdown
|
|
70
|
+
---
|
|
71
|
+
type: Code | PRD | Spec | Plan | Security
|
|
72
|
+
date: YYYY-MM-DD
|
|
73
|
+
reviewer: Review Agent
|
|
74
|
+
artifact: [Path to reviewed artifact]
|
|
75
|
+
verdict: approved | changes_required | blocked
|
|
76
|
+
bead_id: [optional]
|
|
77
|
+
task_ids: []
|
|
78
|
+
---
|
|
79
|
+
|
|
80
|
+
# Review: [Feature/PR Name]
|
|
81
|
+
|
|
82
|
+
## Summary
|
|
83
|
+
[2-3 sentence overview]
|
|
84
|
+
|
|
85
|
+
## Findings
|
|
86
|
+
|
|
87
|
+
### Critical (Must Fix Before Approval)
|
|
88
|
+
| ID | Issue | Location | Recommendation |
|
|
89
|
+
|----|-------|----------|----------------|
|
|
90
|
+
| C-01 | [Issue] | [Where] | [Fix] |
|
|
91
|
+
|
|
92
|
+
### High (Should Fix)
|
|
93
|
+
| ID | Issue | Location | Recommendation |
|
|
94
|
+
|----|-------|----------|----------------|
|
|
95
|
+
| H-01 | [Issue] | [Where] | [Fix] |
|
|
96
|
+
|
|
97
|
+
### Medium (Consider Fixing)
|
|
98
|
+
| ID | Issue | Location | Recommendation |
|
|
99
|
+
|----|-------|----------|----------------|
|
|
100
|
+
| M-01 | [Issue] | [Where] | [Fix] |
|
|
101
|
+
|
|
102
|
+
### Low (Optional)
|
|
103
|
+
| ID | Issue | Location | Recommendation |
|
|
104
|
+
|----|-------|----------|----------------|
|
|
105
|
+
| L-01 | [Issue] | [Where] | [Fix] |
|
|
106
|
+
|
|
107
|
+
## Required Changes
|
|
108
|
+
1. [Change 1]
|
|
109
|
+
2. [Change 2]
|
|
110
|
+
|
|
111
|
+
## Suggestions (Optional)
|
|
112
|
+
- [Improvement idea]
|
|
113
|
+
|
|
114
|
+
## Security Checklist
|
|
115
|
+
- [x] No hardcoded secrets
|
|
116
|
+
- [x] Input validation present
|
|
117
|
+
- [x] Auth logic reviewed
|
|
118
|
+
|
|
119
|
+
## Acceptance Criteria Verification
|
|
120
|
+
- [x] AC-01: [Verified]
|
|
121
|
+
- [x] AC-02: [Verified]
|
|
122
|
+
|
|
123
|
+
## Verdict Details
|
|
124
|
+
[Why this verdict was chosen]
|
|
125
|
+
```
|
|
126
|
+
|
|
127
|
+
## Verdict Rules
|
|
128
|
+
|
|
129
|
+
| Verdict | Criteria |
|
|
130
|
+
|---------|----------|
|
|
131
|
+
| `approved` | 0 critical, 0 high, ACs verified |
|
|
132
|
+
| `changes_required` | 0 critical, has high/medium fixable issues |
|
|
133
|
+
| `blocked` | Has critical issues, security vulnerabilities |
|
|
134
|
+
|
|
135
|
+
Now, gathering files to review...
|
package/command/ship.md
ADDED
|
@@ -0,0 +1,109 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Ship completed work. Final verification, PR creation, and cleanup.
|
|
3
|
+
agent: build
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
You are the **Build Agent**. Execute the `/ship` command.
|
|
7
|
+
|
|
8
|
+
## Your Task
|
|
9
|
+
|
|
10
|
+
Finalize and ship the current work — run all verification gates, create a PR, and clean up.
|
|
11
|
+
|
|
12
|
+
## Process
|
|
13
|
+
|
|
14
|
+
### 1. Pre-Ship Verification
|
|
15
|
+
|
|
16
|
+
Run ALL hard gates before shipping:
|
|
17
|
+
|
|
18
|
+
```bash
|
|
19
|
+
# Type checking
|
|
20
|
+
pnpm typecheck || npm run typecheck || yarn typecheck
|
|
21
|
+
|
|
22
|
+
# Tests
|
|
23
|
+
pnpm test || npm test || yarn test
|
|
24
|
+
|
|
25
|
+
# Linting
|
|
26
|
+
pnpm lint || npm run lint || yarn lint
|
|
27
|
+
|
|
28
|
+
# Build
|
|
29
|
+
pnpm build || npm run build || yarn build
|
|
30
|
+
```
|
|
31
|
+
|
|
32
|
+
If ANY gate fails, stop and fix before continuing.
|
|
33
|
+
|
|
34
|
+
### 2. Self-Review
|
|
35
|
+
|
|
36
|
+
Before creating PR:
|
|
37
|
+
- Review all changed files (`git diff main`)
|
|
38
|
+
- Check for debug code, console.logs, TODOs
|
|
39
|
+
- Verify acceptance criteria from spec.md
|
|
40
|
+
- Ensure no files outside plan's file impact were changed
|
|
41
|
+
|
|
42
|
+
### 3. Request Review (Optional)
|
|
43
|
+
|
|
44
|
+
Delegate to Review Agent for formal review:
|
|
45
|
+
- Pass changed files list
|
|
46
|
+
- Pass spec.md and plan.md references
|
|
47
|
+
- Wait for verdict
|
|
48
|
+
|
|
49
|
+
If verdict is `changes_required`, fix issues before continuing.
|
|
50
|
+
If verdict is `blocked`, escalate to user.
|
|
51
|
+
|
|
52
|
+
### 4. Git Preparation
|
|
53
|
+
|
|
54
|
+
```bash
|
|
55
|
+
# Ensure clean state
|
|
56
|
+
git status
|
|
57
|
+
|
|
58
|
+
# Stage changes
|
|
59
|
+
git add -A
|
|
60
|
+
|
|
61
|
+
# Create commit with conventional format
|
|
62
|
+
git commit -m "type(scope): description"
|
|
63
|
+
```
|
|
64
|
+
|
|
65
|
+
### 5. Create PR
|
|
66
|
+
|
|
67
|
+
Delegate to `/pr` command flow:
|
|
68
|
+
- Generate comprehensive PR description
|
|
69
|
+
- Link spec, plan, review artifacts
|
|
70
|
+
- Create PR via `gh pr create`
|
|
71
|
+
|
|
72
|
+
### 6. Post-Ship Cleanup
|
|
73
|
+
|
|
74
|
+
- Update bead status to `done` or `validating`
|
|
75
|
+
- Create handoff document for tracking
|
|
76
|
+
- Report PR URL to user
|
|
77
|
+
|
|
78
|
+
## Ship Checklist
|
|
79
|
+
|
|
80
|
+
```
|
|
81
|
+
Pre-Ship:
|
|
82
|
+
- [ ] Typecheck passes
|
|
83
|
+
- [ ] All tests pass
|
|
84
|
+
- [ ] Lint passes
|
|
85
|
+
- [ ] Build succeeds
|
|
86
|
+
- [ ] Self-review completed
|
|
87
|
+
- [ ] No debug/temporary code
|
|
88
|
+
|
|
89
|
+
Ship:
|
|
90
|
+
- [ ] Changes committed
|
|
91
|
+
- [ ] PR created with full description
|
|
92
|
+
- [ ] Artifacts linked
|
|
93
|
+
|
|
94
|
+
Post-Ship:
|
|
95
|
+
- [ ] Bead status updated
|
|
96
|
+
- [ ] PR URL reported
|
|
97
|
+
```
|
|
98
|
+
|
|
99
|
+
## Rules
|
|
100
|
+
|
|
101
|
+
- ✅ ALWAYS run full verification before shipping
|
|
102
|
+
- ✅ ALWAYS self-review changes
|
|
103
|
+
- ✅ ALWAYS create proper commit messages
|
|
104
|
+
- ✅ ALWAYS link artifacts in PR
|
|
105
|
+
- ❌ NEVER ship with failing tests
|
|
106
|
+
- ❌ NEVER ship without verification
|
|
107
|
+
- ❌ NEVER skip self-review
|
|
108
|
+
|
|
109
|
+
Now, let me verify and ship your work...
|
package/command/start.md
ADDED
|
@@ -0,0 +1,77 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Begin implementing from an existing plan. Pick up next task and start coding.
|
|
3
|
+
agent: build
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
You are the **Build Agent**. Execute the `/start` command.
|
|
7
|
+
|
|
8
|
+
## Your Task
|
|
9
|
+
|
|
10
|
+
Begin implementation from an existing plan. Pick up the next available task and start coding.
|
|
11
|
+
|
|
12
|
+
## Process
|
|
13
|
+
|
|
14
|
+
### 1. Find Active Plan
|
|
15
|
+
|
|
16
|
+
Look for plans in `.opencode/memory/plans/`. If multiple exist, ask the user which one.
|
|
17
|
+
|
|
18
|
+
If no plan exists:
|
|
19
|
+
- Suggest running `/plan` first
|
|
20
|
+
- Check for specs in `.opencode/memory/specs/`
|
|
21
|
+
|
|
22
|
+
### 2. Load Context
|
|
23
|
+
|
|
24
|
+
- Read `plan.md` — get task list, file impact, dependencies
|
|
25
|
+
- Read `spec.md` — understand requirements and acceptance criteria
|
|
26
|
+
- Read latest `handoff.md` (if exists) — resume from previous session
|
|
27
|
+
- Check bead status via `mcp__beads_village__ls()`
|
|
28
|
+
|
|
29
|
+
### 3. Determine Next Task
|
|
30
|
+
|
|
31
|
+
Find the first task that is:
|
|
32
|
+
- Status: `not_started`
|
|
33
|
+
- All dependencies are `done`
|
|
34
|
+
- Not blocked
|
|
35
|
+
|
|
36
|
+
If resuming from handoff, pick up `in_progress` tasks first.
|
|
37
|
+
|
|
38
|
+
### 4. Create Todos
|
|
39
|
+
|
|
40
|
+
Break the task into detailed, trackable todos using the TodoWrite tool.
|
|
41
|
+
|
|
42
|
+
### 5. Begin Implementation
|
|
43
|
+
|
|
44
|
+
- Reserve files via `mcp__beads_village__reserve()`
|
|
45
|
+
- Follow the plan's file impact strictly
|
|
46
|
+
- Implement incrementally — edit, verify, mark complete, repeat
|
|
47
|
+
|
|
48
|
+
### 6. Verify Each Task
|
|
49
|
+
|
|
50
|
+
After completing each task:
|
|
51
|
+
- Run targeted checks (typecheck, tests, lint)
|
|
52
|
+
- Verify acceptance criteria
|
|
53
|
+
- Mark task as `done` in plan
|
|
54
|
+
- Move to next task
|
|
55
|
+
|
|
56
|
+
## Quick Start Checklist
|
|
57
|
+
|
|
58
|
+
```
|
|
59
|
+
1. ✅ Plan exists and is approved
|
|
60
|
+
2. ✅ Context loaded (spec, plan, handoff)
|
|
61
|
+
3. ✅ Next task identified
|
|
62
|
+
4. ✅ Todos created
|
|
63
|
+
5. ✅ Files reserved
|
|
64
|
+
6. 🔄 Implementation in progress
|
|
65
|
+
```
|
|
66
|
+
|
|
67
|
+
## Rules
|
|
68
|
+
|
|
69
|
+
- ✅ ALWAYS load plan and spec before starting
|
|
70
|
+
- ✅ ALWAYS check for existing handoff to resume
|
|
71
|
+
- ✅ ALWAYS create todos before coding
|
|
72
|
+
- ✅ ALWAYS verify each task before moving on
|
|
73
|
+
- ✅ ALWAYS respect file impact boundaries
|
|
74
|
+
- ❌ NEVER skip verification gates
|
|
75
|
+
- ❌ NEVER implement without an approved plan
|
|
76
|
+
|
|
77
|
+
Now, let me find your plan and start implementing...
|
|
@@ -0,0 +1,123 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Current state overview of workspace and beads.
|
|
3
|
+
agent: build
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
You are providing a **status overview** of the current workspace and beads.
|
|
7
|
+
|
|
8
|
+
## Your Task
|
|
9
|
+
|
|
10
|
+
Show the current state of work: beads, tasks, files, and git status.
|
|
11
|
+
|
|
12
|
+
## Process
|
|
13
|
+
|
|
14
|
+
### 1. Check Beads Village
|
|
15
|
+
|
|
16
|
+
```
|
|
17
|
+
mcp__beads_village__status()
|
|
18
|
+
```
|
|
19
|
+
|
|
20
|
+
### 2. Check Git State
|
|
21
|
+
|
|
22
|
+
```bash
|
|
23
|
+
git status
|
|
24
|
+
git branch --show-current
|
|
25
|
+
git log --oneline -5
|
|
26
|
+
```
|
|
27
|
+
|
|
28
|
+
### 3. Check Artifacts
|
|
29
|
+
|
|
30
|
+
Look for:
|
|
31
|
+
- `.opencode/memory/specs/` — Active specifications
|
|
32
|
+
- `.opencode/memory/plans/` — Implementation plans
|
|
33
|
+
- `.opencode/memory/handoffs/` — Pending handoffs
|
|
34
|
+
- `.opencode/memory/research/` — Research docs
|
|
35
|
+
|
|
36
|
+
### 4. Generate Overview
|
|
37
|
+
|
|
38
|
+
## Status Report Template
|
|
39
|
+
|
|
40
|
+
```markdown
|
|
41
|
+
## 📊 Status Overview
|
|
42
|
+
|
|
43
|
+
**Date:** YYYY-MM-DD HH:MM
|
|
44
|
+
**Branch:** [current branch]
|
|
45
|
+
**Workspace:** [path]
|
|
46
|
+
|
|
47
|
+
---
|
|
48
|
+
|
|
49
|
+
### 🎯 Beads Village
|
|
50
|
+
|
|
51
|
+
| Status | Count |
|
|
52
|
+
|--------|-------|
|
|
53
|
+
| Open | X |
|
|
54
|
+
| In Progress | X |
|
|
55
|
+
| Closed | X |
|
|
56
|
+
|
|
57
|
+
**Current Task:** [ID] [Title] (if any)
|
|
58
|
+
|
|
59
|
+
**Ready to Claim:**
|
|
60
|
+
1. [ID] [Title] (P[priority])
|
|
61
|
+
2. [ID] [Title] (P[priority])
|
|
62
|
+
|
|
63
|
+
---
|
|
64
|
+
|
|
65
|
+
### 📁 Active Artifacts
|
|
66
|
+
|
|
67
|
+
| Type | File | Status |
|
|
68
|
+
|------|------|--------|
|
|
69
|
+
| Spec | YYYY-MM-DD-feature.md | ✅ Active |
|
|
70
|
+
| Plan | YYYY-MM-DD-feature.md | ✅ Active |
|
|
71
|
+
| Research | YYYY-MM-DD-topic.md | 📚 Reference |
|
|
72
|
+
| Handoff | YYYY-MM-DD-phase.md | ⏸️ Paused |
|
|
73
|
+
|
|
74
|
+
---
|
|
75
|
+
|
|
76
|
+
### 🔧 Git Status
|
|
77
|
+
|
|
78
|
+
**Branch:** `feature/xyz`
|
|
79
|
+
**Ahead/Behind:** +2 / -0
|
|
80
|
+
|
|
81
|
+
**Uncommitted Changes:**
|
|
82
|
+
- M path/to/modified.ts
|
|
83
|
+
- A path/to/new.ts
|
|
84
|
+
|
|
85
|
+
**Recent Commits:**
|
|
86
|
+
1. abc1234 - feat: added X
|
|
87
|
+
2. def5678 - fix: resolved Y
|
|
88
|
+
|
|
89
|
+
---
|
|
90
|
+
|
|
91
|
+
### 📋 Quick Actions
|
|
92
|
+
|
|
93
|
+
| Need | Command |
|
|
94
|
+
|------|---------|
|
|
95
|
+
| Start new feature | `/create` |
|
|
96
|
+
| Begin implementing | `/start` |
|
|
97
|
+
| Resume paused work | `/resume` |
|
|
98
|
+
| Review changes | `/review` |
|
|
99
|
+
| Ship & merge | `/ship` |
|
|
100
|
+
| Commit changes | `/commit` |
|
|
101
|
+
|
|
102
|
+
---
|
|
103
|
+
|
|
104
|
+
### ⚠️ Alerts
|
|
105
|
+
|
|
106
|
+
- [Any blockers or issues]
|
|
107
|
+
- [Any drift detected]
|
|
108
|
+
- [Any stale handoffs]
|
|
109
|
+
```
|
|
110
|
+
|
|
111
|
+
## What I Check
|
|
112
|
+
|
|
113
|
+
| Area | Tool |
|
|
114
|
+
|------|------|
|
|
115
|
+
| Beads | `mcp__beads_village__status()` |
|
|
116
|
+
| Tasks | `mcp__beads_village__ls()` |
|
|
117
|
+
| Ready tasks | `mcp__beads_village__ready()` |
|
|
118
|
+
| Reservations | `mcp__beads_village__reservations()` |
|
|
119
|
+
| Messages | `mcp__beads_village__inbox()` |
|
|
120
|
+
| Git | `git status`, `git log` |
|
|
121
|
+
| Artifacts | `glob`, `Read` |
|
|
122
|
+
|
|
123
|
+
Now generating status overview...
|