rails-blocks-cli 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4f132c448c8e065b701c21e0e5b777d24fea8ab32c32870f51c84d4a68626cc
-  data.tar.gz: 4e703d0337c0a8504773d803afef6b83b6cf6a62ec9dca8a2d048fcd4175bec0
+  metadata.gz: 63dd443055d6116deb6a9945e9aa025d6cee699603ac48278dc56d541c567e58
+  data.tar.gz: f8f4993d4a4b63083b8644040e02bf35653cefcbb04ce9f03d319ef3820dacde
 SHA512:
-  metadata.gz: e616247429678b871ce785aa979b2b5922deaaa153b94f0fefeec5261fa8791318a8938ae67c11a602db1a48f884e794334f3df25dac0a82c2be8fa667ab671a
-  data.tar.gz: d8271aef17019e1244f64fb098bdfbd35ce975f16f5b989143a5ba2df72f4946c121a8dd4e9da48a267bc45c78cb3999e83411f385b9302ee330775b3051ef77
+  metadata.gz: ae54a971c219716f16a227a27185078bc00dd33e94a6eb5d858a4e665e653b9ea921485cb3b334e36610d324fa300d8eb691bb50eb89a53d1b0fa18327b53fd6
+  data.tar.gz: 397f72909bbd30ad2e94991e59c34a25ac4e3d3f409062a3dbac3044af7bfb7ada4f22f54c0e6bab7236b446744d09c633bda996def8cecfa07c4d39ccba17af

data/lib/rails_blocks/cli.rb

@@ -2,7 +2,6 @@ require "fileutils"
 require "json"
 require "net/http"
 require "open3"
-require "pathname"
 require "tempfile"
 require "uri"
 require "zip"
@@ -120,11 +119,11 @@ module RailsBlocks
       failures = []
 
       selected_components.each do |component|
-        say_heading("Installing #{component['slug']}#{component['pro'] ? ' (pro)' : ''} as #{implementation}")
+        say_heading("Installing #{component['slug']}#{' (pro)' if component['pro']} as #{implementation}")
         install_component(component, implementation, dry_run: dry_run)
-      rescue StandardError => error
-        failures << [component["slug"], error.message]
-        warn "Failed to install #{component['slug']}: #{error.message}"
+      rescue StandardError => e
+        failures << [component["slug"], e.message]
+        warn "Failed to install #{component['slug']}: #{e.message}"
       end
 
       puts "\nInstalled #{selected_components.size - failures.size} of #{selected_components.size} components."
@@ -172,11 +171,11 @@ module RailsBlocks
       failures = []
 
       selected_components.each do |component|
-        puts "\n#{mode_label(mode)} #{component['slug']}#{component['pro'] ? ' (pro)' : ''} as #{package_implementation}..."
+        puts "\n#{mode_label(mode)} #{component['slug']}#{' (pro)' if component['pro']} as #{package_implementation}..."
         change_component_files(component, mode: mode)
-      rescue StandardError => error
-        failures << [component["slug"], error.message]
-        warn "Failed to #{mode} #{component['slug']}: #{error.message}"
+      rescue StandardError => e
+        failures << [component["slug"], e.message]
+        warn "Failed to #{mode} #{component['slug']}: #{e.message}"
       end
 
       puts "\n#{mode_label(mode)} #{selected_components.size - failures.size} of #{selected_components.size} components."
@@ -320,7 +319,7 @@ module RailsBlocks
 
         destination = stimulus_destination(filename)
         if destination.exist? && !argv.include?("--force")
-          puts "#{muted('Skip')} #{destination} #{muted('(already exists)')}"
+          puts "#{muted('Skip')} #{display_path(destination)} #{muted('(already exists)')}"
           next
         end
 
@@ -329,7 +328,7 @@ module RailsBlocks
     end
 
     def write_file(destination, content, dry_run:)
-      puts "#{dry_run ? muted('Would write') : success('Writing')} #{destination}"
+      puts "#{dry_run ? muted('Would write') : success('Writing')} #{display_path(destination)}"
       return if dry_run
 
       raise "#{destination} already exists. Use --force to overwrite it." if destination.exist? && !argv.include?("--force")
@@ -427,24 +426,28 @@ module RailsBlocks
       old_content = destination.exist? ? File.binread(destination) : ""
       if old_content == new_content
         counts[:unchanged] += 1
-        puts "No changes #{destination}"
+        puts "No changes #{display_path(destination)}"
         return
       end
 
       counts[:changed] += 1
-      puts unified_diff(old_content, new_content, destination.exist? ? destination.to_s : "/dev/null", destination.to_s)
+      puts unified_diff(old_content, new_content, destination.exist? ? display_path(destination) : "/dev/null", display_path(destination))
     end
 
     def update_file(destination, new_content, counts)
       old_content = destination.exist? ? File.binread(destination) : nil
       if old_content == new_content
         counts[:unchanged] += 1
-        puts "Unchanged #{destination}"
+        puts "Unchanged #{display_path(destination)}"
         return
       end
 
       counts[:changed] += 1
-      puts "#{argv.include?('--dry-run') ? 'Would update' : old_content.nil? ? 'Writing' : 'Updating'} #{destination}"
+      puts "#{if argv.include?('--dry-run')
+                'Would update'
+              else
+                old_content.nil? ? 'Writing' : 'Updating'
+              end} #{display_path(destination)}"
       return if argv.include?("--dry-run")
 
       FileUtils.mkdir_p(destination.dirname)
@@ -477,6 +480,13 @@ module RailsBlocks
       mode == :diff ? "Diffing" : "Updating"
     end
 
+    def display_path(path)
+      pathname = Pathname.new(path)
+      pathname.relative_path_from(Pathname.pwd).to_s
+    rescue ArgumentError
+      pathname.to_s
+    end
+
     def components
       registry.fetch("components")
     end
@@ -507,10 +517,10 @@ module RailsBlocks
 
     def registry
       @registry ||= JSON.parse(download(registry_url))
-    rescue DownloadError => error
-      abort "Could not load the Rails Blocks registry from #{registry_url}: #{error.message}"
-    rescue JSON::ParserError => error
-      abort "Could not parse the Rails Blocks registry from #{registry_url}: #{error.message}"
+    rescue DownloadError => e
+      abort "Could not load the Rails Blocks registry from #{registry_url}: #{e.message}"
+    rescue JSON::ParserError => e
+      abort "Could not parse the Rails Blocks registry from #{registry_url}: #{e.message}"
     end
 
     def download(url, auth: false, redirects: 3)
@@ -549,7 +559,7 @@ module RailsBlocks
     end
 
     def api_url_for(path)
-      return path if path.to_s.match?(/\Ahttps?:\/\//)
+      return path if path.to_s.match?(%r{\Ahttps?://})
 
       URI.join(api_url, path).to_s
     end
@@ -578,7 +588,7 @@ module RailsBlocks
       puts bold("Choose an install format")
       puts "  1. ERB partials      #{muted('app/views/shared/<component>/')}"
       puts "  2. ViewComponent     #{muted('app/components/<component>/')}"
-      print "Select an option [1]: "
+      print "Select an option [1 by default]: "
 
       case $stdin.gets&.strip
       when "2", "view_component" then "view_component"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rails-blocks-cli
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Rails Blocks