Icarus-Mod-Tools 2.5.2 → 2.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 791e5af8d7af9b24c8fe53d04fb97e39b14e7ac0773ce682783aeffebd4b2d85
-  data.tar.gz: 7ceede257ab9a52c15e708cf1e346c48b051a0c1c66dd781281eb3d1389ee627
+  metadata.gz: f02e414cd84c6117e8ff650a1048c16eac751dae9cfe99a52a17a3e58682d1b0
+  data.tar.gz: f14a5bf74a4fcd74be9c71aba1debf378d45af6bc54290b35c2d20236e17d3af
 SHA512:
-  metadata.gz: 705e7a9fb7037c1b2f36ea52e883832c56c9720c605c2d88f25c44d8483e500ef1a962d64b0774501a90b5d00495414f24acbac27eda58d4cf2e72d4024e0e61
-  data.tar.gz: 54b2d0f837368e87fc3de253a1bdfd6bb0af60da18b145158b47950a4c1b177fd0fdfab605c96cf1309d22e1f0d7f5a74f9f371d2e53b8cbaec6932343bade5b
+  metadata.gz: 636dfd397c992d7dd15d464584734f7e599417e1145f0cf139ffcf2f8eec9e409b473c4deae9abae453a42abbfe1f04138b36440669ead6db5c4386b6e33b4ca
+  data.tar.gz: b5085429a592dfc1bd8bbf0fde991ade384afcd88aedd50c6f0969a8a786b4c3e98c740fda3053aa48c52ea734dfe74381d2d8797dde0b0e563abc467ec3e75d

data/CHANGELOG.md

@@ -4,6 +4,15 @@ All notable changes to this project will be documented in this file.
 
 ## History (reverse chronological order)
 
+### v2.6.0 - 2026-03-04
+
+- Add `imt sync cleanup` command to remove duplicate entries from mods and tools collections
+  - Groups entries by name+author and keeps the most recently updated
+  - Supports `--dry-run` to preview what would be deleted
+- Fix `find_info` to match by both name AND author, not just name
+  - Previously, mods/tools with the same name but different authors could cause incorrect deletion logic
+  - Now consistent with `info_array` deduplication and `find_by_type` lookups
+
 ### v2.5.2 - 2026-02-09
 
 - Fix duplicate entries created during sync operations

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    Icarus-Mod-Tools (2.5.2)
+    Icarus-Mod-Tools (2.5.3)
       google-cloud-firestore (~> 2.7)
       octokit (~> 6.0)
       paint (~> 2.3)

data/lib/icarus/mod/cli/sync.rb

@@ -52,11 +52,63 @@ module Icarus
           sync_list(:tools)
         end
 
+        desc "cleanup", "Remove duplicate entries from mods and tools collections"
+        def cleanup
+          cleanup_duplicates(:mods)
+          cleanup_duplicates(:tools)
+        end
+
         no_commands do
           def firestore
             $firestore ||= Firestore.new
           end
 
+          def cleanup_duplicates(type)
+            singular_type = type.to_s.chomp("s").to_sym
+            collection = firestore.send(type)
+
+            puts "Scanning #{type} for duplicates..." if verbose?
+
+            # Group by [name, author]
+            grouped = collection.group_by { |item| [item.name, item.author] }
+            duplicates = grouped.select { |_, items| items.length > 1 }
+
+            if duplicates.empty?
+              puts "No duplicate #{type} found." if verbose?
+              return
+            end
+
+            puts "Found #{duplicates.length} duplicate #{singular_type}(s) to clean up:" if verbose?
+
+            duplicates.each do |key, items|
+              name, author = key
+              # Sort by updated_at descending, keep the most recent
+              sorted = items.sort_by { |item| item.updated_at || Time.at(0) }.reverse
+              keeper = sorted.first
+              to_delete = sorted[1..]
+
+              puts "  #{author}/#{name}: #{items.length} entries" if verbose?
+              puts "    Keeping: #{keeper.id} (updated: #{keeper.updated_at})" if verbose?
+
+              to_delete.each do |item|
+                if options[:dry_run]
+                  puts Paint["    Would delete: #{item.id} (updated: #{item.updated_at})", :yellow] if verbose?
+                else
+                  puts "    Deleting: #{item.id} (updated: #{item.updated_at})" if verbose?
+                  response = firestore.delete(singular_type, item)
+                  puts "      #{success_or_failure(response)}" if verbose > 1
+                end
+              end
+            end
+
+            deleted_count = duplicates.values.sum { |items| items.length - 1 }
+            if options[:dry_run]
+              puts Paint["Dry run; no changes made. Would have deleted #{deleted_count} duplicate #{type}.", :yellow] if verbose?
+            else
+              puts "Deleted #{deleted_count} duplicate #{type}." if verbose?
+            end
+          end
+
           def success_or_failure(status)
             format("%<status>10s", status: status ? Paint["Success", :green] : Paint["Failure", :red])
           end

data/lib/icarus/mod/tools/sync/mods.rb

@@ -36,7 +36,7 @@ module Icarus
           end
 
           def find_info(modinfo)
-            @info_array.find { |mod| mod.name == modinfo.name }
+            @info_array.find { |mod| mod.name == modinfo.name && mod.author == modinfo.author }
           end
 
           def update(modinfo)

data/lib/icarus/mod/tools/sync/tools.rb

@@ -38,7 +38,7 @@ module Icarus
           end
 
           def find_info(toolinfo)
-            @info_array.find { |tool| tool.name == toolinfo.name }
+            @info_array.find { |tool| tool.name == toolinfo.name && tool.author == toolinfo.author }
           end
 
           def update(toolinfo)

data/lib/icarus/mod/version.rb

@@ -2,6 +2,6 @@
 
 module Icarus
   module Mod
-    VERSION = "2.5.2"
+    VERSION = "2.6.0"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: Icarus-Mod-Tools
 version: !ruby/object:Gem::Version
-  version: 2.5.2
+  version: 2.6.0
 platform: ruby
 authors:
 - Donovan Young