@jsenv/node-esm-resolution 1.0.4 → 1.0.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/node-esm-resolution",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "license": "MIT",
   "repository": {
     "type": "git",

package/src/default_read_package_json.js

@@ -6,7 +6,7 @@ export const defaultReadPackageJson = (packageUrl) => {
   const string = String(buffer);
   try {
     return JSON.parse(string);
-  } catch (e) {
+  } catch {
     throw new Error(`Invalid package configuration`);
   }
 };

package/src/node_esm_resolution.js

@@ -93,7 +93,7 @@ const applyPackageSpecifierResolution = (specifier, resolutionContext) => {
       type: "absolute_specifier",
       url: urlObject.href,
     };
-  } catch (e) {
+  } catch {
     // bare specifier
     const browserFieldResolution = applyBrowserFieldResolution(
       specifier,
@@ -103,11 +103,9 @@ const applyPackageSpecifierResolution = (specifier, resolutionContext) => {
       return browserFieldResolution;
     }
     const packageResolution = applyPackageResolve(specifier, resolutionContext);
-    if (packageResolution.type !== "relative_specifier") {
-      const search = new URL(specifier, "file://").search;
-      if (search) {
-        packageResolution.url = `${packageResolution.url}${search}`;
-      }
+    const search = new URL(specifier, "file://").search;
+    if (search && !new URL(packageResolution.url).search) {
+      packageResolution.url = `${packageResolution.url}${search}`;
     }
     return packageResolution;
   }

package/src/url_utils.js

@@ -37,7 +37,7 @@ export const isValidUrl = (url) => {
     // eslint-disable-next-line no-new
     new URL(url);
     return true;
-  } catch (e) {
+  } catch {
     return false;
   }
 };