claude-git-hooks 2.14.1 → 2.15.5

package/CHANGELOG.md

@@ -5,58 +5,173 @@ 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.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
+
 ## [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/README.md

@@ -111,6 +111,9 @@ claude-hooks bump-version major --update-changelog
 # Preview without applying
 claude-hooks bump-version patch --dry-run
 
+# Push tag immediately (otherwise pushed by create-pr)
+claude-hooks bump-version patch --push
+
 # Manual workflow (skip automatic commit)
 claude-hooks bump-version patch --no-commit
 ```
@@ -121,7 +124,7 @@ claude-hooks bump-version patch --no-commit
 - Generates CHANGELOG entry with Claude (analyzes commits)
 - Commits changes automatically with conventional commit format
 - Creates annotated Git tag with `v` prefix (e.g., `v2.7.0`)
-- Pushes tag to remote automatically
+- Tags stay local by default (use `--push` to push immediately, or let `create-pr` handle it)
 
 **Version workflow:**
 ```

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';
 

package/lib/commands/bump-version.js

@@ -10,7 +10,7 @@
  * 5. [Optional] Generate and update CHANGELOG
  * 6. Stage and commit changes (skipped with --no-commit)
  * 7. Create annotated Git tag (skipped with --no-tag or --no-commit)
- * 8. Push tag to remote (skipped with --no-push)
+ * 8. Push tag to remote (only if --push flag provided, otherwise pushed by create-pr)
  */
 
 import { execSync } from 'child_process';
@@ -21,7 +21,6 @@ import {
     getCurrentVersion,
     incrementVersion,
     updateVersion,
-    parseVersion,
     getDiscoveredPaths
 } from '../utils/version-manager.js';
 import {
@@ -48,9 +47,7 @@ import logger from '../utils/logger.js';
 import {
     colors,
     error,
-    success,
     info,
-    warning,
     checkGitRepo
 } from './helpers.js';
 
@@ -116,7 +113,7 @@ function validatePrerequisites() {
  * Why: Extracts bump type and options from CLI args
  *
  * @param {Array<string>} args - CLI arguments
- * @returns {Object} Parsed args: { bumpType, suffix, updateChangelog, baseBranch, dryRun, noTag, noPush }
+ * @returns {Object} Parsed args: { bumpType, suffix, updateChangelog, baseBranch, dryRun, noTag, push, noCommit }
  */
 function parseArguments(args) {
     logger.debug('bump-version - parseArguments', 'Parsing arguments', { args });
@@ -128,7 +125,7 @@ function parseArguments(args) {
         baseBranch: 'main',
         dryRun: false,
         noTag: false,
-        noPush: false,
+        push: false,      // Changed: default to NOT push (use --push to enable)
         noCommit: false
     };
 
@@ -160,8 +157,8 @@ function parseArguments(args) {
             parsed.dryRun = true;
         } else if (arg === '--no-tag') {
             parsed.noTag = true;
-        } else if (arg === '--no-push') {
-            parsed.noPush = true;
+        } else if (arg === '--push') {
+            parsed.push = true;
         } else if (arg === '--no-commit') {
             parsed.noCommit = true;
         }
@@ -223,7 +220,7 @@ export async function runBumpVersion(args) {
         console.log('  --update-changelog [branch]  Generate CHANGELOG entry (default branch: main)');
         console.log('  --dry-run                Preview changes without applying');
         console.log('  --no-tag                 Skip Git tag creation');
-        console.log('  --no-push                Create tag but don\'t push to remote');
+        console.log('  --push                   Push tag to remote (default: tags stay local)');
         console.log('  --no-commit              Skip automatic commit (manual workflow)');
         console.log('');
         console.log('Examples:');
@@ -348,6 +345,11 @@ export async function runBumpVersion(args) {
     console.log('');
 
     try {
+        // Load config for default branch
+        const config = await getConfig();
+        const defaultBranch = config.github?.pr?.defaultBase || '{target-branch}';
+        logger.debug('bump-version', 'Default branch for PR', { defaultBranch });
+
         // Step 4: Update version files
         logger.debug('bump-version', 'Step 4: Updating version files');
         showInfo('Updating version files...');
@@ -364,7 +366,6 @@ export async function runBumpVersion(args) {
             logger.debug('bump-version', 'Step 5: Generating CHANGELOG');
             showInfo('Generating CHANGELOG entry...');
 
-            const config = await getConfig();
             const isReleaseVersion = !options.suffix; // Final version if no suffix
 
             const changelogResult = await generateChangelogEntry({
@@ -494,9 +495,9 @@ export async function runBumpVersion(args) {
                     showSuccess(`✓ Tag created: ${tagName}`);
                     console.log('');
 
-                    // Step 7: Push tag (if not disabled)
-                    if (!options.noPush) {
-                        logger.debug('bump-version', 'Step 7: Pushing tag to remote');
+                    // Step 8: Push tag (if --push flag provided)
+                    if (options.push) {
+                        logger.debug('bump-version', 'Step 8: Pushing tag to remote');
                         showInfo('Pushing tag to remote...');
 
                         const pushResult = pushTags(null, tagName);
@@ -510,10 +511,10 @@ export async function runBumpVersion(args) {
                             console.log(`  git push origin ${tagName}`);
                         }
                     } else {
-                        showInfo('Tag push skipped (--no-push)');
+                        showInfo('Tag created locally (use --push to push immediately)');
                         console.log('');
-                        console.log('To push the tag later:');
-                        console.log(`  git push origin ${tagName}`);
+                        console.log('Tag will be pushed automatically when you run:');
+                        console.log('  claude-hooks create-pr [branch]');
                     }
                 } else {
                     showError(`Failed to create tag: ${tagResult.error}`);
@@ -546,18 +547,18 @@ export async function runBumpVersion(args) {
             console.log(`  4. Create tag: git tag -a ${tagName} -m "Release version ${newVersion}"`);
             console.log(`  5. Push: git push origin $(git branch --show-current) ${tagName}`);
             console.log('');
-        } else if (commitCreated && !options.noTag && !options.noPush) {
-            console.log('All done! Changes committed and tag pushed to remote.');
+        } else if (commitCreated && !options.noTag && options.push) {
+            console.log('All done! Changes committed, tagged, and pushed to remote.');
             console.log('');
             console.log('Next steps:');
-            console.log('  1. Create PR: claude-hooks create-pr main');
+            console.log(`  1. Create PR: claude-hooks create-pr ${defaultBranch}`);
             console.log('');
         } else {
             console.log('Next steps:');
-            if (commitCreated && options.noPush) {
-                console.log(`  1. Push changes: git push origin $(git branch --show-current) ${tagName}`);
+            if (commitCreated && !options.push) {
+                console.log(`  1. Create PR: claude-hooks create-pr ${defaultBranch}`);
+                console.log('     (will automatically push tag during PR creation)');
             }
-            console.log('  2. Create PR: claude-hooks create-pr main');
             console.log('');
         }