shopify-cli 1.6.0 → 1.9.1

data/lib/project_types/node/commands/serve.rb

@@ -5,15 +5,15 @@ module Node
       prerequisite_task :ensure_env, :ensure_dev_store
 
       options do |parser, flags|
-        parser.on('--host=HOST') do |h|
-          flags[:host] = h.gsub('"', '')
+        parser.on("--host=HOST") do |h|
+          flags[:host] = h.gsub('"', "")
         end
       end
 
       def call(*)
         project = ShopifyCli::Project.current
         url = options.flags[:host] || ShopifyCli::Tunnel.start(@ctx)
-        @ctx.abort(@ctx.message('node.serve.error.host_must_be_https')) if url.match(/^https/i).nil?
+        @ctx.abort(@ctx.message("node.serve.error.host_must_be_https")) if url.match(/^https/i).nil?
         project.env.update(@ctx, :host, url)
         ShopifyCli::Tasks::UpdateDashboardURLS.call(
           @ctx,
@@ -23,22 +23,22 @@ module Node
 
         if project.env.shop
           project_url = "#{project.env.host}/auth?shop=#{project.env.shop}"
-          @ctx.puts("\n" + @ctx.message('node.serve.open_info', project_url) + "\n")
+          @ctx.puts("\n" + @ctx.message("node.serve.open_info", project_url) + "\n")
         end
 
-        CLI::UI::Frame.open(@ctx.message('node.serve.running_server')) do
+        CLI::UI::Frame.open(@ctx.message("node.serve.running_server")) do
           env = project.env.to_h
-          env['PORT'] = ShopifyCli::Tunnel::PORT.to_s
-          @ctx.system('npm run dev', env: env)
+          env["PORT"] = ShopifyCli::Tunnel::PORT.to_s
+          @ctx.system("npm run dev", env: env)
         end
       end
 
       def self.help
-        ShopifyCli::Context.message('node.serve.help', ShopifyCli::TOOL_NAME)
+        ShopifyCli::Context.message("node.serve.help", ShopifyCli::TOOL_NAME)
       end
 
       def self.extended_help
-        ShopifyCli::Context.message('node.serve.extended_help')
+        ShopifyCli::Context.message("node.serve.extended_help")
       end
     end
   end

data/lib/project_types/node/commands/tunnel.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'shopify_cli'
+require "shopify_cli"
 
 module Node
   module Commands
@@ -10,17 +10,17 @@ module Node
       def call(args, _name)
         subcommand = args.shift
         case subcommand
-        when 'auth'
+        when "auth"
           token = args.shift
           if token.nil?
-            @ctx.puts(@ctx.message('node.tunnel.error.token_argument_missing'))
+            @ctx.puts(@ctx.message("node.tunnel.error.token_argument_missing"))
             @ctx.puts("#{self.class.help}\n#{self.class.extended_help}")
           else
             ShopifyCli::Tunnel.auth(@ctx, token)
           end
-        when 'start'
+        when "start"
           ShopifyCli::Tunnel.start(@ctx)
-        when 'stop'
+        when "stop"
           ShopifyCli::Tunnel.stop(@ctx)
         else
           @ctx.puts(self.class.help)
@@ -28,11 +28,11 @@ module Node
       end
 
       def self.help
-        ShopifyCli::Context.message('node.tunnel.help', ShopifyCli::TOOL_NAME)
+        ShopifyCli::Context.message("node.tunnel.help", ShopifyCli::TOOL_NAME)
       end
 
       def self.extended_help
-        ShopifyCli::Context.message('node.tunnel.extended_help', ShopifyCli::TOOL_NAME)
+        ShopifyCli::Context.message("node.tunnel.extended_help", ShopifyCli::TOOL_NAME)
       end
     end
   end

data/lib/project_types/node/forms/create.rb

@@ -1,4 +1,4 @@
-require 'uri'
+require "uri"
 
 module Node
   module Forms
@@ -7,9 +7,9 @@ module Node
       flag_arguments :title, :organization_id, :shop_domain, :type
 
       def ask
-        self.title ||= CLI::UI::Prompt.ask(ctx.message('node.forms.create.app_name'))
+        self.title ||= CLI::UI::Prompt.ask(ctx.message("node.forms.create.app_name"))
+        self.name = format_name
         self.type = ask_type
-        self.name = self.title.downcase.split(" ").join("_")
         res = ShopifyCli::Tasks::SelectOrgAndShop.call(ctx, organization_id: organization_id, shop_domain: shop_domain)
         self.organization_id = res[:organization_id]
         self.shop_domain = res[:shop_domain]
@@ -17,18 +17,27 @@ module Node
 
       private
 
+      def format_name
+        name = title.downcase.split(" ").join("_")
+
+        if name.include?("shopify")
+          ctx.abort(ctx.message("node.forms.create.error.invalid_app_name"))
+        end
+        name
+      end
+
       def ask_type
         if type.nil?
-          return CLI::UI::Prompt.ask(ctx.message('node.forms.create.app_type.select')) do |handler|
-            handler.option(ctx.message('node.forms.create.app_type.select_public')) { 'public' }
-            handler.option(ctx.message('node.forms.create.app_type.select_custom')) { 'custom' }
+          return CLI::UI::Prompt.ask(ctx.message("node.forms.create.app_type.select")) do |handler|
+            handler.option(ctx.message("node.forms.create.app_type.select_public")) { "public" }
+            handler.option(ctx.message("node.forms.create.app_type.select_custom")) { "custom" }
           end
         end
 
         unless ShopifyCli::Tasks::CreateApiClient::VALID_APP_TYPES.include?(type)
-          ctx.abort(ctx.message('node.forms.create.error.invalid_app_type', type))
+          ctx.abort(ctx.message("node.forms.create.error.invalid_app_type", type))
         end
-        ctx.puts(ctx.message('node.forms.create.app_type.selected', type))
+        ctx.puts(ctx.message("node.forms.create.app_type.selected", type))
         type
       end
     end

data/lib/project_types/node/messages/messages.rb

@@ -21,9 +21,9 @@ module Node
             Usage: {{command:%s create node}}
             Options:
               {{command:--name=NAME}} App name. Any string.
-              {{command:--app_url=APPURL}} App URL. Must be a valid URL.
-              {{command:--organization_id=ID}} Partner organization ID. Must be an existing organization.
-              {{command:--shop_domain=MYSHOPIFYDOMAIN }} Development store URL. Must be an existing development store.
+              {{command:--app-url=APPURL}} App URL. Must be a valid URL.
+              {{command:--organization-id=ID}} Partner organization ID. Must be an existing organization.
+              {{command:--shop-domain=MYSHOPIFYDOMAIN }} Development store URL. Must be an existing development store.
           HELP
           error: {
             node_required: "node is required to create an app project. Download at https://nodejs.org/en/download.",
@@ -59,21 +59,21 @@ module Node
             installed: "Installed Heroku CLI",
             authenticating: "Authenticating with Heroku…",
             authenticated: "{{v}} Authenticated with Heroku",
-            authenticated_with_account: "{{v}} Authenticated with Heroku as `%s`",
+            authenticated_with_account: "{{v}} Authenticated with Heroku as {{green:%s}}",
             deploying: "Deploying to Heroku…",
             deployed: "{{v}} Deployed to Heroku",
             git: {
               checking: "Checking git repo…",
               initialized: "Git repo initialized",
               what_branch: "What branch would you like to deploy?",
-              branch_selected: "{{v}} Git branch `%s` selected for deploy",
+              branch_selected: "{{v}} Git branch {{green:%s}} selected for deploy",
             },
             app: {
               no_apps_found: "No existing Heroku app found. What would you like to do?",
               name: "What is your Heroku app’s name?",
               select: "Specify an existing Heroku app",
-              selecting: "Selecting Heroku app `%s`…",
-              selected: "{{v}} Heroku app `%s` selected",
+              selecting: "Selecting Heroku app %s…",
+              selected: "{{v}} Heroku app {{green:%s}} selected",
               create: "Create a new Heroku app",
               creating: "Creating new Heroku app…",
               created: "{{v}} New Heroku app created",
@@ -202,6 +202,7 @@ module Node
         forms: {
           create: {
             error: {
+              invalid_app_name: "App name cannot contain 'Shopify'",
               invalid_app_type: "Invalid app type %s",
             },
             app_name: "App name",

data/lib/project_types/rails/cli.rb

@@ -1,32 +1,32 @@
 # frozen_string_literal: true
 module Rails
   class Project < ShopifyCli::ProjectType
-    title('Ruby on Rails App')
-    creator('Rails::Commands::Create')
-    connector('Rails::Commands::Connect')
+    title("Ruby on Rails App")
+    creator("Rails::Commands::Create")
+    connector("Rails::Commands::Connect")
 
-    register_command('Rails::Commands::Deploy', "deploy")
-    register_command('Rails::Commands::Generate', "generate")
-    register_command('Rails::Commands::Open', "open")
-    register_command('Rails::Commands::Populate', "populate")
-    register_command('Rails::Commands::Serve', "serve")
-    register_command('Rails::Commands::Tunnel', "tunnel")
+    register_command("Rails::Commands::Deploy", "deploy")
+    register_command("Rails::Commands::Generate", "generate")
+    register_command("Rails::Commands::Open", "open")
+    register_command("Rails::Commands::Populate", "populate")
+    register_command("Rails::Commands::Serve", "serve")
+    register_command("Rails::Commands::Tunnel", "tunnel")
     # register_task('Rails::Tasks::RailsTask', 'rails_task')
 
-    require Project.project_filepath('messages/messages')
+    require Project.project_filepath("messages/messages")
     register_messages(Rails::Messages::MESSAGES)
   end
 
   # define/autoload project specific Commands
   module Commands
-    autoload :Connect, Project.project_filepath('commands/connect')
-    autoload :Create, Project.project_filepath('commands/create')
-    autoload :Deploy, Project.project_filepath('commands/deploy')
-    autoload :Generate, Project.project_filepath('commands/generate')
-    autoload :Open, Project.project_filepath('commands/open')
-    autoload :Populate, Project.project_filepath('commands/populate')
-    autoload :Serve, Project.project_filepath('commands/serve')
-    autoload :Tunnel, Project.project_filepath('commands/tunnel')
+    autoload :Connect, Project.project_filepath("commands/connect")
+    autoload :Create, Project.project_filepath("commands/create")
+    autoload :Deploy, Project.project_filepath("commands/deploy")
+    autoload :Generate, Project.project_filepath("commands/generate")
+    autoload :Open, Project.project_filepath("commands/open")
+    autoload :Populate, Project.project_filepath("commands/populate")
+    autoload :Serve, Project.project_filepath("commands/serve")
+    autoload :Tunnel, Project.project_filepath("commands/tunnel")
   end
 
   # define/autoload project specific Tasks
@@ -35,9 +35,9 @@ module Rails
 
   # define/autoload project specific Forms
   module Forms
-    autoload :Create, Project.project_filepath('forms/create')
+    autoload :Create, Project.project_filepath("forms/create")
   end
 
-  autoload :Ruby, Project.project_filepath('ruby')
-  autoload :Gem, Project.project_filepath('gem')
+  autoload :Ruby, Project.project_filepath("ruby")
+  autoload :Gem, Project.project_filepath("gem")
 end

data/lib/project_types/rails/commands/connect.rb

@@ -4,11 +4,11 @@ module Rails
     class Connect < ShopifyCli::SubCommand
       def call(*)
         if ShopifyCli::Project.has_current? && ShopifyCli::Project.current.env
-          @ctx.puts(@ctx.message('rails.connect.production_warning'))
+          @ctx.puts(@ctx.message("rails.connect.production_warning"))
         end
 
-        app = ShopifyCli::Commands::Connect.new.default_connect('rails')
-        @ctx.done(@ctx.message('rails.connect.connected', app))
+        app = ShopifyCli::Commands::Connect.new.default_connect("rails")
+        @ctx.done(@ctx.message("rails.connect.connected", app))
       end
     end
   end

data/lib/project_types/rails/commands/create.rb

@@ -14,13 +14,16 @@ module Rails
 
       options do |parser, flags|
         # backwards compatibility allow 'title' for now
-        parser.on('--title=TITLE') { |t| flags[:title] = t }
-        parser.on('--name=NAME') { |t| flags[:title] = t }
-        parser.on('--organization_id=ID') { |url| flags[:organization_id] = url }
-        parser.on('--shop_domain=MYSHOPIFYDOMAIN') { |url| flags[:shop_domain] = url }
-        parser.on('--type=APPTYPE') { |url| flags[:type] = url }
-        parser.on('--db=DB') { |db| flags[:db] = db }
-        parser.on('--rails_opts=RAILSOPTS') { |opts| flags[:rails_opts] = opts }
+        parser.on("--title=TITLE") { |t| flags[:title] = t }
+        parser.on("--name=NAME") { |t| flags[:title] = t }
+        parser.on("--organization_id=ID") { |url| flags[:organization_id] = url }
+        parser.on("--organization-id=ID") { |url| flags[:organization_id] = url }
+        parser.on("--shop_domain=MYSHOPIFYDOMAIN") { |url| flags[:shop_domain] = url }
+        parser.on("--shop-domain=MYSHOPIFYDOMAIN") { |url| flags[:shop_domain] = url }
+        parser.on("--type=APPTYPE") { |url| flags[:type] = url }
+        parser.on("--db=DB") { |db| flags[:db] = db }
+        parser.on("--rails_opts=RAILSOPTS") { |opts| flags[:rails_opts] = opts }
+        parser.on("--rails-opts=RAILSOPTS") { |opts| flags[:rails_opts] = opts }
       end
 
       def call(args, _name)
@@ -28,8 +31,8 @@ module Rails
         return @ctx.puts(self.class.help) if form.nil?
 
         ruby_version = Ruby.version(@ctx)
-        @ctx.abort(@ctx.message('rails.create.error.invalid_ruby_version')) unless
-          ruby_version.satisfies?('~>2.5') || ruby_version.satisfies?('~>3.0.0')
+        @ctx.abort(@ctx.message("rails.create.error.invalid_ruby_version")) unless
+          ruby_version.satisfies?("~>2.5") || ruby_version.satisfies?("~>3.0.0")
 
         check_node
         check_yarn
@@ -38,7 +41,7 @@ module Rails
         set_custom_ua
         ShopifyCli::Project.write(
           @ctx,
-          project_type: 'rails',
+          project_type: "rails",
           organization_id: form.organization_id,
         )
 
@@ -53,74 +56,74 @@ module Rails
           api_key: api_client["apiKey"],
           secret: api_client["apiSecretKeys"].first["secret"],
           shop: form.shop_domain,
-          scopes: 'write_products,write_customers,write_draft_orders',
+          scopes: "write_products,write_customers,write_draft_orders",
         ).write(@ctx)
 
-        partners_url = ShopifyCli::PartnersAPI.partners_url_for(form.organization_id, api_client['id'], local_debug?)
+        partners_url = ShopifyCli::PartnersAPI.partners_url_for(form.organization_id, api_client["id"], local_debug?)
 
-        @ctx.puts(@ctx.message('apps.create.info.created', form.title, partners_url))
-        @ctx.puts(@ctx.message('apps.create.info.serve', form.name, ShopifyCli::TOOL_NAME))
+        @ctx.puts(@ctx.message("apps.create.info.created", form.title, partners_url))
+        @ctx.puts(@ctx.message("apps.create.info.serve", form.name, ShopifyCli::TOOL_NAME))
         unless ShopifyCli::Shopifolk.acting_as_shopify_organization?
-          @ctx.puts(@ctx.message('apps.create.info.install', partners_url, form.title))
+          @ctx.puts(@ctx.message("apps.create.info.install", partners_url, form.title))
         end
       end
 
       def self.help
-        ShopifyCli::Context.message('rails.create.help', ShopifyCli::TOOL_NAME, ShopifyCli::TOOL_NAME)
+        ShopifyCli::Context.message("rails.create.help", ShopifyCli::TOOL_NAME, ShopifyCli::TOOL_NAME)
       end
 
       private
 
       def check_node
-        cmd_path = @ctx.which('node')
+        cmd_path = @ctx.which("node")
         if cmd_path.nil?
-          @ctx.abort(@ctx.message('rails.create.error.node_required')) unless @ctx.windows?
-          @ctx.puts("{{x}} {{red:" + @ctx.message('rails.create.error.node_required') + "}}")
-          @ctx.puts(@ctx.message('rails.create.info.open_new_shell', 'node'))
+          @ctx.abort(@ctx.message("rails.create.error.node_required")) unless @ctx.windows?
+          @ctx.puts("{{x}} {{red:" + @ctx.message("rails.create.error.node_required") + "}}")
+          @ctx.puts(@ctx.message("rails.create.info.open_new_shell", "node"))
           raise ShopifyCli::AbortSilent
         end
 
-        version, stat = @ctx.capture2e('node', '-v')
+        version, stat = @ctx.capture2e("node", "-v")
         unless stat.success?
-          @ctx.abort(@ctx.message('rails.create.error.node_version_failure')) unless @ctx.windows?
+          @ctx.abort(@ctx.message("rails.create.error.node_version_failure")) unless @ctx.windows?
           # execution stops above if not Windows
-          @ctx.puts("{{x}} {{red:" + @ctx.message('rails.create.error.node_version_failure') + "}}")
-          @ctx.puts(@ctx.message('rails.create.info.open_new_shell', 'node'))
+          @ctx.puts("{{x}} {{red:" + @ctx.message("rails.create.error.node_version_failure") + "}}")
+          @ctx.puts(@ctx.message("rails.create.info.open_new_shell", "node"))
           raise ShopifyCli::AbortSilent
         end
 
-        @ctx.done(@ctx.message('rails.create.node_version', version))
+        @ctx.done(@ctx.message("rails.create.node_version", version))
       end
 
       def check_yarn
-        cmd_path = @ctx.which('yarn')
+        cmd_path = @ctx.which("yarn")
         if cmd_path.nil?
-          @ctx.abort(@ctx.message('rails.create.error.yarn_required')) unless @ctx.windows?
-          @ctx.puts("{{x}} {{red:" + @ctx.message('rails.create.error.yarn_required') + "}}")
-          @ctx.puts(@ctx.message('rails.create.info.open_new_shell', 'yarn'))
+          @ctx.abort(@ctx.message("rails.create.error.yarn_required")) unless @ctx.windows?
+          @ctx.puts("{{x}} {{red:" + @ctx.message("rails.create.error.yarn_required") + "}}")
+          @ctx.puts(@ctx.message("rails.create.info.open_new_shell", "yarn"))
           raise ShopifyCli::AbortSilent
         end
 
-        version, stat = @ctx.capture2e('yarn', '-v')
+        version, stat = @ctx.capture2e("yarn", "-v")
         unless stat.success?
-          @ctx.abort(@ctx.message('rails.create.error.yarn_version_failure')) unless @ctx.windows?
-          @ctx.puts("{{x}} {{red:" + @ctx.message('rails.create.error.yarn_version_failure') + "}}")
-          @ctx.puts(@ctx.message('rails.create.info.open_new_shell', 'yarn'))
+          @ctx.abort(@ctx.message("rails.create.error.yarn_version_failure")) unless @ctx.windows?
+          @ctx.puts("{{x}} {{red:" + @ctx.message("rails.create.error.yarn_version_failure") + "}}")
+          @ctx.puts(@ctx.message("rails.create.info.open_new_shell", "yarn"))
           raise ShopifyCli::AbortSilent
         end
 
-        @ctx.done(@ctx.message('rails.create.yarn_version', version))
+        @ctx.done(@ctx.message("rails.create.yarn_version", version))
       end
 
       def build(name, db)
-        @ctx.abort(@ctx.message('rails.create.error.install_failure', 'rails')) unless install_gem('rails', '<6.1')
-        @ctx.abort(@ctx.message('rails.create.error.install_failure', 'bundler ~>2.0')) unless
-          install_gem('bundler', '~>2.0')
+        @ctx.abort(@ctx.message("rails.create.error.install_failure", "rails")) unless install_gem("rails", "<6.1")
+        @ctx.abort(@ctx.message("rails.create.error.install_failure", "bundler ~>2.0")) unless
+          install_gem("bundler", "~>2.0")
 
         full_path = File.join(@ctx.root, name)
-        @ctx.abort(@ctx.message('rails.create.error.dir_exists', name)) if Dir.exist?(full_path)
+        @ctx.abort(@ctx.message("rails.create.error.dir_exists", name)) if Dir.exist?(full_path)
 
-        CLI::UI::Frame.open(@ctx.message('rails.create.generating_app', name)) do
+        CLI::UI::Frame.open(@ctx.message("rails.create.generating_app", name)) do
           new_command = %w(rails new)
           new_command += DEFAULT_RAILS_FLAGS
           new_command << "--database=#{db}"
@@ -132,34 +135,34 @@ module Rails
 
         @ctx.root = full_path
 
-        File.open(File.join(@ctx.root, '.gitignore'), 'a') { |f| f.write('.env') }
+        File.open(File.join(@ctx.root, ".gitignore"), "a") { |f| f.write(".env") }
 
-        @ctx.puts(@ctx.message('rails.create.adding_shopify_gem'))
-        File.open(File.join(@ctx.root, 'Gemfile'), 'a') do |f|
+        @ctx.puts(@ctx.message("rails.create.adding_shopify_gem"))
+        File.open(File.join(@ctx.root, "Gemfile"), "a") do |f|
           f.puts "\ngem 'shopify_app', '>=17.0.3'"
         end
-        CLI::UI::Frame.open(@ctx.message('rails.create.running_bundle_install')) do
+        CLI::UI::Frame.open(@ctx.message("rails.create.running_bundle_install")) do
           syscall(%w(bundle install))
         end
 
-        CLI::UI::Frame.open(@ctx.message('rails.create.running_generator')) do
+        CLI::UI::Frame.open(@ctx.message("rails.create.running_generator")) do
           syscall(%w(rails generate shopify_app --new-shopify-cli-app))
         end
 
-        CLI::UI::Frame.open(@ctx.message('rails.create.running_migrations')) do
+        CLI::UI::Frame.open(@ctx.message("rails.create.running_migrations")) do
           syscall(%w(rails db:create))
           syscall(%w(rails db:migrate RAILS_ENV=development))
         end
 
-        unless File.exist?(File.join(@ctx.root, 'config/webpacker.yml'))
-          CLI::UI::Frame.open(@ctx.message('rails.create.running_webpacker_install')) do
+        unless File.exist?(File.join(@ctx.root, "config/webpacker.yml"))
+          CLI::UI::Frame.open(@ctx.message("rails.create.running_webpacker_install")) do
             syscall(%w(rails webpacker:install))
           end
         end
       end
 
       def set_custom_ua
-        ua_path = File.join('config', 'initializers', 'user_agent.rb')
+        ua_path = File.join("config", "initializers", "user_agent.rb")
         @ctx.write(ua_path, USER_AGENT_CODE)
       end