shopify-cli 1.5.0 → 1.6.0

data/lib/shopify-cli/shopifolk.rb

@@ -25,7 +25,8 @@ module ShopifyCli
       #     ShopifyCli::Shopifolk.check
       #
       def check
-        return false unless Feature.enabled?('shopifolk-beta')
+        return false unless ShopifyCli::Config.get_bool('shopifolk-beta', 'enabled')
+
         ShopifyCli::Shopifolk.new.shopifolk?
       end
 
@@ -51,7 +52,7 @@ module ShopifyCli
     # a valid google cloud config file with email ending in "@shopify.com"
     #
     def shopifolk?
-      return false unless Feature.enabled?('shopifolk-beta')
+      return false unless ShopifyCli::Config.get_bool('shopifolk-beta', 'enabled')
       return true if Feature.enabled?(FEATURE_NAME)
 
       if shopifolk_by_gcloud? && shopifolk_by_dev?

data/lib/shopify-cli/tasks/select_org_and_shop.rb

@@ -12,7 +12,9 @@ module ShopifyCli
           Shopifolk.act_as_shopify_organization
         end
         org = get_organization(organization_id)
-        shop_domain ||= get_shop_domain(org)
+        unless Shopifolk.acting_as_shopify_organization?
+          shop_domain ||= get_shop_domain(org)
+        end
         ShopifyCli::Core::Monorail.metadata[:organization_id] = org["id"].to_i
         response(org["id"].to_i, shop_domain)
       end
@@ -20,10 +22,9 @@ module ShopifyCli
       private
 
       def response(organization_id, shop_domain)
-        {
-          organization_id: organization_id,
-          shop_domain: shop_domain,
-        }
+        result = { organization_id: organization_id }
+        result[:shop_domain] = shop_domain if shop_domain
+        result
       end
 
       def organizations

data/lib/shopify-cli/tunnel.rb

@@ -123,7 +123,8 @@ module ShopifyCli
     private
 
     def install(ctx)
-      return if File.exist?(File.join(ShopifyCli.cache_dir, ctx.windows? ? 'ngrok.exe' : 'ngrok'))
+      ngrok = "ngrok#{ctx.executable_file_extension}"
+      return if File.exist?(File.join(ShopifyCli.cache_dir, ngrok))
       check_prereq_command(ctx, 'curl')
       check_prereq_command(ctx, ctx.linux? ? 'unzip' : 'tar')
       spinner = CLI::UI::SpinGroup.new
@@ -141,6 +142,11 @@ module ShopifyCli
         ctx.rm(zip_dest)
       end
       spinner.wait
+
+      # final check to see if ngrok is accessible
+      unless File.exist?(File.join(ShopifyCli.cache_dir, ngrok))
+        ctx.abort(ctx.message('core.tunnel.error.ngrok', ngrok, ShopifyCli.cache_dir))
+      end
     end
 
     def fetch_url(ctx, log_path)

data/lib/shopify-cli/version.rb

@@ -1,3 +1,3 @@
 module ShopifyCli
-  VERSION = '1.5.0'
+  VERSION = '1.6.0'
 end

data/lib/shopify_cli.rb

@@ -15,7 +15,7 @@ ENV['PATH'] = ENV['PATH'].split(':').select { |p| p.start_with?('/', '~') }.join
 vendor_path = File.expand_path("../../vendor/lib", __FILE__)
 $LOAD_PATH.unshift(vendor_path) unless $LOAD_PATH.include?(vendor_path)
 
-deps = %w(cli-ui cli-kit smart_properties)
+deps = %w(cli-ui cli-kit smart_properties webrick)
 deps.each do |dep|
   vendor_path = File.expand_path("../../vendor/deps/#{dep}/lib", __FILE__)
   $LOAD_PATH.unshift(vendor_path) unless $LOAD_PATH.include?(vendor_path)
@@ -108,6 +108,7 @@ module ShopifyCli
   autoload :Heroku, 'shopify-cli/heroku'
   autoload :JsDeps, 'shopify-cli/js_deps'
   autoload :JsSystem, 'shopify-cli/js_system'
+  autoload :MethodObject, 'shopify-cli/method_object'
   autoload :Log, 'shopify-cli/log'
   autoload :OAuth, 'shopify-cli/oauth'
   autoload :Options, 'shopify-cli/options'
@@ -115,7 +116,9 @@ module ShopifyCli
   autoload :ProcessSupervision, 'shopify-cli/process_supervision'
   autoload :Project, 'shopify-cli/project'
   autoload :ProjectType, 'shopify-cli/project_type'
+  autoload :ResolveConstant, 'shopify-cli/resolve_constant'
   autoload :Resources, 'shopify-cli/resources'
+  autoload :Result, 'shopify-cli/result'
   autoload :Shopifolk, 'shopify-cli/shopifolk'
   autoload :SubCommand, 'shopify-cli/sub_command'
   autoload :Task, 'shopify-cli/task'

