rumdl 0.0.106__tar.gz → 0.0.108__tar.gz

{rumdl-0.0.106 → rumdl-0.0.108}/.config/nextest.toml

@@ -43,4 +43,24 @@ default-filter = 'not test(/memory/) and not test(/stress/) and not test(/large/
 test-threads = "num-cpus"
 
 # Less verbose output
-status-level = "fail"
+status-level = "fail"
+
+# Ultra-fast profile for pre-commit hooks
+[profile.pre-commit]
+
+# Only run unit tests in lib, skip all integration tests and binaries
+default-filter = 'package(rumdl) and kind(lib) and not test(/slow|stress|large|integration|comprehensive|cli|config|advanced|commonmark|consistency|performance|benchmark|parity/)'
+
+# Maximum parallelism for speed
+test-threads = "num-cpus"
+
+# Minimal output for speed
+status-level = "fail"
+failure-output = "immediate"
+success-output = "never"
+
+# No retries to save time
+retries = 0
+
+# Fail fast on first failure
+fail-fast = true

{rumdl-0.0.106 → rumdl-0.0.108}/.pre-commit-config.yaml

@@ -1,6 +1,10 @@
 # Pre-commit hooks for rumdl
-# Install with: pre-commit install
-# Run manually: pre-commit run --all-files
+# Install with: 
+#   pre-commit install                    # Install commit hooks
+#   pre-commit install --hook-type pre-push  # Install push hooks
+# Run manually: 
+#   pre-commit run --all-files           # Run commit hooks
+#   pre-commit run --hook-stage push --all-files  # Run push hooks
 
 repos:
   # Rust formatting and linting
@@ -15,26 +19,19 @@ repos:
 
       - id: cargo-clippy
         name: cargo clippy
-        entry: make lint
+        entry: make lint-fast
         language: system
         types: [rust]
         pass_filenames: false
 
       - id: cargo-test-quick
         name: cargo test (quick)
-        entry: make test-quick
+        entry: make test-pre-commit
         language: system
         types: [rust]
         pass_filenames: false
         stages: [pre-commit]
 
-      - id: cargo-check
-        name: cargo check
-        entry: cargo check --all-targets --all-features
-        language: system
-        types: [rust]
-        pass_filenames: false
-
   # General file quality checks
   - repo: https://github.com/pre-commit/pre-commit-hooks
     rev: v4.6.0
@@ -57,3 +54,33 @@ repos:
     hooks:
       - id: rumdl
         exclude: ^(test_tmp|benchmark/test-data|docs/temp|CHANGELOG\.md)
+
+  # Pre-push hooks for comprehensive validation
+  - repo: local
+    hooks:
+      - id: cargo-test-full
+        name: cargo test (full suite)
+        entry: make test-push
+        language: system
+        types: [rust]
+        pass_filenames: false
+        stages: [push]
+        verbose: true
+
+      - id: cargo-lint-full
+        name: cargo clippy (all targets)
+        entry: make lint
+        language: system
+        types: [rust]
+        pass_filenames: false
+        stages: [push]
+        verbose: true
+
+      - id: cargo-doc-check
+        name: cargo doc (check)
+        entry: cargo doc --no-deps --all-features
+        language: system
+        types: [rust]
+        pass_filenames: false
+        stages: [push]
+        verbose: true

{rumdl-0.0.106 → rumdl-0.0.108}/CHANGELOG.md

