FileStorage->cleanup() missing oldest file

[ssnepenthe]

I noticed this on my own custom implementation but just confirmed the same is happening on a fresh laravel install with the latest version of clockwork.

When using file storage with an expiration of 0 and calling ->cleanup(true) (i.e. running php artisan clockwork:clean --all), the index is cleared out and all of the metadata files are removed except for the oldest one.

Expected behavior is for all metadata files to be removed.

To reproduce, create a fresh laravel install, install clockwork, run the dev server, visit the site and refresh a few times to generate a few metadata files and then inspect the storage/clockwork directory.

Now run php artisan clockwork:clean --all and re-inspect the storage/clockwork directory: