claude-git-hooks 2.18.0 → 2.19.0

package/lib/utils/resolution-prompt.js

@@ -46,11 +46,9 @@ class ResolutionPromptError extends Error {
  * Detailed: { message, file, line, method, severity, type, rule }
  */
 const formatBlockingIssues = (issues) => {
-    logger.debug(
-        'resolution-prompt - formatBlockingIssues',
-        'Formatting issues',
-        { issueCount: issues?.length || 0 }
-    );
+    logger.debug('resolution-prompt - formatBlockingIssues', 'Formatting issues', {
+        issueCount: issues?.length || 0
+    });
 
     if (!Array.isArray(issues) || issues.length === 0) {
         return 'No issues found.';
@@ -94,16 +92,15 @@ const getAffectedFiles = (issues) => {
 
     // Extract unique file paths
     const filePaths = issues
-        .map(issue => issue.file)
-        .filter(file => file && typeof file === 'string');
+        .map((issue) => issue.file)
+        .filter((file) => file && typeof file === 'string');
 
     const uniqueFiles = [...new Set(filePaths)];
 
-    logger.debug(
-        'resolution-prompt - getAffectedFiles',
-        'Extracted affected files',
-        { totalIssues: issues.length, uniqueFiles: uniqueFiles.length }
-    );
+    logger.debug('resolution-prompt - getAffectedFiles', 'Extracted affected files', {
+        totalIssues: issues.length,
+        uniqueFiles: uniqueFiles.length
+    });
 
     return uniqueFiles;
 };
@@ -117,11 +114,9 @@ const getAffectedFiles = (issues) => {
  * @returns {Promise<string>} Formatted file contents in markdown
  */
 const formatFileContents = async (filePaths) => {
-    logger.debug(
-        'resolution-prompt - formatFileContents',
-        'Reading file contents',
-        { fileCount: filePaths.length }
-    );
+    logger.debug('resolution-prompt - formatFileContents', 'Reading file contents', {
+        fileCount: filePaths.length
+    });
 
     // Why: Use repo root for absolute paths (works on Windows/PowerShell/Git Bash)
     const repoRoot = getRepoRoot();
@@ -154,15 +149,13 @@ ${content}
     );
 
     const formatted = fileContents
-        .filter(result => result.status === 'fulfilled')
-        .map(result => result.value)
+        .filter((result) => result.status === 'fulfilled')
+        .map((result) => result.value)
         .join('\n');
 
-    logger.debug(
-        'resolution-prompt - formatFileContents',
-        'File contents formatted',
-        { successfulReads: fileContents.filter(r => r.status === 'fulfilled').length }
-    );
+    logger.debug('resolution-prompt - formatFileContents', 'File contents formatted', {
+        successfulReads: fileContents.filter((r) => r.status === 'fulfilled').length
+    });
 
     return formatted;
 };
@@ -202,21 +195,18 @@ const generateResolutionPrompt = async (
 
     // Use details array (all issues) if available, fallback to blockingIssues
     // Why: analyze command provides full details, pre-commit may only have blockingIssues
-    const allIssues = (Array.isArray(analysisResult.details) && analysisResult.details.length > 0)
-        ? analysisResult.details
-        : (analysisResult.blockingIssues || []);
-
-    logger.debug(
-        'resolution-prompt - generateResolutionPrompt',
-        'Generating resolution prompt',
-        {
-            repoRoot,
-            outputPath: absoluteOutputPath,
-            templatePath: absoluteTemplatePath,
-            issueCount: allIssues.length,
-            usingDetails: Array.isArray(analysisResult.details) && analysisResult.details.length > 0
-        }
-    );
+    const allIssues =
+        Array.isArray(analysisResult.details) && analysisResult.details.length > 0
+            ? analysisResult.details
+            : analysisResult.blockingIssues || [];
+
+    logger.debug('resolution-prompt - generateResolutionPrompt', 'Generating resolution prompt', {
+        repoRoot,
+        outputPath: absoluteOutputPath,
+        templatePath: absoluteTemplatePath,
+        issueCount: allIssues.length,
+        usingDetails: Array.isArray(analysisResult.details) && analysisResult.details.length > 0
+    });
 
     try {
         // Load template
@@ -265,7 +255,6 @@ const generateResolutionPrompt = async (
         );
 
         return absoluteOutputPath;
-
     } catch (error) {
         logger.error(
             'resolution-prompt - generateResolutionPrompt',
@@ -295,10 +284,10 @@ const generateResolutionPrompt = async (
  * @returns {boolean} True if should generate prompt
  */
 const shouldGeneratePrompt = (analysisResult) => {
-    const hasBlockingIssues = Array.isArray(analysisResult.blockingIssues) &&
-                              analysisResult.blockingIssues.length > 0;
-    const hasDetailedIssues = Array.isArray(analysisResult.details) &&
-                              analysisResult.details.length > 0;
+    const hasBlockingIssues =
+        Array.isArray(analysisResult.blockingIssues) && analysisResult.blockingIssues.length > 0;
+    const hasDetailedIssues =
+        Array.isArray(analysisResult.details) && analysisResult.details.length > 0;
 
     logger.debug(
         'resolution-prompt - shouldGeneratePrompt',

package/lib/utils/sanitize.js

@@ -19,11 +19,10 @@ import logger from './logger.js';
  * @param {boolean} options.stripControlChars - Remove control characters (default: true)
  * @returns {string} - Sanitized string
  */
-export const sanitizeInput = (input, {
-    maxLength = 65536,
-    allowNewlines = true,
-    stripControlChars = true
-} = {}) => {
+export const sanitizeInput = (
+    input,
+    { maxLength = 65536, allowNewlines = true, stripControlChars = true } = {}
+) => {
     if (typeof input !== 'string') {
         return '';
     }
@@ -62,11 +61,12 @@ export const sanitizeInput = (input, {
  * @param {string} title - Raw title
  * @returns {string} - Sanitized title
  */
-export const sanitizePRTitle = (title) => 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,11 +75,12 @@ export const sanitizePRTitle = (title) => sanitizeInput(title, {
  * @param {string} body - Raw body text
  * @returns {string} - Sanitized body
  */
-export const sanitizePRBody = (body) => 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)
@@ -95,10 +96,10 @@ export const sanitizeStringArray = (items, validPattern = /^[\w.-]+$/) => {
     }
 
     return items
-        .filter(item => typeof item === 'string')
-        .map(item => item.trim())
-        .filter(item => item.length > 0 && item.length <= 100)
-        .filter(item => validPattern.test(item));
+        .filter((item) => typeof item === 'string')
+        .map((item) => item.trim())
+        .filter((item) => item.length > 0 && item.length <= 100)
+        .filter((item) => validPattern.test(item));
 };
 
 /**

package/lib/utils/task-id.js

@@ -86,11 +86,7 @@ export const extractTaskId = (branchName, config = null) => {
         // Get first capture group
         const taskId = match[1];
         if (taskId) {
-            logger.debug(
-                'task-id - extractTaskId',
-                'Task ID found',
-                { branchName, taskId }
-            );
+            logger.debug('task-id - extractTaskId', 'Task ID found', { branchName, taskId });
             return taskId.toUpperCase(); // Normalize to uppercase
         }
     }
@@ -114,11 +110,9 @@ export const extractTaskIdFromCurrentBranch = (config = null) => {
 
         return extractTaskId(branchName, config);
     } catch (error) {
-        logger.debug(
-            'task-id - extractTaskIdFromCurrentBranch',
-            'Failed to get current branch',
-            { error: error.message }
-        );
+        logger.debug('task-id - extractTaskIdFromCurrentBranch', 'Failed to get current branch', {
+            error: error.message
+        });
         return null;
     }
 };
@@ -173,11 +167,10 @@ export const formatWithTaskId = (message, taskId) => {
     // Check if message already has task ID prefix (idempotent)
     const taskIdPrefix = `[${normalizedTaskId}]`;
     if (message.startsWith(taskIdPrefix)) {
-        logger.debug(
-            'task-id - formatWithTaskId',
-            'Message already has task ID prefix',
-            { message, taskId: normalizedTaskId }
-        );
+        logger.debug('task-id - formatWithTaskId', 'Message already has task ID prefix', {
+            message,
+            taskId: normalizedTaskId
+        });
         return message;
     }
 
@@ -186,25 +179,21 @@ export const formatWithTaskId = (message, taskId) => {
     if (existingTaskIdMatch) {
         // Replace existing task ID with new one
         const formattedMessage = message.replace(existingTaskIdMatch[0], `${taskIdPrefix} `);
-        logger.debug(
-            'task-id - formatWithTaskId',
-            'Replaced existing task ID',
-            {
-                oldTaskId: existingTaskIdMatch[1],
-                newTaskId: normalizedTaskId,
-                formattedMessage
-            }
-        );
+        logger.debug('task-id - formatWithTaskId', 'Replaced existing task ID', {
+            oldTaskId: existingTaskIdMatch[1],
+            newTaskId: normalizedTaskId,
+            formattedMessage
+        });
         return formattedMessage;
     }
 
     // Prepend task ID
     const formattedMessage = `${taskIdPrefix} ${message}`;
-    logger.debug(
-        'task-id - formatWithTaskId',
-        'Added task ID prefix',
-        { message, taskId: normalizedTaskId, formattedMessage }
-    );
+    logger.debug('task-id - formatWithTaskId', 'Added task ID prefix', {
+        message,
+        taskId: normalizedTaskId,
+        formattedMessage
+    });
 
     return formattedMessage;
 };
@@ -246,7 +235,9 @@ export const promptForTaskId = async ({
     config = null
 } = {}) => {
     // Try to use /dev/tty for Unix (git hooks), fallback to stdin for Windows
-    let input, output, usingTty = false;
+    let input,
+        output,
+        usingTty = false;
 
     // Check platform
     if (process.platform !== 'win32') {
@@ -281,9 +272,7 @@ export const promptForTaskId = async ({
 
     return new Promise((resolve) => {
         const askQuestion = () => {
-            const promptMessage = allowSkip
-                ? `${message} (press Enter to skip): `
-                : `${message} `;
+            const promptMessage = allowSkip ? `${message} (press Enter to skip): ` : `${message} `;
 
             rl.question(promptMessage, (answer) => {
                 const trimmedAnswer = answer.trim();
@@ -317,11 +306,9 @@ export const promptForTaskId = async ({
 
                 // Valid task ID provided
                 const normalizedTaskId = trimmedAnswer.toUpperCase();
-                logger.debug(
-                    'task-id - promptForTaskId',
-                    'User provided task ID',
-                    { taskId: normalizedTaskId }
-                );
+                logger.debug('task-id - promptForTaskId', 'User provided task ID', {
+                    taskId: normalizedTaskId
+                });
                 rl.close();
                 if (usingTty) {
                     input.close();
@@ -432,8 +419,8 @@ const getExamplesForPattern = (patternName) => {
     const examples = {
         'Jira-style': ['IX-123', 'PROJ-456', 'ABC-789'],
         'GitHub issue': ['#123', 'GH-456'],
-        'Linear': ['LIN-123', 'LIN-456'],
-        'Generic': ['TASK-123', 'BUG-456', 'FEAT-789']
+        Linear: ['LIN-123', 'LIN-456'],
+        Generic: ['TASK-123', 'BUG-456', 'FEAT-789']
     };
     return examples[patternName] || [];
 };

package/lib/utils/telemetry.js

@@ -88,11 +88,9 @@ 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
-    config.system?.telemetry !== false
-;
-
+    config.system?.telemetry !== false;
 /**
  * Ensure telemetry directory exists
  * Why: Create on first use
@@ -115,7 +113,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');
@@ -240,7 +238,7 @@ const readTelemetryEvents = async (maxDays = 7) => {
         const files = await fs.readdir(dir);
 
         // Filter to .jsonl files only
-        const logFiles = files.filter(f => f.endsWith('.jsonl'));
+        const logFiles = files.filter((f) => f.endsWith('.jsonl'));
 
         // Sort by date (newest first)
         logFiles.sort().reverse();
@@ -262,7 +260,11 @@ const readTelemetryEvents = async (maxDays = 7) => {
                         events.push(JSON.parse(line));
                     } catch (parseError) {
                         // Skip invalid lines
-                        logger.debug('telemetry - readTelemetryEvents', 'Invalid JSON line', parseError);
+                        logger.debug(
+                            'telemetry - readTelemetryEvents',
+                            'Invalid JSON line',
+                            parseError
+                        );
                     }
                 }
             }
@@ -286,7 +288,8 @@ export const getStatistics = async (maxDays = 7) => {
     if (!isTelemetryEnabled()) {
         return {
             enabled: false,
-            message: 'Telemetry is disabled. To enable (default), remove or set "system.telemetry: true" in .claude/config.json'
+            message:
+                'Telemetry is disabled. To enable (default), remove or set "system.telemetry: true" in .claude/config.json'
         };
     }
 
@@ -311,14 +314,15 @@ export const getStatistics = async (maxDays = 7) => {
         let totalFilesInFailures = 0;
         let totalFilesInSuccesses = 0;
 
-        events.forEach(event => {
+        events.forEach((event) => {
             if (event.type === 'json_parse_failure') {
                 stats.jsonParseFailures++;
                 totalFilesInFailures += event.data.fileCount || 0;
 
                 // Group by batch size
                 const batchSize = event.data.batchSize || 0;
-                stats.failuresByBatchSize[batchSize] = (stats.failuresByBatchSize[batchSize] || 0) + 1;
+                stats.failuresByBatchSize[batchSize] =
+                    (stats.failuresByBatchSize[batchSize] || 0) + 1;
 
                 // Group by model
                 const model = event.data.model || 'unknown';
@@ -327,7 +331,6 @@ export const getStatistics = async (maxDays = 7) => {
                 // Group by hook
                 const hook = event.data.hook || 'unknown';
                 stats.failuresByHook[hook] = (stats.failuresByHook[hook] || 0) + 1;
-
             } else if (event.type === 'batch_success') {
                 stats.batchSuccesses++;
                 totalFilesInSuccesses += event.data.fileCount || 0;
@@ -340,16 +343,22 @@ export const getStatistics = async (maxDays = 7) => {
 
         // Calculate averages
         if (stats.jsonParseFailures > 0) {
-            stats.avgFilesPerFailure = parseFloat((totalFilesInFailures / stats.jsonParseFailures).toFixed(2));
+            stats.avgFilesPerFailure = parseFloat(
+                (totalFilesInFailures / stats.jsonParseFailures).toFixed(2)
+            );
         }
         if (stats.batchSuccesses > 0) {
-            stats.avgFilesPerSuccess = parseFloat((totalFilesInSuccesses / stats.batchSuccesses).toFixed(2));
+            stats.avgFilesPerSuccess = parseFloat(
+                (totalFilesInSuccesses / stats.batchSuccesses).toFixed(2)
+            );
         }
 
         // Calculate failure rate
         const totalAnalyses = stats.jsonParseFailures + stats.batchSuccesses;
         if (totalAnalyses > 0) {
-            stats.failureRate = parseFloat((stats.jsonParseFailures / totalAnalyses * 100).toFixed(2));
+            stats.failureRate = parseFloat(
+                ((stats.jsonParseFailures / totalAnalyses) * 100).toFixed(2)
+            );
         }
 
         return stats;
@@ -379,7 +388,7 @@ export const rotateTelemetry = async (maxDays = 30) => {
         const files = await fs.readdir(dir);
 
         // Filter to .jsonl files
-        const logFiles = files.filter(f => f.endsWith('.jsonl'));
+        const logFiles = files.filter((f) => f.endsWith('.jsonl'));
 
         // Calculate cutoff date
         const cutoffDate = new Date();
@@ -395,7 +404,10 @@ export const rotateTelemetry = async (maxDays = 30) => {
                 if (fileDate < cutoffStr) {
                     const filePath = path.join(dir, file);
                     await fs.unlink(filePath);
-                    logger.debug('telemetry - rotateTelemetry', `Deleted old telemetry file: ${file}`);
+                    logger.debug(
+                        'telemetry - rotateTelemetry',
+                        `Deleted old telemetry file: ${file}`
+                    );
                 }
             }
         }
@@ -420,7 +432,7 @@ export const clearTelemetry = async () => {
 
         // Delete all .jsonl files
         const files = await fs.readdir(dir);
-        const logFiles = files.filter(f => f.endsWith('.jsonl'));
+        const logFiles = files.filter((f) => f.endsWith('.jsonl'));
 
         for (const file of logFiles) {
             const filePath = path.join(dir, file);