pumuki-ast-hooks 5.3.19 → 5.3.20

package/docs/VIOLATIONS_RESOLUTION_PLAN.md

@@ -10,8 +10,8 @@
 ---
 
 ## 📊 Resumen Ejecutivo
-- **Estado actual:** ⚠️ Acción requerida (91 críticas, 0 altas, 30 medias, 14 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 1: BLOQUEANTES (CRITICAL + HIGH)
+    Resolver CRITICAL                :active,  crit1, 2025-12-30, 1d
+    Resolver HIGH                    :         high1, after crit1, 2d
  
-    section Fase 2: HIGH + MEDIUM
-    Corrección HIGH                  :         high1, after crit5, 1d
-    Refactorización MEDIUM           :         med1, after high1, 5d
+    section Fase 2: MEDIUM
+    Refactorización MEDIUM           :         med1, after high1, 7d
 
     section Fase 3: LOW
     Optimizaciones y documentación   :         low1, after med1, 5d
@@ -46,48 +42,56 @@ gantt
 
 ---
 
-## 🔴 Fase 1: Violaciones CRÍTICAS (91)
-| Estado | Violación | Cant. | Responsable | DOD (Definition of Done) | Doc |
-|--------|-----------|-------|-------------|--------------------------|-----|
-| 🚧 | backend.observability.missing_prometheus | 45 | BE | Añadir métricas Prometheus a todos los servicios críticos; endpoints /metrics funcionando | [Prometheus](../docs/metrics-monitoring.md) |
-| ⏳ | backend.config.missing_env_separation | 27 | BE | Config por entorno (dev/stg/prod), sin secretos hardcode | [Config entornos](../docs/env-configuration.md) |
-| ⏳ | backend.security.missing_audit_logging | 23 | BE | Audit trail en operaciones sensibles + logs estructurados | [Audit logging](../docs/security-auditing.md) |
-| ⏳ | backend.error.custom_exceptions | 16 | BE | CustomError base + reemplazo de `Error` genérico en BE; tests pasando | [Guía de excepciones](../docs/error-handling.md) |
-| ⏳ | backend.reliability.missing_bulkhead | 10 | BE | Limitadores/aislamiento en puntos críticos + pruebas de carga | [Reliability](../docs/reliability-patterns.md) |
-| ⏳ | backend.event.emitter | 4 | BE | try/catch en manejadores de eventos; manejo de errores robusto | [Eventos](../docs/event-handling.md) |
-| ⏳ | backend.observability.missing_alerting | 3 | SRE | Alertas en métricas críticas; umbrales definidos | [Alerting](../docs/alerting-system.md) |
-| ⏳ | backend.config.missing_validation | 1 | BE | Validación de configuración al inicio | [Config validation](../docs/config-validation.md) |
-| ⏳ | backend.config.missing_env_validation | 1 | BE | Validación de variables de entorno requeridas | [Env validation](../docs/env-validation.md) |
-| ⏳ | backend.database.raw_sql | 1 | BE | Uso de ORM en lugar de raw SQL | [Database](../docs/database-layer.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 (30)
+## 🟠 Fase 2: Violaciones MEDIUM (294)
 | Estado | Violación | Cant. | Responsable | DOD | Doc |
 |--------|-----------|-------|-------------|-----|-----|
-| ✅ | HIGH | 0 | BE | No aplica (repo JS puro sin TS); se documenta enfoque de seguridad de tipos | [Type safety](../docs/type-safety.md) |
-| ⏳ | MEDIUM | 30 | BE | Resolver violaciones de complejidad media restantes | [Medium violations](../docs/medium-violations.md) |
+| ⏳ | MEDIUM | 294 | BE | Resolver violaciones de complejidad media restantes | [Medium violations](../docs/medium-violations.md) |
 
 ---
 
-## 🔵 Fase 3: Violaciones LOW (14)
+## 🔵 Fase 3: Violaciones LOW (242)
 | Estado | Violación | Cant. | Responsable | DOD | Doc |
 |--------|-----------|-------|-------------|-----|-----|
-| ⏳ | LOW | 14 | BE/FE | Resolver violaciones de baja prioridad restantes | [Low violations](../docs/low-violations.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 | 91 | 0 | 0% |
-| HIGH + MEDIUM | 30 | 0 | 0% |
-| LOW | 14 | 0 | 0% |
-| **TOTAL** | **135** | **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: añadidas a 27 servicios, pero 45 violaciones restantes indican necesidad de más instrumentación.  
+- 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.19",
+  "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": [

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,8 +1,18 @@
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 const fs = require('fs').promises;
 const path = require('path');
 
 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();
@@ -18,6 +28,7 @@ class DynamicRulesLoader {
             ttl: 60000
         };
         this.lastLoadWarnings = [];
+        m_constructor.success();
     }
 
     resolveRulesDirectories() {

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

@@ -1,8 +1,22 @@
 const { execSync, spawnSync } = require('child_process');
 const { ConfigurationError } = require('../../domain/errors');
 
+const { recordMetric } = require('../../../infrastructure/telemetry/metrics-logger');
+
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class GitFlowService {
     constructor(repoRoot, options = {}, logger = console, gitQuery = null, gitCommand = null, githubAdapter = null) {
+        recordMetric({
+            hook: 'git_flow_service',
+            operation: 'constructor',
+            status: 'started',
+            repoRoot: repoRoot.substring(0, 100),
+            hasOptions: !!options
+        });
+
         this.repoRoot = repoRoot;
         this.logger = logger;
         this.gitQuery = gitQuery;
@@ -16,6 +30,16 @@ class GitFlowService {
         this.requireClean = options.requireClean !== false;
 
         this._validateDependencies();
+
+        recordMetric({
+            hook: 'git_flow_service',
+            operation: 'constructor',
+            status: 'success',
+            developBranch: this.developBranch,
+            mainBranch: this.mainBranch,
+            autoSyncEnabled: this.autoSyncEnabled,
+            autoCleanEnabled: this.autoCleanEnabled
+        });
     }
 
     _validateDependencies() {
@@ -25,9 +49,17 @@ class GitFlowService {
     }
 
     getCurrentBranch() {
+        const m_get_current_branch = createMetricScope({
+            hook: 'git_flow_service',
+            operation: 'get_current_branch'
+        });
+
+        m_get_current_branch.started();
         if (this.gitQuery) {
+            m_get_current_branch.success();
             return this.gitQuery.getCurrentBranch();
         }
+        m_get_current_branch.success();
         return 'unknown';
     }
 
@@ -40,12 +72,31 @@ class GitFlowService {
     }
 
     syncBranches() {
+        recordMetric({
+            hook: 'git_flow_service',
+            operation: 'sync_branches',
+            status: 'started',
+            autoSyncEnabled: this.autoSyncEnabled
+        });
+
         if (!this.autoSyncEnabled) {
+            recordMetric({
+                hook: 'git_flow_service',
+                operation: 'sync_branches',
+                status: 'skipped',
+                reason: 'auto_sync_disabled'
+            });
             return { success: false, message: 'Auto-sync disabled' };
         }
 
         if (!this.isClean()) {
             this.logger.warn('Skipping sync: working directory not clean');
+            recordMetric({
+                hook: 'git_flow_service',
+                operation: 'sync_branches',
+                status: 'skipped',
+                reason: 'working_directory_not_clean'
+            });
             return { success: false, message: 'Working directory not clean' };
         }
 
@@ -64,22 +115,47 @@ class GitFlowService {
 
                 this.gitCommand.checkout(current);
             } else {
+                recordMetric({
+                    hook: 'git_flow_service',
+                    operation: 'sync_branches',
+                    status: 'failed',
+                    reason: 'no_git_command_adapter'
+                });
                 throw new ConfigurationError('GitCommandAdapter is required for branch synchronization', 'gitCommand');
             }
 
             this.logger.info('Branches synchronized successfully', {
                 branches: [this.developBranch, this.mainBranch]
             });
+            recordMetric({
+                hook: 'git_flow_service',
+                operation: 'sync_branches',
+                status: 'success',
+                branches: [this.developBranch, this.mainBranch].join(',')
+            });
             return { success: true, message: 'Branches synchronized' };
         } catch (error) {
             this.logger.error('Branch synchronization failed', { error: error.message });
+            recordMetric({
+                hook: 'git_flow_service',
+                operation: 'sync_branches',
+                status: 'failed',
+                error: error.message.substring(0, 100)
+            });
             return { success: false, message: error.message };
         }
     }
 
     createPullRequest(sourceBranch, targetBranch, title, body) {
+        const m_create_pull_request = createMetricScope({
+            hook: 'git_flow_service',
+            operation: 'create_pull_request'
+        });
+
+        m_create_pull_request.started();
         if (!this.github) {
             this.logger.error('GitHub adapter not available');
+            m_create_pull_request.success();
             return null;
         }
 
@@ -94,11 +170,15 @@ class GitFlowService {
                 this.logger.warn('Pull Request creation failed or returned no URL');
             }
 
+            m_create_pull_request.success();
+
             return prUrl;
         } catch (error) {
             this.logger.error('[GitFlowService] Failed to create PR:', { error: error.message });
+            m_create_pull_request.success();
             return null;
         }
+        m_create_pull_request.success();
     }
 
     mergeDevelopToMain() {

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

@@ -3,6 +3,10 @@ const { execSync } = require('child_process');
 // Import recordMetric for prometheus metrics
 const { recordMetric } = require('../../../infrastructure/telemetry/metrics-logger');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 const extractFilePath = line => {
     recordMetric({
         hook: 'git_tree_state',

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

@@ -1,6 +1,10 @@
 const { recordMetric } = require('../../infrastructure/telemetry/metrics-logger');
 const HookSystemStateMachine = require('../state/HookSystemStateMachine');
 
+const {
+  createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class HookSystemScheduler {
   constructor({ orchestrator, contextEngine, intervalMs = 30000 }) {
     recordMetric({

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

@@ -5,12 +5,23 @@ const FeatureDetector = require('./commit/FeatureDetector');
 const CommitMessageGenerator = require('./commit/CommitMessageGenerator');
 const UnifiedLogger = require('./logging/UnifiedLogger');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class IntelligentCommitAnalyzer {
     constructor({ repoRoot = process.cwd(), logger = null } = {}) {
+        const m_constructor = createMetricScope({
+            hook: 'intelligent_commit_analyzer',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         this.repoRoot = repoRoot;
         this.logger = logger || console;
         this.featureDetector = new FeatureDetector(this.logger);
         this.messageGenerator = new CommitMessageGenerator(this.logger);
+        m_constructor.success();
     }
 
     detectFeature(filePath) {
@@ -137,6 +148,13 @@ class IntelligentCommitAnalyzer {
      * Get ready-to-commit groups (all groups are ready - no verification)
      */
     getReadyCommits(suggestions) {
+        const m_get_ready_commits = createMetricScope({
+            hook: 'intelligent_commit_analyzer',
+            operation: 'get_ready_commits'
+        });
+
+        m_get_ready_commits.started();
+        m_get_ready_commits.success();
         return suggestions;
     }
 
@@ -144,6 +162,13 @@ class IntelligentCommitAnalyzer {
      * Get groups that need attention (none - we don't verify)
      */
     getNeedsAttention(suggestions) {
+        const m_get_needs_attention = createMetricScope({
+            hook: 'intelligent_commit_analyzer',
+            operation: 'get_needs_attention'
+        });
+
+        m_get_needs_attention.started();
+        m_get_needs_attention.success();
         return [];
     }
 }

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

@@ -1,6 +1,10 @@
 const IntelligentCommitAnalyzer = require('./IntelligentCommitAnalyzer');
 const { getGitTreeState } = require('./GitTreeState');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class IntelligentGitTreeMonitor {
     constructor({
         repoRoot = process.cwd(),
@@ -8,11 +12,18 @@ class IntelligentGitTreeMonitor {
         logger = console,
         autoCommitEnabled = false
     } = {}) {
+        const m_constructor = createMetricScope({
+            hook: 'intelligent_git_tree_monitor',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         this.repoRoot = repoRoot;
         this.notifier = notifier;
         this.logger = logger;
         this.autoCommitEnabled = autoCommitEnabled;
         this.analyzer = new IntelligentCommitAnalyzer({ repoRoot, logger });
+        m_constructor.success();
     }
 
     /**

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

@@ -2,10 +2,21 @@ const path = require('path');
 const fs = require('fs');
 const PlatformHeuristics = require('./platform/PlatformHeuristics');
 
+const {
+    createMetricScope: createMetricScope
+} = require('../../../infrastructure/telemetry/metric-scope');
+
 class PlatformAnalysisService {
     constructor(platformDetector) {
+        const m_constructor = createMetricScope({
+            hook: 'platform_analysis_service',
+            operation: 'constructor'
+        });
+
+        m_constructor.started();
         this.platformDetector = platformDetector;
         this.heuristics = new PlatformHeuristics(platformDetector);
+        m_constructor.success();
     }
 
     /**
@@ -130,6 +141,12 @@ class PlatformAnalysisService {
     }
 
     getScoreReasons(platform, context) {
+        const m_get_score_reasons = createMetricScope({
+            hook: 'platform_analysis_service',
+            operation: 'get_score_reasons'
+        });
+
+        m_get_score_reasons.started();
         const reasons = [];
 
         if (context.stagedFiles && context.stagedFiles.length > 0) {
@@ -166,6 +183,8 @@ class PlatformAnalysisService {
             }
         }
 
+        m_get_score_reasons.success();
+
         return reasons;
     }
 }