From e5b857ca8e01981f61ecd4001c834adc8066ed8f Mon Sep 17 00:00:00 2001
From: Keming <kemingy94@gmail.com>
Date: Fri, 11 Apr 2025 19:57:20 +0800
Subject: [PATCH] fix: replace once_cell to make cargo clippy happy

Signed-off-by: Keming <kemingy94@gmail.com>
---
 Cargo.lock                                  |  2 --
 lychee-bin/Cargo.toml                       |  2 +-
 lychee-bin/src/archive/wayback/mod.rs       |  6 +++---
 lychee-bin/src/commands/check.rs            |  2 +-
 lychee-bin/src/formatters/color.rs          | 23 ++++++++++++---------
 lychee-bin/src/formatters/response/color.rs |  2 +-
 lychee-bin/src/formatters/stats/compact.rs  |  4 ++--
 lychee-bin/src/stats.rs                     |  2 +-
 lychee-lib/Cargo.toml                       |  2 +-
 lychee-lib/src/client.rs                    |  2 +-
 lychee-lib/src/extract/markdown.rs          |  2 +-
 lychee-lib/src/filter/mod.rs                | 21 ++++++++++---------
 lychee-lib/src/quirks/mod.rs                | 16 +++++++-------
 lychee-lib/src/types/accept/range.rs        | 10 ++++-----
 lychee-lib/src/types/input.rs               |  2 +-
 lychee-lib/src/types/status.rs              |  9 +++-----
 lychee-lib/src/types/uri/github.rs          |  6 +++---
 lychee-lib/src/types/uri/valid.rs           |  4 ++--
 lychee-lib/src/utils/fragment_checker.rs    |  2 +-
 lychee-lib/src/utils/path.rs                |  6 +++---
 lychee-lib/src/utils/url.rs                 |  4 ++--
 21 files changed, 64 insertions(+), 65 deletions(-)

