rumdl 0.0.96__tar.gz → 0.0.97__tar.gz

{rumdl-0.0.96 → rumdl-0.0.97}/CHANGELOG.md

@@ -7,7 +7,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [0.0.96] - 2025-07-16
+## [0.0.97] - 2025-07-17
+
+### Changed
+- Updated exit code handling for consistency:
+  - Configuration errors now return exit code 2 (was 1)
+  - File not found errors now return exit code 2 (was 1)
+  - Invalid command arguments now return exit code 2 (was 1)
+- Standardized exit codes across all error conditions:
+  - Exit code 1: Reserved for linting issues found
+  - Exit code 2: Tool errors (config parse errors, file not found, invalid arguments)
+
+### Fixed
+- Improved consistency in exit code handling across the entire codebase
+- Updated all tests to expect correct exit codes for different error scenarios
 
 ## [0.0.96] - 2025-07-16
 
@@ -185,7 +198,8 @@ 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.96...HEAD
+[Unreleased]: https://github.com/rvben/rumdl/compare/v0.0.97...HEAD
+[0.0.97]: https://github.com/rvben/rumdl/compare/v0.0.96...v0.0.97
 [0.0.96]: https://github.com/rvben/rumdl/compare/v0.0.95...v0.0.96
 [0.0.95]: https://github.com/rvben/rumdl/compare/v0.0.94...v0.0.95
 [0.0.94]: https://github.com/rvben/rumdl/compare/v0.0.93...v0.0.94

{rumdl-0.0.96 → rumdl-0.0.97}/Cargo.lock

@@ -1493,7 +1493,7 @@ checksum = "2b15c43186be67a4fd63bee50d0303afffcef381492ebe2c5d87f324e1b8815c"
 
 [[package]]
 name = "rumdl"
