SomeExamples.cs

// Filter only active items
record Item(DateTime ActiveUntil);
IReadOnlyCollection<Item> GetActiveItems(IEnumerable<Item> items) => items.Where(item => item.ActiveUntil > DateTime.Now).ToArray();

/* ******************************************************************** */

// Robustly perform an [idempotent] request until timeout is reached
void PerformRequestRobustly(TimeSpan timeout)
{
    var retryUntil = DateTime.UtcNow + timeout;
    while (true)
    {
        try
        {
            PerformRequest();
            break;
        }
        catch (FailedRequestException) when (DateTime.UtcNow < retryUntil)
        {
        }
    }
}

/* ******************************************************************** */

// This is a piece of library code. Is this acceptable?
async Task RobustlyCall(Func<Task> inner)
{
    if (inner is null)
    {
        throw new ArgumentNullException(nameof(inner));
    }
    
    try
    {
        await inner();
    }
    catch
    {
    }
}

/* ******************************************************************** */

// Is this any better? Or worse?
Task RobustlyCall(Func<Task> inner)
{
    if (inner is null)
    {
        throw new ArgumentNullException(nameof(inner));
    }
    
    try
    {
        return inner();
    }
    catch
    {
        return Task.CompletedTask;
    }
}