data/shopify.fish

@@ -7,6 +7,6 @@ if [ $status != "0" ]
   echo "This version of Shopify App CLI is no longer supported. You’ll need to migrate to the new CLI version to continue.
 
 Please visit this page for complete instructions:
-  https://shopify.github.io/shopify-app-cli/migrate/
+  https://shopify.dev/tools/cli/troubleshooting#migrate-from-a-legacy-version
 "
 end

data/shopify.sh

@@ -6,6 +6,6 @@ if [ "$?" != "0" ]; then
   echo "This version of Shopify App CLI is no longer supported. You’ll need to migrate to the new CLI version to continue.
 
 Please visit this page for complete instructions:
-  https://shopify.github.io/shopify-app-cli/migrate/
+  https://shopify.dev/tools/cli/troubleshooting#migrate-from-a-legacy-version
 "
 fi

data/vendor/deps/cli-kit/REVISION

@@ -1 +1 @@
-1013aa5c5664e7034ca3f02fd2e0513361b07e95 (dirty)
+6eadd6c7f7fc9217008c2cfb0bc3101cb6cf3b41

data/vendor/deps/cli-kit/lib/cli/kit/logger.rb

@@ -61,7 +61,7 @@ module CLI
       #
       # @param msg [String] the message to log
       def debug(msg)
-        $stdout.puts CLI::UI.fmt(msg) if is_debug?
+        $stdout.puts CLI::UI.fmt(msg) if debug?
         @debug_logger.debug(format_debug(msg))
       end
 
@@ -73,7 +73,7 @@ module CLI
         "[#{CLI::UI::StdoutRouter.current_id[:id]}] #{msg}"
       end
 
-      def is_debug?
+      def debug?
         val = ENV[@env_debug_name]
         val && val != '0' && val != ''
       end

data/vendor/deps/cli-kit/lib/cli/kit/system.rb

@@ -94,11 +94,11 @@ module CLI
           delegate_open3(*a, sudo: sudo, env: env, method: :popen2, **kwargs, &block)
         end
 
-        def popen2e(*a, sudo: false, env: ENV, **kwargs)
+        def popen2e(*a, sudo: false, env: ENV, **kwargs, &block)
           delegate_open3(*a, sudo: sudo, env: env, method: :popen2e, **kwargs, &block)
         end
 
-        def popen3(*a, sudo: false, env: ENV, **kwargs)
+        def popen3(*a, sudo: false, env: ENV, **kwargs, &block)
           delegate_open3(*a, sudo: sudo, env: env, method: :popen3, **kwargs, &block)
         end
 
@@ -112,7 +112,7 @@ module CLI
         # - `**kwargs`: additional keyword arguments to pass to Process.spawn
         #
         # #### Returns
-        # - `status`: boolean success status of the command execution
+        # - `status`: The `Process:Status` result for the command execution
         #
         # #### Usage
         # `stat = CLI::Kit::System.system('ls', 'a_folder')`

data/vendor/deps/cli-ui/REVISION

@@ -1 +1 @@
-5ea578d3110af35c73837a8008147ce53a4b56ed (dirty)
+168ce68b31d8436b2b222d6f063e5c7225785bd4

data/vendor/deps/cli-ui/lib/cli/ui.rb

@@ -1,18 +1,19 @@
 module CLI
   module UI
-    autoload :ANSI,               'cli/ui/ansi'
-    autoload :Glyph,              'cli/ui/glyph'
-    autoload :Color,              'cli/ui/color'
-    autoload :Frame,              'cli/ui/frame'
-    autoload :OS,                 'cli/ui/os'
-    autoload :Printer,            'cli/ui/printer'
-    autoload :Progress,           'cli/ui/progress'
-    autoload :Prompt,             'cli/ui/prompt'
-    autoload :Terminal,           'cli/ui/terminal'
-    autoload :Truncater,          'cli/ui/truncater'
-    autoload :Formatter,          'cli/ui/formatter'
-    autoload :Spinner,            'cli/ui/spinner'
-    autoload :Widgets,            'cli/ui/widgets'
+    autoload :ANSI,      'cli/ui/ansi'
+    autoload :Glyph,     'cli/ui/glyph'
+    autoload :Color,     'cli/ui/color'
+    autoload :Frame,     'cli/ui/frame'
+    autoload :OS,        'cli/ui/os'
+    autoload :Printer,   'cli/ui/printer'
+    autoload :Progress,  'cli/ui/progress'
+    autoload :Prompt,    'cli/ui/prompt'
+    autoload :Terminal,  'cli/ui/terminal'
+    autoload :Truncater, 'cli/ui/truncater'
+    autoload :Formatter, 'cli/ui/formatter'
+    autoload :Spinner,   'cli/ui/spinner'
+    autoload :Widgets,   'cli/ui/widgets'
+    autoload :Wrap,      'cli/ui/wrap'
 
     # Convenience accessor to +CLI::UI::Spinner::SpinGroup+
     SpinGroup = Spinner::SpinGroup
