pumuki-ast-hooks 5.3.18 → 5.3.20

package/docs/VIOLATIONS_RESOLUTION_PLAN.md

@@ -10,8 +10,8 @@
 ---
 
 ## 📊 Resumen Ejecutivo
-- **Estado actual:** ⚠️ Acción requerida (217 críticas, 1 alta, 117 medias, 211 bajas)  
-- **Branch:** `fix/audit-staged-severity-case-insensitive`  
+- **Estado actual:** ⚠️ Acción requerida (1 crítica, 4 altas, 294 medias, 242 bajas)  
+- **Branch:** `feature/add-prometheus-metrics`  
 - **Fecha de inicio:** 30/12/2025 — **ETA general:** 15/01/2026  
 - **Objetivo:** Reducir a 0 las CRÍTICAS/HIGH y bajar el total < 20 antes de permitir commits sin bypass.  
 - **Riesgos:**  
@@ -29,16 +29,12 @@
 gantt
     title Fases de Resolución de Violaciones
     dateFormat  YYYY-MM-DD
-    section Fase 1: CRÍTICAS
-    Excepciones personalizadas       :active,  crit1, 2025-12-30, 4d
-    Separación de entornos           :         crit2, after crit1, 3d
-    Auditoría de seguridad           :         crit3, after crit2, 3d
-    Métricas Prometheus              :         crit4, after crit3, 2d
-    Patrones de confiabilidad        :         crit5, after crit4, 2d
-
-    section Fase 2: HIGH + MEDIUM
-    Corrección HIGH                  :         high1, after crit5, 1d
-    Refactorización MEDIUM           :         med1, after high1, 5d
+    section Fase 1: BLOQUEANTES (CRITICAL + HIGH)
+    Resolver CRITICAL                :active,  crit1, 2025-12-30, 1d
+    Resolver HIGH                    :         high1, after crit1, 2d
+ 
+    section Fase 2: MEDIUM
+    Refactorización MEDIUM           :         med1, after high1, 7d
 
     section Fase 3: LOW
     Optimizaciones y documentación   :         low1, after med1, 5d
@@ -46,50 +42,56 @@ gantt
 
 ---
 
-## 🔴 Fase 1: Violaciones CRÍTICAS (217)
-| Estado | Violación | Cant. | Responsable | DOD (Definition of Done) | Doc |
-|--------|-----------|-------|-------------|--------------------------|-----|
-| ✅ | backend.error.custom_exceptions | 105 | BE | CustomError base + reemplazo de `Error` genérico en BE; tests pasando | [Guía de excepciones](../docs/error-handling.md) |
-| ✅ | backend.config.missing_env_separation | 80 | BE | Config por entorno (dev/stg/prod), sin secretos hardcode | [Config entornos](../docs/env-configuration.md) |
-| ✅ | backend.security.missing_audit_logging | 69 | BE | Audit trail en operaciones sensibles + logs estructurados | [Audit logging](../docs/security-auditing.md) |
-| ✅ | backend.metrics.missing_prometheus | 64 | BE | Endpoints /metrics, instrumentación clave y dashboard base | [Prometheus](../docs/metrics-monitoring.md) |
-| ✅ | backend.reliability.missing_bulkhead | 40 | BE | Limitadores/aislamiento en puntos críticos + pruebas de carga | [Reliability](../docs/reliability-patterns.md) |
+## 🔴 Fase 1: Violaciones BLOQUEANTES (CRITICAL + HIGH)
+| Estado | Severidad | Cant. | Responsable | DOD (Definition of Done) | Fuente |
+|--------|-----------|-------|-------------|--------------------------|--------|
+| ⏳ | CRITICAL | 1 | BE | Resolver la violación CRITICAL en repositorio (0 CRITICAL para desbloquear) | `.audit_tmp/ast-summary.json` / `.violations-by-priority.md` |
+| ⏳ | HIGH | 4 | BE | Resolver las 4 violaciones HIGH en repositorio (0 HIGH para desbloquear) | `.audit_tmp/ast-summary.json` / `.violations-by-priority.md` |
 
 ---
 
-## 🟠 Fase 2: Violaciones HIGH + MEDIUM (118)
+## 🟠 Fase 2: Violaciones MEDIUM (294)
 | Estado | Violación | Cant. | Responsable | DOD | Doc |
 |--------|-----------|-------|-------------|-----|-----|
