feat(core): load local and production configurations last

Author: innocenzi

packages/core/src/Kernel/LoadConfig.php

@@ -10,6 +10,7 @@
 use SplFileInfo;
 use Tempest\Core\ConfigCache;
 use Tempest\Core\Kernel;
+use Tempest\Support\Arr;
 
 /** @internal */
 final readonly class LoadConfig
@@ -21,10 +22,7 @@ public function __construct(
 
     public function __invoke(): void
     {
-        $configPaths = $this->cache->resolve(
-            'config_cache',
-            fn () => $this->find(),
-        );
+        $configPaths = $this->cache->resolve('config_cache', fn () => $this->find());
 
         foreach ($configPaths as $path) {
             $configFile = require $path;
@@ -55,6 +53,29 @@ public function find(): array
             }
         }
 
-        return $configPaths;
+        return Arr\sort_by_callback($configPaths, function (string $path1, string $path2) {
+            $getPriority = fn (string $path): int => match (true) {
+                str_contains($path, '/vendor/') => 0,
+                str_contains($path, '.local.config.php') => 5,
+                str_contains($path, '.dev.config.php') => 5,
+                str_contains($path, '.production.config.php') => 4,
+                str_contains($path, '.prod.config.php') => 4,
+                str_contains($path, '.prd.config.php') => 4,
+                str_contains($path, '.staging.config.php') => 3,
+                str_contains($path, '.stg.config.php') => 3,
+                str_contains($path, '.test.config.php') => 3,
+                str_contains($path, '.config.php') => 2,
+                default => 1,
+            };
+
+            $priorityA = $getPriority($path1);
+            $priorityB = $getPriority($path2);
+
+            if ($priorityA !== $priorityB) {
+                return $priorityA <=> $priorityB;
+            }
+
+            return strcmp($path1, $path2);
+        });
     }
 }

packages/support/src/Arr/functions.php

@@ -1116,12 +1116,8 @@ function sort(iterable $array, bool $desc = false, ?bool $preserveKeys = null, i
      * @template TValue
      *
      * @param iterable<TKey,TValue> $array
-     * @param callable $callback The function to use for comparing values. It should accept two parameters
-     *                           and return an integer less than, equal to, or greater than zero if the
-     *                           first argument is considered to be respectively less than, equal to, or
-     *                           greater than the second.
-     * @param bool|null $preserveKeys Preserves array keys if `true`; reindexes numerically if `false`.
-     *                                Defaults to `null`, which auto-detects preservation based on array type  (associative or list).
+     * @param \Closure(TValue $a, TValue $b) $callback The function to use for comparing values. It should accept two parameters and return an integer less than, equal to, or greater than zero if the first argument is considered to be respectively less than, equal to, or greater than the second.
+     * @param bool|null $preserveKeys Preserves array keys if `true`; reindexes numerically if `false`. Defaults to `null`, which auto-detects preservation based on array type (associative or list).
      * @return array<array-key, TValue> Key type depends on whether array keys are preserved or not.
      */
     function sort_by_callback(iterable $array, callable $callback, ?bool $preserveKeys = null): array

tests/Integration/Core/Config/Fixtures/0-db.local.config.php

@@ -0,0 +1,7 @@
+<?php
+
+use Tempest\Database\Config\SQLiteConfig;
+
+return new SQLiteConfig(
+    path: 'dblocal.sqlite',
+);

tests/Integration/Core/Config/Fixtures/1-db.stg.config.php

@@ -0,0 +1,7 @@
+<?php
+
+use Tempest\Database\Config\SQLiteConfig;
+
+return new SQLiteConfig(
+    path: 'dbstg.sqlite',
+);

tests/Integration/Core/Config/Fixtures/2-db.config.php

@@ -0,0 +1,7 @@
+<?php
+
+use Tempest\Database\Config\SQLiteConfig;
+
+return new SQLiteConfig(
+    path: 'db1.sqlite',
+);

tests/Integration/Core/Config/LoadConfigTest.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Tests\Tempest\Integration\Core\Config;
+
+use Tempest\Core\ConfigCache;
+use Tempest\Core\Kernel\LoadConfig;
+use Tempest\Discovery\DiscoveryLocation;
+use Tests\Tempest\Integration\FrameworkIntegrationTestCase;
+
+final class LoadConfigTest extends FrameworkIntegrationTestCase
+{
+    public function test_config_loaded_in_order()
+    {
+        $this->container->get(ConfigCache::class)->clear();
+
+        $this->kernel->discoveryLocations = [
+            new DiscoveryLocation('App', __DIR__ . '/Fixtures'),
+        ];
+
+        $config = $this->container->get(LoadConfig::class)->find();
+
+        $this->assertStringContainsString('db.config.php', $config[0]);
+        $this->assertStringContainsString('db.stg.config.php', $config[1]);
+        $this->assertStringContainsString('db.local.config.php', $config[2]);
+    }
+}