claude-git-hooks 2.18.0 → 2.19.0

package/lib/utils/interactive-ui.js

@@ -48,7 +48,7 @@ export const showPRPreview = (prData) => {
 
     // Show first 5 lines of body
     const bodyLines = body.split('\n').slice(0, 5);
-    bodyLines.forEach(line => {
+    bodyLines.forEach((line) => {
         console.log(` ${truncate(line, 61)}`);
     });
 
@@ -66,7 +66,7 @@ export const showPRPreview = (prData) => {
  */
 const truncate = (str, maxLen) => {
     const padded = str.padEnd(maxLen, ' ');
-    return padded.length > maxLen ? `${padded.substring(0, maxLen - 3)  }...` : padded;
+    return padded.length > maxLen ? `${padded.substring(0, maxLen - 3)}...` : padded;
 };
 
 /**
@@ -94,7 +94,9 @@ export const promptConfirmation = async (message, defaultValue = true) => {
 
             // If empty, use default
             if (!trimmed) {
-                logger.debug('interactive-ui - promptConfirmation', 'Using default', { defaultValue });
+                logger.debug('interactive-ui - promptConfirmation', 'Using default', {
+                    defaultValue
+                });
                 resolve(defaultValue);
                 return;
             }
@@ -111,10 +113,14 @@ export const promptConfirmation = async (message, defaultValue = true) => {
                 resolve(false);
             } else {
                 // Invalid input, use default
-                logger.debug('interactive-ui - promptConfirmation', 'Invalid input, using default', {
-                    answer: trimmed,
-                    defaultValue
-                });
+                logger.debug(
+                    'interactive-ui - promptConfirmation',
+                    'Invalid input, using default',
+                    {
+                        answer: trimmed,
+                        defaultValue
+                    }
+                );
                 resolve(defaultValue);
             }
         });
@@ -148,7 +154,9 @@ export const promptEditField = async (fieldName, currentValue) => {
             const trimmed = answer.trim();
 
             if (!trimmed) {
-                logger.debug('interactive-ui - promptEditField', 'Keeping current value', { fieldName });
+                logger.debug('interactive-ui - promptEditField', 'Keeping current value', {
+                    fieldName
+                });
                 resolve(currentValue);
             } else {
                 logger.debug('interactive-ui - promptEditField', 'Updated value', { fieldName });
@@ -179,7 +187,7 @@ export const promptMenu = async (message, options, defaultKey = null) => {
         console.log('');
 
         // Display options
-        options.forEach(opt => {
+        options.forEach((opt) => {
             const isDefault = opt.key === defaultKey;
             const marker = isDefault ? '→' : ' ';
             console.log(`  ${marker} [${opt.key}] ${opt.label}`);
@@ -203,10 +211,12 @@ export const promptMenu = async (message, options, defaultKey = null) => {
             }
 
             // Check if valid option
-            const selectedOption = options.find(opt => opt.key.toLowerCase() === trimmed);
+            const selectedOption = options.find((opt) => opt.key.toLowerCase() === trimmed);
 
             if (selectedOption) {
-                logger.debug('interactive-ui - promptMenu', 'Option selected', { key: selectedOption.key });
+                logger.debug('interactive-ui - promptMenu', 'Option selected', {
+                    key: selectedOption.key
+                });
                 resolve(selectedOption.key);
             } else {
                 // Invalid, use default or first option
@@ -358,7 +368,7 @@ export const promptToggleList = async (message, items) => {
     });
 
     // Clone items to track selection state
-    const state = items.map(item => ({
+    const state = items.map((item) => ({
         key: item.key,
         label: item.label,
         selected: item.selected !== undefined ? item.selected : true
@@ -395,8 +405,8 @@ export const promptToggleList = async (message, items) => {
                 if (!trimmed) {
                     rl.close();
                     const selectedKeys = state
-                        .filter(item => item.selected)
-                        .map(item => item.key);
+                        .filter((item) => item.selected)
+                        .map((item) => item.key);
 
                     logger.debug('interactive-ui - promptToggleList', 'Selection confirmed', {
                         selectedKeys
@@ -425,7 +435,9 @@ export const promptToggleList = async (message, items) => {
                     askForInput();
                 } else {
                     // Invalid input, show message and ask again
-                    console.log(`Invalid input. Enter a number between 1 and ${state.length}, or press Enter to confirm.`);
+                    console.log(
+                        `Invalid input. Enter a number between 1 and ${state.length}, or press Enter to confirm.`
+                    );
                     console.log('');
                     askForInput();
                 }
@@ -452,7 +464,7 @@ export const promptUserConfirmation = async (result) => {
 
     // Calculate total non-blocking issues
     const totalIssues = major + minor + info;
-    const fileCount = new Set(details.map(d => d.file)).size;
+    const fileCount = new Set(details.map((d) => d.file)).size;
 
     console.log('\n');
     console.log('⚠️  Non-blocking issues detected');
@@ -475,7 +487,7 @@ export const promptUserConfirmation = async (result) => {
         const choice = await promptMenu(
             'Do you want to continue with the commit?',
             options,
-            'n'  // Default to abort (safer)
+            'n' // Default to abort (safer)
         );
 
         if (choice === 'y') {

package/lib/utils/logger.js

@@ -96,7 +96,10 @@ class Logger {
         console.error(`${this.colors.red}❌ [${context}] ${message}${this.colors.reset}`);
 
         if (error) {
-            console.error(`${this.colors.red}Error details:${this.colors.reset}`, error.message || error);
+            console.error(
+                `${this.colors.red}Error details:${this.colors.reset}`,
+                error.message || error
+            );
 
             // Show stack trace only in debug mode to avoid overwhelming users
             if (error.stack && this.debugMode) {

package/lib/utils/pr-metadata-engine.js

@@ -62,8 +62,8 @@ import logger from './logger.js';
  * Why: Convention over configuration - sensible defaults for 95% of cases
  */
 const DEFAULTS = {
-    timeout: 180000,                    // 3 minutes
-    maxDiffSize: 50000,                 // 50KB
+    timeout: 180000, // 3 minutes
+    maxDiffSize: 50000 // 50KB
 };
 
 /**
@@ -93,7 +93,7 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
     // Always include file stats (never truncated)
     const statsOutput = getDiffBetweenRefs(baseRef, headRef, { contextLines: 0 })
         .split('\n')
-        .filter(line => line.match(/^\s*[\w/.-]+\s*\|\s*\d+\s*[+-]*$/))
+        .filter((line) => line.match(/^\s*[\w/.-]+\s*\|\s*\d+\s*[+-]*$/))
         .join('\n');
 
     const statsSummary = `Files changed: ${files.length}\n\n${statsOutput}\n\n`;
@@ -125,10 +125,14 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
         };
     }
 
-    logger.debug('pr-metadata-engine - buildDiffPayload', 'Tier 1: Exceeded budget, applying Tier 2', {
-        diffSize: fullDiff.length,
-        maxSize
-    });
+    logger.debug(
+        'pr-metadata-engine - buildDiffPayload',
+        'Tier 1: Exceeded budget, applying Tier 2',
+        {
+            diffSize: fullDiff.length,
+            maxSize
+        }
+    );
 
     // Tier 2: Split diff by file and allocate proportional budget
     const diffPattern = /^diff --git a\/.+ b\/.+$/gm;
@@ -146,9 +150,13 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
         fileDiffs.push(fullDiff.substring(lastIndex));
     }
 
-    logger.debug('pr-metadata-engine - buildDiffPayload', 'Tier 2: Split diff into per-file chunks', {
-        chunkCount: fileDiffs.length
-    });
+    logger.debug(
+        'pr-metadata-engine - buildDiffPayload',
+        'Tier 2: Split diff into per-file chunks',
+        {
+            chunkCount: fileDiffs.length
+        }
+    );
 
     const availableBudget = maxSize - summarySize;
     const budgetPerFile = Math.floor(availableBudget / files.length);
@@ -185,7 +193,10 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
 
     // Tier 3: Replace oversized files with stat-only summary
     if (oversizedFiles.length > 0) {
-        logger.debug('pr-metadata-engine - buildDiffPayload', 'Tier 3: Replacing oversized files with stat-only');
+        logger.debug(
+            'pr-metadata-engine - buildDiffPayload',
+            'Tier 3: Replacing oversized files with stat-only'
+        );
 
         for (const { diff } of oversizedFiles) {
             // Extract file path and stats from diff header
@@ -193,7 +204,7 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
             if (filePathMatch) {
                 const filePath = filePathMatch[2];
                 // Extract stat line for this file
-                const statLine = statsOutput.split('\n').find(line => line.includes(filePath));
+                const statLine = statsOutput.split('\n').find((line) => line.includes(filePath));
                 if (statLine) {
                     finalDiff += `\n--- ${filePath} (truncated - stat only) ---\n${statLine}\n`;
                     statOnlyCount++;
@@ -201,9 +212,13 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
             }
         }
 
-        logger.debug('pr-metadata-engine - buildDiffPayload', 'Tier 3: Stat-only demotion complete', {
-            statOnlyCount
-        });
+        logger.debug(
+            'pr-metadata-engine - buildDiffPayload',
+            'Tier 3: Stat-only demotion complete',
+            {
+                statOnlyCount
+            }
+        );
     }
 
     const payload = statsSummary + finalDiff;
@@ -235,7 +250,9 @@ const buildDiffPayload = (baseRef, headRef, files, maxSize = DEFAULTS.maxDiffSiz
  * @returns {Promise<{ success: boolean, context?: BranchContext, error?: string }>}
  */
 export const getBranchContext = async (targetBranch) => {
-    logger.debug('pr-metadata-engine - getBranchContext', 'Getting branch context', { targetBranch });
+    logger.debug('pr-metadata-engine - getBranchContext', 'Getting branch context', {
+        targetBranch
+    });
 
     try {
         // Get current branch
@@ -248,14 +265,20 @@ export const getBranchContext = async (targetBranch) => {
             };
         }
 
-        logger.debug('pr-metadata-engine - getBranchContext', 'Current branch identified', { currentBranch });
+        logger.debug('pr-metadata-engine - getBranchContext', 'Current branch identified', {
+            currentBranch
+        });
 
         // Fetch remote references
         try {
             fetchRemote();
             logger.debug('pr-metadata-engine - getBranchContext', 'Remote fetched successfully');
         } catch (fetchError) {
-            logger.warning('pr-metadata-engine - getBranchContext', 'Failed to fetch remote', fetchError);
+            logger.warning(
+                'pr-metadata-engine - getBranchContext',
+                'Failed to fetch remote',
+                fetchError
+            );
             // Continue anyway - may work with cached remote refs
         }
 
@@ -288,12 +311,11 @@ export const getBranchContext = async (targetBranch) => {
         logger.debug('pr-metadata-engine - getBranchContext', 'Commits retrieved');
 
         // Build diff payload with tiered reduction
-        const { payload: diff, isTruncated, truncationDetails } = buildDiffPayload(
-            baseBranch,
-            'HEAD',
-            files,
-            DEFAULTS.maxDiffSize
-        );
+        const {
+            payload: diff,
+            isTruncated,
+            truncationDetails
+        } = buildDiffPayload(baseBranch, 'HEAD', files, DEFAULTS.maxDiffSize);
 
         logger.debug('pr-metadata-engine - getBranchContext', 'Diff payload built', {
             diffSize: diff.length,
@@ -321,9 +343,12 @@ export const getBranchContext = async (targetBranch) => {
             success: true,
             context
         };
-
     } catch (error) {
-        logger.error('pr-metadata-engine - getBranchContext', 'Failed to get branch context', error);
+        logger.error(
+            'pr-metadata-engine - getBranchContext',
+            'Failed to get branch context',
+            error
+        );
 
         return {
             success: false,
@@ -411,9 +436,12 @@ export const generatePRMetadata = async (context, options = {}) => {
             testingNotes: result.testingNotes || '',
             context
         };
-
     } catch (error) {
-        logger.error('pr-metadata-engine - generatePRMetadata', 'Failed to generate PR metadata', error);
+        logger.error(
+            'pr-metadata-engine - generatePRMetadata',
+            'Failed to generate PR metadata',
+            error
+        );
         throw error;
     }
 };
@@ -439,9 +467,13 @@ export const analyzeBranchForPR = async (targetBranch, options = {}) => {
         const contextResult = await getBranchContext(targetBranch);
 
         if (!contextResult.success) {
-            logger.error('pr-metadata-engine - analyzeBranchForPR', 'Failed to get branch context', {
-                error: contextResult.error
-            });
+            logger.error(
+                'pr-metadata-engine - analyzeBranchForPR',
+                'Failed to get branch context',
+                {
+                    error: contextResult.error
+                }
+            );
 
             return {
                 success: false,
@@ -456,13 +488,15 @@ export const analyzeBranchForPR = async (targetBranch, options = {}) => {
         // Step 2: Generate PR metadata
         const metadata = await generatePRMetadata(contextResult.context, options);
 
-        logger.debug('pr-metadata-engine - analyzeBranchForPR', 'PR metadata generated successfully');
+        logger.debug(
+            'pr-metadata-engine - analyzeBranchForPR',
+            'PR metadata generated successfully'
+        );
 
         return {
             success: true,
             result: metadata
         };
-
     } catch (error) {
         logger.error('pr-metadata-engine - analyzeBranchForPR', 'PR analysis failed', error);
 

package/lib/utils/preset-loader.js

@@ -40,11 +40,7 @@ class PresetError extends Error {
  * @returns {Promise<Object>} { metadata, config, templates }
  */
 export async function loadPreset(presetName) {
-    logger.debug(
-        'preset-loader - loadPreset',
-        'Loading preset',
-        { presetName }
-    );
+    logger.debug('preset-loader - loadPreset', 'Loading preset', { presetName });
 
     const repoRoot = getRepoRoot();
 
@@ -70,25 +66,16 @@ export async function loadPreset(presetName) {
             templates[key] = path.join(presetDir, templatePath);
         }
 
-        logger.debug(
-            'preset-loader - loadPreset',
-            'Preset loaded successfully',
-            {
-                presetName,
-                displayName: metadata.displayName,
-                fileExtensions: metadata.fileExtensions,
-                templateCount: Object.keys(templates).length
-            }
-        );
+        logger.debug('preset-loader - loadPreset', 'Preset loaded successfully', {
+            presetName,
+            displayName: metadata.displayName,
+            fileExtensions: metadata.fileExtensions,
+            templateCount: Object.keys(templates).length
+        });
 
         return { metadata, config, templates };
-
     } catch (error) {
-        logger.error(
-            'preset-loader - loadPreset',
-            `Failed to load preset: ${presetName}`,
-            error
-        );
+        logger.error('preset-loader - loadPreset', `Failed to load preset: ${presetName}`, error);
 
         throw new PresetError(`Preset "${presetName}" not found or invalid`, {
             presetName,
@@ -104,44 +91,23 @@ export async function loadPreset(presetName) {
  * @returns {Promise<string>} Template content with placeholders replaced
  */
 export async function loadTemplate(templatePath, metadata) {
-    logger.debug(
-        'preset-loader - loadTemplate',
-        'Loading template',
-        { templatePath }
-    );
+    logger.debug('preset-loader - loadTemplate', 'Loading template', { templatePath });
 
     try {
         let template = await fs.readFile(templatePath, 'utf8');
 
         // Replace placeholders
-        template = template.replace(
-            /{{TECH_STACK}}/g,
-            metadata.techStack.join(', ')
-        );
-        template = template.replace(
-            /{{FOCUS_AREAS}}/g,
-            metadata.focusAreas.join(', ')
-        );
-        template = template.replace(
-            /{{FILE_EXTENSIONS}}/g,
-            metadata.fileExtensions.join(', ')
-        );
-        template = template.replace(
-            /{{PRESET_NAME}}/g,
-            metadata.name
-        );
-
-        logger.debug(
-            'preset-loader - loadTemplate',
-            'Template loaded and processed',
-            {
-                templatePath,
-                originalLength: template.length
-            }
-        );
+        template = template.replace(/{{TECH_STACK}}/g, metadata.techStack.join(', '));
+        template = template.replace(/{{FOCUS_AREAS}}/g, metadata.focusAreas.join(', '));
+        template = template.replace(/{{FILE_EXTENSIONS}}/g, metadata.fileExtensions.join(', '));
+        template = template.replace(/{{PRESET_NAME}}/g, metadata.name);
+
+        logger.debug('preset-loader - loadTemplate', 'Template loaded and processed', {
+            templatePath,
+            originalLength: template.length
+        });
 
         return template;
-
     } catch (error) {
         logger.error(
             'preset-loader - loadTemplate',
@@ -195,18 +161,10 @@ export async function listPresets() {
             `Expected location: ${presetsDir}`
         ]);
         logger.warning(errorMsg);
-        logger.debug(
-            'preset-loader - listPresets',
-            'Failed to read presets directory',
-            error
-        );
+        logger.debug('preset-loader - listPresets', 'Failed to read presets directory', error);
     }
 
-    logger.debug(
-        'preset-loader - listPresets',
-        'Presets listed',
-        { count: presets.length }
-    );
+    logger.debug('preset-loader - listPresets', 'Presets listed', { count: presets.length });
 
     return presets;
 }

package/lib/utils/prompt-builder.js

@@ -55,29 +55,23 @@ const loadTemplate = async (templateName, baseDir = '.claude/prompts') => {
         templatePath = path.join(repoRoot, 'templates', templateName);
     }
 
-    logger.debug(
-        'prompt-builder - loadTemplate',
-        'Loading template',
-        { templateName, repoRoot, templatePath }
-    );
+    logger.debug('prompt-builder - loadTemplate', 'Loading template', {
+        templateName,
+        repoRoot,
+        templatePath
+    });
 
     try {
         const content = await fs.readFile(templatePath, 'utf8');
 
-        logger.debug(
-            'prompt-builder - loadTemplate',
-            'Template loaded successfully',
-            { templatePath, contentLength: content.length }
-        );
+        logger.debug('prompt-builder - loadTemplate', 'Template loaded successfully', {
+            templatePath,
+            contentLength: content.length
+        });
 
         return content;
-
     } catch (error) {
-        logger.error(
-            'prompt-builder - loadTemplate',
-            `Template not found: ${templatePath}`,
-            error
-        );
+        logger.error('prompt-builder - loadTemplate', `Template not found: ${templatePath}`, error);
 
         throw new PromptBuilderError('Template file not found', {
             templatePath,
@@ -99,15 +93,14 @@ const loadTemplate = async (templateName, baseDir = '.claude/prompts') => {
  * => 'Repo: my-repo'
  */
 const replaceTemplate = (template, variables) => {
-    logger.debug(
-        'prompt-builder - replaceTemplate',
-        'Replacing template placeholders',
-        { variableCount: Object.keys(variables).length }
-    );
+    logger.debug('prompt-builder - replaceTemplate', 'Replacing template placeholders', {
+        variableCount: Object.keys(variables).length
+    });
 
-    return Object.entries(variables).reduce((result, [key, value]) => 
-        result.replace(new RegExp(`\\{\\{${key}\\}\\}`, 'g'), value)
-    , template);
+    return Object.entries(variables).reduce(
+        (result, [key, value]) => result.replace(new RegExp(`\\{\\{${key}\\}\\}`, 'g'), value),
+        template
+    );
 };
 
 /**
@@ -129,11 +122,10 @@ const replaceTemplate = (template, variables) => {
  * });
  */
 const loadPrompt = async (templateName, variables = {}, baseDir = '.claude/prompts') => {
-    logger.debug(
-        'prompt-builder - loadPrompt',
-        'Loading prompt with variables',
-        { templateName, variableCount: Object.keys(variables).length }
-    );
+    logger.debug('prompt-builder - loadPrompt', 'Loading prompt with variables', {
+        templateName,
+        variableCount: Object.keys(variables).length
+    });
 
     try {
         // Load template
@@ -142,14 +134,12 @@ const loadPrompt = async (templateName, variables = {}, baseDir = '.claude/promp
         // Replace variables
         const prompt = replaceTemplate(template, variables);
 
-        logger.debug(
-            'prompt-builder - loadPrompt',
-            'Prompt loaded successfully',
-            { templateName, promptLength: prompt.length }
-        );
+        logger.debug('prompt-builder - loadPrompt', 'Prompt loaded successfully', {
+            templateName,
+            promptLength: prompt.length
+        });
 
         return prompt;
-
     } catch (error) {
         logger.error(
             'prompt-builder - loadPrompt',
@@ -173,11 +163,12 @@ const loadPrompt = async (templateName, variables = {}, baseDir = '.claude/promp
  * @returns {string} Formatted file section
  */
 const formatFileSection = ({ path, diff, content, isNew }) => {
-    logger.debug(
-        'prompt-builder - formatFileSection',
-        'Formatting file section',
-        { path, isNew, hasDiff: !!diff, hasContent: !!content }
-    );
+    logger.debug('prompt-builder - formatFileSection', 'Formatting file section', {
+        path,
+        isNew,
+        hasDiff: !!diff,
+        hasContent: !!content
+    });
 
     let section = `\n--- Archivo: ${path} ---\n`;
 
@@ -220,11 +211,13 @@ const buildAnalysisPrompt = async ({
     subagentConfig = null,
     baseDir = '.claude/prompts'
 } = {}) => {
-    logger.debug(
-        'prompt-builder - buildAnalysisPrompt',
-        'Building analysis prompt',
-        { templateName, guidelinesName, fileCount: files.length, subagentsEnabled: subagentConfig?.enabled, baseDir }
-    );
+    logger.debug('prompt-builder - buildAnalysisPrompt', 'Building analysis prompt', {
+        templateName,
+        guidelinesName,
+        fileCount: files.length,
+        subagentsEnabled: subagentConfig?.enabled,
+        baseDir
+    });
 
     try {
         // Load template and guidelines
@@ -244,11 +237,15 @@ 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`);
+                logger.info(
+                    `🚀 Batch optimization enabled: ${files.length} files, ${subagentVariables.BATCH_SIZE} per batch, ${subagentVariables.MODEL} model`
+                );
             } catch (error) {
-                logger.warning('Subagent instruction template not found, proceeding without parallel analysis');
+                logger.warning(
+                    'Subagent instruction template not found, proceeding without parallel analysis'
+                );
             }
         }
 
@@ -260,7 +257,7 @@ const buildAnalysisPrompt = async ({
         prompt += '\n\n=== CHANGES TO REVIEW ===\n';
 
         // Add each file
-        files.forEach(fileData => {
+        files.forEach((fileData) => {
             prompt += formatFileSection(fileData);
         });
 
@@ -269,14 +266,12 @@ const buildAnalysisPrompt = async ({
             prompt = replaceTemplate(prompt, metadata);
         }
 
-        logger.debug(
-            'prompt-builder - buildAnalysisPrompt',
-            'Prompt built successfully',
-            { promptLength: prompt.length, fileCount: files.length }
-        );
+        logger.debug('prompt-builder - buildAnalysisPrompt', 'Prompt built successfully', {
+            promptLength: prompt.length,
+            fileCount: files.length
+        });
 
         return prompt;
-
     } catch (error) {
         logger.error(
             'prompt-builder - buildAnalysisPrompt',