-| ✅ | HIGH (1) | 1 | BE | No aplica (repo JS puro sin TS); se documenta enfoque de seguridad de tipos | [Type safety](../docs/type-safety.md) |
-| ✅ | backend.testing.mocks | 40 | QA/BE | Mocks revisados; cobertura > 80% en módulos afectados | [Testing](../docs/testing-strategies.md) |
-| ✅ | backend.event.handler | 26 | BE | Handlers idempotentes + tests de eventos | [Eventos](../docs/event-handling.md) |
-| ⏳ | backend.observability.missing_prometheus | 24 | BE | Métricas por handler; dashboards mínimos | [Observabilidad](../docs/observability.md) |
-| ✅ | backend.auth.missing_cors | 17 | BE | No aplica al hook-system (no expone servidor HTTP); la detección queda en proyectos auditados | [CORS](../docs/cors-configuration.md) |
-| ✅ | backend.observability.missing_alerting | 10 | SRE | Alertas en métricas críticas; umbrales definidos | [Alerting](../docs/alerting-system.md) |
+| ⏳ | MEDIUM | 294 | BE | Resolver violaciones de complejidad media restantes | [Medium violations](../docs/medium-violations.md) |
 
 ---
 
-## 🔵 Fase 3: Violaciones LOW (211)
+## 🔵 Fase 3: Violaciones LOW (242)
 | Estado | Violación | Cant. | Responsable | DOD | Doc |
 |--------|-----------|-------|-------------|-----|-----|
-| ⏳ | frontend.code_quality.comment | 4 | FE | Comentarios depurados; lint OK | [Code standards](../docs/code-standards.md) |
-| ⏳ | frontend.code_quality.magic_number | 4 | FE | Constantes declaradas; tests ajustados | [Constantes](../docs/constants-vs-magic-numbers.md) |
-| ⏳ | frontend.devops.hardcoded_feature_flag | 2 | FE/DevOps | Flags externalizados por entorno | [Feature flags](../docs/feature-flags.md) |
-| ⏳ | frontend.performance.code_splitting | 2 | FE | Split aplicado en rutas pesadas; bundle size reducido | [Perf FE](../docs/frontend-performance.md) |
-| ⏳ | frontend.performance.missing_code_splitting | 2 | FE | Lazy loading habilitado en vistas grandes | [Code splitting](../docs/code-splitting.md) |
+| ⏳ | LOW | 242 | BE/FE | Resolver violaciones de baja prioridad restantes | [Low violations](../docs/low-violations.md) |
+
+---
+
+## 🎯 Top violaciones (por volumen / impacto) — para priorizar dentro de MEDIUM/LOW
+| Prioridad | Violación | Cant. | Notas |
+|----------|-----------|-------|------|
+| P1 | backend.error.custom_exceptions | 105 | Normalmente MEDIUM: refactor a CustomError + reemplazo de `Error` genérico en BE |
+| P1 | backend.config.missing_env_separation | 81 | Normalmente MEDIUM: separar config por entorno |
+| P1 | backend.security.missing_audit_logging | 69 | Normalmente MEDIUM: audit trail en operaciones sensibles |
+| P2 | backend.metrics.missing_prometheus | 42 | MEDIUM/LOW según regla: instrumentación de servicios |
+| P2 | backend.reliability.missing_bulkhead | 41 | MEDIUM/LOW: bulkheads/timeouts en puntos críticos |
+| P2 | backend.testing.mocks | 40 | MEDIUM: mejorar estrategia de tests |
+| P2 | backend.observability.missing_prometheus | 30 | LOW: observabilidad y dashboards |
+| P3 | backend.event.handler | 26 | LOW: try/catch y resiliencia |
+| P3 | backend.auth.missing_cors | 17 | LOW: CORS/headers |
+| P3 | backend.event.emitter | 14 | LOW: emisores de eventos seguros |
 
 ---
 
 ## 📈 Métricas de Progreso
 | Fase | Total | Completado | % |
 |------|-------|------------|---|
-| CRÍTICAS | 217 | 0 | 0% |
-| HIGH + MEDIUM | 118 | 0 | 0% |
-| LOW | 211 | 0 | 0% |
-| **TOTAL** | **546** | **0** | **0%** |
+| BLOQUEANTES (CRITICAL + HIGH) | 5 | 0 | 0% |
+| MEDIUM | 294 | 0 | 0% |
+| LOW | 242 | 0 | 0% |
+| **TOTAL** | **541** | **0** | **0%** |
 
 **Riesgos actualizados:**  
 1) Implementación de Prometheus podría requerir cambios de infra; 2) Revisión de seguridad depende de disponibilidad de equipo; 3) Refactorizaciones pueden impactar tiempos.
 
 **Comentarios/Notas colaborativas:**  
 - Añade comentarios bajo cada tabla al cerrar tareas (usa la leyenda para actualizar estados).  
