Sort versions in output

Otherwise they are ordered by provider source

Author: discordier

src/JsonRepositoryWriter.php

@@ -4,25 +4,29 @@
 
 namespace Phpcq\RepositoryBuilder;
 
+use Composer\Semver\Comparator;
 use Phpcq\RepositoryDefinition\AbstractHash;
 use Phpcq\RepositoryDefinition\Plugin\PhpFilePluginVersion;
 use Phpcq\RepositoryDefinition\Plugin\PluginInterface;
 use Phpcq\RepositoryDefinition\Plugin\PluginRequirements;
 use Phpcq\RepositoryDefinition\Tool\ToolInterface;
 use Phpcq\RepositoryDefinition\Tool\ToolRequirements;
+use Phpcq\RepositoryDefinition\Tool\ToolVersionInterface;
 use Phpcq\RepositoryDefinition\VersionRequirementList;
 use stdClass;
 use Symfony\Component\Filesystem\Filesystem;
 
 use function array_flip;
 use function array_key_exists;
+use function array_values;
 use function glob;
 use function hash_file;
 use function is_array;
 use function is_dir;
 use function is_string;
 use function json_encode;
 use function sprintf;
+use function usort;
 
 use const JSON_UNESCAPED_SLASHES;
 use const JSON_UNESCAPED_UNICODE;
@@ -139,7 +143,8 @@ private function processTool(ToolInterface $tool): ?array
         $data = [
             'tools' => [],
         ];
-        foreach ($tool as $version) {
+
+        foreach ($this->getToolVersions($tool) as $version) {
             if (!isset($data['tools'][$name = $tool->getName()])) {
                 $data['tools'][$name] = [];
             }
@@ -296,4 +301,21 @@ private function dumpFile(string $fileName, array $contents): void
         assert(is_string($encoded));
         $this->filesystem->dumpFile($fullFileName, $encoded);
     }
+
+    /** @return list<ToolVersionInterface> */
+    private function getToolVersions(ToolInterface $tool): array
+    {
+        /** @var array<string, ToolVersionInterface> $versions */
+        $versions = [];
+        foreach ($tool as $version) {
+            $versions[] = $version;
+        }
+        usort(
+            $versions,
+            static fn(ToolVersionInterface $first, ToolVersionInterface $second): int
+                => Comparator::greaterThanOrEqualTo($first->getVersion(), $second->getVersion()) ? 1 : -1
+        );
+
+        return $versions;
+    }
 }