shopify-cli 2.7.0 → 2.7.4

data/lib/project_types/extension/features/argo_serve.rb

@@ -9,7 +9,7 @@ module Extension
       property! :specification_handler, accepts: Extension::Models::SpecificationHandlers::Default
       property :argo_runtime, accepts: -> (runtime) { runtime.class < Features::Runtimes::Base }
       property! :context, accepts: ShopifyCLI::Context
-      property! :port, accepts: Integer, default: 39351
+      property! :port, accepts: Integer, default: ShopifyCLI::Constants::Extension::DEFAULT_PORT
       property  :tunnel_url, accepts: String, default: nil
       property! :js_system, accepts: ->(jss) { jss.respond_to?(:call) }, default: ShopifyCLI::JsSystem
       property :resource_url, accepts: String, default: nil
@@ -95,15 +95,16 @@ module Extension
       end
 
       def new_serve_flow
-        Tasks::RunExtensionCommand.new(
+        Tasks::ExecuteCommands.serve(
           type: specification_handler.specification.identifier,
-          command: "serve",
           context: context,
+          config_file_path: specification_handler.server_config_path,
           port: port,
-          config_file_name: specification_handler.server_config_file,
           resource_url: resource_url,
           tunnel_url: tunnel_url
-        ).call
+        ).unwrap do |error|
+          raise error unless error.nil?
+        end
       end
 
       def supports_development_server?

data/lib/project_types/extension/forms/questions/ask_registration.rb

@@ -34,8 +34,12 @@ module Extension
         end
 
         def load_registrations(type)
-          ctx.puts(@ctx.message("connect.loading_extensions"))
-          registrations = Tasks::GetExtensions.call(context: ctx, type: type)
+          registrations = []
+          loading_extensions = @ctx.message("connect.loading_extensions")
+
+          CLI::UI::Spinner.spin(loading_extensions) do |_spinner|
+            registrations += Tasks::GetExtensions.call(context: ctx, type: type)
+          end
 
           registrations.empty? ? abort_no_registrations : registrations
         end

data/lib/project_types/extension/loaders/project.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+module Extension
+  module Loaders
+    module Project
+      def self.load(context:, directory:, api_key:, registration_id:, api_secret:)
+        env_overrides = {
+          "SHOPIFY_API_KEY" => api_key,
+          "SHOPIFY_API_SECRET" => api_secret,
+          "EXTENSION_ID" => registration_id,
+        }.compact
+        env =
+          begin
+            ShopifyCLI::Resources::EnvFile.read(directory, overrides: env_overrides)
+          rescue Errno::ENOENT
+            ShopifyCLI::Resources::EnvFile.from_hash(env_overrides)
+          end
+        # This is a somewhat uncomfortable hack we use because `Project::at` is
+        # a global cache and we can't rely on this class loading the project
+        # first. Long-term we should move away from that global cache.
+        project = ExtensionProject.at(directory)
+        project.env = env
+        project
+      rescue SmartProperties::InitializationError, SmartProperties::MissingValueError
+        context.abort(context.message("errors.missing_api_key"))
+      end
+    end
+  end
+end

data/lib/project_types/extension/loaders/specification_handler.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Extension
+  module Loaders
+    module SpecificationHandler
+      def self.load(project:, context:)
+        identifier = project.specification_identifier
+        Models::LazySpecificationHandler.new(identifier) do
+          specifications = Models::Specifications.new(
+            fetch_specifications: Tasks::FetchSpecifications.new(api_key: project.app.api_key, context: context)
+          )
+
+          unless specifications.valid?(identifier)
+            raise ShopifyCLI::Abort, context.message("errors.unknown_type", project.specification_identifier)
+          end
+
+          specifications[identifier]
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/messages/messages.rb

@@ -68,8 +68,6 @@ module Extension
         HELP
         frame_title: "Building extension with: %s…",
         build_failure_message: "Failed to build extension code.",
