@jsenv/urls 2.5.1 → 2.5.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/urls",
-  "version": "2.5.1",
+  "version": "2.5.3",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -24,10 +24,8 @@
   "files": [
     "/src/"
   ],
-  "scripts": {
-    "test": "node --conditions=development ./scripts/test.mjs"
-  },
+  "sideEffects": false,
   "dependencies": {
-    "@jsenv/humanize": "1.2.7"
+    "@jsenv/humanize": "1.2.8"
   }
 }

package/src/url_to_basename.js

@@ -7,7 +7,6 @@ export const urlToBasename = (url, removeAllExtensions) => {
     return basename;
   }
   let currentBasename = basename;
-  // eslint-disable-next-line no-constant-condition
   while (true) {
     const nextBasename = filenameToBasename(basename);
     if (nextBasename === currentBasename) {

package/src/url_to_filesystem_path.js

@@ -1,12 +1,20 @@
 import { fileURLToPath } from "node:url";
 
 export const urlToFileSystemPath = (url) => {
-  let urlString = String(url);
-  if (urlString[urlString.length - 1] === "/") {
+  const urlObject = new URL(url);
+  let urlString;
+  if (urlObject.hash) {
+    const origin =
+      urlObject.protocol === "file:" ? "file://" : urlObject.origin;
+    urlString = `${origin}${urlObject.pathname}${urlObject.search}%23${urlObject.hash.slice(1)}`;
+  } else {
+    urlString = urlObject.href;
+  }
+  const fileSystemPath = fileURLToPath(urlString);
+  if (fileSystemPath[fileSystemPath.length - 1] === "/") {
     // remove trailing / so that nodejs path becomes predictable otherwise it logs
     // the trailing slash on linux but does not on windows
-    urlString = urlString.slice(0, -1);
+    return fileSystemPath.slice(0, -1);
   }
-  const fileSystemPath = fileURLToPath(urlString);
   return fileSystemPath;
 };

package/src/url_utils.js

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