@hopla/claude-setup 1.3.0 → 1.3.2

package/cli.js

@@ -140,6 +140,25 @@ const PLANNING_COMMANDS = [
     "hopla-git-pr.md",
 ];
 
+function removeExecutionCommands() {
+    const planningSet = new Set(PLANNING_COMMANDS);
+    const removed = [];
+    if (!fs.existsSync(COMMANDS_DIR)) return;
+    for (const file of fs.readdirSync(COMMANDS_DIR)) {
+        if (file.startsWith("hopla-") && !planningSet.has(file)) {
+            fs.rmSync(path.join(COMMANDS_DIR, file));
+            removed.push(file);
+        }
+    }
+    if (removed.length > 0) {
+        log(`${CYAN}Removing execution commands (planning mode)...${RESET}`);
+        for (const file of removed) {
+            log(`  ${RED}✕${RESET}  Removed: ~/.claude/commands/${file}`);
+        }
+        log("");
+    }
+}
+
 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`);
@@ -151,6 +170,9 @@ async function install() {
     // Remove old non-prefixed commands from previous versions
     removeLegacyFiles();
 
+    // In planning mode, remove any execution commands left from a previous full install
+    if (PLANNING) removeExecutionCommands();
+
     log(`${CYAN}Installing global rules...${RESET}`);
     await installFile(
         path.join(FILES_DIR, "CLAUDE.md"),
@@ -196,6 +218,14 @@ const HOPLA_PERMISSIONS = [
     "Bash(echo *)",
 ];
 
+const PLANNING_PERMISSIONS = [
+    "Bash(git branch*)",
+    "Bash(git log*)",
+    "Bash(git status*)",
+];
+
+const ALL_HOPLA_PERMISSIONS = new Set([...HOPLA_PERMISSIONS, ...PLANNING_PERMISSIONS]);
+
 async function setupPermissions() {
     const settingsPath = path.join(CLAUDE_DIR, "settings.json");
 
@@ -211,30 +241,62 @@ async function setupPermissions() {
     if (!settings.permissions) settings.permissions = {};
     if (!settings.permissions.allow) settings.permissions.allow = [];
 
-    // Find permissions not yet added
-    const existing = new Set(settings.permissions.allow);
-    const toAdd = HOPLA_PERMISSIONS.filter((p) => !existing.has(p));
+    const targetPermissions = PLANNING ? PLANNING_PERMISSIONS : HOPLA_PERMISSIONS;
 
-    if (toAdd.length === 0) {
-        log(`${GREEN}✓${RESET}  Permissions already configured.\n`);
-        return;
-    }
+    if (PLANNING) {
+        // Replace: remove all hopla-owned permissions, then add planning-only ones
+        const cleaned = settings.permissions.allow.filter((p) => !ALL_HOPLA_PERMISSIONS.has(p));
+        const toAdd = targetPermissions.filter((p) => !cleaned.includes(p));
+        const toRemove = settings.permissions.allow.filter((p) => ALL_HOPLA_PERMISSIONS.has(p) && !targetPermissions.includes(p));
 
-    log(`${CYAN}Configuring permissions...${RESET}`);
-    log(`  The following will be added to ~/.claude/settings.json:\n`);
-    for (const p of toAdd) {
-        log(`  ${CYAN}+${RESET}  ${p}`);
-    }
+        if (toAdd.length === 0 && toRemove.length === 0) {
+            log(`${GREEN}✓${RESET}  Permissions already configured.\n`);
+            return;
+        }
 
-    const ok = await confirm(`\n  Add these permissions? (y/N) `);
-    if (!ok) {
-        log(`  ${YELLOW}↷${RESET}  Skipped — you can add them manually to ~/.claude/settings.json\n`);
-        return;
+        log(`${CYAN}Configuring permissions (planning mode)...${RESET}`);
+        log(`  The following changes will be made to ~/.claude/settings.json:\n`);
+        for (const p of toRemove) {
+            log(`  ${RED}-${RESET}  ${p}`);
+        }
+        for (const p of toAdd) {
+            log(`  ${CYAN}+${RESET}  ${p}`);
+        }
+
+        const ok = await confirm(`\n  Apply these permission changes? (y/N) `);
+        if (!ok) {
+            log(`  ${YELLOW}↷${RESET}  Skipped — you can edit ~/.claude/settings.json manually\n`);
+            return;
+        }
+
+        settings.permissions.allow = [...cleaned, ...targetPermissions];
+    } else {
+        // Merge: add missing ones
+        const existing = new Set(settings.permissions.allow);
+        const toAdd = targetPermissions.filter((p) => !existing.has(p));
+
+        if (toAdd.length === 0) {
+            log(`${GREEN}✓${RESET}  Permissions already configured.\n`);
+            return;
+        }
+
+        log(`${CYAN}Configuring permissions...${RESET}`);
+        log(`  The following will be added to ~/.claude/settings.json:\n`);
+        for (const p of toAdd) {
+            log(`  ${CYAN}+${RESET}  ${p}`);
+        }
+
+        const ok = await confirm(`\n  Add these permissions? (y/N) `);
+        if (!ok) {
+            log(`  ${YELLOW}↷${RESET}  Skipped — you can add them manually to ~/.claude/settings.json\n`);
+            return;
+        }
+
+        settings.permissions.allow = [...settings.permissions.allow, ...toAdd];
     }
 
-    settings.permissions.allow = [...settings.permissions.allow, ...toAdd];
     fs.writeFileSync(settingsPath, JSON.stringify(settings, null, 2) + "\n");
-    log(`  ${GREEN}✓${RESET}  Permissions added.\n`);
+    log(`  ${GREEN}✓${RESET}  Permissions configured.\n`);
 }
 
 const run = UNINSTALL ? uninstall : install;

package/package.json

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