pumuki-ast-hooks 5.3.20 → 5.3.22

package/scripts/hooks-system/application/services/monitoring/GitTreeMonitor.js

@@ -2,18 +2,8 @@ const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class GitTreeMonitor {
     constructor(repoRoot, options = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'git_tree_monitor',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.stagedThreshold = options.stagedThreshold || 10;
         this.unstagedThreshold = options.unstagedThreshold || 15;
@@ -27,7 +17,6 @@ class GitTreeMonitor {
         this.timer = null;
         this.lastState = null;
         this.loadState();
-        m_constructor.success();
     }
 
     loadState() {
@@ -50,12 +39,6 @@ class GitTreeMonitor {
     }
 
     getTreeState() {
-        const m_get_tree_state = createMetricScope({
-            hook: 'git_tree_monitor',
-            operation: 'get_tree_state'
-        });
-
-        m_get_tree_state.started();
         try {
             const stagedRaw = execSync('git diff --cached --name-only', {
                 cwd: this.repoRoot,
@@ -75,8 +58,6 @@ class GitTreeMonitor {
             const untracked = untrackedRaw ? untrackedRaw.split('\n').length : 0;
             const total = staged + unstaged + untracked;
 
-            m_get_tree_state.success();
-
             return {
                 staged,
                 unstaged,
@@ -88,7 +69,6 @@ class GitTreeMonitor {
                 timestamp: Date.now()
             };
         } catch (error) {
-            m_get_tree_state.success();
             return {
                 staged: 0,
                 unstaged: 0,
@@ -99,7 +79,6 @@ class GitTreeMonitor {
                 timestamp: Date.now()
             };
         }
-        m_get_tree_state.success();
     }
 
     startMonitoring(onStateChange) {
@@ -130,17 +109,10 @@ class GitTreeMonitor {
     }
 
     stop() {
-        const m_stop = createMetricScope({
-            hook: 'git_tree_monitor',
-            operation: 'stop'
-        });
-
-        m_stop.started();
         if (this.timer) {
             clearInterval(this.timer);
             this.timer = null;
         }
-        m_stop.success();
     }
 
     isActive() {

package/scripts/hooks-system/application/services/monitoring/GitTreeMonitorService.js

@@ -1,7 +1,3 @@
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class GitTreeMonitorService {
     constructor({
         repoRoot = process.cwd(),
@@ -14,12 +10,6 @@ class GitTreeMonitorService {
         logger = console,
         debugLogger = null
     } = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'git_tree_monitor_service',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.limit = limit;
         this.warning = warning;
@@ -33,18 +23,10 @@ class GitTreeMonitorService {
         this.lastCritical = 0;
         this.lastWarning = 0;
         this.wasOverLimit = false;
-        m_constructor.success();
     }
 
     start() {
-        const m_start = createMetricScope({
-            hook: 'git_tree_monitor_service',
-            operation: 'start'
-        });
-
-        m_start.started();
         if (!Number.isFinite(this.limit) || this.limit <= 0 || typeof this.getState !== 'function') {
-            m_start.success();
             return;
         }
         this.check('startup');
@@ -54,21 +36,13 @@ class GitTreeMonitorService {
                 this.timer.unref();
             }
         }
-        m_start.success();
     }
 
     stop() {
-        const m_stop = createMetricScope({
-            hook: 'git_tree_monitor_service',
-            operation: 'stop'
-        });
-
-        m_stop.started();
         if (this.timer) {
             clearInterval(this.timer);
             this.timer = null;
         }
-        m_stop.success();
     }
 
     check(reason) {

package/scripts/hooks-system/application/services/monitoring/HealthCheckProviders.js

@@ -1,10 +1,6 @@
 const fs = require('fs');
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 const defaultHeartbeatPath = repoRoot => path.join(repoRoot, '.audit_tmp', 'guard-heartbeat.json');
 const defaultTokenUsagePath = repoRoot => path.join(repoRoot, '.audit_tmp', 'token-usage.jsonl');
 const defaultEvidencePath = repoRoot => path.join(repoRoot, '.AI_EVIDENCE.json');

package/scripts/hooks-system/application/services/monitoring/HealthCheckService.js

@@ -1,10 +1,6 @@
 const fs = require('fs');
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class HealthCheckService {
     constructor({
         repoRoot = process.cwd(),
@@ -16,12 +12,6 @@ class HealthCheckService {
         intervalMs = 0,
         timers = { setInterval, clearInterval }
     } = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'health_check_service',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.providers = Array.isArray(providers) ? providers : [];
         this.notificationCenter = notificationCenter;
@@ -32,16 +22,9 @@ class HealthCheckService {
         this.timers = timers;
         this.history = [];
         this.timerRef = null;
-        m_constructor.success();
     }
 
     start(reason = 'startup') {
-        const m_start = createMetricScope({
-            hook: 'health_check_service',
-            operation: 'start'
-        });
-
-        m_start.started();
         this.collect(reason);
         if (this.intervalMs > 0) {
             this.timerRef = this.timers.setInterval(() => this.collect('interval'), this.intervalMs);
@@ -49,21 +32,13 @@ class HealthCheckService {
                 this.timerRef.unref();
             }
         }
-        m_start.success();
     }
 
     stop() {
-        const m_stop = createMetricScope({
-            hook: 'health_check_service',
-            operation: 'stop'
-        });
-
-        m_stop.started();
         if (this.timerRef) {
             this.timers.clearInterval(this.timerRef);
             this.timerRef = null;
         }
-        m_stop.success();
     }
 
     async collect(reason = 'manual') {

package/scripts/hooks-system/application/services/monitoring/HeartbeatMonitorService.js

@@ -1,10 +1,6 @@
 const fs = require('fs');
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class HeartbeatMonitorService {
     constructor({
         repoRoot = process.cwd(),
@@ -13,12 +9,6 @@ class HeartbeatMonitorService {
         statusProvider = () => null,
         logger = console
     } = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'heartbeat_monitor_service',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.heartbeatPath = heartbeatPath;
         this.intervalMs = intervalMs;
@@ -26,18 +16,10 @@ class HeartbeatMonitorService {
         this.logger = logger;
         this.timer = null;
         this.lastStatus = null;
-        m_constructor.success();
     }
 
     start() {
-        const m_start = createMetricScope({
-            hook: 'heartbeat_monitor_service',
-            operation: 'start'
-        });
-
-        m_start.started();
         if (!Number.isFinite(this.intervalMs) || this.intervalMs <= 0) {
-            m_start.success();
             return;
         }
         this.emitHeartbeat();
@@ -45,21 +27,13 @@ class HeartbeatMonitorService {
         if (this.timer && typeof this.timer.unref === 'function') {
             this.timer.unref();
         }
-        m_start.success();
     }
 
     stop() {
-        const m_stop = createMetricScope({
-            hook: 'heartbeat_monitor_service',
-            operation: 'stop'
-        });
-
-        m_stop.started();
         if (this.timer) {
             clearInterval(this.timer);
             this.timer = null;
         }
-        m_stop.success();
     }
 
     emitHeartbeat() {

package/scripts/hooks-system/application/services/monitoring/TokenMonitor.js

@@ -3,22 +3,11 @@ const path = require('path');
 const { spawn, spawnSync } = require('child_process');
 const { ConfigurationError } = require('../../../domain/errors');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class TokenMonitor {
     constructor(repoRoot, options = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'token_monitor',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.scriptPath = path.join(repoRoot, 'infrastructure', 'watchdog', 'token-monitor-loop.sh');
         this.process = null;
-        m_constructor.success();
     }
 
     isAvailable() {
@@ -26,12 +15,6 @@ class TokenMonitor {
     }
 
     start() {
-        const m_start = createMetricScope({
-            hook: 'token_monitor',
-            operation: 'start'
-        });
-
-        m_start.started();
         if (!this.isAvailable()) {
             throw new ConfigurationError('Token monitor script not found', 'scriptPath');
         }
@@ -59,23 +42,14 @@ class TokenMonitor {
             this.process = null;
         });
 
-        m_start.success();
-
         return this.process;
     }
 
     stop() {
-        const m_stop = createMetricScope({
-            hook: 'token_monitor',
-            operation: 'stop'
-        });
-
-        m_stop.started();
         if (this.process && this.process.kill) {
             this.process.kill('SIGTERM');
             this.process = null;
         }
-        m_stop.success();
     }
 
     isRunning() {

package/scripts/hooks-system/application/services/notification/MacNotificationSender.js

@@ -1,22 +1,11 @@
 const { spawnSync } = require('child_process');
 const fs = require('fs');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class MacNotificationSender {
     constructor(logger) {
-        const m_constructor = createMetricScope({
-            hook: 'mac_notification_sender',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.logger = logger;
         this.terminalNotifierPath = this.resolveTerminalNotifier();
         this.osascriptPath = this.resolveOsascript();
-        m_constructor.success();
     }
 
     resolveTerminalNotifier() {

package/scripts/hooks-system/application/services/notification/NotificationCenterService.js

@@ -21,18 +21,8 @@ const NotificationQueue = require('./components/NotificationQueue');
 const NotificationRetryExecutor = require('./components/NotificationRetryExecutor');
 const NotificationDispatcher = require('./NotificationDispatcher');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationCenterService {
     constructor(config = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_center_service',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = config.repoRoot || process.cwd();
         this.enabled = config.enabled !== false;
 
@@ -93,7 +83,6 @@ class NotificationCenterService {
             totalFailed: 0,
             totalRetries: 0
         };
-        m_constructor.success();
     }
 
     /**
@@ -184,13 +173,6 @@ class NotificationCenterService {
      * Gets service statistics
      */
     getStats() {
-        const m_get_stats = createMetricScope({
-            hook: 'notification_center_service',
-            operation: 'get_stats'
-        });
-
-        m_get_stats.started();
-        m_get_stats.success();
         return {
             ...this.stats,
             queueSize: this.queue.size(),

package/scripts/hooks-system/application/services/notification/NotificationDispatcher.js

@@ -1,20 +1,9 @@
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationDispatcher {
     constructor(sender, retryExecutor, cooldownManager, logger) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_dispatcher',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.sender = sender;
         this.retryExecutor = retryExecutor;
         this.cooldownManager = cooldownManager;
         this.logger = logger;
-        m_constructor.success();
     }
 
     async dispatch(notification, stats) {

package/scripts/hooks-system/application/services/notification/components/NotificationCooldownManager.js

@@ -1,20 +1,9 @@
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationCooldownManager {
     constructor(defaultCooldownMs = 60000, cooldownsByType = {}, logger = null) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_cooldown_manager',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.cooldowns = new Map();
         this.defaultCooldownMs = defaultCooldownMs;
         this.cooldownsByType = cooldownsByType;
         this.logger = logger;
-        m_constructor.success();
     }
 
     isInCooldown(notification) {
@@ -60,13 +49,6 @@ class NotificationCooldownManager {
     }
 
     getStats() {
-        const m_get_stats = createMetricScope({
-            hook: 'notification_cooldown_manager',
-            operation: 'get_stats'
-        });
-
-        m_get_stats.started();
-        m_get_stats.success();
         return {
             activeCooldowns: this.cooldowns.size
         };

package/scripts/hooks-system/application/services/notification/components/NotificationDeduplicator.js

@@ -1,21 +1,10 @@
 const crypto = require('crypto');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationDeduplicator {
     constructor(windowMs = 5000, logger = null) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_deduplicator',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.deduplicationMap = new Map();
         this.windowMs = windowMs;
         this.logger = logger;
-        m_constructor.success();
     }
 
     isDuplicate(notification) {
@@ -65,13 +54,6 @@ class NotificationDeduplicator {
     }
 
     getStats() {
-        const m_get_stats = createMetricScope({
-            hook: 'notification_deduplicator',
-            operation: 'get_stats'
-        });
-
-        m_get_stats.started();
-        m_get_stats.success();
         return {
             size: this.deduplicationMap.size
         };

package/scripts/hooks-system/application/services/notification/components/NotificationQueue.js

@@ -1,18 +1,7 @@
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationQueue {
     constructor(maxSize = 100) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_queue',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.maxSize = maxSize;
         this.items = [];
-        m_constructor.success();
     }
 
     enqueue(item) {

package/scripts/hooks-system/application/services/notification/components/NotificationRetryExecutor.js

@@ -1,29 +1,12 @@
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class NotificationRetryExecutor {
     constructor(sender, config = {}, logger = null) {
-        const m_constructor = createMetricScope({
-            hook: 'notification_retry_executor',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.sender = sender;
         this.maxRetries = config.maxRetries || 2;
         this.retryDelayMs = config.retryDelayMs || 1000;
         this.logger = logger;
-        m_constructor.success();
     }
 
     async execute(notification, options = {}) {
-        const m_execute = createMetricScope({
-            hook: 'notification_retry_executor',
-            operation: 'execute'
-        });
-
-        m_execute.started();
         const currentRetries = notification.retries || 0;
         const remainingRetries = currentRetries < this.maxRetries ? this.maxRetries - currentRetries : 0;
 
@@ -38,7 +21,6 @@ class NotificationRetryExecutor {
                     if (attempt > 0) {
                         this.logRetrySuccess(notification, attempt);
                     }
-                    m_execute.success();
                     return { success: true, attempts: attempt + 1 };
                 }
             } catch (error) {
@@ -46,8 +28,6 @@ class NotificationRetryExecutor {
             }
         }
 
-        m_execute.success();
-
         return { success: false, attempts: remainingRetries + 1 };
     }
 

package/scripts/hooks-system/application/services/platform/PlatformHeuristics.js

@@ -1,20 +1,9 @@
 const fs = require('fs');
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class PlatformHeuristics {
     constructor(platformDetector) {
-        const m_constructor = createMetricScope({
-            hook: 'platform_heuristics',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.platformDetector = platformDetector;
-        m_constructor.success();
     }
 
     detectFromASTSystemFiles(files) {
@@ -140,12 +129,6 @@ class PlatformHeuristics {
     }
 
     getPlatformFrequencyInHistory(platform, commits) {
-        const m_get_platform_frequency_in_history = createMetricScope({
-            hook: 'platform_heuristics',
-            operation: 'get_platform_frequency_in_history'
-        });
-
-        m_get_platform_frequency_in_history.started();
         if (!commits || commits.length === 0) return 0;
 
         const platformCommits = commits.filter(commit =>
@@ -154,8 +137,6 @@ class PlatformHeuristics {
             )
         );
 
-        m_get_platform_frequency_in_history.success();
-
         return platformCommits.length / commits.length;
     }
 }

package/scripts/hooks-system/application/services/recovery/AutoRecoveryManager.js

@@ -1,9 +1,5 @@
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class AutoRecoveryManager {
     constructor({
         repoRoot = process.cwd(),
@@ -15,12 +11,6 @@ class AutoRecoveryManager {
         baseBackoffMs = 2000,
         jitter = 0.25
     } = {}) {
-        const m_constructor = createMetricScope({
-            hook: 'auto_recovery_manager',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.repoRoot = repoRoot;
         this.logger = logger || console;
         this.notificationCenter = notificationCenter;
@@ -33,23 +23,14 @@ class AutoRecoveryManager {
             : [AutoRecoveryManager.createSupervisorRestartStrategy()];
         this.attempts = new Map();
         this.timeouts = new Map();
-        m_constructor.success();
     }
 
     static createSupervisorRestartStrategy() {
-        const m_create_supervisor_restart_strategy = createMetricScope({
-            hook: 'auto_recovery_manager',
-            operation: 'create_supervisor_restart_strategy'
-        });
-
-        m_create_supervisor_restart_strategy.started();
-        m_create_supervisor_restart_strategy.success();
         return {
             id: 'guard-supervisor-restart',
             condition: ({ reason }) => reason && reason.startsWith('heartbeat-'),
             action: async ({ logger }) => {
                 logger.info('Attempting guard-supervisor restart via start-guards.sh');
-                m_create_supervisor_restart_strategy.success();
                 return AutoRecoveryManager.runScript('start-guards.sh', ['restart']);
             }
         };

package/scripts/hooks-system/application/services/smart-commit/CommitMessageSuggester.js

@@ -1,17 +1,7 @@
 const path = require('path');
 
-const {
-    createMetricScope: createMetricScope
-} = require('../../../infrastructure/telemetry/metric-scope');
-
 class CommitMessageSuggester {
     constructor(featureDetector) {
-        const m_constructor = createMetricScope({
-            hook: 'commit_message_suggester',
-            operation: 'constructor'
-        });
-
-        m_constructor.started();
         this.featureDetector = featureDetector;
         this.commitTypePatterns = {
             feat: ['feature/', 'feat/', 'add', 'new', 'create', 'implement'],
@@ -22,7 +12,6 @@ class CommitMessageSuggester {
             chore: ['chore/', 'config/', 'build/', 'ci/'],
             style: ['style/', 'css/', 'scss/', 'styling']
         };
-        m_constructor.success();
     }
 
     suggest(group) {