shopify-cli 1.7.0 → 1.10.0

data/lib/project_types/script/graphql/script_service_proxy.graphql

@@ -1,7 +1,6 @@
-query ProxyRequest($api_key: String, $shop_domain: String, $query: String!, $variables: String) {
+query ProxyRequest($api_key: String, $query: String!, $variables: String) {
   scriptServiceProxy(
     apiKey: $api_key
-    shopDomain: $shop_domain
     query: $query
     variables: $variables
   )

data/lib/project_types/script/layers/application/create_script.rb

@@ -7,32 +7,27 @@ module Script
     module Application
       class CreateScript
         class << self
-          def call(ctx:, language:, script_name:, extension_point_type:, description:)
-            extension_point = ExtensionPoints.get(type: extension_point_type)
-            project = setup_project(ctx, language, script_name, extension_point, description)
-            project_creator = Infrastructure::ProjectCreator
-              .for(ctx, language, extension_point, script_name, project.directory)
-            install_dependencies(ctx, language, script_name, project_creator)
-            bootstrap(ctx, project_creator)
-            project
+          def call(ctx:, language:, script_name:, extension_point_type:, no_config_ui:)
+            raise Infrastructure::Errors::ScriptProjectAlreadyExistsError, script_name if ctx.dir_exist?(script_name)
+
+            in_new_directory_context(ctx, script_name) do
+              extension_point = ExtensionPoints.get(type: extension_point_type)
+              project = Infrastructure::ScriptProjectRepository.new(ctx: ctx).create(
+                script_name: script_name,
+                extension_point_type: extension_point_type,
+                language: language,
+                no_config_ui: no_config_ui
+              )
+              project_creator = Infrastructure::ProjectCreator
+                .for(ctx, language, extension_point, script_name, project.id)
+              install_dependencies(ctx, language, script_name, project_creator)
+              bootstrap(ctx, project_creator)
+              project
+            end
           end
 
           private
 
-          def setup_project(ctx, language, script_name, extension_point, description)
-            ScriptProject.create(ctx, script_name)
-            ScriptProject.write(
-              ctx,
-              project_type: :script,
-              organization_id: nil, # TODO: can you provide this at creation
-              extension_point_type: extension_point.type,
-              script_name: script_name,
-              language: language,
-              description: description
-            )
-            ScriptProject.current
-          end
-
           def install_dependencies(ctx, language, script_name, project_creator)
             task_runner = Infrastructure::TaskRunner.for(ctx, language, script_name)
             project_creator.setup_dependencies
@@ -45,6 +40,21 @@ module Script
               spinner.update_title(ctx.message("script.create.created"))
             end
           end
+
+          def in_new_directory_context(ctx, directory)
+            initial_directory = ctx.root
+            begin
+              ctx.mkdir_p(directory)
+              ctx.chdir(directory)
+              yield
+            rescue
+              ctx.chdir(initial_directory)
+              ctx.rm_r(directory)
+              raise
+            ensure
+              ctx.chdir(initial_directory)
+            end
+          end
         end
       end
     end

data/lib/project_types/script/layers/application/extension_points.rb

@@ -12,9 +12,10 @@ module Script
           Infrastructure::ExtensionPointRepository.new.extension_point_types
         end
 
-        def self.non_deprecated_types
+        def self.available_types
           Infrastructure::ExtensionPointRepository.new.extension_points.select do |ep|
-            !ep.deprecated?
+            next false if ep.deprecated?
+            !ep.beta? || ShopifyCli::Feature.enabled?(:scripts_beta_extension_points)
           end.map(&:type)
         end
 

data/lib/project_types/script/layers/application/push_script.rb

@@ -6,8 +6,10 @@ module Script
       class PushScript
         class << self
           def call(ctx:, force:)
-            script_project = ScriptProject.current
+            script_project_repo = Infrastructure::ScriptProjectRepository.new(ctx: ctx)
+            script_project = script_project_repo.get
             task_runner = Infrastructure::TaskRunner.for(ctx, script_project.language, script_project.script_name)
+
             ProjectDependencies.install(ctx: ctx, task_runner: task_runner)
             BuildScript.call(ctx: ctx, task_runner: task_runner, script_project: script_project)
 
@@ -15,9 +17,10 @@ module Script
               package = Infrastructure::PushPackageRepository.new(ctx: p_ctx).get_push_package(
                 script_project: script_project,
                 compiled_type: task_runner.compiled_type,
-                metadata: task_runner.metadata
+                metadata: task_runner.metadata,
               )
-              package.push(Infrastructure::ScriptService.new(ctx: p_ctx), script_project.api_key, force)
+              uuid = package.push(Infrastructure::ScriptService.new(ctx: p_ctx), script_project.api_key, force)
+              script_project_repo.update_env(uuid: uuid)
               spinner.update_title(p_ctx.message("script.application.pushed"))
             end
           end

data/lib/project_types/script/layers/domain/config_ui.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Script
+  module Layers
+    module Domain
+      class ConfigUi
+        attr_reader :filename, :content
+
+        def initialize(filename:, content:)
+          @filename = filename
+          @content = content
+        end
+      end
+    end
+  end
+end

data/lib/project_types/script/layers/domain/errors.rb

@@ -14,6 +14,22 @@ module Script
           end
         end
 
+        class InvalidConfigUiDefinitionError < ScriptProjectError
+          attr_reader :filename
+          def initialize(filename)
+            super()
+            @filename = filename
+          end
+        end
+
+        class MissingSpecifiedConfigUiDefinitionError < ScriptProjectError
+          attr_reader :filename
+          def initialize(filename)
+            super()
+            @filename = filename
+          end
+        end
+
         class ScriptNotFoundError < ScriptProjectError
           attr_reader :script_name, :extension_point_type
           def initialize(extension_point_type, script_name)

data/lib/project_types/script/layers/domain/extension_point.rb

@@ -4,79 +4,94 @@ module Script
   module Layers
     module Domain
       class ExtensionPoint
-        attr_reader :type, :deprecated, :sdks
+        attr_reader :type, :beta, :deprecated, :sdks, :domain
 
         def initialize(type, config)
           @type = type
+          @beta = config["beta"] || false
           @deprecated = config["deprecated"] || false
+          @domain = config["domain"] || nil
           @sdks = ExtensionPointSDKs.new(config)
         end
 
+        def beta?
+          @beta
+        end
+
         def deprecated?
           @deprecated
         end
-      end
 
-      class ExtensionPointSDKs
-        def initialize(config)
-          @config = config
+        def dasherize_type
+          @type.gsub("_", "-")
         end
 
-        def all
-          [assemblyscript, rust].compact
-        end
+        class ExtensionPointSDKs
+          def initialize(config)
+            @config = config
+          end
 
-        def assemblyscript
-          @assemblyscript ||= new_sdk(ExtensionPointAssemblyScriptSDK)
-        end
+          def all
+            [assemblyscript, rust].compact
+          end
 
-        def rust
-          @rust ||= new_sdk(ExtensionPointRustSDK)
-        end
+          def assemblyscript
+            @assemblyscript ||= new_sdk(ExtensionPointAssemblyScriptSDK)
+          end
+
+          def rust
+            @rust ||= new_sdk(ExtensionPointRustSDK)
+          end
 
-        private
+          private
 
-        def new_sdk(klass)
-          config = @config[klass.language]
-          return nil if config.nil?
-          klass.new(config)
+          def new_sdk(klass)
+            config = @config[klass.language]
+            return nil if config.nil?
+            klass.new(config)
+          end
         end
-      end
 
-      class ExtensionPointSDK
-        attr_reader :beta, :package
+        class ExtensionPointSDK
+          attr_reader :version, :beta, :package
 
-        def initialize(config)
-          @beta = config["beta"] || false
-          @package = config["package"]
-        end
+          def initialize(config)
+            @beta = config["beta"] || false
+            @package = config["package"]
+            @version = config["package-version"]
+          end
 
-        def beta?
-          @beta
-        end
+          def beta?
+            @beta
+          end
+
+          def versioned?
+            @version
+          end
 
-        def self.language
-          raise NotImplementedError
+          def self.language
+            raise NotImplementedError
+          end
         end
-      end
 
-      class ExtensionPointAssemblyScriptSDK < ExtensionPointSDK
-        attr_reader :sdk_version, :toolchain_version
+        class ExtensionPointAssemblyScriptSDK < ExtensionPointSDK
+          attr_reader :sdk_version, :toolchain_version
 
-        def initialize(config)
-          super
-          @sdk_version = config["sdk-version"]
-          @toolchain_version = config["toolchain-version"]
-        end
+          def initialize(config)
+            super
+            @sdk_version = config["sdk-version"]
+            @toolchain_version = config["toolchain-version"]
+          end
 
-        def self.language
-          "assemblyscript"
+          def self.language
+            "assemblyscript"
+          end
         end
-      end
 
-      class ExtensionPointRustSDK < ExtensionPointSDK
-        def self.language
-          "rust"
+        class ExtensionPointRustSDK < ExtensionPointSDK
+          def self.language
+            "rust"
+          end
         end
       end
     end

data/lib/project_types/script/layers/domain/metadata.rb

@@ -14,39 +14,32 @@ module Script
 
         class << self
           def create_from_json(ctx, metadata_json)
+            err_tag = nil
             metadata_hash = JSON.parse(metadata_json)
-            schema_versions = metadata_hash["schemaVersions"]
-            if schema_versions.nil?
-              err_msg = "script.error.metadata_schema_versions_missing"
-              raise ::Script::Layers::Domain::Errors::MetadataValidationError, ctx.message(err_msg)
-            end
-            # Scripts may be attached to more than one EP in the future but not right now
-            unless schema_versions.count == 1
-              err_msg = "script.error.metadata_schema_versions_single_key"
-              raise ::Script::Layers::Domain::Errors::MetadataValidationError, ctx.message(err_msg)
-            end
 
-            _, version = schema_versions.first
+            use_msgpack = !!metadata_hash.dig("flags", "use_msgpack")
+            schema_versions = metadata_hash["schemaVersions"] || {}
+
+            version = schema_versions.values.first || {}
             schema_major_version = version["major"]
             schema_minor_version = version["minor"]
-            if schema_major_version.nil?
-              err_msg = "script.error.metadata_schema_versions_missing_major"
-              raise ::Script::Layers::Domain::Errors::MetadataValidationError, ctx.message(err_msg)
-            end
 
-            if schema_minor_version.nil?
-              err_msg = "script.error.metadata_schema_versions_missing_minor"
-              raise ::Script::Layers::Domain::Errors::MetadataValidationError, ctx.message(err_msg)
+            if schema_versions.empty?
+              err_tag = "script.error.metadata_schema_versions_missing"
+            elsif schema_versions.count != 1
+              # Scripts may be attached to more than one EP in the future but not right now
+              err_tag = "script.error.metadata_schema_versions_single_key"
+            elsif schema_major_version.nil?
+              err_tag = "script.error.metadata_schema_versions_missing_major"
+            elsif schema_minor_version.nil?
+              err_tag = "script.error.metadata_schema_versions_missing_minor"
             end
 
-            use_msgpack = !!metadata_hash.dig("flags", "use_msgpack")
-
             Metadata.new(schema_major_version, schema_minor_version, use_msgpack)
-          rescue ::Script::Layers::Domain::Errors::MetadataValidationError
-            raise
-          rescue
-            err_msg = "script.error.metadata_validation_cause"
-            raise ::Script::Layers::Domain::Errors::MetadataValidationError, ctx.message(err_msg)
+          rescue JSON::ParserError
+            err_tag = "script.error.metadata_validation_cause"
+          ensure
+            raise Errors::MetadataValidationError, ctx.message(err_tag) if err_tag
           end
         end
       end

data/lib/project_types/script/layers/domain/push_package.rb

@@ -5,9 +5,9 @@ module Script
     module Domain
       class PushPackage
         attr_reader :id,
+          :uuid,
           :extension_point_type,
           :script_name,
-          :description,
           :config_ui,
           :script_content,
           :compiled_type,
@@ -15,18 +15,18 @@ module Script
 
         def initialize(
           id:,
+          uuid:,
           extension_point_type:,
           script_name:,
-          description:,
           script_content:,
           compiled_type:,
           metadata:,
           config_ui:
         )
           @id = id
+          @uuid = uuid
           @extension_point_type = extension_point_type
           @script_name = script_name
-          @description = description
           @script_content = script_content
           @compiled_type = compiled_type
           @metadata = metadata
@@ -35,9 +35,9 @@ module Script
 
         def push(script_service, api_key, force)
           script_service.push(
+            uuid: @uuid,
             extension_point_type: @extension_point_type,
             script_name: @script_name,
-            description: @description,
             script_content: @script_content,
             compiled_type: @compiled_type,
             api_key: api_key,

data/lib/project_types/script/layers/domain/script_project.rb

@@ -0,0 +1,54 @@
+# frozen_string_literal: true
+
+module Script
+  module Layers
+    module Domain
+      class ScriptProject
+        include SmartProperties
+
+        UUID_ENV_KEY = "UUID"
+
+        property! :id, accepts: String
+        property :env, accepts: ShopifyCli::Resources::EnvFile
+
+        property! :extension_point_type, accepts: String
+        property! :script_name, accepts: String
+        property! :language, accepts: String
+
+        property :config_ui, accepts: ConfigUi
+
+        def initialize(*)
+          super
+
+          ShopifyCli::Core::Monorail.metadata = {
+            "script_name" => script_name,
+            "extension_point_type" => extension_point_type,
+            "language" => language,
+          }
+        end
+
+        def api_key
+          env&.api_key
+        end
+
+        def api_secret
+          env&.secret
+        end
+
+        def uuid
+          uuid_defined? && !raw_uuid.empty? ? raw_uuid : nil
+        end
+
+        def uuid_defined?
+          !raw_uuid.nil?
+        end
+
+        private
+
+        def raw_uuid
+          env&.extra&.[](UUID_ENV_KEY)
+        end
+      end
+    end
+  end
+end