Added dependency detection inside *.js, *.xml files.

README.md

@@ -4,10 +4,10 @@ Package allows to run static analysis on Magento 2 Module Packages to provide an
 
 ### Supported tools: ###
 
-- **Composer.json package dependencies checker** - check *.php and *.phtml on a subject if other packages used inside
+- **Composer.json package dependencies checker** - check *.xml, *.js, *.php and *.phtml on a subject if other packages used inside
   and check if corresponding module/package is declared as required in composer.json.
 - **Module.xml dependencies checker** - analyse if packages' etc/module.xml file contains in 'sequence' section all
-  Magento 2 modules which classes are used in *.php and *.phtml files of the package.
+  Magento 2 modules which classes are used in *.xml, *.js, *.php and *.phtml files of the package.
 - **Package structure checker** - verify if all newly added Magento 2 modules has a proper structure with all required
   files.
 

src/Analysis/Service/Dependencies/Scanner/PhpFiles.php

@@ -7,7 +7,16 @@
 
 class PhpFiles implements DependenciesScannerInterface
 {
-    private const FILE_MASKS = ['php', 'phtml'];
+    private const FILE_MASKS = ['php', 'phtml', 'xml', 'js'];
+    private const SKIP_FILES = [
+        'module.xml',
+        'db_schema.xml',
+        'config.xml',
+        'sections.xml',
+        'page_types.xml',
+        'fieldset.xml',
+        'csp_whitelist.xml'
+    ];
 
     private string $regexp;
 
@@ -69,6 +78,9 @@ public function lookupDependencies(Package $package): array
      */
     private function analyzeFile(\SplFileInfo $file, array $currentModuleNamespaces): array
     {
+        if (in_array($file->getFilename(), self::SKIP_FILES)) {
+            return [];
+        }
         $contents = \php_strip_whitespace($file->getPathname());
 
         if ($file->getExtension() === 'phtml') {
@@ -91,7 +103,7 @@ private function analyzeFile(\SplFileInfo $file, array $currentModuleNamespaces)
             $dependenciesInfo[] = $referenceModule;
         }
 
-        return $dependenciesInfo;
+        return array_unique($dependenciesInfo);
     }
 
     /**
@@ -105,8 +117,7 @@ private function stripeHtml(string $contents): string
     {
         return (string)preg_replace_callback(
             '~(<\?(php|=)\s+.*\?>)~sU',
-            function (array $matches) use ($contents, &$contentsWithoutHtml)
-            {
+            function (array $matches) use ($contents, &$contentsWithoutHtml) {
                 $contentsWithoutHtml .= $matches[1];
 
                 return $contents;