@hopla/claude-setup 1.12.1 → 1.14.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/.claude-plugin/marketplace.json +1 -1
- package/.claude-plugin/plugin.json +1 -1
- package/README.md +209 -144
- package/agents/system-reviewer.md +13 -0
- package/cli.js +284 -399
- package/commands/execute.md +22 -3
- package/commands/guide.md +6 -0
- package/commands/init-project.md +9 -7
- package/commands/plan-feature.md +16 -0
- package/commands/rca.md +7 -2
- package/global-rules.md +18 -50
- package/hooks/session-prime.js +39 -12
- package/package.json +1 -1
- package/skills/brainstorm/SKILL.md +5 -0
- package/skills/code-review/SKILL.md +3 -1
- package/skills/debug/SKILL.md +8 -0
- package/skills/execution-report/SKILL.md +1 -1
- package/skills/git/commit.md +20 -3
- package/skills/git/flow-detection.md +68 -0
- package/skills/git/pr.md +46 -16
- package/skills/verify/SKILL.md +10 -0
- package/skills/worktree/SKILL.md +83 -38
- package/commands/end-to-end.md +0 -67
- package/commands/git-commit.md +0 -76
- package/commands/git-pr.md +0 -138
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "hopla",
|
|
3
3
|
"description": "Agentic coding system for Claude Code: PIV loop (Plan → Implement → Validate), TDD, debugging, brainstorming, subagent execution, and team workflows",
|
|
4
|
-
"version": "1.
|
|
4
|
+
"version": "1.14.0",
|
|
5
5
|
"author": {
|
|
6
6
|
"name": "Hopla Tools",
|
|
7
7
|
"email": "julio@hopla.tools"
|
package/README.md
CHANGED
|
@@ -1,107 +1,118 @@
|
|
|
1
1
|
# @hopla/claude-setup
|
|
2
2
|
|
|
3
|
-
Hopla team agentic coding system for Claude Code.
|
|
3
|
+
Hopla team agentic coding system for Claude Code. Delivers commands, skills, agents, hooks, and reference guides via a **Claude Code plugin** (primary channel), with an optional **npm CLI** for the machine-level global rules template.
|
|
4
4
|
|
|
5
|
-
|
|
5
|
+
---
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
## Prerequisites
|
|
8
8
|
|
|
9
|
-
|
|
9
|
+
- **Claude Code CLI** — install from https://claude.com/claude-code
|
|
10
|
+
- **Node.js ≥18** — required for the npm CLI (`node --version`)
|
|
11
|
+
- **git** — plugin installs clone this repo
|
|
10
12
|
|
|
11
|
-
|
|
12
|
-
/plugin marketplace add hopla-marketplace HOPLAtools/claude-setup
|
|
13
|
-
```
|
|
13
|
+
---
|
|
14
14
|
|
|
15
|
-
|
|
15
|
+
## Quick Start
|
|
16
16
|
|
|
17
|
-
|
|
18
|
-
/plugin install hopla@hopla-marketplace
|
|
19
|
-
```
|
|
17
|
+
The full setup is **plugin + CLI** — they deliver different layers:
|
|
20
18
|
|
|
21
|
-
|
|
19
|
+
| Layer | Channel | What it provides |
|
|
20
|
+
|---|---|---|
|
|
21
|
+
| Per-project behavior | Plugin | Commands, skills, agents, hooks |
|
|
22
|
+
| Machine-wide defaults | CLI | `~/.claude/CLAUDE.md` global rules + bash permissions |
|
|
22
23
|
|
|
23
|
-
|
|
24
|
+
**Step 1 — Register the marketplace and install the plugin** (inside Claude Code):
|
|
24
25
|
|
|
25
|
-
```
|
|
26
|
-
|
|
27
|
-
|
|
26
|
+
```
|
|
27
|
+
/plugin marketplace add hopla-marketplace HOPLAtools/claude-setup
|
|
28
|
+
/plugin install hopla@hopla-marketplace
|
|
28
29
|
```
|
|
29
30
|
|
|
30
|
-
|
|
31
|
-
|
|
32
|
-
### Option B: npm CLI only
|
|
31
|
+
**Step 2 — Install the global rules template** (one-time, terminal):
|
|
33
32
|
|
|
34
33
|
```bash
|
|
35
34
|
npm install -g @hopla/claude-setup
|
|
36
|
-
claude-setup
|
|
35
|
+
claude-setup --force
|
|
37
36
|
```
|
|
38
37
|
|
|
39
|
-
|
|
38
|
+
That's it. Commands show as `/hopla:<name>`, skills auto-trigger when relevant.
|
|
40
39
|
|
|
41
|
-
###
|
|
40
|
+
### Plugin only (skip global rules)
|
|
42
41
|
|
|
43
|
-
|
|
44
|
-
npm install -g @hopla/claude-setup
|
|
45
|
-
claude-setup --planning
|
|
46
|
-
```
|
|
42
|
+
If you don't want the machine-wide `~/.claude/CLAUDE.md`, just do Step 1. The plugin works on its own.
|
|
47
43
|
|
|
48
|
-
|
|
44
|
+
### CLI only (no plugin)
|
|
49
45
|
|
|
50
|
-
|
|
46
|
+
If you only want the global rules and not the plugin's commands/skills, just do Step 2. You won't have `/hopla:*` commands.
|
|
51
47
|
|
|
52
|
-
|
|
53
|
-
claude-setup --force
|
|
54
|
-
claude-setup --planning --force
|
|
55
|
-
```
|
|
48
|
+
---
|
|
56
49
|
|
|
57
50
|
## Update
|
|
58
51
|
|
|
59
52
|
### Plugin channel
|
|
60
53
|
|
|
61
|
-
Claude Code caches the marketplace repo locally. To
|
|
54
|
+
Claude Code caches the marketplace repo locally. To pick up new versions:
|
|
62
55
|
|
|
63
56
|
```bash
|
|
64
|
-
# Step 1: Update the local marketplace cache
|
|
65
57
|
cd ~/.claude/plugins/marketplaces/hopla-marketplace && git pull
|
|
58
|
+
```
|
|
59
|
+
|
|
60
|
+
Then inside Claude Code:
|
|
66
61
|
|
|
67
|
-
|
|
62
|
+
```
|
|
68
63
|
/plugin uninstall hopla@hopla-marketplace
|
|
69
64
|
/plugin install hopla@hopla-marketplace
|
|
70
65
|
/reload-plugins
|
|
71
66
|
```
|
|
72
67
|
|
|
73
|
-
> **Known issue:** Claude Code does not automatically `git pull` the marketplace when reinstalling a plugin. The manual `git pull`
|
|
68
|
+
> **Known issue:** Claude Code does not automatically `git pull` the marketplace when reinstalling a plugin. The manual `git pull` above is required.
|
|
69
|
+
|
|
70
|
+
If the `cd` path does not exist, you never installed the marketplace — do Step 1 of Quick Start instead.
|
|
74
71
|
|
|
75
|
-
### CLI channel (
|
|
72
|
+
### CLI channel (global rules)
|
|
76
73
|
|
|
77
74
|
```bash
|
|
78
75
|
npm install -g @hopla/claude-setup@latest --prefer-online && claude-setup --force
|
|
79
76
|
```
|
|
80
77
|
|
|
78
|
+
---
|
|
79
|
+
|
|
81
80
|
## Uninstall
|
|
82
81
|
|
|
83
82
|
**Plugin:**
|
|
83
|
+
|
|
84
84
|
```
|
|
85
85
|
/plugin uninstall hopla@hopla-marketplace
|
|
86
86
|
```
|
|
87
87
|
|
|
88
88
|
**CLI:**
|
|
89
|
+
|
|
89
90
|
```bash
|
|
90
91
|
claude-setup --uninstall
|
|
91
92
|
```
|
|
92
93
|
|
|
94
|
+
Removes `~/.claude/CLAUDE.md` plus legacy `hopla-*` files from older installs.
|
|
95
|
+
|
|
96
|
+
### CLI flags reference
|
|
97
|
+
|
|
98
|
+
| Flag | Purpose |
|
|
99
|
+
|---|---|
|
|
100
|
+
| `claude-setup` | Interactive install of global rules + permissions |
|
|
101
|
+
| `claude-setup --force` | Install without prompts |
|
|
102
|
+
| `claude-setup --migrate` | Remove legacy CLI-installed duplicates only |
|
|
103
|
+
| `claude-setup --uninstall` | Remove global rules + legacy files |
|
|
104
|
+
| `claude-setup --version` | Print package version |
|
|
105
|
+
|
|
93
106
|
---
|
|
94
107
|
|
|
95
108
|
## Naming Convention
|
|
96
109
|
|
|
97
|
-
Skills and commands use short names in source (e.g., `prime`, `execute`, `git`).
|
|
110
|
+
Skills and commands use short names in source (e.g., `prime`, `execute`, `git`). The plugin namespaces them automatically:
|
|
98
111
|
|
|
99
|
-
|
|
|
100
|
-
|
|
101
|
-
| **
|
|
102
|
-
| **
|
|
103
|
-
|
|
104
|
-
The plugin channel uses the plugin name (`hopla:`) as namespace. The CLI channel adds a `hopla-` prefix during installation.
|
|
112
|
+
| Type | Example |
|
|
113
|
+
|---|---|
|
|
114
|
+
| **Skills** | `hopla:prime`, `hopla:git`, `hopla:debug` |
|
|
115
|
+
| **Commands** | `/hopla:execute`, `/hopla:plan-feature` |
|
|
105
116
|
|
|
106
117
|
---
|
|
107
118
|
|
|
@@ -110,19 +121,19 @@ The plugin channel uses the plugin name (`hopla:`) as namespace. The CLI channel
|
|
|
110
121
|
The system uses three levels of CLAUDE.md, each scoped differently:
|
|
111
122
|
|
|
112
123
|
```
|
|
113
|
-
~/.claude/CLAUDE.md ← Machine-level (installed by
|
|
124
|
+
~/.claude/CLAUDE.md ← Machine-level (installed by CLI)
|
|
114
125
|
└── applies to ALL projects on this machine
|
|
115
126
|
|
|
116
|
-
CLAUDE.md (project root) ← Project-level (created with /init-project)
|
|
127
|
+
CLAUDE.md (project root) ← Project-level (created with /hopla:init-project)
|
|
117
128
|
└── applies to THIS project only
|
|
118
129
|
|
|
119
130
|
.claude/CLAUDE.local.md ← Local overrides (personal, gitignored)
|
|
120
131
|
└── your personal tweaks, not shared with team
|
|
121
132
|
```
|
|
122
133
|
|
|
123
|
-
**Machine-level rules** cover: language preferences, tech defaults,
|
|
134
|
+
**Machine-level rules** cover: language preferences, tech defaults, autonomy behavior, context management tips.
|
|
124
135
|
|
|
125
|
-
**Project-level rules** cover: specific stack versions, architecture patterns, naming conventions, logging, testing, dev commands,
|
|
136
|
+
**Project-level rules** cover: specific stack versions, architecture patterns, naming conventions, logging, testing, dev commands, task-specific reference guides.
|
|
126
137
|
|
|
127
138
|
**Local overrides** cover: personal preferences that differ from the team (e.g., verbose logging, different editor settings).
|
|
128
139
|
|
|
@@ -130,20 +141,20 @@ CLAUDE.md (project root) ← Project-level (created with /init-project)
|
|
|
130
141
|
|
|
131
142
|
## The Agentic Coding Framework
|
|
132
143
|
|
|
133
|
-
|
|
144
|
+
Built on two core concepts from the Agentic Coding Course:
|
|
134
145
|
|
|
135
146
|
### AI Layer — What guides the agent
|
|
136
147
|
|
|
137
|
-
| Pillar | What it is |
|
|
138
|
-
|
|
139
|
-
| **Global Rules** | Always-loaded context: language, git flow, tech defaults, autonomy
|
|
140
|
-
| **On-Demand Context** | Task-specific guides loaded when needed
|
|
141
|
-
| **Commands** | Reusable processes that tell the agent *how* to work |
|
|
142
|
-
| **Skills** | Auto-activate by semantic matching — no slash command needed |
|
|
143
|
-
| **Agents** | Specialized subagents for delegation (code review, research, system analysis) |
|
|
144
|
-
| **Hooks** | Run automatically before/after tool use for type checking and protection |
|
|
148
|
+
| Pillar | What it is | Delivered by |
|
|
149
|
+
|---|---|---|
|
|
150
|
+
| **Global Rules** | Always-loaded context: language, git flow, tech defaults, autonomy | CLI → `~/.claude/CLAUDE.md` |
|
|
151
|
+
| **On-Demand Context** | Task-specific guides loaded when needed | Project → `.agents/guides/*.md` |
|
|
152
|
+
| **Commands** | Reusable processes that tell the agent *how* to work | Plugin |
|
|
153
|
+
| **Skills** | Auto-activate by semantic matching — no slash command needed | Plugin |
|
|
154
|
+
| **Agents** | Specialized subagents for delegation (code review, research, system analysis) | Plugin |
|
|
155
|
+
| **Hooks** | Run automatically before/after tool use for type checking and protection | Plugin |
|
|
145
156
|
|
|
146
|
-
The key insight: **commands inject on-demand context deterministically** — when you run `/plan-feature`, it automatically reads the relevant guide from `.agents/guides/` before planning.
|
|
157
|
+
The key insight: **commands inject on-demand context deterministically** — when you run `/hopla:plan-feature`, it automatically reads the relevant guide from `.agents/guides/` before planning.
|
|
147
158
|
|
|
148
159
|
### PIV Loop — How you work
|
|
149
160
|
|
|
@@ -151,19 +162,24 @@ The key insight: **commands inject on-demand context deterministically** — whe
|
|
|
151
162
|
Plan → Implement → Validate → (repeat)
|
|
152
163
|
```
|
|
153
164
|
|
|
154
|
-
- **Plan** (`/plan-feature`) — Research
|
|
155
|
-
- **Implement** (`/execute`) — Delegate coding to the AI, trust but verify at each task
|
|
165
|
+
- **Plan** (`/hopla:plan-feature`) — Research codebase, design approach, create a structured plan
|
|
166
|
+
- **Implement** (`/hopla:execute`) — Delegate coding to the AI, trust but verify at each task
|
|
156
167
|
- **Validate** — AI runs lint → types → tests → integration; human reviews the result
|
|
157
168
|
|
|
158
169
|
### System Evolution
|
|
159
170
|
|
|
160
|
-
After each PIV loop, run `/execution-report` + `/system-review` to find process improvements. Don't just fix bugs — fix the system that allowed them.
|
|
171
|
+
After each PIV loop, run `/hopla:execution-report` + `/hopla:system-review` to find process improvements. Don't just fix bugs — fix the system that allowed them.
|
|
161
172
|
|
|
162
173
|
---
|
|
163
174
|
|
|
164
175
|
## What Gets Installed
|
|
165
176
|
|
|
166
|
-
|
|
177
|
+
### From the CLI (machine-level, one-time)
|
|
178
|
+
|
|
179
|
+
- **`~/.claude/CLAUDE.md`** — Global rules applied to every Claude Code session
|
|
180
|
+
- **`~/.claude/settings.json`** — Bash permissions configured for common dev commands
|
|
181
|
+
|
|
182
|
+
### From the Plugin (per Claude Code session)
|
|
167
183
|
|
|
168
184
|
**Commands** — Reusable commands available in any project:
|
|
169
185
|
|
|
@@ -172,59 +188,56 @@ After each PIV loop, run `/execution-report` + `/system-review` to find process
|
|
|
172
188
|
| `init-project` | Read PRD, recommend stack, create CLAUDE.md and .agents/ structure |
|
|
173
189
|
| `create-prd` | Create a Product Requirements Document through guided questions |
|
|
174
190
|
| `plan-feature` | Research codebase and create a structured implementation plan |
|
|
175
|
-
| `review-plan` | Review a plan before execution — get a
|
|
191
|
+
| `review-plan` | Review a plan before execution — get a summary and approve |
|
|
176
192
|
| `execute` | Execute a structured plan from start to finish with validation |
|
|
177
193
|
| `validate` | Run the validation pyramid: lint → types → tests → integration |
|
|
178
|
-
| `end-to-end` | Full PIV loop in one command: prime → brainstorm → plan → execute → validate → commit |
|
|
179
|
-
| `git-commit` | Create a Conventional Commit with Git Flow awareness |
|
|
180
|
-
| `git-pr` | Create a GitHub Pull Request with a structured description |
|
|
181
194
|
| `code-review-fix` | Fix issues found in a code review report |
|
|
182
|
-
| `rca` | Root Cause Analysis — investigate a bug
|
|
183
|
-
| `guide` | 4D Framework
|
|
195
|
+
| `rca` | Root Cause Analysis — investigate a bug and generate an RCA doc |
|
|
196
|
+
| `guide` | 4D Framework walkthrough for non-technical users |
|
|
184
197
|
| `system-review` | Analyze implementation against plan to find process improvements |
|
|
185
198
|
|
|
186
|
-
>
|
|
199
|
+
> `prime`, `code-review`, and `execution-report` are **skills only** (no slash command needed).
|
|
187
200
|
|
|
188
|
-
**Skills** — Auto-activate by semantic matching
|
|
201
|
+
**Skills** — Auto-activate by semantic matching:
|
|
189
202
|
|
|
190
|
-
| Skill | Auto-activates when you say
|
|
203
|
+
| Skill | Auto-activates when you say… |
|
|
191
204
|
|---|---|
|
|
192
|
-
| `git` | "commit this", "create a PR", "
|
|
205
|
+
| `git` | "commit this", "create a PR", "push changes" |
|
|
206
|
+
| `worktree` | "use a worktree", "isolated branch", "parallel feature work" |
|
|
193
207
|
| `prime` | "orient yourself", "catch me up", "what is this project" |
|
|
194
208
|
| `code-review` | "review the code", "code review", "check these changes" |
|
|
195
|
-
| `execution-report` | "generate the report", "document what was done"
|
|
196
|
-
| `verify` | "verify it works", "make sure it's correct"
|
|
197
|
-
| `brainstorm` | "let's brainstorm", "explore approaches"
|
|
209
|
+
| `execution-report` | "generate the report", "document what was done" |
|
|
210
|
+
| `verify` | "verify it works", "make sure it's correct" |
|
|
211
|
+
| `brainstorm` | "let's brainstorm", "explore approaches" |
|
|
198
212
|
| `debug` | "debug this", "find the bug", "why is this failing" |
|
|
199
213
|
| `tdd` | "write tests first", "TDD", "red-green-refactor" |
|
|
200
|
-
| `subagent-execution` | "use subagents",
|
|
214
|
+
| `subagent-execution` | "use subagents", plans with 5+ tasks |
|
|
201
215
|
| `parallel-dispatch` | "run in parallel", "parallelize this", independent tasks |
|
|
202
|
-
| `worktree` | "use a worktree", "isolated branch", "parallel feature work" |
|
|
203
216
|
|
|
204
|
-
**Hooks** — Run automatically
|
|
217
|
+
**Hooks** — Run automatically:
|
|
205
218
|
|
|
206
219
|
| Hook | Type | What it does |
|
|
207
220
|
|---|---|---|
|
|
208
221
|
| `tsc-check.js` | PostToolUse | Runs `tsc --noEmit` after file edits; feeds errors back to Claude |
|
|
209
222
|
| `env-protect.js` | PreToolUse | Blocks reads/greps targeting `.env` files |
|
|
210
|
-
| `session-prime.js` | SessionStart
|
|
223
|
+
| `session-prime.js` | SessionStart | Loads git context + CLAUDE.md summary + skills list |
|
|
211
224
|
|
|
212
225
|
**Agents** — Specialized subagents for delegation:
|
|
213
226
|
|
|
214
227
|
| Agent | What it does |
|
|
215
228
|
|---|---|
|
|
216
|
-
| `code-reviewer` | Senior code reviewer (read-only).
|
|
217
|
-
| `codebase-researcher` | Fast codebase explorer (read-only). Systematic search
|
|
218
|
-
| `system-reviewer` | System review analyst (read-only).
|
|
229
|
+
| `code-reviewer` | Senior code reviewer (read-only). Plan alignment, quality, architecture, security |
|
|
230
|
+
| `codebase-researcher` | Fast codebase explorer (read-only). Systematic search, structured findings |
|
|
231
|
+
| `system-reviewer` | System review analyst (read-only). Execution vs plan, classifies divergences |
|
|
219
232
|
|
|
220
|
-
**Reference guides** — Loaded on-demand:
|
|
233
|
+
**Reference guides** — Loaded on-demand by commands:
|
|
221
234
|
|
|
222
235
|
| Guide | What it covers |
|
|
223
236
|
|---|---|
|
|
224
|
-
| `mcp-integration.md` |
|
|
237
|
+
| `mcp-integration.md` | Integrating MCP servers into the PIV loop |
|
|
225
238
|
| `ai-optimized-codebase.md` | Vertical slice architecture, LLM-friendly docstrings, strict types |
|
|
226
|
-
| `hooks-reference.md` | All hook types, configuration, input JSON, exit codes
|
|
227
|
-
| `write-skill.md` |
|
|
239
|
+
| `hooks-reference.md` | All hook types, configuration, input JSON, exit codes |
|
|
240
|
+
| `write-skill.md` | Creating new skills with CSO, testing, progressive disclosure |
|
|
228
241
|
| `remote-coding.md` | GitHub-based remote agentic coding with autonomy levels |
|
|
229
242
|
| `scaling-beyond-engineering.md` | Expanding HOPLA to non-technical teams with 4D Framework |
|
|
230
243
|
| `data-audit.md` | Data pipeline audit checklist |
|
|
@@ -235,126 +248,178 @@ After each PIV loop, run `/execution-report` + `/system-review` to find process
|
|
|
235
248
|
## Recommended Workflow
|
|
236
249
|
|
|
237
250
|
### Starting a new project
|
|
251
|
+
|
|
238
252
|
```
|
|
239
|
-
/create-prd
|
|
240
|
-
/init-project
|
|
241
|
-
|
|
253
|
+
/hopla:create-prd → define what you're building (PRD.md)
|
|
254
|
+
/hopla:init-project → reads PRD, recommends stack, creates CLAUDE.md + .agents/
|
|
255
|
+
"commit this" → git skill saves foundation
|
|
242
256
|
```
|
|
243
257
|
|
|
244
258
|
### Feature development (PIV loop)
|
|
245
|
-
```
|
|
246
|
-
"catch me up" → prime skill auto-loads project context
|
|
247
|
-
/plan-feature → research codebase and create plan
|
|
248
|
-
/review-plan → review plan summary and approve
|
|
249
|
-
/execute → implement the plan with validation
|
|
250
|
-
/validate → run lint → types → tests → integration
|
|
251
|
-
"review the code" → code-review skill runs automatically
|
|
252
|
-
/code-review-fix → fix issues found
|
|
253
|
-
/rca → root cause analysis if a bug is found
|
|
254
|
-
"generate the report" → execution-report skill documents what was built
|
|
255
|
-
/git-commit → save to git
|
|
256
|
-
/git-pr → open pull request on GitHub
|
|
257
|
-
```
|
|
258
259
|
|
|
259
|
-
### Full automation (one command)
|
|
260
260
|
```
|
|
261
|
-
|
|
261
|
+
"catch me up" → prime skill loads project context
|
|
262
|
+
"use a worktree" → worktree skill (optional — for isolation or parallel work)
|
|
263
|
+
/hopla:plan-feature → research codebase and create plan
|
|
264
|
+
/hopla:review-plan → review summary and approve
|
|
265
|
+
/hopla:execute → implement plan with validation
|
|
266
|
+
/hopla:validate → lint → types → tests → integration
|
|
267
|
+
"review the code" → code-review skill runs automatically
|
|
268
|
+
/hopla:code-review-fix → fix issues found
|
|
269
|
+
"generate the report" → execution-report skill documents what was built
|
|
270
|
+
"commit this" → git skill handles commits and PRs
|
|
271
|
+
(cleanup including worktree removal happens post-merge)
|
|
262
272
|
```
|
|
263
273
|
|
|
274
|
+
**When to reach for `worktree`:** risky refactors, parallel feature work, or when you need a clean baseline while keeping the current branch untouched. The skill auto-resolves the correct base branch (`feature/*` from `develop`, `hotfix/*` from `main`, etc.).
|
|
275
|
+
|
|
264
276
|
### After implementation
|
|
277
|
+
|
|
265
278
|
```
|
|
266
|
-
/system-review
|
|
279
|
+
/hopla:system-review → analyze plan vs. actual for process improvements
|
|
267
280
|
```
|
|
268
281
|
|
|
269
282
|
### For non-technical users
|
|
283
|
+
|
|
270
284
|
```
|
|
271
|
-
/guide
|
|
285
|
+
/hopla:guide → 4D Framework walkthrough (Description, Discernment, Delegation, Diligence)
|
|
272
286
|
```
|
|
273
287
|
|
|
274
|
-
> **Tip:** Many commands also exist as skills — they auto-activate when you describe what you want in natural language.
|
|
288
|
+
> **Tip:** Many commands also exist as skills — they auto-activate when you describe what you want in natural language. Say "debug this" to trigger the `debug` skill, "let's brainstorm" for `brainstorm`, without typing any slash command.
|
|
275
289
|
|
|
276
290
|
---
|
|
277
291
|
|
|
278
292
|
## Command Chaining
|
|
279
293
|
|
|
280
|
-
Commands are modular — the output of one becomes the input of the next. Some
|
|
294
|
+
Commands are modular — the output of one becomes the input of the next. Some accept arguments (`$1`, `$2`) to receive files generated by previous commands.
|
|
281
295
|
|
|
282
296
|
### Commands that accept arguments
|
|
283
297
|
|
|
284
298
|
| Command | Argument | Example |
|
|
285
299
|
|---|---|---|
|
|
286
|
-
| `/execute` |
|
|
287
|
-
| `/
|
|
288
|
-
| `/
|
|
289
|
-
| `/
|
|
290
|
-
| `/system-review` | Plan file + execution report | `/system-review .agents/plans/auth-feature.md .agents/execution-reports/auth-feature.md` |
|
|
300
|
+
| `/hopla:execute` | Plan file path | `/hopla:execute .agents/plans/auth-feature.md` |
|
|
301
|
+
| `/hopla:code-review-fix` | Review report path | `/hopla:code-review-fix .agents/code-reviews/auth-review.md` |
|
|
302
|
+
| `/hopla:rca` | Bug description | `/hopla:rca "login fails with 403 after token refresh"` |
|
|
303
|
+
| `/hopla:system-review` | Plan + report | `/hopla:system-review .agents/plans/auth.md .agents/execution-reports/auth.md` |
|
|
291
304
|
|
|
292
305
|
### Full PIV loop example
|
|
293
306
|
|
|
294
307
|
```
|
|
295
308
|
# 1. Plan
|
|
296
|
-
/plan-feature add user authentication
|
|
309
|
+
/hopla:plan-feature add user authentication
|
|
297
310
|
→ saves: .agents/plans/add-user-authentication.md
|
|
298
311
|
|
|
299
|
-
# 2. Review
|
|
300
|
-
/review-plan .agents/plans/add-user-authentication.md
|
|
312
|
+
# 2. Review
|
|
313
|
+
/hopla:review-plan .agents/plans/add-user-authentication.md
|
|
301
314
|
|
|
302
315
|
# 3. Execute
|
|
303
|
-
/execute .agents/plans/add-user-authentication.md
|
|
304
|
-
→ implements the plan, runs validation
|
|
316
|
+
/hopla:execute .agents/plans/add-user-authentication.md
|
|
305
317
|
|
|
306
318
|
# 4. Validate
|
|
307
|
-
/validate
|
|
308
|
-
→ runs lint → types → tests → integration
|
|
319
|
+
/hopla:validate
|
|
309
320
|
|
|
310
|
-
# 5. Code review (
|
|
321
|
+
# 5. Code review (skill — just say "review the code")
|
|
311
322
|
→ saves: .agents/code-reviews/add-user-authentication.md
|
|
312
323
|
|
|
313
324
|
# 6. Fix issues
|
|
314
|
-
/code-review-fix .agents/code-reviews/add-user-authentication.md
|
|
325
|
+
/hopla:code-review-fix .agents/code-reviews/add-user-authentication.md
|
|
315
326
|
|
|
316
|
-
# 7. Document (
|
|
327
|
+
# 7. Document (skill — just say "generate the report")
|
|
317
328
|
→ saves: .agents/execution-reports/add-user-authentication.md
|
|
318
329
|
|
|
319
|
-
# 8. Commit
|
|
320
|
-
/git-commit
|
|
330
|
+
# 8. Commit and PR (skill — "commit this", then "create a PR")
|
|
321
331
|
|
|
322
|
-
# 9.
|
|
323
|
-
/
|
|
332
|
+
# 9. Process improvement
|
|
333
|
+
/hopla:system-review .agents/plans/add-user-authentication.md .agents/execution-reports/add-user-authentication.md
|
|
334
|
+
```
|
|
324
335
|
|
|
325
|
-
|
|
326
|
-
|
|
336
|
+
---
|
|
337
|
+
|
|
338
|
+
## Troubleshooting
|
|
339
|
+
|
|
340
|
+
### `/plugin marketplace add` fails with "repo not found"
|
|
341
|
+
|
|
342
|
+
- Confirm the repo URL: `HOPLAtools/claude-setup` (case-sensitive)
|
|
343
|
+
- Check your network — `/plugin marketplace add` does a `git clone` under the hood
|
|
344
|
+
- If you were added to `HOPLAtools` recently, re-authenticate: `gh auth login`
|
|
345
|
+
|
|
346
|
+
### `/plugin install` doesn't show the `hopla` plugin
|
|
347
|
+
|
|
348
|
+
- Run `/plugin marketplace list` — confirm `hopla-marketplace` is registered
|
|
349
|
+
- If missing, re-run `/plugin marketplace add hopla-marketplace HOPLAtools/claude-setup`
|
|
350
|
+
- Restart Claude Code after registering a new marketplace
|
|
351
|
+
|
|
352
|
+
### Commands appear as both `hopla-*` and `hopla:*`
|
|
353
|
+
|
|
354
|
+
You have legacy CLI-installed files from before the plugin refactor. Clean up:
|
|
355
|
+
|
|
356
|
+
```bash
|
|
357
|
+
claude-setup --migrate
|
|
327
358
|
```
|
|
328
359
|
|
|
329
|
-
|
|
360
|
+
This removes `hopla-*` duplicates from `~/.claude/commands/` and `~/.claude/skills/` without touching your global rules.
|
|
361
|
+
|
|
362
|
+
### Update command fails: "no such file or directory"
|
|
363
|
+
|
|
364
|
+
```bash
|
|
365
|
+
cd ~/.claude/plugins/marketplaces/hopla-marketplace && git pull
|
|
366
|
+
# → cd: no such file or directory
|
|
367
|
+
```
|
|
368
|
+
|
|
369
|
+
You never registered the marketplace. Run Quick Start Step 1 first.
|
|
370
|
+
|
|
371
|
+
### `claude-setup: command not found`
|
|
372
|
+
|
|
373
|
+
The npm global bin is not on your PATH. Check with:
|
|
374
|
+
|
|
375
|
+
```bash
|
|
376
|
+
npm config get prefix
|
|
377
|
+
# Add $(npm config get prefix)/bin to your PATH
|
|
378
|
+
```
|
|
379
|
+
|
|
380
|
+
Alternatively, run via npx: `npx @hopla/claude-setup --force`.
|
|
381
|
+
|
|
382
|
+
### Changes to skills/commands don't take effect
|
|
383
|
+
|
|
384
|
+
Claude Code caches plugin content. After a plugin update:
|
|
385
|
+
|
|
386
|
+
```
|
|
387
|
+
/plugin uninstall hopla@hopla-marketplace
|
|
388
|
+
/plugin install hopla@hopla-marketplace
|
|
389
|
+
/reload-plugins
|
|
390
|
+
```
|
|
330
391
|
|
|
331
392
|
---
|
|
332
393
|
|
|
333
394
|
## Roadmap
|
|
334
395
|
|
|
335
|
-
Features under consideration
|
|
396
|
+
Features under consideration:
|
|
336
397
|
|
|
337
|
-
- **Domain-specific agents** — Project-level custom agents beyond the 3 built-in ones (
|
|
338
|
-
- **Hook templates** — Installable hook patterns beyond tsc-check and env-protect
|
|
339
|
-
- **GitHub Actions integration** — Automated PR reviews and `@claude` mentions via GitHub App
|
|
398
|
+
- **Domain-specific agents** — Project-level custom agents beyond the 3 built-in ones (frontend, backend, database)
|
|
399
|
+
- **Hook templates** — Installable hook patterns beyond `tsc-check` and `env-protect`
|
|
400
|
+
- **GitHub Actions integration** — Automated PR reviews and `@claude` mentions via GitHub App
|
|
340
401
|
- **Team dashboards** — Aggregate execution reports and system reviews across team members
|
|
341
402
|
|
|
403
|
+
Issues and contributions welcome: https://github.com/HOPLAtools/claude-setup/issues
|
|
404
|
+
|
|
342
405
|
---
|
|
343
406
|
|
|
344
|
-
## Project Structure (after
|
|
407
|
+
## Project Structure (after `/hopla:create-prd` + `/hopla:init-project`)
|
|
345
408
|
|
|
346
409
|
```
|
|
347
410
|
project/
|
|
348
|
-
├── PRD.md ← Product scope (from /create-prd)
|
|
349
|
-
├── CLAUDE.md ← Project rules and stack (from /init-project)
|
|
411
|
+
├── PRD.md ← Product scope (from /hopla:create-prd)
|
|
412
|
+
├── CLAUDE.md ← Project rules and stack (from /hopla:init-project)
|
|
350
413
|
├── .agents/
|
|
351
|
-
│ ├── plans/ ← Implementation plans (commit
|
|
352
|
-
│ ├──
|
|
353
|
-
│
|
|
354
|
-
│ ├──
|
|
355
|
-
│ ├──
|
|
356
|
-
│ ├──
|
|
357
|
-
│
|
|
414
|
+
│ ├── plans/ ← Implementation plans (commit)
|
|
415
|
+
│ │ ├── done/ ← Archived plans after system-review (commit)
|
|
416
|
+
│ │ └── backlog/ ← Deferred ideas from Scope Guard (commit)
|
|
417
|
+
│ ├── specs/ ← Design specs from brainstorming (commit)
|
|
418
|
+
│ ├── guides/ ← On-demand reference guides (commit)
|
|
419
|
+
│ ├── rca/ ← Root cause analysis docs (commit)
|
|
420
|
+
│ ├── execution-reports/ ← Post-implementation reports (commit)
|
|
421
|
+
│ ├── system-reviews/ ← Process improvement reports (commit)
|
|
422
|
+
│ └── code-reviews/ ← Code review reports (don't commit — ephemeral)
|
|
358
423
|
└── .claude/
|
|
359
424
|
└── commands/ ← Project-specific commands (optional)
|
|
360
425
|
```
|
|
@@ -15,6 +15,7 @@ You are a System Reviewer. Your job is to analyze how well the implementation ma
|
|
|
15
15
|
- **Good divergence**: Found a better approach, justified deviation
|
|
16
16
|
- **Bad divergence**: Missed requirements, ignored constraints, skipped steps
|
|
17
17
|
5. **Trace root causes** — Why did each divergence happen?
|
|
18
|
+
5.5. **Check recommendation history** — Read the 3 most recent system reviews in `.agents/system-reviews/`. If a recommendation appears in 2+ previous reviews and hasn't been applied, flag it as **RECURRING — NOT APPLIED** with escalation priority.
|
|
18
19
|
6. **Suggest improvements** — What should change to prevent bad divergences?
|
|
19
20
|
|
|
20
21
|
## Improvement Decision Matrix
|
|
@@ -60,4 +61,16 @@ Save to `.agents/system-reviews/[feature-name].md`:
|
|
|
60
61
|
|
|
61
62
|
## Key Learnings
|
|
62
63
|
- [Insights worth remembering]
|
|
64
|
+
|
|
65
|
+
## Recommendation Tracking
|
|
66
|
+
|
|
67
|
+
For each recommendation from the 2 most recent system reviews:
|
|
68
|
+
- Was it applied? (Check CLAUDE.md, commands, guides for the suggested change)
|
|
69
|
+
- If not applied, why? (Forgotten, deprioritized, or superseded?)
|
|
70
|
+
- **Recurring unapplied recommendations indicate a broken feedback loop** — escalate these by listing them first in "Recommended Improvements" with a ⚠️ prefix
|
|
71
|
+
|
|
72
|
+
## Next Step
|
|
73
|
+
|
|
74
|
+
After the review is saved, suggest:
|
|
75
|
+
> "System review saved to `.agents/system-reviews/[feature]-review.md`. Plan archived. If recurring recommendations were found, consider applying them before the next feature — they represent known gaps in the process."
|
|
63
76
|
```
|