feat(Config): Add ability to provider app service provider config

- By using `HasConfig` interface on service provider you can merge own provider config and user defined config.
- Also name of the config is same as generated service file name.
- See ValidConfig and LoadProviderConfigTest

src/Config/AbstractProviderConfig.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LaraStrict\Config;
+
+use Illuminate\Config\Repository;
+use LaraStrict\Providers\AbstractServiceProvider;
+use LaraStrict\Providers\Actions\GetAppServiceProviderForClassAction;
+
+abstract class AbstractProviderConfig extends AbstractConfig
+{
+    public function __construct(
+        Repository $config,
+        private readonly GetAppServiceProviderForClassAction $getAppServiceProviderForClassAction
+    ) {
+        parent::__construct($config);
+    }
+
+    /**
+     * @return class-string<AbstractServiceProvider>
+     */
+    abstract protected function getServiceProvider(): string;
+
+    protected function getConfigFileName(): string
+    {
+        $appService = $this->getAppServiceProviderForClassAction->execute($this->getServiceProvider());
+
+        return $appService->serviceFileName;
+    }
+}

src/Config/Laravel/AppConfig.php

@@ -81,6 +81,6 @@ public function getEnvironment(): EnvironmentType|string
 
     protected function getConfigFileName(): string
     {
-        return self::ConfigName;
+        return 'app';
     }
 }

src/Providers/AbstractBaseServiceProvider.php

@@ -10,6 +10,7 @@
 use LaraStrict\Contracts\RunAppServiceProviderPipesActionContract;
 use LaraStrict\Providers\Actions\CreateAppServiceProviderAction;
 use LaraStrict\Providers\Entities\AppServiceProviderEntity;
+use LogicException;
 
 abstract class AbstractBaseServiceProvider extends EventServiceProvider
 {
@@ -60,6 +61,19 @@ public function laraLoadTranslationsFrom(string $path, string $namespace): void
         $this->loadTranslationsFrom($path, $namespace);
     }
 
+    public function laraLoadProviderConfigFrom(string $path, string $namespace): void
+    {
+        $serviceFileName = $namespace . '.php';
+        $configPath = $path . '/Config/' . $serviceFileName;
+        $realPath = realpath($configPath);
+
+        if ($realPath === false) {
+            throw new LogicException('Failed to load config at ' . $configPath);
+        }
+
+        $this->mergeConfigFrom($realPath, $namespace);
+    }
+
     /**
      * @return array<class-string<AppServiceProviderPipeContract>>
      */

src/Providers/AbstractServiceProvider.php

@@ -16,6 +16,7 @@
 use LaraStrict\Providers\Pipes\BootProviderRoutesPipe;
 use LaraStrict\Providers\Pipes\BootProviderViewComponents;
 use LaraStrict\Providers\Pipes\BootViewComposersPipe;
+use LaraStrict\Providers\Pipes\LoadProviderConfig;
 use LaraStrict\Providers\Pipes\LoadProviderTranslations;
 use LaraStrict\Providers\Pipes\LoadProviderViews;
 
@@ -48,7 +49,7 @@ public function boot(): void
 
     protected function registerPipes(): array
     {
-        return [LoadProviderViews::class, LoadProviderTranslations::class];
+        return [LoadProviderViews::class, LoadProviderTranslations::class, LoadProviderConfig::class];
     }
 
     protected function bootPipes(): array

src/Providers/Actions/GetAppServiceProviderForClassAction.php

@@ -0,0 +1,49 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LaraStrict\Providers\Actions;
+
+use Illuminate\Contracts\Foundation\Application;
+use Illuminate\Support\ServiceProvider;
+use LaraStrict\Cache\Contracts\CacheMeServiceContract;
+use LaraStrict\Cache\Enums\CacheMeStrategy;
+use LaraStrict\Providers\AbstractServiceProvider;
+use LaraStrict\Providers\Entities\AppServiceProviderEntity;
+use LogicException;
+
+class GetAppServiceProviderForClassAction
+{
+    public function __construct(
+        private readonly CacheMeServiceContract $cacheMeService,
+    ) {
+    }
+
+    /**
+     * @param class-string<AbstractServiceProvider> $providerClass
+     */
+    public function execute(string $providerClass): AppServiceProviderEntity
+    {
+        return $this->cacheMeService->get(
+            key: 'app-service-provider-' . $providerClass,
+            getValue: static function (Application $application) use ($providerClass) {
+                // TODO add ability to cache getProvider (laravel patch)
+                $serviceProvider = $application->getProvider($providerClass);
+                if ($serviceProvider instanceof ServiceProvider === false) {
+                    throw new LogicException(sprintf('Provider for <%s> not loaded ', $providerClass));
+                }
+
+                if ($serviceProvider instanceof AbstractServiceProvider === false) {
+                    throw new LogicException(sprintf(
+                        'Provider <%s> must use <%s>',
+                        $providerClass,
+                        AbstractServiceProvider::class,
+                    ));
+                }
+
+                return $serviceProvider->getAppServiceProvider();
+            },
+            strategy: CacheMeStrategy::Memory
+        );
+    }
+}

src/Providers/Contracts/HasConfig.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LaraStrict\Providers\Contracts;
+
+interface HasConfig
+{
+}

src/Providers/Pipes/LoadProviderConfig.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace LaraStrict\Providers\Pipes;
+
+use Closure;
+use LaraStrict\Contracts\AppServiceProviderPipeContract;
+use LaraStrict\Providers\Contracts\HasConfig;
+use LaraStrict\Providers\Entities\AppServiceProviderEntity;
+
+class LoadProviderConfig implements AppServiceProviderPipeContract
+{
+    public function handle(AppServiceProviderEntity $appServiceProvider, Closure $next): void
+    {
+        if ($appServiceProvider->serviceProvider instanceof HasConfig) {
+            $appServiceProvider->serviceProvider->laraLoadProviderConfigFrom(
+                path: $appServiceProvider->serviceRootDir,
+                namespace: $appServiceProvider->serviceFileName
+            );
+        }
+
+        $next($appServiceProvider);
+    }
+}

src/Providers/Pipes/LoadProviderTranslations.php

