fix(clippy): resolve lints for Rust 1.96 (io_other_error, derivable_impls, manual_contains, unnecessary_sort_by, while_let_loop)

crates/nuwiki-core/src/ast/block.rs

@@ -120,20 +120,15 @@ pub struct DefinitionItemNode {
     pub definitions: Vec<Vec<InlineNode>>,
 }
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Default)]
 pub enum TableAlign {
+    #[default]
     Default,
     Left,
     Right,
     Center,
 }
 
-impl Default for TableAlign {
-    fn default() -> Self {
-        Self::Default
-    }
-}
-
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct TableNode {
     pub span: Span,
@@ -170,9 +165,9 @@ pub struct CommentNode {
 ///
 /// - `File`        — line 0 or 1 of the document; tags the entire page.
 /// - `Heading(i)`  — within two lines below the i-th `HeadingNode` in the
-///                   document's `children`; tags that heading.
+///   document's `children`; tags that heading.
 /// - `Standalone`  — anywhere else; the tag is its own anchor on the source
-///                   line.
+///   line.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
 pub enum TagScope {
     File,

crates/nuwiki-core/src/ast/link.rs

@@ -9,8 +9,9 @@ use std::collections::HashMap;
 use super::inline::InlineNode;
 use super::span::Span;
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, Default)]
 pub enum LinkKind {
+    #[default]
     Wiki,
     Interwiki,
     Diary,
@@ -31,12 +32,6 @@ pub struct LinkTarget {
     pub is_directory: bool,
 }
 
-impl Default for LinkKind {
-    fn default() -> Self {
-        Self::Wiki
-    }
-}
-
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct WikiLinkNode {
     pub span: Span,

crates/nuwiki-core/src/syntax/registry.rs

@@ -39,7 +39,7 @@ impl SyntaxRegistry {
     pub fn detect_from_extension(&self, ext: &str) -> Option<&dyn SyntaxPlugin> {
         self.plugins
             .iter()
-            .find(|p| p.file_extensions().iter().any(|e| *e == ext))
+            .find(|p| p.file_extensions().contains(&ext))
             .map(|p| p.as_ref())
     }
 

crates/nuwiki-core/src/syntax/vimwiki/parser.rs

@@ -547,8 +547,7 @@ impl<'a> ParseState<'a> {
         // Text). We accept either as a continuation when the implied
         // indent is `> level`.
         let continuation_threshold = level + 1;
-        loop {
-            let Some(first) = self.peek() else { break };
+        while let Some(first) = self.peek() {
             let mut consume_first = false;
             let is_continuation = match &first.kind {
                 K::Text(s) => leading_ws_count(s) >= continuation_threshold,
@@ -561,7 +560,7 @@ impl<'a> ParseState<'a> {
                         consume_first = true;
                         true
                     } else {
-                        false
+                        break;
                     }
                 }
                 _ => false,

crates/nuwiki-ls/Cargo.toml

@@ -17,5 +17,5 @@ name = "nuwiki-ls"
 path = "src/main.rs"
 
 [dependencies]
-nuwiki-lsp = { path = "../nuwiki-lsp", version = "0.3.0" }
+nuwiki-lsp = { path = "../nuwiki-lsp", version = "0.4.1" }
 tokio = { version = "1", features = ["macros", "rt-multi-thread", "io-std"] }

crates/nuwiki-lsp/Cargo.toml

@@ -13,7 +13,7 @@ homepage.workspace = true
 workspace = true
 
 [dependencies]
-nuwiki-core = { path = "../nuwiki-core", version = "0.3.0" }
+nuwiki-core = { path = "../nuwiki-core", version = "0.4.1" }
 tower-lsp = "0.20"
 tokio = { version = "1", features = ["macros", "rt-multi-thread", "io-std", "sync", "fs"] }
 dashmap = "6"

crates/nuwiki-lsp/src/commands.rs

@@ -3762,17 +3762,11 @@ pub mod export_ops {
             .arg("-c")
             .arg(&cmd)
             .status()
-            .map_err(|e| {
-                std::io::Error::new(
-                    std::io::ErrorKind::Other,
-                    format!("custom_wiki2html spawn failed: {e}"),
-                )
-            })?;
+            .map_err(|e| std::io::Error::other(format!("custom_wiki2html spawn failed: {e}")))?;
         if !status.success() {
-            return Err(std::io::Error::new(
-                std::io::ErrorKind::Other,
-                format!("custom_wiki2html exited with {status}"),
-            ));
+            return Err(std::io::Error::other(format!(
+                "custom_wiki2html exited with {status}"
+            )));
         }
         Ok(PageExport {
             page: name.to_string(),
@@ -3821,7 +3815,7 @@ pub mod export_ops {
     /// rendered-HTML body per entry.
     pub fn write_rss(cfg: &WikiConfig, entries: &[DiaryEntry]) -> std::io::Result<PathBuf> {
         let mut sorted: Vec<&DiaryEntry> = entries.iter().collect();
-        sorted.sort_by(|a, b| b.date.cmp(&a.date));
+        sorted.sort_by_key(|a| std::cmp::Reverse(a.date));
         let cap = if cfg.html.rss_max_items == 0 {
             usize::MAX
         } else {

crates/nuwiki-lsp/src/diary.rs

@@ -47,7 +47,7 @@ pub fn list_entries(index: &WorkspaceIndex) -> Vec<DiaryEntry> {
             })
         })
         .collect();
-    out.sort_by(|a, b| a.date.cmp(&b.date));
+    out.sort_by_key(|a| a.date);
     out
 }
 

crates/nuwiki-lsp/src/lib.rs

@@ -406,7 +406,7 @@ impl LanguageServer for Backend {
             .general
             .as_ref()
             .and_then(|g| g.position_encodings.as_ref())
-            .map(|encs| encs.iter().any(|e| *e == PositionEncodingKind::UTF8))
+            .map(|encs| encs.contains(&PositionEncodingKind::UTF8))
             .unwrap_or(false);
         self.use_utf8.store(supports_utf8, Ordering::Relaxed);