claude-git-hooks 2.18.0 → 2.19.0

package/lib/commands/help.js

@@ -14,11 +14,7 @@ import path from 'path';
 import { getPackageJson } from './helpers.js';
 import { executeClaudeWithRetry } from '../utils/claude-client.js';
 import { loadPrompt } from '../utils/prompt-builder.js';
-import {
-    fetchFileContent,
-    fetchDirectoryListing,
-    createIssue
-} from '../utils/github-api.js';
+import { fetchFileContent, fetchDirectoryListing, createIssue } from '../utils/github-api.js';
 import { promptMenu, promptEditField, promptConfirmation } from '../utils/interactive-ui.js';
 import logger from '../utils/logger.js';
 
@@ -147,7 +143,10 @@ const readClaudeMd = async () => {
         logger.debug('help - readClaudeMd', 'CLAUDE.md loaded', { length: content.length });
         return content;
     } catch (error) {
-        logger.debug('help - readClaudeMd', 'CLAUDE.md not found', { path: claudeMdPath, error: error.message });
+        logger.debug('help - readClaudeMd', 'CLAUDE.md not found', {
+            path: claudeMdPath,
+            error: error.message
+        });
         return null;
     }
 };
@@ -170,7 +169,10 @@ async function runAiHelp(question) {
     try {
         const claudeMdContent = await readClaudeMd();
         if (!claudeMdContent) {
-            logger.debug('help - runAiHelp', 'CLAUDE.md not available, falling back to static help');
+            logger.debug(
+                'help - runAiHelp',
+                'CLAUDE.md not available, falling back to static help'
+            );
             showStaticHelp();
             return;
         }
@@ -190,9 +192,15 @@ async function runAiHelp(question) {
         const trimmedResponse = response.trim();
 
         // Check for NEED_MORE_CONTEXT second pass
-        const needMoreLine = trimmedResponse.split('\n').find(l => l.includes('NEED_MORE_CONTEXT'));
+        const needMoreLine = trimmedResponse
+            .split('\n')
+            .find((l) => l.includes('NEED_MORE_CONTEXT'));
         if (needMoreLine) {
-            const enrichedResponse = await handleNeedMoreContext(needMoreLine, question, claudeMdContent);
+            const enrichedResponse = await handleNeedMoreContext(
+                needMoreLine,
+                question,
+                claudeMdContent
+            );
             if (enrichedResponse) {
                 printAiResponse(enrichedResponse, `${localVersion} + source`);
                 return;
@@ -200,7 +208,7 @@ async function runAiHelp(question) {
             // Enrichment failed: show first-pass answer with marker line stripped
             const cleanResponse = trimmedResponse
                 .split('\n')
-                .filter(l => !l.includes('NEED_MORE_CONTEXT'))
+                .filter((l) => !l.includes('NEED_MORE_CONTEXT'))
                 .join('\n')
                 .trim();
             if (cleanResponse) {
@@ -211,7 +219,9 @@ async function runAiHelp(question) {
 
         printAiResponse(trimmedResponse, localVersion);
     } catch (error) {
-        logger.debug('help - runAiHelp', 'AI help failed, falling back to static help', { error: error.message });
+        logger.debug('help - runAiHelp', 'AI help failed, falling back to static help', {
+            error: error.message
+        });
         showStaticHelp();
     }
 }
@@ -228,7 +238,10 @@ async function handleNeedMoreContext(needMoreLine, question, claudeMdContent) {
     try {
         // Parse file paths from line: "NEED_MORE_CONTEXT: file1.js, file2.js"
         const pathsPart = needMoreLine.replace('NEED_MORE_CONTEXT', '').replace(':', '').trim();
-        const filePaths = pathsPart.split(',').map(p => p.trim()).filter(Boolean);
+        const filePaths = pathsPart
+            .split(',')
+            .map((p) => p.trim())
+            .filter(Boolean);
 
         if (filePaths.length === 0) {
             logger.debug('help - handleNeedMoreContext', 'No file paths in NEED_MORE_CONTEXT');
@@ -249,8 +262,8 @@ async function handleNeedMoreContext(needMoreLine, question, claudeMdContent) {
 
         // Build enriched documentation
         const additionalContent = fetchResults
-            .filter(r => r.content !== null)
-            .map(r => `\n--- Source: ${r.filePath} ---\n${r.content}`)
+            .filter((r) => r.content !== null)
+            .map((r) => `\n--- Source: ${r.filePath} ---\n${r.content}`)
             .join('\n');
 
         if (!additionalContent) {
@@ -295,13 +308,15 @@ async function runReportIssue() {
         if (!templates || templates.length === 0) {
             logger.debug('help - runReportIssue', 'No issue templates found');
             console.log('\nNo issue templates found in .github/ISSUE_TEMPLATE/');
-            console.log(`Create issues directly at: https://github.com/${owner}/${repo}/issues/new`);
+            console.log(
+                `Create issues directly at: https://github.com/${owner}/${repo}/issues/new`
+            );
             return;
         }
 
         // Filter to markdown/yaml template files
-        const templateFiles = templates.filter(t =>
-            t.name.endsWith('.md') || t.name.endsWith('.yml') || t.name.endsWith('.yaml')
+        const templateFiles = templates.filter(
+            (t) => t.name.endsWith('.md') || t.name.endsWith('.yml') || t.name.endsWith('.yaml')
         );
 
         if (templateFiles.length === 0) {
@@ -345,12 +360,16 @@ async function runReportIssue() {
             const jsonMatch = questionsResponse.match(/\[[\s\S]*\]/);
             questions = jsonMatch ? JSON.parse(jsonMatch[0]) : null;
         } catch (parseError) {
-            logger.debug('help - runReportIssue', 'Failed to parse questions JSON', { error: parseError.message });
+            logger.debug('help - runReportIssue', 'Failed to parse questions JSON', {
+                error: parseError.message
+            });
             questions = null;
         }
 
         if (!questions || !Array.isArray(questions) || questions.length === 0) {
-            console.log('\nCould not generate questions from template. Please create the issue manually:');
+            console.log(
+                '\nCould not generate questions from template. Please create the issue manually:'
+            );
             console.log(`https://github.com/${owner}/${repo}/issues/new`);
             return;
         }
@@ -359,10 +378,7 @@ async function runReportIssue() {
         console.log('\nPlease answer the following questions:\n');
         const answers = [];
         for (const q of questions) {
-            const answer = await promptEditField(
-                q.section,
-                q.question
-            );
+            const answer = await promptEditField(q.section, q.question);
             answers.push({ section: q.section, answer });
         }
 
@@ -370,7 +386,7 @@ async function runReportIssue() {
         const composePrompt = await loadPrompt('HELP_COMPOSE_ISSUE.md', {
             TEMPLATE_NAME: selectedTemplate.name,
             TEMPLATE_CONTENT: templateContent,
-            USER_ANSWERS: answers.map(a => `${a.section}: ${a.answer}`).join('\n')
+            USER_ANSWERS: answers.map((a) => `${a.section}: ${a.answer}`).join('\n')
         });
 
         console.log('\nComposing issue...\n');
@@ -381,7 +397,9 @@ async function runReportIssue() {
             const jsonMatch = composeResponse.match(/\{[\s\S]*\}/);
             issueData = jsonMatch ? JSON.parse(jsonMatch[0]) : null;
         } catch (parseError) {
-            logger.debug('help - runReportIssue', 'Failed to parse issue JSON', { error: parseError.message });
+            logger.debug('help - runReportIssue', 'Failed to parse issue JSON', {
+                error: parseError.message
+            });
             issueData = null;
         }
 
@@ -413,7 +431,9 @@ async function runReportIssue() {
     } catch (error) {
         logger.debug('help - runReportIssue', 'Report issue failed', { error: error.message });
         console.log(`\nCould not create issue: ${error.message}`);
-        console.log('You can create issues manually at: https://github.com/mscope-S-L/git-hooks/issues/new');
+        console.log(
+            'You can create issues manually at: https://github.com/mscope-S-L/git-hooks/issues/new'
+        );
     }
 }
 
@@ -434,6 +454,6 @@ function extractLabelsFromTemplate(templateContent) {
 
     return labelsMatch[1]
         .split(',')
-        .map(l => l.trim().replace(/^["']|["']$/g, ''))
+        .map((l) => l.trim().replace(/^["']|["']$/g, ''))
         .filter(Boolean);
 }

package/lib/commands/helpers.js

@@ -71,7 +71,10 @@ export function checkGitRepo() {
                     let gitdir = gitContent.substring(8).trim();
                     // Convert Windows path to WSL if needed (C:\ -> /mnt/c/)
                     if (/^[A-Za-z]:/.test(gitdir)) {
-                        gitdir = gitdir.replace(/^([A-Za-z]):/, (_, drive) => `/mnt/${drive.toLowerCase()}`);
+                        gitdir = gitdir.replace(
+                            /^([A-Za-z]):/,
+                            (_, drive) => `/mnt/${drive.toLowerCase()}`
+                        );
                         gitdir = gitdir.replace(/\\/g, '/');
                     }
                     // Verify the gitdir exists
@@ -106,7 +109,10 @@ export function getGitHooksPath() {
         let gitCommonDir = execSync('git rev-parse --git-common-dir', { encoding: 'utf8' }).trim();
         // Handle Windows paths when running under WSL (e.g. C:\... -> /mnt/c/...)
         if (/^[A-Za-z]:/.test(gitCommonDir)) {
-            gitCommonDir = gitCommonDir.replace(/^([A-Za-z]):/, (_, drive) => `/mnt/${drive.toLowerCase()}`);
+            gitCommonDir = gitCommonDir.replace(
+                /^([A-Za-z]):/,
+                (_, drive) => `/mnt/${drive.toLowerCase()}`
+            );
             gitCommonDir = gitCommonDir.replace(/\\/g, '/');
         }
         return path.join(gitCommonDir, 'hooks').replace(/\\/g, '/');
@@ -257,29 +263,33 @@ export async function updateConfig(propertyPath, value, options = {}) {
 export function getLatestVersion(packageName) {
     return new Promise((resolve, reject) => {
         // Use the main NPM API, not /latest
-        https.get(`https://registry.npmjs.org/${packageName}`, (res) => {
-            let data = '';
-            res.on('data', chunk => data += chunk);
-            res.on('end', () => {
-                try {
-                    const json = JSON.parse(data);
-                    // Get the version from the 'latest' tag
-                    if (json['dist-tags'] && json['dist-tags'].latest) {
-                        resolve(json['dist-tags'].latest);
-                    } else {
-                        reject(new Error('Could not get the version'));
-                    }
-                } catch (e) {
-                    // If it fails, try with npm view
+        https
+            .get(`https://registry.npmjs.org/${packageName}`, (res) => {
+                let data = '';
+                res.on('data', (chunk) => (data += chunk));
+                res.on('end', () => {
                     try {
-                        const version = execSync(`npm view ${packageName} version`, { encoding: 'utf8' }).trim();
-                        resolve(version);
-                    } catch (npmError) {
-                        reject(e);
+                        const json = JSON.parse(data);
+                        // Get the version from the 'latest' tag
+                        if (json['dist-tags'] && json['dist-tags'].latest) {
+                            resolve(json['dist-tags'].latest);
+                        } else {
+                            reject(new Error('Could not get the version'));
+                        }
+                    } catch (e) {
+                        // If it fails, try with npm view
+                        try {
+                            const version = execSync(`npm view ${packageName} version`, {
+                                encoding: 'utf8'
+                            }).trim();
+                            resolve(version);
+                        } catch (npmError) {
+                            reject(e);
+                        }
                     }
-                }
-            });
-        }).on('error', reject);
+                });
+            })
+            .on('error', reject);
     });
 }
 
@@ -301,14 +311,18 @@ export class Entertainment {
     static async getJoke() {
         return new Promise((resolve) => {
             // Try to get joke from API
-            const req = https.get('https://icanhazdadjoke.com/', {
-                headers: { 'Accept': 'text/plain' },
-                timeout: 3000
-            }, (res) => {
-                let data = '';
-                res.on('data', chunk => data += chunk);
-                res.on('end', () => resolve(data.trim()));
-            });
+            const req = https.get(
+                'https://icanhazdadjoke.com/',
+                {
+                    headers: { Accept: 'text/plain' },
+                    timeout: 3000
+                },
+                (res) => {
+                    let data = '';
+                    res.on('data', (chunk) => (data += chunk));
+                    res.on('end', () => resolve(data.trim()));
+                }
+            );
 
             req.on('error', () => {
                 // If it fails, use local joke
@@ -332,9 +346,11 @@ export class Entertainment {
         let isFinished = false;
 
         // Get first joke from API without blocking
-        this.getJoke().then(joke => {
-            if (!isFinished) currentJoke = joke;
-        }).catch(() => { }); // If it fails, keep the local one
+        this.getJoke()
+            .then((joke) => {
+                if (!isFinished) currentJoke = joke;
+            })
+            .catch(() => {}); // If it fails, keep the local one
 
         // Hide cursor
         process.stdout.write('\x1B[?25l');
@@ -356,13 +372,16 @@ export class Entertainment {
 
                 // Refresh joke every 10 seconds
                 if (jokeCountdown <= 0) {
-                    this.getJoke().then(joke => {
-                        if (!isFinished) currentJoke = joke;
-                    }).catch(() => {
-                        if (!isFinished) {
-                            currentJoke = this.jokes[Math.floor(Math.random() * this.jokes.length)];
-                        }
-                    });
+                    this.getJoke()
+                        .then((joke) => {
+                            if (!isFinished) currentJoke = joke;
+                        })
+                        .catch(() => {
+                            if (!isFinished) {
+                                currentJoke =
+                                    this.jokes[Math.floor(Math.random() * this.jokes.length)];
+                            }
+                        });
                     jokeCountdown = 10;
                 }
             }
@@ -374,13 +393,17 @@ export class Entertainment {
             const spinner = spinners[spinnerIndex % spinners.length];
 
             // Line 1: Spinner
-            process.stdout.write('\r\x1B[2K' + `${colors.yellow}${spinner} ${message}${colors.reset}\n`);
+            process.stdout.write(
+                '\r\x1B[2K' + `${colors.yellow}${spinner} ${message}${colors.reset}\n`
+            );
 
             // Line 2: Joke
             process.stdout.write('\r\x1B[2K' + `${colors.green}🎭 ${currentJoke}${colors.reset}\n`);
 
             // Line 3: Countdown
-            process.stdout.write('\r\x1B[2K' + `${colors.yellow}⏱️  Next joke in: ${jokeCountdown}s${colors.reset}\n`);
+            process.stdout.write(
+                '\r\x1B[2K' + `${colors.yellow}⏱️  Next joke in: ${jokeCountdown}s${colors.reset}\n`
+            );
         }, 100);
 
         try {

package/lib/commands/hooks.js

@@ -5,14 +5,8 @@
 
 import fs from 'fs';
 import path from 'path';
-import {
-    error,
-    success,
-    info,
-    warning,
-    checkGitRepo,
-    getGitHooksPath
-} from './helpers.js';
+import { error, success, info, warning, checkGitRepo, getGitHooksPath } from './helpers.js';
+import { removeCompletions } from './install.js';
 
 /**
  * Enable command
@@ -26,7 +20,7 @@ export function runEnable(hookName) {
     const hooksDir = getGitHooksPath();
     const hooks = hookName ? [hookName] : ['pre-commit', 'prepare-commit-msg'];
 
-    hooks.forEach(hook => {
+    hooks.forEach((hook) => {
         const disabledPath = `${hooksDir}/${hook}.disabled`;
         const enabledPath = `${hooksDir}/${hook}`;
 
@@ -53,7 +47,7 @@ export function runDisable(hookName) {
     const hooksDir = getGitHooksPath();
     const hooks = hookName ? [hookName] : ['pre-commit', 'prepare-commit-msg'];
 
-    hooks.forEach(hook => {
+    hooks.forEach((hook) => {
         const enabledPath = `${hooksDir}/${hook}`;
         const disabledPath = `${hooksDir}/${hook}.disabled`;
 
@@ -80,7 +74,7 @@ export function runStatus() {
 
     const hooksDir = getGitHooksPath();
     const hooks = ['pre-commit', 'prepare-commit-msg'];
-    hooks.forEach(hook => {
+    hooks.forEach((hook) => {
         const enabledPath = `${hooksDir}/${hook}`;
         const disabledPath = `${hooksDir}/${hook}.disabled`;
 
@@ -96,7 +90,7 @@ export function runStatus() {
     // Check guidelines files
     console.log('\nGuidelines files:');
     const guidelines = ['CLAUDE_PRE_COMMIT.md'];
-    guidelines.forEach(guideline => {
+    guidelines.forEach((guideline) => {
         const promptsPath = path.join('.claude', 'prompts', guideline);
         const legacyPath = path.join('.claude', guideline);
         if (fs.existsSync(promptsPath)) {
@@ -142,7 +136,7 @@ export function runUninstall() {
     const hooksPath = getGitHooksPath();
     const hooks = ['pre-commit', 'prepare-commit-msg'];
 
-    hooks.forEach(hook => {
+    hooks.forEach((hook) => {
         const hookPath = `${hooksPath}/${hook}`;
         if (fs.existsSync(hookPath)) {
             fs.unlinkSync(hookPath);
@@ -150,5 +144,13 @@ export function runUninstall() {
         }
     });
 
+    // Remove shell completion scripts and rc modifications
+    try {
+        removeCompletions();
+        success('Shell completions removed');
+    } catch (e) {
+        warning(`Could not remove shell completions: ${e.message}`);
+    }
+
     success('Claude Git Hooks uninstalled');
 }