From 7ec0b09cf6fa7b27319477e7990ea6391e49f230 Mon Sep 17 00:00:00 2001 From: Paul Keen <125715+pftg@users.noreply.github.com> Date: Sun, 12 Apr 2026 22:51:32 +0200 Subject: [PATCH] fix: deduplicate glob results in snap_diff:clean rake task Overlapping globs (*.diff.png matches *.base.diff.png and *.heatmap.diff.png) produced duplicate paths, causing the "Removed N" count to over-report. Co-Authored-By: Claude Opus 4.6 (1M context) --- Rakefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Rakefile b/Rakefile index a6798fa0..5b2a66f5 100644 --- a/Rakefile +++ b/Rakefile @@ -40,7 +40,7 @@ task "snap_diff:clean" do patterns = ["**/*.diff.png", "**/*.base.diff.png", "**/*.heatmap.diff.png", "**/*.diff.webp", "**/*.base.diff.webp", "**/*.heatmap.diff.webp", "**/snap_diff_report.html"] - removed = patterns.flat_map { |p| Dir.glob("tmp/#{p}") + Dir.glob("doc/screenshots/#{p}") } + removed = patterns.flat_map { |p| Dir.glob("tmp/#{p}") + Dir.glob("doc/screenshots/#{p}") }.uniq removed.each { |f| FileUtils.rm_f(f) } puts "Removed #{removed.size} diff artifacts" end