rumdl 0.0.121__tar.gz → 0.0.123__tar.gz

{rumdl-0.0.121 → rumdl-0.0.123}/CHANGELOG.md

@@ -7,6 +7,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.0.123] - 2025-08-21
+
+## [0.0.123] - 2025-08-21
+
+### Added
+- **MD013**: Comprehensive markdown pattern preservation during text reflow
+  - Preserves reference links, footnotes, math formulas, wiki links, and more
+  - Centralized regex patterns for better maintainability
+
+### Fixed
+- **CLI**: Correct unfixable rules status display and fix counts (closes #56)
+  - Rules marked as unfixable now show `[unfixable]` in yellow instead of `[fixed]`
+  - Fix count now correctly excludes unfixable rules (e.g., "3 of 6" instead of "6 of 6")
+  - Added FixCapability enum to Rule trait for compile-time safety
+- **MD013**: Preserve reference links during text reflow
+  - Reference-style links are now properly preserved when reflowing text
+  - Fixed indicator display to correctly show `[fixed]` when issues are resolved
+- **Tests**: Mark kramdown definition list doctest as text to fix test failures
+
+### Changed
+- **Internal**: Centralized markdown pattern regexes and extended reflow support
+  - Improved code organization and reduced duplication
+  - Better performance through shared regex compilation
+
+## [0.0.122] - 2025-08-19
+
 ## [0.0.121] - 2025-08-19
 
 ### Fixed
@@ -370,7 +396,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Initial implementation of remaining rules for markdownlint parity
 
-[Unreleased]: https://github.com/rvben/rumdl/compare/v0.0.121...HEAD
+[Unreleased]: https://github.com/rvben/rumdl/compare/v0.0.123...HEAD
+[0.0.123]: https://github.com/rvben/rumdl/compare/v0.0.122...v0.0.123
+[0.0.122]: https://github.com/rvben/rumdl/compare/v0.0.121...v0.0.122
 [0.0.121]: https://github.com/rvben/rumdl/compare/v0.0.120...v0.0.121
 [0.0.120]: https://github.com/rvben/rumdl/compare/v0.0.119...v0.0.120
 [0.0.119]: https://github.com/rvben/rumdl/compare/v0.0.118...v0.0.119

{rumdl-0.0.121 → rumdl-0.0.123}/Cargo.lock

@@ -229,9 +229,9 @@ dependencies = [
 
 [[package]]
 name = "cfg-if"
-version = "1.0.1"
+version = "1.0.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9555578bc9e57714c812a1f84e4fc5b4d21fcb063490c624de019f7464c91268"
+checksum = "2fd1289c04a9ea8cb22300a459a72a385d7c73d3259e2ed7dcb2af674838cfa9"
 
 [[package]]
 name = "chrono"
@@ -530,9 +530,9 @@ checksum = "3f9eec918d3f24069decb9af1554cad7c880e2da24a9afd88aca000531ab82c1"
 
 [[package]]
 name = "form_urlencoded"
-version = "1.2.1"
+version = "1.2.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e13624c2627564efccf4934284bdd98cbaa14e79b0b5a141218e507b3a823456"
+checksum = "cb4cb245038516f5f85277875cdaa4f7d2c9a0fa0468de06ed190163b1581fcf"
 dependencies = [
  "percent-encoding",
 ]
@@ -806,9 +806,9 @@ dependencies = [
 
 [[package]]
 name = "idna"
-version = "1.0.3"
+version = "1.1.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "686f825264d630750a544639377bae737628043f20d38bbc029e8f29ea968a7e"
+checksum = "3b0875f23caa03898994f6ddc501886a45c7d3d62d04d2d90788d47be1b1e4de"
 dependencies = [
  "idna_adapter",
  "smallvec",
@@ -1126,9 +1126,9 @@ dependencies = [
 
 [[package]]
 name = "percent-encoding"
-version = "2.3.1"
+version = "2.3.2"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e3148f5046208a5d56bcfc03053e3ca6334e51da8dfb19b6cdc8b306fae3283e"
+checksum = "9b4f627cb1b25917193a259e49bdad08f671f8d9708acfd5fe0a8c1455d87220"
 
 [[package]]
 name = "pin-project"
@@ -1493,7 +1493,7 @@ checksum = "2b15c43186be67a4fd63bee50d0303afffcef381492ebe2c5d87f324e1b8815c"
 
 [[package]]
 name = "rumdl"
-version = "0.0.121"
+version = "0.0.123"
 dependencies = [
  "anyhow",
  "assert_cmd",
@@ -1751,15 +1751,15 @@ checksum = "e502f78cdbb8ba4718f566c418c52bc729126ffd16baee5baa718cf25dd5a69a"
 
 [[package]]
 name = "tempfile"
-version = "3.20.0"
+version = "3.21.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "e8a64e3985349f2441a1a9ef0b853f869006c3855f2cda6862a94d26ebb9d6a1"
+checksum = "15b61f8f20e3a6f7e0649d825294eaf317edce30f82cf6026e7e4cb9222a7d1e"
 dependencies = [
  "fastrand",
  "getrandom",
  "once_cell",
  "rustix",
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -1770,18 +1770,18 @@ checksum = "8f50febec83f5ee1df3015341d8bd429f2d1cc62bcba7ea2076759d315084683"
 
 [[package]]
 name = "thiserror"
-version = "2.0.15"
+version = "2.0.16"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "80d76d3f064b981389ecb4b6b7f45a0bf9fdac1d5b9204c7bd6714fecc302850"
+checksum = "3467d614147380f2e4e374161426ff399c91084acd2363eaf549172b3d5e60c0"
 dependencies = [
  "thiserror-impl",
 ]
 
 [[package]]
 name = "thiserror-impl"
-version = "2.0.15"
+version = "2.0.16"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "44d29feb33e986b6ea906bd9c3559a856983f92371b3eaa5e83782a351623de0"
+checksum = "6c5e1be1c48b9172ee610da68fd9cd2770e7a4056cb3fc98710ee6906f0c7960"
 dependencies = [
  "proc-macro2",
  "quote",
@@ -2080,9 +2080,9 @@ checksum = "673aac59facbab8a9007c7f6108d11f63b603f7cabff99fabf650fea5c32b861"
 
 [[package]]
 name = "url"
-version = "2.5.4"
+version = "2.5.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "32f8b686cadd1473f4bd0117a5d28d36b1ade384ea9b5069a1c40aefed7fda60"
+checksum = "137a3c834eaf7139b73688502f3f1141a0337c5d8e4d9b536f9b8c796e26a7c4"
 dependencies = [
  "form_urlencoded",
  "idna",
@@ -2206,11 +2206,11 @@ dependencies = [
 
 [[package]]
 name = "winapi-util"
-version = "0.1.9"
+version = "0.1.10"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "cf221c93e13a30d793f7645a0e7762c55d169dbb0a49671918a2319d289b10bb"
+checksum = "0978bf7171b3d90bac376700cb56d606feb40f251a475a5d6634613564460b22"
 dependencies = [
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]

{rumdl-0.0.121 → rumdl-0.0.123}/Cargo.toml

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

{rumdl-0.0.121 → rumdl-0.0.123}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rumdl
-Version: 0.0.121
+Version: 0.0.123
 Classifier: Development Status :: 4 - Beta
 Classifier: Environment :: Console
 Classifier: Intended Audience :: Developers
@@ -96,6 +96,8 @@ It offers:
     - [VS Code Extension](#vs-code-extension)
   - [Usage](#usage)
   - [Pre-commit Integration](#pre-commit-integration)
+  - [CI/CD Integration](#cicd-integration)
+    - [GitHub Actions](#github-actions)
   - [Rules](#rules)
   - [Command-line Interface](#command-line-interface)
     - [Commands](#commands)
@@ -270,6 +272,19 @@ repos:
 
 When you run `pre-commit install` or `pre-commit run`, pre-commit will automatically install `rumdl` in an isolated Python environment using pip. You do **not** need to install rumdl manually.
 
+## CI/CD Integration
+
+### GitHub Actions
+
+rumdl can output annotations directly in GitHub Actions format, making issues appear inline in pull requests:
+
+```yaml
+- name: Lint Markdown
+  run: rumdl check --output-format github .
+```
+
+This produces annotations that GitHub automatically displays in the PR's "Files changed" tab. Supports error/warning severity levels and precise issue locations.
+
 ## Rules
 
 rumdl implements 54 lint rules for Markdown files. Here are some key rule categories:

{rumdl-0.0.121 → rumdl-0.0.123}/README.md

@@ -67,6 +67,8 @@ It offers:
     - [VS Code Extension](#vs-code-extension)
   - [Usage](#usage)
   - [Pre-commit Integration](#pre-commit-integration)
+  - [CI/CD Integration](#cicd-integration)
+    - [GitHub Actions](#github-actions)
   - [Rules](#rules)
   - [Command-line Interface](#command-line-interface)
     - [Commands](#commands)
@@ -241,6 +243,19 @@ repos:
 
 When you run `pre-commit install` or `pre-commit run`, pre-commit will automatically install `rumdl` in an isolated Python environment using pip. You do **not** need to install rumdl manually.
 
+## CI/CD Integration
+
+### GitHub Actions
+
+rumdl can output annotations directly in GitHub Actions format, making issues appear inline in pull requests:
+
+```yaml
+- name: Lint Markdown
+  run: rumdl check --output-format github .
+```
+
+This produces annotations that GitHub automatically displays in the PR's "Files changed" tab. Supports error/warning severity levels and precise issue locations.
+
 ## Rules
 
 rumdl implements 54 lint rules for Markdown files. Here are some key rule categories:

{rumdl-0.0.121 → rumdl-0.0.123}/src/main.rs

@@ -2355,6 +2355,26 @@ fn print_statistics(warnings: &[rumdl::rule::LintWarning]) {
     );
 }
 
+// Helper function to check if a rule is actually fixable based on configuration
+fn is_rule_actually_fixable(config: &rumdl_config::Config, rule_name: &str) -> bool {
+    // Check unfixable list
+    if config
+        .global
+        .unfixable
+        .iter()
+        .any(|r| r.eq_ignore_ascii_case(rule_name))
+    {
+        return false;
+    }
+
+    // Check fixable list if specified
+    if !config.global.fixable.is_empty() {
+        return config.global.fixable.iter().any(|r| r.eq_ignore_ascii_case(rule_name));
+    }
+
+    true
+}
+
 // Process file with output formatter
 #[allow(clippy::too_many_arguments)]
 fn process_file_with_formatter(
@@ -2371,7 +2391,7 @@ fn process_file_with_formatter(
 
     // Call the original process_file_inner to get warnings
     let (all_warnings, mut content, total_warnings, fixable_warnings) =
-        process_file_inner(file_path, rules, verbose, quiet);
+        process_file_inner(file_path, rules, verbose, quiet, config);
 
     if total_warnings == 0 {
         return (false, 0, 0, 0, Vec::new());
@@ -2408,14 +2428,25 @@ fn process_file_with_formatter(
             // Create a custom formatter that shows [fixed] instead of [*]
             let mut output = String::new();
             for warning in &all_warnings {
+                let rule_name = warning.rule_name.unwrap_or("unknown");
+
+                // Check if the rule is actually fixable based on configuration
+                let is_fixable = is_rule_actually_fixable(config, rule_name);
+
                 let was_fixed = warning.fix.is_some()
+                    && is_fixable
                     && !remaining_warnings.iter().any(|w| {
                         w.line == warning.line && w.column == warning.column && w.rule_name == warning.rule_name
                     });
 
-                let rule_name = warning.rule_name.unwrap_or("unknown");
-                let fix_indicator = if was_fixed {
-                    " [fixed]".green().to_string()
+                let fix_indicator = if warning.fix.is_some() {
+                    if !is_fixable {
+                        " [unfixable]".yellow().to_string()
+                    } else if was_fixed {
+                        " [fixed]".green().to_string()
+                    } else {
+                        String::new()
+                    }
                 } else {
                     String::new()
                 };
@@ -2454,6 +2485,7 @@ fn process_file_inner(
     rules: &[Box<dyn Rule>],
     verbose: bool,
     quiet: bool,
+    config: &rumdl_config::Config,
 ) -> (Vec<rumdl::rule::LintWarning>, String, usize, usize) {
     use std::time::Instant;
 
@@ -2507,8 +2539,11 @@ fn process_file_inner(
 
     let total_warnings = all_warnings.len();
 
-    // Count fixable issues
-    let fixable_warnings = all_warnings.iter().filter(|w| w.fix.is_some()).count();
+    // Count fixable issues (excluding unfixable rules)
+    let fixable_warnings = all_warnings
+        .iter()
+        .filter(|w| w.fix.is_some() && w.rule_name.is_some_and(|name| is_rule_actually_fixable(config, name)))
+        .count();
 
     let lint_end_time = Instant::now();
     let lint_time = lint_end_time.duration_since(lint_start);

{rumdl-0.0.121 → rumdl-0.0.123}/src/output/formatters/github.rs

@@ -1,10 +1,13 @@
 //! GitHub Actions annotation format
+//!
+//! Outputs annotations in GitHub Actions workflow command format for PR annotations.
+//! See: https://docs.github.com/en/actions/reference/workflow-commands-for-github-actions
 
 use crate::output::OutputFormatter;
-use crate::rule::LintWarning;
+use crate::rule::{LintWarning, Severity};
 
 /// GitHub Actions formatter
-/// Outputs in the format: ::error file=<file>,line=<line>,col=<col>,title=<rule>::<message>
+/// Outputs in the format: ::<level> file=<file>,line=<line>,col=<col>,endLine=<endLine>,endColumn=<endCol>,title=<rule>::<message>
 pub struct GitHubFormatter;
 
 impl Default for GitHubFormatter {
@@ -17,6 +20,24 @@ impl GitHubFormatter {
     pub fn new() -> Self {
         Self
     }
+
+    /// Escape special characters according to GitHub Actions specification
+    /// Percent-encodes: %, \r, \n, :, ,
+    /// Used for property values (file, title, etc.)
+    fn escape_property(value: &str) -> String {
+        value
+            .replace('%', "%25")
+            .replace('\r', "%0D")
+            .replace('\n', "%0A")
+            .replace(':', "%3A")
+            .replace(',', "%2C")
+    }
+
+    /// Escape special characters in the message part
+    /// Percent-encodes: %, \r, \n
+    fn escape_message(value: &str) -> String {
+        value.replace('%', "%25").replace('\r', "%0D").replace('\n', "%0A")
+    }
 }
 
 impl OutputFormatter for GitHubFormatter {
@@ -26,12 +47,38 @@ impl OutputFormatter for GitHubFormatter {
         for warning in warnings {
             let rule_name = warning.rule_name.unwrap_or("unknown");
 
-            // GitHub Actions annotation format
-            // Use "warning" level since these are linting warnings, not errors
-            let line = format!(
-                "::warning file={},line={},col={},title={}::{}",
-                file_path, warning.line, warning.column, rule_name, warning.message
-            );
+            // Map severity to GitHub Actions annotation level
+            let level = match warning.severity {
+                Severity::Error => "error",
+                Severity::Warning => "warning",
+            };
+
+            // Escape special characters in all properties
+            let escaped_file = Self::escape_property(file_path);
+            let escaped_rule = Self::escape_property(rule_name);
+            let escaped_message = Self::escape_message(&warning.message);
+
+            // GitHub Actions annotation format with optional end position
+            let line = if warning.end_line != warning.line || warning.end_column != warning.column {
+                // Include end position if different from start
+                format!(
+                    "::{} file={},line={},col={},endLine={},endColumn={},title={}::{}",
+                    level,
+                    escaped_file,
+                    warning.line,
+                    warning.column,
+                    warning.end_line,
+                    warning.end_column,
+                    escaped_rule,
+                    escaped_message
+                )
+            } else {
+                // Omit end position if same as start
+                format!(
+                    "::{} file={},line={},col={},title={}::{}",
+                    level, escaped_file, warning.line, warning.column, escaped_rule, escaped_message
+                )
+            };
 
             output.push_str(&line);
             output.push('\n');
@@ -88,7 +135,7 @@ mod tests {
         let output = formatter.format_warnings(&warnings, "README.md");
         assert_eq!(
             output,
-            "::warning file=README.md,line=10,col=5,title=MD001::Heading levels should only increment by one level at a time"
+            "::warning file=README.md,line=10,col=5,endLine=10,endColumn=15,title=MD001::Heading levels should only increment by one level at a time"
         );
     }
 
@@ -119,7 +166,7 @@ mod tests {
         ];
 
         let output = formatter.format_warnings(&warnings, "test.md");
-        let expected = "::warning file=test.md,line=5,col=1,title=MD001::First warning\n::warning file=test.md,line=10,col=3,title=MD013::Second warning";
+        let expected = "::warning file=test.md,line=5,col=1,endLine=5,endColumn=10,title=MD001::First warning\n::error file=test.md,line=10,col=3,endLine=10,endColumn=20,title=MD013::Second warning";
         assert_eq!(output, expected);
     }
 
@@ -141,10 +188,10 @@ mod tests {
         }];
 
         let output = formatter.format_warnings(&warnings, "doc.md");
-        // GitHub format doesn't show fix indicator
+        // GitHub format doesn't show fix indicator but includes end position
         assert_eq!(
             output,
-            "::warning file=doc.md,line=15,col=1,title=MD022::Headings should be surrounded by blank lines"
+            "::warning file=doc.md,line=15,col=1,endLine=15,endColumn=10,title=MD022::Headings should be surrounded by blank lines"
         );
     }
 
@@ -165,7 +212,7 @@ mod tests {
         let output = formatter.format_warnings(&warnings, "file.md");
         assert_eq!(
             output,
-            "::warning file=file.md,line=1,col=1,title=unknown::Unknown rule warning"
+            "::warning file=file.md,line=1,col=1,endLine=1,endColumn=5,title=unknown::Unknown rule warning"
         );
     }
 
@@ -188,7 +235,7 @@ mod tests {
         let output = formatter.format_warnings(&warnings, "large.md");
         assert_eq!(
             output,
-            "::warning file=large.md,line=99999,col=12345,title=MD999::Edge case warning"
+            "::error file=large.md,line=99999,col=12345,endLine=100000,endColumn=12350,title=MD999::Edge case warning"
         );
     }
 
@@ -207,10 +254,32 @@ mod tests {
         }];
 
         let output = formatter.format_warnings(&warnings, "test.md");
-        // Note: GitHub Actions should handle special characters in messages
+        // Newline should be escaped as %0A
+        assert_eq!(
+            output,
+            "::warning file=test.md,line=1,col=1,endLine=1,endColumn=5,title=MD001::Warning with \"quotes\" and 'apostrophes' and %0A newline"
+        );
+    }
+
+    #[test]
+    fn test_percent_encoding() {
+        let formatter = GitHubFormatter::new();
+        let warnings = vec![LintWarning {
+            line: 1,
+            column: 1,
+            end_line: 1,
+            end_column: 1,
+            rule_name: Some("MD001"),
+            message: "100% complete\r\nNew line".to_string(),
+            severity: Severity::Warning,
+            fix: None,
+        }];
+
+        let output = formatter.format_warnings(&warnings, "test%.md");
+        // %, \r, and \n should be encoded
         assert_eq!(
             output,
-            "::warning file=test.md,line=1,col=1,title=MD001::Warning with \"quotes\" and 'apostrophes' and \n newline"
+            "::warning file=test%25.md,line=1,col=1,title=MD001::100%25 complete%0D%0ANew line"
         );
     }
 
@@ -231,7 +300,7 @@ mod tests {
         let output = formatter.format_warnings(&warnings, "path/with spaces/and-dashes.md");
         assert_eq!(
             output,
-            "::warning file=path/with spaces/and-dashes.md,line=1,col=1,title=MD001::Test"
+            "::warning file=path/with spaces/and-dashes.md,line=1,col=1,endLine=1,endColumn=5,title=MD001::Test"
         );
     }
 
@@ -256,15 +325,17 @@ mod tests {
         assert!(output.contains("file=test.md"));
         assert!(output.contains("line=42"));
         assert!(output.contains("col=7"));
+        assert!(output.contains("endLine=42"));
+        assert!(output.contains("endColumn=10"));
         assert!(output.contains("title=MD010"));
         assert!(output.ends_with("::Hard tabs"));
     }
 
     #[test]
-    fn test_severity_always_warning() {
+    fn test_severity_mapping() {
         let formatter = GitHubFormatter::new();
 
-        // Test that all severities are output as "warning" in GitHub format
+        // Test that severities are properly mapped
         let warnings = vec![
             LintWarning {
                 line: 1,
@@ -291,16 +362,16 @@ mod tests {
         let output = formatter.format_warnings(&warnings, "test.md");
         let lines: Vec<&str> = output.lines().collect();
 
-        // Both should use ::warning regardless of severity
+        // Warning should use ::warning, Error should use ::error
         assert!(lines[0].starts_with("::warning "));
-        assert!(lines[1].starts_with("::warning "));
+        assert!(lines[1].starts_with("::error "));
     }
 
     #[test]
     fn test_commas_in_parameters() {
         let formatter = GitHubFormatter::new();
 
-        // Test that commas in the title don't break the format
+        // Test that commas in the title and file path are properly escaped
         let warnings = vec![LintWarning {
             line: 1,
             column: 1,
@@ -313,10 +384,80 @@ mod tests {
         }];
 
         let output = formatter.format_warnings(&warnings, "file,with,commas.md");
-        // The format should still be parseable by GitHub Actions
+        // Commas in properties should be escaped as %2C
+        assert_eq!(
+            output,
+            "::warning file=file%2Cwith%2Ccommas.md,line=1,col=1,endLine=1,endColumn=5,title=MD%2C001::Test message, with comma"
+        );
+    }
+
+    #[test]
+    fn test_colons_in_parameters() {
+        let formatter = GitHubFormatter::new();
+
+        // Test that colons in file path and rule name are properly escaped
+        let warnings = vec![LintWarning {
+            line: 1,
+            column: 1,
+            end_line: 1,
+            end_column: 5,
+            rule_name: Some("MD:001"), // Unlikely but test edge case
+            message: "Test message: with colon".to_string(),
+            severity: Severity::Warning,
+            fix: None,
+        }];
+
+        let output = formatter.format_warnings(&warnings, "file:with:colons.md");
+        // Colons in properties should be escaped as %3A, but not in message
+        assert_eq!(
+            output,
+            "::warning file=file%3Awith%3Acolons.md,line=1,col=1,endLine=1,endColumn=5,title=MD%3A001::Test message: with colon"
+        );
+    }
+
+    #[test]
+    fn test_same_start_end_position() {
+        let formatter = GitHubFormatter::new();
+
+        // When start and end are the same, end position should be omitted
+        let warnings = vec![LintWarning {
+            line: 5,
+            column: 10,
+            end_line: 5,
+            end_column: 10,
+            rule_name: Some("MD001"),
+            message: "Single position warning".to_string(),
+            severity: Severity::Warning,
+            fix: None,
+        }];
+
+        let output = formatter.format_warnings(&warnings, "test.md");
+        // Should not include endLine and endColumn when they're the same as line and column
+        assert_eq!(
+            output,
+            "::warning file=test.md,line=5,col=10,title=MD001::Single position warning"
+        );
+    }
+
+    #[test]
+    fn test_error_severity() {
+        let formatter = GitHubFormatter::new();
+
+        let warnings = vec![LintWarning {
+            line: 1,
+            column: 1,
+            end_line: 1,
+            end_column: 5,
+            rule_name: Some("MD001"),
+            message: "Error level issue".to_string(),
+            severity: Severity::Error,
+            fix: None,
+        }];
+
+        let output = formatter.format_warnings(&warnings, "test.md");
         assert_eq!(
             output,
-            "::warning file=file,with,commas.md,line=1,col=1,title=MD,001::Test message, with comma"
+            "::error file=test.md,line=1,col=1,endLine=1,endColumn=5,title=MD001::Error level issue"
         );
     }
 }

{rumdl-0.0.121 → rumdl-0.0.123}/src/rule.rs

@@ -81,6 +81,17 @@ pub enum RuleCategory {
     Other,
 }
 
+/// Capability of a rule to fix issues
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+pub enum FixCapability {
+    /// Rule can automatically fix all violations it detects
+    FullyFixable,
+    /// Rule can fix some violations based on context
+    ConditionallyFixable,
+    /// Rule cannot fix violations (by design)
+    Unfixable,
+}
+
 /// Remove marker /// TRAIT_MARKER_V1
 pub trait Rule: DynClone + Send + Sync {
     fn name(&self) -> &'static str;
@@ -148,6 +159,11 @@ pub trait Rule: DynClone + Send + Sync {
         None
     }
 
+    /// Declares the fix capability of this rule
+    fn fix_capability(&self) -> FixCapability {
+        FixCapability::FullyFixable // Safe default for backward compatibility
+    }
+
     /// Factory: create a rule from config (if present), or use defaults.
     fn from_config(_config: &crate::config::Config) -> Box<dyn Rule>
     where