Merge branch 'feature/installer' of https://github.com/mooxphp/moox into feature/installer

Kim-the-Diamond · Kim-the-Diamond · commit 5eae17775da2 · 2025-12-12T11:09:30.000+01:00
diff --git a/packages/core/config/moox-installer.php b/packages/core/config/moox-installer.php
@@ -113,4 +113,3 @@
         'after_plugins' => null,
     ],
 ];
-
diff --git a/packages/core/src/Console/Commands/MooxInstallCommand.php b/packages/core/src/Console/Commands/MooxInstallCommand.php
@@ -21,7 +21,7 @@
 
 /**
  * Moox Package Installer Command.
- * 
+ *
  * A dynamic, configurable installer for Moox packages.
  * Supports:
  * - Custom installers via traits
@@ -405,4 +405,3 @@ public function getRegistry(): InstallerRegistry
         return $this->registry;
     }
 }
-
diff --git a/packages/core/src/Installer/AbstractAssetInstaller.php b/packages/core/src/Installer/AbstractAssetInstaller.php
@@ -12,7 +12,7 @@
 
 /**
  * Abstract base class for asset installers.
- * 
+ *
  * Provides common functionality for all asset installers including:
  * - Configuration management
  * - Item selection UI
@@ -239,10 +239,10 @@ protected function normalizeItemToString(mixed $item): ?string
 
     /**
      * Publish assets for a package.
-     * 
-     * @param string $packageName The composer package name (e.g., "moox/prompts")
-     * @param string $type The asset type (e.g., "config", "migrations")
-     * @param string|null $publishTag The exact publish tag from mooxInfo (e.g., "moox-prompts-config")
+     *
+     * @param  string  $packageName  The composer package name (e.g., "moox/prompts")
+     * @param  string  $type  The asset type (e.g., "config", "migrations")
+     * @param  string|null  $publishTag  The exact publish tag from mooxInfo (e.g., "moox-prompts-config")
      */
     protected function publishPackageAssets(string $packageName, string $type, ?string $publishTag = null): bool
     {
@@ -277,14 +277,12 @@ protected function publishPackageAssets(string $packageName, string $type, ?stri
 
                 $output = trim(Artisan::output());
 
-                
                 if ($result === 0 && ! str_contains($output, 'Nothing to publish')) {
                     $published = true;
                     break;
-                }else{
+                } else {
                     note($output);
                 }
-
             } catch (\Exception $e) {
                 continue;
             }
@@ -315,4 +313,3 @@ protected function getPackagePath(string $packageName): ?string
         return null;
     }
 }
-
diff --git a/packages/core/src/Installer/Contracts/AssetInstallerInterface.php b/packages/core/src/Installer/Contracts/AssetInstallerInterface.php
@@ -4,7 +4,7 @@
 
 /**
  * Contract for asset installers.
- * 
+ *
  * Implement this interface to create custom asset installers
  * that can be registered with the Moox Installer.
  */
@@ -39,17 +39,17 @@ public function checkExists(string $packageName, array $items): bool;
 
     /**
      * Install the assets.
-     * 
-     * @param array $assets Array of package assets with structure:
-     *                      [['package' => string, 'data' => array, 'provider' => ?string], ...]
+     *
+     * @param  array  $assets  Array of package assets with structure:
+     *                         [['package' => string, 'data' => array, 'provider' => ?string], ...]
      * @return bool True if installation was successful
      */
     public function install(array $assets): bool;
 
     /**
      * Get items from mooxInfo for this installer type.
-     * 
-     * @param array $mooxInfo The mooxInfo array from the service provider
+     *
+     * @param  array  $mooxInfo  The mooxInfo array from the service provider
      * @return array The items relevant to this installer
      */
     public function getItemsFromMooxInfo(array $mooxInfo): array;
@@ -71,4 +71,3 @@ public function getConfig(): array;
      */
     public function setConfig(array $config): void;
 }
