claude-git-hooks 2.18.0 → 2.19.0

package/lib/utils/version-manager.js

@@ -14,6 +14,7 @@ import fs from 'fs';
 import path from 'path';
 import { getRepoRoot } from './git-operations.js';
 import logger from './logger.js';
+import { walkDirectoryTree } from './file-utils.js';
 
 /**
  * Cache for discovery result
@@ -44,16 +45,21 @@ function extractPomProjectVersion(content) {
         // Section-level elements that mark the end of the project coordinates region.
         // Any <version> tag inside these sections belongs to dependencies, plugins, etc.
         const sectionTags = [
-            'properties', 'dependencies', 'dependencyManagement',
-            'build', 'profiles', 'modules', 'reporting',
-            'distributionManagement', 'repositories', 'pluginRepositories',
+            'properties',
+            'dependencies',
+            'dependencyManagement',
+            'build',
+            'profiles',
+            'modules',
+            'reporting',
+            'distributionManagement',
+            'repositories',
+            'pluginRepositories',
             '\\/project'
         ].join('|');
 
         // Extract the "header" region: between </parent> and the first section element
-        const headerRegex = new RegExp(
-            `<\\/parent>([\\s\\S]*?)(?=<(?:${sectionTags})\\b)`
-        );
+        const headerRegex = new RegExp(`<\\/parent>([\\s\\S]*?)(?=<(?:${sectionTags})\\b)`);
         const headerMatch = content.match(headerRegex);
 
         if (headerMatch) {
@@ -98,10 +104,14 @@ const VERSION_FILE_TYPES = {
                 const packageData = JSON.parse(content);
                 return packageData.version || null;
             } catch (error) {
-                logger.debug('version-manager - VERSION_FILE_TYPES', 'Failed to read package.json', {
-                    filePath,
-                    error: error.message
-                });
+                logger.debug(
+                    'version-manager - VERSION_FILE_TYPES',
+                    'Failed to read package.json',
+                    {
+                        filePath,
+                        error: error.message
+                    }
+                );
                 return null;
             }
         },
@@ -136,7 +146,8 @@ const VERSION_FILE_TYPES = {
             let newContent;
             if (inherited) {
                 // Version is inherited from parent — update <parent><version> tag
-                const parentRegex = /(<parent>[\s\S]*?<version>)[^<]+(<\/version>[\s\S]*?<\/parent>)/;
+                const parentRegex =
+                    /(<parent>[\s\S]*?<version>)[^<]+(<\/version>[\s\S]*?<\/parent>)/;
                 newContent = content.replace(parentRegex, `$1${newVersion}$2`);
             } else {
                 const hasParent = /<parent>[\s\S]*?<\/parent>/.test(content);
@@ -144,7 +155,8 @@ const VERSION_FILE_TYPES = {
                     const afterParentRegex = /(<\/parent>[\s\S]*?<version>)[^<]+(<\/version>)/;
                     newContent = content.replace(afterParentRegex, `$1${newVersion}$2`);
                 } else {
-                    const projectVersionRegex = /(<project[^>]*>[\s\S]*?<version>)[^<]+(<\/version>)/;
+                    const projectVersionRegex =
+                        /(<project[^>]*>[\s\S]*?<version>)[^<]+(<\/version>)/;
                     newContent = content.replace(projectVersionRegex, `$1${newVersion}$2`);
                 }
             }
@@ -200,11 +212,7 @@ const VERSION_FILE_TYPES = {
  * @returns {Object} DiscoveryResult: { files, resolvedVersion, mismatch, types }
  */
 export function discoverVersionFiles(options = {}) {
-    const {
-        maxDepth = 3,
-        fileTypes = Object.keys(VERSION_FILE_TYPES),
-        ignoreDirs = []
-    } = options;
+    const { maxDepth = 3, fileTypes = Object.keys(VERSION_FILE_TYPES), ignoreDirs = [] } = options;
 
     logger.debug('version-manager - discoverVersionFiles', 'Starting discovery', {
         maxDepth,
@@ -232,69 +240,38 @@ export function discoverVersionFiles(options = {}) {
     ];
     const ignoreSet = new Set([...defaultIgnore, ...ignoreDirs]);
 
-    /**
-     * Recursively walks directory tree
-     * @param {string} dir - Current directory path
-     * @param {number} depth - Current depth level
-     */
-    function walkDirectory(dir, depth) {
-        if (depth > maxDepth) {
-            return;
-        }
-
-        try {
-            const entries = fs.readdirSync(dir, { withFileTypes: true });
-
-            for (const entry of entries) {
-                // Skip hidden directories and ignored patterns
-                if (entry.name.startsWith('.') || ignoreSet.has(entry.name)) {
-                    continue;
-                }
-
-                const fullPath = path.join(dir, entry.name);
-
-                if (entry.isDirectory()) {
-                    // Recurse into subdirectory
-                    walkDirectory(fullPath, depth + 1);
-                } else if (entry.isFile()) {
-                    // Check if this file matches any registered type
-                    for (const fileType of fileTypes) {
-                        const registry = VERSION_FILE_TYPES[fileType];
-                        if (registry && entry.name === registry.filename) {
-                            // Read version from file
-                            const version = registry.readVersion(fullPath);
-
-                            // Create descriptor
-                            const descriptor = {
-                                path: fullPath,
-                                relativePath: path.relative(repoRoot, fullPath),
-                                type: fileType,
-                                projectLabel: registry.projectLabel,
-                                version,
-                                selected: true // Default: all files selected
-                            };
-
-                            discoveredFiles.push(descriptor);
-
-                            logger.debug('version-manager - discoverVersionFiles', 'Found version file', {
-                                relativePath: descriptor.relativePath,
-                                type: fileType,
-                                version
-                            });
-                        }
-                    }
+    walkDirectoryTree(repoRoot, {
+        maxDepth,
+        ignoreSet,
+        onFile: (entry, fullPath) => {
+            for (const fileType of fileTypes) {
+                const registry = VERSION_FILE_TYPES[fileType];
+                if (registry && entry.name === registry.filename) {
+                    const version = registry.readVersion(fullPath);
+                    const descriptor = {
+                        path: fullPath,
+                        relativePath: path.relative(repoRoot, fullPath),
+                        type: fileType,
+                        projectLabel: registry.projectLabel,
+                        version,
+                        selected: true
+                    };
+                    discoveredFiles.push(descriptor);
+                    logger.debug('version-manager - discoverVersionFiles', 'Found version file', {
+                        relativePath: descriptor.relativePath,
+                        type: fileType,
+                        version
+                    });
                 }
             }
-        } catch (error) {
+        },
+        onError: (dir, error) => {
             logger.debug('version-manager - discoverVersionFiles', 'Error reading directory', {
                 dir,
                 error: error.message
             });
         }
-    }
-
-    // Start walking from repo root at depth 0
-    walkDirectory(repoRoot, 0);
+    });
 
     // Sort results: root files first, then by depth, then alphabetically
     discoveredFiles.sort((a, b) => {
@@ -310,24 +287,22 @@ export function discoverVersionFiles(options = {}) {
 
     // Determine resolved version (prefer root-level file, then first found)
     let resolvedVersion = null;
-    const rootFile = discoveredFiles.find(f => !f.relativePath.includes(path.sep));
+    const rootFile = discoveredFiles.find((f) => !f.relativePath.includes(path.sep));
     if (rootFile && rootFile.version) {
         resolvedVersion = rootFile.version;
     } else if (discoveredFiles.length > 0) {
         // Use first file's version as fallback
-        const firstWithVersion = discoveredFiles.find(f => f.version !== null);
+        const firstWithVersion = discoveredFiles.find((f) => f.version !== null);
         resolvedVersion = firstWithVersion ? firstWithVersion.version : null;
     }
 
     // Check for version mismatch
-    const versions = discoveredFiles
-        .filter(f => f.version !== null)
-        .map(f => f.version);
+    const versions = discoveredFiles.filter((f) => f.version !== null).map((f) => f.version);
     const uniqueVersions = [...new Set(versions)];
     const mismatch = uniqueVersions.length > 1;
 
     // Collect unique types
-    const types = [...new Set(discoveredFiles.map(f => f.type))];
+    const types = [...new Set(discoveredFiles.map((f) => f.type))];
 
     const result = {
         files: discoveredFiles,
@@ -446,7 +421,6 @@ export function updateVersionFiles(files, newVersion) {
                 type: file.type,
                 version: targetVersion
             });
-
         } catch (error) {
             logger.error('version-manager - updateVersionFiles', 'Failed to update file', {
                 path: file.relativePath,
@@ -457,7 +431,7 @@ export function updateVersionFiles(files, newVersion) {
     }
 
     if (errors.length > 0) {
-        const errorMsg = errors.map(e => `${e.file}: ${e.error}`).join(', ');
+        const errorMsg = errors.map((e) => `${e.file}: ${e.error}`).join(', ');
         throw new Error(`Failed to update some files: ${errorMsg}`);
     }
 
@@ -508,7 +482,9 @@ export function modifySuffix(version, options = {}) {
  * @returns {string|null} Version string or null if not found
  */
 function readGradleVersion(filePath) {
-    logger.debug('version-manager - readGradleVersion', 'Reading version from build.gradle', { filePath });
+    logger.debug('version-manager - readGradleVersion', 'Reading version from build.gradle', {
+        filePath
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -523,15 +499,20 @@ function readGradleVersion(filePath) {
         const match = content.match(versionRegex);
 
         if (!match) {
-            logger.debug('version-manager - readGradleVersion', 'Version not found in build.gradle');
+            logger.debug(
+                'version-manager - readGradleVersion',
+                'Version not found in build.gradle'
+            );
             return null;
         }
 
         const version = match[1].trim();
-        logger.debug('version-manager - readGradleVersion', 'Version read', { version, path: filePath });
+        logger.debug('version-manager - readGradleVersion', 'Version read', {
+            version,
+            path: filePath
+        });
 
         return version;
-
     } catch (error) {
         logger.error('version-manager - readGradleVersion', 'Failed to read version', error);
         throw new Error(`Failed to read build.gradle: ${error.message}`);
@@ -546,7 +527,11 @@ function readGradleVersion(filePath) {
  * @returns {string|null} Version string or null if not found
  */
 function readGradleKtsVersion(filePath) {
-    logger.debug('version-manager - readGradleKtsVersion', 'Reading version from build.gradle.kts', { filePath });
+    logger.debug(
+        'version-manager - readGradleKtsVersion',
+        'Reading version from build.gradle.kts',
+        { filePath }
+    );
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -561,15 +546,20 @@ function readGradleKtsVersion(filePath) {
         const match = content.match(versionRegex);
 
         if (!match) {
-            logger.debug('version-manager - readGradleKtsVersion', 'Version not found in build.gradle.kts');
+            logger.debug(
+                'version-manager - readGradleKtsVersion',
+                'Version not found in build.gradle.kts'
+            );
             return null;
         }
 
         const version = match[1].trim();
-        logger.debug('version-manager - readGradleKtsVersion', 'Version read', { version, path: filePath });
+        logger.debug('version-manager - readGradleKtsVersion', 'Version read', {
+            version,
+            path: filePath
+        });
 
         return version;
-
     } catch (error) {
         logger.error('version-manager - readGradleKtsVersion', 'Failed to read version', error);
         throw new Error(`Failed to read build.gradle.kts: ${error.message}`);
@@ -584,7 +574,9 @@ function readGradleKtsVersion(filePath) {
  * @returns {string|null} Version string or null if not found
  */
 function readPyprojectVersion(filePath) {
-    logger.debug('version-manager - readPyprojectVersion', 'Reading version from pyproject.toml', { filePath });
+    logger.debug('version-manager - readPyprojectVersion', 'Reading version from pyproject.toml', {
+        filePath
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -605,15 +597,20 @@ function readPyprojectVersion(filePath) {
         }
 
         if (!match) {
-            logger.debug('version-manager - readPyprojectVersion', 'Version not found in pyproject.toml');
+            logger.debug(
+                'version-manager - readPyprojectVersion',
+                'Version not found in pyproject.toml'
+            );
             return null;
         }
 
         const version = match[1].trim();
-        logger.debug('version-manager - readPyprojectVersion', 'Version read', { version, path: filePath });
+        logger.debug('version-manager - readPyprojectVersion', 'Version read', {
+            version,
+            path: filePath
+        });
 
         return version;
-
     } catch (error) {
         logger.error('version-manager - readPyprojectVersion', 'Failed to read version', error);
         throw new Error(`Failed to read pyproject.toml: ${error.message}`);
@@ -628,7 +625,9 @@ function readPyprojectVersion(filePath) {
  * @returns {string|null} Version string or null if not found
  */
 function readCargoVersion(filePath) {
-    logger.debug('version-manager - readCargoVersion', 'Reading version from Cargo.toml', { filePath });
+    logger.debug('version-manager - readCargoVersion', 'Reading version from Cargo.toml', {
+        filePath
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -648,10 +647,12 @@ function readCargoVersion(filePath) {
         }
 
         const version = match[1].trim();
-        logger.debug('version-manager - readCargoVersion', 'Version read', { version, path: filePath });
+        logger.debug('version-manager - readCargoVersion', 'Version read', {
+            version,
+            path: filePath
+        });
 
         return version;
-
     } catch (error) {
         logger.error('version-manager - readCargoVersion', 'Failed to read version', error);
         throw new Error(`Failed to read Cargo.toml: ${error.message}`);
@@ -666,7 +667,9 @@ function readCargoVersion(filePath) {
  * @returns {string|null} Version string or null if not found
  */
 function readSbtVersion(filePath) {
-    logger.debug('version-manager - readSbtVersion', 'Reading version from version.sbt', { filePath });
+    logger.debug('version-manager - readSbtVersion', 'Reading version from version.sbt', {
+        filePath
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -686,10 +689,12 @@ function readSbtVersion(filePath) {
         }
 
         const version = match[1].trim();
-        logger.debug('version-manager - readSbtVersion', 'Version read', { version, path: filePath });
+        logger.debug('version-manager - readSbtVersion', 'Version read', {
+            version,
+            path: filePath
+        });
 
         return version;
-
     } catch (error) {
         logger.error('version-manager - readSbtVersion', 'Failed to read version', error);
         throw new Error(`Failed to read version.sbt: ${error.message}`);
@@ -722,7 +727,10 @@ function readChangelogVersion() {
         const match = content.match(versionRegex);
 
         if (!match) {
-            logger.debug('version-manager - readChangelogVersion', 'No version found in CHANGELOG.md');
+            logger.debug(
+                'version-manager - readChangelogVersion',
+                'No version found in CHANGELOG.md'
+            );
             return null;
         }
 
@@ -730,14 +738,21 @@ function readChangelogVersion() {
 
         // Skip [Unreleased]
         if (version.toLowerCase() === 'unreleased') {
-            logger.debug('version-manager - readChangelogVersion', 'Latest entry is [Unreleased], looking for next version');
+            logger.debug(
+                'version-manager - readChangelogVersion',
+                'Latest entry is [Unreleased], looking for next version'
+            );
 
             // Find next version entry
             const allMatches = content.match(/##\s*\[([^\]]+)\]/g);
             if (allMatches && allMatches.length > 1) {
                 const secondMatch = allMatches[1].match(/\[([^\]]+)\]/);
                 const actualVersion = secondMatch ? secondMatch[1].trim() : null;
-                logger.debug('version-manager - readChangelogVersion', 'Version read from second entry', { version: actualVersion });
+                logger.debug(
+                    'version-manager - readChangelogVersion',
+                    'Version read from second entry',
+                    { version: actualVersion }
+                );
                 return actualVersion;
             }
 
@@ -746,7 +761,6 @@ function readChangelogVersion() {
 
         logger.debug('version-manager - readChangelogVersion', 'Version read', { version });
         return version;
-
     } catch (error) {
         logger.error('version-manager - readChangelogVersion', 'Failed to read CHANGELOG', error);
         return null; // Non-critical, return null instead of throwing
@@ -846,7 +860,10 @@ export function incrementVersion(currentVersion, bumpType, suffix = null) {
  * @param {string} newVersion - New version string
  */
 function updateGradleVersion(filePath, newVersion) {
-    logger.debug('version-manager - updateGradleVersion', 'Updating build.gradle', { filePath, newVersion });
+    logger.debug('version-manager - updateGradleVersion', 'Updating build.gradle', {
+        filePath,
+        newVersion
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -865,10 +882,15 @@ function updateGradleVersion(filePath, newVersion) {
 
         fs.writeFileSync(filePath, newContent, 'utf8');
 
-        logger.debug('version-manager - updateGradleVersion', 'build.gradle updated', { path: filePath });
-
+        logger.debug('version-manager - updateGradleVersion', 'build.gradle updated', {
+            path: filePath
+        });
     } catch (error) {
-        logger.error('version-manager - updateGradleVersion', 'Failed to update build.gradle', error);
+        logger.error(
+            'version-manager - updateGradleVersion',
+            'Failed to update build.gradle',
+            error
+        );
         throw new Error(`Failed to update build.gradle: ${error.message}`);
     }
 }
@@ -881,7 +903,10 @@ function updateGradleVersion(filePath, newVersion) {
  * @param {string} newVersion - New version string
  */
 function updateGradleKtsVersion(filePath, newVersion) {
-    logger.debug('version-manager - updateGradleKtsVersion', 'Updating build.gradle.kts', { filePath, newVersion });
+    logger.debug('version-manager - updateGradleKtsVersion', 'Updating build.gradle.kts', {
+        filePath,
+        newVersion
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -900,10 +925,15 @@ function updateGradleKtsVersion(filePath, newVersion) {
 
         fs.writeFileSync(filePath, newContent, 'utf8');
 
-        logger.debug('version-manager - updateGradleKtsVersion', 'build.gradle.kts updated', { path: filePath });
-
+        logger.debug('version-manager - updateGradleKtsVersion', 'build.gradle.kts updated', {
+            path: filePath
+        });
     } catch (error) {
-        logger.error('version-manager - updateGradleKtsVersion', 'Failed to update build.gradle.kts', error);
+        logger.error(
+            'version-manager - updateGradleKtsVersion',
+            'Failed to update build.gradle.kts',
+            error
+        );
         throw new Error(`Failed to update build.gradle.kts: ${error.message}`);
     }
 }
@@ -916,7 +946,10 @@ function updateGradleKtsVersion(filePath, newVersion) {
  * @param {string} newVersion - New version string
  */
 function updatePyprojectVersion(filePath, newVersion) {
-    logger.debug('version-manager - updatePyprojectVersion', 'Updating pyproject.toml', { filePath, newVersion });
+    logger.debug('version-manager - updatePyprojectVersion', 'Updating pyproject.toml', {
+        filePath,
+        newVersion
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -945,10 +978,15 @@ function updatePyprojectVersion(filePath, newVersion) {
 
         fs.writeFileSync(filePath, newContent, 'utf8');
 
-        logger.debug('version-manager - updatePyprojectVersion', 'pyproject.toml updated', { path: filePath });
-
+        logger.debug('version-manager - updatePyprojectVersion', 'pyproject.toml updated', {
+            path: filePath
+        });
     } catch (error) {
-        logger.error('version-manager - updatePyprojectVersion', 'Failed to update pyproject.toml', error);
+        logger.error(
+            'version-manager - updatePyprojectVersion',
+            'Failed to update pyproject.toml',
+            error
+        );
         throw new Error(`Failed to update pyproject.toml: ${error.message}`);
     }
 }
@@ -961,7 +999,10 @@ function updatePyprojectVersion(filePath, newVersion) {
  * @param {string} newVersion - New version string
  */
 function updateCargoVersion(filePath, newVersion) {
-    logger.debug('version-manager - updateCargoVersion', 'Updating Cargo.toml', { filePath, newVersion });
+    logger.debug('version-manager - updateCargoVersion', 'Updating Cargo.toml', {
+        filePath,
+        newVersion
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -980,8 +1021,9 @@ function updateCargoVersion(filePath, newVersion) {
 
         fs.writeFileSync(filePath, newContent, 'utf8');
 
-        logger.debug('version-manager - updateCargoVersion', 'Cargo.toml updated', { path: filePath });
-
+        logger.debug('version-manager - updateCargoVersion', 'Cargo.toml updated', {
+            path: filePath
+        });
     } catch (error) {
         logger.error('version-manager - updateCargoVersion', 'Failed to update Cargo.toml', error);
         throw new Error(`Failed to update Cargo.toml: ${error.message}`);
@@ -996,7 +1038,10 @@ function updateCargoVersion(filePath, newVersion) {
  * @param {string} newVersion - New version string
  */
 function updateSbtVersion(filePath, newVersion) {
-    logger.debug('version-manager - updateSbtVersion', 'Updating version.sbt', { filePath, newVersion });
+    logger.debug('version-manager - updateSbtVersion', 'Updating version.sbt', {
+        filePath,
+        newVersion
+    });
 
     try {
         if (!fs.existsSync(filePath)) {
@@ -1015,8 +1060,9 @@ function updateSbtVersion(filePath, newVersion) {
 
         fs.writeFileSync(filePath, newContent, 'utf8');
 
-        logger.debug('version-manager - updateSbtVersion', 'version.sbt updated', { path: filePath });
-
+        logger.debug('version-manager - updateSbtVersion', 'version.sbt updated', {
+            path: filePath
+        });
     } catch (error) {
         logger.error('version-manager - updateSbtVersion', 'Failed to update version.sbt', error);
         throw new Error(`Failed to update version.sbt: ${error.message}`);
@@ -1031,13 +1077,17 @@ function updateSbtVersion(filePath, newVersion) {
  * @returns {boolean} True if valid, false otherwise
  */
 export function validateVersionFormat(version) {
-    logger.debug('version-manager - validateVersionFormat', 'Validating version format', { version });
+    logger.debug('version-manager - validateVersionFormat', 'Validating version format', {
+        version
+    });
 
     try {
         parseVersion(version);
         return true;
     } catch (error) {
-        logger.debug('version-manager - validateVersionFormat', 'Invalid version format', { version });
+        logger.debug('version-manager - validateVersionFormat', 'Invalid version format', {
+            version
+        });
         return false;
     }
 }
@@ -1084,7 +1134,6 @@ export function compareVersions(version1, version2) {
 
         // Both have suffixes or both don't
         return 0;
-
     } catch (error) {
         // Log at debug level instead of error - non-semver versions are expected
         logger.debug('version-manager - compareVersions', 'Cannot compare versions (not semver)', {
@@ -1171,7 +1220,6 @@ export async function validateVersionAlignment() {
         logger.debug('version-manager - validateVersionAlignment', 'Validation complete', result);
 
         return result;
-
     } catch (error) {
         logger.error('version-manager - validateVersionAlignment', 'Validation failed', error);
         return {
@@ -1183,4 +1231,3 @@ export async function validateVersionAlignment() {
         };
     }
 }
-