fix: Accept ReadableStreamDefaultReader instead of ReadableStream

[Copilot]

Update functions to accept ReadableStreamDefaultReader

Plan:

• Update countRows function to accept ReadableStreamDefaultReader<Uint8Array> instead of ReadableStream<Uint8Array>
• Update chunk function to accept ReadableStreamDefaultReader<Uint8Array> instead of ReadableStream<Uint8Array>
• Remove getReader() calls and releaseLock() calls from both functions
• Update all tests to create a reader from the stream before passing to functions
• Update README examples to show getting a reader first
• Rebuild and verify all tests pass
• Run linter to ensure code quality
• Refactor to use async iteration (for await...of) instead of while(true) loops

Changes Made:

• Modified countRows() to accept ReadableStreamDefaultReader<Uint8Array> as the first parameter
• Modified chunk() to accept ReadableStreamDefaultReader<Uint8Array> as the first parameter
• Removed try/finally blocks and releaseLock() calls since the reader is no longer owned by the functions
• Updated test helper from createStreamFromString() to createReaderFromString() to return a reader
• Updated all 22 tests to use the new helper function
• Updated README documentation with new API and examples
• Added makeReaderIterable() helper function to enable async iteration over readers
• Refactored both functions to use for await...of loops instead of manual while(true) with reader.read()
• All tests pass (22/22)
• Linter passes with no errors

Original prompt

Update the functions to take a ReadableStreamDefaultReader instead of a ReadableStream. Be sure to update tests and the README.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[nzakas]

@copilot let's use async iteration (for await...of) instead of while(true);

[Copilot]

Done. Added a makeReaderIterable() helper function to enable async iteration and refactored to use for await...of (ca57ec6).

[nzakas]

@copilot let's use async iteration here too.

[Copilot]

Done. Both functions now use for await...of with the makeReaderIterable() helper (ca57ec6).