refactor(rule001): use new and improved word iterator

Cargo.toml

@@ -14,7 +14,6 @@ glob = "0.3.1"
 itertools = "0.13.0"
 log = "0.4.22"
 markdown = "1.0.0-alpha.21"
-once_cell = "1.20.2"
 regex = "1.11.0"
 serde = { version = "1.0.210", features = ["derive"] }
 serde_json = "1.0.128"

src/config.rs

@@ -176,10 +176,10 @@ impl Config {
                         }
                     });
                 }
-                toml::Value::Boolean(false) if RuleRegistry::is_valid_rule(&key) => {
+                toml::Value::Boolean(false) if registry.is_valid_rule(&key) => {
                     filtered_rules.insert(key.clone());
                 }
-                toml::Value::Table(table) if RuleRegistry::is_valid_rule(&key) => {
+                toml::Value::Table(table) if registry.is_valid_rule(&key) => {
                     let level = table.get("level");
                     if let Some(toml::Value::String(level)) = level.as_ref() {
                         match TryInto::<LintLevel>::try_into(level.as_str()) {

src/geometry.rs

@@ -164,6 +164,21 @@ impl DerefMut for AdjustedRange {
     }
 }
 
+impl From<AdjustedRange> for Range<usize> {
+    fn from(range: AdjustedRange) -> Self {
+        Self::from(&range)
+    }
+}
+
+impl From<&AdjustedRange> for Range<usize> {
+    fn from(range: &AdjustedRange) -> Self {
+        Self {
+            start: range.start.into(),
+            end: range.end.into(),
+        }
+    }
+}
+
 impl AdjustedRange {
     pub fn new(start: AdjustedOffset, end: AdjustedOffset) -> Self {
         Self(Range { start, end })

src/rules.rs

@@ -1,7 +1,6 @@
 use anyhow::Result;
 use log::{debug, error, warn};
 use markdown::mdast::Node;
-use once_cell::sync::Lazy;
 use regex::Regex;
 use std::{collections::HashMap, fmt::Debug};
 
@@ -18,15 +17,15 @@ mod rule002_admonition_types;
 use rule001_heading_case::Rule001HeadingCase;
 use rule002_admonition_types::Rule002AdmonitionTypes;
 
-static ALL_RULES: Lazy<Vec<Box<dyn Rule>>> = Lazy::new(|| {
+fn get_all_rules() -> Vec<Box<dyn Rule>> {
     vec![
         Box::new(Rule001HeadingCase::default()),
         Box::new(Rule002AdmonitionTypes::default()),
     ]
-});
+}
 
 #[allow(private_bounds)] // RuleClone is used within this module tree only
-pub trait Rule: Send + Sync + Debug + RuleName + RuleClone {
+pub trait Rule: Debug + RuleName + RuleClone {
     fn default_level(&self) -> LintLevel;
     fn setup(&mut self, _settings: Option<&RuleSettings>) {}
     fn check(&self, ast: &Node, context: &RuleContext, level: LintLevel) -> Option<Vec<LintError>>;
@@ -236,19 +235,15 @@ enum RuleRegistryState {
 
 impl RuleRegistry {
     pub fn new() -> Self {
-        let mut rules = Vec::<Box<dyn Rule>>::with_capacity(ALL_RULES.len());
-        ALL_RULES.iter().for_each(|rule| {
-            rules.push(rule.clone());
-        });
         Self {
             state: RuleRegistryState::PreSetup,
-            rules,
+            rules: get_all_rules(),
             configured_levels: Default::default(),
         }
     }
 
-    pub fn is_valid_rule(rule_name: &str) -> bool {
-        ALL_RULES.iter().any(|rule| rule.name() == rule_name)
+    pub fn is_valid_rule(&self, rule_name: &str) -> bool {
+        self.rules.iter().any(|rule| rule.name() == rule_name)
     }
 
     pub fn deactivate_rule(&mut self, rule_name: &str) {