claude-git-hooks 2.14.1 → 2.15.5

package/lib/utils/github-client.js

@@ -47,14 +47,14 @@ export const parseGitHubRepo = () => {
             };
         }
 
-        throw new Error('Could not parse GitHub repository from remote URL: ' + remoteUrl);
+        throw new Error(`Could not parse GitHub repository from remote URL: ${  remoteUrl}`);
 
     } catch (error) {
         if (error.message.includes('Could not parse')) {
             throw error;
         }
 
-        throw new Error('Failed to get git remote URL: ' + error.message);
+        throw new Error(`Failed to get git remote URL: ${  error.message}`);
     }
 };
 

package/lib/utils/installation-diagnostics.js

@@ -98,7 +98,7 @@ export const formatError = (errorMessage, additionalContext = []) => {
         lines.push(`  .claude/ exists:   ${diagnostics.claudeDirExists ? '✓' : '✗'}`);
         lines.push(`  presets/ exists:   ${diagnostics.presetsDirExists ? '✓' : '✗'}`);
     } else {
-        lines.push(`  Repository root:   [Not in a git repository]`);
+        lines.push('  Repository root:   [Not in a git repository]');
     }
     lines.push('');
 

package/lib/utils/prompt-builder.js

@@ -105,11 +105,9 @@ const replaceTemplate = (template, variables) => {
         { variableCount: Object.keys(variables).length }
     );
 
-    return Object.entries(variables).reduce((result, [key, value]) => {
-        // Why: Escape special regex characters to prevent regex injection
-        const escapedValue = String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-        return result.replace(new RegExp(`{{${key}}}`, 'g'), value);
-    }, template);
+    return Object.entries(variables).reduce((result, [key, value]) => 
+        result.replace(new RegExp(`\\{\\{${key}\\}\\}`, 'g'), value)
+    , template);
 };
 
 /**
@@ -246,7 +244,7 @@ const buildAnalysisPrompt = async ({
                     BATCH_SIZE: subagentConfig.batchSize || 3,
                     MODEL: subagentConfig.model || 'haiku'
                 };
-                prompt += '\n\n' + replaceTemplate(subagentInstruction, subagentVariables) + '\n';
+                prompt += `\n\n${  replaceTemplate(subagentInstruction, subagentVariables)  }\n`;
 
                 logger.info(`🚀 Batch optimization enabled: ${files.length} files, ${subagentVariables.BATCH_SIZE} per batch, ${subagentVariables.MODEL} model`);
             } catch (error) {

package/lib/utils/sanitize.js

@@ -34,9 +34,11 @@ export const sanitizeInput = (input, {
     if (stripControlChars) {
         if (allowNewlines) {
             // Keep \n, \r, \t but remove other control chars
+            // eslint-disable-next-line no-control-regex
             sanitized = sanitized.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F\x7F]/g, '');
         } else {
             // Remove all control characters including newlines
+            // eslint-disable-next-line no-control-regex
             sanitized = sanitized.replace(/[\x00-\x1F\x7F]/g, ' ');
         }
     }
@@ -60,13 +62,11 @@ export const sanitizeInput = (input, {
  * @param {string} title - Raw title
  * @returns {string} - Sanitized title
  */
-export const sanitizePRTitle = (title) => {
-    return sanitizeInput(title, {
-        maxLength: 256,
-        allowNewlines: false,
-        stripControlChars: true
-    });
-};
+export const sanitizePRTitle = (title) => sanitizeInput(title, {
+    maxLength: 256,
+    allowNewlines: false,
+    stripControlChars: true
+});
 
 /**
  * Sanitize PR body/description for GitHub API
@@ -75,13 +75,11 @@ export const sanitizePRTitle = (title) => {
  * @param {string} body - Raw body text
  * @returns {string} - Sanitized body
  */
