bigpowers 2.2.0 → 2.4.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/.pi/package.json +16 -0
- package/.pi/prompts/assess-impact.md +76 -0
- package/.pi/prompts/audit-code.md +156 -0
- package/.pi/prompts/build-epic.md +44 -0
- package/.pi/prompts/change-request.md +105 -0
- package/.pi/prompts/commit-message.md +135 -0
- package/.pi/prompts/compose-workflow.md +40 -0
- package/.pi/prompts/craft-skill.md +150 -0
- package/.pi/prompts/deepen-architecture.md +235 -0
- package/.pi/prompts/define-language.md +79 -0
- package/.pi/prompts/define-success.md +62 -0
- package/.pi/prompts/delegate-task.md +76 -0
- package/.pi/prompts/design-interface.md +96 -0
- package/.pi/prompts/develop-tdd.md +375 -0
- package/.pi/prompts/diagnose-root.md +23 -0
- package/.pi/prompts/dispatch-agents.md +83 -0
- package/.pi/prompts/edit-document.md +22 -0
- package/.pi/prompts/elaborate-spec.md +81 -0
- package/.pi/prompts/enforce-first.md +77 -0
- package/.pi/prompts/evolve-skill.md +38 -0
- package/.pi/prompts/execute-plan.md +54 -0
- package/.pi/prompts/fix-bug.md +36 -0
- package/.pi/prompts/grill-me.md +95 -0
- package/.pi/prompts/grill-with-docs.md +37 -0
- package/.pi/prompts/guard-git.md +212 -0
- package/.pi/prompts/hook-commits.md +93 -0
- package/.pi/prompts/inspect-quality.md +105 -0
- package/.pi/prompts/investigate-bug.md +117 -0
- package/.pi/prompts/kickoff-branch.md +99 -0
- package/.pi/prompts/map-codebase.md +70 -0
- package/.pi/prompts/migrate-spec.md +482 -0
- package/.pi/prompts/model-domain.md +227 -0
- package/.pi/prompts/orchestrate-project.md +161 -0
- package/.pi/prompts/organize-workspace.md +159 -0
- package/.pi/prompts/plan-refactor.md +77 -0
- package/.pi/prompts/plan-release.md +145 -0
- package/.pi/prompts/plan-work.md +161 -0
- package/.pi/prompts/release-branch.md +158 -0
- package/.pi/prompts/request-review.md +70 -0
- package/.pi/prompts/research-first.md +62 -0
- package/.pi/prompts/reset-baseline.md +20 -0
- package/.pi/prompts/respond-review.md +70 -0
- package/.pi/prompts/run-evals.md +56 -0
- package/.pi/prompts/run-planning.md +26 -0
- package/.pi/prompts/scope-work.md +23 -0
- package/.pi/prompts/search-skills.md +21 -0
- package/.pi/prompts/seed-conventions.md +132 -0
- package/.pi/prompts/session-state.md +146 -0
- package/.pi/prompts/setup-environment.md +23 -0
- package/.pi/prompts/simulate-agents.md +25 -0
- package/.pi/prompts/slice-tasks.md +23 -0
- package/.pi/prompts/spike-prototype.md +94 -0
- package/.pi/prompts/stocktake-skills.md +40 -0
- package/.pi/prompts/survey-context.md +129 -0
- package/.pi/prompts/terse-mode.md +37 -0
- package/.pi/prompts/trace-requirement.md +68 -0
- package/.pi/prompts/using-bigpowers.md +105 -0
- package/.pi/prompts/validate-fix.md +98 -0
- package/.pi/prompts/verify-work.md +125 -0
- package/.pi/prompts/visual-dashboard.md +51 -0
- package/.pi/prompts/wire-observability.md +92 -0
- package/.pi/prompts/write-document.md +244 -0
- package/.pi/skills/assess-impact/SKILL.md +77 -0
- package/.pi/skills/audit-code/SKILL.md +157 -0
- package/.pi/skills/build-epic/SKILL.md +45 -0
- package/.pi/skills/change-request/SKILL.md +106 -0
- package/.pi/skills/commit-message/SKILL.md +136 -0
- package/.pi/skills/compose-workflow/SKILL.md +41 -0
- package/.pi/skills/craft-skill/SKILL.md +151 -0
- package/.pi/skills/deepen-architecture/SKILL.md +236 -0
- package/.pi/skills/define-language/SKILL.md +80 -0
- package/.pi/skills/define-success/SKILL.md +63 -0
- package/.pi/skills/delegate-task/SKILL.md +77 -0
- package/.pi/skills/design-interface/SKILL.md +97 -0
- package/.pi/skills/develop-tdd/SKILL.md +376 -0
- package/.pi/skills/diagnose-root/SKILL.md +24 -0
- package/.pi/skills/dispatch-agents/SKILL.md +84 -0
- package/.pi/skills/edit-document/SKILL.md +23 -0
- package/.pi/skills/elaborate-spec/SKILL.md +82 -0
- package/.pi/skills/enforce-first/SKILL.md +78 -0
- package/.pi/skills/evolve-skill/SKILL.md +39 -0
- package/.pi/skills/execute-plan/SKILL.md +55 -0
- package/.pi/skills/fix-bug/SKILL.md +37 -0
- package/.pi/skills/grill-me/SKILL.md +96 -0
- package/.pi/skills/grill-with-docs/SKILL.md +38 -0
- package/.pi/skills/guard-git/SKILL.md +213 -0
- package/.pi/skills/hook-commits/SKILL.md +94 -0
- package/.pi/skills/inspect-quality/SKILL.md +106 -0
- package/.pi/skills/investigate-bug/SKILL.md +118 -0
- package/.pi/skills/kickoff-branch/SKILL.md +100 -0
- package/.pi/skills/map-codebase/SKILL.md +71 -0
- package/.pi/skills/migrate-spec/SKILL.md +483 -0
- package/.pi/skills/model-domain/SKILL.md +228 -0
- package/.pi/skills/orchestrate-project/SKILL.md +162 -0
- package/.pi/skills/organize-workspace/SKILL.md +160 -0
- package/.pi/skills/plan-refactor/SKILL.md +78 -0
- package/.pi/skills/plan-release/SKILL.md +146 -0
- package/.pi/skills/plan-work/SKILL.md +162 -0
- package/.pi/skills/release-branch/SKILL.md +159 -0
- package/.pi/skills/request-review/SKILL.md +71 -0
- package/.pi/skills/research-first/SKILL.md +63 -0
- package/.pi/skills/reset-baseline/SKILL.md +21 -0
- package/.pi/skills/respond-review/SKILL.md +71 -0
- package/.pi/skills/run-evals/SKILL.md +57 -0
- package/.pi/skills/run-planning/SKILL.md +27 -0
- package/.pi/skills/scope-work/SKILL.md +24 -0
- package/.pi/skills/search-skills/SKILL.md +22 -0
- package/.pi/skills/seed-conventions/SKILL.md +133 -0
- package/.pi/skills/session-state/SKILL.md +147 -0
- package/.pi/skills/setup-environment/SKILL.md +24 -0
- package/.pi/skills/simulate-agents/SKILL.md +26 -0
- package/.pi/skills/slice-tasks/SKILL.md +24 -0
- package/.pi/skills/spike-prototype/SKILL.md +95 -0
- package/.pi/skills/stocktake-skills/SKILL.md +41 -0
- package/.pi/skills/survey-context/SKILL.md +130 -0
- package/.pi/skills/terse-mode/SKILL.md +38 -0
- package/.pi/skills/trace-requirement/SKILL.md +69 -0
- package/.pi/skills/using-bigpowers/SKILL.md +106 -0
- package/.pi/skills/validate-fix/SKILL.md +99 -0
- package/.pi/skills/verify-work/SKILL.md +126 -0
- package/.pi/skills/visual-dashboard/SKILL.md +52 -0
- package/.pi/skills/wire-observability/SKILL.md +93 -0
- package/.pi/skills/write-document/SKILL.md +245 -0
- package/CHANGELOG.md +14 -0
- package/README.md +27 -1
- package/deepen-architecture/SKILL.md +2 -0
- package/define-language/SKILL.md +2 -0
- package/diagnose-root/SKILL.md +2 -0
- package/edit-document/SKILL.md +2 -0
- package/fix-bug/SKILL.md +3 -1
- package/grill-me/SKILL.md +3 -1
- package/grill-with-docs/SKILL.md +3 -1
- package/investigate-bug/SKILL.md +5 -11
- package/map-codebase/SKILL.md +3 -1
- package/model-domain/SKILL.md +2 -0
- package/package.json +11 -2
- package/plan-release/SKILL.md +1 -1
- package/plan-work/SKILL.md +3 -1
- package/release-branch/SKILL.md +4 -2
- package/run-planning/SKILL.md +3 -2
- package/scope-work/SKILL.md +3 -1
- package/scripts/sync-skills.sh +48 -3
- package/scripts/validate-doctrine.sh +24 -9
- package/seed-conventions/SKILL.md +2 -0
- package/slice-tasks/SKILL.md +3 -1
- package/survey-context/SKILL.md +3 -1
- package/write-document/SKILL.md +2 -0
package/.pi/package.json
ADDED
|
@@ -0,0 +1,16 @@
|
|
|
1
|
+
{
|
|
2
|
+
"name": "bigpowers",
|
|
3
|
+
"version": "2.4.0",
|
|
4
|
+
"description": "61 skills — 61 agent skills for spec-driven, test-first software development by solo developers",
|
|
5
|
+
"keywords": [
|
|
6
|
+
"pi-package"
|
|
7
|
+
],
|
|
8
|
+
"pi": {
|
|
9
|
+
"skills": [
|
|
10
|
+
"./skills"
|
|
11
|
+
],
|
|
12
|
+
"prompts": [
|
|
13
|
+
"./prompts"
|
|
14
|
+
]
|
|
15
|
+
}
|
|
16
|
+
}
|
|
@@ -0,0 +1,76 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Analyze the blast radius of a proposed change before any code is written. Maps dependents, affected stories, and test coverage. Produces specs/IMPACT.md. Use before plan-work on any non-trivial change, when touching a shared module, or when the user asks "what does this break?".
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Assess Impact
|
|
7
|
+
|
|
8
|
+
> **HARD GATE** — Run this skill before `plan-work` whenever a change touches an existing module, symbol, or file used by more than one caller. Skip only for net-new code with no existing dependents.
|
|
9
|
+
|
|
10
|
+
Find the blast radius of the proposed change before a single line is written.
|
|
11
|
+
|
|
12
|
+
## Process
|
|
13
|
+
|
|
14
|
+
### 1. Identify the target
|
|
15
|
+
|
|
16
|
+
Name the symbol, module, or file being changed. If the user hasn't specified, ask one question: "What exactly are you changing?"
|
|
17
|
+
|
|
18
|
+
### 2. Find dependents
|
|
19
|
+
|
|
20
|
+
```
|
|
21
|
+
grep -rn "[symbol-name]" . --include="*.ts" | grep -v node_modules
|
|
22
|
+
git log --oneline -10 -- [file-path]
|
|
23
|
+
```
|
|
24
|
+
|
|
25
|
+
→ verify: `grep -rn "[target]" . | wc -l`
|
|
26
|
+
|
|
27
|
+
### 3. Map to release plan stories
|
|
28
|
+
|
|
29
|
+
Read `specs/release-plan.yaml + epic capsule directories` (if it exists). For each dependent found in Step 2, identify which story owns that module. List stories that will be affected by the change.
|
|
30
|
+
|
|
31
|
+
→ verify: `grep -c "Story" specs/release-plan.yaml + epic capsule directories 2>/dev/null || echo "no release plan"`
|
|
32
|
+
|
|
33
|
+
### 4. List test coverage
|
|
34
|
+
|
|
35
|
+
Find tests that exercise the target:
|
|
36
|
+
|
|
37
|
+
```
|
|
38
|
+
grep -rn "[symbol-name]" . --include="*.test.*" --include="*.spec.*"
|
|
39
|
+
```
|
|
40
|
+
|
|
41
|
+
→ verify: `grep -rn "[target]" . --include="*.test.*" | wc -l`
|
|
42
|
+
|
|
43
|
+
### 5. Classify risk
|
|
44
|
+
|
|
45
|
+
| Level | Condition |
|
|
46
|
+
|-------|-----------|
|
|
47
|
+
| Low | ≤ 2 callers, all covered by tests |
|
|
48
|
+
| Medium | 3–10 callers, partial test coverage |
|
|
49
|
+
| High | > 10 callers, or shared API/interface, or no tests |
|
|
50
|
+
|
|
51
|
+
### 6. Write specs/IMPACT.md
|
|
52
|
+
|
|
53
|
+
```
|
|
54
|
+
## Target
|
|
55
|
+
[symbol or file being changed]
|
|
56
|
+
|
|
57
|
+
## Dependents ([count])
|
|
58
|
+
- [file]: [caller or usage]
|
|
59
|
+
|
|
60
|
+
## Affected Stories
|
|
61
|
+
- Story [X.Y]: [title]
|
|
62
|
+
|
|
63
|
+
## Test Coverage
|
|
64
|
+
- [test file]: covers [scenario]
|
|
65
|
+
- Gap: [untested behavior]
|
|
66
|
+
|
|
67
|
+
## Risk: Low / Medium / High
|
|
68
|
+
[One-sentence rationale]
|
|
69
|
+
|
|
70
|
+
## Recommended action
|
|
71
|
+
[Proceed / Add tests first / Discuss design]
|
|
72
|
+
```
|
|
73
|
+
|
|
74
|
+
→ verify: `grep "Risk:" specs/IMPACT.md`
|
|
75
|
+
|
|
76
|
+
Suggest `plan-work` once risk is understood and any test gaps are noted.
|
|
@@ -0,0 +1,156 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Self-review checklist for the coding agent to run before dispatching a reviewer. Checks CONVENTIONS.md compliance, Boy Scout Rule, test coverage, types, and SOLID. Produces a pass/fail checklist. Use before request-review, before committing, or when user asks for a code quality check.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Audit Code
|
|
7
|
+
> **HARD GATE** — **HARD GATE** — Audit must check for: bugs (correctness), security, performance, and clarity. Do NOT skip security review if the code touches user data, auth, or external APIs.
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
Run this self-review before asking anyone else to look at the code. The goal is to catch everything that is clearly wrong or missing — so the reviewer can focus on design and architecture, not hygiene.
|
|
11
|
+
|
|
12
|
+
**Distinct from `request-review`:** This is the coding agent checking its own work. No second agent is involved. Run this first; run `request-review` after this passes.
|
|
13
|
+
|
|
14
|
+
## Modes
|
|
15
|
+
|
|
16
|
+
- Default: full checklist
|
|
17
|
+
- --quick: Run only Supply Chain and Test Coverage. Use for changes under 50 LOC.
|
|
18
|
+
|
|
19
|
+
## Checklist
|
|
20
|
+
|
|
21
|
+
### Supply Chain & Security
|
|
22
|
+
|
|
23
|
+
- [ ] slopcheck run for new dependencies; packages tagged in plan-work: `[OK]`, `[SUS]`, or `[SLOP]`
|
|
24
|
+
- [ ] No `[SLOP]` packages without documented human approval
|
|
25
|
+
- [ ] No secrets in diff (`sk-`, `ghp_`, `AKIA`, `.env` values) — see `guard-git` patterns
|
|
26
|
+
- [ ] OWASP Top 10 spot-check: injection, broken auth, sensitive data exposure, misconfiguration (see `docs/references/security-threats.md`)
|
|
27
|
+
|
|
28
|
+
### Provenance & Metadata
|
|
29
|
+
|
|
30
|
+
- [ ] New plan artefacts include `type:` and `context:` metadata
|
|
31
|
+
- [ ] Implementation steps reference ADR or commit SHA where decisions were made
|
|
32
|
+
|
|
33
|
+
### Law of Demeter
|
|
34
|
+
|
|
35
|
+
- [ ] No method chains through unrelated objects (e.g. `a.getB().getC().doX()`)
|
|
36
|
+
- [ ] Collaborators talk to immediate neighbors only; law violations need explicit justification
|
|
37
|
+
|
|
38
|
+
### CONVENTIONS.md Compliance
|
|
39
|
+
|
|
40
|
+
- [ ] All output files are in `specs/` (no docs written to project root)
|
|
41
|
+
- [ ] No `gh issue create` calls anywhere in new/modified skills or scripts
|
|
42
|
+
- [ ] `gh` used only for PRs and repo clone operations
|
|
43
|
+
- [ ] No GitHub REST API called directly (no curl/fetch to api.github.com)
|
|
44
|
+
|
|
45
|
+
### Scope
|
|
46
|
+
|
|
47
|
+
- [ ] Changes are limited to what was asked — nothing extra refactored or reorganized
|
|
48
|
+
- [ ] No speculative features added
|
|
49
|
+
- [ ] No files touched outside the stated scope
|
|
50
|
+
- [ ] Changes are surgical: only code strictly required for the task; no refactoring, reorganization, or cleanup outside task scope (Boy Scout Rule applied surgically, not broadly)
|
|
51
|
+
|
|
52
|
+
### Boy Scout Rule
|
|
53
|
+
|
|
54
|
+
- [ ] Every file I touched is cleaner than when I found it
|
|
55
|
+
- [ ] No dead code left behind
|
|
56
|
+
- [ ] No commented-out code blocks
|
|
57
|
+
|
|
58
|
+
### Types and Safety
|
|
59
|
+
|
|
60
|
+
- [ ] No `any` types introduced (TypeScript) or untyped public functions (Python/Go/etc.)
|
|
61
|
+
- [ ] No `@ts-ignore` or `// eslint-disable` added
|
|
62
|
+
- [ ] No `as unknown as X` casts that bypass type safety
|
|
63
|
+
|
|
64
|
+
### Test Coverage
|
|
65
|
+
|
|
66
|
+
- [ ] Every new function has at least one test
|
|
67
|
+
- [ ] Every bug fix has a regression test
|
|
68
|
+
- [ ] Tests verify behavior through public interfaces (not implementation details)
|
|
69
|
+
- [ ] Tests are F.I.R.S.T compliant (use `enforce-first` if unsure)
|
|
70
|
+
|
|
71
|
+
### SOLID and Heuristics
|
|
72
|
+
|
|
73
|
+
- [ ] Single Responsibility: no function or module doing two unrelated things
|
|
74
|
+
- [ ] Open/Closed: extended through interfaces, not by modifying stable code
|
|
75
|
+
- [ ] Dependency Inversion: dependencies injected, not imported globally where avoidable
|
|
76
|
+
- [ ] **Chapter 17 Heuristics**: Code is free of smells documented in `audit-code/HEURISTICS.md` (G, N, C, T)
|
|
77
|
+
|
|
78
|
+
### Code Style (CONVENTIONS.md)
|
|
79
|
+
|
|
80
|
+
- [ ] Functions: 4–20 lines; split if longer
|
|
81
|
+
- [ ] Functions: descend exactly one level of abstraction (The Stepdown Rule / G34)
|
|
82
|
+
- [ ] Files: under 300 lines (ideally 200–300)
|
|
83
|
+
- [ ] Names: specific and unique (grep returns < 5 hits for each name)
|
|
84
|
+
- [ ] No duplication — shared logic extracted (DRY / G5)
|
|
85
|
+
- [ ] Early returns over nested ifs; max 2 levels of indentation
|
|
86
|
+
- [ ] Conditionals: expressed as positives (G29)
|
|
87
|
+
- [ ] Comments explain WHY, not WHAT
|
|
88
|
+
|
|
89
|
+
### Agent Readability (Akita's Lens)
|
|
90
|
+
|
|
91
|
+
- [ ] Functions are small enough to fit in a standard context window (4–20 lines)
|
|
92
|
+
- [ ] Names are unique and specific enough to be `grep`-able (grep returns < 5 hits)
|
|
93
|
+
- [ ] Types are explicit (no `any`, no inferred return types for public APIs)
|
|
94
|
+
- [ ] Code avoids deep nesting (max 2 levels) and uses early returns
|
|
95
|
+
|
|
96
|
+
### Red Flags
|
|
97
|
+
|
|
98
|
+
Before reporting, name any rationalization you caught yourself making for skipping a checklist item. Silence is not acceptable — if you skipped an item, state the reason explicitly.
|
|
99
|
+
|
|
100
|
+
## Output
|
|
101
|
+
|
|
102
|
+
Report the checklist with ✓ / ✗ per item. For each ✗, describe what needs to be fixed.
|
|
103
|
+
|
|
104
|
+
If all items pass: suggest running `request-review` for an independent second opinion.
|
|
105
|
+
If any items fail: fix them before proceeding.
|
|
106
|
+
|
|
107
|
+
## Handoff
|
|
108
|
+
|
|
109
|
+
Gate: READY -> next: commit-message
|
|
110
|
+
Writes: state.yaml handoff.next_skill = commit-message
|
|
111
|
+
|
|
112
|
+
---
|
|
113
|
+
|
|
114
|
+
# Clean Code Heuristics (Chapter 17)
|
|
115
|
+
|
|
116
|
+
A summary of Robert C. Martin's catalogue of code smells and heuristics, used as the technical benchmark for `audit-code`.
|
|
117
|
+
|
|
118
|
+
## Comments (C)
|
|
119
|
+
- **C1: Inappropriate Information**: Comments should only hold technical notes. Metadata (author, change history) belongs in Git.
|
|
120
|
+
- **C2: Obsolete Comment**: Update or delete comments that are no longer accurate.
|
|
121
|
+
- **C3: Redundant Comment**: Don't describe code that adequately describes itself (e.g., `i++; // increment i`).
|
|
122
|
+
- **C4: Poorly Written Comment**: If you write a comment, spend time making it the best it can be.
|
|
123
|
+
- **C5: Commented-Out Code**: Delete it. Git remembers it.
|
|
124
|
+
|
|
125
|
+
## Environment (E)
|
|
126
|
+
- **E1: Build Requires More Than One Step**: Building should be a single trivial operation (e.g., `bash install.sh`).
|
|
127
|
+
- **E2: Tests Require More Than One Step**: Running all tests should be one simple command (e.g., `npm test`).
|
|
128
|
+
|
|
129
|
+
## Functions (F)
|
|
130
|
+
- **F1: Too Many Arguments**: 0 is ideal, 1-2 is fine, 3 requires special justification. Never > 3.
|
|
131
|
+
- **F2: Output Arguments**: Avoid them. If a function changes state, it should change the state of its owning object.
|
|
132
|
+
- **F3: Flag Arguments**: Boolean arguments are a smell that the function does > 1 thing.
|
|
133
|
+
- **F4: Dead Function**: Discard methods that are never called.
|
|
134
|
+
|
|
135
|
+
## General (G)
|
|
136
|
+
- **G1: Multiple Languages in One Source File**: Try to minimize the mixing of languages (e.g., HTML inside Java).
|
|
137
|
+
- **G5: Duplication (DRY)**: **The root of all evil.** Every time you see duplication, it's a missed opportunity for abstraction.
|
|
138
|
+
- **G6: Code at Wrong Level of Abstraction**: High-level concepts in base classes; low-level details in derivatives.
|
|
139
|
+
- **G25: Replace Magic Numbers with Named Constants**: No "naked" numbers or strings.
|
|
140
|
+
- **G28: Encapsulate Conditionals**: Prefer `if (shouldBePublished())` over complex boolean logic.
|
|
141
|
+
- **G29: Avoid Negative Conditionals**: Prefer `if (buffer.shouldCompact())` over `if (!buffer.shouldNotCompact())`.
|
|
142
|
+
- **G30: Functions Should Do One Thing**: If a function can be split into sections, it's doing too much.
|
|
143
|
+
- **G31: Hidden Temporal Couplings**: If execution order matters, make the dependency explicit via arguments.
|
|
144
|
+
- **G34: Functions Should Descend Only One Level of Abstraction**: The Stepdown Rule.
|
|
145
|
+
|
|
146
|
+
## Naming (N)
|
|
147
|
+
- **N1: Choose Descriptive Names**: Names should reveal intent and be updated as code evolves.
|
|
148
|
+
- **N4: Unambiguous Names**: Names should make the working of a function/variable clear.
|
|
149
|
+
- **N7: Names Should Describe Side-Effects**: Describe everything the function is or does.
|
|
150
|
+
|
|
151
|
+
## Tests (T)
|
|
152
|
+
- **T1: Insufficient Tests**: A test suite should test everything that could possibly break.
|
|
153
|
+
- **T4: An Ignored Test Is a Question about an Ambiguity**: Document the reason for `@Ignore`.
|
|
154
|
+
- **T5: Test Boundary Conditions**: Most bugs happen at the boundaries; test them exhaustively.
|
|
155
|
+
- **T8: Test Coverage Patterns Can Be Revealing**: Analyze what code is *not* executed to find gaps.
|
|
156
|
+
- **T9: Tests Should Be Fast**: Slow tests don't get run.
|
|
@@ -0,0 +1,44 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Eight-step epic build cycle — reads state.yaml, execution-status.yaml, and one epic capsule; updates status via bp-yaml-set or direct edit. Resume mode runs one step per invocation. Use instead of ad-hoc execute-plan for release work.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Build Epic
|
|
7
|
+
|
|
8
|
+
Scope: one story. Called by orchestrate-project Phase 4. Not a replacement for orchestrate-project.
|
|
9
|
+
|
|
10
|
+
Orchestrates the **build** flow for a single epic: survey → plan tasks → kickoff → TDD → verify → audit → commit → release.
|
|
11
|
+
|
|
12
|
+
> **HARD GATE** — Set `specs/state.yaml` `active_flow: build_epic` and `active_epic: eNN` before starting.
|
|
13
|
+
>
|
|
14
|
+
> **HARD GATE** — Not on `main`/`master` before step 3 (kickoff-branch).
|
|
15
|
+
|
|
16
|
+
## Eight steps (`epic_cycle` in state.yaml)
|
|
17
|
+
|
|
18
|
+
| Step | Skill / action |
|
|
19
|
+
|------|----------------|
|
|
20
|
+
| 1 | `survey-context` — confirm epic + story |
|
|
21
|
+
| 2 | `plan-work` — flesh out story `tasks[]` in `specs/epics/eNN-slug/epic.yaml` |
|
|
22
|
+
| 3 | `kickoff-branch` — feature branch + clean baseline |
|
|
23
|
+
| 4 | `develop-tdd` — red-green per task |
|
|
24
|
+
| 5 | `verify-work` — UAT + mechanical gates |
|
|
25
|
+
| 6 | `audit-code` — self-review checklist |
|
|
26
|
+
| 7 | `commit-message` — Conventional Commits draft |
|
|
27
|
+
| 8 | `release-branch` — PR or solo land |
|
|
28
|
+
|
|
29
|
+
## Process
|
|
30
|
+
|
|
31
|
+
1. Read `specs/state.yaml`, `specs/execution-status.yaml`, `specs/release-plan.yaml`, active `specs/epics/eNN-slug/epic.yaml`.
|
|
32
|
+
2. **BCP Tracking (Step 2):** After `plan-work` completes, read the `bcps:` count (Business Complexity Points story size) from the epic capsule and carry it into `state.yaml` as `epic_cycle.story_bcps = N`.
|
|
33
|
+
3. If `epic_cycle.step` missing, set to `1`.
|
|
34
|
+
4. Run **only the current step** (resume mode) unless user asked for full auto-run.
|
|
35
|
+
5. After step verify passes, increment `epic_cycle.step` in `state.yaml` (or `bash scripts/bp-yaml-set.sh` if available).
|
|
36
|
+
6. On story complete, set `execution-status.yaml` story key to `done`; run `bash scripts/sync-status-from-epics.sh`.
|
|
37
|
+
|
|
38
|
+
## Handoff
|
|
39
|
+
|
|
40
|
+
Write `handoff.next_skill` and `handoff.context` in `state.yaml` when pausing mid-epic.
|
|
41
|
+
|
|
42
|
+
## Verify
|
|
43
|
+
|
|
44
|
+
→ verify: `grep -q 'active_flow: build_epic' specs/state.yaml && test -f specs/epics/*/epic.yaml`
|
|
@@ -0,0 +1,105 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Add a new requirement or reorder epics by WSJF against specs/release-plan.yaml and epic capsule directories. Modes Add and Reorder. Use when a new requirement arrives mid-release or the plan needs prioritization.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Change Request
|
|
7
|
+
|
|
8
|
+
> **HARD GATE** — `specs/release-plan.yaml` must exist before running either mode. If it doesn't, run `plan-release` first.
|
|
9
|
+
>
|
|
10
|
+
> → verify: `[ -f specs/release-plan.yaml ] && echo "ready" || echo "BLOCKED: run plan-release first"`
|
|
11
|
+
|
|
12
|
+
Two modes. State which one you want or the skill will ask.
|
|
13
|
+
|
|
14
|
+
## Mode A — Add
|
|
15
|
+
|
|
16
|
+
Intake a new requirement mid-flight without disrupting work in progress.
|
|
17
|
+
|
|
18
|
+
1. **Capture**: What is the change? What problem does it solve?
|
|
19
|
+
2. **Locate**: Which existing stories in `specs/epics/` does it affect or replace?
|
|
20
|
+
3. **Draft**: Add story + `tasks[]` with Gherkin-style AC in epic YAML (each task has `verify`).
|
|
21
|
+
4. **Place**: Append story under existing epic capsule, or create `specs/epics/eNN-slug.yaml` and register in `release-plan.yaml` `epics[]`.
|
|
22
|
+
5. **Score**: Compute WSJF; note if it outranks in-progress work.
|
|
23
|
+
|
|
24
|
+
→ verify: `grep -c 'stories:' specs/epics/*/epic.yaml`
|
|
25
|
+
|
|
26
|
+
## Mode B — Reorder
|
|
27
|
+
|
|
28
|
+
Value-engineering pass over the full release using WSJF.
|
|
29
|
+
|
|
30
|
+
See [REFERENCE.md](REFERENCE.md) for the full WSJF scoring rubric.
|
|
31
|
+
|
|
32
|
+
1. **Score** each epic/story: BV + TC + RR / Job Size.
|
|
33
|
+
2. **Re-sort** `release-plan.yaml` `epics[]` and per-epic `wsjf` fields.
|
|
34
|
+
3. **Flag cut candidates**: WSJF < 1.5.
|
|
35
|
+
4. **Update** `specs/release-plan.yaml` and epic `wsjf` keys with rationale.
|
|
36
|
+
5. **Report** the delta.
|
|
37
|
+
|
|
38
|
+
→ verify: `grep -c 'wsjf' specs/release-plan.yaml specs/epics/*/epic.yaml`
|
|
39
|
+
|
|
40
|
+
## After either mode
|
|
41
|
+
|
|
42
|
+
Run `bash scripts/sync-status-from-epics.sh`. Suggest `plan-work` or `build-epic` for the top-ranked unstarted story.
|
|
43
|
+
|
|
44
|
+
---
|
|
45
|
+
|
|
46
|
+
# WSJF Scoring Reference
|
|
47
|
+
|
|
48
|
+
Weighted Shortest Job First: **WSJF = (Business Value + Time Criticality + Risk Reduction) / Job Size**
|
|
49
|
+
|
|
50
|
+
All dimensions scored 1–10 using a Fibonacci-like scale: 1, 2, 3, 5, 8, 10.
|
|
51
|
+
|
|
52
|
+
## Business Value
|
|
53
|
+
|
|
54
|
+
| Score | Meaning |
|
|
55
|
+
|-------|---------|
|
|
56
|
+
| 10 | Core revenue path, legal requirement, blocking launch |
|
|
57
|
+
| 8 | Significant user value, major pain point removed |
|
|
58
|
+
| 5 | Notable improvement, medium user segment affected |
|
|
59
|
+
| 3 | Nice-to-have, minor convenience |
|
|
60
|
+
| 1 | Cosmetic or speculative |
|
|
61
|
+
|
|
62
|
+
## Time Criticality
|
|
63
|
+
|
|
64
|
+
| Score | Meaning |
|
|
65
|
+
|-------|---------|
|
|
66
|
+
| 10 | Hard deadline (regulatory, contract, launch date) |
|
|
67
|
+
| 8 | Competitive window closing, partner dependency |
|
|
68
|
+
| 5 | Would delay next milestone if deferred |
|
|
69
|
+
| 3 | Flexible, can slip one sprint |
|
|
70
|
+
| 1 | No urgency |
|
|
71
|
+
|
|
72
|
+
## Risk Reduction (or Opportunity Enablement)
|
|
73
|
+
|
|
74
|
+
| Score | Meaning |
|
|
75
|
+
|-------|---------|
|
|
76
|
+
| 10 | Eliminates critical architectural risk or enables a new capability |
|
|
77
|
+
| 8 | Reduces a known failure mode or unblocks high-value work |
|
|
78
|
+
| 5 | Moderate risk mitigation or enablement |
|
|
79
|
+
| 3 | Low risk reduction |
|
|
80
|
+
| 1 | No risk relevance |
|
|
81
|
+
|
|
82
|
+
## Job Size (effort denominator — larger = lower WSJF)
|
|
83
|
+
|
|
84
|
+
| Score | Meaning |
|
|
85
|
+
|-------|---------|
|
|
86
|
+
| 1 | Hours |
|
|
87
|
+
| 2 | 1 day |
|
|
88
|
+
| 3 | 2–3 days |
|
|
89
|
+
| 5 | 1 week |
|
|
90
|
+
| 8 | 2 weeks |
|
|
91
|
+
| 10 | 1 month+ |
|
|
92
|
+
|
|
93
|
+
## Cut threshold
|
|
94
|
+
|
|
95
|
+
Stories with WSJF < 1.5 are cut candidates: high effort, low combined value.
|
|
96
|
+
|
|
97
|
+
## Example
|
|
98
|
+
|
|
99
|
+
Story: "Add OAuth login"
|
|
100
|
+
- Business Value: 8 (removes major sign-up friction)
|
|
101
|
+
- Time Criticality: 5 (Q3 launch nice, not hard)
|
|
102
|
+
- Risk Reduction: 3 (minor security improvement)
|
|
103
|
+
- Job Size: 5 (one week)
|
|
104
|
+
|
|
105
|
+
WSJF = (8 + 5 + 3) / 5 = **3.2** — healthy, implement early.
|
|
@@ -0,0 +1,135 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Reviews working-tree changes, then drafts a Conventional Commits title/body and states the semantic-release version bump a single such commit would imply. Also notes which defensive-code categories were touched. Use when the user wants to commit recent work, prepare a Conventional Commits message, or asks for semantic-release / semver-consistent messaging before git commit.
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Commit Message
|
|
7
|
+
> **HARD GATE** — **HARD GATE** — Commits must follow Conventional Commits spec (type(scope): description). Do NOT use vague messages like 'fix' or 'updates.' The message must explain the 'why,' not the 'what.'
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
## Modes
|
|
11
|
+
|
|
12
|
+
- Default: standard Conventional Commits message
|
|
13
|
+
- --fix-type: Forces type=fix. Use when commit type is unambiguous.
|
|
14
|
+
|
|
15
|
+
## What "last chat" means
|
|
16
|
+
|
|
17
|
+
- **Primary source of truth:** `git status`, `git diff` (unstaged), and `git diff --cached` (staged). Run these in the repo root (or the paths the user changed).
|
|
18
|
+
- **Context:** use the current conversation to summarize *intent* and to spot **breaking** API/behavior changes that diff alone may not show.
|
|
19
|
+
- If the user tracks a session baseline (e.g. branch, tag, or `git stash create` at start), you may `git diff <baseline>..HEAD` plus uncommitted diffs; otherwise use only the index and working tree.
|
|
20
|
+
|
|
21
|
+
## Quick workflow
|
|
22
|
+
|
|
23
|
+
1. **Inventory** — List changed paths; group by feature vs chore vs docs vs test-only.
|
|
24
|
+
2. **Decide commit shape** — One atomic commit is ideal. If the diff mixes unrelated concerns, recommend **multiple commits** (each with its own type/scope) before suggesting one message.
|
|
25
|
+
3. **Classify for semantic release** — `fix` → patch, `feat` → minor, **breaking** → major.
|
|
26
|
+
4. **Write the message** — `type(optional-scope)!: description` (see [REFERENCE.md](REFERENCE.md#message-format)). Use `!` or a `BREAKING CHANGE:` footer when behavior contracts change.
|
|
27
|
+
5. **Note defensive-code categories touched** — from CONVENTIONS.md: Rate limit | Retry with backoff | Circuit breaker | Timeout | Graceful degradation
|
|
28
|
+
6. **Deliver** — Output:
|
|
29
|
+
- Proposed **full commit message** (title + optional body + footers).
|
|
30
|
+
- **Release bump** this commit would drive: `patch` | `minor` | `major` | `none`.
|
|
31
|
+
- Optional `git add …` and `git commit -m` instructions; do **not** run destructive git commands unless the user asked.
|
|
32
|
+
|
|
33
|
+
## Checklist before finalizing
|
|
34
|
+
|
|
35
|
+
- [ ] Type matches the **dominant** user-visible outcome (`feat` vs `fix` vs `perf`, etc.).
|
|
36
|
+
- [ ] **Scope** is a short noun in parentheses if it helps (e.g. `fix(api): …`).
|
|
37
|
+
- [ ] Breaking changes are explicit (`!` and/or `BREAKING CHANGE:` in the body/footer).
|
|
38
|
+
- [ ] Description is imperative, lowercase start after the prefix, no trailing period in the title line.
|
|
39
|
+
|
|
40
|
+
## When not to invent a bump
|
|
41
|
+
|
|
42
|
+
If the repo uses a custom `@semantic-release/commit-analyzer` preset, note that your bump is **heuristic** and they should match `.releaserc` / `release.config.*`. See [REFERENCE.md](REFERENCE.md#custom-repositories).
|
|
43
|
+
|
|
44
|
+
## Further reading
|
|
45
|
+
|
|
46
|
+
- [REFERENCE.md](REFERENCE.md) — Message shape, footers, release mapping, squashing notes.
|
|
47
|
+
|
|
48
|
+
## Handoff
|
|
49
|
+
|
|
50
|
+
Gate: READY -> next: release-branch
|
|
51
|
+
Writes: state.yaml handoff.next_skill = release-branch
|
|
52
|
+
|
|
53
|
+
---
|
|
54
|
+
|
|
55
|
+
# Conventional Commits + semantic-style release (reference)
|
|
56
|
+
|
|
57
|
+
## Message format
|
|
58
|
+
|
|
59
|
+
From [Conventional Commits 1.0.0](https://www.conventionalcommits.org/en/v1.0.0/#specification):
|
|
60
|
+
|
|
61
|
+
```text
|
|
62
|
+
<type>[optional scope][optional !]: <description>
|
|
63
|
+
|
|
64
|
+
[optional body]
|
|
65
|
+
|
|
66
|
+
[optional footer(s)]
|
|
67
|
+
```
|
|
68
|
+
|
|
69
|
+
- **Scope:** parenthesized noun, e.g. `feat(parser): …`.
|
|
70
|
+
- **Breaking:** `!` before `:` (e.g. `feat(api)!: …`) and/or footer `BREAKING CHANGE: description` (token must be uppercase per spec for that footer name).
|
|
71
|
+
- **Description:** short summary; body explains *why* or migration steps.
|
|
72
|
+
|
|
73
|
+
Common **types** (not exhaustive): `feat`, `fix`, `docs`, `style`, `refactor`, `perf`, `test`, `build`, `ci`, `chore` — as in [Angular / commitlint conventions](https://github.com/conventional-changelog/commitlint).
|
|
74
|
+
|
|
75
|
+
## Advanced Specification Patterns
|
|
76
|
+
|
|
77
|
+
### Reverts
|
|
78
|
+
If the commit reverts a previous commit, it should begin with `revert:`, followed by the header of the reverted commit. In the body, it should say: `This reverts commit <hash>.`.
|
|
79
|
+
|
|
80
|
+
```text
|
|
81
|
+
revert: feat(api): add user endpoint
|
|
82
|
+
|
|
83
|
+
This reverts commit 676104e.
|
|
84
|
+
```
|
|
85
|
+
|
|
86
|
+
### Breaking Changes
|
|
87
|
+
A breaking change can be signaled by:
|
|
88
|
+
1. A **`BREAKING CHANGE:`** footer (must be uppercase, at the start of the footer). This is the **most compatible** way to trigger a Major release in `semantic-release` (Angular preset).
|
|
89
|
+
2. A **`!`** after the type/scope: `feat(api)!: change user response shape`.
|
|
90
|
+
|
|
91
|
+
**Pro-tip:** For maximum compatibility with all tooling (older and newer), use BOTH the `!` and the `BREAKING CHANGE:` footer.
|
|
92
|
+
|
|
93
|
+
### Footers (Tokens & Values)
|
|
94
|
+
Footers follow the same `Token: value` pattern as Git Trailers. Common tokens:
|
|
95
|
+
- `Refs: #123`
|
|
96
|
+
- `See-also: docs/ADR-001.md`
|
|
97
|
+
- `Signed-off-by: Name <email>`
|
|
98
|
+
|
|
99
|
+
**Multi-line footers:** If a footer value spans multiple lines, each subsequent line must be indented.
|
|
100
|
+
|
|
101
|
+
### Squashing & History
|
|
102
|
+
When using `gh pr merge --squash`, the PR title is usually used as the commit subject.
|
|
103
|
+
- **PR Title:** MUST follow `<type>(<scope>): <description>`
|
|
104
|
+
- **PR Body:** Content will be moved to the commit body.
|
|
105
|
+
|
|
106
|
+
## Release Type Mapping (Default Angular Preset)
|
|
107
|
+
|
|
108
|
+
This table reflects the **out-of-the-box** behavior of `semantic-release` using the `@semantic-release/commit-analyzer` default (Angular) rules.
|
|
109
|
+
|
|
110
|
+
| Commit pattern | Release | Notes |
|
|
111
|
+
|----------------|---------|-------|
|
|
112
|
+
| `fix:` | **Patch** | Bug fixes |
|
|
113
|
+
| `feat:` | **Minor** | New features |
|
|
114
|
+
| `perf:` | **Patch** | Performance improvements |
|
|
115
|
+
| `any type` + `BREAKING CHANGE:` footer | **Major** | **Mandatory** for Major version bumps in default configs. |
|
|
116
|
+
| `any type!:` (exclamation mark) | **Major** | Supported by modern CC parsers, but use footer for max safety. |
|
|
117
|
+
| `docs:`, `chore:`, `test:`, `ci:`, `refactor:`, `style:` | **None** | Does not trigger a new release by default. |
|
|
118
|
+
|
|
119
|
+
> **Warning:** While `refactor:` and `style:` improve code, they do NOT trigger a release in the default Angular preset. Use `fix:` if a refactor also fixes a bug, or `feat:` if it adds new behavior.
|
|
120
|
+
|
|
121
|
+
## Custom Repositories
|
|
122
|
+
|
|
123
|
+
- Read `release.config.js`, `.releaserc`, or `package.json` → `release` / `semantic-release` config.
|
|
124
|
+
- The **@semantic-release/commit-analyzer** preset may map types differently; prefer **their** rules when they conflict with this reference.
|
|
125
|
+
|
|
126
|
+
## Squash and PR titles
|
|
127
|
+
|
|
128
|
+
- If the team squashes on merge, the **PR title** often becomes the single squashed commit subject — it should still follow `type(scope): description` for tooling.
|
|
129
|
+
- `revert:` type and `Refs:` footers are valid patterns; revert handling varies by [tooling](https://www.conventionalcommits.org/en/v1.0.0/#specification).
|
|
130
|
+
|
|
131
|
+
## Links
|
|
132
|
+
|
|
133
|
+
- [Conventional Commits — specification](https://www.conventionalcommits.org/en/v1.0.0/#specification)
|
|
134
|
+
- [semantic-release — README (commit format & flow)](https://github.com/semantic-release/semantic-release#commit-message-format)
|
|
135
|
+
- Fork pointer: [semantic-release-baby](https://github.com/danielvm-git/semantic-release-baby) (automation and docs align with upstream semantic-release)
|
|
@@ -0,0 +1,40 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Chain multiple bigpowers skills into a custom workflow recipe saved in specs/. Use when a project repeats a non-standard skill sequence, or user wants a documented playbook beyond orchestrate-project modes.model: sonnet
|
|
3
|
+
---
|
|
4
|
+
|
|
5
|
+
|
|
6
|
+
# Compose Workflow
|
|
7
|
+
> **HARD GATE** — **HARD GATE** — Workflows are orchestration, not automation. Do NOT create workflows for tasks that should be single skills. Workflow complexity must be justified.
|
|
8
|
+
|
|
9
|
+
|
|
10
|
+
## Process
|
|
11
|
+
|
|
12
|
+
1. Interview: goal, phases, which skills, gates between steps.
|
|
13
|
+
2. Write `specs/WORKFLOW-<name>.md`:
|
|
14
|
+
- Trigger ("Use when...")
|
|
15
|
+
- Ordered steps: `skill → artefact → verify`
|
|
16
|
+
- HARD GATEs between phases
|
|
17
|
+
3. Register in state.yaml Active Decisions.
|
|
18
|
+
4. Optional: reference from `orchestrate-project` Ad-Hoc mode.
|
|
19
|
+
|
|
20
|
+
## Verify
|
|
21
|
+
|
|
22
|
+
→ verify: `test -f specs/WORKFLOW-*.md && grep -c "verify:" specs/WORKFLOW-*.md | awk '{if($1>0) print "OK"}'`
|
|
23
|
+
|
|
24
|
+
See [REFERENCE.md](REFERENCE.md) for template.
|
|
25
|
+
|
|
26
|
+
---
|
|
27
|
+
|
|
28
|
+
# Workflow template
|
|
29
|
+
|
|
30
|
+
```markdown
|
|
31
|
+
# WORKFLOW: <name>
|
|
32
|
+
|
|
33
|
+
**Trigger:** Use when ...
|
|
34
|
+
|
|
35
|
+
| Step | Skill | Output | verify |
|
|
36
|
+
|------|-------|--------|--------|
|
|
37
|
+
| 1 | survey-context | state.yaml handoff | ... |
|
|
38
|
+
| 2 | research-first | Prior Art in SCOPE_LATEST.yaml | ... |
|
|
39
|
+
| 3 | plan-work | epics/eNN-*.yaml tasks | ... |
|
|
40
|
+
```
|