@@ -29,7 +30,7 @@ module CLI
     end
 
     # Color resolution using +CLI::UI::Color.lookup+
-    # Will lookup using +Color.lookup+ if a symbol, otherwise we assume it is a valid color and return it
+    # Will lookup using +Color.lookup+ unless it's already a CLI::UI::Color (or nil)
     #
     # ==== Attributes
     #
@@ -37,26 +38,25 @@ module CLI
     #
     def self.resolve_color(input)
       case input
-      when Symbol
-        CLI::UI::Color.lookup(input)
-      else
+      when CLI::UI::Color, nil
         input
+      else
+        CLI::UI::Color.lookup(input)
       end
     end
 
     # Frame style resolution using +CLI::UI::Frame::FrameStyle.lookup+.
-    # Will lookup using +FrameStyle.lookup+ if the input is a symbol.  Otherwise,
-    # we assume it's a valid FrameStyle
+    # Will lookup using +FrameStyle.lookup+ unless it's already a CLI::UI::Frame::FrameStyle(or nil)
     #
     # ==== Attributes
     #
     # * +input+ - frame style to resolve
     def self.resolve_style(input)
       case input
-      when Symbol
-        CLI::UI::Frame::FrameStyle.lookup(input)
-      else
+      when CLI::UI::Frame::FrameStyle, nil
         input
+      else
+        CLI::UI::Frame::FrameStyle.lookup(input)
       end
     end
 
@@ -114,6 +114,10 @@ module CLI
       CLI::UI::Formatter.new(input).format(enable_color: enable_color)
     end
 
+    def self.wrap(input)
+      CLI::UI::Wrap.new(input).wrap
+    end
+
     # Convenience Method for +CLI::UI::Printer.puts+
     #
     # ==== Attributes

data/vendor/deps/cli-ui/lib/cli/ui/ansi.rb

@@ -21,6 +21,8 @@ module CLI
           when 0x200d # zero-width joiner
             zwj = true
             acc
+          when "\n"
+            acc
           else
             acc + 1
           end
@@ -138,17 +140,13 @@ module CLI
       # Move to the next line
       #
       def self.next_line
-        cmd = cursor_down + control('1', 'G')
-        cmd += control('1', 'D') if CLI::UI::OS.current.shift_cursor_on_line_reset?
-        cmd
+        cursor_down + cursor_horizontal_absolute
       end
 
       # Move to the previous line
       #
       def self.previous_line
-        cmd = cursor_up + control('1', 'G')
-        cmd += control('1', 'D') if CLI::UI::OS.current.shift_cursor_on_line_reset?
-        cmd
+        cursor_up + cursor_horizontal_absolute
       end
 
       def self.clear_to_end_of_line

data/vendor/deps/cli-ui/lib/cli/ui/frame.rb

@@ -90,7 +90,7 @@ module CLI
 
           t_start = Time.now
           CLI::UI.raw do
-            print prefix.chop
+            print(prefix.chop)
             puts frame_style.open(text, color: color)
           end
           FrameStack.push(color: color, style: frame_style)
@@ -153,7 +153,7 @@ module CLI
           frame_style = CLI::UI.resolve_style(frame_style) || fs_item.frame_style
 
           CLI::UI.raw do
-            print prefix.chop
+            print(prefix.chop)
             puts frame_style.divider(text, color: color)
           end
 
@@ -197,7 +197,7 @@ module CLI
           end
 
           CLI::UI.raw do
-            print prefix.chop
+            print(prefix.chop)
             puts frame_style.close(text, color: color, **kwargs)
           end
         end

data/vendor/deps/cli-ui/lib/cli/ui/frame/frame_stack.rb

@@ -5,13 +5,12 @@ module CLI
         COLOR_ENVVAR = 'CLI_FRAME_STACK'
         STYLE_ENVVAR = 'CLI_STYLE_STACK'
 
-        StackItem = Struct.new(:color_name, :style_name) do
-          def color
-            @color ||= CLI::UI.resolve_color(color_name)
-          end
+        class StackItem
+          attr_reader :color, :frame_style
 
-          def frame_style
-            @frame_style ||= CLI::UI.resolve_style(style_name)
+          def initialize(color_name, style_name)
+            @color = CLI::UI.resolve_color(color_name)
+            @frame_style = CLI::UI.resolve_style(style_name)
           end
         end
 
@@ -55,7 +54,7 @@ module CLI
               end
             end
 
-            item ||= StackItem.new(color.name, style.name)
+            item ||= StackItem.new(color, style)
 
             curr = items
             curr << item
