@jsenv/filesystem 4.15.6 → 4.15.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/filesystem",
-  "version": "4.15.6",
+  "version": "4.15.8",
   "repository": {
     "type": "git",
     "url": "https://github.com/jsenv/core",
@@ -26,7 +26,7 @@
   "dependencies": {
     "@jsenv/abort": "4.3.1",
     "@jsenv/url-meta": "8.5.7",
-    "@jsenv/urls": "2.9.1",
+    "@jsenv/urls": "2.9.2",
     "@jsenv/utils": "2.3.1"
   },
   "devDependencies": {

package/src/path_and_url/lookup_package_directory.js

@@ -49,14 +49,23 @@ export const createLookupPackageDirectory = () => {
 };
 
 export const readPackageAtOrNull = (packageDirectoryUrl) => {
+  const packageJsonFileUrl = new URL("./package.json", packageDirectoryUrl);
+  let packageJsonFileContentBuffer;
   try {
-    const packageFileContent = readFileSync(
-      new URL("./package.json", packageDirectoryUrl),
-      "utf8",
+    packageJsonFileContentBuffer = readFileSync(packageJsonFileUrl, "utf8");
+  } catch (e) {
+    if (e.code === "ENOENT") {
+      return null;
+    }
+    throw e;
+  }
+  const packageJsonFileContentString = String(packageJsonFileContentBuffer);
+  try {
+    const packageJsonFileContentObject = JSON.parse(
+      packageJsonFileContentString,
     );
-    const packageJSON = JSON.parse(packageFileContent);
-    return packageJSON;
+    return packageJsonFileContentObject;
   } catch {
-    return null;
+    throw new Error(`Invalid package configuration at ${packageJsonFileUrl}`);
   }
 };

package/src/read_write/read_symbolic_link_sync.js

@@ -9,7 +9,13 @@ import { assertAndNormalizeFileUrl } from "../path_and_url/file_url_validation.j
 
 export const readSymbolicLinkSync = (url) => {
   const symbolicLinkUrl = assertAndNormalizeFileUrl(url);
-  const resolvedPath = readlinkSync(new URL(symbolicLinkUrl));
+  const resolvedPath = readlinkSync(
+    new URL(
+      symbolicLinkUrl.endsWith("/")
+        ? symbolicLinkUrl.slice(0, -1)
+        : symbolicLinkUrl,
+    ),
+  );
   return isFileSystemPath(resolvedPath)
     ? fileSystemPathToUrl(resolvedPath)
     : resolvedPath.replace(/\\/g, "/"); // replace back slashes with slashes

package/src/read_write/write_symbolic_link_sync.js

@@ -28,9 +28,20 @@ export const writeSymbolicLinkSync = ({
   type,
   allowUseless = false,
   allowOverwrite = false,
+  throwWhenTargetNotFound = true,
 }) => {
   const fromUrl = assertAndNormalizeFileUrl(from);
   const toInfo = getToInfo(to, fromUrl);
+  if (throwWhenTargetNotFound) {
+    const toStats = readEntryStatSync(toInfo.url, {
+      nullIfNotFound: true,
+    });
+    if (toStats === null) {
+      throw new Error(
+        `symbolic link target not found: ${toInfo.url} (from: ${fromUrl})`,
+      );
+    }
+  }
   // Node.js doc at https://nodejs.org/api/fs.html#fssymlinktarget-path-type-callback
   // states the following:
   // "If the type argument is not set, Node.js will autodetect