Fix publish command (#93)

* Fix publish command * Fix styling --------- Co-authored-by: simonhamp <[email protected]>
NativePHP · May 6, 2024 · 9d42e65 · 9d42e65
1 parent e897c65
commit 9d42e65
Showing 1 changed file with 18 additions and 5 deletions.
diff --git a/src/Commands/PublishCommand.php b/src/Commands/PublishCommand.php
@@ -12,7 +12,7 @@ class PublishCommand extends Command
 {
     use LocatesPhpBinary;
 
-    protected $signature = 'native:publish {os=all : The operating system to build for (all, linux, mac, windows)}';
+    protected $signature = 'native:publish {os? : The operating system to build for (linux, mac, win)}';
 
     public function handle(): void
     {
@@ -30,16 +30,19 @@ public function handle(): void
                 echo $output;
             });
 
-        $buildCommand = 'npm run build';
-        if ($this->argument('os')) {
-            $buildCommand .= ':'.$this->argument('os');
+        if (! $os = $this->argument('os')) {
+            $os = select(
+                label: 'Please select the operating system to build for',
+                options: ['win', 'linux', 'mac', 'all'],
+                default: $this->getDefaultOs(),
+            );
         }
 
         Process::path(__DIR__.'/../../resources/js/')
             ->env($this->getEnvironmentVariables())
             ->forever()
             ->tty(PHP_OS_FAMILY != 'Windows' && ! $this->option('no-interaction'))
-            ->run($buildCommand, function (string $type, string $output) {
+            ->run("npm run publish:{$os}", function (string $type, string $output) {
                 echo $output;
             });
     }
@@ -62,4 +65,14 @@ protected function getEnvironmentVariables(): array
             Updater::environmentVariables(),
         );
     }
+
+    protected function getDefaultOs(): string
+    {
+        return match (PHP_OS_FAMILY) {
+            'Windows' => 'win',
+            'Darwin' => 'mac',
+            'Linux' => 'linux',
+            default => 'all',
+        };
+    }
 }