// Inject ILogger<T> via constructor
private readonly ILogger<TankService> _logger;

public TankService(ILogger<TankService> logger)
{
    _logger = logger;
}

// Use semantic message templates - property names in braces
_logger.LogInformation("Tank {TankId} updated to {Level}%", tankId, level);
_logger.LogWarning("Tank {TankName} below threshold: {CurrentLevel}% < {Threshold}%", 
    tank.Name, tank.CurrentLevel, tank.AlertLevel);

try
{
    await plugin.SetStateAsync(controlId, state, ct);
}
catch (Exception ex)
{
    // Exception as FIRST parameter, then message template
    _logger.LogError(ex, "Failed to set state for control {ControlId}", controlId);
    throw;
}

public async Task<Tank> UpdateTankAsync(Tank tank, CancellationToken ct)
{
    _logger.LogDebug("Updating tank {TankId}: {TankName}", tank.Id, tank.Name);
    
    // Business logic...
    
    _logger.LogInformation("Tank {TankId} saved successfully", tank.Id);
    return tank;
}

public async Task InitializeAsync(Dictionary<string, object> config, CancellationToken ct)
{
    _logger.LogInformation("Initializing {PluginName} v{Version}", Name, Version);
    
    if (!config.TryGetValue("MqttBroker", out var broker))
    {
        _logger.LogWarning("{PluginName} missing MqttBroker config, using default", Name);
    }
}