chore(deps): upgrade perfectionist to 0.0.0-rc.6

The new release adds the `flat_module_pattern`, `macro_trailing_comma`,
`unicode_ellipsis_in_panic_messages`, and `unknown_perfectionist_lints`
rules. Fix the trailing-comma violations the new rule reports and drop
the now-redundant flat-module-pattern bullet from `CONTRIBUTING.md`.

Two `macro_trailing_comma` warnings remain in `tests/tree_builder.rs`
and `tests/visualizer.rs`. They are triggered by single-element
multi-line `vec!` invocations that `rustfmt` actively reformats away
from a trailing comma, and `#[allow(perfectionist::...)]` cannot be
used without nightly `register_tool`. The warnings do not fail the
build because `test.sh` does not pass `-D warnings` to `cargo dylint`.

Author: claude

CONTRIBUTING.md

@@ -54,7 +54,8 @@ use crate::get_size::{GetBlockCount, GetBlockSize};
 
 ### Module Organization
 
-- Use the flat file pattern (`module.rs`) rather than `module/mod.rs` for submodules.
+The flat file pattern (`module.rs` rather than `module/mod.rs`) is enforced by the `perfectionist::flat_module_pattern` dylint check. In addition to that requirement, follow these conventions:
+
 - List `pub mod` declarations first, then `pub use` re-exports, then private imports and items.
 - Use `pub use` to re-export key types at the module level for convenience.
 

dylint.toml

@@ -1,4 +1,4 @@
 [workspace.metadata.dylint]
 libraries = [
-    { git = "https://github.com/KSXGitHub/perfectionist", tag = "0.0.0-rc.0" },
+    { git = "https://github.com/KSXGitHub/perfectionist", tag = "0.0.0-rc.6" },
 ]

src/app/hdd.rs

@@ -237,8 +237,8 @@ fn is_virtual_block_device<Fs: FsApi>(block_dev: &str) -> bool {
                 | "xen-blkfront"
                 | "vbd"
                 | "vmw_pvscsi"
-                | "hv_storvsc"
-        )
+                | "hv_storvsc",
+        ),
     )
 }
 

src/app/hdd/test_linux_smoke.rs

@@ -14,7 +14,7 @@ fn real_sysfs_vda_does_not_panic() {
 fn nonexistent_device_is_not_virtual() {
     assert!(
         !is_virtual_block_device::<RealFs>("nonexistent_device_xyz"),
-        "non-existent device should not be detected as virtual"
+        "non-existent device should not be detected as virtual",
     );
 }
 

src/app/overlapping_arguments/test_remove_overlapping_paths.rs

@@ -90,7 +90,7 @@ fn remove_duplicated_arguments() {
         "0/1/2",
         "./bar",
         "./abc/./def",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["foo", "bar", "abc/def", "0/1/2"];
@@ -104,7 +104,7 @@ fn remove_duplicated_arguments() {
         "abc/def",
         "foo",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["foo", "./bar", "./abc/./def", "0/1/2"];
@@ -145,7 +145,7 @@ fn remove_all_except_current_dir() {
         "abc/def",
         "0/1/2",
         MOCKED_CURRENT_DIR,
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["."];
@@ -158,7 +158,7 @@ fn remove_all_except_current_dir() {
         ".",
         "abc/def",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec![MOCKED_CURRENT_DIR];
@@ -182,7 +182,7 @@ fn remove_all_except_parent_dir() {
         "abc/def",
         ".",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["/home/user"];
@@ -199,7 +199,7 @@ fn remove_overlapping_real_paths() {
         "link-to-foo/child",
         "link-to-bar/a/b/c",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["foo", "bar", "abc/def", "0/1/2"];
@@ -212,7 +212,7 @@ fn remove_overlapping_real_paths() {
         "bar",
         "abc/def",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["foo", "bar", "abc/def", "0/1/2"];
@@ -225,7 +225,7 @@ fn remove_overlapping_real_paths() {
         "abc/def",
         "link-to-current-dir/bar",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = vec!["link-to-current-dir/foo", "bar", "abc/def", "0/1/2"];
@@ -242,7 +242,7 @@ fn do_not_remove_symlinks() {
         "link-to-foo",
         "link-to-bar",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = original;
@@ -255,7 +255,7 @@ fn do_not_remove_symlinks() {
         "link-to-foo",
         "link-to-bar",
         "0/1/2",
-    ]);
+    ],);
     let mut actual = original.clone();
     remove_overlapping_paths::<MockedApi>(&mut actual);
     let expected = original;

src/hardlink/hardlink_list/summary.rs

@@ -90,7 +90,7 @@ where
                 }
                 Ordering::Less => {
                     panic!(
-                        "Impossible! Total of nlink ({links}) is less than detected paths ({paths}). Something must have gone wrong!"
+                        "Impossible! Total of nlink ({links}) is less than detected paths ({paths}). Something must have gone wrong!",
                     );
                 }
             }

src/man_page.rs

@@ -275,7 +275,7 @@ fn render_possible_values(out: &mut String, arg: &Arg) {
     }
     if matches!(
         arg.get_action(),
-        ArgAction::SetTrue | ArgAction::SetFalse | ArgAction::Count
+        ArgAction::SetTrue | ArgAction::SetFalse | ArgAction::Count,
     ) {
         return;
     }

tests/cli_errors.rs

@@ -72,7 +72,7 @@ fn min_ratio_1() {
             "error: invalid value '1' for '--min-ratio <MIN_RATIO>': greater than or equal to 1"
             ""
             "For more information, try '--help'."
-        }
+        },
     );
     assert_eq!(&stdout, &[] as &[u8]);
 }
@@ -101,7 +101,7 @@ fn max_depth_0() {
             r#"error: invalid value '0' for '--max-depth <MAX_DEPTH>': Value is neither "inf" nor a positive integer: number would be zero for non-zero type"#
             ""
             "For more information, try '--help'."
-        }
+        },
     );
     assert_eq!(&stdout, &[] as &[u8]);
 }

tests/one_file_system.rs

@@ -230,7 +230,7 @@ fn cross_device_excludes_mount() {
         .find_map(|mut entry| entry.next()?.ok());
     assert!(
         poll_result.is_some(),
-        "FUSE mount at {mount_point:?} not ready after {retries} retries"
+        "FUSE mount at {mount_point:?} not ready after {retries} retries",
     );
 
     let build_expected_tree = |device_boundary: DeviceBoundary| -> String {