Skip to content

Commit d1fa9a6

Browse files
committed
vfs: import getVirtualFd eagerly in streams
internal/vfs/fd.js doesn't require any internal/vfs/* modules so there's no circular dependency to defer. Replace the getLazy wrapper with a direct import. Assisted-by: Claude-Opus4.7
1 parent 72e8449 commit d1fa9a6

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

lib/internal/vfs/streams.js

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,15 @@ const {
77
const { Buffer } = require('buffer');
88
const { Readable, Writable } = require('stream');
99
const { createEBADF } = require('internal/vfs/errors');
10-
const { getLazy, kEmptyObject } = require('internal/util');
10+
const { getVirtualFd } = require('internal/vfs/fd');
11+
const { kEmptyObject } = require('internal/util');
1112
const { validateInteger } = require('internal/validators');
1213
const {
1314
codes: {
1415
ERR_OUT_OF_RANGE,
1516
},
1617
} = require('internal/errors');
1718

18-
// Lazy-load fd module to avoid circular dependency
19-
const lazyGetVirtualFd = getLazy(
20-
() => require('internal/vfs/fd').getVirtualFd,
21-
);
22-
2319
/**
2420
* A readable stream for virtual files.
2521
*/
@@ -133,7 +129,6 @@ class VirtualReadStream extends Readable {
133129
// Load content on first read (lazy loading)
134130
if (this.#content === null) {
135131
try {
136-
const getVirtualFd = lazyGetVirtualFd();
137132
const vfd = getVirtualFd(this.#fd);
138133
if (!vfd) {
139134
this.destroy(createEBADF('read'));
@@ -279,7 +274,6 @@ class VirtualWriteStream extends Writable {
279274
* @param {number} pos The position to set
280275
*/
281276
#setPosition(pos) {
282-
const getVirtualFd = lazyGetVirtualFd();
283277
const vfd = getVirtualFd(this.#fd);
284278
if (vfd) {
285279
vfd.entry.position = pos;

0 commit comments

Comments
 (0)