claude-git-hooks 2.14.1 → 2.15.5

package/lib/commands/create-pr.js

@@ -43,7 +43,7 @@ export async function runCreatePr(args) {
 
         // Import GitHub API module
         logger.debug('create-pr', 'Importing GitHub API modules');
-        const { createPullRequest, GitHubAPIError, validateToken, findExistingPR } = await import('../utils/github-api.js');
+        const { createPullRequest, validateToken, findExistingPR } = await import('../utils/github-api.js');
         const { parseGitHubRepo } = await import('../utils/github-client.js');
 
         showInfo('🚀 Creating Pull Request...');
@@ -300,39 +300,181 @@ export async function runCreatePr(args) {
             }
         }
 
-        // Step 5.7: Check for unpushed tags (Issue #44)
-        logger.debug('create-pr', 'Step 5.7: Checking for unpushed tags');
-        const { compareLocalAndRemoteTags, pushTags: pushTagsUtil } = await import('../utils/git-tag-manager.js');
+        // Step 5.7: Smart tag pushing (Issue #44)
+        logger.debug('create-pr', 'Step 5.7: Checking and pushing unpushed tags');
+        const {
+            compareLocalAndRemoteTags,
+            pushTags: pushTagsUtil,
+            getLatestLocalTag,
+            getLatestRemoteTag,
+            parseTagVersion
+        } = await import('../utils/git-tag-manager.js');
+        const { compareVersions } = await import('../utils/version-manager.js');
+
         const tagComparison = await compareLocalAndRemoteTags();
 
         if (tagComparison.localNewer.length > 0) {
-            showWarning(`Local tags not pushed: ${tagComparison.localNewer.join(', ')}`);
-            console.log('');
+            // Get latest local and remote tags for comparison
+            const latestLocalTag = getLatestLocalTag();
+            const latestRemoteTag = await getLatestRemoteTag();
+
+            const localVersion = latestLocalTag ? parseTagVersion(latestLocalTag) : null;
+            const remoteVersion = latestRemoteTag ? parseTagVersion(latestRemoteTag) : null;
+
+            logger.debug('create-pr', 'Tag comparison details', {
+                localTag: latestLocalTag,
+                remoteTag: latestRemoteTag,
+                localVersion,
+                remoteVersion,
+                unpushedTags: tagComparison.localNewer
+            });
 
-            const shouldPushTags = await promptConfirmation(
-                'Push tags to remote?',
-                true  // default yes
-            );
+            let shouldPush = false;
+            let userChoice = null;
+
+            // Case 1: Local tag > Remote tag → Auto-push
+            if (localVersion && remoteVersion && compareVersions(localVersion, remoteVersion) > 0) {
+                logger.debug('create-pr', 'Local version > remote version, auto-pushing', {
+                    localVersion,
+                    remoteVersion
+                });
+
+                showInfo(`Local tag ${latestLocalTag} is newer than remote ${latestRemoteTag}`);
+                showInfo('Auto-pushing tag to remote...');
+                shouldPush = true;
+
+            // Case 2: Local tag = Remote tag → Prompt with warning
+            } else if (localVersion && remoteVersion && compareVersions(localVersion, remoteVersion) === 0) {
+                logger.debug('create-pr', 'Local version = remote version, prompting user', {
+                    localVersion,
+                    remoteVersion
+                });
+
+                showWarning('Local and remote tags have the same version:');
+                console.log(`  Local:  ${latestLocalTag} (${localVersion})`);
+                console.log(`  Remote: ${latestRemoteTag} (${remoteVersion})`);
+                console.log('');
+                console.log('This might indicate the tag was already pushed.');
+                console.log('');
+
+                userChoice = await promptMenu(
+                    'What would you like to do?',
+                    [
+                        { key: 'c', label: 'Continue without pushing' },
+                        { key: 'p', label: 'Force push tag' },
+                        { key: 'a', label: 'Abort PR creation' }
+                    ],
+                    'c'
+                );
+
+                if (userChoice === 'a') {
+                    showInfo('PR creation cancelled');
+                    process.exit(0);
+                } else if (userChoice === 'p') {
+                    shouldPush = true;
+                }
+
+            // Case 3: Local tag < Remote tag → Prompt with error
+            } else if (localVersion && remoteVersion && compareVersions(localVersion, remoteVersion) < 0) {
+                logger.debug('create-pr', 'Local version < remote version, prompting user', {
+                    localVersion,
+                    remoteVersion
+                });
+
+                showError('Local tag is older than remote tag:');
+                console.log(`  Local:  ${latestLocalTag} (${localVersion})`);
+                console.log(`  Remote: ${latestRemoteTag} (${remoteVersion})`);
+                console.log('');
+                console.log('This usually means you need to pull latest tags:');
+                console.log('  git fetch --tags');
+                console.log('');
+
+                userChoice = await promptMenu(
+                    'What would you like to do?',
+                    [
+                        { key: 'a', label: 'Abort PR creation' },
+                        { key: 'c', label: 'Continue anyway (not recommended)' }
+                    ],
+                    'a'
+                );
+
+                if (userChoice === 'a') {
+                    showInfo('PR creation cancelled');
+                    process.exit(0);
+                }
+
+            // Case 4: No version comparison possible → Prompt with info
+            } else {
+                logger.debug('create-pr', 'Cannot compare versions, prompting user', {
+                    localTag: latestLocalTag,
+                    remoteTag: latestRemoteTag,
+                    hasLocalVersion: !!localVersion,
+                    hasRemoteVersion: !!remoteVersion
+                });
+
+                showWarning('Unable to compare tag versions:');
+                console.log(`  Local tag:  ${latestLocalTag || 'none'}`);
+                console.log(`  Remote tag: ${latestRemoteTag || 'none'}`);
+                console.log('');
+                console.log('Unpushed tags:', tagComparison.localNewer.join(', '));
+                console.log('');
+
+                userChoice = await promptMenu(
+                    'What would you like to do?',
+                    [
+                        { key: 'p', label: 'Push tags to remote' },
+                        { key: 'c', label: 'Continue without pushing' },
+                        { key: 'a', label: 'Abort PR creation' }
+                    ],
+                    'p'
+                );
+
+                if (userChoice === 'a') {
+                    showInfo('PR creation cancelled');
+                    process.exit(0);
+                } else if (userChoice === 'p') {
+                    shouldPush = true;
+                }
+            }
+
+            // Execute push if decided
+            if (shouldPush) {
+                console.log('');
+                showInfo('Pushing tags to remote...');
 
-            if (shouldPushTags) {
-                showInfo('Pushing tags...');
                 const pushResult = pushTagsUtil(null, tagComparison.localNewer);
 
                 if (pushResult.success) {
-                    showSuccess('Tags pushed successfully');
+                    showSuccess('✓ Tags pushed successfully');
+                    logger.debug('create-pr', 'Tags pushed', {
+                        pushed: tagComparison.localNewer
+                    });
                 } else {
                     showError(`Failed to push some tags: ${pushResult.error}`);
                     if (pushResult.failed.length > 0) {
                         console.log('Failed tags:');
                         pushResult.failed.forEach(f => console.log(`  - ${f.tag}: ${f.error}`));
                     }
+
+                    const shouldContinue = await promptConfirmation(
+                        'Continue creating PR despite push failure?',
+                        false
+                    );
+
+                    if (!shouldContinue) {
+                        showInfo('PR creation cancelled');
+                        process.exit(0);
+                    }
                 }
 
                 console.log('');
-            } else {
-                showInfo('Tag push skipped');
+            } else if (userChoice !== 'c') {
+                // Auto-push failed or user chose to continue
+                logger.debug('create-pr', 'Tag push skipped', { reason: 'user choice or condition' });
                 console.log('');
             }
+        } else {
+            logger.debug('create-pr', 'No unpushed tags found, continuing');
         }
 
         // Step 6: Generate PR metadata using engine

package/lib/commands/debug.js

@@ -30,7 +30,7 @@ export async function runSetDebug(value) {
             const config = await getConfig();
             const isEnabled = config.system.debug || false;
             console.log('');
-            info(`Debug mode: ${isEnabled ? colors.green + 'enabled' + colors.reset : colors.red + 'disabled' + colors.reset}`);
+            info(`Debug mode: ${isEnabled ? `${colors.green  }enabled${  colors.reset}` : `${colors.red  }disabled${  colors.reset}`}`);
             console.log('');
         } catch (err) {
             error(`Failed to check debug status: ${err.message}`);

package/lib/commands/helpers.js

@@ -193,10 +193,15 @@ export async function updateConfig(propertyPath, value, options = {}) {
         }
 
         // Set value at propertyPath (support dot notation like 'system.debug')
-        // For v2.8.0 format, write inside 'overrides' so loadUserConfig picks it up
+        // For v2.8.0 format, write inside 'overrides' EXCEPT for top-level fields
         const pathParts = propertyPath.split('.');
         let current = config;
-        if (config.version === '2.8.0') {
+
+        // Top-level fields that should NOT go into overrides
+        const topLevelFields = ['preset', 'version'];
+        const isTopLevelField = pathParts.length === 1 && topLevelFields.includes(pathParts[0]);
+
+        if (config.version === '2.8.0' && !isTopLevelField) {
             if (!config.overrides || typeof config.overrides !== 'object') {
                 config.overrides = {};
             }
@@ -263,12 +268,12 @@ export function getLatestVersion(packageName) {
  */
 export class Entertainment {
     static jokes = [
-        "Why do programmers prefer dark mode? Because light attracts bugs!",
-        "A QA engineer walks into a bar. Orders 1 beer. Orders 0 beers. Orders -1 beers.",
+        'Why do programmers prefer dark mode? Because light attracts bugs!',
+        'A QA engineer walks into a bar. Orders 1 beer. Orders 0 beers. Orders -1 beers.',
         "What's a pirate's favorite programming language? R!",
         "There are 10 types of people: those who understand binary and those who don't.",
-        "Why do programmers confuse Halloween with Christmas? Because Oct 31 = Dec 25",
-        "What does one bit say to another? See you on the bus!",
+        'Why do programmers confuse Halloween with Christmas? Because Oct 31 = Dec 25',
+        'What does one bit say to another? See you on the bus!',
         "Why don't Java and C++ get along? Because they have different views on pointers.",
         "My code doesn't have bugs, just undocumented features."
     ];

package/lib/commands/install.js

@@ -63,7 +63,7 @@ async function checkVersionAndPromptUpdate() {
                         success('Update completed. Please run your command again.');
                         process.exit(0); // Exit so user restarts the process
                     } catch (e) {
-                        error('Error updating: ' + e.message);
+                        error(`Error updating: ${  e.message}`);
                         resolve(false);
                     }
                 } else {
@@ -158,10 +158,9 @@ async function checkClaudeAuth() {
 
 /**
  * Check complete dependencies (like setup-wsl.sh)
- * @param {string|null} sudoPassword - Not used in v2.0.0+
  * @param {boolean} skipAuth - Skip Claude verification
  */
-async function checkAndInstallDependencies(sudoPassword = null, skipAuth = false) {
+async function checkAndInstallDependencies(skipAuth = false) {
     info('Checking system dependencies...');
 
     // Check Node.js
@@ -252,7 +251,7 @@ function updateGitignore() {
         }
 
         // Add the missing entries
-        gitignoreContent += missingEntries.join('\n') + '\n';
+        gitignoreContent += `${missingEntries.join('\n')  }\n`;
 
         // Write the updated file
         fs.writeFileSync(gitignorePath, gitignoreContent);
@@ -294,7 +293,7 @@ function configureGit() {
         }
 
     } catch (e) {
-        warning('Error configuring Git: ' + e.message);
+        warning(`Error configuring Git: ${  e.message}`);
     }
 }
 
@@ -399,7 +398,7 @@ export async function runInstall(args) {
 
     // v2.0.0+: No sudo needed (pure Node.js, no system packages required)
     // Check dependencies
-    await checkAndInstallDependencies(null, skipAuth);
+    await checkAndInstallDependencies(skipAuth);
 
     const templatesPath = getTemplatesPath();
     const hooksPath = '.git/hooks';
@@ -641,8 +640,8 @@ export async function runInstall(args) {
     const settingsLocalPath = path.join(claudeDir, 'settings.local.json');
     if (!fs.existsSync(settingsLocalPath)) {
         const settingsLocalContent = {
-            "_comment": "Local settings - DO NOT COMMIT. This file is gitignored.",
-            "githubToken": ""
+            '_comment': 'Local settings - DO NOT COMMIT. This file is gitignored.',
+            'githubToken': ''
         };
         fs.writeFileSync(settingsLocalPath, JSON.stringify(settingsLocalContent, null, 2));
         info('settings.local.json created (add your GitHub token here)');

package/lib/commands/migrate-config.js

@@ -36,7 +36,29 @@ export async function runMigrateConfig() {
 
         // Check if already in v2.8.0 format
         if (rawConfig.version === '2.8.0') {
-            success('✅ Config is already in v2.8.0 format.');
+            // Bug fix: Check if preset was incorrectly placed in overrides
+            let fixed = false;
+            if (rawConfig.overrides?.preset) {
+                info('🔧 Fixing incorrectly placed preset field...');
+
+                // If top-level preset doesn't exist or differs, use the one from overrides
+                if (!rawConfig.preset || rawConfig.preset !== rawConfig.overrides.preset) {
+                    info(`  Moving preset '${rawConfig.overrides.preset}' from overrides to top level`);
+                    rawConfig.preset = rawConfig.overrides.preset;
+                }
+
+                // Remove from overrides
+                delete rawConfig.overrides.preset;
+
+                // Write back
+                fs.writeFileSync(configPath, JSON.stringify(rawConfig, null, 4));
+                fixed = true;
+                success('✅ Config fixed: preset moved to correct location.');
+            }
+
+            if (!fixed) {
+                success('✅ Config is already in v2.8.0 format.');
+            }
             return;
         }
 
@@ -87,7 +109,7 @@ export async function runMigrateConfig() {
             info('📖 See .claude/config.advanced.example.json for documentation');
         }
 
-        console.log(`\n✨ New config:`);
+        console.log('\n✨ New config:');
         console.log(JSON.stringify(newConfig, null, 2));
         console.log(`\n💾 Old config backed up to: ${backupPath}`);
         console.log('\n💡 Many parameters are now hardcoded with sensible defaults');

package/lib/commands/setup-github.js

@@ -71,7 +71,7 @@ export async function runSetupGitHub() {
     // Save token
     const saveResult = saveGitHubToken(trimmedToken);
     if (!saveResult.success) {
-        showError('Failed to save token: ' + saveResult.error);
+        showError(`Failed to save token: ${  saveResult.error}`);
         return;
     }
     showSuccess(`Token saved to ${saveResult.path}`);

package/lib/commands/update.js

@@ -61,7 +61,7 @@ export async function runUpdate() {
             success('Update completed');
             await runInstall(['--force']);
         } catch (updateError) {
-            error('Error updating: ' + updateError.message);
+            error(`Error updating: ${  updateError.message}`);
         }
     }
 }

package/lib/config.js

@@ -26,12 +26,8 @@
 
 import fs from 'fs';
 import path from 'path';
-import { fileURLToPath } from 'url';
 import logger from './utils/logger.js';
 
-const __filename = fileURLToPath(import.meta.url);
-const __dirname = path.dirname(__filename);
-
 /**
  * Hardcoded defaults (v3.0.0)
  * These are NOT user-configurable - sensible defaults that work for everyone

package/lib/hooks/prepare-commit-msg.js

@@ -184,7 +184,7 @@ const main = async () => {
         const taskId = await getOrPromptTaskId({
             prompt: false,  // Don't prompt - just detect from branch
             required: false,
-            config: config  // Pass config for custom pattern
+            config  // Pass config for custom pattern
         });
 
         // Note: getOrPromptTaskId() already logs the task ID if found
@@ -302,7 +302,7 @@ const main = async () => {
         }
 
         // Write to commit message file
-        await fs.writeFile(commitMsgFile, message + '\n', 'utf8');
+        await fs.writeFile(commitMsgFile, `${message  }\n`, 'utf8');
 
         const duration = ((Date.now() - startTime) / 1000).toFixed(2);
         console.error(`⏱️  Message generation completed in ${duration}s`);

package/lib/utils/changelog-generator.js

@@ -175,7 +175,7 @@ export function formatChangelogSection(version, date, categories, isUnreleased =
     });
 
     const header = isUnreleased
-        ? `## [Unreleased]\n`
+        ? '## [Unreleased]\n'
         : `## [${version}] - ${date}\n`;
 
     const sections = [];
@@ -200,7 +200,7 @@ export function formatChangelogSection(version, date, categories, isUnreleased =
         }
     }
 
-    const content = header + '\n' + sections.join('');
+    const content = `${header  }\n${  sections.join('')}`;
 
     logger.debug('changelog-generator - formatChangelogSection', 'Section formatted');
 
@@ -215,7 +215,6 @@ export function formatChangelogSection(version, date, categories, isUnreleased =
  * @param {string} options.version - Version string
  * @param {boolean} options.isReleaseVersion - True if final version (no suffix)
  * @param {string} options.baseBranch - Base branch for diff (default: 'main')
- * @param {Object} options.config - Configuration object
  * @returns {Promise<Object>} Generated entry: { content, categories }
  */
 export async function generateChangelogEntry(options) {
@@ -223,7 +222,6 @@ export async function generateChangelogEntry(options) {
         version,
         isReleaseVersion = false,
         baseBranch = 'main',
-        config = {}
     } = options;
 
     logger.debug('changelog-generator - generateChangelogEntry', 'Generating CHANGELOG entry', {
@@ -270,7 +268,7 @@ export async function generateChangelogEntry(options) {
 
         // Truncate diff if too large
         const truncatedDiff = fullDiff.length > 30000
-            ? fullDiff.substring(0, 30000) + '\n... (truncated)'
+            ? `${fullDiff.substring(0, 30000)  }\n... (truncated)`
             : fullDiff;
 
         // Load prompt template
@@ -359,14 +357,14 @@ y este proyecto adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.0.h
             if (headerEnd !== -1) {
                 const before = existingContent.substring(0, match.index + headerEnd);
                 const after = existingContent.substring(match.index + headerEnd);
-                updatedContent = before + newContent + '\n' + after;
+                updatedContent = `${before + newContent  }\n${  after}`;
             } else {
                 // No existing versions, append after header
-                updatedContent = existingContent + '\n' + newContent;
+                updatedContent = `${existingContent  }\n${  newContent}`;
             }
         } else {
             // No header found, prepend content
-            updatedContent = newContent + '\n' + existingContent;
+            updatedContent = `${newContent  }\n${  existingContent}`;
         }
 
         fs.writeFileSync(changelogPath, updatedContent, 'utf8');

package/lib/utils/claude-client.js

@@ -246,6 +246,7 @@ const executeClaude = (prompt, { timeout = 120000, allowedTools = [] } = {}) =>
 
     // Handle process completion
     claude.on('close', (code) => {
+        clearTimeout(timeoutId);
         const elapsedTime = Date.now() - startTime;
 
         // Check for "Execution error" even when exit code is 0
@@ -342,6 +343,7 @@ const executeClaude = (prompt, { timeout = 120000, allowedTools = [] } = {}) =>
 
     // Handle errors
     claude.on('error', (error) => {
+        clearTimeout(timeoutId);
         const elapsedTime = Date.now() - startTime;
 
         logger.error(
@@ -390,9 +392,6 @@ const executeClaude = (prompt, { timeout = 120000, allowedTools = [] } = {}) =>
         }));
     }, timeout);
 
-    // Clear timeout if process completes
-    claude.on('close', () => clearTimeout(timeoutId));
-
     // Write prompt to stdin
     // Why: Claude CLI reads prompt from stdin, not command arguments
 
@@ -400,6 +399,7 @@ const executeClaude = (prompt, { timeout = 120000, allowedTools = [] } = {}) =>
     // Why: write() failures can emit 'error' events asynchronously
     // Common in parallel execution with large prompts
     claude.stdin.on('error', (error) => {
+        clearTimeout(timeoutId);
         logger.error(
             'claude-client - executeClaude',
             'stdin stream error (process may have terminated early)',
@@ -426,6 +426,7 @@ const executeClaude = (prompt, { timeout = 120000, allowedTools = [] } = {}) =>
         claude.stdin.write(prompt);
         claude.stdin.end();
     } catch (error) {
+        clearTimeout(timeoutId);
         logger.error(
             'claude-client - executeClaude',
             'Failed to write prompt to Claude CLI stdin (synchronous error)',
@@ -536,7 +537,7 @@ const executeClaudeInteractive = (prompt, { timeout = 300000 } = {}) => new Prom
  * @returns {Object} Parsed JSON object
  * @throws {ClaudeClientError} If no valid JSON found
  */
-const extractJSON = (response, telemetryContext = {}) => {
+const extractJSON = (response) => {
     logger.debug(
         'claude-client - extractJSON',
         'Extracting JSON from response',
@@ -614,7 +615,7 @@ const extractJSON = (response, telemetryContext = {}) => {
     // Telemetry is now recorded by withRetry wrapper
     throw new ClaudeClientError('No valid JSON found in Claude response', {
         context: {
-            response: response,
+            response,
             errorInfo: {
                 type: 'JSON_PARSE_ERROR'
             }
@@ -797,7 +798,7 @@ const executeClaudeWithRetry = async (prompt, options = {}) => {
         () => executeClaude(prompt, executeOptions),
         {
             operationName: 'executeClaude',
-            telemetryContext: telemetryContext
+            telemetryContext
         }
     );
 };

package/lib/utils/claude-diagnostics.js

@@ -150,27 +150,26 @@ const formatTimingInfo = (errorInfo) => {
  * @returns {string} Formatted error message
  */
 export const formatClaudeError = (errorInfo) => {
-    const lines = [];
 
     switch (errorInfo.type) {
-        case ClaudeErrorType.EXECUTION_ERROR:
-            return formatExecutionError(errorInfo);
+    case ClaudeErrorType.EXECUTION_ERROR:
+        return formatExecutionError(errorInfo);
 
-        case ClaudeErrorType.RATE_LIMIT:
-            return formatRateLimitError(errorInfo);
+    case ClaudeErrorType.RATE_LIMIT:
+        return formatRateLimitError(errorInfo);
 
-        case ClaudeErrorType.AUTH_FAILED:
-            return formatAuthError(errorInfo);
+    case ClaudeErrorType.AUTH_FAILED:
+        return formatAuthError(errorInfo);
 
-        case ClaudeErrorType.NETWORK:
-            return formatNetworkError(errorInfo);
+    case ClaudeErrorType.NETWORK:
+        return formatNetworkError(errorInfo);
 
-        case ClaudeErrorType.INVALID_RESPONSE:
-            return formatInvalidResponseError(errorInfo);
+    case ClaudeErrorType.INVALID_RESPONSE:
+        return formatInvalidResponseError(errorInfo);
 
-        case ClaudeErrorType.GENERIC:
-        default:
-            return formatGenericError(errorInfo);
+    case ClaudeErrorType.GENERIC:
+    default:
+        return formatGenericError(errorInfo);
     }
 };
 
@@ -178,7 +177,6 @@ export const formatClaudeError = (errorInfo) => {
  * Format execution error
  */
 const formatExecutionError = (errorInfo) => {
-    const { exitCode, stdout } = errorInfo;
     const lines = [];
 
     lines.push('❌ Claude CLI execution error');
@@ -243,7 +241,6 @@ const formatRateLimitError = (errorInfo) => {
  * Format authentication error
  */
 const formatAuthError = (errorInfo) => {
-    const { exitCode } = errorInfo;
     const lines = [];
 
     lines.push('❌ Claude CLI authentication failed');
@@ -266,7 +263,6 @@ const formatAuthError = (errorInfo) => {
  * Format network error
  */
 const formatNetworkError = (errorInfo) => {
-    const { exitCode } = errorInfo;
     const lines = [];
 
     lines.push('❌ Network error connecting to Claude API');
@@ -290,7 +286,6 @@ const formatNetworkError = (errorInfo) => {
  * Format invalid response error
  */
 const formatInvalidResponseError = (errorInfo) => {
-    const { exitCode } = errorInfo;
     const lines = [];
 
     lines.push('❌ Claude returned invalid response');
@@ -350,9 +345,7 @@ const formatGenericError = (errorInfo) => {
  * @param {Object} errorInfo - Output from detectClaudeError()
  * @returns {boolean} True if error might resolve with retry
  */
-export const isRecoverableError = (errorInfo) => {
-    return errorInfo.type === ClaudeErrorType.EXECUTION_ERROR ||
+export const isRecoverableError = (errorInfo) => errorInfo.type === ClaudeErrorType.EXECUTION_ERROR ||
            errorInfo.type === ClaudeErrorType.RATE_LIMIT ||
            errorInfo.type === ClaudeErrorType.NETWORK ||
            errorInfo.type === ClaudeErrorType.TIMEOUT;
-};

package/lib/utils/file-operations.js

@@ -174,7 +174,7 @@ const filterFiles = async (files, { maxSize = 100000, extensions = [] } = {}) =>
             maxSize,
             extensions,
             'process.cwd()': process.cwd(),
-            files: files
+            files
         }
     );
 

package/lib/utils/file-utils.js

@@ -4,7 +4,6 @@
  */
 
 import fs from 'fs/promises';
-import fsSync from 'fs';
 import path from 'path';
 import { getRepoRoot } from './git-operations.js';
 import logger from './logger.js';

package/lib/utils/git-operations.js

@@ -374,7 +374,6 @@ const getStagedStats = () => {
 
     try {
         const shortstat = execGitCommand('diff --cached --shortstat');
-        const numstat = execGitCommand('diff --cached --numstat');
         const nameStatus = execGitCommand('diff --cached --name-status');
 
         // Parse shortstat: "X files changed, Y insertions(+), Z deletions(-)"

package/lib/utils/github-api.js

@@ -322,7 +322,7 @@ export const createPullRequest = async ({
     logger.debug('github-api - createPullRequest', 'Starting PR creation', {
         owner,
         repo,
-        titlePreview: title.substring(0, 50) + '...',
+        titlePreview: `${title.substring(0, 50)  }...`,
         titleLength: title.length,
         bodyLength: body.length,
         head,
@@ -424,8 +424,8 @@ export const createPullRequest = async ({
             title: pr.title,
             head: pr.head.ref,
             base: pr.base.ref,
-            labels: labels,
-            reviewers: reviewers
+            labels,
+            reviewers
         };
 
         logger.debug('github-api - createPullRequest', 'PR creation completed', result);