Revert "perf(linter): use OsStr for faster path comparison and hash…

…ing (#9685)"

This reverts commit bcbb468.

crates/oxc_linter/src/module_graph_visitor.rs

@@ -1,4 +1,4 @@
-use std::{ffi::OsStr, marker::PhantomData, sync::Arc};
+use std::{marker::PhantomData, path::PathBuf, sync::Arc};
 
 use oxc_span::CompactStr;
 use rustc_hash::FxHashSet;
@@ -124,7 +124,7 @@ impl<T> Default for ModuleGraphVisitorBuilder<'_, T> {
 
 pub struct ModuleGraphVisitResult<T> {
     pub result: T,
-    pub _traversed: FxHashSet<Box<OsStr>>,
+    pub _traversed: FxHashSet<PathBuf>,
     pub _max_depth: u32,
 }
 
@@ -136,7 +136,7 @@ impl<T> ModuleGraphVisitResult<T> {
 
 #[derive(Debug)]
 struct ModuleGraphVisitor {
-    traversed: FxHashSet<Box<OsStr>>,
+    traversed: FxHashSet<PathBuf>,
     depth: u32,
     max_depth: u32,
 }
@@ -203,7 +203,7 @@ impl ModuleGraphVisitor {
             }
 
             let path = &pair.1.resolved_absolute_path;
-            if !self.traversed.insert(path.clone().into_os_string().into_boxed_os_str()) {
+            if !self.traversed.insert(path.clone()) {
                 continue;
             }
 

crates/oxc_linter/src/rules/import/no_cycle.rs

@@ -1,5 +1,5 @@
 #![expect(clippy::cast_possible_truncation)]
-use std::{env, ffi::OsStr, path::Component, sync::Arc};
+use std::{ffi::OsStr, path::Component, sync::Arc};
 
 use cow_utils::CowUtils;
 use oxc_diagnostics::OxcDiagnostic;
@@ -119,7 +119,7 @@ impl Rule for NoCycle {
         let module_record = ctx.module_record();
 
         let needle = &module_record.resolved_absolute_path;
-        let cwd = env::current_dir().unwrap();
+        let cwd = std::env::current_dir().unwrap();
 
         let mut stack = Vec::new();
         let ignore_types = self.ignore_types;
@@ -161,7 +161,7 @@ impl Rule for NoCycle {
             })
             .visit_fold(false, module_record, |_, (_, val), _| {
                 let path = &val.resolved_absolute_path;
-                if path.as_os_str() == needle.as_os_str() {
+                if path == needle {
                     VisitFoldWhile::Stop(true)
                 } else {
                     VisitFoldWhile::Next(false)