spatie / laravel-db-snapshots

src/Commands/Delete.php

<?php

namespace Spatie\DbSnapshots\Commands;

use Illuminate\Console\Command;
use Spatie\DbSnapshots\Commands\Concerns\AsksForSnapshotName;
use Spatie\DbSnapshots\SnapshotRepository;

class Delete extends Command
{
    use AsksForSnapshotName;

    protected $signature = 'snapshot:delete {name?}';

    protected $description = 'Delete a snapshot.';

    public function handle()
    {
        $snapShots = app(SnapshotRepository::class)->getAll();

The assignment to $snapShots is dead and can be removed.

        if (app(SnapshotRepository::class)->getAll()->isEmpty()) {
            $this->warn('No snapshots found. Run `snapshot:create` to create snapshots.');

            return;
        }

        $name = $this->argument('name') ?: $this->askForSnapshotName();

        $snapshot = app(SnapshotRepository::class)->findByName($name);

It seems like $name can also be of type string[]; however, parameter $name of Spatie\DbSnapshots\SnapshotRepository::findByName() does only seem to accept string, maybe add an additional type check?

        if (! $snapshot) {
            $this->warn("Snapshot `{$name}` does not exist!");

            return;
        }

        $snapshot->delete();

        $this->info("Snapshot `{$snapshot->name}` deleted!");
    }
}