style: fix RuboCop offenses in CLI code

Use idiomatic Ruby patterns to satisfy linter:
- Replace each+append with map for building arrays
- Use annotated format tokens (%<name>s) over positional ones
- Use %r{} for regex containing slashes

Author: cardmagic

lib/classifier/cli.rb

@@ -397,22 +397,22 @@ def list_local_models
         return
       end
 
-      models = [] #: Array[{name: String, registry: String?, path: String}]
-
       # Find models from default registry
-      Dir.glob(File.join(models_dir, '*.json')).each do |path|
-        models << { name: File.basename(path, '.json'), registry: nil, path: path }
+      default_models = Dir.glob(File.join(models_dir, '*.json')).map do |path|
+        { name: File.basename(path, '.json'), registry: nil, path: path }
       end
 
       # Find models from custom registries (@user/repo structure)
-      Dir.glob(File.join(models_dir, '@*', '*', '*.json')).each do |path|
+      custom_models = Dir.glob(File.join(models_dir, '@*', '*', '*.json')).map do |path|
         # Extract registry from path: .../models/@user/repo/model.json
         repo_dir = File.dirname(path)
         user_dir = File.dirname(repo_dir)
         registry = "#{File.basename(user_dir).delete_prefix('@')}/#{File.basename(repo_dir)}"
-        models << { name: File.basename(path, '.json'), registry: registry, path: path }
+        { name: File.basename(path, '.json'), registry: registry, path: path }
       end
 
+      models = default_models + custom_models #: Array[{name: String, registry: String?, path: String}]
+
       if models.empty?
         @output << 'No local models found'
         return
@@ -443,7 +443,7 @@ def human_size(bytes)
         unit_index += 1
       end
 
-      format('%.1f %s', size, units[unit_index])
+      format('%<size>.1f %<unit>s', size: size, unit: units[unit_index])
     end
 
     def command_pull

test/cli/registry_commands_test.rb

@@ -131,7 +131,7 @@ def test_models_local_lists_models_from_custom_registries
     result = run_cli('models', '--local')
 
     assert_equal 0, result[:exit_code]
-    assert_match(/@someone\/models:custom-model/, result[:output])
+    assert_match(%r{@someone/models:custom-model}, result[:output])
   end
 
   def test_models_local_shows_no_models_when_cache_empty