@jspg-ai/coding-bb 0.0.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +41 -0
- package/cbb/dev-standards/rules/cbb-ai-behavior.md +104 -0
- package/cbb/dev-standards/rules/cbb-coding-rule.md +50 -0
- package/cbb/dev-standards/rules/cbb-priority.md +56 -0
- package/cbb/dev-standards/skills/architecture-specs/SKILL.md +129 -0
- package/cbb/dev-standards/skills/coding-specs/SKILL.md +376 -0
- package/cbb/dev-standards/skills/coding-specs/references/concurrency.md +53 -0
- package/cbb/dev-standards/skills/coding-specs/references/config-center.md +16 -0
- package/cbb/dev-standards/skills/coding-specs/references/distributed.md +42 -0
- package/cbb/dev-standards/skills/coding-specs/references/es-coding.md +60 -0
- package/cbb/dev-standards/skills/coding-specs/references/scheduled-task.md +25 -0
- package/cbb/dev-standards/skills/coding-specs/references/security.md +11 -0
- package/cbb/dev-standards/skills/coding-specs/references/unit-testing.md +43 -0
- package/cbb/dev-standards/skills/es-design-specs/SKILL.md +104 -0
- package/cbb/dev-standards/skills/mysql-design-specs/SKILL.md +89 -0
- package/cbb/lib/install/claude-code.js +20 -0
- package/cbb/lib/install/cleanup.js +64 -0
- package/cbb/lib/install/init.js +1875 -0
- package/cbb/lib/install/qoder.js +20 -0
- package/cbb/lib/install/workspaces.js +232 -0
- package/cbb/lib/openspec/index.js +554 -0
- package/cbb/lib/superpowers/index.js +265 -0
- package/cbb/lib/utils/check-update.js +147 -0
- package/cbb/lib/utils/checkbox.js +383 -0
- package/cbb/lib/utils/gitignore.js +69 -0
- package/cbb/lib/utils/output.js +64 -0
- package/cbb/lib/utils/settings.js +119 -0
- package/cbb/lib/utils/version.js +135 -0
- package/cbb/lib/wiki/api-client.js +358 -0
- package/cbb/lib/wiki/cli.js +427 -0
- package/cbb/lib/wiki/convert.js +464 -0
- package/cbb/lib/wiki/index.js +218 -0
- package/cbb/lib/wiki/mermaid-guard.js +103 -0
- package/cbb/lib/wiki/split.js +131 -0
- package/cbb/tools/cbb-decompile-jar/SKILL.md +220 -0
- package/cbb/tools/cbb-decompile-jar/scripts/decompile.py +863 -0
- package/cbb/tools/cbb-design-to-wiki/SKILL.md +180 -0
- package/cbb/tools/cbb-mvn-guide/SKILL.md +281 -0
- package/cbb/tools/cbb-mvn-guide/scripts/mvn_jdk_manager.py +378 -0
- package/cbb/tools/cbb-mvn-guide/scripts/mvn_pom_jdk_reader.py +197 -0
- package/cbb/tools/cbb-plantuml-authoring/SKILL.md +448 -0
- package/cbb/tools/cbb-plantuml-authoring/references/drawing-templates.md +479 -0
- package/cbb/tools/cbb-wiki-ops/SKILL.md +134 -0
- package/cbb/tools/cbb-wiki-ops/scripts/check-auth.js +44 -0
- package/cbb/worktrees/_shared/scripts/find-target-worktree.js +96 -0
- package/cbb/worktrees/_shared/scripts/find-workspace-root.js +67 -0
- package/cbb/worktrees/_shared/scripts/push-core.js +136 -0
- package/cbb/worktrees/_shared/scripts/silence-popup.js +23 -0
- package/cbb/worktrees/commands/close.md +64 -0
- package/cbb/worktrees/commands/extend.md +65 -0
- package/cbb/worktrees/commands/init.md +55 -0
- package/cbb/worktrees/commands/push.md +42 -0
- package/cbb/worktrees/skills/openspec-close-worktree/SKILL.md +449 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/check-env.js +76 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/check-unarchived.js +48 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/delete-branches.js +136 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/discover-apps.js +76 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/find-target-worktree.js +96 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/find-workspace-root.js +67 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/remove-worktrees.js +124 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/safety-check.js +174 -0
- package/cbb/worktrees/skills/openspec-close-worktree/scripts/silence-popup.js +23 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/SKILL.md +390 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/check-env.js +95 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/check-repos.js +98 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/create-branches-and-worktrees.js +135 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/find-target-worktree.js +96 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/find-workspace-root.js +67 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/install-ai.js +150 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/list-available-apps.js +88 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/push-core.js +136 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/silence-popup.js +23 -0
- package/cbb/worktrees/skills/openspec-extend-worktree/scripts/sync-repos.js +93 -0
- package/cbb/worktrees/skills/openspec-init-worktree/SKILL.md +539 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/auto-open.js +136 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/check-env-deep.js +87 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/check-env.js +90 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/check-repos.js +171 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/create-branches.js +103 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/create-worktrees.js +154 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/find-workspace-root.js +67 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/generate-app-options.js +93 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/install-ai.js +177 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/parse-config.js +67 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/push-branches.js +101 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/push-core.js +136 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/silence-popup.js +23 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/sync-repos.js +95 -0
- package/cbb/worktrees/skills/openspec-init-worktree/scripts/update-gitignore.js +134 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/SKILL.md +307 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/commit-worktrees.js +269 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/find-target-worktree.js +96 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/find-workspace-root.js +67 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/push-core.js +136 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/push-worktrees.js +159 -0
- package/cbb/worktrees/skills/openspec-push-worktrees/scripts/silence-popup.js +23 -0
- package/config/cbb.yaml +6 -0
- package/config/config.sample.json +16 -0
- package/config/openspec/config.yaml +28 -0
- package/config/openspec/schemas/spec-driven/schema.yaml +247 -0
- package/config/openspec/schemas/spec-driven/templates/design.md +475 -0
- package/config/openspec/schemas/spec-driven/templates/proposal.md +58 -0
- package/config/openspec/schemas/spec-driven/templates/spec.md +8 -0
- package/config/openspec/schemas/spec-driven/templates/tasks.md +9 -0
- package/config/workspaces.json +15 -0
- package/openspec/.version +6 -0
- package/openspec/commands/apply.md +182 -0
- package/openspec/commands/archive.md +223 -0
- package/openspec/commands/bulk-archive.md +334 -0
- package/openspec/commands/continue.md +112 -0
- package/openspec/commands/explore.md +206 -0
- package/openspec/commands/ff.md +111 -0
- package/openspec/commands/new.md +70 -0
- package/openspec/commands/onboard.md +555 -0
- package/openspec/commands/propose.md +157 -0
- package/openspec/commands/sync.md +256 -0
- package/openspec/commands/update.md +85 -0
- package/openspec/commands/verify.md +169 -0
- package/openspec/skills/openspec-apply-change/SKILL.md +187 -0
- package/openspec/skills/openspec-archive-change/SKILL.md +181 -0
- package/openspec/skills/openspec-bulk-archive-change/SKILL.md +338 -0
- package/openspec/skills/openspec-continue-change/SKILL.md +117 -0
- package/openspec/skills/openspec-explore/SKILL.md +342 -0
- package/openspec/skills/openspec-ff-change/SKILL.md +116 -0
- package/openspec/skills/openspec-new-change/SKILL.md +76 -0
- package/openspec/skills/openspec-onboard/SKILL.md +560 -0
- package/openspec/skills/openspec-propose/SKILL.md +162 -0
- package/openspec/skills/openspec-sync-specs/SKILL.md +261 -0
- package/openspec/skills/openspec-update-change/SKILL.md +90 -0
- package/openspec/skills/openspec-verify-change/SKILL.md +174 -0
- package/package.json +46 -0
- package/superpowers/.version +6 -0
- package/superpowers/skills/brainstorming/SKILL.md +250 -0
- package/superpowers/skills/brainstorming/scripts/frame-template.html +213 -0
- package/superpowers/skills/brainstorming/scripts/helper.js +167 -0
- package/superpowers/skills/brainstorming/scripts/server.cjs +723 -0
- package/superpowers/skills/brainstorming/scripts/start-server.sh +209 -0
- package/superpowers/skills/brainstorming/scripts/stop-server.sh +120 -0
- package/superpowers/skills/brainstorming/spec-document-reviewer-prompt.md +49 -0
- package/superpowers/skills/brainstorming/visual-companion.md +299 -0
- package/superpowers/skills/dispatching-parallel-agents/SKILL.md +167 -0
- package/superpowers/skills/executing-plans/SKILL.md +64 -0
- package/superpowers/skills/finishing-a-development-branch/SKILL.md +225 -0
- package/superpowers/skills/receiving-code-review/SKILL.md +205 -0
- package/superpowers/skills/requesting-code-review/SKILL.md +95 -0
- package/superpowers/skills/requesting-code-review/code-reviewer.md +181 -0
- package/superpowers/skills/subagent-driven-development/SKILL.md +568 -0
- package/superpowers/skills/subagent-driven-development/implementer-prompt.md +154 -0
- package/superpowers/skills/subagent-driven-development/re-review-prompt.md +115 -0
- package/superpowers/skills/subagent-driven-development/scripts/review-package +46 -0
- package/superpowers/skills/subagent-driven-development/scripts/sdd-workspace +40 -0
- package/superpowers/skills/subagent-driven-development/scripts/task-brief +41 -0
- package/superpowers/skills/subagent-driven-development/task-reviewer-prompt.md +207 -0
- package/superpowers/skills/systematic-debugging/CREATION-LOG.md +119 -0
- package/superpowers/skills/systematic-debugging/SKILL.md +283 -0
- package/superpowers/skills/systematic-debugging/condition-based-waiting-example.ts +158 -0
- package/superpowers/skills/systematic-debugging/condition-based-waiting.md +115 -0
- package/superpowers/skills/systematic-debugging/defense-in-depth.md +122 -0
- package/superpowers/skills/systematic-debugging/find-polluter.sh +72 -0
- package/superpowers/skills/systematic-debugging/root-cause-tracing.md +169 -0
- package/superpowers/skills/systematic-debugging/test-academic.md +14 -0
- package/superpowers/skills/systematic-debugging/test-pressure-1.md +58 -0
- package/superpowers/skills/systematic-debugging/test-pressure-2.md +68 -0
- package/superpowers/skills/systematic-debugging/test-pressure-3.md +69 -0
- package/superpowers/skills/test-driven-development/SKILL.md +320 -0
- package/superpowers/skills/test-driven-development/writing-good-tests.md +198 -0
- package/superpowers/skills/using-git-worktrees/SKILL.md +167 -0
- package/superpowers/skills/using-superpowers/SKILL.md +63 -0
- package/superpowers/skills/using-superpowers/references/antigravity-tools.md +23 -0
- package/superpowers/skills/using-superpowers/references/codex-tools.md +108 -0
- package/superpowers/skills/using-superpowers/references/gemini-tools.md +63 -0
- package/superpowers/skills/using-superpowers/references/hermes-tools.md +56 -0
- package/superpowers/skills/using-superpowers/references/pi-tools.md +16 -0
- package/superpowers/skills/verification-before-completion/SKILL.md +120 -0
- package/superpowers/skills/writing-plans/SKILL.md +171 -0
- package/superpowers/skills/writing-plans/plan-document-reviewer-prompt.md +49 -0
- package/superpowers/skills/writing-skills/SKILL.md +679 -0
- package/superpowers/skills/writing-skills/anthropic-best-practices.md +1150 -0
- package/superpowers/skills/writing-skills/examples/CLAUDE_MD_TESTING.md +189 -0
- package/superpowers/skills/writing-skills/graphviz-conventions.dot +172 -0
- package/superpowers/skills/writing-skills/persuasion-principles.md +187 -0
- package/superpowers/skills/writing-skills/render-graphs.js +169 -0
- package/superpowers/skills/writing-skills/testing-skills-with-subagents.md +384 -0
|
@@ -0,0 +1,174 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: openspec-verify-change
|
|
3
|
+
description: Verify implementation matches change artifacts. Use when the user wants to validate that implementation is complete, correct, and coherent before archiving.
|
|
4
|
+
allowed-tools: Bash(openspec:*)
|
|
5
|
+
license: MIT
|
|
6
|
+
compatibility: Requires openspec CLI.
|
|
7
|
+
metadata:
|
|
8
|
+
author: openspec
|
|
9
|
+
version: "1.0"
|
|
10
|
+
---
|
|
11
|
+
|
|
12
|
+
Verify that an implementation matches the change artifacts (specs, tasks, design).
|
|
13
|
+
|
|
14
|
+
**Store selection:** If the user names a store (a store is a standalone OpenSpec repo registered on this machine) or the work lives in one, run `openspec store list --json` to discover registered store ids, then pass `--store <id>` on the commands that read or write specs and changes (`new change`, `status`, `instructions`, `list`, `show`, `validate`, `archive`, `doctor`, `context`, `schemas`, `view`). Once selected, treat `--store <id>` as sticky for the rest of the workflow. Every unscoped example of those commands below is shorthand: before running it, append the flag. For example, run `openspec status --change "<name>" --json --store "<id>"`, not the unscoped form shown below. Other commands do not take the flag. Hints printed by commands already carry the flag; keep it on follow-ups. Without a store, commands act on the nearest local `openspec/` root.
|
|
15
|
+
|
|
16
|
+
**Input**: Optionally specify a change name. If omitted, check if it can be inferred from conversation context. If vague or ambiguous you MUST prompt for available changes.
|
|
17
|
+
|
|
18
|
+
**Steps**
|
|
19
|
+
|
|
20
|
+
1. **Select the change**
|
|
21
|
+
|
|
22
|
+
If a name is provided, use it. Otherwise:
|
|
23
|
+
- Infer from conversation context if the user mentioned a change
|
|
24
|
+
- Auto-select if only one active change exists
|
|
25
|
+
- If ambiguous, run `openspec list --json` to get available changes and ask the user to select one
|
|
26
|
+
|
|
27
|
+
When prompting, show changes that have implementation tasks (tasks artifact exists).
|
|
28
|
+
Include the schema used for each change if available.
|
|
29
|
+
Mark changes with incomplete tasks as "(In Progress)".
|
|
30
|
+
|
|
31
|
+
Always announce: "Using change: <name>" and how to override (e.g., `/openspec-verify-change <other>`).
|
|
32
|
+
|
|
33
|
+
2. **Check status to understand the schema**
|
|
34
|
+
```bash
|
|
35
|
+
openspec status --change "<name>" --json
|
|
36
|
+
```
|
|
37
|
+
Parse the JSON to understand:
|
|
38
|
+
- `schemaName`: The workflow being used (e.g., "spec-driven")
|
|
39
|
+
- `planningHome`, `changeRoot`, `artifactPaths`, and `actionContext`: path and scope context
|
|
40
|
+
- Which artifacts exist for this change
|
|
41
|
+
|
|
42
|
+
3. **Get planning context and load artifacts**
|
|
43
|
+
|
|
44
|
+
```bash
|
|
45
|
+
openspec instructions apply --change "<name>" --json
|
|
46
|
+
```
|
|
47
|
+
|
|
48
|
+
This returns the change directory and `contextFiles` (artifact ID -> array of concrete file paths). Read all available artifacts from `contextFiles`.
|
|
49
|
+
|
|
50
|
+
4. **Initialize verification report structure**
|
|
51
|
+
|
|
52
|
+
Create a report structure with three dimensions:
|
|
53
|
+
- **Completeness**: Track tasks and spec coverage
|
|
54
|
+
- **Correctness**: Track requirement implementation and scenario coverage
|
|
55
|
+
- **Coherence**: Track design adherence and pattern consistency
|
|
56
|
+
|
|
57
|
+
Each dimension can have CRITICAL, WARNING, or SUGGESTION issues.
|
|
58
|
+
|
|
59
|
+
5. **Verify Completeness**
|
|
60
|
+
|
|
61
|
+
**Task Completion**:
|
|
62
|
+
- If `contextFiles.tasks` exists, read every file path in it
|
|
63
|
+
- Parse checkboxes: `- [ ]` (incomplete) vs `- [x]` (complete)
|
|
64
|
+
- Count complete vs total tasks
|
|
65
|
+
- If incomplete tasks exist:
|
|
66
|
+
- Add CRITICAL issue for each incomplete task
|
|
67
|
+
- Recommendation: "Complete task: <description>" or "Mark as done if already implemented"
|
|
68
|
+
|
|
69
|
+
**Spec Coverage**:
|
|
70
|
+
- If delta specs exist in `contextFiles.specs`:
|
|
71
|
+
- Extract all requirements (marked with "### Requirement:")
|
|
72
|
+
- For each requirement:
|
|
73
|
+
- Search codebase for keywords related to the requirement
|
|
74
|
+
- Assess if implementation likely exists
|
|
75
|
+
- If requirements appear unimplemented:
|
|
76
|
+
- Add CRITICAL issue: "Requirement not found: <requirement name>"
|
|
77
|
+
- Recommendation: "Implement requirement X: <description>"
|
|
78
|
+
|
|
79
|
+
6. **Verify Correctness**
|
|
80
|
+
|
|
81
|
+
**Requirement Implementation Mapping**:
|
|
82
|
+
- For each requirement from delta specs:
|
|
83
|
+
- Search codebase for implementation evidence
|
|
84
|
+
- If found, note file paths and line ranges
|
|
85
|
+
- Assess if implementation matches requirement intent
|
|
86
|
+
- If divergence detected:
|
|
87
|
+
- Add WARNING: "Implementation may diverge from spec: <details>"
|
|
88
|
+
- Recommendation: "Review <file>:<lines> against requirement X"
|
|
89
|
+
|
|
90
|
+
**Scenario Coverage**:
|
|
91
|
+
- For each scenario in delta specs (marked with "#### Scenario:"):
|
|
92
|
+
- Check if conditions are handled in code
|
|
93
|
+
- Check if tests exist covering the scenario
|
|
94
|
+
- If scenario appears uncovered:
|
|
95
|
+
- Add WARNING: "Scenario not covered: <scenario name>"
|
|
96
|
+
- Recommendation: "Add test or implementation for scenario: <description>"
|
|
97
|
+
|
|
98
|
+
7. **Verify Coherence**
|
|
99
|
+
|
|
100
|
+
**Design Adherence**:
|
|
101
|
+
- If `contextFiles.design` exists:
|
|
102
|
+
- Extract key decisions (look for sections like "Decision:", "Approach:", "Architecture:")
|
|
103
|
+
- Verify implementation follows those decisions
|
|
104
|
+
- If contradiction detected:
|
|
105
|
+
- Add WARNING: "Design decision not followed: <decision>"
|
|
106
|
+
- Recommendation: "Update implementation or revise design.md to match reality"
|
|
107
|
+
- If no design.md: Skip design adherence check, note "No design.md to verify against"
|
|
108
|
+
|
|
109
|
+
**Code Pattern Consistency**:
|
|
110
|
+
- Review new code for consistency with project patterns
|
|
111
|
+
- Check file naming, directory structure, coding style
|
|
112
|
+
- If significant deviations found:
|
|
113
|
+
- Add SUGGESTION: "Code pattern deviation: <details>"
|
|
114
|
+
- Recommendation: "Consider following project pattern: <example>"
|
|
115
|
+
|
|
116
|
+
8. **Generate Verification Report**
|
|
117
|
+
|
|
118
|
+
**Summary Scorecard**:
|
|
119
|
+
```markdown
|
|
120
|
+
## Verification Report: <change-name>
|
|
121
|
+
|
|
122
|
+
### Summary
|
|
123
|
+
| Dimension | Status |
|
|
124
|
+
|--------------|------------------|
|
|
125
|
+
| Completeness | X/Y tasks, N reqs|
|
|
126
|
+
| Correctness | M/N reqs covered |
|
|
127
|
+
| Coherence | Followed/Issues |
|
|
128
|
+
```
|
|
129
|
+
|
|
130
|
+
**Issues by Priority**:
|
|
131
|
+
|
|
132
|
+
1. **CRITICAL** (Must fix before archive):
|
|
133
|
+
- Incomplete tasks
|
|
134
|
+
- Missing requirement implementations
|
|
135
|
+
- Each with specific, actionable recommendation
|
|
136
|
+
|
|
137
|
+
2. **WARNING** (Should fix):
|
|
138
|
+
- Spec/design divergences
|
|
139
|
+
- Missing scenario coverage
|
|
140
|
+
- Each with specific recommendation
|
|
141
|
+
|
|
142
|
+
3. **SUGGESTION** (Nice to fix):
|
|
143
|
+
- Pattern inconsistencies
|
|
144
|
+
- Minor improvements
|
|
145
|
+
- Each with specific recommendation
|
|
146
|
+
|
|
147
|
+
**Final Assessment**:
|
|
148
|
+
- If CRITICAL issues: "X critical issue(s) found. Fix before archiving."
|
|
149
|
+
- If only warnings: "No critical issues. Y warning(s) to consider. Ready for archive (with noted improvements)."
|
|
150
|
+
- If all clear: "All checks passed. Ready for archive."
|
|
151
|
+
|
|
152
|
+
**Verification Heuristics**
|
|
153
|
+
|
|
154
|
+
- **Completeness**: Focus on objective checklist items (checkboxes, requirements list)
|
|
155
|
+
- **Correctness**: Use keyword search, file path analysis, reasonable inference - don't require perfect certainty
|
|
156
|
+
- **Coherence**: Look for glaring inconsistencies, don't nitpick style
|
|
157
|
+
- **False Positives**: When uncertain, prefer SUGGESTION over WARNING, WARNING over CRITICAL
|
|
158
|
+
- **Actionability**: Every issue must have a specific recommendation with file/line references where applicable
|
|
159
|
+
|
|
160
|
+
**Graceful Degradation**
|
|
161
|
+
|
|
162
|
+
- If only tasks.md exists: verify task completion only, skip spec/design checks
|
|
163
|
+
- If tasks + specs exist: verify completeness and correctness, skip design
|
|
164
|
+
- If full artifacts: verify all three dimensions
|
|
165
|
+
- Always note which checks were skipped and why
|
|
166
|
+
|
|
167
|
+
**Output Format**
|
|
168
|
+
|
|
169
|
+
Use clear markdown with:
|
|
170
|
+
- Table for summary scorecard
|
|
171
|
+
- Grouped lists for issues (CRITICAL/WARNING/SUGGESTION)
|
|
172
|
+
- Code references in format: `file.ts:123`
|
|
173
|
+
- Specific, actionable recommendations
|
|
174
|
+
- No vague suggestions like "consider reviewing"
|
package/package.json
ADDED
|
@@ -0,0 +1,46 @@
|
|
|
1
|
+
{
|
|
2
|
+
"name": "@jspg-ai/coding-bb",
|
|
3
|
+
"version": "0.0.1",
|
|
4
|
+
"description": "整合业界热门且高价值的工具、框架与技能,为 AI CODING AGENT 提供统一的行为准则与工作流,辅助开发者将需求高效落地为符合规范的代码",
|
|
5
|
+
"main": "cbb/lib/install/init.js",
|
|
6
|
+
"bin": {
|
|
7
|
+
"cbb": "cbb/lib/install/init.js"
|
|
8
|
+
},
|
|
9
|
+
"files": [
|
|
10
|
+
"cbb/",
|
|
11
|
+
"openspec/",
|
|
12
|
+
"config/",
|
|
13
|
+
"superpowers/",
|
|
14
|
+
"!**/__pycache__/",
|
|
15
|
+
"!**/*.pyc",
|
|
16
|
+
"!**/*.pyo"
|
|
17
|
+
],
|
|
18
|
+
"scripts": {
|
|
19
|
+
"sync:shared": "node scripts/sync-shared.js",
|
|
20
|
+
"pretest": "node scripts/sync-shared.js --check",
|
|
21
|
+
"test": "node test/lib/install/cleanup.test.js && node test/lib/utils/settings.test.js && node test/lib/utils/gitignore.test.js && node test/lib/utils/version.test.js && node test/lib/install/workspaces.test.js && node test/lib/utils/checkbox.test.js && node test/lib/install/cli-help.test.js && node test/lib/wiki/wiki-publish.test.js && node test/lib/wiki/wiki-parse-args.test.js && node test/lib/wiki/wiki-split.test.js && node test/lib/superpowers/superpowers.test.js && node test/worktrees/push-branches.test.js && node test/worktrees/commit-worktrees.test.js && node test/worktrees/push-worktrees.test.js && node test/worktrees/install-ai.test.js && node test/worktrees/install-ai-extend.test.js && node test/worktrees/check-env-deep.test.js && node test/lib/openspec/openspec-render.test.js"
|
|
22
|
+
},
|
|
23
|
+
"keywords": [
|
|
24
|
+
"cbb",
|
|
25
|
+
"coding-bb",
|
|
26
|
+
"ai",
|
|
27
|
+
"coding-assistant",
|
|
28
|
+
"openspec",
|
|
29
|
+
"superpowers",
|
|
30
|
+
"spec-driven",
|
|
31
|
+
"backend",
|
|
32
|
+
"java"
|
|
33
|
+
],
|
|
34
|
+
"author": "Contributors",
|
|
35
|
+
"license": "MIT",
|
|
36
|
+
"publishConfig": {
|
|
37
|
+
"registry": "https://registry.npmjs.org"
|
|
38
|
+
},
|
|
39
|
+
"engines": {
|
|
40
|
+
"node": ">=16.7.0"
|
|
41
|
+
},
|
|
42
|
+
"dependencies": {
|
|
43
|
+
"js-yaml": "^4.1.1",
|
|
44
|
+
"koffi": "^3.1.0"
|
|
45
|
+
}
|
|
46
|
+
}
|
|
@@ -0,0 +1,250 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: brainstorming
|
|
3
|
+
description: "You MUST use this before any creative work - creating features, building components, adding functionality, or modifying behavior. Explores user intent, requirements and design before implementation."
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Brainstorming Ideas Into Designs
|
|
7
|
+
|
|
8
|
+
Help turn ideas into fully formed designs and specs through natural collaborative dialogue.
|
|
9
|
+
|
|
10
|
+
Start by classifying how much process the request needs, then work
|
|
11
|
+
through your path: understand the context, refine the idea, present a
|
|
12
|
+
design, and get your human partner's approval.
|
|
13
|
+
|
|
14
|
+
<HARD-GATE>
|
|
15
|
+
Do NOT invoke any implementation skill, write any code, scaffold any
|
|
16
|
+
project, or take any implementation action until you have told your
|
|
17
|
+
human partner what you intend and they have approved it. This applies
|
|
18
|
+
to EVERY task on EVERY path below — the ceremony scales with the task;
|
|
19
|
+
the approval gate never does.
|
|
20
|
+
</HARD-GATE>
|
|
21
|
+
|
|
22
|
+
## Three Paths
|
|
23
|
+
|
|
24
|
+
Before your first question, classify the request and say the
|
|
25
|
+
classification out loud — "this looks bounded, so I'll present a short
|
|
26
|
+
design here rather than write a spec" — so your human partner can
|
|
27
|
+
override it:
|
|
28
|
+
|
|
29
|
+
- **Spike** — a feasibility question ("can we...", "is it possible...",
|
|
30
|
+
"quick and dirty is fine") whose output is an answer, not code you
|
|
31
|
+
keep. Present the question and what you'll try in 2-3 sentences, get
|
|
32
|
+
a nod, then find out as cheaply as correctness allows. No design
|
|
33
|
+
doc, no spec file. Report findings as a recommendation; anything you
|
|
34
|
+
built stays labeled throwaway.
|
|
35
|
+
- **Bounded** — a well-scoped change to code that already exists in
|
|
36
|
+
this repo: a new flag, a small endpoint, a one-file fix.
|
|
37
|
+
Understanding the kind of app is not enough — bounded means the flow
|
|
38
|
+
you are changing is already here to read. If there is no existing
|
|
39
|
+
flow to change, the task is not bounded. Ask the clarifying
|
|
40
|
+
questions that matter, present a short design IN CHAT (a few
|
|
41
|
+
sentences to a few short paragraphs), and STOP. Implementation
|
|
42
|
+
starts only after your human partner says yes to that design — a
|
|
43
|
+
bounded task's approval is as hard a gate as an architectural
|
|
44
|
+
one. No spec file, no implementation plan document.
|
|
45
|
+
- **Architectural** — new projects, new subsystems, changes that
|
|
46
|
+
restructure how components fit together or alter interfaces others
|
|
47
|
+
depend on. Follow the full process: questions, approaches, sectioned
|
|
48
|
+
design, written spec, then the writing-plans skill.
|
|
49
|
+
|
|
50
|
+
When in doubt between two paths, take the heavier one. The ratchet is
|
|
51
|
+
one-way: hidden complexity discovered mid-task upgrades the path —
|
|
52
|
+
stop, say so, and step up. Nothing downgrades mid-task.
|
|
53
|
+
|
|
54
|
+
## Anti-Pattern: "Too Simple To Need Approval"
|
|
55
|
+
|
|
56
|
+
Every path ends with your human partner approving your intent before
|
|
57
|
+
implementation. A todo list, a single-function utility, a config
|
|
58
|
+
change — the design may be two sentences in chat, but you MUST present
|
|
59
|
+
it and get approval. "Simple" tasks are where unexamined assumptions
|
|
60
|
+
cause the most wasted work. What scales with simplicity is the
|
|
61
|
+
artifact, never the approval.
|
|
62
|
+
|
|
63
|
+
## Red Flags
|
|
64
|
+
|
|
65
|
+
| Thought | Reality |
|
|
66
|
+
|---------|---------|
|
|
67
|
+
| "This is too simple to need a design" | Simple means a short design, not no design. Two sentences in chat, then approval. |
|
|
68
|
+
| "I'll call it bounded and skip the spec" | Reaching for a label to skip work IS the doubt — take the heavier path. |
|
|
69
|
+
| "It's bounded and the design is obvious — I'll start while they read it" | The gate is the approval, not the design's length. Present, then stop until you hear yes. |
|
|
70
|
+
| "I understand this kind of app, so it's bounded" | Bounded measures the repo, not your familiarity. A new project has no existing flow — it is architectural. |
|
|
71
|
+
| "The spike works, so I'll keep the code" | A spike's output is an answer. Keeping the code is a new request — classify it. |
|
|
72
|
+
| "It grew, but I'm almost done — no need to re-classify" | Hidden complexity upgrades the path mid-task. Stop and say so. |
|
|
73
|
+
| "They approved the spike, so the follow-up change is approved too" | Each task gets its own classification and its own approval. |
|
|
74
|
+
|
|
75
|
+
## Checklist
|
|
76
|
+
|
|
77
|
+
Classify first, announce the path, then create a task for each item on
|
|
78
|
+
your path and complete them in order.
|
|
79
|
+
|
|
80
|
+
**Spike:**
|
|
81
|
+
1. **Explore project context** — enough to frame the probe
|
|
82
|
+
2. **Present question + probe plan** — 2-3 sentences
|
|
83
|
+
3. **Get approval** — a nod is enough
|
|
84
|
+
4. **Investigate** — as cheaply as correctness allows
|
|
85
|
+
5. **Report findings** — a recommendation; label anything built as throwaway
|
|
86
|
+
|
|
87
|
+
**Bounded:**
|
|
88
|
+
1. **Explore project context** — check files, docs, recent commits
|
|
89
|
+
2. **Ask clarifying questions** — one at a time, the ones that matter
|
|
90
|
+
3. **Present short design in chat** — approach, files touched, testing
|
|
91
|
+
4. **Get approval** — STOP and wait for an explicit yes; presenting the design and starting in the same breath is skipping the gate
|
|
92
|
+
5. **Implement** — proceed with the normal development workflow (TDD applies); no plan document
|
|
93
|
+
|
|
94
|
+
**Architectural:**
|
|
95
|
+
1. **Explore project context** — check files, docs, recent commits
|
|
96
|
+
2. **Offer the visual companion just-in-time** — NOT upfront. The first time a question would genuinely be clearer shown than described, offer it then (its own message); on approval its browser tab opens for you. If no visual question ever arises, never offer it. See the Visual Companion section below.
|
|
97
|
+
3. **Ask clarifying questions** — one at a time, understand purpose/constraints/success criteria
|
|
98
|
+
4. **Propose 2-3 approaches** — with trade-offs and your recommendation
|
|
99
|
+
5. **Present design** — in sections scaled to their complexity, get user approval after each section
|
|
100
|
+
6. **Write design doc** — save to `docs/superpowers/specs/YYYY-MM-DD-<topic>-design.md` and commit
|
|
101
|
+
7. **Spec self-review** — quick inline check for placeholders, contradictions, ambiguity, scope (see below)
|
|
102
|
+
8. **User reviews written spec** — ask user to review the spec file before proceeding
|
|
103
|
+
9. **Transition to implementation** — invoke writing-plans skill to create implementation plan
|
|
104
|
+
|
|
105
|
+
## Process Flow
|
|
106
|
+
|
|
107
|
+
```dot
|
|
108
|
+
digraph brainstorming {
|
|
109
|
+
"Classify: spike / bounded / architectural" [shape=diamond];
|
|
110
|
+
"Present question + probe (2-3 sentences)" [shape=box];
|
|
111
|
+
"Ask clarifying questions (bounded)" [shape=box];
|
|
112
|
+
"Present short design in chat" [shape=box];
|
|
113
|
+
"Human approves?" [shape=diamond];
|
|
114
|
+
"Investigate; report recommendation" [shape=doublecircle];
|
|
115
|
+
"Implement via normal workflow (no plan doc)" [shape=doublecircle];
|
|
116
|
+
"Explore project context" [shape=box];
|
|
117
|
+
"Ask clarifying questions" [shape=box];
|
|
118
|
+
"Propose 2-3 approaches" [shape=box];
|
|
119
|
+
"Present design sections" [shape=box];
|
|
120
|
+
"User approves design?" [shape=diamond];
|
|
121
|
+
"Write design doc" [shape=box];
|
|
122
|
+
"Spec self-review\n(fix inline)" [shape=box];
|
|
123
|
+
"User reviews spec?" [shape=diamond];
|
|
124
|
+
"Invoke writing-plans skill" [shape=doublecircle];
|
|
125
|
+
"Hidden complexity? Upgrade path" [shape=box];
|
|
126
|
+
|
|
127
|
+
"Classify: spike / bounded / architectural" -> "Present question + probe (2-3 sentences)" [label="spike"];
|
|
128
|
+
"Classify: spike / bounded / architectural" -> "Ask clarifying questions (bounded)" [label="bounded"];
|
|
129
|
+
"Classify: spike / bounded / architectural" -> "Explore project context" [label="architectural"];
|
|
130
|
+
"Present question + probe (2-3 sentences)" -> "Human approves?";
|
|
131
|
+
"Ask clarifying questions (bounded)" -> "Present short design in chat";
|
|
132
|
+
"Present short design in chat" -> "Human approves?";
|
|
133
|
+
"Human approves?" -> "Investigate; report recommendation" [label="spike: yes"];
|
|
134
|
+
"Human approves?" -> "Implement via normal workflow (no plan doc)" [label="bounded: yes"];
|
|
135
|
+
"Hidden complexity? Upgrade path" -> "Classify: spike / bounded / architectural";
|
|
136
|
+
"Explore project context" -> "Ask clarifying questions";
|
|
137
|
+
"Ask clarifying questions" -> "Propose 2-3 approaches";
|
|
138
|
+
"Propose 2-3 approaches" -> "Present design sections";
|
|
139
|
+
"Present design sections" -> "User approves design?";
|
|
140
|
+
"User approves design?" -> "Present design sections" [label="no, revise"];
|
|
141
|
+
"User approves design?" -> "Write design doc" [label="yes"];
|
|
142
|
+
"Write design doc" -> "Spec self-review\n(fix inline)";
|
|
143
|
+
"Spec self-review\n(fix inline)" -> "User reviews spec?";
|
|
144
|
+
"User reviews spec?" -> "Write design doc" [label="changes requested"];
|
|
145
|
+
"User reviews spec?" -> "Invoke writing-plans skill" [label="approved"];
|
|
146
|
+
}
|
|
147
|
+
```
|
|
148
|
+
|
|
149
|
+
**Terminal states are path-bound.** Architectural: the ONLY skill you
|
|
150
|
+
invoke after brainstorming is writing-plans — never frontend-design,
|
|
151
|
+
mcp-builder, or any other implementation skill. Bounded: after
|
|
152
|
+
approval, implementation proceeds directly through the normal
|
|
153
|
+
development workflow; no plan document. Spike: the terminal state is a
|
|
154
|
+
reported recommendation.
|
|
155
|
+
|
|
156
|
+
## The Process
|
|
157
|
+
|
|
158
|
+
The subsections below serve the bounded and architectural paths (a
|
|
159
|
+
spike stops at "present the probe, get a nod"). Sections from
|
|
160
|
+
**Exploring approaches** onward are architectural-path depth — for
|
|
161
|
+
bounded work, context plus a few questions plus a short in-chat design
|
|
162
|
+
is the whole process.
|
|
163
|
+
|
|
164
|
+
**Understanding the idea:**
|
|
165
|
+
|
|
166
|
+
- Check out the current project state first (files, docs, recent commits)
|
|
167
|
+
- Before asking detailed questions, assess scope: if the request describes multiple independent subsystems (e.g., "build a platform with chat, file storage, billing, and analytics"), flag this immediately. Don't spend questions refining details of a project that needs to be decomposed first.
|
|
168
|
+
- If the project is too large for a single spec, help the user decompose into sub-projects: what are the independent pieces, how do they relate, what order should they be built? Then brainstorm the first sub-project through the normal design flow. Each sub-project gets its own spec → plan → implementation cycle.
|
|
169
|
+
- For appropriately-scoped projects, ask questions one at a time to refine the idea
|
|
170
|
+
- Prefer multiple choice questions when possible, but open-ended is fine too
|
|
171
|
+
- Only one question per message - if a topic needs more exploration, break it into multiple questions
|
|
172
|
+
- Focus on understanding: purpose, constraints, success criteria
|
|
173
|
+
|
|
174
|
+
**Exploring approaches:**
|
|
175
|
+
|
|
176
|
+
- Propose 2-3 different approaches with trade-offs
|
|
177
|
+
- Present options conversationally with your recommendation and reasoning
|
|
178
|
+
- Lead with your recommended option and explain why
|
|
179
|
+
- YAGNI ruthlessly - remove unnecessary features from every approach and design
|
|
180
|
+
|
|
181
|
+
**Presenting the design:**
|
|
182
|
+
|
|
183
|
+
- Once you believe you understand what you're building, present the design
|
|
184
|
+
- Scale each section to its complexity: a few sentences if straightforward, up to 200-300 words if nuanced
|
|
185
|
+
- Ask after each section whether it looks right so far
|
|
186
|
+
- Cover: architecture, components, data flow, error handling, testing
|
|
187
|
+
- Be ready to go back and clarify if something doesn't make sense
|
|
188
|
+
|
|
189
|
+
**Design for isolation and clarity:**
|
|
190
|
+
|
|
191
|
+
- Break the system into smaller units that each have one clear purpose, communicate through well-defined interfaces, and can be understood and tested independently
|
|
192
|
+
- For each unit, you should be able to answer: what does it do, how do you use it, and what does it depend on?
|
|
193
|
+
- Can someone understand what a unit does without reading its internals? Can you change the internals without breaking consumers? If not, the boundaries need work.
|
|
194
|
+
- Smaller, well-bounded units are also easier for you to work with - you reason better about code you can hold in context at once, and your edits are more reliable when files are focused. When a file grows large, that's often a signal that it's doing too much.
|
|
195
|
+
|
|
196
|
+
**Working in existing codebases:**
|
|
197
|
+
|
|
198
|
+
- Explore the current structure before proposing changes. Follow existing patterns.
|
|
199
|
+
- Where existing code has problems that affect the work (e.g., a file that's grown too large, unclear boundaries, tangled responsibilities), include targeted improvements as part of the design - the way a good developer improves code they're working in.
|
|
200
|
+
- Don't propose unrelated refactoring. Stay focused on what serves the current goal.
|
|
201
|
+
|
|
202
|
+
## After the Design (architectural path)
|
|
203
|
+
|
|
204
|
+
**Documentation:**
|
|
205
|
+
|
|
206
|
+
- Write the validated design (spec) to `docs/superpowers/specs/YYYY-MM-DD-<topic>-design.md`
|
|
207
|
+
- (User preferences for spec location override this default)
|
|
208
|
+
- Use elements-of-style:writing-clearly-and-concisely skill if available
|
|
209
|
+
- Commit the design document to git
|
|
210
|
+
|
|
211
|
+
**Spec Self-Review:**
|
|
212
|
+
After writing the spec document, look at it with fresh eyes:
|
|
213
|
+
|
|
214
|
+
1. **Placeholder scan:** Any "TBD", "TODO", incomplete sections, or vague requirements? Fix them.
|
|
215
|
+
2. **Internal consistency:** Do any sections contradict each other? Does the architecture match the feature descriptions?
|
|
216
|
+
3. **Scope check:** Is this focused enough for a single implementation plan, or does it need decomposition?
|
|
217
|
+
4. **Ambiguity check:** Could any requirement be interpreted two different ways? If so, pick one and make it explicit.
|
|
218
|
+
|
|
219
|
+
Fix any issues inline. No need to re-review — just fix and move on.
|
|
220
|
+
|
|
221
|
+
**User Review Gate:**
|
|
222
|
+
After the spec review loop passes, ask the user to review the written spec before proceeding:
|
|
223
|
+
|
|
224
|
+
> "Spec written and committed to `<path>`. Please review it and let me know if you want to make any changes before we start writing out the implementation plan."
|
|
225
|
+
|
|
226
|
+
Wait for the user's response. If they request changes, make them and re-run the spec review loop. Only proceed once the user approves.
|
|
227
|
+
|
|
228
|
+
**Implementation:**
|
|
229
|
+
|
|
230
|
+
- Invoke the writing-plans skill to create a detailed implementation plan
|
|
231
|
+
- Do NOT invoke any other skill. writing-plans is the next step.
|
|
232
|
+
|
|
233
|
+
## Visual Companion
|
|
234
|
+
|
|
235
|
+
A browser-based companion for showing mockups, diagrams, and visual options during brainstorming. Available as a tool — not a mode. Accepting the companion means it's available for questions that benefit from visual treatment; it does NOT mean every question goes through the browser.
|
|
236
|
+
|
|
237
|
+
**Offering the companion (just-in-time):** Do NOT offer it upfront. Wait until a question would genuinely be clearer shown than told — a real mockup / layout / diagram question, not merely a UI *topic*. The first time that happens, offer it then, as its own message:
|
|
238
|
+
> "This next part might be easier if I show you — I can put together mockups, diagrams, and comparisons in a browser tab as we go. It's still new and can be token-intensive. Want me to? I'll open it for you."
|
|
239
|
+
|
|
240
|
+
**This offer MUST be its own message.** Only the offer — no clarifying question, summary, or other content. Wait for the user's response. If they accept, start the server with `--open` so their browser opens to the first screen automatically. If they decline, continue text-only and don't offer again unless they raise it.
|
|
241
|
+
|
|
242
|
+
**Per-question decision:** Even after the user accepts, decide FOR EACH QUESTION whether to use the browser or the terminal. The test: **would the user understand this better by seeing it than reading it?**
|
|
243
|
+
|
|
244
|
+
- **Use the browser** for content that IS visual — mockups, wireframes, layout comparisons, architecture diagrams, side-by-side visual designs
|
|
245
|
+
- **Use the terminal** for content that is text — requirements questions, conceptual choices, tradeoff lists, A/B/C/D text options, scope decisions
|
|
246
|
+
|
|
247
|
+
A question about a UI topic is not automatically a visual question. "What does personality mean in this context?" is a conceptual question — use the terminal. "Which wizard layout works better?" is a visual question — use the browser.
|
|
248
|
+
|
|
249
|
+
If they agree to the companion, read the detailed guide before proceeding:
|
|
250
|
+
`skills/brainstorming/visual-companion.md`
|