Fix ecs issues.

src/Asset/AbstractAssetManager.php

@@ -173,11 +173,11 @@ protected function buildCommand(string $defaultBin, string $action, array|string
         $bin = $this->config->get('manager-bin', $defaultBin);
         $bin = Platform::isWindows() ? str_replace('/', '\\', $bin) : $bin;
         $gOptions = trim($this->config->get('manager-options', ''));
-        $options = trim($this->config->get('manager-'.$action.'-options', ''));
+        $options = trim($this->config->get('manager-' . $action . '-options', ''));
 
-        return $bin.' '.implode(' ', (array) $command)
-            .(empty($gOptions) ? '' : ' '.$gOptions)
-            .(empty($options) ? '' : ' '.$options);
+        return $bin . ' ' . implode(' ', (array) $command)
+            . (empty($gOptions) ? '' : ' ' . $gOptions)
+            . (empty($options) ? '' : ' ' . $options);
     }
 
     protected function getVersion(): string|null

src/Asset/AssetManagerFinder.php

@@ -51,8 +51,6 @@ public function addManager(AssetManagerInterface $manager): void
      *
      * @param null|string $manager The name of the asset manager
      *
-     * @return AssetManagerInterface
-     *
      * @throws RuntimeException When the asset manager does not exist
      * @throws RuntimeException When the asset manager is not found
      */
@@ -72,8 +70,6 @@ public function findManager(string $manager = null): AssetManagerInterface
     /**
      * Find the available asset manager.
      *
-     * @return AssetManagerInterface
-     *
      * @throws RuntimeException When no asset manager is found
      */
     private function findAvailableManager(): AssetManagerInterface

src/Asset/AssetManagerInterface.php

@@ -91,8 +91,6 @@ public function validate(): void;
      *
      * @param RootPackageInterface $rootPackage  The composer root package
      * @param array $dependencies The asset local dependencies
-     *
-     * @return AssetPackageInterface
      */
     public function addDependencies(RootPackageInterface $rootPackage, array $dependencies): AssetPackageInterface;
 

src/Asset/AssetPackage.php

@@ -88,7 +88,7 @@ public function addNewDependencies(array $dependencies): array
             if (isset($installedAssets[$name])) {
                 $existingPackages[] = $name;
             } else {
-                $this->package[self::SECTION_DEPENDENCIES][$name] = 'file:./'.\dirname($path);
+                $this->package[self::SECTION_DEPENDENCIES][$name] = 'file:./' . \dirname($path);
             }
         }
 

src/Config/Config.php