diff --git a/Cargo.lock b/Cargo.lock
index a5ac06eeb6..97c8b86517 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -2534,7 +2534,6 @@ dependencies = [
  "indicatif",
  "log",
  "lychee-lib",
- "once_cell",
  "openssl-sys",
  "pad",
  "predicates",
@@ -2582,7 +2581,6 @@ dependencies = [
  "linkify",
  "log",
  "octocrab",
- "once_cell",
  "openssl-sys",
  "par-stream",
  "path-clean",
diff --git a/lychee-bin/Cargo.toml b/lychee-bin/Cargo.toml
index c406433a0e..231145a107 100644
--- a/lychee-bin/Cargo.toml
+++ b/lychee-bin/Cargo.toml
@@ -10,6 +10,7 @@ license = "Apache-2.0 OR MIT"
 repository = "https://github.com/lycheeverse/lychee"
 readme = "../README.md"
 version.workspace = true
+rust-version = "1.83.0"
 
 [dependencies]
 # NOTE: We need to specify the version of lychee-lib here because crates.io
@@ -31,7 +32,6 @@ humantime = "2.2.0"
 humantime-serde = "1.1.1"
 indicatif = "0.17.11"
 log = "0.4.27"
-once_cell = "1.21.3"
 openssl-sys = { version = "0.9.106", optional = true }
 pad = "0.1.6"
 regex = "1.11.1"
diff --git a/lychee-bin/src/archive/wayback/mod.rs b/lychee-bin/src/archive/wayback/mod.rs
index 3dfec70bd2..f818e7b306 100644
--- a/lychee-bin/src/archive/wayback/mod.rs
+++ b/lychee-bin/src/archive/wayback/mod.rs
@@ -1,13 +1,13 @@
 use std::time::Duration;
 
-use once_cell::sync::Lazy;
 use serde::de::Error as SerdeError;
 use serde::{Deserialize, Deserializer};
+use std::sync::LazyLock;
 
 use http::StatusCode;
 use reqwest::{Client, Error, Url};
-static WAYBACK_URL: Lazy<Url> =
-    Lazy::new(|| Url::parse("https://archive.org/wayback/available").unwrap());
+static WAYBACK_URL: LazyLock<Url> =
+    LazyLock::new(|| Url::parse("https://archive.org/wayback/available").unwrap());
 
 pub(crate) async fn get_wayback_link(url: &Url, timeout: Duration) -> Result<Option<Url>, Error> {
     let mut archive_url: Url = WAYBACK_URL.clone();
diff --git a/lychee-bin/src/commands/check.rs b/lychee-bin/src/commands/check.rs
index 641c762a46..4cb47468c1 100644
--- a/lychee-bin/src/commands/check.rs
+++ b/lychee-bin/src/commands/check.rs
@@ -183,7 +183,7 @@ where
         if let Some(pb) = &bar {
             pb.inc_length(1);
             pb.set_message(request.to_string());
-        };
+        }
         send_req
             .send(Ok(request))
             .await
diff --git a/lychee-bin/src/formatters/color.rs b/lychee-bin/src/formatters/color.rs
index eaa9929c27..4797d52988 100644
--- a/lychee-bin/src/formatters/color.rs
+++ b/lychee-bin/src/formatters/color.rs
@@ -2,20 +2,23 @@
 
 use console::Style;
 use log::Level;
-use once_cell::sync::Lazy;
+use std::sync::LazyLock;
 
-pub(crate) static NORMAL: Lazy<Style> = Lazy::new(Style::new);
-pub(crate) static DIM: Lazy<Style> = Lazy::new(|| Style::new().dim());
+pub(crate) static NORMAL: LazyLock<Style> = LazyLock::new(Style::new);
+pub(crate) static DIM: LazyLock<Style> = LazyLock::new(|| Style::new().dim());
 
-pub(crate) static GREEN: Lazy<Style> = Lazy::new(|| Style::new().color256(2).bold().bright());
-pub(crate) static BOLD_GREEN: Lazy<Style> = Lazy::new(|| Style::new().color256(82).bold().bright());
-pub(crate) static YELLOW: Lazy<Style> = Lazy::new(|| Style::new().yellow().bright());
-pub(crate) static BOLD_YELLOW: Lazy<Style> = Lazy::new(|| Style::new().yellow().bold().bright());
-pub(crate) static PINK: Lazy<Style> = Lazy::new(|| Style::new().color256(197));
-pub(crate) static BOLD_PINK: Lazy<Style> = Lazy::new(|| Style::new().color256(197).bold());
+pub(crate) static GREEN: LazyLock<Style> =
+    LazyLock::new(|| Style::new().color256(2).bold().bright());
+pub(crate) static BOLD_GREEN: LazyLock<Style> =
+    LazyLock::new(|| Style::new().color256(82).bold().bright());
+pub(crate) static YELLOW: LazyLock<Style> = LazyLock::new(|| Style::new().yellow().bright());
+pub(crate) static BOLD_YELLOW: LazyLock<Style> =
+    LazyLock::new(|| Style::new().yellow().bold().bright());
+pub(crate) static PINK: LazyLock<Style> = LazyLock::new(|| Style::new().color256(197));
+pub(crate) static BOLD_PINK: LazyLock<Style> = LazyLock::new(|| Style::new().color256(197).bold());
 
 // Used for debug log messages
-pub(crate) static BLUE: Lazy<Style> = Lazy::new(|| Style::new().blue().bright());
+pub(crate) static BLUE: LazyLock<Style> = LazyLock::new(|| Style::new().blue().bright());
 
 // Write output using predefined colors
 macro_rules! color {
diff --git a/lychee-bin/src/formatters/response/color.rs b/lychee-bin/src/formatters/response/color.rs
index 824a2f2ac2..7ebeb0f512 100644
--- a/lychee-bin/src/formatters/response/color.rs
+++ b/lychee-bin/src/formatters/response/color.rs
@@ -13,7 +13,7 @@ pub(crate) struct ColorFormatter;
 impl ColorFormatter {
     /// Determine the color for formatted output based on the status of the
     /// response.
-    fn status_color(status: &Status) -> &'static once_cell::sync::Lazy<console::Style> {
+    fn status_color(status: &Status) -> &'static std::sync::LazyLock<console::Style> {
         match status {
             Status::Ok(_) | Status::Cached(CacheStatus::Ok(_)) => &GREEN,
             Status::Excluded
diff --git a/lychee-bin/src/formatters/stats/compact.rs b/lychee-bin/src/formatters/stats/compact.rs
index 9b1ed905de..396931f493 100644
--- a/lychee-bin/src/formatters/stats/compact.rs
+++ b/lychee-bin/src/formatters/stats/compact.rs
@@ -1,6 +1,6 @@
 use anyhow::Result;
 use console::Style;
-use once_cell::sync::Lazy;
+use std::sync::LazyLock;
 use std::{
     fmt::{self, Display},
     time::Duration,
@@ -89,7 +89,7 @@ fn write_if_any(
     value: usize,
     symbol: &str,
     text: &str,
-    style: &Lazy<Style>,
+    style: &LazyLock<Style>,
     f: &mut fmt::Formatter<'_>,
 ) -> Result<(), fmt::Error> {
     if value > 0 {
diff --git a/lychee-bin/src/stats.rs b/lychee-bin/src/stats.rs
index fc5839902d..f339080ad0 100644
--- a/lychee-bin/src/stats.rs
+++ b/lychee-bin/src/stats.rs
@@ -65,7 +65,7 @@ impl ResponseStats {
     ///
     /// This function is used to update the counters (success, error, etc.)
     /// based on the given response status.
-    pub(crate) fn increment_status_counters(&mut self, status: &Status) {
+    pub(crate) const fn increment_status_counters(&mut self, status: &Status) {
         match status {
             Status::Ok(_) => self.successful += 1,
             Status::Error(_) => self.errors += 1,
diff --git a/lychee-lib/Cargo.toml b/lychee-lib/Cargo.toml
index b61e42c4cc..0058a78587 100644
--- a/lychee-lib/Cargo.toml
+++ b/lychee-lib/Cargo.toml
@@ -10,6 +10,7 @@ license = "Apache-2.0 OR MIT"
 repository = "https://github.com/lycheeverse/lychee"
 readme = "../README.md"
 version.workspace = true
+rust-version = "1.83.0"
 
 [dependencies]
 async-stream = "0.3.6"
@@ -30,7 +31,6 @@ ip_network = "0.4.1"
 linkify = "0.10.0"
 log = "0.4.27"
 octocrab = "0.44.0"
-once_cell = "1.21.3"
 openssl-sys = { version = "0.9.106", optional = true }
 path-clean = "1.0.1"
 percent-encoding = "2.3.1"
diff --git a/lychee-lib/src/client.rs b/lychee-lib/src/client.rs
index f9566f06e4..7fe8f08ca0 100644
--- a/lychee-lib/src/client.rs
+++ b/lychee-lib/src/client.rs
@@ -322,7 +322,7 @@ impl ClientBuilder {
                 "Found user-agent in headers: {}. Overriding it with {user_agent}.",
                 prev_user_agent.to_str().unwrap_or("�"),
             );
-        };
+        }
 
         headers.insert(
             header::TRANSFER_ENCODING,
diff --git a/lychee-lib/src/extract/markdown.rs b/lychee-lib/src/extract/markdown.rs
index ba3d8ed2b9..efd482366a 100644
--- a/lychee-lib/src/extract/markdown.rs
+++ b/lychee-lib/src/extract/markdown.rs
@@ -167,7 +167,7 @@ pub(crate) fn extract_markdown_fragments(input: &str) -> HashSet<String> {
             Event::Text(text) | Event::Code(text) => {
                 if in_heading {
                     heading_text.push_str(&text);
-                };
+                }
             }
 
             // An HTML node
diff --git a/lychee-lib/src/filter/mod.rs b/lychee-lib/src/filter/mod.rs
index f5f0165024..ed0dcfeb68 100644
--- a/lychee-lib/src/filter/mod.rs
+++ b/lychee-lib/src/filter/mod.rs
@@ -1,9 +1,9 @@
 mod excludes;
 mod includes;
 
-use once_cell::sync::Lazy;
 use regex::RegexSet;
 use std::collections::HashSet;
+use std::sync::LazyLock;
 
 pub use excludes::Excludes;
 pub use includes::Includes;
@@ -14,23 +14,24 @@ use crate::Uri;
 /// These domains are explicitly defined by RFC 2606, section 3 Reserved Example
 /// Second Level Domain Names for describing example cases and should not be
 /// dereferenced as they should not have content.
-static EXAMPLE_DOMAINS: Lazy<HashSet<&'static str>> =
-    Lazy::new(|| HashSet::from_iter(["example.com", "example.org", "example.net", "example.edu"]));
+static EXAMPLE_DOMAINS: LazyLock<HashSet<&'static str>> = LazyLock::new(|| {
+    HashSet::from_iter(["example.com", "example.org", "example.net", "example.edu"])
+});
 
 #[cfg(all(not(test), not(feature = "check_example_domains")))]
 /// We also exclude the example TLDs in section 2 of the same RFC.
 /// This exclusion gets subsumed by the `check_example_domains` feature.
-static EXAMPLE_TLDS: Lazy<HashSet<&'static str>> =
-    Lazy::new(|| HashSet::from_iter([".test", ".example", ".invalid", ".localhost"]));
+static EXAMPLE_TLDS: LazyLock<HashSet<&'static str>> =
+    LazyLock::new(|| HashSet::from_iter([".test", ".example", ".invalid", ".localhost"]));
 
 // Allow usage of example domains in tests
 #[cfg(any(test, feature = "check_example_domains"))]
-static EXAMPLE_DOMAINS: Lazy<HashSet<&'static str>> = Lazy::new(HashSet::new);
+static EXAMPLE_DOMAINS: LazyLock<HashSet<&'static str>> = LazyLock::new(HashSet::new);
 
 #[cfg(any(test, feature = "check_example_domains"))]
-static EXAMPLE_TLDS: Lazy<HashSet<&'static str>> = Lazy::new(HashSet::new);
+static EXAMPLE_TLDS: LazyLock<HashSet<&'static str>> = LazyLock::new(HashSet::new);
 
-static UNSUPPORTED_DOMAINS: Lazy<HashSet<&'static str>> = Lazy::new(|| {
+static UNSUPPORTED_DOMAINS: LazyLock<HashSet<&'static str>> = LazyLock::new(|| {
     HashSet::from_iter([
         // Twitter requires an account to view tweets
         // https://news.ycombinator.com/item?id=36540957
@@ -50,8 +51,8 @@ const FALSE_POSITIVE_PAT: &[&str] = &[
     r"^https?://(.*)/xmlrpc.php$",
 ];
 
-static FALSE_POSITIVE_SET: Lazy<RegexSet> =
-    Lazy::new(|| regex::RegexSet::new(FALSE_POSITIVE_PAT).expect("Failed to create RegexSet"));
+static FALSE_POSITIVE_SET: LazyLock<RegexSet> =
+    LazyLock::new(|| regex::RegexSet::new(FALSE_POSITIVE_PAT).expect("Failed to create RegexSet"));
 
 #[inline]
 #[must_use]
diff --git a/lychee-lib/src/quirks/mod.rs b/lychee-lib/src/quirks/mod.rs
index d9f96de07f..e161369ed6 100644
--- a/lychee-lib/src/quirks/mod.rs
+++ b/lychee-lib/src/quirks/mod.rs
@@ -5,17 +5,17 @@ use crate::{
 use async_trait::async_trait;
 use header::HeaderValue;
 use http::header;
-use once_cell::sync::Lazy;
 use regex::Regex;
 use reqwest::{Request, Url};
 use std::collections::HashMap;
+use std::sync::LazyLock;
 
-static CRATES_PATTERN: Lazy<Regex> =
-    Lazy::new(|| Regex::new(r"^(https?://)?(www\.)?crates.io").unwrap());
-static YOUTUBE_PATTERN: Lazy<Regex> =
-    Lazy::new(|| Regex::new(r"^(https?://)?(www\.)?youtube(-nocookie)?\.com").unwrap());
-static YOUTUBE_SHORT_PATTERN: Lazy<Regex> =
-    Lazy::new(|| Regex::new(r"^(https?://)?(www\.)?(youtu\.?be)").unwrap());
+static CRATES_PATTERN: LazyLock<Regex> =
+    LazyLock::new(|| Regex::new(r"^(https?://)?(www\.)?crates.io").unwrap());
+static YOUTUBE_PATTERN: LazyLock<Regex> =
+    LazyLock::new(|| Regex::new(r"^(https?://)?(www\.)?youtube(-nocookie)?\.com").unwrap());
+static YOUTUBE_SHORT_PATTERN: LazyLock<Regex> =
+    LazyLock::new(|| Regex::new(r"^(https?://)?(www\.)?(youtu\.?be)").unwrap());
 
 // Retrieve a map of query params for the given request
 fn query(request: &Request) -> HashMap<String, String> {
@@ -24,7 +24,7 @@ fn query(request: &Request) -> HashMap<String, String> {
 
 #[derive(Debug, Clone)]
 pub(crate) struct Quirk {
-    pub(crate) pattern: &'static Lazy<Regex>,
+    pub(crate) pattern: &'static LazyLock<Regex>,
     pub(crate) rewrite: fn(Request) -> Request,
 }
 
diff --git a/lychee-lib/src/types/accept/range.rs b/lychee-lib/src/types/accept/range.rs
index dff721ad7e..c68cd35c5d 100644
--- a/lychee-lib/src/types/accept/range.rs
+++ b/lychee-lib/src/types/accept/range.rs
@@ -1,11 +1,11 @@
 use std::{fmt::Display, num::ParseIntError, ops::RangeInclusive, str::FromStr};
 
-use once_cell::sync::Lazy;
 use regex::Regex;
+use std::sync::LazyLock;
 use thiserror::Error;
 
-static RANGE_PATTERN: Lazy<Regex> =
-    Lazy::new(|| Regex::new(r"^([0-9]{3})?\.\.((=?)([0-9]{3}))?$|^([0-9]{3})$").unwrap());
+static RANGE_PATTERN: LazyLock<Regex> =
+    LazyLock::new(|| Regex::new(r"^([0-9]{3})?\.\.((=?)([0-9]{3}))?$|^([0-9]{3})$").unwrap());
 
 /// Indicates that the parsing process of an [`AcceptRange`]  from a string
 /// failed due to various underlying reasons.
@@ -113,7 +113,7 @@ impl AcceptRange {
         self.0
     }
 
-    pub(crate) fn update_start(&mut self, new_start: u16) -> Result<(), AcceptRangeError> {
+    pub(crate) const fn update_start(&mut self, new_start: u16) -> Result<(), AcceptRangeError> {
         let end = *self.end();
 
         if new_start > end {
@@ -124,7 +124,7 @@ impl AcceptRange {
         Ok(())
     }
 
-    pub(crate) fn update_end(&mut self, new_end: u16) -> Result<(), AcceptRangeError> {
+    pub(crate) const fn update_end(&mut self, new_end: u16) -> Result<(), AcceptRangeError> {
         let start = *self.start();
 
         if start > new_end {
diff --git a/lychee-lib/src/types/input.rs b/lychee-lib/src/types/input.rs
index effce2f1f0..104f788b07 100644
--- a/lychee-lib/src/types/input.rs
+++ b/lychee-lib/src/types/input.rs
@@ -263,7 +263,7 @@ impl Input {
                             Err(_) if skip_missing => (),
                             Err(e) => Err(e)?,
                             Ok(content) => yield content,
-                        };
+                        }
                     }
                 },
                 InputSource::Stdin => {
diff --git a/lychee-lib/src/types/status.rs b/lychee-lib/src/types/status.rs
index bb845c715e..38535b64b3 100644
--- a/lychee-lib/src/types/status.rs
+++ b/lychee-lib/src/types/status.rs
@@ -113,14 +113,14 @@ impl Status {
             CacheStatus::Ok(code) => {
                 if matches!(s, CacheStatus::Ok(_)) || accepted.contains(&code) {
                     return Self::Cached(CacheStatus::Ok(code));
-                };
+                }
                 Self::Cached(CacheStatus::Error(Some(code)))
             }
             CacheStatus::Error(code) => {
                 if let Some(code) = code {
                     if accepted.contains(&code) {
                         return Self::Cached(CacheStatus::Ok(code));
-                    };
+                    }
                 }
                 Self::Cached(CacheStatus::Error(code))
             }
@@ -225,10 +225,7 @@ impl Status {
                 }
             }
             Status::Cached(CacheStatus::Ok(code) | CacheStatus::Error(Some(code))) => {
-                match StatusCode::from_u16(*code) {
-                    Ok(code) => Some(code),
-                    Err(_) => None,
-                }
+                StatusCode::from_u16(*code).ok()
             }
             _ => None,
         }
diff --git a/lychee-lib/src/types/uri/github.rs b/lychee-lib/src/types/uri/github.rs
index 514f2e0e76..867c4c63d2 100644
--- a/lychee-lib/src/types/uri/github.rs
+++ b/lychee-lib/src/types/uri/github.rs
@@ -1,10 +1,10 @@
 use std::collections::HashSet;
 
-use once_cell::sync::Lazy;
+use std::sync::LazyLock;
 
 use crate::{ErrorKind, Result, Uri};
 
-static GITHUB_API_EXCLUDED_ENDPOINTS: Lazy<HashSet<&'static str>> = Lazy::new(|| {
+static GITHUB_API_EXCLUDED_ENDPOINTS: LazyLock<HashSet<&'static str>> = LazyLock::new(|| {
     HashSet::from_iter([
         "about",
         "collections",
@@ -74,7 +74,7 @@ impl GithubUri {
             "github.com" | "www.github.com" | "raw.githubusercontent.com"
         ) {
             return Err(ErrorKind::InvalidGithubUrl(uri.to_string()));
-        };
+        }
 
         let parts: Vec<_> = match uri.path_segments() {
             Some(parts) => parts.collect(),
diff --git a/lychee-lib/src/types/uri/valid.rs b/lychee-lib/src/types/uri/valid.rs
index 522acc4849..74fb8dd81d 100644
--- a/lychee-lib/src/types/uri/valid.rs
+++ b/lychee-lib/src/types/uri/valid.rs
@@ -254,8 +254,8 @@ impl TryFrom<&str> for Uri {
                     // which will allow `Url::parse` to parse them.
                     if let Ok(uri) = Url::parse(&format!("mailto:{s}")) {
                         return Ok(uri.into());
-                    };
-                };
+                    }
+                }
 
                 // We do not handle relative URLs here, as we do not know the base URL.
                 Err(ErrorKind::ParseUrl(err, s.to_owned()))
diff --git a/lychee-lib/src/utils/fragment_checker.rs b/lychee-lib/src/utils/fragment_checker.rs
index 789b52bb84..5e1a7a4a6a 100644
--- a/lychee-lib/src/utils/fragment_checker.rs
+++ b/lychee-lib/src/utils/fragment_checker.rs
@@ -50,7 +50,7 @@ impl FragmentChecker {
         };
         if fragment.is_empty() || fragment.eq_ignore_ascii_case("top") {
             return Ok(true);
-        };
+        }
         let mut fragment_decoded = percent_decode_str(fragment).decode_utf8()?;
         let url_without_frag = Self::remove_fragment(url.clone());
 
diff --git a/lychee-lib/src/utils/path.rs b/lychee-lib/src/utils/path.rs
index daa4f7fde6..c25745b792 100644
--- a/lychee-lib/src/utils/path.rs
+++ b/lychee-lib/src/utils/path.rs
@@ -1,13 +1,13 @@
 use crate::{ErrorKind, Result};
 use cached::proc_macro::cached;
-use once_cell::sync::Lazy;
 use path_clean::PathClean;
 use std::env;
 use std::fs;
 use std::path::{Path, PathBuf};
+use std::sync::LazyLock;
 
-static CURRENT_DIR: Lazy<PathBuf> =
-    Lazy::new(|| env::current_dir().expect("cannot get current dir from environment"));
+static CURRENT_DIR: LazyLock<PathBuf> =
+    LazyLock::new(|| env::current_dir().expect("cannot get current dir from environment"));
 
 /// Create an absolute path out of a `PathBuf`.
 ///
diff --git a/lychee-lib/src/utils/url.rs b/lychee-lib/src/utils/url.rs
index c27f0a0202..26cc2920a1 100644
--- a/lychee-lib/src/utils/url.rs
+++ b/lychee-lib/src/utils/url.rs
@@ -1,8 +1,8 @@
 use linkify::LinkFinder;
 
-use once_cell::sync::Lazy;
+use std::sync::LazyLock;
 
-static LINK_FINDER: Lazy<LinkFinder> = Lazy::new(LinkFinder::new);
+static LINK_FINDER: LazyLock<LinkFinder> = LazyLock::new(LinkFinder::new);
 
 /// Remove all GET parameters from a URL and separates out the fragment.
 /// The link is not a URL but a String as it may not have a base domain.