claude-git-hooks 2.21.0 → 2.30.2

package/lib/utils/judge.js

@@ -22,11 +22,7 @@ import path from 'path';
 import { execSync } from 'child_process';
 import { executeClaudeWithRetry, extractJSON } from './claude-client.js';
 import { loadPrompt } from './prompt-builder.js';
-import {
-    formatBlockingIssues,
-    getAffectedFiles,
-    formatFileContents
-} from './resolution-prompt.js';
+import { formatBlockingIssues, getAffectedFiles, formatFileContents } from './resolution-prompt.js';
 import { getRepoRoot, getRepoName, getCurrentBranch } from './git-operations.js';
 import logger from './logger.js';
 import { recordEvent } from './telemetry.js';
@@ -55,6 +51,29 @@ const applyFix = async (fix, repoRoot) => {
         const firstIndex = content.indexOf(search);
 
         if (firstIndex === -1) {
+            // Debug: diagnose why the search string didn't match
+            if (logger.isDebugMode()) {
+                const fileCRLF = content.includes('\r\n');
+                const searchCRLF = search.includes('\r\n');
+                const searchLines = search.split(/\r?\n/);
+                const firstLine = searchLines[0].trim();
+                // Try to locate the first line loosely in the file
+                const looseIndex = content.indexOf(firstLine);
+                logger.debug('judge - applyFix', 'Search mismatch diagnosis', {
+                    file,
+                    searchLength: search.length,
+                    searchLines: searchLines.length,
+                    fileCRLF,
+                    searchCRLF,
+                    crlfMismatch: fileCRLF !== searchCRLF,
+                    firstLineFound: looseIndex !== -1,
+                    firstLineFoundAt:
+                        looseIndex !== -1
+                            ? `char ${looseIndex} (approx line ${content.substring(0, looseIndex).split('\n').length})`
+                            : null,
+                    searchFirst200Hex: Buffer.from(search.substring(0, 200)).toString('hex')
+                });
+            }
             return { applied: false, reason: 'search string not found in file' };
         }
 
@@ -122,6 +141,42 @@ const judgeAndFix = async (analysisResult, filesData, config) => {
     const parsed = extractJSON(response);
     const fixes = parsed?.fixes || [];
 
+    // Debug: dump raw LLM response and parsed fixes for diagnosis
+    if (logger.isDebugMode()) {
+        try {
+            const debugDir = path.join(repoRoot, '.claude', 'out');
+            await fs.mkdir(debugDir, { recursive: true });
+            await fs.writeFile(
+                path.join(debugDir, 'judge-debug.json'),
+                JSON.stringify(
+                    {
+                        timestamp: new Date().toISOString(),
+                        model,
+                        issueCount: allIssues.length,
+                        fixCount: fixes.length,
+                        rawResponse: response,
+                        parsedFixes: fixes.map((f) => ({
+                            file: f.file,
+                            assessment: f.assessment,
+                            searchLength: f.search?.length ?? null,
+                            replaceLength: f.replace?.length ?? null,
+                            searchHex: f.search
+                                ? Buffer.from(f.search.substring(0, 300)).toString('hex')
+                                : null,
+                            search: f.search,
+                            replace: f.replace
+                        }))
+                    },
+                    null,
+                    2
+                )
+            );
+            logger.debug('judge', 'Debug dump written to .claude/out/judge-debug.json');
+        } catch (debugErr) {
+            logger.debug('judge', 'Failed to write debug dump', debugErr);
+        }
+    }
+
     let fixedCount = 0;
     let falsePositiveCount = 0;
     const resolvedIndices = new Set();
@@ -173,11 +228,15 @@ const judgeAndFix = async (analysisResult, filesData, config) => {
     );
 
     if (fixedCount > 0) {
-        logger.info(`   ${fixedCount} fix${fixedCount !== 1 ? 'es' : ''} staged for commit — review with: git diff --cached`);
+        logger.info(
+            `   ${fixedCount} fix${fixedCount !== 1 ? 'es' : ''} staged for commit — review with: git diff --cached`
+        );
     }
 
     if (unresolvedCount > 0) {
-        logger.warning(`   ${unresolvedCount} unresolved issue${unresolvedCount !== 1 ? 's' : ''} — commit will be blocked`);
+        logger.warning(
+            `   ${unresolvedCount} unresolved issue${unresolvedCount !== 1 ? 's' : ''} — commit will be blocked`
+        );
     }
 
     // Record telemetry

package/lib/utils/linear-connector.js

@@ -268,10 +268,7 @@ export const fetchTicket = async (identifier) => {
 
             if (attempt < MAX_RETRIES) {
                 // Steps c/d: Retry with reconnection
-                logger.debug(
-                    'linear-connector - fetchTicket',
-                    'Retrying after connection failure'
-                );
+                logger.debug('linear-connector - fetchTicket', 'Retrying after connection failure');
                 // Small delay before retry
                 await new Promise((resolve) => setTimeout(resolve, 1000));
             }

package/lib/utils/package-info.js

@@ -61,4 +61,3 @@ export const getPackageName = async () => {
     const pkg = await getPackageJson();
     return pkg.name;
 };
-

package/lib/utils/token-store.js

@@ -123,7 +123,10 @@ export const saveToken = (settingsKey, token) => {
             try {
                 settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
             } catch {
-                logger.debug('token-store - saveToken', 'Invalid existing settings, starting fresh');
+                logger.debug(
+                    'token-store - saveToken',
+                    'Invalid existing settings, starting fresh'
+                );
                 settings = {};
             }
         }
@@ -155,5 +158,4 @@ export const saveToken = (settingsKey, token) => {
  * @param {string[]} envVars - Environment variable names to check
  * @returns {boolean}
  */
-export const hasToken = (settingsKey, envVars = []) =>
-    loadToken(settingsKey, envVars) !== null;
+export const hasToken = (settingsKey, envVars = []) => loadToken(settingsKey, envVars) !== null;

package/package.json

@@ -1,69 +1,69 @@
-{
-    "name": "claude-git-hooks",
-    "version": "2.21.0",
-    "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
-    "type": "module",
-    "bin": {
-        "claude-hooks": "./bin/claude-hooks"
-    },
-    "scripts": {
-        "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/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",
-        "hooks",
-        "claude",
-        "ai",
-        "code-review",
-        "commit-messages",
-        "pre-commit",
-        "automation"
-    ],
-    "author": "Pablo Rovito",
-    "license": "MIT",
-    "repository": {
-        "type": "git",
-        "url": "https://github.com/mscope-S-L/git-hooks.git"
-    },
-    "engines": {
-        "node": ">=16.9.0"
-    },
-    "engineStrict": false,
-    "os": [
-        "darwin",
-        "linux",
-        "win32"
-    ],
-    "preferGlobal": true,
-    "files": [
-        "bin/",
-        "lib/",
-        "templates/",
-        "README.md",
-        "CHANGELOG.md",
-        "CLAUDE.md",
-        "LICENSE"
-    ],
-    "dependencies": {
-        "@octokit/rest": "^21.0.0"
-    },
-    "devDependencies": {
-        "@types/jest": "^29.5.0",
-        "eslint": "^8.57.0",
-        "jest": "^29.7.0",
-        "prettier": "^3.2.0"
-    }
-}
+{
+    "name": "claude-git-hooks",
+    "version": "2.30.2",
+    "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
+    "type": "module",
+    "bin": {
+        "claude-hooks": "./bin/claude-hooks"
+    },
+    "scripts": {
+        "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/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",
+        "hooks",
+        "claude",
+        "ai",
+        "code-review",
+        "commit-messages",
+        "pre-commit",
+        "automation"
+    ],
+    "author": "Pablo Rovito",
+    "license": "MIT",
+    "repository": {
+        "type": "git",
+        "url": "https://github.com/mscope-S-L/git-hooks.git"
+    },
+    "engines": {
+        "node": ">=16.9.0"
+    },
+    "engineStrict": false,
+    "os": [
+        "darwin",
+        "linux",
+        "win32"
+    ],
+    "preferGlobal": true,
+    "files": [
+        "bin/",
+        "lib/",
+        "templates/",
+        "README.md",
+        "CHANGELOG.md",
+        "CLAUDE.md",
+        "LICENSE"
+    ],
+    "dependencies": {
+        "@octokit/rest": "^21.0.0"
+    },
+    "devDependencies": {
+        "@types/jest": "^29.5.0",
+        "eslint": "^8.57.0",
+        "jest": "^29.7.0",
+        "prettier": "^3.2.0"
+    }
+}