feat: Session 7 - Add return type declarations to Actions & Jobs

PHPStan Error Resolution Session 7: Return Type Declarations

## Summary
- Baseline: 6,805 errors
- After: 6,726 errors
- Reduction: 79 errors

## Changes Made

### Database Actions (13 files)
- StartClickhouse, StartDragonfly, StartKeydb, StartMariadb, StartMongodb,
  StartMysql, StartPostgresql, StartRedis: Added Activity return types and
  PHPDoc annotations for array return types on helper methods
- RestartDatabase, StartDatabase: Added string|Activity|null return types
- StopDatabase: Already had return types (verified)
- StartDatabaseProxy, StopDatabaseProxy: Added void return types

### Application Actions (5 files)
- GenerateConfig: Added string|array return type
- LoadComposeFile: Added void return type
- StopApplication, StopApplicationOneServer: Added ?string return types
- IsHorizonQueueEmpty: Added bool return type

### Service Actions (4 files)
- StartService, RestartService: Added Activity return types
- StopService: Added ?string return type
- DeleteService: Added void return type

### Server Actions (14 files)
- CheckUpdates: Added array return type with PHPDoc
- CleanupDocker: Added array return type with PHPDoc
- DeleteServer, ResourcesCheck, RestartContainer, StopSentinel,
  StartSentinel, UpdateCoolify: Added void return types
- InstallDocker, InstallPrerequisites, RunCommand: Added Activity return types
- StartLogDrain: Added string|Activity return type
- StopLogDrain: Added ?string return type
- ValidateServer: Added string return type

### Proxy Actions (1 file)
- StopProxy: Added void return type

### Shared Actions (1 file)
- ComplexStatusCheck: Added void return type

### Jobs (16 files)
- Added void return types to all Job handle() methods
- Fixed return statements in void functions (changed 'return value' to 'return')

## Technical Notes
- Used Spatie\Activitylog\Contracts\Activity for remote_process() returns
- Nullable return types (?Activity) used where SSL errors cause early returns
- PHPDoc @return annotations added for complex array types

Relates to #203

Author: johnproblems

app/Actions/Application/GenerateConfig.php