-
diff --git a/packages/core/src/Installer/Contracts/PanelAwareInstallerInterface.php b/packages/core/src/Installer/Contracts/PanelAwareInstallerInterface.php
@@ -4,7 +4,7 @@
 
 /**
  * Contract for installers that need panel awareness.
- * 
+ *
  * Implement this interface for installers that need to interact
  * with Filament panels (e.g., plugin installers).
  */
@@ -25,4 +25,3 @@ public function getPanelPath(): ?string;
      */
     public function requiresPanelSelection(): bool;
 }
-
diff --git a/packages/core/src/Installer/InstallerRegistry.php b/packages/core/src/Installer/InstallerRegistry.php
@@ -11,7 +11,7 @@
 
 /**
  * Registry for managing asset installers.
- * 
+ *
  * This registry provides a centralized way to:
  * - Register custom installers
  * - Configure installer behavior
@@ -58,7 +58,7 @@ public static function resetInstance(): void
 
     /**
      * Register the default installers.
-     * 
+     *
      * Installers are only enabled if explicitly defined in the config.
      * If an installer is not in the config, it will be disabled by default.
      */
@@ -78,12 +78,12 @@ protected function registerDefaultInstallers(): void
             // Only enable installer if it's explicitly defined in config
             $isConfigured = array_key_exists($type, $configuredInstallers);
             $config = $configuredInstallers[$type] ?? [];
-            
+
             // If not in config, disable by default
             if (! $isConfigured) {
                 $config['enabled'] = false;
             }
-            
+
             $this->register($type, new $installerClass($config));
         }
     }
@@ -126,7 +126,7 @@ public function has(string $type): bool
 
     /**
      * Get all registered installers.
-     * 
+     *
      * @return array<string, AssetInstallerInterface>
      */
     public function all(): array
@@ -136,15 +136,14 @@ public function all(): array
 
     /**
      * Get all enabled installers sorted by priority.
-     * 
+     *
      * @return array<string, AssetInstallerInterface>
      */
     public function getEnabled(): array
     {
         $enabled = array_filter(
             $this->installers,
-            fn (AssetInstallerInterface $installer, string $type) => 
-                $installer->isEnabled() && ! ($this->skipped[$type] ?? false),
+            fn (AssetInstallerInterface $installer, string $type) => $installer->isEnabled() && ! ($this->skipped[$type] ?? false),
             ARRAY_FILTER_USE_BOTH
         );
 
@@ -331,4 +330,3 @@ public function without(array $types): self
         return $new;
     }
 }
-
diff --git a/packages/core/src/Installer/Installers/ConfigInstaller.php b/packages/core/src/Installer/Installers/ConfigInstaller.php
@@ -7,7 +7,6 @@
 
 use function Moox\Prompts\info;
 use function Moox\Prompts\note;
