@photostructure/fs-metadata 1.2.0 → 1.4.0

package/CHANGELOG.md

@@ -14,6 +14,43 @@ Fixed for any bug fixes.
 Security in case of vulnerabilities.
 -->
 
+## 1.4.0 - 2026-04-20
+
+### Removed
+
+- **Linux GIO/GLib integration removed.** The optional `enable_gio` build-time
+  flag, the `getGioMountPoints` native binding, and the entire `src/linux/gio_*`
+  source tree have been deleted. The Linux mount-points and volume-metadata
+  paths now use `/proc/self/mounts` parsing exclusively.
+
+  Background: GIO was originally added to surface auto-mounted volumes (USB
+  sticks, external SSDs) that appeared to be missing from `/proc/mounts`. The
+  earlier mount-table read order has since been fixed, and direct verification
+  on Ubuntu 24 confirmed that udisks2-mounted devices (`/media/$USER/...`) are
+  fully visible via `/proc/self/mounts` without GIO. The remaining GIO code
+  path was a thin wrapper around `g_unix_mounts_get()` (which itself reads
+  `/proc/self/mountinfo`) — functionally redundant with the existing mtab
+  parser. The GVolumeMonitor enrichment path that originally justified GIO had
+  already been removed for thread-safety reasons in an earlier release.
+
+  Net effect: smaller native module, no `libglib2.0-dev` build dependency, no
+  `libgio` runtime dependency, simpler Linux build matrix. No public API
+  changes — `getVolumeMountPoints()` and `getVolumeMetadata()` return the same
+  data as before.
+
+### Changed
+
+- Linux build no longer requires `libglib2.0-dev`. Only `libblkid-dev` (and
+  `uuid-dev` for the dev environment) is needed when compiling from source.
+- `scripts/setup-native.mjs` removed — its only purpose was GIO autodetection.
+  `npm run build` and `npm run build:native` no longer invoke it.
+
+## 1.3.0 - 2026-03-26
+
+### Added
+
+- New optional `mountPoints` field on `Options`: pass pre-fetched mount points to `getMountPointForPath()` and `getVolumeMetadataForPath()` to avoid redundant system queries when resolving multiple paths. Obtain via `getVolumeMountPoints({ includeSystemVolumes: true })`.
+
 ## 1.2.0 - 2026-03-26
 
 ### Added

package/CONTRIBUTING.md

@@ -41,7 +41,7 @@ Install the Xcode Command Line Tools, and then
 
 ### On Ubuntu/Debian
 
-    sudo apt-get install bear build-essential clang-format libglib2.0-dev libblkid-dev uuid-dev
+    sudo apt-get install bear build-essential clang-format libblkid-dev uuid-dev
 
 ## Before submitting your PR
 
@@ -57,7 +57,7 @@ into account both Windows and POSIX systems.
 
 - macOS on x86 and Apple Silicon
 - Windows on x64
-- glibc Linux on x64 and arm64, with or without Gnome GIO support
+- glibc Linux on x64 and arm64
 - MUSL Alpine Linux on x64 and arm64
 
 ## Development Gotchas
@@ -121,7 +121,7 @@ Actions that have multiple targets can be run in parallel using wildcards:
 
 ### Naming Guidelines
 
-- Use explicit names to avoid ambiguity (e.g., `setup:native` instead of just `setup`)
+- Use explicit names to avoid ambiguity (e.g., `build:native` instead of just `build`)
 - Group related scripts by action prefix for easy wildcard execution
 - Avoid names that could cause npm lifecycle conflicts (e.g., `prebuild` vs `build`)
 - Use descriptive suffixes that clearly indicate the target or purpose

package/binding.gyp

@@ -1,7 +1,4 @@
 {
-  "variables": {
-    "enable_gio%": "false"
-  },
   "targets": [
     {
       "target_name": "fs_metadata",
@@ -57,25 +54,6 @@
                   "cflags": ["-mbranch-protection=standard"],
                   "cflags_cc": ["-mbranch-protection=standard"]
                 }
-              ],
-              [
-                "enable_gio=='true'",
-                {
-                  "sources": [
-                    "src/linux/gio_utils.cpp",
-                    "src/linux/gio_mount_points.cpp",
-                    "src/linux/gio_volume_metadata.cpp"
-                  ],
-                  "defines": [
-                    "ENABLE_GIO=1"
-                  ],
-                  "libraries": [
-                    "<!@(pkg-config --libs gio-unix-2.0)"
-                  ],
-                  "cflags": [
-                    "<!@(pkg-config --cflags gio-unix-2.0)"
-                  ]
-                }
               ]
             ]
           }

