refactor(diagnostics): remove thiserror

Cargo.lock

@@ -1368,7 +1368,6 @@ dependencies = [
  "miette",
  "owo-colors",
  "textwrap",
- "thiserror",
  "unicode-width",
 ]
 

Cargo.toml

@@ -132,7 +132,6 @@ serde               = "1.0.199"
 serde_json          = "1.0.116"
 syn                 = { version = "2.0.58", default-features = false }
 tempfile            = "3.10.1"
-thiserror           = "1.0.59"
 tokio               = "1"
 tower-lsp           = "0.20.0"
 trybuild            = "1.0.93"

crates/oxc_diagnostics/Cargo.toml

@@ -18,7 +18,6 @@ workspace = true
 doctest = false
 
 [dependencies]
-thiserror = { workspace = true }
 miette = { workspace = true }
 
 unicode-width = { workspace = true }

crates/oxc_diagnostics/src/lib.rs

@@ -1,5 +1,5 @@
 //! Diagnostics Wrapper
-//! Exports `thiserror` and `miette`
+//! Exports `miette`
 
 mod graphic_reporter;
 mod graphical_theme;
@@ -9,11 +9,9 @@ mod service;
 use std::{
     fmt::{self, Display},
     ops::Deref,
-    path::PathBuf,
 };
 
 pub use miette;
-pub use thiserror;
 
 pub use crate::{
     graphic_reporter::GraphicalReportHandler,
@@ -29,17 +27,6 @@ pub type Result<T> = std::result::Result<T, OxcDiagnostic>;
 
 use miette::{Diagnostic, SourceCode};
 pub use miette::{LabeledSpan, NamedSource};
-use thiserror::Error;
-
-#[derive(Debug, Error, Diagnostic)]
-#[error("File is too long to fit on the screen")]
-#[diagnostic(help("{0:?} seems like a minified file"))]
-pub struct MinifiedFileError(pub PathBuf);
-
-#[derive(Debug, Error, Diagnostic)]
-#[error("Failed to open file {0:?} with error \"{1}\"")]
-#[diagnostic(help("Failed to open file {0:?} with error \"{1}\""))]
-pub struct FailedToOpenFileError(pub PathBuf, pub std::io::Error);
 
 #[derive(Debug, Clone)]
 pub struct OxcDiagnostic {

crates/oxc_diagnostics/src/service.rs

@@ -10,7 +10,7 @@ use crate::{
         CheckstyleReporter, DiagnosticReporter, GithubReporter, GraphicalReporter, JsonReporter,
         UnixReporter,
     },
-    Error, MinifiedFileError, Severity,
+    Error, OxcDiagnostic, Severity,
 };
 
 pub type DiagnosticTuple = (PathBuf, Vec<Error>);
@@ -139,7 +139,10 @@ impl DiagnosticService {
                 if let Some(mut err_str) = self.reporter.render_error(diagnostic) {
                     // Skip large output and print only once
                     if err_str.lines().any(|line| line.len() >= 400) {
-                        let minified_diagnostic = Error::new(MinifiedFileError(path.clone()));
+                        let minified_diagnostic = Error::new(
+                            OxcDiagnostic::warning("File is too long to fit on the screen")
+                                .with_help(format!("{path:?} seems like a minified file")),
+                        );
                         err_str = format!("{minified_diagnostic:?}");
                         output = err_str;
                         break;

crates/oxc_linter/src/service.rs

@@ -12,7 +12,7 @@ use rayon::{iter::ParallelBridge, prelude::ParallelIterator};
 use rustc_hash::FxHashSet;
 
 use oxc_allocator::Allocator;
-use oxc_diagnostics::{DiagnosticSender, DiagnosticService, Error, FailedToOpenFileError};
+use oxc_diagnostics::{DiagnosticSender, DiagnosticService, Error, OxcDiagnostic};
 use oxc_parser::Parser;
 use oxc_resolver::Resolver;
 use oxc_semantic::{ModuleRecord, SemanticBuilder};
@@ -177,8 +177,11 @@ impl Runtime {
             return None;
         }
         let source_type = source_type.unwrap_or_default();
-        let file_result = fs::read_to_string(path)
+        let file_result = fs::read_to_string(path).map_err(|e| {
-            .map_err(|e| Error::new(FailedToOpenFileError(path.to_path_buf(), e)));
+            Error::new(OxcDiagnostic::error(format!(
+                "Failed to open file {path:?} with error \"{e}\""
+            )))
+        });
         Some(match file_result {
             Ok(source_text) => Ok((source_type, source_text)),
             Err(e) => Err(e),