[Bug] VipsDriver save() without path fails and format() is ignored

[dimitrykostin]

Hi! Found a bug in spatie/image Vips driver (v3.9.1).

Environment

• spatie/image: 3.9.1
• php-vips: 2.6.1
• PHP: 8.5

Problem

save() without path fails after loadFile().

Image::useImageDriver(\Spatie\Image\Drivers\Vips\VipsDriver::class)
    ->loadFile('storage/watermarks/watermark.png')
    ->save();

Actual: UnsupportedImageFormat: Unsupported format 'unknown'.
Expected: same behavior as GD/Imagick (save back to loaded path).

Why it matters
In Laravel Media Library pipeline, save() is often called without explicit path, so manipulations (watermark/format) may be dropped in queued conversions.
I can open a PR with tests if this behavior should be fixed.

looks similar to #228/#236, but affects VipsDriver

[freekmurze]

This has been fixed in 3.9.2. The VipsDriver was missing originalPath tracking and format fallback logic that GdDriver and ImagickDriver both have.

You can update with:

composer update spatie/image

[dimitrykostin]

Thanks for 3.9.2 — I can confirm save() without path is fixed.

However, there is still one Vips issue:
format('webp') is ignored when saving to a .jpg path.

Repro (VipsDriver):

use Spatie\Image\Image;
use Spatie\Image\Drivers\Vips\VipsDriver;

$src = 'storage/watermarks/watermark.png';
$path = tempnam(sys_get_temp_dir(), 'fmt-') . '.jpg';
copy($src, $path);

Image::useImageDriver(VipsDriver::class)
    ->loadFile($path)
    ->format('webp')
    ->save($path);

$data = file_get_contents($path);

echo 'is_webp=' . ((substr($data, 0, 4) === 'RIFF' && substr($data, 8, 4) === 'WEBP') ? '1' : '0') . PHP_EOL;
echo 'is_jpeg=' . ((substr($data, 0, 2) === "\xFF\xD8") ? '1' : '0') . PHP_EOL;

Output: is_webp=0 is_jpeg=1
Control check (ImagickDriver) with same repro: is_webp=1 is_jpeg=0
So 3.9.2 fixed originalPath fallback, but explicit format() still appears not to be respected by Vips when path extension differs.

[freekmurze]

This has been fixed in 3.9.3. The VipsDriver now uses writeToBuffer() with the correct format suffix when format() differs from the path extension.

You can update with:

composer update spatie/image

[dimitrykostin]

Confirmed, fixed in 3.9.3. Thanks.