@@ -15,8 +15,8 @@ public function handle(AppServiceProviderEntity $appServiceProvider, Closure $ne
     {
         if ($appServiceProvider->serviceProvider instanceof HasTranslations) {
             $appServiceProvider->serviceProvider->laraLoadTranslationsFrom(
-                $appServiceProvider->serviceRootDir . '/Translations',
-                $appServiceProvider->serviceName
+                path: $appServiceProvider->serviceRootDir . '/Translations',
+                namespace: $appServiceProvider->serviceName
             );
         }
 

src/Translations/AbstractProviderTranslations.php

@@ -5,38 +5,22 @@
 namespace LaraStrict\Translations;
 
 use Illuminate\Contracts\Translation\Translator;
-use Illuminate\Foundation\Application;
-use Illuminate\Support\ServiceProvider;
 use LaraStrict\Providers\AbstractServiceProvider;
-use LogicException;
+use LaraStrict\Providers\Actions\GetAppServiceProviderForClassAction;
 
 abstract class AbstractProviderTranslations extends AbstractTranslations
 {
     protected readonly string $providerKey;
 
-    public function __construct(Translator $translator, Application $application)
-    {
+    public function __construct(
+        Translator $translator,
+        GetAppServiceProviderForClassAction $getAppServiceProviderForClassAction
+    ) {
         parent::__construct($translator);
 
-        // TODO add ability to cache getProvider (laravel patch)
-        $providerClass = $this->getProviderClass();
-        $serviceProvider = $application->getProvider($providerClass);
-
-        if ($serviceProvider instanceof ServiceProvider === false) {
-            throw new LogicException(sprintf('Provider for translation <%s> not loaded ', $providerClass));
-        }
-
-        if ($serviceProvider instanceof AbstractServiceProvider === false) {
-            throw new LogicException(sprintf(
-                'Provider <%s> for translation must use <%s>',
-                $providerClass,
-                AbstractServiceProvider::class,
-            ));
-        }
-
-        $this->providerKey = $serviceProvider
-            ->getAppServiceProvider()
-            ->serviceName;
+        $appService = $getAppServiceProviderForClassAction->execute($this->getProviderClass());
+
+        $this->providerKey = $appService->serviceName;
     }
 
     /**

tests/Feature/Config/AbstractProviderConfigTest.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Config;
+
+use Tests\LaraStrict\Feature\Config\NotUsingPipe\Config\NotUsingPipeConfig;
+use Tests\LaraStrict\Feature\Config\NotUsingPipe\NotUsingPipeServiceProvider;
+use Tests\LaraStrict\Feature\Config\Valid\Config\ValidConfig;
+use Tests\LaraStrict\Feature\Config\Valid\ValidConfigServiceProvider;
+use Tests\LaraStrict\Feature\TestCase;
+
+class AbstractProviderConfigTest extends TestCase
+{
+    public function testValid(): void
+    {
+        $this->app()
+            ->register(provider: ValidConfigServiceProvider::class);
+        $config = $this->app()
+            ->make(ValidConfig::class);
+
+        $this->assertEquals('value!', $config->getTest());
+    }
+
+    public function testNotUsingPipe(): void
+    {
+        $this->app()
+            ->register(provider: NotUsingPipeServiceProvider::class);
+        $config = $this->app()
+            ->make(NotUsingPipeConfig::class);
+
+        $this->assertEquals('missing file, this is a default', $config->getTest());
+    }
+}

tests/Feature/Config/Laravel/AppConfigTest.php

@@ -156,6 +156,6 @@ protected function setEnv(mixed $value): void
 
     protected function getConfigName(): string
     {
-        return AppConfig::ConfigName;
+        return 'app';
     }
 }

tests/Feature/Config/NotUsingPipe/Config/NotUsingPipeConfig.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Config\NotUsingPipe\Config;
+
+use LaraStrict\Config\AbstractProviderConfig;
+use Tests\LaraStrict\Feature\Config\NotUsingPipe\NotUsingPipeServiceProvider;
+
+class NotUsingPipeConfig extends AbstractProviderConfig
+{
+    public function getTest(): string
+    {
+        return $this->get(keyOrPath: ['test', 'sub-key'], default: 'missing file, this is a default');
+    }
+
+    protected function getServiceProvider(): string
+    {
+        return NotUsingPipeServiceProvider::class;
+    }
+}

tests/Feature/Config/NotUsingPipe/NotUsingPipeServiceProvider.php

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Config\NotUsingPipe;
+
+use LaraStrict\Providers\AbstractServiceProvider;
+
+class NotUsingPipeServiceProvider extends AbstractServiceProvider
+{
+}

tests/Feature/Config/Valid/Config/ValidConfig.php

@@ -0,0 +1,23 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Config\Valid\Config;
+
+use LaraStrict\Config\AbstractProviderConfig;
+use Tests\LaraStrict\Feature\Config\Valid\ValidConfigServiceProvider;
+
+class ValidConfig extends AbstractProviderConfig
+{
+    final public const KeyTest = 'test';
+
+    public function getTest(): string
+    {
+        return $this->get(self::KeyTest);
+    }
+
+    protected function getServiceProvider(): string
+    {
+        return ValidConfigServiceProvider::class;
+    }
+}

tests/Feature/Config/Valid/Config/valid.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+use Tests\LaraStrict\Feature\Config\Valid\Config\ValidConfig;
+
+return [
+    ValidConfig::KeyTest => 'value!',
+];

tests/Feature/Config/Valid/ValidConfigServiceProvider.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Config\Valid;
+
+use LaraStrict\Providers\AbstractServiceProvider;
+use LaraStrict\Providers\Contracts\HasConfig;
+
+class ValidConfigServiceProvider extends AbstractServiceProvider implements HasConfig
+{
+}

tests/Feature/Providers/Pipes/LoadProviderConfig/Invalid/InvalidConfigServiceProvider.php

@@ -0,0 +1,12 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\LaraStrict\Feature\Providers\Pipes\LoadProviderConfig\Invalid;
+
+use Illuminate\Support\ServiceProvider;
+use LaraStrict\Providers\Contracts\HasConfig;
+
+class InvalidConfigServiceProvider extends ServiceProvider implements HasConfig
+{
+}