@@ -7,6 +7,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.0.107] - 2025-08-06
+
+## [0.0.107] - 2025-08-06
+
+### Fixed
+- MD036: Remove automatic fix to prevent document corruption when bold/italic text is used as image captions, labels, or warnings (#23)
+- MD011: No longer flags patterns like `()[1]` inside inline code as reversed links (#19)
+- MD052: No longer flags reference patterns inside HTML comments as undefined references (#20)
+
 ## [0.0.106] - 2025-08-05
 
 ### Changed
@@ -289,7 +298,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.105...HEAD
+[Unreleased]: https://github.com/rvben/rumdl/compare/v0.0.107...HEAD
+[0.0.107]: https://github.com/rvben/rumdl/compare/v0.0.106...v0.0.107
+[0.0.107]: https://github.com/rvben/rumdl/compare/v0.0.106...v0.0.107
 [0.0.105]: https://github.com/rvben/rumdl/compare/v0.0.104...v0.0.105
 [0.0.105]: https://github.com/rvben/rumdl/compare/v0.0.104...v0.0.105
 [0.0.104]: https://github.com/rvben/rumdl/compare/v0.0.103...v0.0.104

{rumdl-0.0.106 → rumdl-0.0.108}/Cargo.lock

@@ -49,9 +49,9 @@ checksum = "4b46cbb362ab8752921c97e041f5e366ee6297bd428a31275b9fcf1e380f7299"
 
 [[package]]
 name = "anstream"
-version = "0.6.19"
+version = "0.6.20"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "301af1932e46185686725e0fad2f8f2aa7da69dd70bf6ecc44d6b703844a3933"
+checksum = "3ae563653d1938f79b1ab1b5e668c87c76a9930414574a6583a7b7e11a8e6192"
 dependencies = [
  "anstyle",
  "anstyle-parse",
@@ -79,22 +79,22 @@ dependencies = [
 
 [[package]]
 name = "anstyle-query"
-version = "1.1.3"
+version = "1.1.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6c8bdeb6047d8983be085bab0ba1472e6dc604e7041dbf6fcd5e71523014fae9"
+checksum = "9e231f6134f61b71076a3eab506c379d4f36122f2af15a9ff04415ea4c3339e2"
 dependencies = [
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
 name = "anstyle-wincon"
-version = "3.0.9"
+version = "3.0.10"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "403f75924867bb1033c59fbf0797484329750cfbe3c4325cd33127941fabc882"
+checksum = "3e0633414522a32ffaac8ac6cc8f748e090c5717661fddeea04219e2344f5f2a"
 dependencies = [
  "anstyle",
  "once_cell_polyfill",
- "windows-sys 0.59.0",
+ "windows-sys 0.60.2",
 ]
 
 [[package]]
@@ -277,9 +277,9 @@ dependencies = [
 
 [[package]]
 name = "clap"
-version = "4.5.42"
+version = "4.5.43"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ed87a9d530bb41a67537289bafcac159cb3ee28460e0a4571123d2a778a6a882"
+checksum = "50fd97c9dc2399518aa331917ac6f274280ec5eb34e555dd291899745c48ec6f"
 dependencies = [
  "clap_builder",
  "clap_derive",
@@ -287,9 +287,9 @@ dependencies = [
 
 [[package]]
 name = "clap_builder"
-version = "4.5.42"
+version = "4.5.43"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "64f4f3f3c77c94aff3c7e9aac9a2ca1974a5adf392a8bb751e827d6d127ab966"
+checksum = "c35b5830294e1fa0462034af85cc95225a4cb07092c088c55bda3147cfcd8f65"
 dependencies = [
  "anstream",
  "anstyle",
@@ -1493,7 +1493,7 @@ checksum = "2b15c43186be67a4fd63bee50d0303afffcef381492ebe2c5d87f324e1b8815c"
 
 [[package]]
 name = "rumdl"
-version = "0.0.106"
+version = "0.0.108"
 dependencies = [
  "anyhow",
  "assert_cmd",
@@ -2527,9 +2527,9 @@ dependencies = [
 
 [[package]]
 name = "zerovec"
-version = "0.11.3"
+version = "0.11.4"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bdbb9122ea75b11bf96e7492afb723e8a7fbe12c67417aa95e7e3d18144d37cd"
+checksum = "e7aa2bd55086f1ab526693ecbe444205da57e25f4489879da80635a46d90e73b"
 dependencies = [
  "yoke",
  "zerofrom",

{rumdl-0.0.106 → rumdl-0.0.108}/Cargo.toml

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

{rumdl-0.0.106 → rumdl-0.0.108}/Makefile

@@ -69,6 +69,13 @@ test-nextest:
 test-quick:
 	cargo nextest run --profile quick
 
+test-pre-commit:
+	cargo nextest run --profile pre-commit
+
+test-push:
+	@echo "Running full test suite (this may take a few minutes)..."
+	cargo nextest run
+
 test-ci:
 	cargo nextest run --profile ci
 
@@ -81,7 +88,10 @@ fmt:
 	cargo fix --allow-dirty --allow-staged
 
 lint:
-	cargo clippy --all-targets --all-features -- -D warnings
+	CARGO_INCREMENTAL=1 cargo clippy --all-targets --all-features -- -D warnings
+
+lint-fast:
+	CARGO_INCREMENTAL=1 cargo clippy --workspace --lib --bins -- -D warnings
 
 check:
 	cargo check --all-targets --all-features

{rumdl-0.0.106 → rumdl-0.0.108}/PKG-INFO

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

{rumdl-0.0.106 → rumdl-0.0.108}/docs/md036.md

@@ -1,8 +1,10 @@
 # MD036 - Use real headings, not just bold text
 
+> **Note:** This rule does not provide automatic fixes because it cannot reliably determine user intent. Bold text on its own line has many legitimate uses (image captions, labels, warnings, etc.).
+
 ## What this rule does
 
-Detects when bold or italic text is used as a standalone heading and converts it to proper heading syntax.
+Detects when bold or italic text is used as a standalone heading.
 
 ## Why this matters
 
@@ -43,9 +45,9 @@ Using emphasis as headings:
 **Configuration:**
 ```
 
-### 🔧 Fixed
+### 🔧 Manual Fix Required
 
-Converted to proper headings:
+These should be manually converted to proper headings:
 
 ```markdown
 ## This looks like a heading
@@ -57,28 +59,29 @@ Converted to proper headings:
 ## Configuration
 ```
 
+**Note:** rumdl will detect these issues but won't automatically fix them. You need to manually decide if the emphasized text should be a heading.
+
 <!-- rumdl-enable MD036 -->
 
 ## Configuration
 
-```yaml
-MD036:
-  punctuation: ".,;:!?"  # Characters to remove from heading ends
+```toml
+# .rumdl.toml
+[MD036]
+punctuation = ".,;:!?"  # Configure punctuation checking
 ```
 
 ### Options
 
-- `punctuation`: String of punctuation marks to remove when converting (default: `".,;:!?"`)
-  - Set to `""` to keep all punctuation
+- `punctuation`: String of punctuation marks that, when found at the end of emphasized text, prevent it from being flagged (default: `".,;:!?"`)
+  - Set to `""` to flag all emphasized lines regardless of punctuation
   - Customize to remove only specific marks
 
 ## Automatic fixes
 
-This rule will:
-- Convert standalone bold/italic lines to proper headings
-- Remove trailing punctuation based on your configuration
-- Choose appropriate heading level (## for bold, # for italic)
-- Preserve common labels like "Table of Contents" as-is
+This rule does **not** provide automatic fixes. Converting bold or italic text to headings is too risky as it can corrupt documents. For example, bold text is often used for image captions, labels, or other legitimate emphasis that should not be converted to headings.
+
+Users should manually review each warning and decide whether the emphasized text should be converted to a heading.
 
 ## Learn more
 

rumdl-0.0.108/src/bin/debug_frontmatter.rs

@@ -0,0 +1,42 @@
+use rumdl::rules::front_matter_utils::FrontMatterUtils;
+use std::env;
+use std::fs;
+
+fn main() {
+    let args: Vec<String> = env::args().collect();
+    let content = if args.len() > 1 {
+        fs::read_to_string(&args[1]).expect("Failed to read file")
+    } else {
+        r#"+++
+title = "My Post"
+tags = ["example", "test"]
++++
+
+# Content
+
+[missing] reference should be flagged."#
+            .to_string()
+    };
+
+    println!("Content:");
+    for (i, line) in content.lines().enumerate() {
+        println!("Line {}: {}", i + 1, line);
+    }
+    println!();
+
+    let line_count = content.lines().count();
+
+    // Test with 0-based indexing (what the function actually uses)
+    println!("Testing is_in_front_matter with 0-based indexing:");
+    for i in 0..line_count {
+        let in_frontmatter = FrontMatterUtils::is_in_front_matter(&content, i);
+        println!("Line {} (0-based={}): in_frontmatter={}", i + 1, i, in_frontmatter);
+    }
+
+    // Test also with 1-based indexing (what we pass from MD052)
+    println!("\nTesting is_in_front_matter with 1-based indexing:");
+    for i in 1..=line_count {
+        let in_frontmatter = FrontMatterUtils::is_in_front_matter(&content, i);
+        println!("Line {i}: in_frontmatter={in_frontmatter}");
+    }
+}

{rumdl-0.0.106 → rumdl-0.0.108}/src/config.rs

@@ -862,6 +862,46 @@ local_time = 07:32:00
             // Note: local_date and local_time might not be parsed by the current implementation
         }
     }
+
+    #[test]
+    fn test_default_config_passes_validation() {
+        use crate::rules;
+
+        let temp_dir = tempdir().unwrap();
+        let config_path = temp_dir.path().join(".rumdl.toml");
+        let config_path_str = config_path.to_str().unwrap();
+
+        // Create the default config using the same function that `rumdl init` uses
+        create_default_config(config_path_str).unwrap();
+
+        // Load it back as a SourcedConfig
+        let sourced =
+            SourcedConfig::load(Some(config_path_str), None).expect("Default config should load successfully");
+
+        // Create the rule registry
+        let all_rules = rules::all_rules(&Config::default());
+        let registry = RuleRegistry::from_rules(&all_rules);
+
+        // Validate the config
+        let warnings = validate_config_sourced(&sourced, &registry);
+
+        // The default config should have no warnings
+        if !warnings.is_empty() {
+            for warning in &warnings {
+                eprintln!("Config validation warning: {}", warning.message);
+                if let Some(rule) = &warning.rule {
+                    eprintln!("  Rule: {rule}");
+                }
+                if let Some(key) = &warning.key {
+                    eprintln!("  Key: {key}");
+                }
+            }
+        }
+        assert!(
+            warnings.is_empty(),
+            "Default config from rumdl init should pass validation without warnings"
+        );
+    }
 }
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]

{rumdl-0.0.106 → rumdl-0.0.108}/src/lib.rs

@@ -1,6 +1,5 @@
 pub mod config;
 pub mod exit_codes;
-pub mod init;
 pub mod inline_config;
 pub mod lint_context;
 pub mod lsp;
@@ -190,8 +189,18 @@ pub fn lint(content: &str, rules: &[Box<dyn Rule>], _verbose: bool) -> LintResul
                 let filtered_warnings: Vec<_> = rule_warnings
                     .into_iter()
                     .filter(|warning| {
+                        // Use the warning's rule_name if available, otherwise use the rule's name
+                        let rule_name_to_check = warning.rule_name.unwrap_or(rule.name());
+
+                        // Extract the base rule name for sub-rules like "MD029-style" -> "MD029"
+                        let base_rule_name = if let Some(dash_pos) = rule_name_to_check.find('-') {
+                            &rule_name_to_check[..dash_pos]
+                        } else {
+                            rule_name_to_check
+                        };
+
                         !inline_config.is_rule_disabled(
-                            rule.name(),
+                            base_rule_name,
                             warning.line, // Already 1-indexed
                         )
                     })

{rumdl-0.0.106 → rumdl-0.0.108}/src/rules/md011_no_reversed_links.rs

@@ -204,18 +204,18 @@ impl Rule for MD011NoReversedLinks {
         let mut byte_pos = 0;
 
         for (line_num, line) in content.lines().enumerate() {
-            // Skip if this line is in a code block
-            if ctx.is_in_code_block_or_span(byte_pos) {
-                byte_pos += line.len() + 1;
-                continue;
-            }
-
             // Part 1: Check for existing perfectly formed reversed links
             for cap in REVERSED_LINK_CHECK_REGEX.captures_iter(line) {
                 let match_obj = cap.get(0).unwrap();
                 let match_start = match_obj.start();
                 let match_end = match_obj.end();
 
+                // Check if this specific match is within a code block or inline code span
+                let match_byte_pos = byte_pos + match_start;
+                if ctx.is_in_code_block_or_span(match_byte_pos) {
+                    continue;
+                }
+
                 // Check if the match contains escaped brackets or parentheses
                 let match_text = match_obj.as_str();
 
@@ -277,6 +277,12 @@ impl Rule for MD011NoReversedLinks {
             // Part 2: Check for malformed link attempts where user intent is clear
             let malformed_attempts = self.detect_malformed_link_attempts(line);
             for (start, len, url, text) in malformed_attempts {
+                // Check if this specific match is within a code block or inline code span
+                let match_byte_pos = byte_pos + start;
+                if ctx.is_in_code_block_or_span(match_byte_pos) {
+                    continue;
+                }
+
                 // Calculate precise character range for the malformed syntax
                 let (start_line, start_col, end_line, end_col) = calculate_match_range(line_num + 1, line, start, len);
 
@@ -593,4 +599,45 @@ But this (https://example.com)[reversed link] should be flagged."#;
         let result = rule.check(&ctx).unwrap();
         assert_eq!(result.len(), 1, "Should still flag URLs with nested parentheses");
     }
+
+    #[test]
+    fn test_inline_code_patterns() {
+        // Test for issue #19 - MD011 should not flag patterns inside inline code
+        let rule = MD011NoReversedLinks;
+
+        // Test the exact case from issue #19
+        let content = "I find `inspect.stack()[1].frame` a lot easier to understand (or at least guess about) at a glance than `inspect.stack()[1][0]`.";
+        let ctx = LintContext::new(content);
+        let result = rule.check(&ctx).unwrap();
+        assert_eq!(result.len(), 0, "Should not flag ()[1] patterns inside inline code");
+
+        // Test other patterns that might look like reversed links in code
+        let content = "Use `array()[0]` or `func()[1]` to access elements.";
+        let ctx = LintContext::new(content);
+        let result = rule.check(&ctx).unwrap();
+        assert_eq!(result.len(), 0, "Should not flag array access patterns in inline code");
+
+        // Test that actual reversed links outside code are still caught
+        let content = "Check out (https://example.com)[this link] and use `array()[1]`.";
+        let ctx = LintContext::new(content);
+        let result = rule.check(&ctx).unwrap();
+        assert_eq!(result.len(), 1, "Should flag actual reversed link but not code pattern");
+        assert!(result[0].message.contains("Reversed link syntax"));
+
+        // Test mixed scenario with code blocks
+        let content = r#"
+Here's some code: `func()[1]` and `other()[2]`.
+
+But this is wrong: (https://example.com)[Click here]
+
+```python
+# This should not be flagged
+result = inspect.stack()[1]
+```
+"#;
+        let ctx = LintContext::new(content);
+        let result = rule.check(&ctx).unwrap();
+        assert_eq!(result.len(), 1, "Should only flag the actual reversed link");
+        assert_eq!(result[0].line, 4, "Should flag the reversed link on line 4");
+    }
 }