<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

/**
 * Quién hizo cada cosa, en una columna.
 *
 * La tabla `logs` guardaba `level`, `code`, `msg`, `entity`, `entity_id`, `trace` e `ip`,
 * y **el usuario responsable iba dentro del texto del mensaje** —«… (por Antonio
 * Manzano)»—, porque cada sitio que escribe un log lo iba concatenando a mano.
 *
 * Eso hace tres cosas mal:
 *
 * 1. **No se puede filtrar** por persona, que es la primera pregunta al revisar un
 *    registro de acciones: «¿qué ha tocado esta cuenta?».
 * 2. **No distingue una persona de una tarea programada.** Un log escrito por
 *    `licences:check-expiry` y uno escrito por alguien a las tres de la tarde se leen
 *    igual, y la diferencia importa.
 * 3. Depende de que quien escriba el log se acuerde de poner el nombre, y no todos lo
 *    hacen.
 *
 * `nullOnDelete`: si la cuenta se borra, la entrada del registro **no desaparece** —lo
 * que importa es que la acción ocurrió—. El nombre sigue además en el texto de los logs
 * antiguos, que no se reescriben: no se puede reconstruir con fiabilidad quién era, y
 * adivinarlo parseando el mensaje sería peor que dejarlo vacío.
 */
return new class extends Migration
{
    public function up(): void
    {
        Schema::table('logs', function (Blueprint $table) {
            $table->foreignId('user_id')
                ->nullable()
                ->after('code')
                ->constrained('users')
                ->nullOnDelete();

            // Se filtra por persona y se ordena por fecha: el índice cubre las dos cosas.
            $table->index(['user_id', 'created_at']);
        });
    }

    public function down(): void
    {
        Schema::table('logs', function (Blueprint $table) {
            $table->dropIndex(['user_id', 'created_at']);
            $table->dropConstrainedForeignId('user_id');
        });
    }
};