-use function Moox\Prompts\warning;
 
 /**
  * Installer for configuration files.
@@ -48,7 +47,7 @@ public function checkExists(string $packageName, array $items): bool
             // Handle both array format and string format
             $configName = is_array($configFile) ? ($configFile['name'] ?? '') : $configFile;
             $configPath = config_path($configName.'.php');
-            
+
             if (File::exists($configPath)) {
                 return true;
             }
@@ -101,7 +100,7 @@ public function install(array $assets): bool
                 $published = true;
                 $publishedPackages[] = $packageName;
                 note('    ✅ Published');
-            } 
+            }
         }
 
         // Show summary
diff --git a/packages/core/src/Installer/Installers/MigrationInstaller.php b/packages/core/src/Installer/Installers/MigrationInstaller.php
@@ -58,7 +58,7 @@ public function checkExists(string $packageName, array $items): bool
             if (is_array($migrationName)) {
                 $migrationName = $migrationName['name'] ?? '';
             }
-            
+
             foreach ($existingFiles as $file) {
                 $filename = $file->getFilename();
                 // Remove timestamp prefix if exists (format: YYYY_MM_DD_HHMMSS_name.php)
diff --git a/packages/core/src/Installer/Installers/PluginInstaller.php b/packages/core/src/Installer/Installers/PluginInstaller.php
@@ -2,33 +2,28 @@
 
 namespace Moox\Core\Installer\Installers;
 
+use Filament\Support\Commands\Concerns\CanGeneratePanels;
+use Filament\Support\Commands\Concerns\CanManipulateFiles;
+use Illuminate\Support\Facades\File;
+use Moox\Core\Installer\AbstractAssetInstaller;
+use Moox\Core\Installer\Contracts\PanelAwareInstallerInterface;
+
+use function Moox\Prompts\confirm;
 use function Moox\Prompts\info;
+use function Moox\Prompts\multiselect;
 use function Moox\Prompts\note;
-use function Moox\Prompts\text;
 use function Moox\Prompts\select;
-
-use function Moox\Prompts\confirm;
+use function Moox\Prompts\text;
 use function Moox\Prompts\warning;
-use Illuminate\Console\OutputStyle;
-use Illuminate\Support\Facades\File;
-use function Moox\Prompts\multiselect;
-use Filament\Commands\MakePanelCommand;
-use Illuminate\Support\Facades\Artisan;
-use Moox\Core\Installer\AbstractAssetInstaller;
-use Symfony\Component\Console\Input\ArrayInput;
-use Symfony\Component\Console\Output\ConsoleOutput;
-use Filament\Support\Commands\Concerns\CanGeneratePanels;
-use Filament\Support\Commands\Concerns\CanManipulateFiles;
-use Moox\Core\Installer\Contracts\PanelAwareInstallerInterface;
 
 /**
  * Installer for Filament plugins.
  */
 class PluginInstaller extends AbstractAssetInstaller implements PanelAwareInstallerInterface
 {
-
     use CanGeneratePanels;
     use CanManipulateFiles;
+
     protected ?string $panelPath = null;
 
     protected $panelSelector = null;
@@ -368,4 +363,3 @@ protected function registerPluginsInPanel(array $pluginClasses, string $panelPat
         }
     }
 }
-
diff --git a/packages/core/src/Installer/Installers/SeederInstaller.php b/packages/core/src/Installer/Installers/SeederInstaller.php
@@ -155,4 +155,3 @@ protected function resolveSeederClass(string $packageName, string $seeder): ?str
         return null;
     }
 }
-
diff --git a/packages/core/src/Installer/Installers/TranslationInstaller.php b/packages/core/src/Installer/Installers/TranslationInstaller.php
@@ -7,7 +7,6 @@
 
 use function Moox\Prompts\info;
 use function Moox\Prompts\note;
-use function Moox\Prompts\warning;
 
 /**
  * Installer for translation files.
diff --git a/packages/core/src/Installer/Traits/HasConfigurableInstallers.php b/packages/core/src/Installer/Traits/HasConfigurableInstallers.php
@@ -6,10 +6,10 @@
 
 /**
  * Trait for commands that want configurable installer behavior.
- * 
+ *
  * Use this trait to allow configuration-driven installer setup.
  * Configuration can come from config files, command options, or both.
- * 
+ *
  * Example config (config/moox-installer.php):
  * ```php
  * return [
@@ -80,4 +80,3 @@ protected function buildConfiguredRegistry(): InstallerRegistry
         return $registry;
     }
 }
-
diff --git a/packages/core/src/Installer/Traits/HasCustomInstallers.php b/packages/core/src/Installer/Traits/HasCustomInstallers.php
@@ -7,16 +7,16 @@
 
 /**
  * Trait for commands that want to add custom installers.
- * 
+ *
  * Use this trait in your install command to register package-specific
  * installers that will be processed alongside the default installers.
- * 
+ *
  * Example:
  * ```php
  * class MyPackageInstallCommand extends Command
  * {
  *     use HasCustomInstallers;
- * 
+ *
  *     protected function getCustomInstallers(): array
  *     {
  *         return [
@@ -31,7 +31,7 @@ trait HasCustomInstallers
     /**
      * Get custom installers to register.
      * Override this method to provide package-specific installers.
-     * 
+     *
      * @return array<AssetInstallerInterface>
      */
     protected function getCustomInstallers(): array