-export const sanitizePRBody = (body) => {
-    return sanitizeInput(body, {
-        maxLength: 65536,  // GitHub's limit is 65536 chars
-        allowNewlines: true,
-        stripControlChars: true
-    });
-};
+export const sanitizePRBody = (body) => sanitizeInput(body, {
+    maxLength: 65536,  // GitHub's limit is 65536 chars
+    allowNewlines: true,
+    stripControlChars: true
+});
 
 /**
  * Sanitize array of strings (labels, reviewers)
@@ -149,6 +147,7 @@ export const sanitizeLabel = (label) => {
     }
 
     // Remove problematic characters
+    // eslint-disable-next-line no-control-regex
     return trimmed.replace(/[,\x00-\x1F\x7F]/g, '');
 };
 

package/lib/utils/task-id.js

@@ -405,26 +405,24 @@ export const parseTaskIdArg = async (argTaskId, { prompt = true, required = fals
  *
  * @returns {Array<Object>} - Array of pattern info
  */
-export const getSupportedPatterns = () => {
-    return [
-        {
-            name: 'Jira-style',
-            examples: getExamplesForPattern('Jira-style')
-        },
-        {
-            name: 'GitHub issue',
-            examples: getExamplesForPattern('GitHub issue')
-        },
-        {
-            name: 'Linear',
-            examples: getExamplesForPattern('Linear')
-        },
-        {
-            name: 'Generic',
-            examples: getExamplesForPattern('Generic')
-        }
-    ];
-};
+export const getSupportedPatterns = () => [
+    {
+        name: 'Jira-style',
+        examples: getExamplesForPattern('Jira-style')
+    },
+    {
+        name: 'GitHub issue',
+        examples: getExamplesForPattern('GitHub issue')
+    },
+    {
+        name: 'Linear',
+        examples: getExamplesForPattern('Linear')
+    },
+    {
+        name: 'Generic',
+        examples: getExamplesForPattern('Generic')
+    }
+];
 
 /**
  * Get example task IDs for a pattern

package/lib/utils/telemetry.js

@@ -69,9 +69,7 @@ const generateEventId = () => {
  *
  * @returns {string} Telemetry directory path
  */
-const getTelemetryDir = () => {
-    return path.join(process.cwd(), '.claude', 'telemetry');
-};
+const getTelemetryDir = () => path.join(process.cwd(), '.claude', 'telemetry');
 
 /**
  * Get current telemetry log file path
@@ -90,10 +88,10 @@ const getCurrentLogFile = () => {
  *
  * @returns {boolean} True if telemetry is enabled (default: true)
  */