-version = "0.0.96"
+version = "0.0.97"
 dependencies = [
  "anyhow",
  "assert_cmd",

{rumdl-0.0.96 → rumdl-0.0.97}/Cargo.toml

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

{rumdl-0.0.96 → rumdl-0.0.97}/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2025 Ruben J. Jongejan
+Copyright (c) 2024-2025 Ruben J. Jongejan
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

{rumdl-0.0.96 → rumdl-0.0.97}/PKG-INFO

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

{rumdl-0.0.96 → rumdl-0.0.97}/src/rules/md013_line_length.rs

@@ -1023,14 +1023,22 @@ And a bullet list:
         let lines: Vec<&str> = fixed.lines().collect();
         for (i, line) in lines.iter().enumerate() {
             if line.trim().starts_with("1.") || line.trim().starts_with("2.") || 
-               line.trim().starts_with("3.") || line.trim().starts_with("-") {
-                // Check if next line is a continuation (should be indented)
+               line.trim().starts_with("3.") {
+                // Check if next line is a continuation (should be indented with 3 spaces for numbered lists)
                 if i + 1 < lines.len() && !lines[i + 1].trim().is_empty() &&
                    !lines[i + 1].trim().starts_with(char::is_numeric) &&
                    !lines[i + 1].trim().starts_with("-") {
-                    // Continuation lines should have proper indentation
+                    // Numbered list continuation lines should have 3 spaces
                     assert!(lines[i + 1].starts_with("   ") || lines[i + 1].trim().is_empty());
                 }
+            } else if line.trim().starts_with("-") {
+                // Check if next line is a continuation (should be indented with 2 spaces for dash lists)
+                if i + 1 < lines.len() && !lines[i + 1].trim().is_empty() &&
+                   !lines[i + 1].trim().starts_with(char::is_numeric) &&
+                   !lines[i + 1].trim().starts_with("-") {
+                    // Dash list continuation lines should have 2 spaces
+                    assert!(lines[i + 1].starts_with("  ") || lines[i + 1].trim().is_empty());
+                }
             }
         }
     }

{rumdl-0.0.96 → rumdl-0.0.97}/src/utils/text_reflow.rs

@@ -324,13 +324,18 @@ pub fn reflow_markdown(content: &str, options: &ReflowOptions) -> String {
             let marker = &line[indent..content_start];
             let content = &line[content_start..].trim_start();
             
+            // Calculate the proper indentation for continuation lines
+            // We need to align with the text after the marker
+            let trimmed_marker = marker.trim_end();
+            let continuation_spaces = indent + trimmed_marker.len() + 1; // +1 for the space after marker
+            
             let reflowed = reflow_line(content, options);
             for (j, reflowed_line) in reflowed.iter().enumerate() {
                 if j == 0 {
-                    result.push(format!("{}{} {}", indent_str, marker.trim_end(), reflowed_line));
+                    result.push(format!("{}{} {}", indent_str, trimmed_marker, reflowed_line));
                 } else {
-                    // Continuation lines get extra indentation
-                    let continuation_indent = " ".repeat(indent + marker.len() + 1);
+                    // Continuation lines aligned with text after marker
+                    let continuation_indent = " ".repeat(continuation_spaces);
                     result.push(format!("{}{}", continuation_indent, reflowed_line));
                 }
             }

{rumdl-0.0.96 → rumdl-0.0.97}/tests/comprehensive_output_format_tests.rs

@@ -215,7 +215,7 @@ fn test_invalid_output_format() {
         .arg(&test_file);
 
     cmd.assert()
-        .failure()
+        .code(2)  // Should exit with code 2 for invalid command argument
         .stderr(predicate::str::contains("invalid value 'invalid_format'"));
 }
 

{rumdl-0.0.96 → rumdl-0.0.97}/tests/config_file_command_test.rs

@@ -56,7 +56,8 @@ fn test_config_file_command_with_nonexistent_config() {
         .output()
         .expect("Failed to execute command");
 
-    assert!(!output.status.success());
+    // Should exit with code 2 for file not found (tool error)
+    assert_eq!(output.status.code(), Some(2), "Expected exit code 2 for file not found");
 
     let stderr = String::from_utf8(output.stderr).unwrap();
     assert!(stderr.contains("Config error"));

{rumdl-0.0.96 → rumdl-0.0.97}/tests/config_tests.rs

@@ -721,7 +721,8 @@ line-length:
             .output()
             .expect("Failed to execute command");
 
-        assert!(!output.status.success(), "Command should fail with invalid config");
+        // Should exit with code 2 for configuration error
+        assert_eq!(output.status.code(), Some(2), "Expected exit code 2 for invalid JSON config");
         let stderr = String::from_utf8_lossy(&output.stderr);
         let stdout = String::from_utf8_lossy(&output.stdout);
         let combined_output = format!("{stderr}{stdout}");
@@ -739,7 +740,8 @@ line-length:
             .output()
             .expect("Failed to execute command");
 
-        assert!(!output.status.success(), "Command should fail with invalid YAML config");
+        // Should exit with code 2 for configuration error
+        assert_eq!(output.status.code(), Some(2), "Expected exit code 2 for invalid YAML config");
         let stderr = String::from_utf8_lossy(&output.stderr);
         let stdout = String::from_utf8_lossy(&output.stdout);
         let combined_output = format!("{stderr}{stdout}");
@@ -754,7 +756,8 @@ line-length:
             .output()
             .expect("Failed to execute command");
 
-        assert!(!output.status.success(), "Command should fail with nonexistent config");
+        // Should exit with code 2 for file not found
+        assert_eq!(output.status.code(), Some(2), "Expected exit code 2 for nonexistent config file");
         let stderr = String::from_utf8_lossy(&output.stderr);
         let stdout = String::from_utf8_lossy(&output.stdout);
         let combined_output = format!("{stderr}{stdout}");

{rumdl-0.0.96 → rumdl-0.0.97}/tests/markdownlint_cli_integration.rs

@@ -164,8 +164,8 @@ fn test_invalid_markdownlint_json_prints_helpful_error() {
         .output()
         .expect("Failed to run rumdl CLI");
 
-    // Should exit with code 1 (config load/parse error)
-    assert_eq!(output.status.code(), Some(1), "Expected exit code 1 for parse error");
+    // Should exit with code 2 (tool error - config load/parse error)
+    assert_eq!(output.status.code(), Some(2), "Expected exit code 2 for parse error");
     let stderr = String::from_utf8_lossy(&output.stderr);
     // Accept any error message that contains 'Failed to parse JSON' and the filename
     assert!(