shopify-cli 2.2.0 → 2.4.0

data/lib/project_types/extension/commands/build.rb

@@ -12,6 +12,34 @@ module Extension
       NPM_BUILD_COMMAND = %w(run-script build)
 
       def call(_args, _command_name)
+        project = ExtensionProject.current(force_reload: true)
+        return run_new_flow(project) if supports_development_server?(project.specification_identifier)
+        run_legacy_flow
+      end
+
+      def self.help
+        ShopifyCli::Context.new.message("build.help", ShopifyCli::TOOL_NAME)
+      end
+
+      private
+
+      def run_new_flow(project)
+        Tasks::RunExtensionCommand.new(
+          type: project.specification_identifier.downcase,
+          command: "build"
+        ).call
+
+        @ctx.puts(@ctx.message("build.build_success_message"))
+      rescue => error
+        if error.message.include?("no such file or directory")
+          @ctx.abort(@ctx.message("build.directory_not_found"))
+        else
+          @ctx.debug(error)
+          @ctx.abort(@ctx.message("build.build_failure_message"))
+        end
+      end
+
+      def run_legacy_flow
         system = ShopifyCli::JsSystem.new(ctx: @ctx)
 
         CLI::UI::Frame.open(@ctx.message("build.frame_title", system.package_manager)) do
@@ -20,8 +48,8 @@ module Extension
         end
       end
 
-      def self.help
-        ShopifyCli::Context.new.message("build.help", ShopifyCli::TOOL_NAME)
+      def supports_development_server?(type)
+        Models::DevelopmentServerRequirements.supported?(type)
       end
     end
   end

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

@@ -3,6 +3,10 @@
 module Extension
   class Command
     class Create < ShopifyCli::SubCommand
+      DEVELOPMENT_SERVER_SUPPORTED_TYPES = [
+        "checkout_ui_extension",
+      ]
+
       prerequisite_task :ensure_authenticated
 
       options do |parser, flags|
@@ -18,6 +22,8 @@ module Extension
             @ctx.abort(message_for_extension["create.errors.directory_exists", form.directory_name])
           end
 
