feat: Add the ability to disable dependency extraction

Add a `dependencyExtractionEnabled` property to `Script` constructor that will disable automatic dependency extraction if set to `false`

Author: nlemoine

src/Script.php

@@ -35,6 +35,19 @@ class Script extends BaseAsset implements Asset, DataAwareAsset, FilterAwareAsse
         'path' => null,
     ];
 
+    protected bool $dependencyExtractionEnabled = false;
+
+    public function __construct(
+        string $handle,
+        string $url,
+        int $location = Asset::FRONTEND | Asset::ACTIVATE,
+        bool $dependencyExtractionEnabled = true
+    ) {
+
+        parent::__construct($handle, $url, $location);
+        $this->dependencyExtractionEnabled = $dependencyExtractionEnabled;
+    }
+
     /**
      * @return array<string, mixed>
      */
@@ -199,7 +212,7 @@ public function useDependencyExtractionPlugin(): Script
      */
     public function version(): ?string
     {
-        $this->resolveDependencyExtractionPlugin();
+        $this->dependencyExtractionEnabled and $this->resolveDependencyExtractionPlugin();
 
         return parent::version();
     }
@@ -209,7 +222,7 @@ public function version(): ?string
      */
     public function dependencies(): array
     {
-        $this->resolveDependencyExtractionPlugin();
+        $this->dependencyExtractionEnabled and $this->resolveDependencyExtractionPlugin();
 
         return parent::dependencies();
     }

tests/phpunit/Unit/Asset/ScriptTest.php

@@ -404,4 +404,59 @@ public function provideVersions(): \Generator
             '1.0',
         ];
     }
+
+    /**
+     * @test
+     */
+    public function testDependencyExtractionCanBeDisabled(): void
+    {
+        $expectedDependencies = ['foo', 'bar', 'baz'];
+        $expectedVersion = '1.0';
+
+        vfsStream::newFile('script.asset.json')
+            ->withContent(
+                json_encode(
+                    [
+                        'dependencies' => $expectedDependencies,
+                        'version' => $expectedVersion,
+                    ]
+                )
+            )
+            ->at($this->root);
+
+        $expectedFile = vfsStream::newFile('script.js')->at($this->root);
+
+        $testee = new Script('script', $expectedFile->url(), Asset::FRONTEND, false);
+        $testee->withFilePath($expectedFile->url());
+
+        // Dependencies should not be loaded from the .asset.json file
+        static::assertEmpty($testee->dependencies());
+
+        // Version is still autodiscovered from file modification time, not from .asset.json
+        // To verify it's not from .asset.json, we check it's not the expected version
+        static::assertNotEquals($expectedVersion, $testee->version());
+    }
+
+    /**
+     * @test
+     */
+    public function testDependencyExtractionEnabledByDefault(): void
+    {
+        $expectedDependencies = ['foo', 'bar', 'baz'];
+
+        vfsStream::newFile('script.asset.json')
+            ->withContent(json_encode(['dependencies' => $expectedDependencies]))
+            ->at($this->root);
+
+        $expectedFile = vfsStream::newFile('script.js')->at($this->root);
+
+        $testee = new Script('script', $expectedFile->url());
+        $testee->withFilePath($expectedFile->url());
+
+        // Should load dependencies by default
+        static::assertEqualsCanonicalizing(
+            $expectedDependencies,
+            $testee->dependencies()
+        );
+    }
 }