@hopla/claude-setup 1.11.2 → 1.12.0

package/.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
     {
       "name": "hopla",
       "description": "Agentic coding system: PIV loop, TDD, debugging, brainstorming, subagent execution, and team workflows",
-      "version": "1.11.2",
+      "version": "1.12.0",
       "source": "./",
       "author": {
         "name": "Hopla Tools",

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "hopla",
   "description": "Agentic coding system for Claude Code: PIV loop (Plan → Implement → Validate), TDD, debugging, brainstorming, subagent execution, and team workflows",
-  "version": "1.11.2",
+  "version": "1.12.0",
   "author": {
     "name": "Hopla Tools",
     "email": "julio@hopla.tools"

package/agents/code-reviewer.md

@@ -1,7 +1,6 @@
 ---
 name: code-reviewer
 description: "Senior code reviewer agent for thorough code quality analysis. Use this agent to review completed code changes with fresh context, catching issues the implementer might miss."
-model: sonnet
 allowed-tools: Read, Grep, Glob, Bash
 ---
 

package/agents/codebase-researcher.md

@@ -1,7 +1,6 @@
 ---
 name: codebase-researcher
 description: "Fast codebase exploration agent for research tasks. Use this agent to investigate code, find patterns, map dependencies, or gather context without polluting the main conversation."
-model: haiku
 allowed-tools: Read, Grep, Glob, Bash
 ---
 

package/agents/system-reviewer.md

@@ -1,7 +1,6 @@
 ---
 name: system-reviewer
 description: "System review agent that analyzes execution reports against plans to identify process improvements. Use after feature completion to find patterns and improve the development system."
-model: sonnet
 allowed-tools: Read, Grep, Glob, Bash
 ---
 

package/cli.js

@@ -74,26 +74,30 @@ async function uninstall() {
     log(`\n${BOLD}@hopla/claude-setup${RESET} — Uninstall\n`);
 
     const srcEntries = fs.readdirSync(path.join(REPO_ROOT, "commands"));
-    const srcFiles = srcEntries.filter((f) =>
-        fs.statSync(path.join(REPO_ROOT, "commands", f)).isFile()
-    );
+    const srcFiles = srcEntries.filter((f) => {
+        if (f.startsWith(".")) return false;
+        return fs.statSync(path.join(REPO_ROOT, "commands", f)).isFile();
+    });
     const srcDirs = srcEntries.filter((f) =>
         fs.statSync(path.join(REPO_ROOT, "commands", f)).isDirectory()
     );
 
+    // Map source files to their installed names (with hopla- prefix)
+    const installedNames = new Set(srcFiles.map((f) => withPrefix(f)));
+
     // Also include any hopla-* files in ~/.claude/commands/ not in current source
     // (leftovers from previous versions)
     const installedHoplaFiles = fs.existsSync(COMMANDS_DIR)
         ? fs.readdirSync(COMMANDS_DIR).filter((f) =>
-            f.startsWith("hopla-") && fs.statSync(path.join(COMMANDS_DIR, f)).isFile() && !srcFiles.includes(f)
+            f.startsWith("hopla-") && fs.statSync(path.join(COMMANDS_DIR, f)).isFile() && !installedNames.has(f)
         )
         : [];
 
     const itemsToRemove = [
         { dest: path.join(CLAUDE_DIR, "CLAUDE.md"), label: "~/.claude/CLAUDE.md", isDir: false },
         ...srcFiles.map((file) => ({
-            dest: path.join(COMMANDS_DIR, file),
-            label: `~/.claude/commands/${file}`,
+            dest: path.join(COMMANDS_DIR, withPrefix(file)),
+            label: `~/.claude/commands/${withPrefix(file)}`,
             isDir: false,
         })),
         ...installedHoplaFiles.map((file) => ({
@@ -145,7 +149,7 @@ async function uninstall() {
 
 function removeStaleCommands(currentCommandFiles) {
     if (!fs.existsSync(COMMANDS_DIR)) return;
-    const currentSet = new Set(currentCommandFiles);
+    const currentSet = new Set(currentCommandFiles.map((f) => withPrefix(f)));
     const removed = [];
     for (const file of fs.readdirSync(COMMANDS_DIR)) {
         const filePath = path.join(COMMANDS_DIR, file);
@@ -165,15 +169,20 @@ function removeStaleCommands(currentCommandFiles) {
 }
 
 const PLANNING_COMMANDS = [
-    "hopla-init-project.md",
-    "hopla-create-prd.md",
-    "hopla-plan-feature.md",
-    "hopla-review-plan.md",
-    "hopla-git-commit.md",
-    "hopla-git-pr.md",
-    "hopla-guide.md",
+    "init-project.md",
+    "create-prd.md",
+    "plan-feature.md",
+    "review-plan.md",
+    "git-commit.md",
+    "git-pr.md",
+    "guide.md",
 ];
 
+// CLI channel adds hopla- prefix so skills/commands keep their namespace
+function withPrefix(name) {
+    return `hopla-${name}`;
+}
+
 async function install() {
     const modeLabel = PLANNING ? "Planning Mode (Robert)" : "Full Install";
     log(`\n${BOLD}@hopla/claude-setup${RESET} — Agentic Coding System ${CYAN}[${modeLabel}]${RESET}\n`);
@@ -209,10 +218,11 @@ async function install() {
 
     log(`\n${CYAN}Installing commands...${RESET}`);
     for (const file of commandFiles.sort()) {
+        const destFile = withPrefix(file);
         await installFile(
             path.join(REPO_ROOT, "commands", file),
-            path.join(COMMANDS_DIR, file),
-            `~/.claude/commands/${file}`
+            path.join(COMMANDS_DIR, destFile),
+            `~/.claude/commands/${destFile}`
         );
     }
     // Install subdirectories (e.g. guides/)
@@ -231,7 +241,7 @@ async function install() {
 
     log(`\n${GREEN}${BOLD}Done!${RESET} Commands available in any Claude Code session:\n`);
     for (const file of commandFiles.sort()) {
-        const name = file.replace(".md", "");
+        const name = withPrefix(file).replace(".md", "");
         log(`  ${CYAN}/${name}${RESET}`);
     }
     if (PLANNING) {
@@ -247,7 +257,7 @@ async function install() {
 }
 
 // Skills to install in planning mode (subset)
-const PLANNING_SKILLS = ["hopla-prime", "hopla-brainstorm"];
+const PLANNING_SKILLS = ["prime", "brainstorm"];
 
 async function installSkills() {
     const skillsSrcDir = path.join(REPO_ROOT, "skills");
@@ -268,20 +278,21 @@ async function installSkills() {
     log(`\n${CYAN}Installing skills...${RESET}`);
     for (const skillName of skillsToInstall.sort()) {
         const srcDir = path.join(skillsSrcDir, skillName);
-        const destDir = path.join(SKILLS_DIR, skillName);
+        const destName = withPrefix(skillName);
+        const destDir = path.join(SKILLS_DIR, destName);
         fs.mkdirSync(destDir, { recursive: true });
         for (const file of fs.readdirSync(srcDir).sort()) {
             await installFile(
                 path.join(srcDir, file),
                 path.join(destDir, file),
-                `~/.claude/skills/${skillName}/${file}`
+                `~/.claude/skills/${destName}/${file}`
             );
         }
     }
 
     log(`\n${GREEN}${BOLD}Skills installed!${RESET} Auto-activate without a slash command:\n`);
     for (const skillName of skillsToInstall.sort()) {
-        log(`  ${CYAN}${skillName}${RESET}`);
+        log(`  ${CYAN}${withPrefix(skillName)}${RESET}`);
     }
 }
 

package/commands/guides/write-skill.md

@@ -20,7 +20,7 @@ skill-name/
 name: skill-name
 description: "100-150 words. Start with what it does, then 'Use when...' + specific triggers. End with 'Do NOT use for...' anti-triggers."
 allowed-tools: Read, Grep, Glob, Bash  # Optional: restrict tools
-model: sonnet  # Optional: force specific model
+# Do NOT hardcode `model:` — let skills inherit the user's configured model
 ---
 ```
 

package/hooks/session-prime.js

@@ -42,14 +42,14 @@ async function main() {
 
     // Available skills reminder
     lines.push(`📦 HOPLA Skills Available:
-- hopla-prime: Project orientation (trigger: "orient", "get context", "load project")
-- hopla-git: Git operations (trigger: "commit", "PR", "push")
-- hopla-code-review: Code review (trigger: "review code", "code review")
-- hopla-execution-report: Post-implementation docs (trigger: "generate report")
-- hopla-verify: Completion verification (trigger: any "done"/"listo"/"finished" claim)
-- hopla-brainstorm: Design exploration (trigger: "new feature", "brainstorm", "explore options")
-- hopla-debug: Systematic debugging (trigger: "bug", "error", "debug")
-- hopla-tdd: Test-driven development (trigger: implementing with tests)
+- prime: Project orientation (trigger: "orient", "get context", "load project")
+- git: Git operations (trigger: "commit", "PR", "push")
+- code-review: Code review (trigger: "review code", "code review")
+- execution-report: Post-implementation docs (trigger: "generate report")
+- verify: Completion verification (trigger: any "done"/"listo"/"finished" claim)
+- brainstorm: Design exploration (trigger: "new feature", "brainstorm", "explore options")
+- debug: Systematic debugging (trigger: "bug", "error", "debug")
+- tdd: Test-driven development (trigger: implementing with tests)
 
 ⚠️ If a skill applies to the current task, you MUST use it.`);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hopla/claude-setup",
-  "version": "1.11.2",
+  "version": "1.12.0",
   "description": "Hopla team agentic coding system for Claude Code",
   "type": "module",
   "bin": {

package/skills/{hopla-brainstorm → brainstorm}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-brainstorm
+name: brainstorm
 description: "Design exploration and brainstorming before planning. Use when the user wants to explore options for a new feature, discuss approaches, design a solution, brainstorm ideas, or evaluate trade-offs. Trigger on: 'new feature', 'brainstorm', 'explore options', 'design', 'how should we', 'what approach', 'trade-offs', 'quiero agregar', 'diseñar', 'explorar opciones'. Do NOT use when the user already has a clear plan or is asking to execute existing work."
 ---
 

package/skills/{hopla-code-review → code-review}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-code-review
+name: code-review
 description: "Technical code review on changed files. Use when the user says 'review code', 'code review', 'check my code', 'review changes', 'look for bugs', or 'audit code'. Also use after completing implementation when validation passes. Do NOT use for reviewing plans or documents — only code."
 allowed-tools: Read, Grep, Glob, Bash
 ---

package/skills/{hopla-debug → debug}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-debug
+name: debug
 description: "Systematic debugging methodology for finding and fixing bugs. Use when encountering errors, bugs, failures, unexpected behavior, or when the user says 'bug', 'error', 'not working', 'failing', 'debug', 'fix', 'broken', 'falla', 'no funciona', 'error'. Do NOT use for planned feature work or refactoring — only for diagnosing and fixing unexpected problems."
 ---
 

package/skills/{hopla-execution-report → execution-report}/SKILL.md

@@ -1,8 +1,7 @@
 ---
-name: hopla-execution-report
+name: execution-report
 description: "Post-implementation documentation generator. Use when the user says 'generate report', 'document what was done', 'execution report', 'what changed', or after a feature implementation is complete and validated. Do NOT use during implementation — only after completion."
 allowed-tools: Read, Grep, Glob, Bash
-model: sonnet
 ---
 
 > 🌐 **Language:** All user-facing output must match the user's language. Code, paths, and commands stay in English.

package/skills/{hopla-git → git}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-git
+name: git
 description: "Git operations handler for commits and pull requests. Use when the user mentions 'commit', 'save changes', 'create commit', 'PR', 'pull request', 'push', 'merge request', or any git workflow action. Do NOT use for git status checks or branch management — only for commits and PRs."
 ---
 
@@ -20,7 +20,7 @@ Detect the user's intent and execute the appropriate git workflow.
 ## File References
 
 The full step-by-step instructions for each workflow are in:
-- `~/.claude/skills/hopla-git/commit.md` — conventional commit with Git Flow awareness
-- `~/.claude/skills/hopla-git/pr.md` — GitHub PR creation with structured description
+- `commit.md` (in this skill's directory) — conventional commit with Git Flow awareness
+- `pr.md` (in this skill's directory) — GitHub PR creation with structured description
 
 Read the relevant file now and follow its instructions completely.

package/skills/{hopla-parallel-dispatch → parallel-dispatch}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-parallel-dispatch
+name: parallel-dispatch
 description: "Parallel agent dispatch for independent tasks. Use when 2+ tasks have no shared state and can run simultaneously, during brainstorming to explore multiple approaches, or when the user says 'in parallel', 'simultaneously', 'at the same time'. Do NOT use when tasks have dependencies or share state."
 ---
 

package/skills/{hopla-prime → prime}/SKILL.md

@@ -1,8 +1,7 @@
 ---
-name: hopla-prime
+name: prime
 description: "Project orientation and context loading. Use when starting a session, onboarding to a project, needing to understand the codebase, or when the user says 'orient', 'get context', 'load project', 'what is this project', 'prime', or 'start'. Do NOT use mid-task when the project is already understood."
 allowed-tools: Read, Grep, Glob, Bash
-model: sonnet
 ---
 
 > 🌐 **Language:** All user-facing output must match the user's language. Code, paths, and commands stay in English.

package/skills/{hopla-subagent-execution → subagent-execution}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-subagent-execution
+name: subagent-execution
 description: "Subagent-driven execution for large plans. Use when executing plans with 5+ tasks to maintain context quality, when the user says 'use subagents', 'parallel execution', or when context degradation is a concern in long implementations. Do NOT use for small plans (< 5 tasks) or quick fixes."
 ---
 

package/skills/{hopla-tdd → tdd}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-tdd
+name: tdd
 description: "Test-driven development workflow using the RED-GREEN-REFACTOR cycle. Use when implementing features that require tests, when the plan specifies test tasks, when the user mentions 'TDD', 'test first', 'write tests', or when working on bug fixes that should have regression tests. Do NOT use for documentation, configuration, or tasks where tests don't apply."
 ---
 

package/skills/{hopla-verify → verify}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-verify
+name: verify
 description: "Verification gate that ensures all completion claims are backed by fresh evidence. Use when the agent is about to declare work as done, finished, complete, ready, or implemented. Also use when hearing 'listo', 'terminé', 'ya está', 'done', 'finished', 'all tests pass', 'everything works', or any completion claim. Do NOT use for intermediate progress updates or partial task completion."
 ---
 

package/skills/{hopla-worktree → worktree}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: hopla-worktree
+name: worktree
 description: "Git worktree management for isolated feature development. Use when starting a new feature that benefits from isolation, when the user says 'worktree', 'isolated branch', 'parallel development', or when implementing multiple features simultaneously. Do NOT use for quick fixes or single-file changes."
 ---