claude-git-hooks 2.44.0 → 2.51.2

package/lib/utils/diff-analysis-orchestrator.js

@@ -23,10 +23,11 @@ import path from 'path';
 import { executeClaudeWithRetry, extractJSON } from './claude-client.js';
 import { loadPrompt } from './prompt-builder.js';
 import logger from './logger.js';
+import { getDefaultSection, resolveSection } from './config-registry.js';
 
-// Orchestration model — Opus for semantic grouping quality, not user-configurable
-const ORCHESTRATOR_MODEL = 'opus';
-const ORCHESTRATOR_TIMEOUT = 60000; // 60s — lightweight call, just file overview
+// Orchestration config from lib/defaults.json (sync fallback)
+const { model: ORCHESTRATOR_MODEL, timeout: ORCHESTRATOR_TIMEOUT } =
+    getDefaultSection('orchestrator');
 
 /**
  * Counts added/removed lines in a unified diff string
@@ -184,6 +185,10 @@ const _buildCommonContext = (filesData, batchGroups) => {
  * @returns {Promise<{batches: Array<{files: FileData[], rationale: string, model: string}>, commonContext: string}>}
  */
 export const orchestrateBatches = async (filesData, { headless = false } = {}) => {
+    // Resolve orchestrator config: remote settings.json > local defaults.json
+    const orchConfig = await resolveSection('orchestrator');
+    const orchModel = orchConfig?.model || ORCHESTRATOR_MODEL;
+    const orchTimeout = orchConfig?.timeout || ORCHESTRATOR_TIMEOUT;
     logger.debug('diff-analysis-orchestrator - orchestrateBatches', 'Building file overview', {
         fileCount: filesData.length
     });
@@ -214,20 +219,20 @@ export const orchestrateBatches = async (filesData, { headless = false } = {}) =
     }
 
     logger.debug('diff-analysis-orchestrator - orchestrateBatches', 'Calling Opus orchestrator', {
-        model: ORCHESTRATOR_MODEL,
-        timeout: ORCHESTRATOR_TIMEOUT
+        model: orchModel,
+        timeout: orchTimeout
     });
 
     let rawResponse;
     try {
         rawResponse = await executeClaudeWithRetry(prompt, {
-            model: ORCHESTRATOR_MODEL,
-            timeout: ORCHESTRATOR_TIMEOUT,
+            model: orchModel,
+            timeout: orchTimeout,
             headless,
             telemetryContext: {
                 hook: 'orchestrator',
                 fileCount: filesData.length,
-                model: ORCHESTRATOR_MODEL
+                model: orchModel
             }
         });
     } catch (err) {

package/lib/utils/judge.js

@@ -26,9 +26,9 @@ import { formatBlockingIssues, getAffectedFiles, formatFileContents } from './re
 import { getRepoRoot, getRepoName, getCurrentBranch } from './git-operations.js';
 import logger from './logger.js';
 import { recordMetric } from './metrics.js';
+import { getDefaultSection, resolveSection } from './config-registry.js';
 
-const JUDGE_DEFAULT_MODEL = 'sonnet';
-const JUDGE_TIMEOUT = 360000;
+const { model: JUDGE_DEFAULT_MODEL, timeout: JUDGE_TIMEOUT } = getDefaultSection('judge');
 
 /**
  * Applies a single search/replace fix to a file and stages it
@@ -103,7 +103,10 @@ const applyFix = async (fix, repoRoot) => {
  * @returns {Promise<{fixedCount: number, falsePositiveCount: number, remainingIssues: Array, verdicts: Array}>}
  */
 const judgeAndFix = async (analysisResult, filesData, config, { headless = false } = {}) => {
-    const model = config.judge?.model || JUDGE_DEFAULT_MODEL;
+    // Resolve judge config: remote settings.json > local defaults.json
+    const judgeConfig = await resolveSection('judge');
+    const model = config.judge?.model ?? judgeConfig?.model ?? JUDGE_DEFAULT_MODEL;
+    const judgeTimeout = judgeConfig?.timeout ?? JUDGE_TIMEOUT;
     const repoRoot = getRepoRoot();
 
     // Get all issues: prefer details (all severities), fallback to blockingIssues
@@ -135,7 +138,7 @@ const judgeAndFix = async (analysisResult, filesData, config, { headless = false
     // Call LLM
     const response = await executeClaudeWithRetry(prompt, {
         model,
-        timeout: JUDGE_TIMEOUT,
+        timeout: judgeTimeout,
         headless
     });
 

package/lib/utils/linear-connector.js

@@ -19,11 +19,13 @@
 import https from 'https';
 import logger from './logger.js';
 import { loadToken } from './token-store.js';
+import { getDefaultSection } from './config-registry.js';
 
-const LINEAR_HOSTNAME = 'api.linear.app';
-const LINEAR_PATH = '/graphql';
-const LINEAR_TIMEOUT = 10000;
-const MAX_RETRIES = 2;
+const linearDefaults = getDefaultSection('linear') || {};
+const LINEAR_HOSTNAME = linearDefaults.hostname || 'api.linear.app';
+const LINEAR_PATH = linearDefaults.path || '/graphql';
+const LINEAR_TIMEOUT = linearDefaults.timeout || 10000;
+const MAX_RETRIES = linearDefaults.maxRetries || 2;
 
 /**
  * Custom error for Linear connector failures

package/lib/utils/linter-runner.js

@@ -27,6 +27,7 @@ import {
 import { getRepoRoot } from './git-operations.js';
 import { which } from './which-command.js';
 import { fetchRemoteConfig } from './remote-config.js';
+import { resolveSection } from './config-registry.js';
 import logger from './logger.js';
 
 /**
@@ -422,6 +423,9 @@ export async function runLinters(files, config, presetName = 'default') {
     const timeout = lintConfig.timeout || 30000;
 
     const tools = await getLinterToolsForPreset(presetName);
+
+    // Merge remote tools.json overrides (timeout, perFile, enabled) into tool definitions
+    const remoteToolsConfig = await resolveSection('tools');
     const results = [];
 
     logger.debug('linter-runner - runLinters', 'Starting linters', {
@@ -431,7 +435,18 @@ export async function runLinters(files, config, presetName = 'default') {
         autoFix
     });
 
-    for (const toolDef of tools) {
+    for (let toolDef of tools) {
+        // Apply remote tool config overrides (remote > local)
+        const remoteToolOverride = remoteToolsConfig?.[toolDef.name];
+        if (remoteToolOverride) {
+            toolDef = { ...toolDef };
+            if (remoteToolOverride.timeout !== undefined) toolDef.timeout = remoteToolOverride.timeout;
+            if (remoteToolOverride.perFile !== undefined) toolDef.perFile = remoteToolOverride.perFile;
+            if (remoteToolOverride.enabled === false) {
+                logger.info(`Skipping ${toolDef.name} (disabled by remote config)`);
+                continue;
+            }
+        }
         // Filter files by tool's extensions
         const matchingFiles = filterFilesByTool(files, toolDef);
 

package/lib/utils/pr-metadata-engine.js

@@ -32,6 +32,7 @@ import { executeClaudeWithRetry, extractJSON } from './claude-client.js';
 import { loadPrompt } from './prompt-builder.js';
 import { getConfig } from '../config.js';
 import logger from './logger.js';
+import { getDefaultSection, resolveSection } from './config-registry.js';
 
 /**
  * @typedef {Object} BranchContext
@@ -58,13 +59,11 @@ import logger from './logger.js';
  */
 
 /**
- * Internal defaults (not documented in config.example.json)
+ * Internal defaults loaded from lib/defaults.json
  * Why: Convention over configuration - sensible defaults for 95% of cases
+ * Fallbacks: timeout=180000 (3 min), maxDiffSize=51200 (50KB)
  */
-const DEFAULTS = {
-    timeout: 180000, // 3 minutes
-    maxDiffSize: 50000 // 50KB
-};
+const DEFAULTS = getDefaultSection('prMetadata') || { timeout: 180000, maxDiffSize: 50000 };
 
 /**
  * Builds diff payload with tiered reduction strategy
@@ -310,12 +309,14 @@ export const getBranchContext = async (targetBranch) => {
 
         logger.debug('pr-metadata-engine - getBranchContext', 'Commits retrieved');
 
-        // Build diff payload with tiered reduction
+        // Build diff payload with tiered reduction (remote settings.json > local defaults)
+        const resolvedPrMeta = await resolveSection('prMetadata');
+        const maxDiffSize = resolvedPrMeta?.maxDiffSize || DEFAULTS.maxDiffSize;
         const {
             payload: diff,
             isTruncated,
             truncationDetails
-        } = buildDiffPayload(baseBranch, 'HEAD', files, DEFAULTS.maxDiffSize);
+        } = buildDiffPayload(baseBranch, 'HEAD', files, maxDiffSize);
 
         logger.debug('pr-metadata-engine - getBranchContext', 'Diff payload built', {
             diffSize: diff.length,
@@ -371,7 +372,9 @@ export const getBranchContext = async (targetBranch) => {
 export const generatePRMetadata = async (context, options = {}) => {
     const config = await getConfig();
     const configTimeout = config.analysis?.timeout;
-    const { timeout = configTimeout || DEFAULTS.timeout, hook = 'pr-metadata', headless = false, costTracker = null } = options;
+    const resolvedPrMetaGen = await resolveSection('prMetadata');
+    const defaultTimeout = resolvedPrMetaGen?.timeout || DEFAULTS.timeout;
+    const { timeout = configTimeout || defaultTimeout, hook = 'pr-metadata', headless = false, costTracker = null } = options;
 
     logger.debug('pr-metadata-engine - generatePRMetadata', 'Generating PR metadata', {
         filesCount: context.filesCount,

package/lib/utils/version-manager.js

@@ -377,11 +377,13 @@ export function writeVersionToFile(filePath, type, newVersion) {
 }
 
 /**
- * Updates version in selected files
- * Why: Applies version update to specific VersionFileDescriptor[] subset
+ * Updates version in the given files (resolved mutations from the selector)
+ * Why: Applies version update to the concrete list of files the caller provides.
+ *      Every item in the array is a mutation to apply — the caller is responsible
+ *      for filtering before calling this function.
  *
- * @param {Array} files - Array of VersionFileDescriptor objects with selected=true
- * @param {string} newVersion - New version string
+ * @param {Array} files - Array of VersionFileDescriptor objects to update
+ * @param {string} newVersion - New version string (used when file has no targetVersion)
  */
 export function updateVersionFiles(files, newVersion) {
     logger.debug('version-manager - updateVersionFiles', 'Updating version files', {
@@ -402,10 +404,6 @@ export function updateVersionFiles(files, newVersion) {
     const errors = [];
 
     for (const file of files) {
-        if (!file.selected) {
-            continue; // Skip unselected files
-        }
-
         try {
             // Verify file still exists
             if (!fs.existsSync(file.path)) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-git-hooks",
-    "version": "2.44.0",
+    "version": "2.51.2",
     "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
     "type": "module",
     "bin": {
@@ -21,8 +21,14 @@
         "format": "prettier --write \"lib/**/*.js\" \"bin/**\" \"test/**/*.js\"",
         "precommit": "npm run lint && npm run test:smoke",
         "prepublishOnly": "npm run test:all",
-        "library:tokens": "node .library/tools/measure-tokens.js",
-        "library:graph": "node .library/tools/generate-graph.js"
+        "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",
@@ -70,6 +76,8 @@
         "jest": "^29.7.0",
         "js-tiktoken": "^1.0.18",
         "madge": "^8.0.0",
-        "prettier": "^3.2.0"
+        "prettier": "^3.2.0",
+        "tree-sitter-wasms": "^0.1.13",
+        "web-tree-sitter": "^0.24.7"
     }
 }