@@ -60,4 +60,3 @@ protected function configureRegistry(InstallerRegistry $registry): void
         // Override in your command to customize
     }
 }
-
diff --git a/packages/core/src/Installer/Traits/HasInstallationHooks.php b/packages/core/src/Installer/Traits/HasInstallationHooks.php
@@ -9,21 +9,21 @@
 
 /**
  * Trait for adding lifecycle hooks to the installation process.
- * 
+ *
  * Use this trait to add custom logic before/after installation
  * of specific asset types or the entire installation process.
- * 
+ *
  * Example:
  * ```php
  * class MyInstallCommand extends Command
  * {
  *     use HasInstallationHooks;
- * 
+ *
  *     protected function beforeInstall(): void
  *     {
  *         $this->info('Starting installation...');
  *     }
- * 
+ *
  *     protected function afterMigrations(): void
  *     {
  *         // Run custom migrations or setup
@@ -75,7 +75,7 @@ protected function afterInstaller(string $type): void
 
     /**
      * Run installation with hooks.
-     * 
+     *
      * Uses graceful degradation - if one installer fails, continues with the next.
      */
     protected function runWithHooks(InstallerRegistry $registry, array $assets, array $selectedTypes): void
@@ -119,10 +119,10 @@ protected function runWithHooks(InstallerRegistry $registry, array $assets, arra
 
     /**
      * Filter assets for a specific installer type.
-     * 
+     *
      * This default implementation looks for assets stored with the type as key.
      * Override in your command for custom filtering logic.
-     * 
+     *
      * Expected $assets structure:
      * [
      *     'package/name' => [
@@ -161,4 +161,3 @@ protected function filterAssetsByType(array $assets, string $type, $installer):
         return $typeAssets;
     }
 }
-
diff --git a/packages/core/src/Installer/Traits/HasSkippableInstallers.php b/packages/core/src/Installer/Traits/HasSkippableInstallers.php
diff --git a/packages/core/src/MooxServiceProvider.php b/packages/core/src/MooxServiceProvider.php

-Original file line number
+Diff line change
         'after_plugins' => null,
     ],
 ];
+-
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@`
`21`	`21`
`22`	`22`	`/**`
`23`	`23`	`* Moox Package Installer Command.`
`24`		`- *`
	`24`	`+ *`
`25`	`25`	`* A dynamic, configurable installer for Moox packages.`
`26`	`26`	`* Supports:`
`27`	`27`	`* - Custom installers via traits`
`@@ -405,4 +405,3 @@ public function getRegistry(): InstallerRegistry`
`405`	`405`	`return $this->registry;`
`406`	`406`	`}`
`407`	`407`	`}`
`408`		`-`
Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ public function checkExists(string $packageName, array $items): bool`
`58`	`58`	`if (is_array($migrationName)) {`
`59`	`59`	`$migrationName = $migrationName['name'] ?? '';`
`60`	`60`	`}`
`61`		`-`
	`61`	`+`
`62`	`62`	`foreach ($existingFiles as $file) {`
`63`	`63`	`$filename = $file->getFilename();`
`64`	`64`	`// Remove timestamp prefix if exists (format: YYYY_MM_DD_HHMMSS_name.php)`
Original file line number	Diff line number	Diff line change
`@@ -155,4 +155,3 @@ protected function resolveSeederClass(string $packageName, string $seeder): ?str`
`155`	`155`	`return null;`
`156`	`156`	`}`
`157`	`157`	`}`
`158`		`-`