claude-git-hooks 2.51.2 → 2.66.1

package/lib/utils/git-tag-manager.js

@@ -196,6 +196,51 @@ export function getLatestLocalTag() {
     }
 }
 
+/**
+ * Gets latest local tag reachable from HEAD
+ * Why: Excludes rogue tags from unrelated branches that were fetched locally
+ *
+ * @returns {string|null} Latest semver tag reachable from HEAD, or null
+ */
+export function getLatestLocalTagOnBranch() {
+    logger.debug('git-tag-manager - getLatestLocalTagOnBranch', 'Getting latest local tag on HEAD');
+
+    try {
+        const output = execGitTagCommand('git tag --merged HEAD --sort=-v:refname');
+
+        if (!output) {
+            logger.debug('git-tag-manager - getLatestLocalTagOnBranch', 'No tags on HEAD');
+            return null;
+        }
+
+        const tags = output.split(/\r?\n/).filter((t) => t.length > 0);
+        const semverTags = tags.filter(isSemverTag);
+
+        if (semverTags.length === 0) {
+            logger.debug('git-tag-manager - getLatestLocalTagOnBranch', 'No semver tags on HEAD', {
+                totalTags: tags.length
+            });
+            return null;
+        }
+
+        const latestTag = semverTags[0];
+
+        logger.debug('git-tag-manager - getLatestLocalTagOnBranch', 'Latest tag on HEAD', {
+            latestTag,
+            semverTags: semverTags.length
+        });
+
+        return latestTag;
+    } catch (error) {
+        logger.error(
+            'git-tag-manager - getLatestLocalTagOnBranch',
+            'Failed to get tags on HEAD',
+            error
+        );
+        return null;
+    }
+}
+
 /**
  * Gets all remote tags
  * Why: Compare local tags with remote for push status
@@ -311,6 +356,65 @@ export async function getLatestRemoteTag(remoteName = null) {
     }
 }
 
+/**
+ * Gets latest remote tag reachable from a specific branch
+ * Why: Scoped comparison — avoids rogue tags pushed from unmerged feature branches
+ *
+ * @param {string} baseBranch - Branch to scope tags to (e.g., 'develop')
+ * @param {string} remoteName - Remote name (default: 'origin')
+ * @returns {string|null} Latest semver tag name reachable from the branch, or null
+ */
+export function getLatestRemoteTagOnBranch(baseBranch, remoteName = null) {
+    const remote = remoteName || getRemoteName();
+    logger.debug('git-tag-manager - getLatestRemoteTagOnBranch', 'Getting latest tag on branch', {
+        baseBranch,
+        remote
+    });
+
+    try {
+        // Ensure remote branch ref is fresh
+        execGitTagCommand(`git fetch ${remote} ${baseBranch} --quiet`);
+
+        // Get tags merged into the remote branch, sorted by version descending
+        const output = execGitTagCommand(
+            `git tag --merged ${remote}/${baseBranch} --sort=-v:refname`
+        );
+
+        if (!output) {
+            logger.debug('git-tag-manager - getLatestRemoteTagOnBranch', 'No tags on branch');
+            return null;
+        }
+
+        const tags = output.split(/\r?\n/).filter((line) => line.length > 0);
+        const semverTags = tags.filter(isSemverTag);
+
+        if (semverTags.length === 0) {
+            logger.debug('git-tag-manager - getLatestRemoteTagOnBranch', 'No semver tags on branch', {
+                totalTags: tags.length
+            });
+            return null;
+        }
+
+        // Already sorted by git --sort=-v:refname (descending), first is latest
+        const latestTag = semverTags[0];
+
+        logger.debug('git-tag-manager - getLatestRemoteTagOnBranch', 'Latest tag on branch', {
+            baseBranch,
+            latestTag,
+            semverTags: semverTags.length
+        });
+
+        return latestTag;
+    } catch (error) {
+        logger.error(
+            'git-tag-manager - getLatestRemoteTagOnBranch',
+            'Failed to get tags on branch',
+            error
+        );
+        return null;
+    }
+}
+
 /**
  * Checks if tag exists
  * Why: Prevents duplicate tag creation

package/lib/utils/github-api.js

@@ -940,6 +940,36 @@ export const findExistingPR = async ({ owner, repo, head, base }) => {
     }
 };
 
+/**
+ * Update the body of an existing pull request
+ * @param {string} owner - Repository owner
+ * @param {string} repo - Repository name
+ * @param {number} number - PR number
+ * @param {string} body - New PR body content
+ * @returns {Promise<Object>} Updated PR data
+ */
+export const updatePullRequestBody = async (owner, repo, number, body) => {
+    logger.debug('github-api - updatePullRequestBody', 'Updating PR body', {
+        owner, repo, number, bodyLength: body.length
+    });
+
+    const octokit = getOctokit();
+
+    const { data } = await octokit.pulls.update({
+        owner,
+        repo,
+        pull_number: number,
+        body
+    });
+
+    logger.debug('github-api - updatePullRequestBody', 'PR body updated', {
+        number: data.number,
+        url: data.html_url
+    });
+
+    return data;
+};
+
 /**
  * Get repository information
  * Why: Fetch repo metadata for validation and context

package/lib/utils/judge.js

@@ -139,7 +139,8 @@ const judgeAndFix = async (analysisResult, filesData, config, { headless = false
     const response = await executeClaudeWithRetry(prompt, {
         model,
         timeout: judgeTimeout,
-        headless
+        headless,
+        print: true
     });
 
     const parsed = extractJSON(response);

package/lib/utils/linter-runner.js

@@ -45,6 +45,12 @@ export function parseEslintOutput(stdout) {
 
         for (const fileResult of results) {
             for (const msg of fileResult.messages || []) {
+                // Skip ESLint meta-warnings about ignored files — not code quality issues.
+                // Dot-directory files (e.g. .library/) trigger "File ignored by default"
+                // even when ignorePatterns negation is set, because ESLint's default
+                // dot-directory ignore takes precedence for explicitly passed file paths.
+                if (msg.message && msg.message.startsWith('File ignored')) continue;
+
                 const issue = {
                     file: fileResult.filePath || '',
                     line: msg.line,

package/lib/utils/version-manager.js

@@ -248,13 +248,14 @@ export function discoverVersionFiles(options = {}) {
                 const registry = VERSION_FILE_TYPES[fileType];
                 if (registry && entry.name === registry.filename) {
                     const version = registry.readVersion(fullPath);
+                    const isSemver = version !== null && validateVersionFormat(version);
                     const descriptor = {
                         path: fullPath,
                         relativePath: path.relative(repoRoot, fullPath),
                         type: fileType,
                         projectLabel: registry.projectLabel,
                         version,
-                        selected: true
+                        selected: isSemver
                     };
                     discoveredFiles.push(descriptor);
                     logger.debug('version-manager - discoverVersionFiles', 'Found version file', {
@@ -285,19 +286,19 @@ export function discoverVersionFiles(options = {}) {
         return a.relativePath.localeCompare(b.relativePath);
     });
 
-    // Determine resolved version (prefer root-level file, then first found)
+    // Determine resolved version from semver files only (prefer root-level, then first found)
     let resolvedVersion = null;
-    const rootFile = discoveredFiles.find((f) => !f.relativePath.includes(path.sep));
+    const semverFiles = discoveredFiles.filter((f) => f.selected);
+    const rootFile = semverFiles.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);
+    } else if (semverFiles.length > 0) {
+        const firstWithVersion = semverFiles.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);
+    // Check for version mismatch (only among semver files)
+    const versions = semverFiles.filter((f) => f.version !== null).map((f) => f.version);
     const uniqueVersions = [...new Set(versions)];
     const mismatch = uniqueVersions.length > 1;
 
@@ -1149,24 +1150,36 @@ export function compareVersions(version1, version2) {
  *
  * @returns {Promise<Object>} Validation result with alignment status and issues
  */
