fs.createReadStream throws with start parameter

[Indidev]

Apparently the node implementation of Buffer.copy changed somewhere between v18.12 and v18.20, which leads to all copy calls with sourceStart > buffer.length throwing an Error.

This affects the fs.createReadStream method when called with a start parameter:

Example:

Replace the code inside of createReadstream.ts in the demo folder to:

import { fs } from '../src/index';

fs.writeFileSync('/streamTest', '# Hello World');
const rs = fs.createReadStream('/streamTest', {encoding: 'utf8', start: 0});
rs.on('data', data => {
  console.log(`data: "${data}"`)
});

Run with ts-node demo/createReadStream.ts (or transpile and run with node) results in:

 ts-node demo/createReadStream.ts 
data # Hello World
RangeError [ERR_OUT_OF_RANGE]: The value of "sourceStart" is out of range. It must be >= 0 && <= 13. Received 65536
    at new NodeError (node:internal/errors:405:5)
    at _copy (node:buffer:227:13)
    at Buffer.copy (node:buffer:821:12)
    at Node.read (/home/breitdo/dev/git/test/memfs/src/node.ts:195:14)
    at File.read (/home/breitdo/dev/git/test/memfs/src/node.ts:542:29)
    at Volume.readBase (/home/breitdo/dev/git/test/memfs/src/volume.ts:861:17)
    at Immediate.<anonymous> (/home/breitdo/dev/git/test/memfs/src/volume.ts:899:28)
    at processImmediate (node:internal/timers:476:21) {
  code: 'ERR_OUT_OF_RANGE'
}

Issue:
The culprit is in src/node.ts:194:

this.buf.copy(buf2, off, pos, pos + actualLen);
// First call: pos = 0, actualLen = 13
// Second call: pos = 65536 (default chunk size), actualLen = -65523 (buffer.length - pos)

Possible fix:
Inserting another check after line 185 fixes this issue:

if (pos >= this.buf.length) return 0;