metro 0.84.2 → 0.84.3

package/src/lib/getAppendScripts.js.flow

@@ -18,7 +18,7 @@ import CountingSet from './CountingSet';
 import countLines from './countLines';
 import nullthrows from 'nullthrows';
 
-type Options<T: number | string> = Readonly<{
+type Options<T extends number | string> = Readonly<{
   asyncRequireModulePath: string,
   createModuleId: string => T,
   getRunModuleStatement: (moduleId: T, globalPrefix: string) => string,
@@ -33,7 +33,7 @@ type Options<T: number | string> = Readonly<{
   ...
 }>;
 
-export default function getAppendScripts<T: number | string>(
+export default function getAppendScripts<T extends number | string>(
   entryPoint: string,
   modules: ReadonlyArray<Module<>>,
   options: Options<T>,

package/src/node-haste/DependencyGraph/ModuleResolution.d.ts

@@ -6,7 +6,7 @@
  *
  * @noformat
  * @oncall react_native
- * @generated SignedSource<<d5224d8913b7e0c52ac84b215b356422>>
+ * @generated SignedSource<<0024fd05b95efe19a24f9acc84ff474b>>
  *
  * This file was translated from Flow by scripts/generateTypeScriptDefinitions.js
  * Original file: packages/metro/src/node-haste/DependencyGraph/ModuleResolution.js
@@ -32,19 +32,7 @@ import type {
 import type {PackageForModule, PackageJson} from 'metro-resolver/private/types';
 
 export type DirExistsFn = (filePath: string) => boolean;
-export type Packageish = {path: string; read(): PackageJson};
-export type Moduleish = {readonly path: string};
-export type PackageishCache<TPackage> = {
-  getPackage(
-    name: string,
-    platform?: string,
-    supportsNativePlatform?: boolean,
-  ): TPackage;
-  getPackageOf(
-    absolutePath: string,
-  ): null | undefined | {pkg: TPackage; packageRelativePath: string};
-};
-type Options<TPackage> = Readonly<{
+type Options = Readonly<{
   assetExts: ReadonlySet<string>;
   dirExists: DirExistsFn;
   disableHierarchicalLookup: boolean;
@@ -61,7 +49,10 @@ type Options<TPackage> = Readonly<{
     platform: null | undefined | string,
   ) => null | undefined | string;
   mainFields: ReadonlyArray<string>;
-  packageCache: PackageishCache<TPackage>;
+  getPackage: (packageJsonPath: string) => null | undefined | PackageJson;
+  getPackageForModule: (
+    absolutePath: string,
+  ) => null | undefined | PackageForModule;
   nodeModulesPaths: ReadonlyArray<string>;
   preferNativePlatform: boolean;
   projectRoot: string;
@@ -76,11 +67,11 @@ type Options<TPackage> = Readonly<{
   unstable_enablePackageExports: boolean;
   unstable_incrementalResolution: boolean;
 }>;
-export declare class ModuleResolver<TPackage extends Packageish> {
-  _options: Options<TPackage>;
+export declare class ModuleResolver {
+  _options: Options;
   _projectRootFakeModulePath: string;
   _cachedEmptyModule: null | undefined | BundlerResolution;
-  constructor(options: Options<TPackage>);
+  constructor(options: Options);
   _getEmptyModule(): BundlerResolution;
   resolveDependency(
     originModulePath: string,
@@ -89,10 +80,6 @@ export declare class ModuleResolver<TPackage extends Packageish> {
     platform: string | null,
     resolverOptions: ResolverInputOptions,
   ): BundlerResolution;
-  _getPackage: (packageJsonPath: string) => null | undefined | PackageJson;
-  _getPackageForModule: (
-    absolutePath: string,
-  ) => null | undefined | PackageForModule;
   /**
    * TODO: Return Resolution instead of coercing to BundlerResolution here
    */

package/src/node-haste/DependencyGraph/ModuleResolution.js

@@ -86,6 +86,8 @@ class ModuleResolver {
       doesFileExist,
       extraNodeModules,
       fileSystemLookup,
+      getPackage,
+      getPackageForModule,
       mainFields,
       nodeModulesPaths,
       preferNativePlatform,
@@ -109,9 +111,8 @@ class ModuleResolver {
             doesFileExist,
             extraNodeModules,
             fileSystemLookup,
-            getPackage: this._getPackage,
-            getPackageForModule: (absoluteModulePath) =>
-              this._getPackageForModule(absoluteModulePath),
+            getPackage,
+            getPackageForModule,
             isESMImport: dependency.data.isESMImport,
             mainFields,
             nodeModulesPaths,
@@ -191,25 +192,6 @@ class ModuleResolver {
       throw error;
     }
   }
-  _getPackage = (packageJsonPath) => {
-    try {
-      return this._options.packageCache.getPackage(packageJsonPath).read();
-    } catch (e) {}
-    return null;
-  };
-  _getPackageForModule = (absolutePath) => {
-    let result;
-    try {
-      result = this._options.packageCache.getPackageOf(absolutePath);
-    } catch (e) {}
-    return result != null
-      ? {
-          packageJson: result.pkg.read(),
-          packageRelativePath: result.packageRelativePath,
-          rootPath: _path.default.dirname(result.pkg.path),
-        }
-      : null;
-  };
   _getFileResolvedModule(resolution) {
     switch (resolution.type) {
       case "sourceFile":

package/src/node-haste/DependencyGraph/ModuleResolution.js.flow

@@ -35,28 +35,7 @@ import util from 'util';
 
 export type DirExistsFn = (filePath: string) => boolean;
 
-export type Packageish = interface {
-  path: string,
-  read(): PackageJson,
-};
-
-export type Moduleish = interface {
-  +path: string,
-};
-
-export type PackageishCache<TPackage> = interface {
-  getPackage(
-    name: string,
-    platform?: string,
-    supportsNativePlatform?: boolean,
-  ): TPackage,
-  getPackageOf(absolutePath: string): ?{
-    pkg: TPackage,
-    packageRelativePath: string,
-  },
-};
-
-type Options<TPackage> = Readonly<{
+type Options = Readonly<{
   assetExts: ReadonlySet<string>,
   dirExists: DirExistsFn,
   disableHierarchicalLookup: boolean,
@@ -67,7 +46,8 @@ type Options<TPackage> = Readonly<{
   getHasteModulePath: (name: string, platform: ?string) => ?string,
   getHastePackagePath: (name: string, platform: ?string) => ?string,
   mainFields: ReadonlyArray<string>,
-  packageCache: PackageishCache<TPackage>,
+  getPackage: (packageJsonPath: string) => ?PackageJson,
+  getPackageForModule: (absolutePath: string) => ?PackageForModule,
   nodeModulesPaths: ReadonlyArray<string>,
   preferNativePlatform: boolean,
   projectRoot: string,
@@ -83,14 +63,14 @@ type Options<TPackage> = Readonly<{
   unstable_incrementalResolution: boolean,
 }>;
 
-export class ModuleResolver<TPackage: Packageish> {
-  _options: Options<TPackage>;
+export class ModuleResolver {
+  _options: Options;
   // A module representing the project root, used as the origin when resolving `emptyModulePath`.
   _projectRootFakeModulePath: string;
   // An empty module, the result of resolving `emptyModulePath` from the project root.
   _cachedEmptyModule: ?BundlerResolution;
 
-  constructor(options: Options<TPackage>) {
+  constructor(options: Options) {
     this._options = options;
     const {projectRoot} = this._options;
     this._projectRootFakeModulePath = path.join(projectRoot, '_');
@@ -132,6 +112,8 @@ export class ModuleResolver<TPackage: Packageish> {
       doesFileExist,
       extraNodeModules,
       fileSystemLookup,
+      getPackage,
+      getPackageForModule,
       mainFields,
       nodeModulesPaths,
       preferNativePlatform,
@@ -156,9 +138,8 @@ export class ModuleResolver<TPackage: Packageish> {
             doesFileExist,
             extraNodeModules,
             fileSystemLookup,
-            getPackage: this._getPackage,
-            getPackageForModule: (absoluteModulePath: string) =>
-              this._getPackageForModule(absoluteModulePath),
+            getPackage,
+            getPackageForModule,
             isESMImport: dependency.data.isESMImport,
             mainFields,
             nodeModulesPaths,
@@ -240,36 +221,6 @@ export class ModuleResolver<TPackage: Packageish> {
     }
   }
 
-  _getPackage = (packageJsonPath: string): ?PackageJson => {
-    try {
-      return this._options.packageCache.getPackage(packageJsonPath).read();
-    } catch (e) {
-      // Do nothing. The standard module cache does not trigger any error, but
-      // the ModuleGraph one does, if the module does not exist.
-    }
-
-    return null;
-  };
-
-  _getPackageForModule = (absolutePath: string): ?PackageForModule => {
-    let result;
-
-    try {
-      result = this._options.packageCache.getPackageOf(absolutePath);
-    } catch (e) {
-      // Do nothing. The standard module cache does not trigger any error, but
-      // the ModuleGraph one does, if the module does not exist.
-    }
-
-    return result != null
-      ? {
-          packageJson: result.pkg.read(),
-          packageRelativePath: result.packageRelativePath,
-          rootPath: path.dirname(result.pkg.path),
-        }
-      : null;
-  };
-
   /**
    * TODO: Return Resolution instead of coercing to BundlerResolution here
    */

package/src/node-haste/DependencyGraph/createFileMap.js

@@ -71,7 +71,7 @@ function createFileMap(config, options) {
   const { enabled: autoSaveEnabled, ...autoSaveOpts } =
     config.watcher.unstable_autoSaveCache ?? {};
   const autoSave = watch && autoSaveEnabled ? autoSaveOpts : false;
-  const plugins = [];
+  const plugins = [...(config.unstable_fileMapPlugins ?? [])];
   let dependencyPlugin = null;
   if (
     config.resolver.dependencyExtractor != null &&

package/src/node-haste/DependencyGraph/createFileMap.js.flow

@@ -10,7 +10,7 @@
  */
 
 import type {ConfigT} from 'metro-config';
-import type {HasteMap} from 'metro-file-map';
+import type {HasteMap, InputFileMapPlugin} from 'metro-file-map';
 
 import ci from 'ci-info';
 import MetroFileMap, {
@@ -75,7 +75,9 @@ export default function createFileMap(
     config.watcher.unstable_autoSaveCache ?? {};
   const autoSave = watch && autoSaveEnabled ? autoSaveOpts : false;
 
-  const plugins: Array<DependencyPlugin | HastePlugin> = [];
+  const plugins: Array<InputFileMapPlugin> = [
+    ...(config.unstable_fileMapPlugins ?? []),
+  ];
 
   let dependencyPlugin = null;
   // Add DependencyPlugin if dependencies should be extracted

package/src/node-haste/DependencyGraph.d.ts

@@ -6,7 +6,7 @@
  *
  * @noformat
  * @oncall react_native
- * @generated SignedSource<<eb945b3566e9ed370894adaada197736>>
+ * @generated SignedSource<<13f1483d2a732241f8d9eae463399b0e>>
  *
  * This file was translated from Flow by scripts/generateTypeScriptDefinitions.js
  * Original file: packages/metro/src/node-haste/DependencyGraph.js
@@ -20,7 +20,6 @@ import type {
   TransformResultDependency,
 } from '../DeltaBundler/types';
 import type {ResolverInputOptions} from '../shared/types';
-import type Package from './Package';
 import type {ConfigT} from 'metro-config';
 import type {
   ChangeEvent,
@@ -32,7 +31,6 @@ import type {
 } from 'metro-file-map';
 
 import {ModuleResolver} from './DependencyGraph/ModuleResolution';
-import {PackageCache} from './PackageCache';
 import EventEmitter from 'events';
 
 declare class DependencyGraph extends EventEmitter {
@@ -40,7 +38,7 @@ declare class DependencyGraph extends EventEmitter {
   _haste: MetroFileMap;
   _fileSystem: FileSystem;
   _hasteMap: HasteMap;
-  _moduleResolver: ModuleResolver<Package>;
+  _moduleResolver: ModuleResolver;
   _resolutionCache: Map<
     string | symbol,
     Map<
@@ -64,7 +62,6 @@ declare class DependencyGraph extends EventEmitter {
   _getClosestPackage(
     absoluteModulePath: string,
   ): null | undefined | {packageJsonPath: string; packageRelativePath: string};
-  _createPackageCache(): PackageCache;
   getAllFiles(): Array<string>;
   /**
    * Used when watcher.unstable_lazySha1 is true

package/src/node-haste/DependencyGraph.js

@@ -71,7 +71,10 @@ class DependencyGraph extends _events.default {
         this._onWatcherHealthCheck(result),
       );
       this._resolutionCache = new Map();
-      this.#packageCache = this._createPackageCache();
+      this.#packageCache = new _PackageCache.PackageCache({
+        getClosestPackage: (absoluteModulePath) =>
+          this._getClosestPackage(absoluteModulePath),
+      });
       this._createModuleResolver();
     });
   }
@@ -90,10 +93,14 @@ class DependencyGraph extends _events.default {
   async ready() {
     await this._initializedPromise;
   }
-  _onHasteChange({ eventsQueue }) {
+  _onHasteChange({ changes, rootDir }) {
     this._resolutionCache = new Map();
-    eventsQueue.forEach(({ filePath }) =>
-      this.#packageCache.invalidate(filePath),
+    [
+      ...changes.addedFiles,
+      ...changes.modifiedFiles,
+      ...changes.removedFiles,
+    ].forEach(([canonicalPath]) =>
+      this.#packageCache.invalidate(_path.default.join(rootDir, canonicalPath)),
     );
     this._createModuleResolver();
     this.emit("change");
@@ -130,9 +137,19 @@ class DependencyGraph extends _events.default {
         this._hasteMap.getModule(name, platform, true),
       getHastePackagePath: (name, platform) =>
         this._hasteMap.getPackage(name, platform, true),
+      getPackage: (packageJsonPath) => {
+        try {
+          return (
+            this.#packageCache.getPackage(packageJsonPath).packageJson ?? null
+          );
+        } catch {
+          return null;
+        }
+      },
+      getPackageForModule: (absolutePath) =>
+        this.#packageCache.getPackageForModule(absolutePath),
       mainFields: this._config.resolver.resolverMainFields,
       nodeModulesPaths: this._config.resolver.nodeModulesPaths,
-      packageCache: this.#packageCache,
       preferNativePlatform: true,
       projectRoot: this._config.projectRoot,
       reporter: this._config.reporter,
@@ -176,12 +193,6 @@ class DependencyGraph extends _events.default {
         }
       : null;
   }
-  _createPackageCache() {
-    return new _PackageCache.PackageCache({
-      getClosestPackage: (absolutePath) =>
-        this._getClosestPackage(absolutePath),
-    });
-  }
   getAllFiles() {
     return (0, _nullthrows.default)(this._fileSystem).getAllFiles();
   }

package/src/node-haste/DependencyGraph.js.flow

@@ -14,7 +14,6 @@ import type {
   TransformResultDependency,
 } from '../DeltaBundler/types';
 import type {ResolverInputOptions} from '../shared/types';
-import type Package from './Package';
 import type {ConfigT} from 'metro-config';
 import type {
   ChangeEvent,
@@ -66,7 +65,7 @@ export default class DependencyGraph extends EventEmitter {
   #packageCache: PackageCache;
   _hasteMap: HasteMap;
   #dependencyPlugin: ?DependencyPlugin;
-  _moduleResolver: ModuleResolver<Package>;
+  _moduleResolver: ModuleResolver;
   _resolutionCache: Map<
     // Custom resolver options
     string | symbol,
@@ -131,7 +130,10 @@ export default class DependencyGraph extends EventEmitter {
         this._onWatcherHealthCheck(result),
       );
       this._resolutionCache = new Map();
-      this.#packageCache = this._createPackageCache();
+      this.#packageCache = new PackageCache({
+        getClosestPackage: absoluteModulePath =>
+          this._getClosestPackage(absoluteModulePath),
+      });
       this._createModuleResolver();
     });
   }
@@ -150,10 +152,14 @@ export default class DependencyGraph extends EventEmitter {
     await this._initializedPromise;
   }
 
-  _onHasteChange({eventsQueue}: ChangeEvent) {
+  _onHasteChange({changes, rootDir}: ChangeEvent) {
     this._resolutionCache = new Map();
-    eventsQueue.forEach(({filePath}) =>
-      this.#packageCache.invalidate(filePath),
+    [
+      ...changes.addedFiles,
+      ...changes.modifiedFiles,
+      ...changes.removedFiles,
+    ].forEach(([canonicalPath]) =>
+      this.#packageCache.invalidate(path.join(rootDir, canonicalPath)),
     );
     this._createModuleResolver();
     this.emit('change');
@@ -190,9 +196,20 @@ export default class DependencyGraph extends EventEmitter {
         this._hasteMap.getModule(name, platform, true),
       getHastePackagePath: (name, platform) =>
         this._hasteMap.getPackage(name, platform, true),
+      getPackage: (packageJsonPath: string) => {
+        try {
+          return (
+            this.#packageCache.getPackage(packageJsonPath).packageJson ?? null
+          );
+        } catch {
+          // Non-existence or malformed JSON, we treat both as non-existent
+          return null;
+        }
+      },
+      getPackageForModule: (absolutePath: string) =>
+        this.#packageCache.getPackageForModule(absolutePath),
       mainFields: this._config.resolver.resolverMainFields,
       nodeModulesPaths: this._config.resolver.nodeModulesPaths,
-      packageCache: this.#packageCache,
       preferNativePlatform: true,
       projectRoot: this._config.projectRoot,
       reporter: this._config.reporter,
@@ -241,12 +258,6 @@ export default class DependencyGraph extends EventEmitter {
       : null;
   }
 
-  _createPackageCache(): PackageCache {
-    return new PackageCache({
-      getClosestPackage: absolutePath => this._getClosestPackage(absolutePath),
-    });
-  }
-
   getAllFiles(): Array<string> {
     return nullthrows(this._fileSystem).getAllFiles();
   }

package/src/node-haste/PackageCache.d.ts

@@ -6,7 +6,7 @@
  *
  * @noformat
  * @oncall react_native
- * @generated SignedSource<<eaea6d7e01d54353f700cdadd60c4bf2>>
+ * @generated SignedSource<<768dba0958b531c8edd43c2df24e25f6>>
  *
  * This file was translated from Flow by scripts/generateTypeScriptDefinitions.js
  * Original file: packages/metro/src/node-haste/PackageCache.js
@@ -15,27 +15,23 @@
  *   yarn run build-ts-defs (OSS) 
  */
 
-import Package from './Package';
+import type {PackageJson} from 'metro-resolver/private/types';
 
 type GetClosestPackageFn = (
   absoluteFilePath: string,
 ) => null | undefined | {packageJsonPath: string; packageRelativePath: string};
+type PackageForModule = Readonly<{
+  packageJson: PackageJson;
+  rootPath: string;
+  packageRelativePath: string;
+}>;
 export declare class PackageCache {
-  _getClosestPackage: GetClosestPackageFn;
-  _packageCache: {[filePath: string]: Package};
-  _packagePathAndSubpathByModulePath: {
-    [filePath: string]:
-      | null
-      | undefined
-      | {packageJsonPath: string; packageRelativePath: string};
-  };
-  _modulePathsByPackagePath: {
-    [filePath: string]: Set<string>;
-  };
   constructor(options: {getClosestPackage: GetClosestPackageFn});
-  getPackage(filePath: string): Package;
-  getPackageOf(
+  getPackage(
+    filePath: string,
+  ): Readonly<{rootPath: string; packageJson: PackageJson}>;
+  getPackageForModule(
     absoluteModulePath: string,
-  ): null | undefined | {pkg: Package; packageRelativePath: string};
+  ): null | undefined | PackageForModule;
   invalidate(filePath: string): void;
 }

package/src/node-haste/PackageCache.js

@@ -4,78 +4,89 @@ Object.defineProperty(exports, "__esModule", {
   value: true,
 });
 exports.PackageCache = void 0;
-var _Package = _interopRequireDefault(require("./Package"));
-function _interopRequireDefault(e) {
-  return e && e.__esModule ? e : { default: e };
-}
+var _fs = require("fs");
+var _path = require("path");
 class PackageCache {
+  #getClosestPackage;
+  #packageCache;
+  #resultByModulePath;
+  #modulePathsByPackagePath;
+  #modulePathsWithNoPackage;
   constructor(options) {
-    this._getClosestPackage = options.getClosestPackage;
-    this._packageCache = Object.create(null);
-    this._packagePathAndSubpathByModulePath = Object.create(null);
-    this._modulePathsByPackagePath = Object.create(null);
+    this.#getClosestPackage = options.getClosestPackage;
+    this.#packageCache = new Map();
+    this.#resultByModulePath = new Map();
+    this.#modulePathsByPackagePath = new Map();
+    this.#modulePathsWithNoPackage = new Set();
   }
   getPackage(filePath) {
-    if (!this._packageCache[filePath]) {
-      this._packageCache[filePath] = new _Package.default({
-        file: filePath,
-      });
+    let cached = this.#packageCache.get(filePath);
+    if (cached == null) {
+      cached = {
+        rootPath: (0, _path.dirname)(filePath),
+        packageJson: JSON.parse((0, _fs.readFileSync)(filePath, "utf8")),
+      };
+      this.#packageCache.set(filePath, cached);
     }
-    return this._packageCache[filePath];
+    return cached;
   }
-  getPackageOf(absoluteModulePath) {
-    let packagePathAndSubpath =
-      this._packagePathAndSubpathByModulePath[absoluteModulePath];
-    if (
-      packagePathAndSubpath &&
-      this._packageCache[packagePathAndSubpath.packageJsonPath]
-    ) {
-      return {
-        pkg: this._packageCache[packagePathAndSubpath.packageJsonPath],
-        packageRelativePath: packagePathAndSubpath.packageRelativePath,
-      };
+  getPackageForModule(absoluteModulePath) {
+    const cached = this.#resultByModulePath.get(absoluteModulePath);
+    if (cached !== undefined) {
+      return cached;
     }
-    packagePathAndSubpath = this._getClosestPackage(absoluteModulePath);
-    if (!packagePathAndSubpath) {
+    const closest = this.#getClosestPackage(absoluteModulePath);
+    if (closest == null) {
+      this.#resultByModulePath.set(absoluteModulePath, null);
+      this.#modulePathsWithNoPackage.add(absoluteModulePath);
       return null;
     }
-    const packagePath = packagePathAndSubpath.packageJsonPath;
-    this._packagePathAndSubpathByModulePath[absoluteModulePath] =
-      packagePathAndSubpath;
-    const modulePaths =
-      this._modulePathsByPackagePath[packagePath] ?? new Set();
+    const packagePath = closest.packageJsonPath;
+    let modulePaths = this.#modulePathsByPackagePath.get(packagePath);
+    if (modulePaths == null) {
+      modulePaths = new Set();
+      this.#modulePathsByPackagePath.set(packagePath, modulePaths);
+    }
     modulePaths.add(absoluteModulePath);
-    this._modulePathsByPackagePath[packagePath] = modulePaths;
-    return {
-      pkg: this.getPackage(packagePath),
-      packageRelativePath: packagePathAndSubpath.packageRelativePath,
+    const pkg = this.getPackage(packagePath);
+    if (pkg == null) {
+      return null;
+    }
+    const result = {
+      packageJson: pkg.packageJson,
+      packageRelativePath: closest.packageRelativePath,
+      rootPath: pkg.rootPath,
     };
+    this.#resultByModulePath.set(absoluteModulePath, result);
+    return result;
   }
   invalidate(filePath) {
-    if (this._packageCache[filePath]) {
-      this._packageCache[filePath].invalidate();
-      delete this._packageCache[filePath];
-    }
-    const packagePathAndSubpath =
-      this._packagePathAndSubpathByModulePath[filePath];
-    if (packagePathAndSubpath) {
-      const packagePath = packagePathAndSubpath.packageJsonPath;
-      delete this._packagePathAndSubpathByModulePath[filePath];
-      const modulePaths = this._modulePathsByPackagePath[packagePath];
-      if (modulePaths) {
-        modulePaths.delete(filePath);
-        if (modulePaths.size === 0) {
-          delete this._modulePathsByPackagePath[packagePath];
+    this.#packageCache.delete(filePath);
+    const cachedResult = this.#resultByModulePath.get(filePath);
+    this.#resultByModulePath.delete(filePath);
+    this.#modulePathsWithNoPackage.delete(filePath);
+    if (cachedResult != null) {
+      const packagePath = cachedResult.rootPath + _path.sep + "package.json";
+      const modules = this.#modulePathsByPackagePath.get(packagePath);
+      if (modules != null) {
+        modules.delete(filePath);
+        if (modules.size === 0) {
+          this.#modulePathsByPackagePath.delete(packagePath);
         }
       }
     }
-    if (this._modulePathsByPackagePath[filePath]) {
-      const modulePaths = this._modulePathsByPackagePath[filePath];
+    const modulePaths = this.#modulePathsByPackagePath.get(filePath);
+    if (modulePaths != null) {
       for (const modulePath of modulePaths) {
-        delete this._packagePathAndSubpathByModulePath[modulePath];
+        this.#resultByModulePath.delete(modulePath);
+      }
+      this.#modulePathsByPackagePath.delete(filePath);
+    }
+    if (filePath.endsWith(_path.sep + "package.json")) {
+      for (const modulePath of this.#modulePathsWithNoPackage) {
+        this.#resultByModulePath.delete(modulePath);
       }
-      modulePaths.clear();
-      delete this._modulePathsByPackagePath[filePath];
+      this.#modulePathsWithNoPackage.clear();
     }
   }
 }