chore(deps): point perfectionist at KSXGitHub/perfectionist#25

PR #25 (commit d8c545f) fixes both issues filed in #408 and #409:
macro_trailing_comma now skips rustfmt-compatible compact layouts
(single-element multi-line bracket macros) and properly fulfills
cfg_attr-gated expects.

* Drop the crate-level suppression scaffolding in
  tests/tree_builder.rs and tests/visualizer.rs: the rule no longer
  fires on vec![Single { ... }] / vec![single(...)] shapes that
  rustfmt actively collapses.
* Drop the trailing commas added to dbg!(vec![...]) call sites in
  test_remove_overlapping_paths.rs, which fell into the same
  compact-layout category and are no longer flagged.

The single-argument multi-line panic! in
src/hardlink/hardlink_list/summary.rs still requires the trailing
comma because PR #25 only relaxes the rule for bracket-delimited
compact layouts, not parenthesised single-argument macros.

Author: claude

dylint.toml

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

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;

tests/tree_builder.rs

@@ -1,24 +1,3 @@
-#![cfg_attr(dylint_lib = "perfectionist", feature(register_tool))]
-#![cfg_attr(dylint_lib = "perfectionist", register_tool(perfectionist))]
-#![cfg_attr(
-    dylint_lib = "perfectionist",
-    expect(
-        perfectionist::macro_trailing_comma,
-        reason = "rustfmt collapses single-element multi-line vec! and removes the trailing comma; see https://github.com/KSXGitHub/perfectionist/issues"
-    )
-)]
-#![cfg_attr(
-    dylint_lib = "perfectionist",
-    allow(
-        unfulfilled_lint_expectations,
-        reason = "perfectionist's macro_trailing_comma short-circuits when its lint is allowed/expected at crate scope, so #[expect] cannot be fulfilled even though violations exist"
-    )
-)]
-#![allow(
-    unexpected_cfgs,
-    reason = "`dylint_lib` is set by dylint's compiler wrapper and is not in --check-cfg"
-)]
-
 use build_fs_tree::{FileSystemTree, dir, file};
 use derive_more::From;
 use parallel_disk_usage::{

tests/visualizer.rs

@@ -1,24 +1,3 @@
-#![cfg_attr(dylint_lib = "perfectionist", feature(register_tool))]
-#![cfg_attr(dylint_lib = "perfectionist", register_tool(perfectionist))]
-#![cfg_attr(
-    dylint_lib = "perfectionist",
-    expect(
-        perfectionist::macro_trailing_comma,
-        reason = "rustfmt collapses single-element multi-line vec! and removes the trailing comma; see https://github.com/KSXGitHub/perfectionist/issues"
-    )
-)]
-#![cfg_attr(
-    dylint_lib = "perfectionist",
-    allow(
-        unfulfilled_lint_expectations,
-        reason = "perfectionist's macro_trailing_comma short-circuits when its lint is allowed/expected at crate scope, so #[expect] cannot be fulfilled even though violations exist"
-    )
-)]
-#![allow(
-    unexpected_cfgs,
-    reason = "`dylint_lib` is set by dylint's compiler wrapper and is not in --check-cfg"
-)]
-
 use parallel_disk_usage::{
     bytes_format::BytesFormat::*,
     data_tree::DataTree,