Merge pull request #373 from fsprojects/repo-assist/perf-chunkby-reuse-buffer-20260331-571b7516713643ae

[Repo Assist] perf: reuse ResizeArray buffer in chunkBy and chunkByAsync

Author: dsyme

release-notes.txt

@@ -2,6 +2,7 @@
 Release notes:
 
 1.0.0
+    - perf: TaskSeq.chunkBy and chunkByAsync reuse the ResizeArray buffer between chunks, reducing allocations on sequences with many chunk boundaries
     - fixes: TaskSeq.insertAt, insertManyAt, removeAt, removeManyAt, updateAt now raise ArgumentNullException (not NullReferenceException) when given a null source; insertManyAt also validates the values argument
     - refactor: simplify lengthBy and lengthBeforeMax to use while! and remove the redundant mutable 'go' and initial MoveNextAsync
     - adds TaskSeq.distinctUntilChangedWith and TaskSeq.distinctUntilChangedWithAsync, #345

src/FSharp.Control.TaskSeq/TaskSeqInternal.fs

@@ -1754,7 +1754,7 @@ module internal TaskSeqInternal =
                         currentChunk.Add item
                     else
                         yield prevKey, currentChunk.ToArray()
-                        currentChunk <- ResizeArray<'T>()
+                        currentChunk.Clear() // reuse backing array; ToArray() already captured a snapshot
                         currentChunk.Add item
                         maybeCurrentKey <- ValueSome key
 
@@ -1782,7 +1782,7 @@ module internal TaskSeqInternal =
                         currentChunk.Add item
                     else
                         yield prevKey, currentChunk.ToArray()
-                        currentChunk <- ResizeArray<'T>()
+                        currentChunk.Clear() // reuse backing array; ToArray() already captured a snapshot
                         currentChunk.Add item
                         maybeCurrentKey <- ValueSome key