pumuki-ast-hooks 5.3.18 → 5.3.19

package/scripts/hooks-system/infrastructure/ast/ios/analyzers/iOSArchitectureRules.js

@@ -7,8 +7,6 @@ class iOSArchitectureRules {
   }
 
   runRules(files) {
-    console.log(`[iOS Architecture] Detected pattern: ${this.pattern}`);
-
     switch (this.pattern) {
       case 'FEATURE_FIRST_CLEAN_DDD':
         this.checkFeatureFirstCleanDDDRules(files);
@@ -16,9 +14,6 @@ class iOSArchitectureRules {
       case 'MVVM':
         this.checkMVVMRules(files);
         break;
-      case 'MVVM-C':
-        this.checkMVVMCRules(files);
-        break;
       case 'MVP':
         this.checkMVPRules(files);
         break;
@@ -38,7 +33,7 @@ class iOSArchitectureRules {
         this.checkMixedArchitectureRules(files);
         break;
       default:
-        console.log(`[iOS Architecture] No specific rules for pattern: ${this.pattern}`);
+        break;
     }
   }
 
@@ -68,10 +63,10 @@ class iOSArchitectureRules {
         });
 
         if (file.includes('/domain/') && !file.includes('/entities/') &&
-            !file.includes('/value-objects/') && !file.includes('/interfaces/')) {
+          !file.includes('/value-objects/') && !file.includes('/interfaces/')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.domain_structure',
-            severity: 'medium',
+            severity: 'critical',
             message: 'Archivo en domain/ sin estructura correcta. Usar entities/, value-objects/, interfaces/',
             filePath: file,
             line: 1
@@ -86,7 +81,7 @@ class iOSArchitectureRules {
         if (hasProperties > 3 && hasMethods === 0) {
           pushFinding(this.findings, {
             ruleId: 'ios.ddd.anemic_entity',
-            severity: 'high',
+            severity: 'critical',
             message: 'Entity anémica (solo properties, sin comportamiento). Añadir métodos de negocio.',
             filePath: file,
             line: 1,
@@ -99,7 +94,7 @@ class iOSArchitectureRules {
         if (content.includes('var ') && !content.includes('private(set)')) {
           pushFinding(this.findings, {
             ruleId: 'ios.ddd.mutable_value_object',
-            severity: 'high',
+            severity: 'critical',
             message: 'Value Object con properties mutables. VOs deben ser inmutables (usar let).',
             filePath: file,
             line: 1,
@@ -110,7 +105,7 @@ class iOSArchitectureRules {
         if (!content.includes('init(') || !content.includes('throw')) {
           pushFinding(this.findings, {
             ruleId: 'ios.ddd.value_object_no_validation',
-            severity: 'medium',
+            severity: 'critical',
             message: 'Value Object sin validación en init(). VOs deben garantizar invariantes.',
             filePath: file,
             line: 1,
@@ -128,7 +123,7 @@ class iOSArchitectureRules {
         if (!content.includes('func execute(')) {
           pushFinding(this.findings, {
             ruleId: 'ios.ddd.usecase_missing_execute',
-            severity: 'high',
+            severity: 'critical',
             message: 'Use Case sin método execute(). Convención: func execute(input: Input) async throws -> Output',
             filePath: file,
             line: 1
@@ -138,7 +133,7 @@ class iOSArchitectureRules {
         if (content.includes('UIKit') || content.includes('SwiftUI')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.usecase_ui_dependency',
-            severity: 'high',
+            severity: 'critical',
             message: 'Use Case depende de UI framework. Application layer debe ser UI-agnostic.',
             filePath: file,
             line: 1
@@ -150,7 +145,7 @@ class iOSArchitectureRules {
         if (!file.includes('/infrastructure/')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.repository_wrong_layer',
-            severity: 'high',
+            severity: 'critical',
             message: 'Repository implementation fuera de infrastructure/. Mover a infrastructure/repositories/',
             filePath: file,
             line: 1
@@ -162,7 +157,7 @@ class iOSArchitectureRules {
         if (!file.includes('/domain/')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.repository_interface_wrong_layer',
-            severity: 'high',
+            severity: 'critical',
             message: 'Repository protocol fuera de domain/. Mover a domain/interfaces/',
             filePath: file,
             line: 1
@@ -174,7 +169,7 @@ class iOSArchitectureRules {
         if (!file.includes('/application/')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.dto_wrong_layer',
-            severity: 'medium',
+            severity: 'critical',
             message: 'DTO fuera de application/. Mover a application/dto/',
             filePath: file,
             line: 1
@@ -213,7 +208,7 @@ class iOSArchitectureRules {
         if (complexMethods.length > 0) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.infrastructure_business_logic',
-            severity: 'high',
+            severity: 'critical',
             message: 'Infrastructure con lógica de negocio compleja. Mover a domain/ o application/',
             filePath: file,
             line: 1,
@@ -226,7 +221,7 @@ class iOSArchitectureRules {
         if (content.includes('Entity') && !content.includes('DTO') && !content.includes('Dto')) {
           pushFinding(this.findings, {
             ruleId: 'ios.clean.presentation_uses_entity',
-            severity: 'medium',
+            severity: 'critical',
             message: 'Presentation usando Entities de domain directamente. Usar DTOs para desacoplar.',
             filePath: file,
             line: 1,
@@ -245,7 +240,7 @@ class iOSArchitectureRules {
         if (!content.includes('ObservableObject') && !content.includes('@Observable')) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvvm.viewmodel_not_observable',
-            severity: 'high',
+            severity: 'critical',
             message: 'ViewModel debe conformar ObservableObject o usar @Observable macro (iOS 17+)',
             filePath: file,
             line: 1
@@ -255,7 +250,7 @@ class iOSArchitectureRules {
         if (content.match(/import\s+UIKit/) && !content.includes('#if canImport(UIKit)')) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvvm.viewmodel_uikit_dependency',
-            severity: 'high',
+            severity: 'critical',
             message: 'ViewModel NO debe depender de UIKit. Usar tipos agnósticos de plataforma.',
             filePath: file,
             line: content.split('\n').findIndex(line => line.includes('import UIKit')) + 1
@@ -266,7 +261,7 @@ class iOSArchitectureRules {
         if (classMatch && content.includes('var ') && !content.includes('@Published')) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvvm.missing_published',
-            severity: 'medium',
+            severity: 'critical',
             message: 'ViewModel properties que cambian deben usar @Published para notificar a la View',
             filePath: file,
             line: 1
@@ -276,7 +271,7 @@ class iOSArchitectureRules {
 
       if (file.includes('View.swift') || content.includes('struct ') && content.includes(': View')) {
         const hasBusinessLogic =
-          /func\s+\w+\([^)]*\)\s*->\s*\w+\s*{[\s\S]{100,}/.test(content) || 
+          /func\s+\w+\([^)]*\)\s*->\s*\w+\s*{[\s\S]{100,}/.test(content) ||
           content.includes('URLSession') ||
           content.includes('CoreData') ||
           /\.save\(|\.fetch\(|\.delete\(/.test(content);
@@ -284,7 +279,7 @@ class iOSArchitectureRules {
         if (hasBusinessLogic) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvvm.view_business_logic',
-            severity: 'high',
+            severity: 'critical',
             message: 'View contiene lógica de negocio. Mover al ViewModel.',
             filePath: file,
             line: 1
@@ -294,63 +289,6 @@ class iOSArchitectureRules {
     });
   }
 
-  checkMVVMCRules(files) {
-    this.checkMVVMRules(files);
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-
-      if (file.includes('Coordinator.swift')) {
-        if (!content.includes('protocol Coordinator') && !content.includes(': Coordinator')) {
-          pushFinding(this.findings, {
-            ruleId: 'ios.mvvmc.coordinator_protocol',
-            severity: 'medium',
-            message: 'Coordinator debe conformar protocol Coordinator con start() y navigate(to:)',
-            filePath: file,
-            line: 1
-          });
-        }
-
-        if (!/func\s+start\(\)/.test(content)) {
-          pushFinding(this.findings, {
-            ruleId: 'ios.mvvmc.coordinator_missing_start',
-            severity: 'high',
-            message: 'Coordinator debe implementar func start() para iniciar el flujo',
-            filePath: file,
-            line: 1
-          });
-        }
-
-        const hasBusinessLogic =
-          content.includes('URLSession') ||
-          content.includes('CoreData') ||
-          /\.save\(|\.fetch\(|\.delete\(/.test(content);
-
-        if (hasBusinessLogic) {
-          pushFinding(this.findings, {
-            ruleId: 'ios.mvvmc.coordinator_business_logic',
-            severity: 'high',
-            message: 'Coordinator NO debe contener lógica de negocio. Solo navegación.',
-            filePath: file,
-            line: 1
-          });
-        }
-      }
-
-      if (file.includes('ViewModel.swift')) {
-        if (content.includes('navigationController') || content.includes('.present(')) {
-          pushFinding(this.findings, {
-            ruleId: 'ios.mvvmc.viewmodel_navigation',
-            severity: 'high',
-            message: 'ViewModel NO debe manejar navegación. Delegar al Coordinator.',
-            filePath: file,
-            line: 1
-          });
-        }
-      }
-    });
-  }
-
   checkMVPRules(files) {
     files.forEach(file => {
       const content = this.readFile(file);
@@ -359,7 +297,7 @@ class iOSArchitectureRules {
         if (!content.includes('protocol ') && content.includes('View')) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvp.view_not_protocol',
-            severity: 'high',
+            severity: 'critical',
             message: 'En MVP, View debe ser un protocol implementado por ViewController',
             filePath: file,
             line: 1
@@ -371,7 +309,7 @@ class iOSArchitectureRules {
         if (content.includes('var view:') && !content.includes('weak var view')) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvp.presenter_strong_view',
-            severity: 'high',
+            severity: 'critical',
             message: 'Presenter debe tener referencia weak a View para evitar retain cycles',
             filePath: file,
             line: content.split('\n').findIndex(line => line.includes('var view:')) + 1
@@ -385,7 +323,7 @@ class iOSArchitectureRules {
         if (hasLogic < 3) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvp.presenter_thin',
-            severity: 'medium',
+            severity: 'critical',
             message: 'Presenter parece tener poca lógica. En MVP, Presenter debe contener toda la lógica de presentación.',
             filePath: file,
             line: 1
@@ -402,7 +340,7 @@ class iOSArchitectureRules {
         if (hasBusinessLogic) {
           pushFinding(this.findings, {
             ruleId: 'ios.mvp.viewcontroller_business_logic',
-            severity: 'high',
+            severity: 'critical',
             message: 'ViewController NO debe contener lógica de negocio. Delegar al Presenter.',
             filePath: file,
             line: 1
@@ -494,7 +432,7 @@ class iOSArchitectureRules {
 
       if (file.includes('Entity.swift')) {
         const hasMethods = (content.match(/func\s+/g) || []).length;
-        if (hasMethods > 2) { 
+        if (hasMethods > 2) {
           pushFinding(this.findings, {
             ruleId: 'ios.viper.entity_with_logic',
             severity: 'medium',
@@ -537,7 +475,7 @@ class iOSArchitectureRules {
 
       if (content.includes(': Reducer')) {
         if ((content.includes('URLSession') || content.includes('async ')) &&
-            !content.includes('Effect')) {
+          !content.includes('Effect')) {
           pushFinding(this.findings, {
             ruleId: 'ios.tca.missing_effect',
             severity: 'high',

package/scripts/hooks-system/infrastructure/hooks/skill-activation-prompt.js

@@ -3,9 +3,8 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
-const env = require('../../config/env');
 
-const projectDir = env.get('CLAUDE_PROJECT_DIR', process.cwd());
+const projectDir = process.env.CLAUDE_PROJECT_DIR || process.cwd();
 const rulesPath = path.join(projectDir, '.cursor', 'ai-skills', 'skill-rules.json');
 const mdcRulesDir = path.join(projectDir, '.cursor', 'rules');
 const debugLogPath = path.join(projectDir, '.audit_tmp', 'skill-activation-debug.log');
@@ -55,7 +54,7 @@ function sendMacNotification(title, message) {
         return;
     }
 
-    const enabled = env.getBool('HOOK_GUARD_ENABLE_SKILL_ACTIVATION', true);
+    const enabled = process.env.HOOK_GUARD_ENABLE_SKILL_ACTIVATION !== '0';
     if (!enabled) {
         appendDebugLog('Notifications disabled via HOOK_GUARD_ENABLE_SKILL_ACTIVATION');
         return;

package/scripts/hooks-system/infrastructure/logging/UnifiedLoggerFactory.js

@@ -1,26 +1,13 @@
 const fs = require('fs');
 const path = require('path');
-const env = require('../../config/env');
-
-// AuditLogger import for logging critical operations
-const AuditLogger = require('../services/logging/AuditLogger');
-
-// Initialize audit logger
-const auditLogger = new AuditLogger({
-    repoRoot: process.cwd(),
-    filename: path.join('.audit_tmp', 'logger-factory-audit.log')
-});
-
-// Import recordMetric for prometheus metrics
-const { recordMetric } = require('../telemetry/metrics-logger');
 
 const UnifiedLogger = require('../../application/services/logging/UnifiedLogger');
 
 function createUnifiedLogger({
     repoRoot = process.cwd(),
     component = 'HookSystem',
-    consoleLevel = env.get('HOOK_LOG_CONSOLE_LEVEL', 'info'),
-    fileLevel = env.get('HOOK_LOG_FILE_LEVEL', 'debug'),
+    consoleLevel = process.env.HOOK_LOG_CONSOLE_LEVEL || 'info',
+    fileLevel = process.env.HOOK_LOG_FILE_LEVEL || 'debug',
     fileName = null,
     defaultData = {}
 } = {}) {
@@ -30,10 +17,10 @@ function createUnifiedLogger({
     const logFileName = fileName || `${component.replace(/\s+/g, '-').toLowerCase()}.log`;
     const filePath = path.join(reportsDir, logFileName);
 
-    const maxSizeBytes = env.getNumber('HOOK_LOG_MAX_SIZE', 5 * 1024 * 1024);
-    const maxFiles = env.getNumber('HOOK_LOG_MAX_FILES', 5);
+    const maxSizeBytes = Number(process.env.HOOK_LOG_MAX_SIZE || 5 * 1024 * 1024);
+    const maxFiles = Number(process.env.HOOK_LOG_MAX_FILES || 5);
 
-    const logger = new UnifiedLogger({
+    return new UnifiedLogger({
         component,
         console: {
             enabled: true,
@@ -48,23 +35,6 @@ function createUnifiedLogger({
         },
         defaultData
     });
-
-    auditLogger.log({
-        action: 'logger_created',
-        category: 'system',
-        severity: 'info',
-        message: `UnifiedLogger created for component: ${component}`,
-        metadata: {
-            component,
-            filePath,
-            consoleLevel,
-            fileLevel,
-            maxSizeBytes,
-            maxFiles
-        }
-    });
-
-    return logger;
 }
 
 module.exports = { createUnifiedLogger };

package/scripts/hooks-system/infrastructure/orchestration/intelligent-audit.js

@@ -8,16 +8,9 @@ const { TokenManager } = require('../utils/token-manager');
 const { toErrorMessage } = require('../utils/error-utils');
 const fs = require('fs');
 const path = require('path');
-const env = require('../../config/env');
-
-// AuditLogger import for logging critical operations
-const AuditLogger = require('../services/logging/AuditLogger');
-
-// Import recordMetric for prometheus metrics
-const { recordMetric } = require('../telemetry/metrics-logger');
 
 function resolveAuditTmpDir() {
-  const configured = (env.get('AUDIT_TMP', '') || '').trim();
+  const configured = (process.env.AUDIT_TMP || '').trim();
   if (configured.length > 0) {
     return path.isAbsolute(configured) ? configured : path.join(process.cwd(), configured);
   }
@@ -29,37 +22,13 @@ function resolveAuditTmpDir() {
  * Called by audit.sh after AST analysis completes
  */
 async function runIntelligentAudit() {
-  const auditLogger = new AuditLogger({
-    repoRoot: process.cwd(),
-    filename: path.join('.audit_tmp', 'audit.log')
-  });
-
   try {
     console.log('[Intelligent Audit] Starting severity evaluation...');
 
-    auditLogger.log({
-      action: 'audit_start',
-      category: 'system',
-      severity: 'info',
-      message: 'Intelligent audit orchestration started',
-      metadata: {
-        timestamp: new Date().toISOString(),
-        scope: env.get('AI_GATE_SCOPE', 'staging')
-      }
-    });
-
     const rawViolations = loadRawViolations();
     console.log(`[Intelligent Audit] Loaded ${rawViolations.length} violations from AST`);
 
-    auditLogger.log({
-      action: 'violations_loaded',
-      category: 'data',
-      severity: 'info',
-      message: `Loaded ${rawViolations.length} violations from AST analysis`,
-      metadata: { violationCount: rawViolations.length }
-    });
-
-    const gateScope = String(env.get('AI_GATE_SCOPE', 'staging') || 'staging').trim().toLowerCase();
+    const gateScope = String(process.env.AI_GATE_SCOPE || 'staging').trim().toLowerCase();
     const isRepoScope = gateScope === 'repo' || gateScope === 'repository';
 
     let violationsForGate = [];
@@ -107,26 +76,6 @@ async function runIntelligentAudit() {
       console.log(`[Intelligent Audit] Blocked by: ${gateResult.blockedBy} violations`);
     }
 
-    auditLogger.log({
-      action: 'quality_gate_applied',
-      category: 'security',
-      severity: gateResult.passed ? 'info' : 'warn',
-      message: `Quality gate ${gateResult.passed ? 'PASSED' : 'FAILED'}${gateResult.blockedBy ? ` - blocked by ${gateResult.blockedBy}` : ''}`,
-      metadata: {
-        passed: gateResult.passed,
-        blockedBy: gateResult.blockedBy,
-        violationCount: enhancedViolations.length
-      }
-    });
-
-    // Record prometheus metrics for gate result
-    recordMetric({
-      hook: 'quality_gate',
-      status: gateResult.passed ? 'passed' : 'failed',
-      violationCount: enhancedViolations.length,
-      blockedBy: gateResult.blockedBy || 'none'
-    });
-
     console.log('[Intelligent Audit] Generating reports...');
     const reportGenerator = new ReportGenerator();
     const reportPaths = reportGenerator.save(enhancedViolations, gateResult);
@@ -135,25 +84,6 @@ async function runIntelligentAudit() {
     console.log(`  - JSON: ${reportPaths.jsonPath}`);
     console.log(`  - Text: ${reportPaths.textPath}`);
 
-    auditLogger.log({
-      action: 'reports_generated',
-      category: 'system',
-      severity: 'info',
-      message: 'Audit reports generated and saved',
-      metadata: {
-        jsonPath: reportPaths.jsonPath,
-        textPath: reportPaths.textPath,
-        violationCount: enhancedViolations.length
-      }
-    });
-
-    // Record prometheus metrics for report generation
-    recordMetric({
-      hook: 'audit_reports',
-      status: 'generated',
-      violationCount: enhancedViolations.length
-    });
-
     const tracker = new SeverityTracker();
     tracker.record(enhancedViolations, gateResult);
 
@@ -282,21 +212,21 @@ function updateAIEvidence(violations, gateResult, tokenUsage) {
     const currentBranch = execSync('git branch --show-current', { encoding: 'utf8' }).trim();
 
     const resolveBaseBranch = () => {
-      const configured = env.get('AST_BASE_BRANCH', '');
-      if (configured && configured.trim().length > 0) {
-        return configured.trim();
-      }
-      try {
-        execSync('git show-ref --verify --quiet refs/heads/develop', { stdio: 'ignore' });
-        return 'develop';
-      } catch {
+        const configured = process.env.AST_BASE_BRANCH;
+        if (configured && configured.trim().length > 0) {
+            return configured.trim();
+        }
         try {
-          execSync('git show-ref --verify --quiet refs/heads/main', { stdio: 'ignore' });
-          return 'main';
+            execSync('git show-ref --verify --quiet refs/heads/develop', { stdio: 'ignore' });
+            return 'develop';
         } catch {
-          return 'main';
+            try {
+                execSync('git show-ref --verify --quiet refs/heads/main', { stdio: 'ignore' });
+                return 'main';
+            } catch {
+                return 'main';
+            }
         }
-      }
     };
     const baseBranch = resolveBaseBranch();
     const isProtected = ['main', 'master', baseBranch].includes(currentBranch);
@@ -304,12 +234,12 @@ function updateAIEvidence(violations, gateResult, tokenUsage) {
     const highViolations = violations.filter(v => v.severity === 'HIGH');
     const blockingViolations = [...criticalViolations, ...highViolations].slice(0, 50);
 
-    const gateScope = String(env.get('AI_GATE_SCOPE', 'staging') || 'staging').trim().toLowerCase();
+    const gateScope = String(process.env.AI_GATE_SCOPE || 'staging').trim().toLowerCase();
 
     const existingGate = evidence.ai_gate && typeof evidence.ai_gate === 'object' ? evidence.ai_gate : null;
     let preserveExistingRepoGate = false;
     if (gateScope !== 'repo' && gateScope !== 'repository' && existingGate && existingGate.scope === 'repo' && existingGate.status === 'BLOCKED') {
-      const preserveMs = env.getNumber('AI_GATE_REPO_PRESERVE_MS', 600000);
+      const preserveMs = Number(process.env.AI_GATE_REPO_PRESERVE_MS || 600000);
       const lastCheckMs = Date.parse(existingGate.last_check || '');
       if (!Number.isNaN(preserveMs) && preserveMs > 0 && !Number.isNaN(lastCheckMs)) {
         const ageMs = Date.now() - lastCheckMs;

package/scripts/hooks-system/infrastructure/telemetry/metrics-server.js

@@ -3,22 +3,9 @@
 const http = require('http');
 const fs = require('fs');
 const path = require('path');
-const env = require('../../config/env');
 
-// AuditLogger import for logging critical operations
-const AuditLogger = require('../services/logging/AuditLogger');
-
-// Import recordMetric for prometheus metrics
-const { recordMetric } = require('./metrics-logger');
-
-const PORT = env.getNumber('HOOK_METRICS_PORT', 9464);
-const METRICS_FILE = path.join(process.cwd(), env.get('HOOK_METRICS_FILE', '.audit_tmp/hook-metrics.jsonl'));
-
-// Initialize audit logger
-const auditLogger = new AuditLogger({
-  repoRoot: process.cwd(),
-  filename: path.join('.audit_tmp', 'metrics-server-audit.log')
-});
+const PORT = Number(process.env.HOOK_METRICS_PORT || 9464);
+const METRICS_FILE = path.join(process.cwd(), '.audit_tmp', 'hook-metrics.jsonl');
 
 function loadMetrics() {
   if (!fs.existsSync(METRICS_FILE)) return [];
@@ -62,49 +49,13 @@ const server = http.createServer((req, res) => {
     const body = buildPrometheusMetrics();
     res.writeHead(200, { 'Content-Type': 'text/plain' });
     res.end(body);
-
-    auditLogger.log({
-      action: 'metrics_served',
-      category: 'observability',
-      severity: 'info',
-      message: 'Prometheus metrics served successfully',
-      metadata: {
-        port: PORT,
-        metricsCount: body.split('\n').filter(line => line.trim() && !line.startsWith('#')).length,
-        endpoint: '/metrics'
-      }
-    });
     return;
   }
 
   res.writeHead(404);
   res.end();
-
-  auditLogger.log({
-    action: 'invalid_request',
-    category: 'observability',
-    severity: 'warn',
-    message: 'Invalid request to metrics server',
-    metadata: {
-      url: req.url,
-      method: req.method,
-      port: PORT
-    }
-  });
 });
 
 server.listen(PORT, () => {
   console.log(`Hook-System metrics server running on http://localhost:${PORT}/metrics`);
-
-  auditLogger.log({
-    action: 'server_started',
-    category: 'system',
-    severity: 'info',
-    message: 'Metrics server started successfully',
-    metadata: {
-      port: PORT,
-      endpoint: '/metrics',
-      metricsFile: METRICS_FILE
-    }
-  });
 });

package/scripts/hooks-system/infrastructure/validators/enforce-english-literals.js

@@ -4,9 +4,8 @@
 const fs = require('fs');
 const path = require('path');
 const { execSync } = require('child_process');
-const env = require('../../config/env');
 
-const REPO_ROOT = env.get('HOOK_GUARD_REPO_ROOT', process.cwd());
+const REPO_ROOT = process.env.HOOK_GUARD_REPO_ROOT || process.cwd();
 const CONFIG_PATH = path.join(REPO_ROOT, 'scripts', 'hooks-system', 'config', 'language-guard.json');
 const DEFAULT_IGNORED_SEGMENTS = [
     `${path.sep}node_modules${path.sep}`,
@@ -23,7 +22,7 @@ function decodeUnicode(value) {
     try {
         return JSON.parse(`"${value}"`);
     } catch (error) {
-        if (env.isDev || env.getBool('DEBUG', false)) {
+        if (process.env.NODE_ENV === 'development' || process.env.DEBUG) {
             console.debug(`[enforce-english-literals] Failed to decode Unicode value "${value}": ${error.message}`);
         }
         return value;
@@ -120,10 +119,13 @@ function analyzeFile(relativePath, config) {
 
 function collectStagedFiles() {
     try {
-        const stagedFilesRaw = execSync('git diff --cached --name-only', { encoding: 'utf8' });
-        return stagedFilesRaw.split('\n').filter(Boolean).map(file => file.trim());
+        const raw = execSync('git diff --cached --name-only --diff-filter=ACMR', {
+            cwd: REPO_ROOT,
+            encoding: 'utf8'
+        });
+        return raw.split('\n').map(entry => entry.trim()).filter(Boolean);
     } catch (error) {
-        if (env.isDev || env.getBool('DEBUG', false)) {
+        if (process.env.NODE_ENV === 'development' || process.env.DEBUG) {
             console.debug(`[enforce-english-literals] Failed to collect staged files: ${error.message}`);
         }
         return [];