TASK-1623 — Hotfix upload image Livewire stockage distant

[cslucki]

ROOT_CAUSE — mesurée, pas supposée

Sur un environnement où Livewire stocke ses fichiers temporaires sur un disque distant, tout upload d'image levait :

Intervention\Image\Exceptions\DirectoryNotFoundException
The directory "livewire-tmp" contained in SplFileInfo does not exist

Le mécanisme tient en deux lignes de vendor :

// TemporaryUploadedFile
public function getPathname(): string { return $this->storage->path($this->path); }
// FilesystemAdapter:282
public function path($path) { return $this->prefixer->prefixPath($path); }

path() ne fait que préfixer avec la racine du disque. Absolue en local → le chemin existe. Vide sur s3/gcs → le chemin rendu est relatif (livewire-tmp/xxx.png). Intervention résout un SplFileInfo par son getPathname(), teste is_dir(dirname($path)), et lève.

Pourquoi aucun test existant ne POUVAIT l'attraper

Livewire::test(...)->set('image', $f) passe par FileUploadConfiguration::disk(), qui rend 'tmp-for-tests' en dur dès que l'app tourne en test, et storage() en fait un Storage::fake() — donc un disque local. Le harnais de test de Livewire ne peut structurellement pas produire la condition de production.

Le banc construit donc un vrai TemporaryUploadedFile sur un disque dont le FilesystemAdapter reçoit root vide : path() rend un chemin relatif, comme s3, sans réseau, tandis que le contenu est réellement servi.

FIX

Quatre corrections d'une ligne : Image::decode($file) → Image::decode($file->get()). Le contenu, jamais le chemin. C'est l'idiome déjà en place (GenerateServiceThumbnail lit son image par $disk->get($path)). Pas de service générique, pas d'abstraction, pas de refactor.

Hors scope, vérifié et non modifié : ServiceController et ProfileController reçoivent un UploadedFile d'une requête HTTP classique, dans le tmp local de PHP — pas le même défaut.

Mesures

Banc avant fix	5 rouges, 2 verts
Banc après fix	7 verts (27 assertions)
Voisins directs	199 verts, 1 rouge #[Group('ci-known-red')] pré-existant sur develop (rejoué sur le fichier nu pour le prouver)
PostgreSQL	118 verts (286 assertions)
Sabotage	redonner l'objet fichier fait rougir la garde de non-retour

Recette navigateur réelle

ChatLoop et création d'annonce sur https://test.laravel : image jointe, aperçu, envoi, RIFF Web/P VP8 640x400 produit dans les deux cas. Zéro erreur console, zéro erreur Laravel. Artefacts de recette supprimés en base.

L'environnement local utilise un disque temporaire local : la recette navigateur couvre les critères 1, 3, 4 et 6 ; le critère 2 (stockage distant) est couvert par le banc, seul endroit où la condition soit reproductible sans S3.

Multi-tenant

Aucune ligne touchée. Organization = Tenant, Loop != Tenant, community_id = dette legacy temporaire, aucune migration Community → Organization.

🤖 Generated with Claude Code

https://claude.ai/code/session_01JqwjarKwhSbpzdPxLAmNh3