Merge pull request #50 from tonysm/tm/fix-previous-version-detection

Fix outdated command breaks on previous comment format

Author: tonysm

src/Commands/PinCommand.php

@@ -84,7 +84,7 @@ private function importPackages(Packager $packager, Collection $imports): void
     private function pattern(string $package): string
     {
         return sprintf(
-            '#Importmap::pin\([\'\"]%s[\'\"].*$#',
+            '#.*pin\([\'\"]%s[\'\"].*#',
             preg_quote($package),
         );
     }

src/Npm.php

@@ -95,7 +95,7 @@ private function findPackagesFromCdnMatches(string $content)
 
     private function findPackagesFromLocalMatches(string $content)
     {
-        preg_match_all('/^Importmap::pin\(.+\)\;\s*\/\/\s*(.+?)@(.+?)\s+.*\r?$/m', $content, $matches);
+        preg_match_all('/pin\([\'\"](.*?)[\'\"].*\);\s+\/\/\s+.*?@(\d+\.\d+\.\d+.*?)\s/m', $content, $matches);
 
         if (count($matches) !== 3) {
             return collect();

src/Packager.php

@@ -123,7 +123,7 @@ private function packageFilename(string $package): string
 
     private function extractPackageVersionFrom(string $url): string
     {
-        preg_match('#(@\d+\.\d+\.\d+)/#', $url, $matches);
+        preg_match('#(@\d+\.\d+\.\d+.*?)/#', $url, $matches);
 
         if (! ($matches[1] ?? false)) {
             return 'Unknown Version';

tests/fixtures/npm/audit-importmap.php

@@ -3,4 +3,4 @@
 use Tonysm\ImportmapLaravel\Facades\Importmap;
 
 Importmap::pin('is-svg', to: 'https://cdn.skypack.dev/[email protected]', preload: true);
-Importmap::pin('lodash', to: '/js/vendor/lodash.js'); // lodash@4.17.12
+Importmap::pin('lodash', to: '/js/vendor/lodash.js'); // @4.17.12

tests/fixtures/npm/outdated-importmap.php

@@ -3,4 +3,4 @@
 use Tonysm\ImportmapLaravel\Facades\Importmap;
 
 Importmap::pin('is-svg', to: 'https://cdn.skypack.dev/[email protected]', preload: true);
-Importmap::pin('lodash', to: '/js/vendor/lodash.js'); // lodash@4.0.0
+Importmap::pin('lodash', to: '/js/vendor/lodash.js'); // @4.0.0