Feat/cron job names (#994)

* Optional cron job naming added

* Fix: CronJob model, test and form formatting

Author: tomasdav9

app/Actions/CronJob/CreateCronJob.php

@@ -29,6 +29,7 @@ public function create(Server $server, array $input, ?Site $site = null): CronJo
         }
 
         $cronJob = new CronJob([
+            'name' => $input['name'] ?? null,
             'server_id' => $server->id,
             'site_id' => $siteId,
             'user' => $input['user'],
@@ -59,6 +60,11 @@ private function validate(array $input, Server $server, ?Site $site = null): voi
                 'required',
                 new CronRule(acceptCustom: true),
             ],
+            'name' => [
+                'nullable',
+                'string',
+                'max:255',
+            ],
         ];
 
         // Add site_id validation if provided in input

app/Actions/CronJob/EditCronJob.php

@@ -38,6 +38,7 @@ public function edit(Server $server, CronJob $cronJob, array $input, ?Site $site
 
         $cronJob->update([
             'site_id' => $siteId,
+            'name' => $input['name'] ?? null,
             'user' => $input['user'],
             'command' => $input['command'],
             'frequency' => $input['frequency'] == 'custom' ? $input['custom'] : $input['frequency'],
@@ -72,6 +73,11 @@ private function validate(array $input, Server $server, ?Site $site = null): voi
                 'required',
                 new CronRule(acceptCustom: true),
             ],
+            'name' => [
+                'nullable',
+                'string',
+                'max:255',
+            ],
         ];
 
         // Add site_id validation if provided in input

app/Http/Resources/CronJobResource.php

@@ -16,6 +16,7 @@ public function toArray(Request $request): array
     {
         return [
             'id' => $this->id,
+            'name' => $this->name,
             'server_id' => $this->server_id,
             'site_id' => $this->site_id,
             'command' => $this->command,

app/Models/CronJob.php

@@ -14,6 +14,7 @@
  * @property string $user
  * @property string $frequency
  * @property bool $hidden
+ * @property string|null $name
  * @property CronjobStatus $status
  * @property string $crontab
  * @property Server $server
@@ -32,6 +33,7 @@ class CronJob extends AbstractModel
         'frequency',
         'hidden',
         'status',
+        'name',
     ];
 
     protected $casts = [

database/factories/CronJobFactory.php

@@ -16,6 +16,7 @@ class CronJobFactory extends Factory
     public function definition(): array
     {
         return [
+            'name' => $this->faker->optional()->sentence(3),
             'server_id' => 1,
             'site_id' => null,
             'command' => 'ls -la',

database/migrations/2026_01_09_170832_add_name_to_cron_jobs_table.php

@@ -0,0 +1,28 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    /**
+     * Run the migrations.
+     */
+    public function up(): void
+    {
+        Schema::table('cron_jobs', function (Blueprint $table) {
+            $table->string('name')->nullable()->index();
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     */
+    public function down(): void
+    {
+        Schema::table('cron_jobs', function (Blueprint $table) {
+            $table->dropColumn('name');
+        });
+    }
+};

resources/js/pages/cronjobs/components/columns.tsx

@@ -96,6 +96,15 @@ function Delete({ cronJob, site }: { cronJob: CronJob; site?: Site }) {
 
 function getColumns(site?: Site, sites?: Array<{ id: number; domain: string }>): ColumnDef<CronJob>[] {
   return [
+    {
+      accessorKey: 'name',
+      header: 'Name',
+      enableColumnFilter: true,
+      enableSorting: true,
+      cell: ({ row }) => {
+        return <span>{row.original.name || <i className="text-muted-foreground">No name</i>}</span>;
+      },
+    },
     {
       accessorKey: 'command',
       header: 'Command',

resources/js/pages/cronjobs/components/form.tsx

@@ -36,12 +36,14 @@ export default function CronJobForm({
   const page = usePage<SharedData & { server: Server; sites?: Array<{ id: number; domain: string }> }>();
   const [open, setOpen] = useState(false);
   const form = useForm<{
+    name: string;
     command: string;
     user: string;
     frequency: string;
     custom: string;
     site_id: string;
   }>({
+    name: cronJob?.name || '',
     command: cronJob?.command || '',
     user: cronJob?.user || '',
     frequency: cronJob ? (page.props.configs.cronjob_intervals[cronJob.frequency] ? cronJob.frequency : 'custom') : '',
@@ -84,6 +86,20 @@ export default function CronJobForm({
         </DialogHeader>
         <Form id="cronjob-form" onSubmit={submit} className="p-4">
           <FormFields>
+            {/* Name */}
+            <FormField>
+              <Label htmlFor="name">Name</Label>
+              <Input
+                type="text"
+                id="name"
+                value={form.data.name}
+                onChange={(e) => form.setData('name', e.target.value)}
+                placeholder="Optional name for the cron job"
+              />
+              <InputError message={form.errors.name} />
+            </FormField>
+
+            {/* Command */}
             <FormField>
               <Label htmlFor="command">Command</Label>
               <Input type="text" id="command" value={form.data.command} onChange={(e) => form.setData('command', e.target.value)} />

resources/js/types/cronjob.d.ts

@@ -1,5 +1,6 @@
 export interface CronJob {
   id: number;
+  name: string | null;
   server_id: number;
   site_id: number | null;
   command: string;

tests/Feature/API/SiteCronjobTest.php

@@ -74,6 +74,7 @@ public function test_create_site_cronjob(): void
             'command' => 'ls -la',
             'user' => 'vito',
             'frequency' => '* * * * *',
+            'name' => 'My Site Cronjob',
         ])
             ->assertSuccessful()
             ->assertJsonFragment([
@@ -91,6 +92,7 @@ public function test_create_site_cronjob(): void
             'user' => 'vito',
             'frequency' => '* * * * *',
             'status' => CronjobStatus::READY,
+            'name' => 'My Site Cronjob',
         ]);
     }