pumuki-ast-hooks 5.5.46 → 5.5.48

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

@@ -1,12 +1,22 @@
 const fs = require('fs');
 const path = require('path');
 const glob = require('glob');
+const {
+  detectFeatureFirstClean,
+  detectTCA,
+  detectVIPER,
+  detectCleanSwift,
+  detectMVP,
+  detectMVVMC,
+  detectMVVM,
+  detectMVC,
+} = require('../detectors/ios-architecture-strategies');
 
 class iOSArchitectureDetector {
   constructor(projectRoot) {
     this.projectRoot = projectRoot;
     this.patterns = {
-      featureFirstClean: 0,  
+      featureFirstClean: 0,
       mvvm: 0,
       mvvmc: 0,
       mvp: 0,
@@ -62,249 +72,36 @@ class iOSArchitectureDetector {
   }
 
   detectFeatureFirstClean(files) {
-    const hasFeaturesFolders = files.some(f =>
-      /\/Features?\/\w+\/(domain|application|infrastructure|presentation)\//.test(f)
-    );
-
-    const cleanArchFolders = ['domain', 'application', 'infrastructure', 'presentation'];
-    const foundCleanFolders = cleanArchFolders.filter(folder => {
-      return files.some(f => f.includes(`/${folder}/`));
-    });
-
-    const dddConcepts = files.filter(f =>
-      f.includes('/entities/') ||
-      f.includes('/value-objects/') ||
-      f.includes('/use-cases/') ||
-      f.includes('Entity.swift') ||
-      f.includes('VO.swift') ||
-      f.includes('UseCase.swift')
-    );
-
-    if (hasFeaturesFolders) {
-      this.patterns.featureFirstClean += 10; 
-    }
-
-    if (foundCleanFolders.length >= 3) {
-      this.patterns.featureFirstClean += foundCleanFolders.length * 3;
-    }
-
-    if (dddConcepts.length > 0) {
-      this.patterns.featureFirstClean += dddConcepts.length * 2;
-    }
-
-    const featureNames = new Set();
-    files.forEach(f => {
-      const match = f.match(/\/Features?\/(\w+)\//);
-      if (match) {
-        featureNames.add(match[1]);
-      }
-    });
-
-    if (featureNames.size >= 2) {
-      this.patterns.featureFirstClean += featureNames.size * 4;
-    }
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-
-      if (content.includes('struct ') && content.includes('VO')) {
-        this.patterns.featureFirstClean += 2;
-      }
-
-      if (file.includes('Entity.swift') && content.includes('func ')) {
-        this.patterns.featureFirstClean += 2;
-      }
-
-      if (file.includes('/domain/') && content.includes('protocol ') && content.includes('Repository')) {
-        this.patterns.featureFirstClean += 3;
-      }
-
-      if (file.includes('/application/') && content.includes('UseCase')) {
-        this.patterns.featureFirstClean += 2;
-      }
-    });
-
+    detectFeatureFirstClean(files, this.patterns, this.projectRoot);
     console.log(`[Architecture Detection] Feature-First + Clean + DDD score: ${this.patterns.featureFirstClean}`);
   }
 
   detectTCA(files) {
-    const tcaIndicators = [
-      'import ComposableArchitecture',
-      'Store<',
-      'struct.*State',
-      'enum.*Action',
-      ': Reducer',
-      'Effect<'
-    ];
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-      const matches = tcaIndicators.filter(indicator =>
-        new RegExp(indicator).test(content)
-      ).length;
-
-      if (matches >= 3) {
-        this.patterns.tca += matches;
-      }
-    });
+    detectTCA(files, this.patterns);
   }
 
   detectVIPER(files) {
-    const viperFiles = files.filter(f =>
-      /Presenter\.swift$|Interactor\.swift$|Router\.swift$|Entity\.swift$/.test(f)
-    );
-
-    if (viperFiles.length >= 4) {
-      this.patterns.viper += viperFiles.length;
-    }
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-      const viperProtocols = [
-        'ViewProtocol',
-        'PresenterProtocol',
-        'InteractorProtocol',
-        'RouterProtocol'
-      ];
-
-      const matches = viperProtocols.filter(proto =>
-        content.includes(proto)
-      ).length;
-
-      if (matches >= 2) {
-        this.patterns.viper += matches * 2;
-      }
-    });
-
-    const viperFolders = ['View', 'Interactor', 'Presenter', 'Entity', 'Router'];
-    const hasViperStructure = viperFolders.filter(folder => {
-      const folderPath = path.join(this.projectRoot, folder);
-      return fs.existsSync(folderPath);
-    }).length;
-
-    if (hasViperStructure >= 3) {
-      this.patterns.viper += hasViperStructure * 3;
-    }
+    detectVIPER(files, this.patterns, this.projectRoot);
   }
 
   detectCleanSwift(files) {
-    const cleanSwiftFiles = files.filter(f =>
-      /Models\.swift$/.test(f)
-    );
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-      const cleanSwiftIndicators = [
-        'DisplayLogic',
-        'BusinessLogic',
-        'PresentationLogic',
-        'Request\\s*{',
-        'Response\\s*{',
-        'ViewModel\\s*{'
-      ];
-
-      const matches = cleanSwiftIndicators.filter(indicator =>
-        new RegExp(indicator).test(content)
-      ).length;
-
-      if (matches >= 3) {
-        this.patterns.cleanSwift += matches * 2;
-      }
-    });
+    detectCleanSwift(files, this.patterns);
   }
 
   detectMVP(files) {
-    const presenterFiles = files.filter(f => /Presenter\.swift$/.test(f));
-    const interactorFiles = files.filter(f => /Interactor\.swift$/.test(f));
-    const routerFiles = files.filter(f => /Router\.swift$/.test(f));
-
-    if (presenterFiles.length >= 2 && interactorFiles.length === 0 && routerFiles.length === 0) {
-      this.patterns.mvp += presenterFiles.length * 3;
-    }
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-
-      const hasMVPProtocols =
-        content.includes('ViewProtocol') &&
-        content.includes('PresenterProtocol') &&
-        !content.includes('InteractorProtocol');
-
-      if (hasMVPProtocols) {
-        this.patterns.mvp += 3;
-      }
-    });
+    detectMVP(files, this.patterns);
   }
 
   detectMVVMC(files) {
-    const coordinatorFiles = files.filter(f => /Coordinator\.swift$/.test(f));
-
-    if (coordinatorFiles.length >= 1) {
-      this.patterns.mvvmc += coordinatorFiles.length * 3;
-    }
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-
-      if (content.includes('protocol Coordinator') ||
-          content.includes(': Coordinator') ||
-          /func\s+start\(\)/.test(content) && /func\s+navigate/.test(content)) {
-        this.patterns.mvvmc += 2;
-      }
-    });
+    detectMVVMC(files, this.patterns);
   }
 
   detectMVVM(files) {
-    const viewModelFiles = files.filter(f => /ViewModel\.swift$/.test(f));
-
-    if (viewModelFiles.length >= 2) {
-      this.patterns.mvvm += viewModelFiles.length * 2;
-    }
-
-    files.forEach(file => {
-      const content = this.readFile(file);
-
-      const mvvmIndicators = [
-        '@Published',
-        ': ObservableObject',
-        'import Combine',
-        'class.*ViewModel'
-      ];
-
-      const matches = mvvmIndicators.filter(indicator =>
-        new RegExp(indicator).test(content)
-      ).length;
-
-      if (matches >= 2) {
-        this.patterns.mvvm += matches;
-      }
-    });
+    detectMVVM(files, this.patterns);
   }
 
   detectMVC(files) {
-    const viewControllerFiles = files.filter(f => /ViewController\.swift$/.test(f));
-    const viewModelFiles = files.filter(f => /ViewModel\.swift$/.test(f));
-    const presenterFiles = files.filter(f => /Presenter\.swift$/.test(f));
-    const interactorFiles = files.filter(f => /Interactor\.swift$/.test(f));
-
-    if (viewControllerFiles.length >= 2 &&
-        viewModelFiles.length === 0 &&
-        presenterFiles.length === 0 &&
-        interactorFiles.length === 0) {
-
-      viewControllerFiles.forEach(file => {
-        const content = this.readFile(file);
-        const lines = content.split('\n').length;
-
-        if (lines > 300) {
-          this.patterns.mvc += 3; 
-        } else if (lines > 150) {
-          this.patterns.mvc += 2;
-        } else {
-          this.patterns.mvc += 1;
-        }
-      });
-    }
+    detectMVC(files, this.patterns);
   }
 
   getDominantPattern() {
@@ -323,7 +120,7 @@ class iOSArchitectureDetector {
     }
 
     if (sorted.length > 1 && sorted[1][1] >= dominantScore * 0.7) {
-      return 'MIXED'; 
+      return 'MIXED';
     }
 
     return this.normalizePatternName(dominant);
@@ -331,7 +128,7 @@ class iOSArchitectureDetector {
 
   normalizePatternName(pattern) {
     const mapping = {
-      'featureFirstClean': 'FEATURE_FIRST_CLEAN_DDD', 
+      'featureFirstClean': 'FEATURE_FIRST_CLEAN_DDD',
       'mvvm': 'MVVM',
       'mvvmc': 'MVVM-C',
       'mvp': 'MVP',