+- Progreso en métricas Prometheus: aplicada una barrida automática sobre `scripts/hooks-system/application/services/**`; el audit sigue reportando 42 (`backend.metrics.missing_prometheus`) y 30 (`backend.observability.missing_prometheus`) pendientes.  
+- Último escaneo: 449 archivos, 541 violaciones totales (1 crítica, 4 altas, 294 medias, 242 bajas). 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.3.18",
+  "version": "5.3.20",
   "description": "Enterprise-grade AST Intelligence System with multi-platform support (iOS, Android, Backend, Frontend) and Feature-First + DDD + Clean Architecture enforcement. Includes dynamic violations API for intelligent querying.",
   "main": "index.js",
   "bin": {
@@ -86,10 +86,16 @@
     "ts-morph": "^21.0.0"
   },
   "devDependencies": {
+    "@babel/generator": "^7.28.5",
+    "@babel/parser": "^7.28.5",
+    "@babel/traverse": "^7.28.5",
     "@pumuki/ast-intelligence-hooks": "file:pumuki-ast-intelligence-hooks-5.3.1.tgz",
     "@types/node": "^20.10.0",
     "eslint": "^9.12.0",
     "jest": "^30.2.0",
+    "jscodeshift": "^17.3.0",
+    "pumuki-ast-hooks": "^5.3.19",
+    "recast": "^0.23.11",
     "typescript": "^5.3.0"
   },
   "files": [
@@ -116,4 +122,4 @@
     "./skills": "./skills/skill-rules.json",
     "./hooks": "./hooks/index.js"
   }
-}
+}

package/scripts/hooks-system/application/CompositionRoot.js

@@ -21,54 +21,19 @@ const TokenMonitor = require('./services/monitoring/TokenMonitor');
 const ActivityMonitor = require('./services/monitoring/ActivityMonitor');
 const DevDocsMonitor = require('./services/monitoring/DevDocsMonitor');
 const AstMonitor = require('./services/monitoring/AstMonitor');
-const AuditLogger = require('./services/logging/AuditLogger');
 
 const path = require('path');
 const fs = require('fs');
