fixup: remove changes for get full packagejson

lib/internal/modules/package_json_reader.js

@@ -12,18 +12,16 @@ const { resolve, sep } = require('path');
 const { kEmptyObject } = require('internal/util');
 
 /**
- * @typedef {import('typings/internalBinding/modules').FullPackageConfig} FullPackageConfig
  * @typedef {import('typings/internalBinding/modules').PackageConfig} PackageConfig
  * @typedef {import('typings/internalBinding/modules').SerializedPackageConfig} SerializedPackageConfig
  */
 
 /**
  * @param {string} path
  * @param {SerializedPackageConfig} contents
- * @param {boolean} everything
- * @returns {everything extends true ? FullPackageConfig : PackageConfig}
+ * @returns {PackageConfig}
  */
-function deserializePackageJSON(path, contents, everything = false) {
+function deserializePackageJSON(path, contents) {
   if (contents === undefined) {
     return {
       __proto__: null,
@@ -71,7 +69,6 @@ function deserializePackageJSON(path, contents, everything = false) {
       ObjectDefineProperty(this, 'exports', { __proto__: null, value });
       return this.exports;
     },
-    ...(everything && contents[6]),
   };
 }
 
@@ -113,17 +110,16 @@ function readPackage(requestPath) {
  * Get the nearest parent package.json file from a given path.
  * Return the package.json data and the path to the package.json file, or undefined.
  * @param {string} checkPath The path to start searching from.
- * @param {boolean} everything Whether to include unrecognised fields.
  * @returns {undefined | {data: PackageConfig, path: string}}
  */
-function getNearestParentPackageJSON(checkPath, everything = false) {
+function getNearestParentPackageJSON(checkPath) {
   const result = modulesBinding.getNearestParentPackageJSON(checkPath);
 
   if (result === undefined) {
     return undefined;
   }
 
-  const data = deserializePackageJSON(checkPath, result, everything);
+  const data = deserializePackageJSON(checkPath, result);
 
   // Path should be the root folder of the matched package.json
   // For example for ~/path/package.json, it should be ~/path
@@ -144,14 +140,13 @@ function findNearestPackageJSON(origin) {
 /**
  * Returns the package configuration for the given resolved URL.
  * @param {URL | string} resolved - The resolved URL.
- * @param {boolean} everything - Whether to include unrecognised fields.
  * @returns {PackageConfig} - The package configuration.
  */
-function getPackageScopeConfig(resolved, everything = false) {
+function getPackageScopeConfig(resolved) {
   const result = modulesBinding.getPackageScopeConfig(`${resolved}`);
 
   if (ArrayIsArray(result)) {
-    return deserializePackageJSON(`${resolved}`, result, everything);
+    return deserializePackageJSON(`${resolved}`, result);
   }
 
   // This means that the response is a string

lib/module.js

@@ -11,7 +11,6 @@ const {
   getCompileCacheDir,
 } = require('internal/modules/helpers');
 const {
-  getPackageScopeConfig,
   findNearestPackageJSON,
 } = require('internal/modules/package_json_reader');
 
@@ -23,6 +22,5 @@ Module.enableCompileCache = enableCompileCache;
 Module.flushCompileCache = flushCompileCache;
 
 Module.getCompileCacheDir = getCompileCacheDir;
-Module.getPackageScopeConfig = getPackageScopeConfig;
 Module.findNearestPackageJSON = findNearestPackageJSON;
 module.exports = Module;

typings/internalBinding/modules.d.ts

@@ -8,9 +8,6 @@ export type PackageConfig = {
   exports?: string | string[] | Record<string, unknown>
   imports?: string | string[] | Record<string, unknown>
 }
-export type FullPackageConfig = PackageConfig & {
-  [key: string]: unknown,
-}
 export type SerializedPackageConfig = [
   PackageConfig['name'],
   PackageConfig['main'],