pumuki-ast-hooks 5.5.58 → 5.5.60

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.5.58",
+  "version": "5.5.60",
   "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": {
@@ -134,4 +134,4 @@
     "./skills": "./skills/skill-rules.json",
     "./hooks": "./hooks/index.js"
   }
-}
+}

package/scripts/hooks-system/application/services/installation/McpConfigurator.js

@@ -3,7 +3,6 @@ const path = require('path');
 const AuditLogger = require('../logging/AuditLogger');
 const McpServerConfigBuilder = require('./mcp/McpServerConfigBuilder');
 const McpProjectConfigWriter = require('./mcp/McpProjectConfigWriter');
-const McpGlobalConfigCleaner = require('./mcp/McpGlobalConfigCleaner');
 
 const COLORS = {
     reset: '\x1b[0m',
@@ -19,7 +18,6 @@ class McpConfigurator {
         this.logger = logger;
         this.serverConfigBuilder = new McpServerConfigBuilder(targetRoot, hookSystemRoot, logger);
         this.projectWriter = new McpProjectConfigWriter(targetRoot, logger, this.logSuccess.bind(this), this.logWarning.bind(this), this.logInfo.bind(this));
-        this.globalCleaner = new McpGlobalConfigCleaner(targetRoot, logger, this.logInfo.bind(this));
     }
 
     configure() {
@@ -27,7 +25,6 @@ class McpConfigurator {
 
         const { serverId, mcpConfig } = this.serverConfigBuilder.build();
         this.projectWriter.configureProjectScoped(mcpConfig, serverId);
-        this.globalCleaner.cleanupGlobalConfig(serverId);
     }
 
     detectIDEs() {

package/scripts/hooks-system/infrastructure/ast/common/ast-common.js

@@ -281,6 +281,11 @@ function runCommonIntelligence(project, findings) {
 
     const full = sf.getFullText();
     const isSpecFile = /\.(spec|test)\.(ts|tsx|js|jsx)$/.test(filePath);
+    const isSwiftFile = /\.swift$/i.test(filePath);
+
+    // Skip secret detection for Swift struct/class properties - they're not hardcoded secrets
+    if (isSwiftFile) return;
+
     const secretPattern = /(PASSWORD|TOKEN|SECRET|API_KEY)\s*[:=]\s*['"]([^'"]{8,})['"]/gi;
     const matches = Array.from(full.matchAll(secretPattern));
 

package/scripts/hooks-system/infrastructure/ast/ios/detectors/ios-ast-intelligent-strategies.js

@@ -153,7 +153,7 @@ function analyzeClassAST(analyzer, node, filePath) {
         return true;
     });
 
-    if (name && !/Spec$|Test$|Mock/.test(name)) {
+    if (name && !/Spec$|Test$|Mock/.test(name) && !name.includes('Coordinator')) {
         const complexity = calculateComplexityAST(substructure);
         analyzer.godClassCandidates.push({
             name,
@@ -233,8 +233,15 @@ function analyzeClassAST(analyzer, node, filePath) {
     }
 
     // Skip ISP validation for test files - spies/mocks are allowed to have unused properties
+<<<<<<< HEAD
+    // Also skip ObservableObject classes - their @Published properties are inherently observed externally
+    const isTestFile = /Tests?\/|Spec|Mock|Spy|Stub|Fake|Dummy/.test(filePath);
+    const isObservableObject = inheritedTypes.some((t) => t['key.name'] === 'ObservableObject');
+    if (!isTestFile && !isObservableObject) {
+=======
     const isTestFile = /Tests?\/|Spec|Mock|Spy|Stub|Fake|Dummy/.test(filePath);
     if (!isTestFile) {
+>>>>>>> origin/main
         const unusedProps = findUnusedPropertiesAST(analyzer, properties, methods);
         for (const prop of unusedProps) {
             analyzer.pushFinding(