@@ -82,7 +82,7 @@ public function get(string $key, mixed $default = null): mixed
      */
     private function convertEnvKey(string $key): string
     {
-        return 'FOXY__'.strtoupper(str_replace('-', '_', $key));
+        return 'FOXY__' . strtoupper(str_replace('-', '_', $key));
     }
 
     /**

src/Config/ConfigBuilder.php

@@ -66,7 +66,7 @@ private static function getConfigBase(Composer $composer, IOInterface $io = null
     private static function getGlobalConfig(Composer $composer, string $filename, IOInterface $io = null): array
     {
         $home = self::getComposerHome($composer);
-        $file = new JsonFile($home.'/'.$filename.'.json');
+        $file = new JsonFile($home . '/' . $filename . '.json');
         $config = array();
 
         if ($file->exists()) {
@@ -76,7 +76,7 @@ private static function getGlobalConfig(Composer $composer, string $filename, IO
                 $config = $data['config']['foxy'];
 
                 if ($io instanceof IOInterface && $io->isDebug()) {
-                    $io->writeError('Loading Foxy config in file '.$file->getPath());
+                    $io->writeError('Loading Foxy config in file ' . $file->getPath());
                 }
             }
         }

src/Converter/SemverConverter.php

@@ -23,7 +23,7 @@ final class SemverConverter implements VersionConverterInterface
     public function convertVersion(string $version = null): string
     {
         if (\in_array($version, array(null, '', 'latest'), true)) {
-            return ('latest' === $version ? 'default || ' : '').'*';
+            return ('latest' === $version ? 'default || ' : '') . '*';
         }
 
         $version = str_replace('–', '-', $version);
@@ -32,6 +32,6 @@ public function convertVersion(string $version = null): string
         $version = SemverUtil::convertVersionMetadata($version);
         $version = SemverUtil::convertDateVersion($version);
 
-        return $prefix.$version;
+        return $prefix . $version;
     }
 }

src/Converter/SemverUtil.php

@@ -31,16 +31,14 @@ public static function convertDateVersion(string $version): string
     {
         if (preg_match('/^\d{7,}\./', $version)) {
             $pos = strpos($version, '.');
-            $version = substr($version, 0, $pos).self::convertDateMinorVersion(substr($version, $pos + 1));
+            $version = substr($version, 0, $pos) . self::convertDateMinorVersion(substr($version, $pos + 1));
         }
 
         return $version;
     }
 
     /**
      * Converts the version metadata.
-     *
-     * @param string $version
      */
     public static function convertVersionMetadata(string $version): string
     {
@@ -75,10 +73,10 @@ public static function convertVersionMetadata(string $version): string
     public static function createPattern(string $pattern): string
     {
         $numVer = '([0-9]+|x|\*)';
-        $numVer2 = '('.$numVer.'\.'.$numVer.')';
-        $numVer3 = '('.$numVer.'\.'.$numVer.'\.'.$numVer.')';
+        $numVer2 = '(' . $numVer . '\.' . $numVer . ')';
+        $numVer3 = '(' . $numVer . '\.' . $numVer . '\.' . $numVer . ')';
 
-        return '/^('.$numVer.'|'.$numVer2.'|'.$numVer3.')'.$pattern.'/';
+        return '/^(' . $numVer . '|' . $numVer2 . '|' . $numVer3 . ')' . $pattern . '/';
     }
 
     /**
@@ -108,7 +106,7 @@ private static function cleanWildcard(string $version): string
     private static function cleanVersion(string $version, array $matches): array
     {
         $end = substr($version, \strlen($matches[1][0][0]));
-        $version = $matches[1][0][0].'-';
+        $version = $matches[1][0][0] . '-';
 
         $matches = array();
         if (preg_match('/^([-+])/', $end, $matches)) {
@@ -178,6 +176,6 @@ private static function convertDateMinorVersion(string $minor): string
         $minor = (int) $split[0];
         $revision = isset($split[1]) ? (int) $split[1] : 0;
 
-        return '.'.sprintf('%03d', $minor).sprintf('%03d', $revision);
+        return '.' . sprintf('%03d', $minor) . sprintf('%03d', $revision);
     }
 }

src/Event/AbstractSolveEvent.php

@@ -29,7 +29,7 @@ abstract class AbstractSolveEvent extends Event
      * @param string $name The event name.
      * @param string $assetDir The directory of mock assets.
      * @param array $packages All installed Composer packages.
-     * 
+     *
      * @psalm-param PackageInterface[] $packages All installed Composer packages.
      */
     public function __construct(string $name, private string $assetDir, private array $packages = [])

src/Event/GetAssetsEvent.php

@@ -29,7 +29,7 @@ final class GetAssetsEvent extends AbstractSolveEvent
      * @param string $assetDir The directory of mock assets.
      * @param array $packages All installed Composer packages.
      * @param array $assets The map of asset package name and the asset package path.
-     * 
+     *
      * @psalm-param PackageInterface[] $packages All installed Composer packages.
      */
     public function __construct(string $assetDir, array $packages, private array $assets = [])

src/Event/PostSolveEvent.php

@@ -29,7 +29,7 @@ final class PostSolveEvent extends AbstractSolveEvent
      * @param string $assetDir The directory of mock assets.
      * @param array $packages  All installed Composer packages.
      * @param int $runResult The process result of asset manager execution.
-     * 
+     *
      * @psalm-param PackageInterface[] $packages All installed Composer packages.
      */
     public function __construct($assetDir, array $packages, private int $runResult)

src/Event/PreSolveEvent.php

@@ -28,7 +28,7 @@ final class PreSolveEvent extends AbstractSolveEvent
      *
      * @param string $assetDir The directory of mock assets.
      * @param array $packages All installed Composer packages.
-     * 
+     *
      * @psalm-param PackageInterface[] $packages All installed Composer packages.
      */
     public function __construct(string $assetDir, array $packages = [])

src/Fallback/AssetFallback.php

@@ -16,7 +16,6 @@
 use Composer\IO\IOInterface;
 use Composer\Util\Filesystem;
 use Foxy\Config\Config;
-use PHPUnit\TextUI\XmlConfiguration\File;
 
 /**
  * Asset fallback.

src/Json/JsonFile.php

@@ -75,9 +75,7 @@ public function write(array $hash, int $options = 448): void
         self::$encodeIndent = 4;
     }
 
-    /**
-     * {@inheritdoc}
-     */
+
     public static function encode(mixed $data, int $options = 448, string $indent = self::INDENT_DEFAULT): string
     {
         $result = parent::encode($data, $options, $indent);

src/Json/JsonFormatter.php

@@ -62,8 +62,6 @@ public static function getIndent(string $content): int
      * @param array $arrayKeys The list of keys to be retained with an array representation if they are empty.
      * @param int $indent The space count for indent.
      * @param bool $formatJson Check if the json must be formatted.
-     *
-     * @return string
      */
     public static function format(
         string $json,
@@ -125,8 +123,6 @@ static function (mixed $match) {
      * @param array $arrayKeys The list of keys to be retained with an array representation if they are empty.
      *
      * @psalm-param string[] $arrayKeys The list of keys to be retained with an array representation if they are empty.
-     *
-     * @return string
      */
     private static function replaceArrayByMap(string $json, array $arrayKeys): string
     {

src/Solver/Solver.php

@@ -37,10 +37,9 @@ final class Solver implements SolverInterface
     /**
      * Constructor.
      *
-     * @param AssetManagerInterface  $assetManager     The asset manager
-     * @param Config                 $config           The config
-     * @param Filesystem             $filesystem       The composer filesystem
-     * @param null|FallbackInterface $composerFallback The composer fallback
+     * @param AssetManagerInterface $assetManager The asset manager instance.
+     * @param Config $config The config instance.
+     * @param null|FallbackInterface $composerFallback The composer fallback instance.
      */
     public function __construct(
         protected AssetManagerInterface $assetManager,
@@ -66,7 +65,7 @@ public function solve(Composer $composer, IOInterface $io): void
         $dispatcher = $composer->getEventDispatcher();
         $packages = $composer->getRepositoryManager()->getLocalRepository()->getCanonicalPackages();
         $vendorDir = $composer->getConfig()->get('vendor-dir');
-        $assetDir = $this->config->get('composer-asset-dir', $vendorDir.'/foxy/composer-asset/');
+        $assetDir = $this->config->get('composer-asset-dir', $vendorDir . '/foxy/composer-asset/');
         $dispatcher->dispatch(FoxyEvents::PRE_SOLVE, new PreSolveEvent($assetDir, $packages));
         $this->fs->remove($assetDir);
 
@@ -125,8 +124,8 @@ protected function getAssets(Composer $composer, string $assetDir, array $packag
     protected function getMockPackagePath(PackageInterface $package, string $assetDir, string $filename): array
     {
         $packageName = AssetUtil::getName($package);
-        $packagePath = \rtrim($assetDir, '/').'/'.$package->getName();
-        $newFilename = $packagePath.'/'. \basename($filename);
+        $packagePath = \rtrim($assetDir, '/') . '/' . $package->getName();
+        $newFilename = $packagePath . '/' . \basename($filename);
 
         \mkdir($packagePath, 0777, true);
         \copy($filename, $newFilename);

src/Util/AssetUtil.php

@@ -31,7 +31,7 @@ final class AssetUtil
      *
      * @param PackageInterface $package The package instance.
      */
-    public static function getName(PackageInterface $package): string 
+    public static function getName(PackageInterface $package): string
     {
         return AssetPackage::COMPOSER_PREFIX . \str_replace(array('/'), '--', $package->getName());
     }
@@ -45,16 +45,16 @@ public static function getName(PackageInterface $package): string
      * @param array $configPackages The packages defined in config.
      */
     public static function getPath(
-        InstallationManager $installationManager, 
-        AssetManagerInterface $assetManager, 
-        PackageInterface $package, 
+        InstallationManager $installationManager,
+        AssetManagerInterface $assetManager,
+        PackageInterface $package,
         array $configPackages = []
     ): string|null {
         $path = null;
 
         if (static::isAsset($package, $configPackages)) {
             $installPath = $installationManager->getInstallPath($package);
-            $filename = $installPath.'/'.$assetManager->getPackageName();
+            $filename = $installPath . '/' . $assetManager->getPackageName();
             $path = \file_exists($filename) ? \str_replace('\\', '/', \realpath($filename)) : null;
         }
 

src/Util/LockerUtil.php

@@ -13,12 +13,10 @@
 
 namespace Foxy\Util;
 
-use Composer\Composer;
 use Composer\Installer\InstallationManager;
 use Composer\IO\IOInterface;
 use Composer\Json\JsonFile;
 use Composer\Package\Locker;
-use Composer\Repository\RepositoryManager;
 
 /**
  * Helper for Locker.