GetRecordAsync

[sandrosxila]

CsvHelper has a GetRecordsAsync method to get all records asynchronously. It works fine, but if there's an error in an uploaded CSV file, it throws an exception about the first error and doesn't continue reading other rows to get all errors. It would be convenient if we had a GetRecordAsync function that would work like an async GetRecord.

I implement my method like this

    ...
    public class ParsedRecord<T>(T? record, bool isValid, string errorMessage) : ICsvFileImportService.IParsedRecord<T>
    {
        public T? Record { get; set; } = record;
        public bool IsValid { get; set; } = isValid;
        public string ErrorMessage { get; set; } = errorMessage;
    }
    
    public async IAsyncEnumerable<TParsedRecord> ReadRecordsAsync<TRecord, TParsedRecord>(
        Stream stream,
        ClassMap<TRecord> classMap,
        [EnumeratorCancellation] CancellationToken cancellationToken = default
    ) where TParsedRecord : ICsvFileImportService.IParsedRecord<TRecord>
    {
        using var reader = new StreamReader(stream);
        using var csvReader = new CsvReader(reader, _csvConfiguration);
        csvReader.Context.TypeConverterCache.AddConverter<bool>(new CommonValuesBooleanConverter());
        csvReader.Context.RegisterClassMap(classMap);

        if (!await csvReader.ReadAsync().ConfigureAwait(false))
        {
            yield break;
        }
        
        csvReader.ReadHeader();
        csvReader.ValidateHeader<TRecord>();

        while (await csvReader.ReadAsync().ConfigureAwait(false))
        {
            cancellationToken.ThrowIfCancellationRequested();
            TParsedRecord parsedRecord;
            
            try
            {
                TRecord record = csvReader.GetRecord<TRecord>();
                parsedRecord = (TParsedRecord)Activator.CreateInstance(typeof(TParsedRecord), record, true, string.Empty)!;
            } catch (Exception ex)
            {
                parsedRecord = (TParsedRecord)Activator.CreateInstance(typeof(TParsedRecord), null, false, ex.Message)!;
            }
            
            yield return parsedRecord;
        }
    }
    ....

But I'm not sure if it's the correct way to do that.