@@ -9,7 +9,7 @@ class GenerateConfig
 {
     use AsAction;
 
-    public function handle(Application $application, bool $is_json = false)
+    public function handle(Application $application, bool $is_json = false): string|array
     {
         return $application->generateConfig(is_json: $is_json);
     }

app/Actions/Application/IsHorizonQueueEmpty.php

@@ -9,7 +9,7 @@ class IsHorizonQueueEmpty
 {
     use AsAction;
 
-    public function handle()
+    public function handle(): bool
     {
         $hostname = gethostname();
         $recent = app(JobRepository::class)->getRecent();

app/Actions/Application/LoadComposeFile.php

@@ -9,7 +9,7 @@ class LoadComposeFile
 {
     use AsAction;
 
-    public function handle(Application $application)
+    public function handle(Application $application): void
     {
         $application->loadComposeFile();
     }

app/Actions/Application/StopApplication.php

@@ -13,7 +13,7 @@ class StopApplication
 
     public string $jobQueue = 'high';
 
-    public function handle(Application $application, bool $previewDeployments = false, bool $dockerCleanup = true)
+    public function handle(Application $application, bool $previewDeployments = false, bool $dockerCleanup = true): ?string
     {
         $servers = collect([$application->destination->server]);
         if ($application?->additional_servers?->count() > 0) {
@@ -28,7 +28,7 @@ public function handle(Application $application, bool $previewDeployments = fals
                 if ($server->isSwarm()) {
                     instant_remote_process(["docker stack rm {$application->uuid}"], $server);
 
-                    return;
+                    return null;
                 }
 
                 $containers = $previewDeployments
@@ -56,5 +56,7 @@ public function handle(Application $application, bool $previewDeployments = fals
             }
         }
         ServiceStatusChanged::dispatch($application->environment->project->team->id);
+
+        return null;
     }
 }

app/Actions/Application/StopApplicationOneServer.php

@@ -10,10 +10,10 @@ class StopApplicationOneServer
 {
     use AsAction;
 
-    public function handle(Application $application, Server $server)
+    public function handle(Application $application, Server $server): ?string
     {
         if ($application->destination->server->isSwarm()) {
-            return;
+            return null;
         }
         if (! $server->isFunctional()) {
             return 'Server is not functional';
@@ -37,5 +37,7 @@ public function handle(Application $application, Server $server)
         } catch (\Exception $e) {
             return $e->getMessage();
         }
+
+        return null;
     }
 }

app/Actions/Database/RestartDatabase.php

@@ -11,12 +11,13 @@
 use App\Models\StandalonePostgresql;
 use App\Models\StandaloneRedis;
 use Lorisleiva\Actions\Concerns\AsAction;
+use Spatie\Activitylog\Contracts\Activity;
 
 class RestartDatabase
 {
     use AsAction;
 
-    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse $database)
+    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse $database): string|Activity|null
     {
         $server = $database->destination->server;
         if (! $server->isFunctional()) {

app/Actions/Database/StartClickhouse.php

@@ -4,6 +4,7 @@
 
 use App\Models\StandaloneClickhouse;
 use Lorisleiva\Actions\Concerns\AsAction;
+use Spatie\Activitylog\Contracts\Activity;
 use Symfony\Component\Yaml\Yaml;
 
 class StartClickhouse
@@ -16,7 +17,7 @@ class StartClickhouse
 
     public string $configuration_dir;
 
-    public function handle(StandaloneClickhouse $database)
+    public function handle(StandaloneClickhouse $database): Activity
     {
         $this->database = $database;
 
@@ -113,7 +114,10 @@ public function handle(StandaloneClickhouse $database)
         return remote_process($this->commands, $database->destination->server, callEventOnFinish: 'DatabaseStatusChanged');
     }
 
-    private function generate_local_persistent_volumes()
+    /**
+     * @return array<int, string>
+     */
+    private function generate_local_persistent_volumes(): array
     {
         $local_persistent_volumes = [];
         foreach ($this->database->persistentStorages as $persistentStorage) {
@@ -128,7 +132,10 @@ private function generate_local_persistent_volumes()
         return $local_persistent_volumes;
     }
 
-    private function generate_local_persistent_volumes_only_volume_names()
+    /**
+     * @return array<string, array<string, bool|string>>
+     */
+    private function generate_local_persistent_volumes_only_volume_names(): array
     {
         $local_persistent_volumes_names = [];
         foreach ($this->database->persistentStorages as $persistentStorage) {
@@ -145,7 +152,10 @@ private function generate_local_persistent_volumes_only_volume_names()
         return $local_persistent_volumes_names;
     }
 
-    private function generate_environment_variables()
+    /**
+     * @return array<int, string>
+     */
+    private function generate_environment_variables(): array
     {
         $environment_variables = collect();
         foreach ($this->database->runtime_environment_variables as $env) {

app/Actions/Database/StartDatabase.php

@@ -11,14 +11,15 @@
 use App\Models\StandalonePostgresql;
 use App\Models\StandaloneRedis;
 use Lorisleiva\Actions\Concerns\AsAction;
+use Spatie\Activitylog\Contracts\Activity;
 
 class StartDatabase
 {
     use AsAction;
 
     public string $jobQueue = 'high';
 
-    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse $database)
+    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse $database): string|Activity|null
     {
         $server = $database->destination->server;
         if (! $server->isFunctional()) {

app/Actions/Database/StartDatabaseProxy.php

@@ -20,7 +20,7 @@ class StartDatabaseProxy
 
     public string $jobQueue = 'high';
 
-    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse|ServiceDatabase $database)
+    public function handle(StandaloneRedis|StandalonePostgresql|StandaloneMongodb|StandaloneMysql|StandaloneMariadb|StandaloneKeydb|StandaloneDragonfly|StandaloneClickhouse|ServiceDatabase $database): void
     {
         $databaseType = $database->database_type;
         $network = data_get($database, 'destination.network');

app/Actions/Database/StartDragonfly.php

@@ -6,6 +6,7 @@
 use App\Models\SslCertificate;
 use App\Models\StandaloneDragonfly;
 use Lorisleiva\Actions\Concerns\AsAction;
+use Spatie\Activitylog\Contracts\Activity;
 use Symfony\Component\Yaml\Yaml;
 
 class StartDragonfly
@@ -20,7 +21,7 @@ class StartDragonfly
 
     private ?SslCertificate $ssl_certificate = null;
 
-    public function handle(StandaloneDragonfly $database)
+    public function handle(StandaloneDragonfly $database): ?Activity
     {
         $this->database = $database;
 
@@ -65,7 +66,7 @@ public function handle(StandaloneDragonfly $database)
             if (! $caCert) {
                 $this->dispatch('error', 'No CA certificate found for this database. Please generate a CA certificate for this server in the server/advanced page.');
 
-                return;
+                return null;
             }
 
             $this->ssl_certificate = $this->database->sslCertificates()->first();
@@ -217,7 +218,10 @@ private function buildStartCommand(): string
         return $command;
     }
 
-    private function generate_local_persistent_volumes()
+    /**
+     * @return array<int, string>
+     */
+    private function generate_local_persistent_volumes(): array
     {
         $local_persistent_volumes = [];
         foreach ($this->database->persistentStorages as $persistentStorage) {
@@ -232,7 +236,10 @@ private function generate_local_persistent_volumes()
         return $local_persistent_volumes;
     }
 
-    private function generate_local_persistent_volumes_only_volume_names()
+    /**
+     * @return array<string, array<string, bool|string>>
+     */
+    private function generate_local_persistent_volumes_only_volume_names(): array
     {
         $local_persistent_volumes_names = [];
         foreach ($this->database->persistentStorages as $persistentStorage) {
@@ -249,7 +256,10 @@ private function generate_local_persistent_volumes_only_volume_names()
         return $local_persistent_volumes_names;
     }
 
-    private function generate_environment_variables()
+    /**
+     * @return array<int, string>
+     */
+    private function generate_environment_variables(): array
     {
         $environment_variables = collect();
         foreach ($this->database->runtime_environment_variables as $env) {