Add DeserializeEnumerableAsync<T> methods for streaming sequences

[AArnott]

See #237

[Copilot]

Copilot reviewed 5 out of 13 changed files in this pull request and generated 1 comment.

Files not reviewed (8)

• src/Nerdbank.MessagePack/MessagePackSerializer.AutomatedFriendlyOverloads.tt: Language not supported
• src/Nerdbank.MessagePack/net8.0/PublicAPI.Unshipped.txt: Language not supported
• src/Nerdbank.MessagePack/netstandard2.0/PublicAPI.Unshipped.txt: Language not supported
• docfx/docs/features.md: Evaluated as low risk
• docfx/docs/migrating.md: Evaluated as low risk
• docfx/docs/streaming-deserialization.md: Evaluated as low risk
• docfx/docs/toc.yml: Evaluated as low risk
• samples/StreamingDeserialization.cs: Evaluated as low risk

Comments suppressed due to low confidence (4)

src/Nerdbank.MessagePack/MessagePackSerializer.cs:470

• [nitpick] The XML documentation comments for the parameters 'reader', 'shape', and 'cancellationToken' are missing. Please add these comments for clarity.

public IAsyncEnumerable<T?> DeserializeEnumerableAsync<T>(PipeReader reader, ITypeShape<T> shape, CancellationToken cancellationToken = default)

src/Nerdbank.MessagePack/MessagePackSerializer.cs:475

• [nitpick] The XML documentation comments for the parameters 'reader', 'provider', and 'cancellationToken' are missing. Please add these comments for clarity.

public IAsyncEnumerable<T?> DeserializeEnumerableAsync<T>(PipeReader reader, ITypeShapeProvider provider, CancellationToken cancellationToken = default)

test/Nerdbank.MessagePack.Tests/AsyncSerializationTests.cs:129

• The test method DeserializeEnumerableAsync_Array is marked as skipped. Please implement this test or remove it if it's not needed.

[Fact(Skip = "Not ready")]

src/Nerdbank.MessagePack/MessagePackAsyncReader.cs:218

• [nitpick] The method name 'GetIsEndOfStreamAsync' could be simplified to 'IsEndOfStreamAsync' for better readability.

internal async ValueTask<bool> GetIsEndOfStreamAsync()

[codecov]

Codecov Report

Attention: Patch coverage is 78.12500% with 7 lines in your changes missing coverage. Please review.

Project coverage is 84.32%. Comparing base (997b760) to head (e5e693e).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...agePack/MessagePackSerializer.FriendlyOverloads.cs	0.00%	5 Missing ⚠️
src/Nerdbank.MessagePack/MessagePackSerializer.cs	90.00%	1 Missing ⚠️
...dbank.MessagePack.Tests/AsyncSerializationTests.cs	92.30%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   84.34%   84.32%   -0.02%     
==========================================
  Files         164      164              
  Lines       12907    12939      +32     
  Branches     1405     1410       +5     
==========================================
+ Hits        10886    10911      +25     
- Misses       1992     1999       +7     
  Partials       29       29              

Flag	Coverage Δ
Linux	82.52% <78.12%> (-0.04%)	⬇️
Windows	84.31% <78.12%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.