+          return use_new_create_flow(form, message_for_extension) if supports_development_server?(form.type)
+
           if form.type.create(form.directory_name, @ctx, getting_started: options.flags[:getting_started])
             ExtensionProject.write_cli_file(context: @ctx, type: form.type.identifier)
             ExtensionProject.write_env_file(
@@ -47,6 +53,25 @@ module Extension
 
         yield form, form.type.method(:message_for_extension)
       end
+
+      def supports_development_server?(type)
+        Models::DevelopmentServerRequirements.supported?(type.identifier)
+      end
+
+      def use_new_create_flow(form, msg)
+        Tasks::RunExtensionCommand.new(
+          root_dir: form.directory_name,
+          template: form.template,
+          type: form.type.identifier.downcase,
+          command: "create"
+        ).call
+
+        @ctx.puts(msg["create.ready_to_start", form.directory_name, form.name])
+        @ctx.puts(msg["create.learn_more", form.type.name])
+      rescue => error
+        @ctx.debug(error)
+        @ctx.puts(msg["create.try_again"])
+      end
     end
   end
 end

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

@@ -11,7 +11,6 @@ module Extension
       def call(args, name)
         Command::Register.new(@ctx).call(args, name) unless project.registered?
         Command::Build.new(@ctx).call(args, name) unless specification_handler.specification.options[:skip_build]
-
         CLI::UI::Frame.open(@ctx.message("push.frame_title")) do
           updated_draft_version = update_draft
           show_message(updated_draft_version)

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

@@ -19,16 +19,6 @@ module Extension
       YARN_RUN_SCRIPT_NAME = %w(build).freeze
       private_constant :YARN_INSTALL_COMMAND, :YARN_INSTALL_PARAMETERS, :YARN_RUN_COMMAND, :YARN_RUN_SCRIPT_NAME
 
-      UI_EXTENSIONS_CHECKOUT = "@shopify/checkout-ui-extensions"
-      UI_EXTENSIONS_ADMIN = "@shopify/admin-ui-extensions"
-      UI_EXTENSIONS_POST_PURCHASE = "@shopify/post-purchase-ui-extensions"
-
-      PACKAGE_NAMES = [
-        UI_EXTENSIONS_CHECKOUT,
-        UI_EXTENSIONS_ADMIN,
-        UI_EXTENSIONS_POST_PURCHASE,
-      ].freeze
-
       def create(directory_name, identifier, context)
         Features::ArgoSetup.new(git_template: git_template).call(directory_name, identifier, context)
       end
@@ -54,7 +44,7 @@ module Extension
       def renderer_package(context)
         js_system = ShopifyCli::JsSystem.new(ctx: context)
         Tasks::FindNpmPackages
-          .exactly_one_of(*PACKAGE_NAMES, js_system: js_system)
+          .exactly_one_of(renderer_package_name, js_system: js_system)
           .unwrap { |err| raise err }
       rescue Extension::PackageResolutionFailed
         context.abort(

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

@@ -3,7 +3,7 @@
 module Extension
   module Forms
     class Create < ShopifyCli::Form
-      flag_arguments :name, :type, :api_key
+      flag_arguments :name, :type, :api_key, :template
 
       attr_reader :app
 
@@ -13,6 +13,7 @@ module Extension
         property :app, accepts: Models::App
         property :name, accepts: String
         property :type, accepts: Models::SpecificationHandlers::Default
+        property :template, accepts: String
 
         def complete?
           !!(app && name && type)
@@ -23,6 +24,7 @@ module Extension
         ShopifyCli::Result.wrap(ExtensionProjectDetails.new)
           .then(&Questions::AskApp.new(ctx: ctx, api_key: api_key))
           .then(&Questions::AskType.new(ctx: ctx, type: type))
+          .then(&Questions::AskTemplate.new(ctx: ctx))
           .then(&Questions::AskName.new(ctx: ctx, name: name))
           .unwrap { |e| raise e }
           .tap do |project_details|
@@ -30,6 +32,7 @@ module Extension
 
             self.app = project_details.app
             self.type = project_details.type
+            self.template = project_details.template
             self.name = project_details.name
           end
       end

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

@@ -0,0 +1,44 @@
+module Extension
+  module Forms
+    module Questions
+      class AskTemplate
+        include ShopifyCli::MethodObject
+
+        TEMPLATE_REQUIRED_TYPES = [
+          "checkout_ui_extension",
+        ]
+
+        property! :ctx
+        property :prompt,
+          accepts: ->(prompt) { prompt.respond_to?(:call) },
+          default: -> { CLI::UI::Prompt.method(:ask) }
+
+        def call(project_details)
+          return project_details unless template_required?(project_details)
+          project_details.template = choose_interactively
+          project_details
+        end
+
+        private
+
+        def template_required?(project_details)
+          return false unless extension_server_beta?
+          type = project_details&.type&.identifier
+          TEMPLATE_REQUIRED_TYPES.include?(type.downcase)
+        end
+
+        def extension_server_beta?
+          ShopifyCli::Shopifolk.check && ShopifyCli::Feature.enabled?(:extension_server_beta)
+        end
+
+        def choose_interactively
+          prompt.call(ctx.message("create.ask_template")) do |handler|
+            Models::ServerConfig::Development::VALID_TEMPLATES.each do |template|
+              handler.option(template) { template }
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -45,6 +45,7 @@ module Extension
           "or try creating a new app using {{command:shopify [node|rails] create}}.",
         loading_apps: "Loading your apps…",
         no_available_extensions: "{{x}} There are no available extensions for this app.",
+        ask_template: "Select a template to use for your extension",
       },
       connect: {
         connected: "Project now connected to {{green:%s: %s}}",
@@ -67,6 +68,8 @@ 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,

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

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Extension
+  module Models
+    class DevelopmentServer
+      class DevelopmentServerError < StandardError; end
+
+      include SmartProperties
+
+      EXECUTABLE_PATH = "../../../../../ext/shopify-cli/shopify-extensions/shopify-extensions"
+
+      property! :executable, converts: :to_s, default: File.expand_path(EXECUTABLE_PATH, __FILE__)
+
+      def create(server_config)
+        CLI::Kit::System.capture3(executable, "create", "-", stdin_data: server_config.to_yaml)
+      rescue StandardError => error
+        raise error
+      end
+
+      def build(server_config)
+        _, error, pid = CLI::Kit::System.capture3(executable, "build", "-", stdin_data: server_config.to_yaml)
+        return if pid.success?
+        raise DevelopmentServerError, error
+      end
+
+      def serve
+        raise NotImplementedError
+      end
+
+      def version
+        raise NotImplementedError
+      end
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Models
+    class DevelopmentServerRequirements
+      SUPPORTED_EXTENSION_TYPES = [
+        "checkout_ui_extension",
+      ]
+
+      def self.supported?(type)
+        return false unless SUPPORTED_EXTENSION_TYPES.include?(type.downcase)
+        ShopifyCli::Shopifolk.check && ShopifyCli::Feature.enabled?(:extension_server_beta)
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/base.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Extension
+  module Models
+    module ServerConfig
+      class Base
+        def to_h
+          to_hash
+        end
+
+        def to_hash
+          is_hashable = ->(obj) { obj.respond_to?(:to_hash) }
+          is_collection_of_hashables = ->(obj) { obj.is_a?(Enumerable) && obj.all?(&is_hashable) }
+
+          self.class.properties.each.reduce({}) do |data, (_, property)|
+            data.merge(property.name.to_s => send(property.reader).yield_self do |value|
+              case value
+              when is_collection_of_hashables
+                value.map { |element| element.to_hash.transform_keys(&:to_s) }
+              when is_hashable
+                value.to_hash.transform_keys(&:to_s)
+              else
+                value
+              end
+            end)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/development.rb

@@ -0,0 +1,23 @@
+module Extension
+  module Models
+    module ServerConfig
+      class Development < Base
+        include SmartProperties
+        VALID_TEMPLATES = [
+          "javascript",
+          "javascript-react",
+          "typescript",
+          "typescript-react",
+        ]
+
+        CURRENT_DIRECTORY = "."
+
+        property :root_dir, accepts: String, default: CURRENT_DIRECTORY
+        property! :build_dir, accepts: String, default: "build"
+        property :template, accepts: VALID_TEMPLATES
+        property :renderer, accepts: ServerConfig::DevelopmentRenderer
+        property :entries, accepts: ServerConfig::DevelopmentEntries
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/development_entries.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Extension
+  module Models
+    module ServerConfig
+      class DevelopmentEntries < Base
+        include SmartProperties
+
+        JAVASCRIPT = "javascript"
+        JAVASCRIPT_REACT = "javascript-react"
+        TYPESCRIPT = "typescript"
+        TYPESCRIPT_REACT = "typescript-react"
+
+        VALID_ENTRY_POINTS = [
+          "src/index.js",
+          "src/index.jsx",
+          "src/index.ts",
+          "src/index.tsx",
+        ]
+
+        property! :main, accepts: VALID_ENTRY_POINTS
+
+        def self.find(template)
+          case template
+          when JAVASCRIPT
+            new(main: "src/index.js")
+          when JAVASCRIPT_REACT
+            new(main: "src/index.jsx")
+          when TYPESCRIPT
+            new(main: "src/index.ts")
+          when TYPESCRIPT_REACT
+            new(main: "src/index.tsx")
+          end
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/development_renderer.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module Extension
+  module Models
+    module ServerConfig
+      class DevelopmentRenderer < Base
+        include SmartProperties
+
+        VALID_RENDERERS = [
+          "@shopify/admin-ui-extensions",
+          "@shopify/post-purchase-ui-extensions",
+          "@shopify/checkout-ui-extensions",
+        ]
+
+        property! :name, accepts: VALID_RENDERERS
+
+        def self.find(type)
+          case type.downcase
+          when "admin_ui_extension"
+            new(name: "@shopify/admin-ui-extensions")
+          when "checkout_ui_extension"
+            new(name: "@shopify/checkout-ui-extensions")
+          when "checkout_post_purchase"
+            new(name: "@shopify/post-purchase-ui-extensions")
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,35 @@
+require "securerandom"
+
+module Extension
+  module Models
+    module ServerConfig
+      class Extension < Base
+        include SmartProperties
+        property! :uuid, accepts: String
+        property! :type, accepts: String
+        property! :user, accepts: ServerConfig::User
+        property! :development, accepts: ServerConfig::Development
+
+        def self.build(uuid: "", template:, type:, root_dir:)
+          renderer = ServerConfig::DevelopmentRenderer.find(type)
+          entry = ServerConfig::DevelopmentEntries.find(template)
+          new(
+            uuid: uuid.empty? ? generate_dev_uuid : uuid,
+            type: type.downcase,
+            user: ServerConfig::User.new,
+            development: ServerConfig::Development.new(
+              root_dir: root_dir,
+              template: template,
+              renderer: renderer,
+              entries: entry
+            )
+          )
+        end
+
+        def self.generate_dev_uuid
+          "dev-#{SecureRandom.uuid}"
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/root.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Extension
+  module Models
+    module ServerConfig
+      class Root < Base
+        include SmartProperties
+
+        property! :port, accepts: Integer, default: 39351
+        property! :extensions, accepts: Array, default: -> { [] }
+
+        def to_yaml
+          to_h.to_yaml.gsub("---\n", "")
+        end
+      end
+    end
+  end
+end

data/lib/project_types/extension/models/server_config/user.rb

@@ -0,0 +1,10 @@
+module Extension
+  module Models
+    module ServerConfig
+      class User < Base
+        include SmartProperties
+        property! :metafields, accepts: Array, default: -> { [] }
+      end
+    end
+  end
+end

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

@@ -6,6 +6,7 @@ module Extension
     module SpecificationHandlers
       class CheckoutPostPurchase < Default
         PERMITTED_CONFIG_KEYS = [:metafields]
+        RENDERER_PACKAGE_NAME = "@shopify/post-purchase-ui-extensions"
 
         def config(context)
           {
@@ -13,6 +14,15 @@ module Extension
             **argo.config(context),
           }
         end
+
+        protected
+
+        def argo
+          Features::Argo.new(
+            git_template: specification.features.argo.git_template,
+            renderer_package_name: RENDERER_PACKAGE_NAME
+          )
+        end
       end
     end
   end

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

@@ -4,7 +4,7 @@ module Extension
   module Models
     module SpecificationHandlers
       class CheckoutUiExtension < Default
-        PERMITTED_CONFIG_KEYS = [:metafields, :extension_points]
+        PERMITTED_CONFIG_KEYS = [:extension_points, :metafields, :name]
 
         def config(context)
           {

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

@@ -9,7 +9,7 @@ module Extension
 
       property! :from
       property! :to, default: -> { from + 10 }
-      property! :host, default: "localhost"
+      property! :host, default: "127.0.0.1"
 
       def call
         available_port = port_range(from: from, to: to).find { |p| available?(host, p) }

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

@@ -0,0 +1,58 @@
+
+# frozen_string_literal: true
+require "shopify_cli"
+
+module Extension
+  module Tasks
+    class RunExtensionCommand < ShopifyCli::Task
+      include SmartProperties
+
+      SUPPORTED_EXTENSION_TYPES = [
+        "checkout_ui_extension",
+      ]
+
+      SUPPORTED_COMMANDS = [
+        "create",
+        "build",
+      ]
+
+      property :root_dir, accepts: String
+      property :template, accepts: Models::ServerConfig::Development::VALID_TEMPLATES
+      property! :type, accepts: SUPPORTED_EXTENSION_TYPES
+      property! :command, accepts: SUPPORTED_COMMANDS
+
+      def call
+        ShopifyCli::Result
+          .call(&method(:build_extension))
+          .then(&method(:build_server_config))
+          .then(&method(:run_command))
+          .unwrap do |error|
+            raise error unless error.nil?
+          end
+      end
+
+      private
+
+      def build_extension
+        Models::ServerConfig::Extension.build(
+          template: template,
+          type: type,
+          root_dir: root_dir,
+        )
+      end
+
+      def build_server_config(extension)
+        Models::ServerConfig::Root.new(extensions: [extension])
+      end
+
+      def run_command(server_config)
+        case command
+        when "create"
+          Models::DevelopmentServer.new.create(server_config)
+        when "build"
+          Models::DevelopmentServer.new.build(server_config)
+        end
+      end
+    end
+  end
+end

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

@@ -46,7 +46,7 @@ module Node
           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"])
 
         @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, "node"))
@@ -120,10 +120,6 @@ module Node
           @ctx.debug(e)
         end
       end
-
-      def local_debug?
-        @ctx.getenv(ShopifyCli::PartnersAPI::LOCAL_DEBUG)
-      end
     end
   end
 end

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

@@ -63,7 +63,7 @@ module Rails
           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"])
 
         @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, "rails"))
