Fixed lint issues and failing tests

Author: pabloerhard

lib/get-exports.js

@@ -55,7 +55,7 @@ function resolvePackageImports (specifier, fromUrl) {
 
           // Look for path inside packageJson
           let resolvedPath
-          if (typeof imports == 'object') {
+          if (Object.prototype.toString.call(imports) === '[object Object]') {
             const requireSpecifier = imports.require
             const importSpecifier = imports.import
             // look for the possibility of require and import which is standard for CJS/ESM
@@ -65,11 +65,11 @@ function resolvePackageImports (specifier, fromUrl) {
             } else if (imports.node || imports.default) {
               resolvedPath = imports.node || imports.default
             }
-          } else if (typeof imports == 'string') {
+          } else if (typeof imports === 'string') {
             resolvedPath = imports
           }
 
-          if (existsSync(resolvedPath)) {
+          if (resolvedPath) {
             return resolvedPath
           }
         }
@@ -105,7 +105,6 @@ async function getCjsExports (url, context, parentLoad, source) {
           re = './'
         }
 
-        let newUrl
         // Entries in the import field should always start with #
         if (re.startsWith('#')) {
           re = resolvePackageImports(re, url)
@@ -115,8 +114,7 @@ async function getCjsExports (url, context, parentLoad, source) {
           }
         }
 
-        // Resolve the re-exported module relative to the current module.
-        newUrl = pathToFileURL(require.resolve(re, { paths: [dirname(fileURLToPath(url))] })).href
+        const newUrl = pathToFileURL(require.resolve(re, { paths: [dirname(fileURLToPath(url))] })).href
 
         if (newUrl.endsWith('.node') || newUrl.endsWith('.json')) {
           return