rumdl 0.0.149__tar.gz → 0.0.153__tar.gz

{rumdl-0.0.149 → rumdl-0.0.153}/Cargo.lock

@@ -1644,7 +1644,7 @@ checksum = "caf4aa5b0f434c91fe5c7f1ecb6a5ece2130b02ad2a590589dda5146df959001"
 
 [[package]]
 name = "rumdl"
-version = "0.0.149"
+version = "0.0.153"
 dependencies = [
  "anyhow",
  "assert_cmd",

{rumdl-0.0.149 → rumdl-0.0.153}/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rumdl"
-version = "0.0.149"
+version = "0.0.153"
 edition = "2024"
 rust-version = "1.89.0"
 description = "A fast Markdown linter written in Rust (Ru(st) MarkDown Linter)"

{rumdl-0.0.149 → rumdl-0.0.153}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rumdl
-Version: 0.0.149
+Version: 0.0.153
 Classifier: Development Status :: 4 - Beta
 Classifier: Environment :: Console
 Classifier: Intended Audience :: Developers

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md001_heading_increment.rs

@@ -179,8 +179,12 @@ impl Rule for MD001HeadingIncrement {
     }
 
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        // Skip if content is empty or has no headings
-        ctx.content.is_empty() || !ctx.lines.iter().any(|line| line.heading.is_some())
+        // Fast path: check if document likely has headings
+        if ctx.content.is_empty() || !ctx.likely_has_headings() {
+            return true;
+        }
+        // Verify headings actually exist
+        !ctx.lines.iter().any(|line| line.heading.is_some())
     }
 
     fn as_any(&self) -> &dyn std::any::Any {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md002_first_heading_h1.rs

@@ -281,8 +281,8 @@ impl Rule for MD002FirstHeadingH1 {
 
     /// Check if this rule should be skipped
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        let content = ctx.content;
-        content.is_empty() || (!content.contains('#') && !content.contains('=') && !content.contains('-'))
+        // Check for heading indicators: # for ATX, = or - for Setext
+        ctx.content.is_empty() || (!ctx.has_char('#') && !ctx.has_char('=') && !ctx.has_char('-'))
     }
 
     fn as_any(&self) -> &dyn std::any::Any {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md003_heading_style.rs

@@ -235,8 +235,12 @@ impl Rule for MD003HeadingStyle {
     }
 
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        // Skip if content is empty or has no headings
-        ctx.content.is_empty() || !ctx.lines.iter().any(|line| line.heading.is_some())
+        // Fast path: check if document likely has headings using character frequency
+        if ctx.content.is_empty() || !ctx.likely_has_headings() {
+            return true;
+        }
+        // Verify headings actually exist (handles false positives from character frequency)
+        !ctx.lines.iter().any(|line| line.heading.is_some())
     }
 
     fn as_any(&self) -> &dyn std::any::Any {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md004_unordered_list_style.rs

@@ -104,7 +104,7 @@ impl Rule for MD004UnorderedListStyle {
         }
 
         // Quick check for any list markers before processing
-        if !ctx.content.contains(['*', '-', '+']) {
+        if !ctx.likely_has_lists() {
             return Ok(Vec::new());
         }
 
@@ -314,7 +314,7 @@ impl Rule for MD004UnorderedListStyle {
 
     /// Check if this rule should be skipped
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        ctx.content.is_empty() || !ctx.content.contains(['*', '-', '+'])
+        ctx.content.is_empty() || !ctx.likely_has_lists()
     }
 
     fn as_any(&self) -> &dyn std::any::Any {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md006_start_bullets.rs

@@ -337,8 +337,7 @@ impl Rule for MD006StartBullets {
 
     /// Check if this rule should be skipped
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        let content = ctx.content;
-        content.is_empty() || (!content.contains('*') && !content.contains('-') && !content.contains('+'))
+        ctx.content.is_empty() || !ctx.likely_has_lists()
     }
 
     fn as_any(&self) -> &dyn std::any::Any {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md007_ul_indent.rs

@@ -124,9 +124,17 @@ impl Rule for MD007ULIndent {
                     Self::char_pos_to_visual_column(&line_info.content, list_item.content_column)
                 };
 
+                // For nesting detection, treat 1-space indent as if it's at column 0
+                // because 1 space is insufficient to establish a nesting relationship
+                let visual_marker_for_nesting = if visual_marker_column == 1 {
+                    0
+                } else {
+                    visual_marker_column
+                };
+
                 // Clean up stack - remove items at same or deeper indentation
                 while let Some(&(indent, _, _, _)) = list_stack.last() {
-                    if indent >= visual_marker_column {
+                    if indent >= visual_marker_for_nesting {
                         list_stack.pop();
                     } else {
                         break;
@@ -178,13 +186,14 @@ impl Rule for MD007ULIndent {
 
                     // Add current item to stack
                     // Use actual marker position for cleanup logic
-                    // For text-aligned children, store the ACTUAL content position
-                    // to match markdownlint's behavior (allows cascade)
-                    let actual_content_visual_col = visual_marker_column + 2; // where content ACTUALLY is
-                    list_stack.push((visual_marker_column, line_idx, false, actual_content_visual_col));
+                    // For text-aligned children, store the EXPECTED content position after fix
+                    // (not the actual position) to prevent error cascade
+                    let expected_content_visual_col = expected_indent + 2; // where content SHOULD be after fix
+                    list_stack.push((visual_marker_column, line_idx, false, expected_content_visual_col));
 
                     // Skip first level check if start_indented is false
-                    if !self.config.start_indented && nesting_level == 0 {
+                    // BUT always check items with 1 space indent (insufficient for nesting)
+                    if !self.config.start_indented && nesting_level == 0 && visual_marker_column != 1 {
                         continue;
                     }
 
@@ -295,12 +304,14 @@ impl Rule for MD007ULIndent {
 
     /// Check if this rule should be skipped
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        // Skip if content is empty or has no unordered list items
-        ctx.content.is_empty()
-            || !ctx
-                .lines
-                .iter()
-                .any(|line| line.list_item.as_ref().is_some_and(|item| !item.is_ordered))
+        // Fast path: check if document likely has lists
+        if ctx.content.is_empty() || !ctx.likely_has_lists() {
+            return true;
+        }
+        // Verify unordered list items actually exist
+        !ctx.lines
+            .iter()
+            .any(|line| line.list_item.as_ref().is_some_and(|item| !item.is_ordered))
     }
 
     fn as_any(&self) -> &dyn std::any::Any {
@@ -395,10 +406,10 @@ mod tests {
         let content = "* Item 1\n   * Item 2\n      * Item 3";
         let ctx = LintContext::new(content, crate::config::MarkdownFlavor::Standard);
         let result = rule.fix(&ctx).unwrap();
-        // With text-aligned style (cascade behavior like markdownlint):
+        // With text-aligned style and non-cascade:
         // Item 2 aligns with Item 1's text (2 spaces)
-        // Item 3 aligns with Item 2's ACTUAL text position (5 spaces)
-        let expected = "* Item 1\n  * Item 2\n     * Item 3";
+        // Item 3 aligns with Item 2's expected text position (4 spaces)
+        let expected = "* Item 1\n  * Item 2\n    * Item 3";
         assert_eq!(result, expected);
     }
 
@@ -608,17 +619,17 @@ repos:
         let content_multi = "* Item 1\n\t* Item 2\n\t\t* Item 3";
         let ctx = LintContext::new(content_multi, crate::config::MarkdownFlavor::Standard);
         let fixed = rule.fix(&ctx).unwrap();
-        // With cascade: Item 2 is at column 4 (tab), content at 6
-        // Item 3 aligns with Item 2's actual content at column 6
-        assert_eq!(fixed, "* Item 1\n  * Item 2\n      * Item 3");
+        // With non-cascade: Item 2 at 2 spaces, content at 4
+        // Item 3 aligns with Item 2's expected content at 4 spaces
+        assert_eq!(fixed, "* Item 1\n  * Item 2\n    * Item 3");
 
         // Mixed tabs and spaces
         let content_mixed = "* Item 1\n \t* Item 2\n\t * Item 3";
         let ctx = LintContext::new(content_mixed, crate::config::MarkdownFlavor::Standard);
         let fixed = rule.fix(&ctx).unwrap();
-        // With cascade: Item 2 at column 4 (space+tab), content at 6
-        // Item 3 at column 5 (tab+space), child of Item 2, aligns with content at 6
-        assert_eq!(fixed, "* Item 1\n  * Item 2\n      * Item 3");
+        // With non-cascade: Item 2 at 2 spaces, content at 4
+        // Item 3 aligns with Item 2's expected content at 4 spaces
+        assert_eq!(fixed, "* Item 1\n  * Item 2\n    * Item 3");
     }
 
     #[test]
@@ -722,9 +733,9 @@ tags:
         let content = "* Item 1 with **bold** and *italic*\n   * Item 2 with `code`\n     * Item 3 with [link](url)";
         let ctx = LintContext::new(content, crate::config::MarkdownFlavor::Standard);
         let fixed = rule.fix(&ctx).unwrap();
-        // With cascade: Item 2 is at column 3, content at 5
-        // Item 3 aligns with Item 2's actual content at column 5
-        let expected = "* Item 1 with **bold** and *italic*\n  * Item 2 with `code`\n     * Item 3 with [link](url)";
+        // With non-cascade: Item 2 at 2 spaces, content at 4
+        // Item 3 aligns with Item 2's expected content at 4 spaces
+        let expected = "* Item 1 with **bold** and *italic*\n  * Item 2 with `code`\n    * Item 3 with [link](url)";
         assert_eq!(fixed, expected, "Fix should only change indentation, not content");
     }
 
@@ -836,17 +847,17 @@ tags:
         assert!(result[0].message.contains("Expected 2 spaces"));
         assert!(result[0].message.contains("found 3"));
 
-        // Test insufficient indentation (1 space instead of 2)
+        // Test insufficient indentation (1 space is treated as level 0, should be 0)
         let content = "- Item 1\n - Item 2 with 1 space";
         let ctx = LintContext::new(content, crate::config::MarkdownFlavor::Standard);
         let result = rule.check(&ctx).unwrap();
         assert_eq!(
             result.len(),
             1,
-            "Should detect insufficient indentation (1 instead of 2)"
+            "Should detect 1-space indent (insufficient for nesting, expected 0)"
         );
         assert_eq!(result[0].line, 2);
-        assert!(result[0].message.contains("Expected 2 spaces"));
+        assert!(result[0].message.contains("Expected 0 spaces"));
         assert!(result[0].message.contains("found 1"));
     }
 

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md010_no_hard_tabs.rs

@@ -241,7 +241,7 @@ impl Rule for MD010NoHardTabs {
 
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
         // Skip if content is empty or has no tabs
-        ctx.content.is_empty() || !ctx.content.contains('\t')
+        ctx.content.is_empty() || !ctx.has_char('\t')
     }
 
     fn category(&self) -> RuleCategory {

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md011_no_reversed_links.rs

@@ -194,6 +194,10 @@ impl Rule for MD011NoReversedLinks {
         Ok(result)
     }
 
+    fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
+        ctx.content.is_empty() || !ctx.likely_has_links_or_images()
+    }
+
     fn as_any(&self) -> &dyn std::any::Any {
         self
     }

{rumdl-0.0.149 → rumdl-0.0.153}/src/rules/md012_no_multiple_blanks.rs

@@ -436,8 +436,8 @@ impl Rule for MD012NoMultipleBlanks {
     }
 
     fn should_skip(&self, ctx: &crate::lint_context::LintContext) -> bool {
-        // Skip if content is empty or doesn't have consecutive newlines
-        ctx.content.is_empty() || !ctx.content.contains("\n\n")
+        // Skip if content is empty or doesn't have newlines (single line can't have multiple blanks)
+        ctx.content.is_empty() || !ctx.has_char('\n')
     }
 
     fn default_config_section(&self) -> Option<(String, toml::Value)> {