@@ -85,8 +84,8 @@ module CLI
             styles = []
 
             items.each do |item|
-              colors << item.color_name
-              styles << item.style_name
+              colors << item.color.name
+              styles << item.frame_style.name
             end
 
             ENV[COLOR_ENVVAR] = colors.join(':')

data/vendor/deps/cli-ui/lib/cli/ui/frame/frame_style.rb

@@ -39,7 +39,7 @@ module CLI
           def message
             keys = FrameStyle.loaded_styles.map(&:inspect).join(',')
             "invalid frame style: #{@name.inspect}" \
-              "-- must be one of CLI::UI::Frame::FrameStyle.loaded_styles " \
+              " -- must be one of CLI::UI::Frame::FrameStyle.loaded_styles " \
               "(#{keys})"
           end
         end

data/vendor/deps/cli-ui/lib/cli/ui/glyph.rb

@@ -57,6 +57,7 @@ module CLI
       BUG       = new('b', 0x1f41b,          '!', Color::WHITE)  # Bug emoji (🐛)
       CHEVRON   = new('>', 0xbb,             '»', Color::YELLOW) # RIGHT-POINTING DOUBLE ANGLE QUOTATION MARK (»)
       HOURGLASS = new('H', [0x231b, 0xfe0e], 'H', Color::BLUE)   # HOURGLASS + VARIATION SELECTOR 15 (⌛︎)
+      WARNING   = new('!', [0x26a0, 0xfe0f], '!', Color::YELLOW) # WARNING SIGN + VARIATION SELECTOR 16 (⚠️ )
 
       # Looks up a glyph by name
       #

data/vendor/deps/cli-ui/lib/cli/ui/printer.rb

@@ -18,6 +18,7 @@ module CLI
       # * +:encoding+ - Force the output to be in a certain encoding. Defaults to UTF-8.
       # * +:format+ - Whether to format the string using CLI::UI.fmt. Defaults to true.
       # * +:graceful+ - Whether to gracefully ignore common I/O errors. Defaults to true.
+      # * +:wrap+ - Whether to wrap text at word boundaries to terminal width. Defaults to true.
       #
       # ==== Returns
       # Returns whether the message was successfully printed,
@@ -25,11 +26,22 @@ module CLI
       #
       # ==== Example
       #
-      #   CLI::UI::Printer.puts('{x} Ouch', stream: $stderr, color: :red)
-      #
-      def self.puts(msg, frame_color: nil, to: $stdout, encoding: Encoding::UTF_8, format: true, graceful: true)
+      #   CLI::UI::Printer.puts('{{x}} Ouch', to: $stderr)
+      #
+      def self.puts(
+        msg,
+        frame_color:
+        nil,
+        to:
+        $stdout,
+        encoding: Encoding::UTF_8,
+        format: true,
+        graceful: true,
+        wrap: true
+      )
         msg = (+msg).force_encoding(encoding) if encoding
         msg = CLI::UI.fmt(msg) if format
+        msg = CLI::UI.wrap(msg) if wrap
 
         if frame_color
           CLI::UI::Frame.with_frame_color_override(frame_color) { to.puts(msg) }

data/vendor/deps/cli-ui/lib/cli/ui/prompt/interactive_options.rb

@@ -334,22 +334,15 @@ module CLI
         end
 
         def read_char
-          raw_tty! do
-            getc = $stdin.getc
-            getc ? getc.chr : :timeout
+          if $stdin.tty? && !ENV['TEST']
+            $stdin.getch # raw mode for tty
+          else
+            $stdin.getc
           end
         rescue IOError
           "\e"
         end
 
-        def raw_tty!
-          if ENV['TEST'] || !$stdin.tty?
-            yield
-          else
-            $stdin.raw { yield }
-          end
-        end
-
         def presented_options(recalculate: false)
           return @presented_options unless recalculate
 

data/vendor/deps/cli-ui/lib/cli/ui/spinner.rb

@@ -12,11 +12,9 @@ module CLI
 
       RUNES = CLI::UI::OS.current.supports_emoji? ? %w(⠋ ⠙ ⠹ ⠸ ⠼ ⠴ ⠦ ⠧ ⠇ ⠏).freeze : %w(\\ | / - \\ | / -).freeze
 
-      begin
-        colors = [CLI::UI::Color::CYAN.code] * (RUNES.size / 2).ceil +
-          [CLI::UI::Color::MAGENTA.code] * (RUNES.size / 2).to_i
-        GLYPHS = colors.zip(RUNES).map(&:join)
-      end
+      colors = [CLI::UI::Color::CYAN.code] * (RUNES.size / 2).ceil +
+        [CLI::UI::Color::MAGENTA.code] * (RUNES.size / 2).to_i
+      GLYPHS = colors.zip(RUNES).map(&:join)
 
       class << self
         attr_accessor(:index)