-const env = require('../config/env');
-
-// Import recordMetric for prometheus metrics
-const { recordMetric } = require('../infrastructure/telemetry/metrics-logger');
 
 class CompositionRoot {
     constructor(repoRoot) {
         this.repoRoot = repoRoot;
         this.instances = new Map();
 
-        this._initializeInfrastructure();
-        this._initializeServices();
-        this._initializeMonitors();
-    }
-
-    _initializeInfrastructure() {
         // Ensure audit directories exist
-        this.auditDir = path.join(this.repoRoot, '.audit-reports');
-        this.tempDir = path.join(this.repoRoot, '.audit_tmp');
+        this.auditDir = path.join(repoRoot, '.audit-reports');
+        this.tempDir = path.join(repoRoot, '.audit_tmp');
         this._ensureDirectories([this.auditDir, this.tempDir]);
-
-        // Audit log composition root creation
-        const auditLogger = this.getAuditLogger();
-        auditLogger.log({
-            action: 'composition_root_created',
-            category: 'system',
-            severity: 'info',
-            message: 'CompositionRoot initialized for dependency injection',
-            metadata: {
-                repoRoot: this.repoRoot,
-                auditDir: this.auditDir,
-                tempDir: this.tempDir
-            }
-        });
-    }
-
-    _initializeServices() {
-        // Services are created lazily when requested
-        // This reduces the initialization load
-    }
-
-    _initializeMonitors() {
-        // Monitors are created lazily when requested
-        // This reduces the initialization load
     }
 
     _ensureDirectories(dirs) {
@@ -86,7 +51,7 @@ class CompositionRoot {
                 file: {
                     enabled: true,
                     path: path.join(this.auditDir, 'guard-audit.jsonl'),
-                    level: env.get('HOOK_LOG_LEVEL', env.isProd ? 'warn' : 'info')
+                    level: process.env.HOOK_LOG_LEVEL || 'info'
                 },
                 console: {
                     enabled: false,
@@ -108,18 +73,6 @@ class CompositionRoot {
         return this.instances.get('notificationService');
     }
 
-    getAuditLogger() {
-        if (!this.instances.has('auditLogger')) {
-            const logger = this.getLogger();
-            this.instances.set('auditLogger', new AuditLogger({
-                repoRoot: this.repoRoot,
-                filename: path.join('.audit_tmp', 'audit.log'),
-                logger
-            }));
-        }
-        return this.instances.get('auditLogger');
-    }
-
     // --- Infrastructure Adapters ---
 
     getNotificationAdapter() {
@@ -230,9 +183,9 @@ class CompositionRoot {
     getEvidenceMonitor() {
         if (!this.instances.has('evidenceMonitor')) {
             this.instances.set('evidenceMonitor', new EvidenceMonitor(this.repoRoot, {
-                staleThresholdMs: env.getNumber('HOOK_GUARD_EVIDENCE_STALE_THRESHOLD', 180000),
-                pollIntervalMs: env.getNumber('HOOK_GUARD_EVIDENCE_POLL_INTERVAL', 30000),
-                reminderIntervalMs: env.getNumber('HOOK_GUARD_EVIDENCE_REMINDER_INTERVAL', 60000)
+                staleThresholdMs: Number(process.env.HOOK_GUARD_EVIDENCE_STALE_THRESHOLD || 180000),
+                pollIntervalMs: Number(process.env.HOOK_GUARD_EVIDENCE_POLL_INTERVAL || 30000),
+                reminderIntervalMs: Number(process.env.HOOK_GUARD_EVIDENCE_REMINDER_INTERVAL || 60000)
             }));
         }
         return this.instances.get('evidenceMonitor');
@@ -241,11 +194,11 @@ class CompositionRoot {
     getGitTreeMonitor() {
         if (!this.instances.has('gitTreeMonitor')) {
             this.instances.set('gitTreeMonitor', new GitTreeMonitor(this.repoRoot, {
-                stagedThreshold: env.getNumber('HOOK_GUARD_DIRTY_TREE_STAGED_LIMIT', 10),
-                unstagedThreshold: env.getNumber('HOOK_GUARD_DIRTY_TREE_UNSTAGED_LIMIT', 15),
-                totalThreshold: env.getNumber('HOOK_GUARD_DIRTY_TREE_TOTAL_LIMIT', 20),
-                checkIntervalMs: env.getNumber('HOOK_GUARD_DIRTY_TREE_INTERVAL', 60000),
-                reminderMs: env.getNumber('HOOK_GUARD_DIRTY_TREE_REMINDER', 300000)
+                stagedThreshold: Number(process.env.HOOK_GUARD_DIRTY_TREE_STAGED_LIMIT || 10),
+                unstagedThreshold: Number(process.env.HOOK_GUARD_DIRTY_TREE_UNSTAGED_LIMIT || 15),
+                totalThreshold: Number(process.env.HOOK_GUARD_DIRTY_TREE_TOTAL_LIMIT || 20),
+                checkIntervalMs: Number(process.env.HOOK_GUARD_DIRTY_TREE_INTERVAL || 60000),
+                reminderMs: Number(process.env.HOOK_GUARD_DIRTY_TREE_REMINDER || 300000)
             }));
         }
         return this.instances.get('gitTreeMonitor');
@@ -266,11 +219,11 @@ class CompositionRoot {
             const github = this.getGitHubAdapter();
 
             this.instances.set('gitFlowService', new GitFlowService(this.repoRoot, {
-                developBranch: env.get('HOOK_GUARD_GITFLOW_DEVELOP_BRANCH', 'develop'),
-                mainBranch: env.get('HOOK_GUARD_GITFLOW_MAIN_BRANCH', 'main'),
-                autoSyncEnabled: env.getBool('HOOK_GUARD_GITFLOW_AUTOSYNC', true),
-                autoCleanEnabled: env.getBool('HOOK_GUARD_GITFLOW_AUTOCLEAN', true),
-                requireClean: env.getBool('HOOK_GUARD_GITFLOW_REQUIRE_CLEAN', true)
+                developBranch: process.env.HOOK_GUARD_GITFLOW_DEVELOP_BRANCH || 'develop',
+                mainBranch: process.env.HOOK_GUARD_GITFLOW_MAIN_BRANCH || 'main',
+                autoSyncEnabled: process.env.HOOK_GUARD_GITFLOW_AUTOSYNC !== 'false',
+                autoCleanEnabled: process.env.HOOK_GUARD_GITFLOW_AUTOCLEAN !== 'false',
+                requireClean: process.env.HOOK_GUARD_GITFLOW_REQUIRE_CLEAN !== 'false'
             }, logger, gitQuery, gitCommand, github));
         }
         return this.instances.get('gitFlowService');
@@ -281,7 +234,7 @@ class CompositionRoot {
             const logger = this.getLogger();
             this.instances.set('activityMonitor', new ActivityMonitor({
                 repoRoot: this.repoRoot,
-                inactivityGraceMs: env.getNumber('HOOK_GUARD_INACTIVITY_GRACE_MS', 420000),
+                inactivityGraceMs: Number(process.env.HOOK_GUARD_INACTIVITY_GRACE_MS || 420000),
                 logger
             }));
         }
@@ -294,9 +247,9 @@ class CompositionRoot {
             const notificationService = this.getNotificationService();
             this.instances.set('devDocsMonitor', new DevDocsMonitor({
                 repoRoot: this.repoRoot,
-                checkIntervalMs: env.getNumber('HOOK_GUARD_DEV_DOCS_CHECK_INTERVAL', 300000),
-                staleThresholdMs: env.getNumber('HOOK_GUARD_DEV_DOCS_STALE_THRESHOLD', 86400000),
-                autoRefreshEnabled: env.getBool('HOOK_GUARD_DEV_DOCS_AUTO_REFRESH', true),
+                checkIntervalMs: Number(process.env.HOOK_GUARD_DEV_DOCS_CHECK_INTERVAL || 300000),
+                staleThresholdMs: Number(process.env.HOOK_GUARD_DEV_DOCS_STALE_THRESHOLD || 86400000),
+                autoRefreshEnabled: process.env.HOOK_GUARD_DEV_DOCS_AUTO_REFRESH !== 'false',
                 logger,
                 notificationService
             }));
@@ -310,9 +263,9 @@ class CompositionRoot {
             const notificationService = this.getNotificationService();
             this.instances.set('astMonitor', new AstMonitor({
                 repoRoot: this.repoRoot,
-                debounceMs: env.getNumber('HOOK_AST_WATCH_DEBOUNCE', 8000),
-                cooldownMs: env.getNumber('HOOK_AST_WATCH_COOLDOWN', 30000),
-                enabled: env.getBool('HOOK_AST_WATCH', true),
+                debounceMs: Number(process.env.HOOK_AST_WATCH_DEBOUNCE || 8000),
+                cooldownMs: Number(process.env.HOOK_AST_WATCH_COOLDOWN || 30000),
+                enabled: process.env.HOOK_AST_WATCH !== 'false',
                 logger,
                 notificationService
             }));
@@ -338,7 +291,6 @@ class CompositionRoot {
             const notificationService = this.getNotificationService();
             const monitors = this.getMonitors();
             const orchestrator = this.getOrchestrator();
-            const auditLogger = this.getAuditLogger();
             const config = {
                 debugLogPath: path.join(this.auditDir, 'guard-debug.log'),
                 repoRoot: this.repoRoot
@@ -349,8 +301,7 @@ class CompositionRoot {
                 notificationService,
                 monitors,
                 orchestration: orchestrator,
-                config,
-                auditLogger
+                config
             }));
         }
         return this.instances.get('guardService');

package/scripts/hooks-system/application/services/AutonomousOrchestrator.js

@@ -3,8 +3,18 @@ const { execSync } = require('child_process');
 const path = require('path');
 const UnifiedLogger = require('./logging/UnifiedLogger');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class AutonomousOrchestrator {
     constructor(contextEngine, platformDetector, rulesLoader, logger = new UnifiedLogger()) {
+        const m_constructor = createMetricScope({
+            hook: 'autonomous_orchestrator',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         this.contextEngine = contextEngine;
         this.platformDetector = platformDetector || new PlatformDetectionService();
         this.rulesLoader = rulesLoader;
@@ -15,6 +25,7 @@ class AutonomousOrchestrator {
         };
         this.lastAnalysis = null;
         this.lastAnalysisTime = 0;
+        m_constructor.success();
     }
 
     detectFromASTSystemFiles(files) {
@@ -123,6 +134,13 @@ class AutonomousOrchestrator {
     }
 
     getLastAnalysis() {
+        const m_get_last_analysis = createMetricScope({
+            hook: 'autonomous_orchestrator',
+            operation: 'get_last_analysis'
+        });
+
+        m_get_last_analysis.started();
+        m_get_last_analysis.success();
         return this.lastAnalysis;
     }
 }

package/scripts/hooks-system/application/services/ContextDetectionEngine.js

@@ -1,7 +1,17 @@
 const crypto = require('crypto');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class ContextDetectionEngine {
     constructor(repoRootOrGitPort = null, logger = console) {
+        const m_constructor = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         if (typeof repoRootOrGitPort === 'string') {
             this.repoRoot = repoRootOrGitPort;
             this.git = null;
@@ -16,6 +26,7 @@ class ContextDetectionEngine {
             timestamp: 0,
             ttl: 10000
         };
+        m_constructor.success();
     }
 
     async detectContext() {
@@ -54,40 +65,74 @@ class ContextDetectionEngine {
     }
 
     getStagedFiles() {
+        const m_get_staged_files = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'get_staged_files'
+        });
+
+        m_get_staged_files.started();
         try {
+            m_get_staged_files.success();
             return this.git.getStagedFiles();
         } catch (error) {
             this.logger.error('ContextDetectionEngine: Failed to get staged files', error);
+            m_get_staged_files.success();
             return [];
         }
+        m_get_staged_files.success();
     }
 
     getStagedSignature() {
+        const m_get_staged_signature = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'get_staged_signature'
+        });
+
+        m_get_staged_signature.started();
         try {
             const patch = this.git.getDiff(true);
             if (!patch) return '';
+            m_get_staged_signature.success();
             return crypto.createHash('sha1').update(patch).digest('hex');
         } catch (error) {
+            m_get_staged_signature.success();
             return '';
         }
+        m_get_staged_signature.success();
     }
 
     getRecentlyModifiedFiles() {
+        const m_get_recently_modified_files = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'get_recently_modified_files'
+        });
+
+        m_get_recently_modified_files.started();
         try {
             const output = this.git.getStatusShort();
             if (!output) return [];
 
+            m_get_recently_modified_files.success();
+
             return output
                 .split('\n')
                 .filter(Boolean)
                 .map(line => line.trim().substring(2)) // Remove status code
                 .filter(file => !file.startsWith('.git'));
         } catch (error) {
+            m_get_recently_modified_files.success();
             return [];
         }
+        m_get_recently_modified_files.success();
     }
 
     getRecentCommitPatterns() {
+        const m_get_recent_commit_patterns = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'get_recent_commit_patterns'
+        });
+
+        m_get_recent_commit_patterns.started();
         try {
             const output = this.git.getLog(10);
             if (!output) return [];
@@ -116,18 +161,31 @@ class ContextDetectionEngine {
                 commits.push(currentCommit);
             }
 
+            m_get_recent_commit_patterns.success();
+
             return commits.slice(0, 10);
         } catch (error) {
+            m_get_recent_commit_patterns.success();
             return [];
         }
+        m_get_recent_commit_patterns.success();
     }
 
     getCurrentBranch() {
+        const m_get_current_branch = createMetricScope({
+            hook: 'context_detection_engine',
+            operation: 'get_current_branch'
+        });
+
+        m_get_current_branch.started();
         try {
+            m_get_current_branch.success();
             return this.git.getCurrentBranch();
         } catch (error) {
+            m_get_current_branch.success();
             return 'unknown';
         }
+        m_get_current_branch.success();
     }
 
     inferFromGitStatus() {

package/scripts/hooks-system/application/services/DynamicRulesLoader.js

@@ -1,9 +1,18 @@
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 const fs = require('fs').promises;
 const path = require('path');
-const env = require('../config/env');
 
 class DynamicRulesLoader {
     constructor(rulesDirectory, logger = console) {
+        const m_constructor = createMetricScope({
+            hook: 'dynamic_rules_loader',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         this.rulesDirectory = rulesDirectory || null;
         this.logger = logger;
         this.rulesDirectories = this.resolveRulesDirectories();
@@ -19,6 +28,7 @@ class DynamicRulesLoader {
             ttl: 60000
         };
         this.lastLoadWarnings = [];
+        m_constructor.success();
     }
 
     resolveRulesDirectories() {
@@ -28,7 +38,7 @@ class DynamicRulesLoader {
             return [this.rulesDirectory];
         }
 
-        const envRaw = env.get('AST_RULES_DIRECTORIES');
+        const envRaw = process.env.AST_RULES_DIRECTORIES;
         if (envRaw && typeof envRaw === 'string' && envRaw.trim().length > 0) {
             return envRaw
                 .split(',')