-        build_success_message: "Build was successful!",
-        directory_not_found: "Build directory not found.",
       },
       register: {
         help: <<~HELP,
@@ -91,6 +89,9 @@ module Extension
         help: <<~HELP,
           Push the current extension to Shopify.
             Usage: {{command:%s extension push}}
+            Options:
+              {{command:--api-key=API_KEY}} Connect your extension and app by inserting your app's API key (which you can get from your app setup page on shopify.dev).
+              {{command:--registration-id=REGISTRATION_ID}} The id of the extension's registration.
         HELP
         frame_title: "Pushing your extension to Shopify",
         waiting_text: "Pushing code to Shopify…",
@@ -175,6 +176,7 @@ module Extension
       errors: {
         unknown_type: "Unknown extension type %s",
         package_not_found: "`%s` package not found.",
+        missing_api_key: "Missing api_key.",
         module_not_found: "Unable to find module %s. Ensure your dependencies are up-to-date and try again.",
       },
       warnings: {

data/lib/project_types/extension/models/app.rb

@@ -6,7 +6,7 @@ module Extension
       include SmartProperties
 
       property! :api_key, accepts: String
-      property! :secret, accepts: String
+      property :secret, accepts: String
       property :title, accepts: String
       property :business_name, accepts: String
     end

data/lib/project_types/extension/models/development_server.rb

@@ -33,8 +33,8 @@ module Extension
       end
 
       def build(server_config)
-        _, error, status = CLI::Kit::System.capture3(executable, "build", "-", stdin_data: server_config.to_yaml)
-        return if status.success?
+        output, error, status = CLI::Kit::System.capture3(executable, "build", "-", stdin_data: server_config.to_yaml)
+        return output if status.success?
         raise DevelopmentServerError, error
       end
 

data/lib/project_types/extension/models/specification_handlers/default.rb

@@ -103,6 +103,10 @@ module Extension
           raise NotImplementedError
         end
 
+        def server_config_path(base_dir = Dir.pwd)
+          File.join(base_dir, server_config_file)
+        end
+
         def server_config_file
           "shopifile.yml"
         end

data/lib/project_types/extension/tasks/convert_server_config.rb

@@ -9,11 +9,12 @@ module Extension
       property! :api_key, accepts: String
       property! :context, accepts: ShopifyCLI::Context
       property! :hash, accepts: Hash
+      property  :port, accepts: Integer, default: ShopifyCLI::Constants::Extension::DEFAULT_PORT
       property! :registration_uuid, accepts: String
       property  :resource_url, accepts: String
       property! :store, accepts: String
       property! :title, accepts: String
-      property! :tunnel_url, accepts: String
+      property  :tunnel_url, accepts: String
       property! :type, accepts: String
 
       def self.call(*args)
@@ -46,6 +47,7 @@ module Extension
         end
         server_config = Models::ServerConfig::Root.new(
           extensions: [extension],
+          port: port,
           public_url: tunnel_url,
           store: store
         )

data/lib/project_types/extension/tasks/execute_commands/base.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Tasks
+    module ExecuteCommands
+      class Base
+        include SmartProperties
+        property! :type, accepts: Models::DevelopmentServerRequirements::SUPPORTED_EXTENSION_TYPES
+      end
+    end
+  end
+end

data/lib/project_types/extension/tasks/execute_commands/build.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Tasks
+    module ExecuteCommands
+      class Build < Base
+        property! :context, accepts: ShopifyCLI::Context
+        property! :config_file_path, accepts: String
+
+        def call
+          ShopifyCLI::Result
+            .call(&method(:merge_server_config))
+            .then { |server_config| Models::DevelopmentServer.new.build(server_config) }
+        end
+
+        private
+
+        def merge_server_config
+          Tasks::MergeServerConfig.call(
+            context: context,
+            file_path: config_file_path,
+            type: type
+          )
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/tasks/execute_commands/create.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Tasks
+    module ExecuteCommands
+      class Create < Base
+        property!  :template, accepts: Models::ServerConfig::Development::VALID_TEMPLATES
+        property!  :root_dir, accepts: String
+
+        def call
+          ShopifyCLI::Result.success(generate_config)
+            .then { |server_config| Models::DevelopmentServer.new.create(server_config) }
+            .unwrap do |error|
+              raise error unless error.nil?
+            end
+        end
+
+        private
+
+        def generate_config
+          extension = Models::ServerConfig::Extension.build(
+            template: template,
+            type: type,
+            root_dir: root_dir,
+          )
+
+          Models::ServerConfig::Root.new(extensions: [extension])
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/tasks/execute_commands/serve.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Tasks
+    module ExecuteCommands
+      class Serve < Base
+        property! :context, accepts: ShopifyCLI::Context
+        property! :config_file_path, accepts: String
+        property  :port, accepts: Integer, default: ShopifyCLI::Constants::Extension::DEFAULT_PORT
+        property  :resource_url, accepts: String
+        property! :tunnel_url, accepts: String
+
+        def call
+          ShopifyCLI::Result
+            .call(&method(:merge_server_config))
+            .then { |server_config| Models::DevelopmentServer.new.serve(context, server_config) }
+        end
+
+        private
+
+        def merge_server_config
+          Tasks::MergeServerConfig.call(
+            context: context,
+            file_path: config_file_path,
+            port: port,
+            resource_url: resource_url,
+            tunnel_url: tunnel_url,
+            type: type
+          )
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/tasks/merge_server_config.rb

@@ -4,28 +4,39 @@ require "yaml"
 
 module Extension
   module Tasks
-    class MergeServerConfig < ShopifyCLI::Task
-      class << self
-        def call(context:, file_name:, resource_url:, tunnel_url:, type:)
-          config = YAML.load_file(file_name)
-          project = ExtensionProject.current
-          Tasks::ConvertServerConfig.call(
-            api_key: project.env.api_key,
-            context: context,
-            hash: config,
-            registration_uuid: project.registration_uuid,
-            resource_url: resource_url || project.resource_url,
-            store: project.env.shop || "",
-            title: project.title,
-            tunnel_url: tunnel_url,
-            type: type
-          )
-        rescue Psych::SyntaxError => e
-          raise(
-            ShopifyCLI::Abort,
-            ShopifyCLI::Context.message("core.yaml.error.invalid", file_name, e.message)
-          )
-        end
+    class MergeServerConfig
+      include SmartProperties
+
+      property! :context, accepts: ShopifyCLI::Context
+      property! :file_path, accepts: ->(path) { Pathname(path).yield_self { |pn| pn.absolute? && pn.file? } }
+      property  :port, accepts: Integer, default: ShopifyCLI::Constants::Extension::DEFAULT_PORT
+      property  :resource_url, accepts: String
+      property  :tunnel_url, accepts: String
+      property! :type, accepts: Models::DevelopmentServerRequirements::SUPPORTED_EXTENSION_TYPES
+
+      def self.call(*args)
+        new(*args).call
+      end
+
+      def call
+        config = YAML.load_file(file_path)
+        project = ExtensionProject.current
+        Tasks::ConvertServerConfig.call(
+          api_key: project.env.api_key,
+          context: context,
+          hash: config,
+          registration_uuid: project.registration_uuid,
+          resource_url: resource_url || project.resource_url,
+          store: project.env.shop || "",
+          title: project.title,
+          tunnel_url: tunnel_url,
+          type: type
+        )
+      rescue Psych::SyntaxError => e
+        raise(
+          ShopifyCLI::Abort,
+          ShopifyCLI::Context.message("core.yaml.error.invalid", file_name, e.message)
+        )
       end
     end
   end

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

@@ -2,9 +2,7 @@
 module Rails
   class Command
     class Create < ShopifyCLI::Command::AppSubCommand
-      unless ShopifyCLI::Environment.acceptance_test?
-        prerequisite_task :ensure_authenticated
-      end
+      prerequisite_task :ensure_authenticated
 
       USER_AGENT_CODE = <<~USERAGENT
         module ShopifyAPI
@@ -158,10 +156,10 @@ module Rails
 
         CLI::UI::Frame.open(@ctx.message("rails.create.generating_app", name)) do
           new_command = %w(rails new)
+          new_command << name
           new_command += DEFAULT_RAILS_FLAGS
           new_command << "--database=#{db}"
           new_command += options.flags[:rails_opts].split unless options.flags[:rails_opts].nil?
-          new_command << name
 
           syscall(new_command)
         end

data/lib/project_types/script/cli.rb

@@ -13,12 +13,19 @@ module Script
     hidden_feature(feature_set: :script_project)
     subcommand :Create, "create", Project.project_filepath("commands/create")
     subcommand :Push, "push", Project.project_filepath("commands/push")
+    subcommand :Connect, "connect", Project.project_filepath("commands/connect")
+    subcommand :Javy, "javy", Project.project_filepath("commands/javy")
   end
   ShopifyCLI::Commands.register("Script::Command", "script")
 
   # define/autoload project specific Forms
   module Forms
+    autoload :AskOrg, Project.project_filepath("forms/ask_org")
+    autoload :AskApp, Project.project_filepath("forms/ask_app")
+    autoload :AskScriptUuid, Project.project_filepath("forms/ask_script_uuid")
+    autoload :RunAgainstShopifyOrg, Project.project_filepath("forms/run_against_shopify_org")
     autoload :Create, Project.project_filepath("forms/create")
+    autoload :Connect, Project.project_filepath("forms/connect")
     autoload :ScriptForm, Project.project_filepath("forms/script_form")
   end
 
@@ -29,6 +36,7 @@ module Script
   module Layers
     module Application
       autoload :BuildScript, Project.project_filepath("layers/application/build_script")
+      autoload :ConnectApp, Project.project_filepath("layers/application/connect_app")
       autoload :CreateScript, Project.project_filepath("layers/application/create_script")
       autoload :PushScript, Project.project_filepath("layers/application/push_script")
       autoload :ExtensionPoints, Project.project_filepath("layers/application/extension_points")
@@ -40,7 +48,7 @@ module Script
       autoload :PushPackage, Project.project_filepath("layers/domain/push_package")
       autoload :Metadata, Project.project_filepath("layers/domain/metadata")
       autoload :ExtensionPoint, Project.project_filepath("layers/domain/extension_point")
-      autoload :ScriptJson, Project.project_filepath("layers/domain/script_json")
+      autoload :ScriptConfig, Project.project_filepath("layers/domain/script_config")
       autoload :ScriptProject, Project.project_filepath("layers/domain/script_project")
     end
 

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

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+module Script
+  class Command
+    class Connect < ShopifyCLI::Command::SubCommand
+      prerequisite_task :ensure_authenticated
+      prerequisite_task ensure_project_type: :script
+
+      def call(_args, _)
+        Layers::Application::ConnectApp.call(ctx: @ctx, force: true, strict: true)
+      rescue StandardError => e
+        UI::ErrorHandler.pretty_print_and_raise(e, failed_op: @ctx.message("script.connect.error.operation_failed"))
+      end
+
+      def self.help
+        ShopifyCLI::Context.new.message("connect.help", ShopifyCLI::TOOL_NAME, ShopifyCLI::TOOL_NAME)
+      end
+    end
+  end
+end

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

@@ -3,9 +3,7 @@
 module Script
   class Command
     class Create < ShopifyCLI::Command::SubCommand
-      unless ShopifyCLI::Environment.acceptance_test?
-        prerequisite_task :ensure_authenticated
-      end
+      prerequisite_task :ensure_authenticated
 
       options do |parser, flags|
         parser.on("--name=NAME") { |name| flags[:name] = name }

data/lib/project_types/script/commands/javy.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+require "shopify_cli"
+require_relative "../../../../ext/javy/javy.rb"
+
+module Script
+  class Command
+    class Javy < ShopifyCLI::Command::SubCommand
+      hidden_feature
+
+      options do |parser, flags|
+        parser.on("--in=IN") { |in_file| flags[:in_file] = in_file }
+        parser.on("--out=OUT") { |out_file| flags[:out_file] = out_file }
+      end
+
+      def call(*)
+        source = options.flags[:in_file]
+        dest = options.flags[:out_file]
+
+        @ctx.abort(@ctx.message("script.javy.errors.invalid_arguments", ShopifyCLI::TOOL_NAME)) unless source
+
+        ::Javy.build(source: source, dest: dest).unwrap { |e| @ctx.abort(e.message) }
+      end
+
+      def self.help
+        ShopifyCLI::Context.message("script.javy.help", ShopifyCLI::TOOL_NAME)
+      end
+    end
+  end
+end

data/lib/project_types/script/commands/push.rb

@@ -10,7 +10,8 @@ module Script
       end
 
       def call(_args, _name)
-        fresh_env = Tasks::EnsureEnv.call(@ctx)
+        fresh_env = Layers::Application::ConnectApp.call(ctx: @ctx)
+
         force = options.flags.key?(:force) || !!fresh_env
 
         api_key = Layers::Infrastructure::ScriptProjectRepository.new(ctx: @ctx).get.api_key

data/lib/project_types/script/config/extension_points.yml

@@ -1,29 +1,3 @@
-discount:
-  deprecated: true
-  libraries:
-    assemblyscript:
-      package: "@shopify/extension-point-as-discount"
-unit_limit_per_order:
-  beta: true
-  libraries:
-    assemblyscript:
-      package: "@shopify/extension-point-as-unit-limit-per-order"
-payment_filter:
-  deprecated: true
-  libraries:
-    assemblyscript:
-      package: "@shopify/extension-point-as-payment-filter"
-shipping_filter:
-  deprecated: true
-  libraries:
-    assemblyscript:
-      package: "@shopify/extension-point-as-shipping-filter"
-tax_filter:
-  beta: true
-  libraries:
-    assemblyscript:
-      repo: "https://github.com/Shopify/extension-points.git"
-      package: "@shopify/extension-point-as-tax-filter"
 payment_methods:
   domain: 'checkout'
   libraries:
@@ -32,7 +6,7 @@ payment_methods:
       package: "@shopify/scripts-checkout-apis"
     typescript:
       beta: true
-      package: "@shopify/scripts-checkout-apis-temp"
+      package: "@shopify/scripts-checkout-apis"
       repo: "https://github.com/Shopify/scripts-apis-examples"
 shipping_methods:
   domain: 'checkout'
@@ -42,13 +16,21 @@ shipping_methods:
       package: "@shopify/scripts-checkout-apis"
     typescript:
       beta: true
-      package: "@shopify/scripts-checkout-apis-temp"
+      package: "@shopify/scripts-checkout-apis"
+      repo: "https://github.com/Shopify/scripts-apis-examples"
+merchandise_discount_types:
+  beta: true
+  domain: 'discounts'
+  libraries:
+    typescript:
+      beta: true
+      package: "@shopify/scripts-discounts-apis"
       repo: "https://github.com/Shopify/scripts-apis-examples"
-discount_types:
+delivery_discount_types:
   beta: true
   domain: 'discounts'
   libraries:
     typescript:
       beta: true
-      package: "@shopify/scripts-discount-apis"
+      package: "@shopify/scripts-discounts-apis"
       repo: "https://github.com/Shopify/scripts-apis-examples"

data/lib/project_types/script/forms/ask_app.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module Script
+  module Forms
+    class AskApp < ShopifyCLI::Form
+      attr_reader :app
+
+      def ask
+        apps = @xargs.fetch(:apps)
+
+        unless @xargs[:acting_as_shopify_organization]
+          apps = apps.select { |app| app["appType"] == "custom" }
+        end
+
+        raise Errors::NoExistingAppsError if apps.empty?
+
+        @app =
+          if apps.count > 1
+            CLI::UI::Prompt.ask(ctx.message("script.application.ensure_env.app_select")) do |handler|
+              apps.each do |app|
+                handler.option(app["title"]) { app }
+              end
+            end
+          else
+            apps.first.tap do |app|
+              ctx.puts(ctx.message("script.application.ensure_env.app", app["title"]))
+            end
+          end
+      end
+    end
+  end
+end

data/lib/project_types/script/forms/ask_org.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Script
+  module Forms
+    class AskOrg < ShopifyCLI::Form
+      attr_reader :org
+
+      BUSINESS_NAME = "businessName"
+      ID = "id"
+
+      def ask
+        orgs = @xargs
+        @org =
+          if orgs.count == 1
+            orgs.first.tap do |org|
+              ctx.puts(ctx.message("script.application.ensure_env.organization", org[BUSINESS_NAME], org[ID]))
+            end
+          elsif orgs.count > 0
+            CLI::UI::Prompt.ask(ctx.message("script.application.ensure_env.organization_select")) do |handler|
+              orgs.each do |org|
+                handler.option("#{org[BUSINESS_NAME]} (#{org[ID]})") { org }
+              end
+            end
+          else
+            raise Errors::NoExistingOrganizationsError
+          end
+      end
+    end
+  end
+end

data/lib/project_types/script/forms/ask_script_uuid.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Script
+  module Forms
+    class AskScriptUuid < ShopifyCLI::Form
+      attr_reader :uuid
+      def ask
+        scripts = @xargs
+
+        return if scripts.empty? ||
+          !CLI::UI::Prompt.confirm(ctx.message("script.application.ensure_env.ask_connect_to_existing_script"))
+
+        @uuid =
+          CLI::UI::Prompt.ask(ctx.message("script.application.ensure_env.ask_which_script_to_connect_to")) do |handler|
+            scripts.each do |script|
+              handler.option("#{script["title"]} (#{script["uuid"]})") { script["uuid"] }
+            end
+          end
+      end
+    end
+  end
+end

data/lib/project_types/script/forms/run_against_shopify_org.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Script
+  module Forms
+    class RunAgainstShopifyOrg < ShopifyCLI::Form
+      attr_reader :response
+      def ask
+        @ctx.puts(@ctx.message("core.tasks.select_org_and_shop.identified_as_shopify"))
+        message = @ctx.message("core.tasks.select_org_and_shop.first_party")
+        @response = CLI::UI::Prompt.confirm(message, default: false)
+      end
+    end
+  end
+end