claude-git-hooks 2.30.2 → 2.33.0

package/lib/utils/telemetry.js

@@ -24,7 +24,7 @@ import fs from 'fs/promises';
 import fsSync from 'fs';
 import path from 'path';
 import logger from './logger.js';
-import config from '../config.js';
+import { recordMetric, rotateMetrics as rotateMetricsData, clearMetrics as clearMetricsData } from './metrics.js';
 
 /**
  * Telemetry event structure
@@ -82,15 +82,6 @@ const getCurrentLogFile = () => {
     return path.join(getTelemetryDir(), `telemetry-${date}.jsonl`);
 };
 
-/**
- * Check if telemetry is enabled
- * Why: Enabled by default, users must explicitly disable
- *
- * @returns {boolean} True if telemetry is enabled (default: true)
- */
-const isTelemetryEnabled = () =>
-    // Enabled by default - only disabled if explicitly set to false
-    config.system?.telemetry !== false;
 /**
  * Ensure telemetry directory exists
  * Why: Create on first use
@@ -134,11 +125,6 @@ const appendEvent = async (event) => {
  * @param {Object} metadata - Event metadata (success, retryAttempt, totalRetries)
  */
 export const recordEvent = async (eventType, eventData = {}, metadata = {}) => {
-    // Check opt-in
-    if (!isTelemetryEnabled()) {
-        return;
-    }
-
     try {
         // Ensure directory exists
         await ensureTelemetryDir();
@@ -156,6 +142,11 @@ export const recordEvent = async (eventType, eventData = {}, metadata = {}) => {
 
         // Append to log
         await appendEvent(event);
+
+        // Dual-write to unified metrics
+        recordMetric(eventType, { ...eventData, _meta: metadata }).catch((err) => {
+            logger.debug('telemetry - recordEvent', 'Metrics write failed', err);
+        });
     } catch (error) {
         // Don't fail on telemetry errors
         logger.debug('telemetry - recordEvent', 'Failed to record event', error);
@@ -289,14 +280,6 @@ const readTelemetryEvents = async (maxDays = 7) => {
  * @returns {Promise<Object>} Statistics object
  */
 export const getStatistics = async (maxDays = 7) => {
-    if (!isTelemetryEnabled()) {
-        return {
-            enabled: false,
-            message:
-                'Telemetry is disabled. To enable (default), remove or set "system.telemetry: true" in .claude/config.json'
-        };
-    }
-
     try {
         const events = await readTelemetryEvents(maxDays);
 
@@ -415,10 +398,6 @@ export const getStatistics = async (maxDays = 7) => {
  * @param {number} maxDays - Keep files newer than this many days (default: 30)
  */
 export const rotateTelemetry = async (maxDays = 30) => {
-    if (!isTelemetryEnabled()) {
-        return;
-    }
-
     try {
         const dir = getTelemetryDir();
         const files = await fs.readdir(dir);
@@ -450,6 +429,9 @@ export const rotateTelemetry = async (maxDays = 30) => {
     } catch (error) {
         logger.debug('telemetry - rotateTelemetry', 'Failed to rotate telemetry', error);
     }
+
+    // Also rotate unified metrics (90-day retention)
+    await rotateMetricsData().catch(() => {});
 };
 
 /**
@@ -479,6 +461,9 @@ export const clearTelemetry = async () => {
     } catch (error) {
         logger.error('telemetry - clearTelemetry', 'Failed to clear telemetry', error);
     }
+
+    // Also clear unified metrics
+    await clearMetricsData().catch(() => {});
 };
 
 /**
@@ -492,18 +477,6 @@ export const displayStatistics = async () => {
     console.log('║                     TELEMETRY STATISTICS                           ║');
     console.log('╚════════════════════════════════════════════════════════════════════╝\n');
 
-    if (!stats.enabled) {
-        console.log(stats.message);
-        console.log('\nTelemetry is disabled in your configuration.');
-        console.log('To re-enable (default), remove or set to true in .claude/config.json:');
-        console.log('{');
-        console.log('  "system": {');
-        console.log('    "telemetry": true');
-        console.log('  }');
-        console.log('}\n');
-        return;
-    }
-
     if (stats.error) {
         console.log(`Error: ${stats.error}`);
         console.log(`Details: ${stats.details}\n`);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-git-hooks",
-    "version": "2.30.2",
+    "version": "2.33.0",
     "description": "Git hooks with Claude CLI for code analysis and automatic commit messages",
     "type": "module",
     "bin": {