fix: mixed posix/win32 namespaces (#197)

Author: pi0

src/path.ts src/_path.ts

@@ -26,13 +26,6 @@ const _PATH_ROOT_RE = /^[/\\]|^[a-zA-Z]:[/\\]/;
  */
 export const sep = "/";
 
-/**
- * The platform-specific file delimiter.
- *
- * Equals to `";"` in windows and `":"` in all other platforms.
- */
-export const delimiter = globalThis.process?.platform === "win32" ? ";" : ":";
-
 export const normalize: typeof path.normalize = function (path: string) {
   if (path.length === 0) {
     return ".";

src/index.ts

@@ -1,13 +1,36 @@
-import type NodePath from "node:path";
+import type { posix as Posix, win32 as Win32, PlatformPath } from "node:path";
 
-import * as path from "./path";
+import * as _path from "./_path";
 
-export * from "./path";
+export * from "./_path";
 
-export type Pathe = Omit<typeof NodePath, "posix" | "win32">;
+/**
+ * The platform-specific file delimiter.
+ *
+ * Equals to `";"` in windows and `":"` in all other platforms.
+ */
+export const delimiter: ";" | ":" = /* @__PURE__ */ (() =>
+  globalThis.process?.platform === "win32" ? ";" : ":")();
 
-export const posix = path satisfies Pathe;
+// Mix namespaces without side-effects of object to allow tree-shaking
 
-export const win32 = path satisfies Pathe;
+const _platforms = { posix: undefined, win32: undefined } as {
+  posix: typeof Posix;
+  win32: typeof Win32;
+};
 
-export default path satisfies Pathe;
+const mix = (del: ";" | ":" = delimiter) => {
+  return new Proxy(_path, {
+    get(_, prop) {
+      if (prop === "delimiter") return del;
+      if (prop === "posix") return posix;
+      if (prop === "win32") return win32;
+      return _platforms[prop] || _path[prop];
+    },
+  }) as unknown as PlatformPath;
+};
+
+export const posix = /* @__PURE__ */ mix(":") as typeof Posix;
+export const win32 = /* @__PURE__ */ mix(";") as typeof Win32;
+
+export default posix;

src/utils.ts

@@ -1,4 +1,4 @@
-import { join } from "./path";
+import { join } from "./_path";
 import { normalizeWindowsPath } from "./_internal";
 
 const pathSeparators = new Set(["/", "\\", undefined]);

test/index.spec.ts

@@ -16,6 +16,8 @@ import {
   toNamespacedPath,
   normalizeString,
   matchesGlob,
+  posix,
+  win32,
 } from "../src";
 
 import { normalizeWindowsPath } from "../src/_internal";
@@ -397,6 +399,8 @@ runTest("toNamespacedPath", toNamespacedPath, {
 
 describe("constants", () => {
   it("delimiter", () => {
+    expect(posix.delimiter).to.equal(":");
+    expect(win32.delimiter).to.equal(";");
     expect(delimiter).to.equal(
       globalThis.process?.platform === "win32" ? ";" : ":",
     );
@@ -407,6 +411,13 @@ describe("constants", () => {
   });
 });
 
+describe("mixed namespaces", () => {
+  it("nested namespaces work", () => {
+    expect(posix.win32.posix.delimiter).to.equal(":");
+    expect(win32.posix.win32.delimiter).to.equal(";");
+  });
+});
+
 runTest("matchesGlob", matchesGlob, [
   ["/foo/bar", "/foo/**", true],
   [String.raw`\foo\bar`, "/foo/**", true],