pumuki-ast-hooks 5.5.54 → 5.5.56

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.5.54",
+  "version": "5.5.56",
   "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": {

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

@@ -145,27 +145,30 @@ function runCommonIntelligence(project, findings) {
       }
     });
 
-    sf.getDescendantsOfKind(SyntaxKind.CatchClause).forEach((clause) => {
-      const block = typeof clause.getBlock === 'function' ? clause.getBlock() : null;
-      const statements = block && typeof block.getStatements === 'function' ? block.getStatements() : [];
-
-      if ((statements || []).length === 0) {
-        const blockText = block ? block.getText() : '';
-        const hasTestAssertions = /XCTFail|XCTAssert|guard\s+case|expect\(|assert/i.test(blockText);
-        const hasErrorHandling = /throw|console\.|logger\.|log\(|print\(/i.test(blockText);
-
-        if (!hasTestAssertions && !hasErrorHandling) {
-          pushFinding(
-            'common.error.empty_catch',
-            'critical',
-            sf,
-            clause,
-            'Empty catch block detected - always handle errors (log, rethrow, wrap, or return Result)',
-            findings
-          );
+    // Skip Swift files - they should only be analyzed by iOS-specific detectors
+    if (!/\.swift$/i.test(filePath)) {
+      sf.getDescendantsOfKind(SyntaxKind.CatchClause).forEach((clause) => {
+        const block = typeof clause.getBlock === 'function' ? clause.getBlock() : null;
+        const statements = block && typeof block.getStatements === 'function' ? block.getStatements() : [];
+
+        if ((statements || []).length === 0) {
+          const blockText = block ? block.getText() : '';
+          const hasTestAssertions = /XCTFail|XCTAssert|guard\s+case|expect\(|assert/i.test(blockText);
+          const hasErrorHandling = /throw|console\.|logger\.|log\(|print\(/i.test(blockText);
+
+          if (!hasTestAssertions && !hasErrorHandling) {
+            pushFinding(
+              'common.error.empty_catch',
+              'critical',
+              sf,
+              clause,
+              'Empty catch block detected - always handle errors (log, rethrow, wrap, or return Result)',
+              findings
+            );
+          }
         }
-      }
-    });
+      });
+    }
 
     sf.getDescendantsOfKind(SyntaxKind.AnyKeyword).forEach((node) => {
 

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

@@ -658,6 +658,17 @@ function checkDependencyInjectionAST(analyzer, properties, filePath, className,
             continue;
         }
 
+        // Skip generic type parameters (e.g., "Client" in LoginUseCaseImpl<Client: APIClientProtocol>)
+        // These are not concrete dependencies - they are constrained by protocols
+        const propName = prop['key.name'] || '';
+        const isGenericTypeParameter = typename.length === 1 ||
+            (typename.match(/^[A-Z][a-z]*$/) && !typename.includes('Impl') &&
+                (className.includes('<') || propName === 'apiClient' || propName === 'client'));
+
+        if (isGenericTypeParameter) {
+            continue;
+        }
+
         const isConcreteService = /Service$|Repository$|UseCase$|Client$/.test(typename) && !typename.includes('Protocol') && !typename.includes('any ') && !typename.includes('some ');
 
         if (isConcreteService) {