-export async function validateVersionAlignment() {
-    logger.debug('version-manager - validateVersionAlignment', 'Validating version alignment');
+export async function validateVersionAlignment(baseBranch = null) {
+    logger.debug('version-manager - validateVersionAlignment', 'Validating version alignment', {
+        baseBranch
+    });
 
     try {
         // Discover all version files
         const discovery = discoverVersionFiles();
 
-        // Get git tag version (parseTagVersion returns null for non-semver tags)
-        const { getLatestLocalTag, parseTagVersion } = await import('./git-tag-manager.js');
-        const latestTag = getLatestLocalTag();
+        // Get git tag version scoped to HEAD when baseBranch is provided
+        const {
+            getLatestLocalTag, getLatestLocalTagOnBranch,
+            parseTagVersion,
+            getLatestRemoteTag, getLatestRemoteTagOnBranch
+        } = await import('./git-tag-manager.js');
+        const latestTag = baseBranch ? getLatestLocalTagOnBranch(baseBranch) : getLatestLocalTag();
         const tagVersion = latestTag ? parseTagVersion(latestTag) : null;
 
         // Get CHANGELOG version
         const changelogVersion = readChangelogVersion();
-
-        // Get remote tag version (parseTagVersion returns null for non-semver tags)
-        const { getLatestRemoteTag } = await import('./git-tag-manager.js');
-        const latestRemoteTag = await getLatestRemoteTag();
+        let latestRemoteTag;
+        if (baseBranch) {
+            latestRemoteTag = getLatestRemoteTagOnBranch(baseBranch);
+            // Fall back to global if branch-scoped lookup returns nothing
+            if (!latestRemoteTag) {
+                latestRemoteTag = await getLatestRemoteTag();
+            }
+        } else {
+            latestRemoteTag = await getLatestRemoteTag();
+        }
         const remoteVersion = latestRemoteTag ? parseTagVersion(latestRemoteTag) : null;
 
         // Collect all local versions

package/package.json

@@ -1,83 +1,85 @@
-{
-    "name": "claude-git-hooks",
-    "version": "2.51.2",
-    "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
-    "type": "module",
-    "bin": {
-        "claude-hooks": "./bin/claude-hooks"
-    },
-    "scripts": {
-        "test": "npm run test:all",
-        "test:all": "npm run lint && npm run test:smoke && npm run test:unit && npm run test:integration",
-        "test:smoke": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/smoke --maxWorkers=1",
-        "test:unit": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --forceExit",
-        "test:integration": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration --maxWorkers=1 --testTimeout=30000 --forceExit",
-        "test:integration:ci": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration/ci-safe.test.js --maxWorkers=1 --testTimeout=30000 --forceExit",
-        "test:changed": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --changedSince=main --forceExit",
-        "test:watch": "node --experimental-vm-modules node_modules/jest/bin/jest.js --watch",
-        "test:coverage": "node --experimental-vm-modules node_modules/jest/bin/jest.js --coverage",
-        "lint": "eslint lib/ bin/claude-hooks",
-        "lint:fix": "eslint lib/ bin/claude-hooks --fix",
-        "format": "prettier --write \"lib/**/*.js\" \"bin/**\" \"test/**/*.js\"",
-        "precommit": "npm run lint && npm run test:smoke",
-        "prepublishOnly": "npm run test:all",
-        "library:check": "node .library/bin/library check",
-        "library:regenerate": "node .library/bin/library regenerate",
-        "library:extract": "node .library/bin/library extract",
-        "library:tokens": "node .library/bin/library tokens",
-        "library:graph": "node .library/bin/library graph",
-        "library:inject": "node .library/bin/library inject",
-        "library:validate": "node .library/bin/library validate",
-        "library:report": "node .library/bin/library report"
-    },
-    "keywords": [
-        "git",
-        "hooks",
-        "claude",
-        "ai",
-        "code-review",
-        "commit-messages",
-        "pre-commit",
-        "automation"
-    ],
-    "author": "Pablo Rovito",
-    "license": "MIT",
-    "repository": {
-        "type": "git",
-        "url": "https://github.com/mscope-S-L/git-hooks.git"
-    },
-    "engines": {
-        "node": ">=16.9.0"
-    },
-    "engineStrict": false,
-    "os": [
-        "darwin",
-        "linux",
-        "win32"
-    ],
-    "preferGlobal": true,
-    "files": [
-        "bin/",
-        "lib/",
-        "templates/",
-        "README.md",
-        "CHANGELOG.md",
-        "CLAUDE.md",
-        "LICENSE"
-    ],
-    "dependencies": {
-        "@anthropic-ai/sdk": "^0.91.0",
-        "@octokit/rest": "^21.0.0",
-        "langfuse": "^3.38.20"
-    },
-    "devDependencies": {
-        "@types/jest": "^29.5.0",
-        "eslint": "^8.57.1",
-        "jest": "^29.7.0",
-        "js-tiktoken": "^1.0.18",
-        "madge": "^8.0.0",
-        "prettier": "^3.2.0",
-        "tree-sitter-wasms": "^0.1.13",
-        "web-tree-sitter": "^0.24.7"
-    }
-}
+{
+    "name": "claude-git-hooks",
+    "version": "2.66.1",
+    "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
+    "type": "module",
+    "bin": {
+        "claude-hooks": "./bin/claude-hooks"
+    },
+    "scripts": {
+        "test": "npm run test:all",
+        "test:all": "npm run lint && npm run test:smoke && npm run test:unit && npm run test:integration",
+        "test:smoke": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/smoke --maxWorkers=1",
+        "test:unit": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --forceExit",
+        "test:integration": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration --maxWorkers=1 --testTimeout=30000 --forceExit",
+        "test:integration:ci": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/integration/ci-safe.test.js --maxWorkers=1 --testTimeout=30000 --forceExit",
+        "test:changed": "node --experimental-vm-modules node_modules/jest/bin/jest.js test/unit --changedSince=main --forceExit",
+        "test:watch": "node --experimental-vm-modules node_modules/jest/bin/jest.js --watch",
+        "test:coverage": "node --experimental-vm-modules node_modules/jest/bin/jest.js --coverage",
+        "test:e2e": "bash test/manual/sdlc-stability-check.sh",
+        "test:full": "npm run test:all && npm run test:e2e",
+        "lint": "eslint lib/ bin/claude-hooks .library/librarian/",
+        "lint:fix": "eslint lib/ bin/claude-hooks .library/librarian/ --fix",
+        "format": "prettier --write \"lib/**/*.js\" \"bin/**\" \"test/**/*.js\"",
+        "precommit": "npm run lint && npm run test:smoke",
+        "prepublishOnly": "npm run test:all",
+        "library:check": "node .library/bin/library check",
+        "library:regenerate": "node .library/bin/library regenerate",
+        "library:extract": "node .library/bin/library extract",
+        "library:tokens": "node .library/bin/library tokens",
+        "library:graph": "node .library/bin/library graph",
+        "library:inject": "node .library/bin/library inject",
+        "library:validate": "node .library/bin/library validate",
+        "library:report": "node .library/bin/library report"
+    },
+    "keywords": [
+        "git",
+        "hooks",
+        "claude",
+        "ai",
+        "code-review",
+        "commit-messages",
+        "pre-commit",
+        "automation"
+    ],
+    "author": "Pablo Rovito",
+    "license": "MIT",
+    "repository": {
+        "type": "git",
+        "url": "https://github.com/mscope-S-L/git-hooks.git"
+    },
+    "engines": {
+        "node": ">=16.9.0"
+    },
+    "engineStrict": false,
+    "os": [
+        "darwin",
+        "linux",
+        "win32"
+    ],
+    "preferGlobal": true,
+    "files": [
+        "bin/",
+        "lib/",
+        "templates/",
+        "README.md",
+        "CHANGELOG.md",
+        "CLAUDE.md",
+        "LICENSE"
+    ],
+    "dependencies": {
+        "@anthropic-ai/sdk": "^0.91.0",
+        "@octokit/rest": "^21.0.0",
+        "langfuse": "^3.38.20"
+    },
+    "devDependencies": {
+        "@types/jest": "^29.5.0",
+        "eslint": "^8.57.1",
+        "jest": "^29.7.0",
+        "js-tiktoken": "^1.0.18",
+        "madge": "^8.0.0",
+        "prettier": "^3.2.0",
+        "tree-sitter-wasms": "^0.1.13",
+        "web-tree-sitter": "^0.24.7"
+    }
+}