@anionzo/skill 1.4.0 → 1.6.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/CONTRIBUTING.md +2 -1
- package/README.md +21 -9
- package/docs/design-brief.md +19 -13
- package/i18n/CONTRIBUTING.vi.md +2 -1
- package/i18n/README.vi.md +21 -9
- package/i18n/design-brief.vi.md +19 -13
- package/knowledge/global/skill-triggering-rules.md +2 -1
- package/package.json +1 -1
- package/skills/brainstorming/SKILL.md +176 -13
- package/skills/brainstorming/meta.yaml +19 -10
- package/skills/code-review/SKILL.md +214 -19
- package/skills/code-review/meta.yaml +21 -9
- package/skills/commit/SKILL.md +187 -0
- package/skills/commit/examples.md +62 -0
- package/skills/commit/meta.yaml +30 -0
- package/skills/commit/references/output-template.md +14 -0
- package/skills/debug/SKILL.md +252 -0
- package/skills/debug/examples.md +83 -0
- package/skills/debug/meta.yaml +38 -0
- package/skills/debug/references/output-template.md +16 -0
- package/skills/docs-writer/SKILL.md +85 -10
- package/skills/docs-writer/meta.yaml +16 -12
- package/skills/extract/SKILL.md +161 -0
- package/skills/extract/examples.md +47 -0
- package/skills/extract/meta.yaml +27 -0
- package/skills/extract/references/output-template.md +24 -0
- package/skills/feature-delivery/SKILL.md +10 -5
- package/skills/feature-delivery/meta.yaml +5 -0
- package/skills/go-pipeline/SKILL.md +156 -0
- package/skills/go-pipeline/examples.md +56 -0
- package/skills/go-pipeline/meta.yaml +27 -0
- package/skills/go-pipeline/references/output-template.md +17 -0
- package/skills/planning/SKILL.md +128 -17
- package/skills/planning/meta.yaml +15 -6
- package/skills/refactor-safe/SKILL.md +10 -7
- package/skills/repo-onboarding/SKILL.md +11 -7
- package/skills/repo-onboarding/meta.yaml +2 -0
- package/skills/research/SKILL.md +100 -0
- package/skills/research/examples.md +79 -0
- package/skills/research/meta.yaml +27 -0
- package/skills/research/references/output-template.md +23 -0
- package/skills/test-driven-development/SKILL.md +194 -0
- package/skills/test-driven-development/examples.md +77 -0
- package/skills/test-driven-development/meta.yaml +31 -0
- package/skills/test-driven-development/references/.gitkeep +0 -0
- package/skills/test-driven-development/references/output-template.md +31 -0
- package/skills/using-skills/SKILL.md +32 -14
- package/skills/using-skills/examples.md +3 -3
- package/skills/using-skills/meta.yaml +8 -3
- package/skills/verification-before-completion/SKILL.md +127 -13
- package/skills/verification-before-completion/meta.yaml +24 -14
- package/templates/SKILL.md +8 -1
- package/skills/bug-triage/SKILL.md +0 -47
- package/skills/bug-triage/examples.md +0 -68
- package/skills/bug-triage/meta.yaml +0 -25
- package/skills/bug-triage/references/output-template.md +0 -26
|
@@ -0,0 +1,161 @@
|
|
|
1
|
+
# Extract
|
|
2
|
+
|
|
3
|
+
## Purpose
|
|
4
|
+
|
|
5
|
+
Extract reusable patterns, significant decisions, and failure learnings from completed work into documentation that compounds knowledge over time.
|
|
6
|
+
|
|
7
|
+
This skill exists to turn individual task outcomes into organizational memory — not just code patterns, but also the decisions and mistakes that are expensive to repeat.
|
|
8
|
+
|
|
9
|
+
## When To Use
|
|
10
|
+
|
|
11
|
+
Load this skill when:
|
|
12
|
+
|
|
13
|
+
- a task or feature is complete and had notable learnings
|
|
14
|
+
- a debugging session revealed a pattern worth remembering
|
|
15
|
+
- a decision was made that future work should know about
|
|
16
|
+
- a mistake was made that should not be repeated
|
|
17
|
+
- the user says "extract learnings", "document what we learned", or "capture this pattern"
|
|
18
|
+
|
|
19
|
+
Skip this skill when work was routine with no surprises or novel patterns.
|
|
20
|
+
|
|
21
|
+
## Workflow
|
|
22
|
+
|
|
23
|
+
1. Identify the source of knowledge (completed task, code change, debugging session).
|
|
24
|
+
2. Analyze across three categories: patterns, decisions, and failures.
|
|
25
|
+
3. Check for existing documentation that should be updated (avoid duplicates).
|
|
26
|
+
4. Create or update knowledge documentation.
|
|
27
|
+
5. Promote critical learnings that would save significant time if known in advance.
|
|
28
|
+
|
|
29
|
+
## Three-Category Analysis
|
|
30
|
+
|
|
31
|
+
### Patterns
|
|
32
|
+
|
|
33
|
+
Reusable approaches worth standardizing:
|
|
34
|
+
|
|
35
|
+
- **Code patterns** — new utilities, abstractions, integration techniques
|
|
36
|
+
- **Architecture patterns** — structural decisions that worked
|
|
37
|
+
- **Process patterns** — workflow approaches that saved time
|
|
38
|
+
|
|
39
|
+
For each pattern, document:
|
|
40
|
+
- What it is
|
|
41
|
+
- When to use it
|
|
42
|
+
- A concrete example
|
|
43
|
+
- Where it was first used
|
|
44
|
+
|
|
45
|
+
### Decisions
|
|
46
|
+
|
|
47
|
+
Significant choices and their outcomes:
|
|
48
|
+
|
|
49
|
+
- **GOOD_CALL** — decisions that proved correct or saved time
|
|
50
|
+
- **BAD_CALL** — decisions that required rework
|
|
51
|
+
- **SURPRISE** — things that turned out differently than expected
|
|
52
|
+
- **TRADEOFF** — conscious choices where alternatives were considered
|
|
53
|
+
|
|
54
|
+
For each decision, document:
|
|
55
|
+
- What was chosen
|
|
56
|
+
- What was rejected
|
|
57
|
+
- How it played out
|
|
58
|
+
- Recommendation for future work
|
|
59
|
+
|
|
60
|
+
### Failures
|
|
61
|
+
|
|
62
|
+
Mistakes and wasted effort worth preventing:
|
|
63
|
+
|
|
64
|
+
- Bugs introduced and their root causes
|
|
65
|
+
- Wrong assumptions that required backtracking
|
|
66
|
+
- Missing prerequisites discovered mid-execution
|
|
67
|
+
- Test gaps that allowed regressions
|
|
68
|
+
|
|
69
|
+
For each failure, document:
|
|
70
|
+
- What went wrong
|
|
71
|
+
- Root cause (not just symptom)
|
|
72
|
+
- Time cost estimate
|
|
73
|
+
- How to prevent it
|
|
74
|
+
|
|
75
|
+
## Knowledge Document Template
|
|
76
|
+
|
|
77
|
+
```markdown
|
|
78
|
+
## Patterns
|
|
79
|
+
|
|
80
|
+
### [Pattern Name]
|
|
81
|
+
- **What:** [description]
|
|
82
|
+
- **When to use:** [applicable conditions]
|
|
83
|
+
- **Example:** [concrete code or workflow example]
|
|
84
|
+
- **Source:** [task or feature reference]
|
|
85
|
+
|
|
86
|
+
## Decisions
|
|
87
|
+
|
|
88
|
+
### [Decision]
|
|
89
|
+
- **Chose:** [what was chosen]
|
|
90
|
+
- **Over:** [what was rejected]
|
|
91
|
+
- **Tag:** GOOD_CALL / BAD_CALL / SURPRISE / TRADEOFF
|
|
92
|
+
- **Outcome:** [how it played out]
|
|
93
|
+
- **Recommendation:** [for future work]
|
|
94
|
+
|
|
95
|
+
## Failures
|
|
96
|
+
|
|
97
|
+
### [Failure]
|
|
98
|
+
- **What went wrong:** [description]
|
|
99
|
+
- **Root cause:** [not just symptom]
|
|
100
|
+
- **Time lost:** [estimate]
|
|
101
|
+
- **Prevention:** [what to do differently]
|
|
102
|
+
```
|
|
103
|
+
|
|
104
|
+
## Extraction Rules
|
|
105
|
+
|
|
106
|
+
- Extract patterns, decisions, AND failures — not just code patterns.
|
|
107
|
+
- Search for existing docs before creating new ones — update instead of duplicating.
|
|
108
|
+
- Link extracted knowledge back to its source.
|
|
109
|
+
- Only document genuinely reusable knowledge — do not fabricate findings.
|
|
110
|
+
- A short learning with 2 genuine entries is better than a long doc with invented ones.
|
|
111
|
+
|
|
112
|
+
## Critical Learning Promotion
|
|
113
|
+
|
|
114
|
+
For findings that meet ALL criteria:
|
|
115
|
+
- Affects more than one future feature
|
|
116
|
+
- Would cause 30+ minutes wasted effort if unknown
|
|
117
|
+
- Is generalizable, not implementation-specific
|
|
118
|
+
|
|
119
|
+
These should be marked as critical and placed where they will be seen at the start of future work sessions.
|
|
120
|
+
|
|
121
|
+
**Calibration:** Do NOT promote everything. If critical learnings grow past 20-30 entries, they become noise. Only promote learnings that would have saved 30+ minutes if known in advance.
|
|
122
|
+
|
|
123
|
+
## Output Format
|
|
124
|
+
|
|
125
|
+
Present results using the Shared Output Contract:
|
|
126
|
+
|
|
127
|
+
1. **Goal/Result** — what knowledge was extracted, updated, or intentionally not extracted
|
|
128
|
+
2. **Key Details:**
|
|
129
|
+
- what was extracted (patterns, decisions, failures)
|
|
130
|
+
- whether docs were created or updated
|
|
131
|
+
- whether critical learnings were promoted
|
|
132
|
+
- where the canonical knowledge now lives
|
|
133
|
+
3. **Next Action** — only when findings lead somewhere:
|
|
134
|
+
- extracted from completed task → `commit` if changes pending
|
|
135
|
+
- no clear handoff → stop after the result
|
|
136
|
+
|
|
137
|
+
## No-Op Case
|
|
138
|
+
|
|
139
|
+
If the work was too routine to generalize, say so explicitly and do not force a new document. Do NOT fabricate findings. If the task ran smoothly with no surprises, write that.
|
|
140
|
+
|
|
141
|
+
## Red Flags
|
|
142
|
+
|
|
143
|
+
- only extracting code patterns, ignoring decisions and failures
|
|
144
|
+
- promoting everything as critical (noise kills the learning loop)
|
|
145
|
+
- writing generic learnings like "test more carefully" (worthless)
|
|
146
|
+
- fabricating findings when the task was straightforward
|
|
147
|
+
- not checking existing docs before creating duplicates
|
|
148
|
+
- extracting implementation-specific details that will never be reused
|
|
149
|
+
|
|
150
|
+
## Checklist
|
|
151
|
+
|
|
152
|
+
- [ ] Three categories analyzed (patterns, decisions, failures)
|
|
153
|
+
- [ ] Existing docs checked for duplicates
|
|
154
|
+
- [ ] Knowledge is genuinely generalizable
|
|
155
|
+
- [ ] Includes concrete examples (for patterns)
|
|
156
|
+
- [ ] Links back to source
|
|
157
|
+
- [ ] Critical learnings promoted (if applicable)
|
|
158
|
+
|
|
159
|
+
## Done Criteria
|
|
160
|
+
|
|
161
|
+
This skill is complete when the three-category analysis is done and any findings worth preserving are documented in the appropriate location. If nothing worth extracting was found, an explicit "no-op" statement is the valid completion.
|
|
@@ -0,0 +1,47 @@
|
|
|
1
|
+
# Examples
|
|
2
|
+
|
|
3
|
+
## Example 1
|
|
4
|
+
|
|
5
|
+
User request:
|
|
6
|
+
|
|
7
|
+
```text
|
|
8
|
+
We just finished the auth refactor. Extract what we learned.
|
|
9
|
+
```
|
|
10
|
+
|
|
11
|
+
Expected result:
|
|
12
|
+
|
|
13
|
+
- analyze the completed work across three categories
|
|
14
|
+
- document patterns, decisions, and failures
|
|
15
|
+
- promote critical learnings if applicable
|
|
16
|
+
|
|
17
|
+
Sample completed output:
|
|
18
|
+
|
|
19
|
+
```
|
|
20
|
+
Goal/Result: Extracted 2 patterns, 1 decision, and 1 failure from the auth refactor.
|
|
21
|
+
|
|
22
|
+
Key Details:
|
|
23
|
+
Patterns:
|
|
24
|
+
1. Provider Registry Pattern — centralized auth provider registration
|
|
25
|
+
in src/auth/registry.ts. New providers implement AuthProvider
|
|
26
|
+
interface and register in one place. Reusable for any plugin system.
|
|
27
|
+
2. Token Rotation Pattern — refresh tokens are rotated on each use
|
|
28
|
+
(one-time use). Prevents replay attacks.
|
|
29
|
+
|
|
30
|
+
Decisions:
|
|
31
|
+
1. TRADEOFF: Chose passport.js over custom auth middleware.
|
|
32
|
+
- Pros: ecosystem support, community maintenance
|
|
33
|
+
- Cons: callback-heavy API, harder to test
|
|
34
|
+
- Outcome: Worked well for OAuth but added friction for custom flows
|
|
35
|
+
- Recommendation: Use passport for OAuth, custom middleware for API keys
|
|
36
|
+
|
|
37
|
+
Failures:
|
|
38
|
+
1. Assumed session store was thread-safe — it was not.
|
|
39
|
+
- Root cause: Redis client was shared without connection pooling
|
|
40
|
+
- Time lost: ~4 hours debugging intermittent auth failures
|
|
41
|
+
- Prevention: Always verify concurrent access patterns for shared state
|
|
42
|
+
|
|
43
|
+
Critical learning promoted: Redis connection pooling requirement.
|
|
44
|
+
(Would have saved 4+ hours if known in advance.)
|
|
45
|
+
|
|
46
|
+
Next Action: No follow-up needed — knowledge documented.
|
|
47
|
+
```
|
|
@@ -0,0 +1,27 @@
|
|
|
1
|
+
name: extract
|
|
2
|
+
version: 0.1.0
|
|
3
|
+
category: knowledge
|
|
4
|
+
summary: Extract reusable patterns, decisions, and failure learnings from completed work to compound knowledge.
|
|
5
|
+
summary_vi: "Tr\xEDch xu\u1EA5t pattern, quy\u1EBFt \u0111\u1ECBnh, v\xE0 b\xE0i h\u1ECDc t\u1EEB c\xF4ng vi\u1EC7c ho\xE0n th\xE0nh \u0111\u1EC3 t\xEDch l\u0169y ki\u1EBFn th\u1EE9c."
|
|
6
|
+
triggers:
|
|
7
|
+
- extract learnings from this task
|
|
8
|
+
- document what we learned
|
|
9
|
+
- capture this pattern
|
|
10
|
+
- what did we learn
|
|
11
|
+
inputs:
|
|
12
|
+
- completed task or feature
|
|
13
|
+
- code changes or debugging session
|
|
14
|
+
outputs:
|
|
15
|
+
- patterns documented
|
|
16
|
+
- decisions recorded
|
|
17
|
+
- failures captured
|
|
18
|
+
- critical learnings promoted
|
|
19
|
+
constraints:
|
|
20
|
+
- search existing docs before creating new ones
|
|
21
|
+
- do not fabricate findings
|
|
22
|
+
- three-category analysis required
|
|
23
|
+
related_skills:
|
|
24
|
+
- using-skills
|
|
25
|
+
- commit
|
|
26
|
+
- debug
|
|
27
|
+
- research
|
|
@@ -0,0 +1,24 @@
|
|
|
1
|
+
## Goal/Result
|
|
2
|
+
|
|
3
|
+
- Knowledge extracted:
|
|
4
|
+
|
|
5
|
+
## Key Details
|
|
6
|
+
|
|
7
|
+
### Patterns
|
|
8
|
+
- Pattern 1:
|
|
9
|
+
|
|
10
|
+
### Decisions
|
|
11
|
+
- Decision 1:
|
|
12
|
+
|
|
13
|
+
### Failures
|
|
14
|
+
- Failure 1:
|
|
15
|
+
|
|
16
|
+
### Critical Learnings Promoted
|
|
17
|
+
-
|
|
18
|
+
|
|
19
|
+
### Docs Created/Updated
|
|
20
|
+
-
|
|
21
|
+
|
|
22
|
+
## Next Action
|
|
23
|
+
|
|
24
|
+
- Recommended follow-up:
|
|
@@ -19,11 +19,16 @@ Load this skill when the user wants code changed to add or update behavior.
|
|
|
19
19
|
|
|
20
20
|
## Output Format
|
|
21
21
|
|
|
22
|
-
|
|
23
|
-
|
|
24
|
-
|
|
25
|
-
|
|
26
|
-
-
|
|
22
|
+
Present results using the Shared Output Contract:
|
|
23
|
+
|
|
24
|
+
1. **Goal/Result** — the feature implemented and its status
|
|
25
|
+
2. **Key Details:**
|
|
26
|
+
- goal and scope
|
|
27
|
+
- chosen approach
|
|
28
|
+
- files changed
|
|
29
|
+
- verification performed
|
|
30
|
+
- follow-up or risk note
|
|
31
|
+
3. **Next Action** — `verification-before-completion` to confirm, then `code-review` or `commit`
|
|
27
32
|
|
|
28
33
|
## Red Flags
|
|
29
34
|
|
|
@@ -20,7 +20,12 @@ constraints:
|
|
|
20
20
|
- ask only one blocking question when necessary
|
|
21
21
|
related_skills:
|
|
22
22
|
- using-skills
|
|
23
|
+
- brainstorming
|
|
23
24
|
- planning
|
|
24
25
|
- repo-onboarding
|
|
26
|
+
- debug
|
|
25
27
|
- verification-before-completion
|
|
26
28
|
- code-review
|
|
29
|
+
- docs-writer
|
|
30
|
+
- go-pipeline
|
|
31
|
+
- test-driven-development
|
|
@@ -0,0 +1,156 @@
|
|
|
1
|
+
# Go Pipeline
|
|
2
|
+
|
|
3
|
+
## Purpose
|
|
4
|
+
|
|
5
|
+
Execute a full development pipeline from an approved spec: break into tasks, plan each, implement each, verify all, and commit — in one continuous run with minimal review gates.
|
|
6
|
+
|
|
7
|
+
This skill exists for when the spec is approved and the user wants execution, not incremental review at each step.
|
|
8
|
+
|
|
9
|
+
## When To Use
|
|
10
|
+
|
|
11
|
+
Load this skill when:
|
|
12
|
+
|
|
13
|
+
- the user has an approved spec and wants to execute everything in one shot
|
|
14
|
+
- the user says "run all", "go mode", "execute everything", or "just build it"
|
|
15
|
+
- all requirements are clear and decisions are locked
|
|
16
|
+
|
|
17
|
+
## When NOT To Use
|
|
18
|
+
|
|
19
|
+
- Spec is still a draft — redirect to `brainstorming` first.
|
|
20
|
+
- User wants to review each task individually — use `planning` + `feature-delivery` per task.
|
|
21
|
+
- Spec has unresolved open questions — resolve them first.
|
|
22
|
+
- The scope is too large for one context window — use `planning` + `feature-delivery` incrementally.
|
|
23
|
+
|
|
24
|
+
## Workflow
|
|
25
|
+
|
|
26
|
+
### Phase 1: Validate Spec
|
|
27
|
+
|
|
28
|
+
Before starting, verify the spec is ready:
|
|
29
|
+
|
|
30
|
+
- [ ] Spec exists and is approved
|
|
31
|
+
- [ ] Has acceptance criteria defined
|
|
32
|
+
- [ ] No unresolved blocking questions
|
|
33
|
+
- [ ] Requirements are specific enough to implement
|
|
34
|
+
|
|
35
|
+
If any check fails, STOP and redirect:
|
|
36
|
+
> "Spec not ready. [specific issue]. Run `brainstorming` first."
|
|
37
|
+
|
|
38
|
+
### Phase 2: Generate Tasks
|
|
39
|
+
|
|
40
|
+
Parse the spec and break it into ordered implementation tasks:
|
|
41
|
+
|
|
42
|
+
- Group related requirements into logical tasks
|
|
43
|
+
- Order by dependency (foundational first, dependent last)
|
|
44
|
+
- Each task should be completable in a single implementation session
|
|
45
|
+
- Map each task to the spec acceptance criteria it fulfills
|
|
46
|
+
|
|
47
|
+
**Report:** "Created X tasks from spec. Starting implementation..."
|
|
48
|
+
|
|
49
|
+
### Phase 3: Plan + Implement Each Task
|
|
50
|
+
|
|
51
|
+
Loop through all tasks in dependency order:
|
|
52
|
+
|
|
53
|
+
For each task:
|
|
54
|
+
1. **Research context** — check related code, patterns, docs
|
|
55
|
+
2. **Draft plan** — concrete steps, files to change, tests to add
|
|
56
|
+
3. **Implement** — work through plan steps
|
|
57
|
+
4. **Verify** — run tests/lint/build after each task
|
|
58
|
+
5. **Mark complete** — note what was done
|
|
59
|
+
|
|
60
|
+
**Progress report between tasks:**
|
|
61
|
+
> "Task X/Y done: [title]. Continuing..."
|
|
62
|
+
|
|
63
|
+
### Phase 4: Full Verification
|
|
64
|
+
|
|
65
|
+
After all tasks complete:
|
|
66
|
+
|
|
67
|
+
1. Run the full test suite
|
|
68
|
+
2. Run the build
|
|
69
|
+
3. Run linting
|
|
70
|
+
4. Check acceptance criteria coverage:
|
|
71
|
+
|
|
72
|
+
```
|
|
73
|
+
Coverage Report
|
|
74
|
+
===============
|
|
75
|
+
Spec: [name]
|
|
76
|
+
Tasks: X/X complete (100%)
|
|
77
|
+
ACs: Y/Z verified
|
|
78
|
+
```
|
|
79
|
+
|
|
80
|
+
If coverage < 100%, identify and address gaps.
|
|
81
|
+
|
|
82
|
+
### Phase 5: Commit
|
|
83
|
+
|
|
84
|
+
Stage all changes and prepare a single conventional commit:
|
|
85
|
+
|
|
86
|
+
```
|
|
87
|
+
feat(<scope>): implement <spec-name>
|
|
88
|
+
|
|
89
|
+
- Task 1: <title>
|
|
90
|
+
- Task 2: <title>
|
|
91
|
+
- All ACs verified
|
|
92
|
+
```
|
|
93
|
+
|
|
94
|
+
**This is the ONE gate in go-pipeline — ask user before committing:**
|
|
95
|
+
|
|
96
|
+
> Pipeline complete. X tasks done, all verified.
|
|
97
|
+
> Ready to commit. Proceed? (yes / no / edit)
|
|
98
|
+
|
|
99
|
+
## Context Budget Management
|
|
100
|
+
|
|
101
|
+
If context exceeds approximately 60% during implementation:
|
|
102
|
+
|
|
103
|
+
1. Finish the current task
|
|
104
|
+
2. Commit completed work so far
|
|
105
|
+
3. Report progress and remaining tasks
|
|
106
|
+
4. Suggest: "Run `go-pipeline` again to continue remaining tasks."
|
|
107
|
+
|
|
108
|
+
The skill should detect already-completed tasks and skip them on re-run.
|
|
109
|
+
|
|
110
|
+
## Error Handling
|
|
111
|
+
|
|
112
|
+
- **Build/test fails during a task:** Fix the error, re-run tests. If unfixable, mark task as blocked, note why, continue to next task.
|
|
113
|
+
- **Spec has conflicting requirements:** STOP and ask user to clarify.
|
|
114
|
+
- **Task depends on blocked task:** Skip and report at the end.
|
|
115
|
+
|
|
116
|
+
## Output Format
|
|
117
|
+
|
|
118
|
+
Present results using the Shared Output Contract:
|
|
119
|
+
|
|
120
|
+
1. **Goal/Result** — what was completed across the full pipeline run
|
|
121
|
+
2. **Key Details:**
|
|
122
|
+
- total tasks created and completed
|
|
123
|
+
- any blocked or skipped tasks
|
|
124
|
+
- acceptance criteria coverage percentage
|
|
125
|
+
- build/test/lint status
|
|
126
|
+
- commit proposal
|
|
127
|
+
3. **Next Action:**
|
|
128
|
+
- if complete → commit confirmation
|
|
129
|
+
- if interrupted → how to resume
|
|
130
|
+
- if blocked → what to resolve
|
|
131
|
+
|
|
132
|
+
## Red Flags
|
|
133
|
+
|
|
134
|
+
- running on a draft or unapproved spec
|
|
135
|
+
- skipping verification between tasks
|
|
136
|
+
- not checking acceptance criteria before marking tasks done
|
|
137
|
+
- committing without user approval
|
|
138
|
+
- ignoring build/test failures
|
|
139
|
+
- not reporting progress between tasks
|
|
140
|
+
- continuing past context budget limit without checkpointing
|
|
141
|
+
- treating this as a shortcut to skip proper spec work
|
|
142
|
+
|
|
143
|
+
## Checklist
|
|
144
|
+
|
|
145
|
+
- [ ] Spec is approved
|
|
146
|
+
- [ ] Spec validated (requirements clear, ACs defined)
|
|
147
|
+
- [ ] Tasks generated with AC mapping
|
|
148
|
+
- [ ] Each task: planned, implemented, verified, completed
|
|
149
|
+
- [ ] Full verification passed (tests, build, lint)
|
|
150
|
+
- [ ] AC coverage reported
|
|
151
|
+
- [ ] User approved commit
|
|
152
|
+
- [ ] Commit created
|
|
153
|
+
|
|
154
|
+
## Done Criteria
|
|
155
|
+
|
|
156
|
+
This skill is complete when all tasks from the spec are implemented, verified, and committed with user approval — or when progress is checkpointed and the user is informed how to resume.
|
|
@@ -0,0 +1,56 @@
|
|
|
1
|
+
# Examples
|
|
2
|
+
|
|
3
|
+
## Example 1
|
|
4
|
+
|
|
5
|
+
User request:
|
|
6
|
+
|
|
7
|
+
```text
|
|
8
|
+
The user-notifications spec is approved. Just build it all.
|
|
9
|
+
```
|
|
10
|
+
|
|
11
|
+
Expected result:
|
|
12
|
+
|
|
13
|
+
- validate the spec
|
|
14
|
+
- generate tasks from requirements
|
|
15
|
+
- plan and implement each task
|
|
16
|
+
- verify all ACs
|
|
17
|
+
- present commit for approval
|
|
18
|
+
|
|
19
|
+
Sample completed output:
|
|
20
|
+
|
|
21
|
+
```
|
|
22
|
+
Goal/Result: Pipeline complete — 4 tasks implemented, all ACs verified.
|
|
23
|
+
|
|
24
|
+
Key Details:
|
|
25
|
+
Spec: user-notifications (approved)
|
|
26
|
+
|
|
27
|
+
Tasks completed: 4/4
|
|
28
|
+
1. Add notification model and database migration — DONE
|
|
29
|
+
2. Implement notification service with email + in-app delivery — DONE
|
|
30
|
+
3. Add user notification preferences API — DONE
|
|
31
|
+
4. Wire notification triggers into existing user events — DONE
|
|
32
|
+
|
|
33
|
+
Tasks blocked: 0
|
|
34
|
+
|
|
35
|
+
AC Coverage: 5/5 (100%)
|
|
36
|
+
- AC-1: Notifications created on user events — VERIFIED
|
|
37
|
+
- AC-2: Email delivery via SendGrid — VERIFIED
|
|
38
|
+
- AC-3: In-app notifications visible in UI — VERIFIED
|
|
39
|
+
- AC-4: User preferences respected — VERIFIED
|
|
40
|
+
- AC-5: Delivery failures logged — VERIFIED
|
|
41
|
+
|
|
42
|
+
Build: PASS
|
|
43
|
+
Tests: 58/58 PASS (12 new)
|
|
44
|
+
Lint: PASS
|
|
45
|
+
|
|
46
|
+
Commit proposal:
|
|
47
|
+
feat(notifications): implement user notification system
|
|
48
|
+
|
|
49
|
+
- Add notification model with email + in-app channels
|
|
50
|
+
- Implement notification service with delivery tracking
|
|
51
|
+
- Add user preferences API for per-channel control
|
|
52
|
+
- Wire notifications into user registration and order events
|
|
53
|
+
- All 5 acceptance criteria verified
|
|
54
|
+
|
|
55
|
+
Proceed? (yes / no / edit)
|
|
56
|
+
```
|
|
@@ -0,0 +1,27 @@
|
|
|
1
|
+
name: go-pipeline
|
|
2
|
+
version: 0.1.0
|
|
3
|
+
category: workflow
|
|
4
|
+
summary: Execute a full spec-to-commit pipeline in one continuous run with minimal review gates.
|
|
5
|
+
summary_vi: "Th\u1EF1c thi to\xE0n b\u1ED9 pipeline t\u1EEB spec \u0111\u1EBFn commit trong m\u1ED9t l\u01B0\u1EE3t li\xEAn t\u1EE5c v\u1EDBi \xEDt gate review."
|
|
6
|
+
triggers:
|
|
7
|
+
- go mode
|
|
8
|
+
- run all
|
|
9
|
+
- execute everything
|
|
10
|
+
- just build it
|
|
11
|
+
inputs:
|
|
12
|
+
- approved spec document
|
|
13
|
+
outputs:
|
|
14
|
+
- implemented tasks
|
|
15
|
+
- verification results
|
|
16
|
+
- conventional commit
|
|
17
|
+
constraints:
|
|
18
|
+
- spec must be approved before starting
|
|
19
|
+
- commit requires user approval
|
|
20
|
+
- checkpoint on context budget
|
|
21
|
+
related_skills:
|
|
22
|
+
- using-skills
|
|
23
|
+
- brainstorming
|
|
24
|
+
- planning
|
|
25
|
+
- feature-delivery
|
|
26
|
+
- verification-before-completion
|
|
27
|
+
- commit
|