package/dist/index.cjs

@@ -694,11 +694,6 @@ async function* readLinks(directory) {
 // src/linux/mount_points.ts
 var import_promises4 = require("fs/promises");
 
-// src/mount_point.ts
-function isMountPoint(obj) {
-  return isObject(obj) && "mountPoint" in obj && isNotBlank(obj.mountPoint);
-}
-
 // src/options.ts
 var import_node_os2 = require("os");
 var import_node_process3 = require("process");
@@ -1180,49 +1175,25 @@ function parseMtab(content) {
 }
 
 // src/linux/mount_points.ts
-async function getLinuxMountPoints(native, opts) {
+async function getLinuxMountPoints(opts) {
   const o = optionsWithDefaults(opts);
-  const raw = [];
-  try {
-    const arr = await (await native()).getGioMountPoints?.();
-    debug("[getLinuxMountPoints] GIO mount points: %o", arr);
-    if (arr != null) raw.push(...arr);
-  } catch (error) {
-    debug("Failed to get GIO mount points: %s", error);
-  }
   let cause;
   for (const input of o.linuxMountTablePaths) {
     try {
       const mtabContent = await (0, import_promises4.readFile)(input, "utf8");
-      const arr = parseMtab(mtabContent).map((ea) => mountEntryToMountPoint(ea)).filter((ea) => ea != null);
-      debug("[getLinuxMountPoints] %s mount points: %o", input, arr);
-      if (arr.length > 0) {
-        raw.push(...arr);
-        break;
+      const results = parseMtab(mtabContent).map((ea) => mountEntryToMountPoint(ea)).filter((ea) => ea != null);
+      debug("[getLinuxMountPoints] %s mount points: %o", input, results);
+      if (results.length > 0) {
+        return results;
       }
     } catch (error) {
       cause ??= toError(error);
     }
   }
-  const byMountPoint = /* @__PURE__ */ new Map();
-  for (const ea of raw) {
-    const prior = byMountPoint.get(ea.mountPoint);
-    const merged = { ...compactValues(prior), ...compactValues(ea) };
-    if (isMountPoint(merged)) {
-      byMountPoint.set(merged.mountPoint, merged);
-    }
-  }
-  if (byMountPoint.size === 0) {
-    throw new WrappedError(
-      `Failed to find any mount points (tried: ${JSON.stringify(o.linuxMountTablePaths)})`,
-      { cause }
-    );
-  }
-  const results = [...byMountPoint.values()];
-  debug("[getLinuxMountPoints] %o", {
-    results: results.map((ea) => ea.mountPoint)
-  });
-  return results;
+  throw new WrappedError(
+    `Failed to find any mount points (tried: ${JSON.stringify(o.linuxMountTablePaths)})`,
+    { cause }
+  );
 }
 async function getLinuxMtabMetadata(mountPoint, opts) {
   let caughtError;
@@ -1349,7 +1320,7 @@ async function _getVolumeMountPoints(o, nativeFn2) {
       points.length
     );
     return points;
-  })() : getLinuxMountPoints(nativeFn2, o));
+  })() : getLinuxMountPoints(o));
   debug("[getVolumeMountPoints] raw mount points: %o", raw);
   const compacted = raw.map((ea) => compactValues(ea)).filter((ea) => isNotBlank(ea.mountPoint));
   for (const ea of compacted) {
@@ -1502,7 +1473,7 @@ async function getVolumeMetadataForPathImpl(pathname, opts, nativeFn2) {
 }
 async function findMountPointByDeviceId(resolved, resolvedStat, opts, nativeFn2) {
   const targetDev = resolvedStat.dev;
-  const mountPoints = await getVolumeMountPointsImpl(
+  const mountPoints = opts.mountPoints ?? await getVolumeMountPointsImpl(
     { ...opts, includeSystemVolumes: true },
     nativeFn2
   );