feat(fslib): add FileHandle.readLines (#5121)

Author: merceyz

.pnp.cjs

@@ -0,0 +1,39 @@
+releases:
+  "@yarnpkg/cli": patch
+  "@yarnpkg/core": patch
+  "@yarnpkg/fslib": minor
+  "@yarnpkg/plugin-pnp": patch
+  "@yarnpkg/pnp": patch
+
+declined:
+  - "@yarnpkg/plugin-compat"
+  - "@yarnpkg/plugin-constraints"
+  - "@yarnpkg/plugin-dlx"
+  - "@yarnpkg/plugin-essentials"
+  - "@yarnpkg/plugin-exec"
+  - "@yarnpkg/plugin-file"
+  - "@yarnpkg/plugin-git"
+  - "@yarnpkg/plugin-github"
+  - "@yarnpkg/plugin-http"
+  - "@yarnpkg/plugin-init"
+  - "@yarnpkg/plugin-interactive-tools"
+  - "@yarnpkg/plugin-link"
+  - "@yarnpkg/plugin-nm"
+  - "@yarnpkg/plugin-npm"
+  - "@yarnpkg/plugin-npm-cli"
+  - "@yarnpkg/plugin-pack"
+  - "@yarnpkg/plugin-patch"
+  - "@yarnpkg/plugin-pnpm"
+  - "@yarnpkg/plugin-stage"
+  - "@yarnpkg/plugin-typescript"
+  - "@yarnpkg/plugin-version"
+  - "@yarnpkg/plugin-workspace-tools"
+  - vscode-zipfs
+  - "@yarnpkg/builder"
+  - "@yarnpkg/doctor"
+  - "@yarnpkg/extensions"
+  - "@yarnpkg/libzip"
+  - "@yarnpkg/nm"
+  - "@yarnpkg/pnpify"
+  - "@yarnpkg/sdks"
+  - "@yarnpkg/shell"

.yarn/cache/@types-node-npm-18.11.11-e5325451ab-7ff6dfb616.zip

@@ -79,6 +79,7 @@ The following changes only affect people writing Yarn plugins:
 ### Compatibility
 
 - Updates the PnP compatibility layer for TypeScript v4.9.4.
+- The patched filesystem now supports `FileHandle.readLines`.
 
 ## 3.3.0
 

.yarn/cache/@types-node-npm-18.7.6-c1eafa3d9a-5a8817b093.zip

@@ -14,7 +14,7 @@
     "@babel/preset-typescript": "^7.18.6",
     "@babel/register": "^7.18.9",
     "@types/jest": "^28.1.6",
-    "@types/node": "^18.7.6",
+    "@types/node": "^18.11.11",
     "@yarnpkg/cli": "workspace:^",
     "@yarnpkg/core": "workspace:^",
     "@yarnpkg/eslint-config": "workspace:^",

.yarn/versions/f12a955a.yml

@@ -44,7 +44,7 @@
     "@types/diff": "^5.0.0",
     "@types/lodash": "^4.14.136",
     "@types/micromatch": "^4.0.1",
-    "@types/node": "^18.7.6",
+    "@types/node": "^18.11.11",
     "@types/tar": "^4.0.4",
     "@types/tunnel": "^0.0.0",
     "@yarnpkg/cli": "workspace:^",

CHANGELOG.md

@@ -1,4 +1,5 @@
 import type {BigIntStats, ReadStream, StatOptions, Stats, WriteStream, WriteVResult} from 'fs';
+import {createInterface}                                                             from 'readline';
 
 import type {CreateReadStreamOptions, CreateWriteStreamOptions, FakeFS}              from '../FakeFS';
 import type {Path}                                                                   from '../path';
@@ -212,6 +213,13 @@ export class FileHandle<P extends Path> {
     }
   }
 
+  readLines(options?: CreateReadStreamOptions) {
+    return createInterface({
+      input: this.createReadStream(options),
+      crlfDelay: Infinity,
+    });
+  }
+
   stat(
     opts?: StatOptions & {
       bigint?: false | undefined;

package.json

@@ -553,6 +553,23 @@ describe(`patchedFs`, () => {
     });
   });
 
+  it(`should support FileHandle.readLines`, async () => {
+    const patchedFs = extendFs(fs, new PosixFS(new NodeFS()));
+
+    await xfs.mktempPromise(async dir => {
+      const filepath = npath.join(npath.fromPortablePath(dir), `foo.txt`);
+      await patchedFs.promises.writeFile(filepath, `1\n\n2\n`);
+
+      const fd = await patchedFs.promises.open(filepath);
+
+      const lines: Array<string> = [];
+      for await (const line of fd.readLines())
+        lines.push(line);
+
+      expect(lines).toStrictEqual([`1`, ``, `2`]);
+    });
+  });
+
   ifNotWin32It(`should support FileHandle.chmod`, async () => {
     const patchedFs = extendFs(fs, new PosixFS(new NodeFS()));
 

packages/yarnpkg-core/package.json

@@ -10,7 +10,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@types/node": "^18.7.6",
+    "@types/node": "^18.11.11",
     "@yarnpkg/fslib": "workspace:^"
   },
   "devDependencies": {