fix(path): replace win32 usages with custom functions, prettier

Author: jonluca

CLAUDE.md

@@ -5,12 +5,14 @@ This file provides guidance to Claude Code (claude.ai/code) when working with co
 ## Development Commands
 
 ### Building
+
 ```bash
 yarn build          # Compile TypeScript to dist/
 yarn typecheck      # Run TypeScript type checking without emitting files
 ```
 
 ### Testing
+
 ```bash
 yarn test           # Run all tests with coverage (Vitest)
 yarn test:node      # Run tests in Node.js environment
@@ -20,26 +22,32 @@ yarn test:update    # Update test snapshots
 ```
 
 ### Code Quality
+
 ```bash
 yarn lint           # Run ESLint on lib/ directory
 yarn prettier       # Format all code files
 ```
 
 ### Running Individual Tests
+
 To run a single test file:
+
 ```bash
 npx vitest test/specs/circular/circular.spec.ts
 ```
 
 To run tests matching a pattern:
+
 ```bash
 npx vitest --grep "circular"
 ```
 
 ## Architecture Overview
 
 ### Core Purpose
+
 This library parses, resolves, and dereferences JSON Schema `$ref` pointers. It handles references to:
+
 - External files (local filesystem)
 - HTTP/HTTPS URLs
 - Internal JSON pointers within schemas
@@ -49,7 +57,9 @@ This library parses, resolves, and dereferences JSON Schema `$ref` pointers. It
 ### Key Architecture Components
 
 #### 1. $RefParser (lib/index.ts)
+
 The main entry point and orchestrator class. Provides four primary operations:
+
 - **parse()**: Reads a single schema file (JSON/YAML) without resolving references
 - **resolve()**: Parses and resolves all `$ref` pointers, returns a `$Refs` object mapping references to values
 - **bundle()**: Converts external `$ref` pointers to internal ones (single file output)
@@ -58,35 +68,44 @@ The main entry point and orchestrator class. Provides four primary operations:
 All methods support both callback and Promise-based APIs, with multiple overload signatures.
 
 #### 2. $Refs (lib/refs.ts)
+
 A map/registry of all resolved JSON references and their values. Tracks:
+
 - All file paths/URLs encountered
 - Circular reference detection
 - Helper methods to query references by type (file, http, etc.)
 
 #### 3. Pointer (lib/pointer.ts)
+
 Represents a single JSON pointer (`#/definitions/person`) and implements JSON Pointer RFC 6901 spec:
+
 - Parses JSON pointer syntax (`/`, `~0`, `~1` escaping)
 - Resolves pointers to actual values within objects
 - Handles edge cases (null values, missing properties)
 
 #### 4. $Ref (lib/ref.ts)
+
 Wraps a single reference with metadata:
+
 - The reference path/URL
 - The resolved value
 - Path type (file, http, etc.)
 - Error information (when continueOnError is enabled)
 
 #### 5. Plugin System
+
 Two types of plugins, both configurable via options:
 
 **Parsers** (lib/parsers/):
+
 - JSON parser (json.ts)
 - YAML parser (yaml.ts) - uses js-yaml
 - Text parser (text.ts)
 - Binary parser (binary.ts)
 - Execute in order based on `order` property and `canParse()` matching
 
 **Resolvers** (lib/resolvers/):
+
 - File resolver (file.ts) - reads from filesystem (Node.js only)
 - HTTP resolver (http.ts) - fetches from URLs using native fetch
 - Custom resolvers can be added via options
@@ -100,7 +119,9 @@ Two types of plugins, both configurable via options:
 **lib/dereference.ts**: Replaces all `$ref` pointers with actual values, handles circular references
 
 #### 7. Options System (lib/options.ts)
+
 Hierarchical configuration with defaults for:
+
 - Which parsers/resolvers to enable
 - Circular reference handling (boolean or "ignore")
 - External reference resolution (relative vs root)
@@ -132,12 +153,14 @@ Hierarchical configuration with defaults for:
 ## Testing Strategy
 
 Tests are organized in test/specs/ by scenario:
-- Each scenario has test files (*.spec.ts) and fixture data
+
+- Each scenario has test files (\*.spec.ts) and fixture data
 - Tests validate parse, resolve, bundle, and dereference operations
 - Extensive coverage of edge cases: circular refs, deep nesting, special characters in paths
 - Browser-specific tests use test/fixtures/server.ts for HTTP mocking
 
 Test utilities:
+
 - test/utils/helper.js: Common test patterns
 - test/utils/path.js: Path handling for cross-platform tests
 - test/utils/serializeJson.ts: Custom snapshot serializer

lib/util/convert-path-to-posix.ts

@@ -1,11 +1,12 @@
 import path from "path";
 
+const win32Sep = "\\";
 export default function convertPathToPosix(filePath: string) {
   const isExtendedLengthPath = filePath.startsWith("\\\\?\\");
 
   if (isExtendedLengthPath) {
     return filePath;
   }
 
-  return filePath.split(path?.win32?.sep).join(path?.posix?.sep ?? "/");
+  return filePath.split(win32Sep).join(path?.posix?.sep ?? "/");
 }

lib/util/url.ts

@@ -1,5 +1,5 @@
 import convertPathToPosix from "./convert-path-to-posix.js";
-import path, { win32 } from "path";
+import path from "path";
 
 const forwardSlashPattern = /\//g;
 const protocolPattern = /^(\w{2,}):\/\//i;
@@ -9,6 +9,8 @@ const jsonPointerTilde = /~0/g;
 import { join } from "path";
 import { isWindows } from "./is-windows.js";
 
+const isAbsoluteWin32Path = /^[a-zA-Z]:\\/;
+
 // RegExp patterns to URL-encode special characters in local filesystem paths
 const urlEncodePatterns = [
   [/\?/g, "%3F"],
@@ -391,7 +393,7 @@ export function fromFileSystemPath(path: string) {
     const hasProjectDir = upperPath.includes(posixUpper);
     const hasProjectUri = upperPath.includes(posixUpper);
     const isAbsolutePath =
-      win32?.isAbsolute(path) ||
+      isAbsoluteWin32Path.test(path) ||
       path.startsWith("http://") ||
       path.startsWith("https://") ||
       path.startsWith("file://");

test/specs/bundle-defs-encoding/bundle-defs-encoding.spec.ts

@@ -10,7 +10,7 @@ describe("Bundle $defs encoding", () => {
     const bundled = await parser.bundle(schema);
 
     // Debug: log the bundled schema
-    console.log('Bundled schema:', JSON.stringify(bundled, null, 2));
+    console.log("Bundled schema:", JSON.stringify(bundled, null, 2));
 
     // The bundled schema should have $defs section with myschema that references otherschema
     expect(bundled).toHaveProperty("$defs");

test/specs/defs/dereferencedA.json

@@ -116,4 +116,4 @@
       ]
     }
   }
-}
+}

test/specs/defs/dereferencedB.json

@@ -116,4 +116,4 @@
       ]
     }
   }
-}
+}