-const isTelemetryEnabled = () => {
+const isTelemetryEnabled = () => 
     // Enabled by default - only disabled if explicitly set to false
-    return config.system?.telemetry !== false;
-};
+    config.system?.telemetry !== false
+;
 
 /**
  * Ensure telemetry directory exists
@@ -117,7 +115,7 @@ const ensureTelemetryDir = async () => {
 const appendEvent = async (event) => {
     try {
         const logFile = getCurrentLogFile();
-        const line = JSON.stringify(event) + '\n';
+        const line = `${JSON.stringify(event)  }\n`;
 
         // Append to file (create if doesn't exist)
         await fs.appendFile(logFile, line, 'utf8');

package/package.json

@@ -1,18 +1,26 @@
 {
     "name": "claude-git-hooks",
-    "version": "2.14.1",
+    "version": "2.15.5",
     "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
     "type": "module",
     "bin": {
         "claude-hooks": "./bin/claude-hooks"
     },
     "scripts": {
-        "test": "node --experimental-vm-modules node_modules/jest/bin/jest.js",
+        "test": "npm run test:all",
+        "test:all": "npm run lint && npm run test:smoke && npm run test:unit && npm run test:integration",
+        "test:smoke": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/smoke --maxWorkers=1",
+        "test:unit": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --forceExit",
+        "test:integration": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration --maxWorkers=1 --testTimeout=30000 --forceExit",
+        "test:integration:ci": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration/ci-safe.test.js --maxWorkers=1 --testTimeout=30000 --forceExit",
+        "test:changed": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --changedSince=main --forceExit",
         "test:watch": "node --experimental-vm-modules node_modules/jest/bin/jest.js --watch",
         "test:coverage": "node --experimental-vm-modules node_modules/jest/bin/jest.js --coverage",
-        "lint": "eslint lib/ bin/",
-        "lint:fix": "eslint lib/ bin/ --fix",
-        "format": "prettier --write \"lib/**/*.js\" \"bin/**\""
+        "lint": "eslint lib/ bin/claude-hooks",
+        "lint:fix": "eslint lib/ bin/claude-hooks --fix",
+        "format": "prettier --write \"lib/**/*.js\" \"bin/**\" \"test/**/*.js\"",
+        "precommit": "npm run lint && npm run test:smoke",
+        "prepublishOnly": "npm run test:all"
     },
     "keywords": [
         "git",

package/templates/config.advanced.example.json

@@ -1,113 +1,113 @@
-{
-    "$schema": "Configuration v2.8.0 - Advanced parameters",
-    "$comment": "Advanced configuration for power users and tinkerers",
-    "$warning": "Most users don't need these settings. Use at your own risk.",
-
-    "version": "2.8.0",
-    "preset": "backend",
-
-    "overrides": {
-        "analysis": {
-            "ignoreExtensions": [
-                ".min.js",
-                ".map",
-                "package-lock.json",
-                "yarn.lock",
-                ".generated.ts"
-            ]
-        },
-
-        "commitMessage": {
-            "taskIdPattern": "(CUSTOM-[0-9]{4})"
-        },
-
-        "subagents": {
-            "model": "sonnet",
-            "batchSize": 2
-        },
-
-        "github": {
-            "pr": {
-                "defaultBase": "develop",
-                "reviewers": ["tech-lead"]
-            }
-        }
-    },
-
-    "_documentation": {
-        "analysis.ignoreExtensions": {
-            "description": "File extensions to exclude from analysis",
-            "default": "[]",
-            "examples": [".min.js", ".lock", ".map"],
-            "use_case": "Skip minified files, lock files, or generated code"
-        },
-
-        "commitMessage.taskIdPattern": {
-            "description": "Regex pattern to extract task-id from branch name",
-            "default": "([A-Z]{1,3}[-\\s]\\d{3,5})",
-            "examples": [
-                "IX-123 -> IX-123",
-                "PROJ-4567 -> PROJ-4567",
-                "(CUSTOM-[0-9]{4}) for CUSTOM-1234"
-            ],
-            "use_case": "Custom task-id format (not Jira/GitHub/Linear)"
-        },
-
-        "subagents.model": {
-            "description": "Claude model for parallel analysis",
-            "default": "haiku",
-            "options": ["haiku", "sonnet", "opus"],
-            "trade_offs": {
-                "haiku": "Fast and cheap, good for most code",
-                "sonnet": "Balanced speed and quality",
-                "opus": "Slowest but most thorough analysis"
-            },
-            "use_case": "Trade speed/cost vs analysis quality"
-        },
-
-        "subagents.batchSize": {
-            "description": "Number of files per parallel batch",
-            "default": "Preset-specific (1-4)",
-            "range": "1-10",
-            "trade_offs": {
-                "1": "Maximum parallelization, fastest but most expensive",
-                "2-4": "Balanced parallelization (preset defaults)",
-                "5+": "Less parallel, slower but cheaper"
-            },
-            "use_case": "Fine-tune cost vs speed for your project size"
-        }
-    },
-
-    "_examples": {
-        "skip_generated_code": {
-            "version": "2.8.0",
-            "preset": "frontend",
-            "overrides": {
-                "analysis": {
-                    "ignoreExtensions": [".generated.ts", ".d.ts", ".min.js"]
-                }
-            }
-        },
-
-        "custom_jira_format": {
-            "version": "2.8.0",
-            "preset": "backend",
-            "overrides": {
-                "commitMessage": {
-                    "taskIdPattern": "(MYPROJ-[0-9]{1,5})"
-                }
-            }
-        },
-
-        "high_quality_analysis": {
-            "version": "2.8.0",
-            "preset": "fullstack",
-            "overrides": {
-                "subagents": {
-                    "model": "sonnet",
-                    "batchSize": 2
-                }
-            }
-        }
-    }
-}
+{
+    "$schema": "Configuration v2.8.0 - Advanced parameters",
+    "$comment": "Advanced configuration for power users and tinkerers",
+    "$warning": "Most users don't need these settings. Use at your own risk.",
+
+    "version": "2.8.0",
+    "preset": "backend",
+
+    "overrides": {
+        "analysis": {
+            "ignoreExtensions": [
+                ".min.js",
+                ".map",
+                "package-lock.json",
+                "yarn.lock",
+                ".generated.ts"
+            ]
+        },
+
+        "commitMessage": {
+            "taskIdPattern": "(CUSTOM-[0-9]{4})"
+        },
+
+        "subagents": {
+            "model": "sonnet",
+            "batchSize": 2
+        },
+
+        "github": {
+            "pr": {
+                "defaultBase": "develop",
+                "reviewers": ["tech-lead"]
+            }
+        }
+    },
+
+    "_documentation": {
+        "analysis.ignoreExtensions": {
+            "description": "File extensions to exclude from analysis",
+            "default": "[]",
+            "examples": [".min.js", ".lock", ".map"],
+            "use_case": "Skip minified files, lock files, or generated code"
+        },
+
+        "commitMessage.taskIdPattern": {
+            "description": "Regex pattern to extract task-id from branch name",
+            "default": "([A-Z]{1,3}[-\\s]\\d{3,5})",
+            "examples": [
+                "IX-123 -> IX-123",
+                "PROJ-4567 -> PROJ-4567",
+                "(CUSTOM-[0-9]{4}) for CUSTOM-1234"
+            ],
+            "use_case": "Custom task-id format (not Jira/GitHub/Linear)"
+        },
+
+        "subagents.model": {
+            "description": "Claude model for parallel analysis",
+            "default": "haiku",
+            "options": ["haiku", "sonnet", "opus"],
+            "trade_offs": {
+                "haiku": "Fast and cheap, good for most code",
+                "sonnet": "Balanced speed and quality",
+                "opus": "Slowest but most thorough analysis"
+            },
+            "use_case": "Trade speed/cost vs analysis quality"
+        },
+
+        "subagents.batchSize": {
+            "description": "Number of files per parallel batch",
+            "default": "Preset-specific (1-4)",
+            "range": "1-10",
+            "trade_offs": {
+                "1": "Maximum parallelization, fastest but most expensive",
+                "2-4": "Balanced parallelization (preset defaults)",
+                "5+": "Less parallel, slower but cheaper"
+            },
+            "use_case": "Fine-tune cost vs speed for your project size"
+        }
+    },
+
+    "_examples": {
+        "skip_generated_code": {
+            "version": "2.8.0",
+            "preset": "frontend",
+            "overrides": {
+                "analysis": {
+                    "ignoreExtensions": [".generated.ts", ".d.ts", ".min.js"]
+                }
+            }
+        },
+
+        "custom_jira_format": {
+            "version": "2.8.0",
+            "preset": "backend",
+            "overrides": {
+                "commitMessage": {
+                    "taskIdPattern": "(MYPROJ-[0-9]{1,5})"
+                }
+            }
+        },
+
+        "high_quality_analysis": {
+            "version": "2.8.0",
+            "preset": "fullstack",
+            "overrides": {
+                "subagents": {
+                    "model": "sonnet",
+                    "batchSize": 2
+                }
+            }
+        }
+    }
+}

package/templates/pre-commit

@@ -112,6 +112,13 @@ if [ -z "$NODE_HOOK" ]; then
     exit 1
 fi
 
+# Run tests before Claude analysis (optional - uncomment to enable)
+# echo "Running tests..."
+# npm run test:unit || {
+#     echo -e "${RED}Tests failed! Fix tests before committing.${NC}"
+#     exit 1
+# }
+
 # Execute the Node.js script
 # Why: Pass all arguments ($@) to support future hook extensions
 exec node "$NODE_HOOK" "$@"

package/templates/presets/ai/config.json

@@ -1,5 +1,5 @@
-{
-    "subagents": {
-        "batchSize": 1
-    }
-}
+{
+    "subagents": {
+        "batchSize": 1
+    }
+}

package/templates/presets/backend/config.json

@@ -1,5 +1,5 @@
-{
-    "subagents": {
-        "batchSize": 3
-    }
-}
+{
+    "subagents": {
+        "batchSize": 3
+    }
+}

package/templates/presets/backend/preset.json

@@ -1,49 +1,49 @@
-{
-  "name": "backend",
-  "displayName": "Backend (Spring Boot + SQL Server)",
-  "description": "Java backend with Spring Boot, JPA, SQL Server, AWS",
-  "version": "1.0.0",
-
-  "techStack": [
-    "Spring Boot 2.6+",
-    "JPA",
-    "Hibernate",
-    "SQL Server",
-    "Spring Security",
-    "JWT",
-    "MapStruct",
-    "Lombok",
-    "AWS SDK",
-    "Maven",
-    "Cucumber",
-    "JUnit",
-    "JaCoCo"
-  ],
-
-  "fileExtensions": [
-    ".java",
-    ".xml",
-    ".yml",
-    ".yaml"
-  ],
-
-  "focusAreas": [
-    "REST API design and best practices",
-    "JPA entities and repositories",
-    "Service layer patterns",
-    "Security vulnerabilities (OWASP Top 10)",
-    "SQL injection prevention",
-    "Performance (threads, async operations)",
-    "Transaction management",
-    "DTO mappings",
-    "Test coverage"
-  ],
-
-  "templates": {
-    "analysis": "ANALYSIS_PROMPT.md",
-    "guidelines": "PRE_COMMIT_GUIDELINES.md",
-    "commitMessage": "../shared/COMMIT_MESSAGE.md",
-    "analyzeDiff": "../shared/ANALYZE_DIFF.md",
-    "resolution": "../shared/RESOLUTION_PROMPT.md"
-  }
-}
+{
+  "name": "backend",
+  "displayName": "Backend (Spring Boot + SQL Server)",
+  "description": "Java backend with Spring Boot, JPA, SQL Server, AWS",
+  "version": "1.0.0",
+
+  "techStack": [
+    "Spring Boot 2.6+",
+    "JPA",
+    "Hibernate",
+    "SQL Server",
+    "Spring Security",
+    "JWT",
+    "MapStruct",
+    "Lombok",
+    "AWS SDK",
+    "Maven",
+    "Cucumber",
+    "JUnit",
+    "JaCoCo"
+  ],
+
+  "fileExtensions": [
+    ".java",
+    ".xml",
+    ".yml",
+    ".yaml"
+  ],
+
+  "focusAreas": [
+    "REST API design and best practices",
+    "JPA entities and repositories",
+    "Service layer patterns",
+    "Security vulnerabilities (OWASP Top 10)",
+    "SQL injection prevention",
+    "Performance (threads, async operations)",
+    "Transaction management",
+    "DTO mappings",
+    "Test coverage"
+  ],
+
+  "templates": {
+    "analysis": "ANALYSIS_PROMPT.md",
+    "guidelines": "PRE_COMMIT_GUIDELINES.md",
+    "commitMessage": "../shared/COMMIT_MESSAGE.md",
+    "analyzeDiff": "../shared/ANALYZE_DIFF.md",
+    "resolution": "../shared/RESOLUTION_PROMPT.md"
+  }
+}

package/templates/presets/database/config.json

@@ -1,5 +1,5 @@
-{
-    "subagents": {
-        "batchSize": 2
-    }
-}
+{
+    "subagents": {
+        "batchSize": 2
+    }
+}