@@ -178,10 +178,6 @@ module Rails
       def install_gem(name, version = nil)
         Gem.install(@ctx, name, version)
       end
-
-      def local_debug?
-        @ctx.getenv(ShopifyCli::PartnersAPI::LOCAL_DEBUG)
-      end
     end
   end
 end

data/lib/project_types/script/cli.rb

@@ -45,12 +45,14 @@ module Script
     end
 
     module Infrastructure
+      autoload :ApiClients, Project.project_filepath("layers/infrastructure/api_clients")
       autoload :Errors, Project.project_filepath("layers/infrastructure/errors")
       autoload :CommandRunner, Project.project_filepath("layers/infrastructure/command_runner")
       autoload :PushPackageRepository, Project.project_filepath("layers/infrastructure/push_package_repository")
       autoload :ExtensionPointRepository, Project.project_filepath("layers/infrastructure/extension_point_repository")
       autoload :ScriptProjectRepository, Project.project_filepath("layers/infrastructure/script_project_repository")
       autoload :ScriptService, Project.project_filepath("layers/infrastructure/script_service")
+      autoload :ScriptUploader, Project.project_filepath("layers/infrastructure/script_uploader")
 
       module Languages
         autoload :AssemblyScriptProjectCreator,

data/lib/project_types/script/graphql/app_script_set.graphql

@@ -0,0 +1,40 @@
+mutation AppScriptSet(
+  $uuid: String
+  $extensionPointName: ExtensionPointName!,
+  $title: String!,
+  $description: String,
+  $force: Boolean,
+  $schemaMajorVersion: String,
+  $schemaMinorVersion: String,
+  $scriptJsonVersion: String!,
+  $configurationUi: Boolean!,
+  $configurationDefinition: String!,
+  $moduleUploadUrl: String!,
+) {
+  appScriptSet(
+    uuid: $uuid
+    extensionPointName: $extensionPointName
+    title: $title
+    description: $description
+    force: $force
+    schemaMajorVersion: $schemaMajorVersion
+    schemaMinorVersion: $schemaMinorVersion,
+    scriptJsonVersion: $scriptJsonVersion,
+    configurationUi: $configurationUi,
+    configurationDefinition: $configurationDefinition,
+    moduleUploadUrl: $moduleUploadUrl,
+) {
+    userErrors {
+      field
+      message
+      tag
+    }
+    appScript {
+      uuid
+      appKey
+      configSchema
+      extensionPointName
+      title
+    }
+  }
+}