claude-git-hooks 2.14.4 → 2.16.0

package/CHANGELOG.md

@@ -5,84 +5,195 @@ Todos los cambios notables en este proyecto se documentarán en este archivo.
 El formato está basado en [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 y este proyecto adhiere a [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [2.14.4] - 2026-02-06
+## [2.16.0] - 2026-02-15
+
+### ✨ Added
+- Multi-file version discovery with recursive scanning up to 3 levels deep
+- Per-file version editing with `--interactive` flag for advanced control
+- Support for additional project types: build.gradle, build.gradle.kts, pyproject.toml, Cargo.toml, version.sbt
+- Suffix-only operations with `--remove-suffix` and `--set-suffix` flags
+- Interactive file selection menu when version mismatch detected or `--interactive` enabled
+- Version files discovery table showing all detected files with versions
+- Automatic rollback of version changes on failure
+- New UI components: `promptToggleList()` and `promptEditField()` in interactive-ui.js
+
+### 🔧 Changed
+- Refactored version-manager.js to use new discovery-based API with `discoverVersionFiles()`, `updateVersionFiles()`, and `modifySuffix()`
+- Bump-version workflow now supports selecting specific files to update
+- Each file can have an independent target version when using per-file editing mode
+- Updated CLAUDE.md documentation with new version bump flow and API changes
+
+### ⚠️ Deprecated
+- Legacy version-manager functions `detectProjectType()`, `getCurrentVersion()`, `updateVersion()`, and `getDiscoveredPaths()` replaced by new discovery API
+
+
+## [2.15.5] - 2026-02-12
+
+### ✨ Added
+- Added comprehensive CI-safe integration test suite that runs without Claude CLI dependency
+- Added new test scripts: `test:integration:ci` for CI environments and `test:changed` for testing only modified files
+
+### 🔧 Changed
+- Optimized CI matrix to run full OS/Node.js combinations only on push events, reducing PR build times
+- Refactored integration tests to verify installation workflow, hook lifecycle, config persistence, and CLI routing
+- Improved timeout handling in Claude CLI client by clearing timeouts consistently on all exit paths
+- Simplified template variable replacement in prompt-builder with cleaner regex handling
+- Reduced Jest verbosity and coverage reporters for cleaner CI output
+
+### 🐛 Fixed
+- Fixed potential timeout leak in claude-client.js where timeouts were not cleared on error or stdin failures
+
+### 🗑️ Removed
+- Removed unused variables and parameters across multiple modules (analyze-diff, create-pr, install, config, changelog-generator, claude-client, claude-diagnostics, file-utils, git-operations, prompt-builder)
+
+
+## [2.15.4] - 2026-02-11
+
+### ✨ Added
+- Added comprehensive unit tests for claude-diagnostics.js covering error detection, formatting, and recovery classification
+- Added unit tests for file-utils.js covering directory creation and file writing operations
+
+
+## [2.15.3] - 2026-02-10
+
+### ✨ Added
+- Unit tests for `generate-changelog.js` command covering CHANGELOG generation, version detection, and error handling
+- Unit tests for `github-client.js` covering CODEOWNERS parsing, reviewer detection, and GitHub repo parsing
+- Unit tests for `hooks.js` command covering enable, disable, status, and uninstall operations
+
+
+## [2.15.2] - 2026-02-10
 
 ### 🔧 Changed
 - The bump-version command now uses the configured default branch (from github.pr.defaultBase) in the 'Next steps' output instead of hardcoded 'main'
 
+### 🐛 Fixed
+- Fixed cross-platform path assertions in unit tests to use partial matching instead of exact paths, improving Windows compatibility
+
+
+## [2.15.1] - 2026-02-10
+
+### 🐛 Fixed
+
+- Fixed cross-platform test compatibility for Windows environments by updating path assertions to use partial matching instead of exact Unix paths
+- Fixed CI workflow to properly skip integration tests that require Claude CLI, with clear documentation about manual testing requirements
+- Fixed Unix-specific tests to be skipped on Windows platform
+- Fixed executable file permissions for bin/claude-hooks and hook templates (pre-commit, prepare-commit-msg)
+
+## [2.15.0] - 2026-02-10
+
+### ✨ Added
+
+- Comprehensive testing suite with GitHub Actions CI workflow including lint, smoke tests, unit tests, integration tests, and dogfooding checks
+- Cross-platform unit tests running on Ubuntu, Windows, and macOS with Node.js 18, 20, and 22
+- Codecov integration for coverage reporting on unit and integration tests
+- Per-module coverage thresholds for critical modules (config.js, helpers.js, git-operations.js, preset-loader.js, analysis-engine.js)
+
+### 🔧 Changed
+
+- Updated ESLint configuration from ECMAScript 2021 to 2022
+- Improved Jest configuration with separate coverage reporters (text, text-summary, lcov, html) and adjusted global coverage thresholds
+- Updated .gitignore to track test files (\*.test.js) while ignoring coverage directory
+
+### 🐛 Fixed
+
+- Fixed ESLint linting issues across utility and command modules including string concatenation, switch case indentation, and arrow function formatting (#68)
+
+## [2.14.4] - 2026-02-09
+
+### 🔧 Changed
+
+- The migrate-config command now detects and fixes incorrectly placed preset fields in config overrides
+
+### 🐛 Fixed
+
+- Fixed preset field being incorrectly placed inside overrides when using --set-preset command (#66)
 
 ## [2.14.3] - 2026-02-06
 
+### 🔧 Changed
+
+- The bump-version command now uses the configured default branch (from github.pr.defaultBase) in the 'Next steps' output instead of hardcoded 'main'
 
 ## [2.14.2] - 2026-02-06
 
 ### ✨ Added
+
 - New `--push` flag for `bump-version` command to explicitly push tags to remote when desired
 
 ### 🔧 Changed
+
 - Changed `bump-version` default behavior to keep tags local instead of automatically pushing to remote (#65)
 - Tags are now pushed by `create-pr` command or manually with `--push` flag, providing better control over the release workflow
 - Updated documentation in README.md, README-NPM.md, and CLAUDE.md to reflect new tag push behavior
 
 ### ⚠️ Deprecated
+
 - The `--no-push` flag for `bump-version` is no longer needed as tags now stay local by default
 
 ### 🗑️ Removed
-- Removed RFC-001 documentation file (pr-metadata-engine-refactor.md) as the refactor has been completed
 
+- Removed RFC-001 documentation file (pr-metadata-engine-refactor.md) as the refactor has been completed
 
 ## [2.14.1] - 2026-02-06
 
 ### 🔧 Changed
+
 - Improved `bump-version` command workflow - now automatically stages and commits version changes with conventional commit format before tag creation
 - Enhanced `bump-version` command flexibility - added `--no-commit` flag for manual workflows where users prefer to commit changes themselves
 - Improved error handling in `bump-version` - provides clearer guidance when staging or committing fails, with manual fallback instructions
 - Enhanced next-steps guidance in `bump-version` output - adapts instructions based on flags used (--no-commit, --no-push, --no-tag)
 
 ### 🐛 Fixed
+
 - Fixed `bump-version` workflow inconsistency - prevented tag creation when using `--no-commit` flag, as tags should only be created after changes are committed
 - Fixed potential git history issues - version bump commits now use `--no-verify` flag to bypass pre-commit hooks and prevent circular execution
 
 ### 🗑️ Removed
-- Removed redundant fallback warnings from `analyze-diff` and `create-pr` commands - warnings were unnecessary as base branch resolution now throws clear errors with suggestions instead
 
+- Removed redundant fallback warnings from `analyze-diff` and `create-pr` commands - warnings were unnecessary as base branch resolution now throws clear errors with suggestions instead
 
 ## [2.14.0] - 2026-02-06
 
 ### ✨ Added
+
 - PR metadata engine (`pr-metadata-engine.js`) - centralized module for generating PR titles, descriptions, and test plans from branch diffs with timeout resilience (#63)
 - Tiered diff reduction system - automatically truncates large diffs using context reduction, proportional budgets, and stat-only summaries to prevent timeouts
 - Extended git operations - added `fetchRemote()`, `branchExists()`, `resolveBaseBranch()`, `getChangedFilesBetweenRefs()`, `getDiffBetweenRefs()`, and `getCommitsBetweenRefs()` to `git-operations.js`
 - RFC-001 documentation - comprehensive design specification for PR metadata engine refactor with timeout resilience strategy
 
 ### 🔧 Changed
+
 - Refactored `analyze-diff` command to use shared PR metadata engine - reduced from 262 to ~80 lines by eliminating duplicate git logic
 - Refactored `create-pr` command to use shared PR metadata engine - eliminated duplicate diff extraction logic (lines 340-420)
 - Improved timeout error handling - timeout errors now include `errorInfo` and are classified as recoverable for automatic retry with exponential backoff
 - Updated CLAUDE.md with PR metadata engine architecture, new git operations exports, and tiered diff reduction strategy
 
 ### 🐛 Fixed
+
 - Fixed timeout failures in `analyze-diff` for large diffs - now uses tiered reduction instead of monolithic diff processing
 - Fixed inconsistent behavior between `analyze-diff` and `create-pr` commands - both now share identical analysis logic via unified engine
 
 ### 🗑️ Removed
-- Removed unreliable `SUBAGENT_INSTRUCTION` text hint - replaced with deterministic tiered diff reduction for parallel processing
 
+- Removed unreliable `SUBAGENT_INSTRUCTION` text hint - replaced with deterministic tiered diff reduction for parallel processing
 
 ## [2.13.0] - 2026-02-05
 
 ### ✨ Added
+
 - Interactive analysis command `claude-hooks analyze` - review all issues (INFO to BLOCKER) interactively before committing with auto-commit option
 - Shared analysis engine module (`analysis-engine.js`) - centralized file data building, analysis orchestration, and results display for both pre-commit hooks and interactive analysis
 - Support for version files in subdirectories - automatically searches parent directories when version files not found in repo root
 - Auto-commit functionality after interactive analysis - creates commit with auto-generated message when user confirms
 
 ### 🔧 Changed
+
 - Improved version detection logic for pom.xml files - enhanced reliability of version extraction in Maven projects
 
 ### 🐛 Fixed
-- Fixed version search logic for pom.xml files - corrected parsing issues in Maven version detection
 
+- Fixed version search logic for pom.xml files - corrected parsing issues in Maven version detection
 
 ## [2.12.0] - 2026-02-03
 

package/LICENSE

@@ -1,21 +1,21 @@
-MIT License
-
-Copyright (c) 2024 Pablo Rovito
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+MIT License
+
+Copyright (c) 2024 Pablo Rovito
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.

package/bin/claude-hooks

@@ -34,90 +34,90 @@ async function main() {
     const command = args[0];
 
     switch (command) {
-        case 'install':
-            await runInstall(args.slice(1));
-            break;
-        case 'update':
-            await runUpdate();
-            break;
-        case 'uninstall':
-            runUninstall();
-            break;
-        case 'enable':
-            runEnable(args[1]);
-            break;
-        case 'disable':
-            runDisable(args[1]);
-            break;
-        case 'status':
-            runStatus();
-            break;
-        case 'analyze':
-            await runAnalyze({
-                staged: !args.includes('--unstaged') && !args.includes('--all'),
-                unstaged: args.includes('--unstaged'),
-                all: args.includes('--all')
-            });
-            break;
-        case 'analyze-diff':
-            await runAnalyzeDiff(args.slice(1));
-            break;
-        case 'create-pr':
-            await runCreatePr(args.slice(1));
-            break;
-        case 'setup-github':
-            await runSetupGitHub();
-            break;
-        case 'presets':
-            await runShowPresets();
-            break;
-        case '--set-preset':
-            await runSetPreset(args[1]);
-            break;
-        case 'preset':
-            // Handle subcommands: preset current
-            if (args[1] === 'current') {
-                await runCurrentPreset();
-            } else {
-                error(`Unknown preset subcommand: ${args[1]}`);
-            }
-            break;
-        case 'migrate-config':
-            await runMigrateConfig();
-            break;
-        case 'bump-version':
-            await runBumpVersion(args.slice(1));
-            break;
-        case 'generate-changelog':
-            await runGenerateChangelog(args.slice(1));
-            break;
-        case 'telemetry':
-            // Handle subcommands: telemetry show, telemetry clear
-            if (args[1] === 'show' || args[1] === undefined) {
-                await runShowTelemetry();
-            } else if (args[1] === 'clear') {
-                await runClearTelemetry();
-            } else {
-                error(`Unknown telemetry subcommand: ${args[1]}`);
-            }
-            break;
-        case '--debug':
-            await runSetDebug(args[1]);
-            break;
-        case '--version':
-        case '-v':
-        case 'version':
-            runShowVersion();
-            break;
-        case 'help':
-        case '--help':
-        case '-h':
-        case undefined:
-            runShowHelp();
-            break;
-        default:
-            error(`Unknown command: ${command}`);
-            runShowHelp();
+    case 'install':
+        await runInstall(args.slice(1));
+        break;
+    case 'update':
+        await runUpdate();
+        break;
+    case 'uninstall':
+        runUninstall();
+        break;
+    case 'enable':
+        runEnable(args[1]);
+        break;
+    case 'disable':
+        runDisable(args[1]);
+        break;
+    case 'status':
+        runStatus();
+        break;
+    case 'analyze':
+        await runAnalyze({
+            staged: !args.includes('--unstaged') && !args.includes('--all'),
+            unstaged: args.includes('--unstaged'),
+            all: args.includes('--all')
+        });
+        break;
+    case 'analyze-diff':
+        await runAnalyzeDiff(args.slice(1));
+        break;
+    case 'create-pr':
+        await runCreatePr(args.slice(1));
+        break;
+    case 'setup-github':
+        await runSetupGitHub();
+        break;
+    case 'presets':
+        await runShowPresets();
+        break;
+    case '--set-preset':
+        await runSetPreset(args[1]);
+        break;
+    case 'preset':
+        // Handle subcommands: preset current
+        if (args[1] === 'current') {
+            await runCurrentPreset();
+        } else {
+            error(`Unknown preset subcommand: ${args[1]}`);
+        }
+        break;
+    case 'migrate-config':
+        await runMigrateConfig();
+        break;
+    case 'bump-version':
+        await runBumpVersion(args.slice(1));
+        break;
+    case 'generate-changelog':
+        await runGenerateChangelog(args.slice(1));
+        break;
+    case 'telemetry':
+        // Handle subcommands: telemetry show, telemetry clear
+        if (args[1] === 'show' || args[1] === undefined) {
+            await runShowTelemetry();
+        } else if (args[1] === 'clear') {
+            await runClearTelemetry();
+        } else {
+            error(`Unknown telemetry subcommand: ${args[1]}`);
+        }
+        break;
+    case '--debug':
+        await runSetDebug(args[1]);
+        break;
+    case '--version':
+    case '-v':
+    case 'version':
+        runShowVersion();
+        break;
+    case 'help':
+    case '--help':
+    case '-h':
+    case undefined:
+        runShowHelp();
+        break;
+    default:
+        error(`Unknown command: ${command}`);
+        runShowHelp();
     }
 }
 

package/lib/commands/analyze-diff.js

@@ -11,7 +11,6 @@ import {
     colors,
     error,
     info,
-    warning,
     checkGitRepo
 } from './helpers.js';