disco_app 0.15.2 → 0.16.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ffe2c53f6a63376cf9484d15fbfb9030d7e4b5c07e0c38cccbc3f2ef095ac00f
-  data.tar.gz: 91019e6d795d0bd4b930badc400d8add493da10965312747db9b67ed58a8addc
+  metadata.gz: '058c0c520f0593e746cf7273793c2cefdd47af9e816224f8d3402c485e20a20e'
+  data.tar.gz: 9683bca6877f2b8c806a01024abab14fbedaed90d6c8a0505d6219011d9b4120
 SHA512:
-  metadata.gz: dba9193914aa136db5dd59dd9c25b49e70396967f0aa2f0cbe05682408864c93d6e57f6afb771ad5c19b3e00074efb4b1d9f6e76f3805aed7473a390993a2183
-  data.tar.gz: 3b171fa44ccab7a8af5470c50106ff1023370a29c95da68e9f2909287d551d5858ac840063daf9c091ed2184157eb25184278b4ac05916ffe887b1fe6df1b09d
+  metadata.gz: 82a23148e598236b391871d345deb085792afc6d3e422743410aca316f2d67a67e17a399649ee35d5ce5341ec3fa6d0f644e892592c6cdb29f4c49116bdf6e60
+  data.tar.gz: 670725a58544ac120136ab6cc67cf4c8b173a8d96e411041578f63c021b4480dbaf4e5979b901303634b5146aeea947a8dee22bb8b9b7c99682e2fb3cb98dace

data/app/clients/disco_app/graphql_client.rb

@@ -0,0 +1,85 @@
+require 'rest-client'
+
+##
+# This file defines a very simple GraphQL API client to support a single type
+# of GraphQL API call for a Shopify store - sending a Shopify Flow trigger.
+#
+# We use this simple approach rather than using an existing GraphQL client
+# library such as https://github.com/github/graphql-client (either standalone
+# or as bundled with the Shopify API gem) for a couple of reasons:
+#
+#   - These libraries tend to presume that a single client instance is
+#     instantiated once and then reused across the application, which isn't the
+#     case when we're making API calls once per trigger for each background
+#     job.
+#   - These libraries make an API call to fetch the Shopify GraphQL schema on
+#     initialisation. The schema is very large, so the API call takes a number
+#     of seconds to complete and when parsed consumes a large amount of memory.
+#   - These libraries do not natively work well with the idea of a dynamic API
+#     endpoint (ie, changing the request URL frequently), which is required
+#     when making many requests to different Shopify stores.
+#
+module DiscoApp
+  class GraphqlClient
+
+    def initialize(shop)
+      @shop = shop
+    end
+
+    ##
+    # Fire a Shopify Flow Trigger.
+    # Returns a tuple {Boolean, Array} representing {success, errors}.
+    def create_flow_trigger(title, resource_name, resource_url, properties)
+      body = {
+        trigger_title: title,
+        resources: [
+          {
+            name: resource_name,
+            url: resource_url
+          }
+        ],
+        properties: properties
+      }
+
+      # The double .to_json.to_json below looks odd but is required to properly escape the JSON hash
+      # when inserting it into the GraphQL mutation call.
+      response = execute(%Q(
+        mutation {
+          flowTriggerReceive(body: #{body.to_json.to_json}) {
+            userErrors {
+              field,
+              message
+            }
+          }
+        }
+      ))
+
+      errors = response.dig(:data, :flowTriggerReceive, :userErrors)
+      [errors.empty?, errors]
+    end
+
+    private
+
+      def execute(query)
+        response = RestClient::Request.execute(
+          method: :post,
+          headers: headers,
+          url: url,
+          payload: { query: query }.to_json
+        )
+        JSON.parse(response.body).with_indifferent_access
+      end
+
+      def headers
+        {
+          'Content-Type' => 'application/json',
+          'X-Shopify-Access-Token' => @shop.shopify_token
+        }
+      end
+
+      def url
+        "https://#{@shop.shopify_domain}/admin/api/graphql.json"
+      end
+
+  end
+end

data/app/controllers/disco_app/flow/actions_controller.rb

@@ -0,0 +1,7 @@
+module DiscoApp
+  module Flow
+    class ActionsController < ActionController::Base
+      include DiscoApp::Flow::Concerns::ActionsController
+    end
+  end
+end

data/app/controllers/disco_app/flow/concerns/actions_controller.rb

@@ -0,0 +1,47 @@
+module DiscoApp
+  module Flow
+    module Concerns
+      module ActionsController
+
+        extend ActiveSupport::Concern
+
+        included do
+          before_action :verify_flow_action
+          before_action :find_shop
+          protect_from_forgery with: :null_session
+        end
+
+        def create_flow_action
+          DiscoApp::Flow::CreateAction.call(
+            shop: @shop,
+            action_id: params[:id],
+            action_run_id: params[:action_run_id],
+            properties: params[:properties]
+          )
+
+          head :ok
+        end
+
+        private
+
+          def verify_flow_action
+            unless flow_action_is_valid?
+              head :unauthorized
+            end
+            request.body.rewind
+          end
+
+          # Shopify Flow action endpoints use the same verification method as webhooks, which is why we reuse this
+          # service method here.
+          def flow_action_is_valid?
+            DiscoApp::WebhookService.is_valid_hmac?(request.body.read.to_s, ShopifyApp.configuration.secret, request.headers['HTTP_X_SHOPIFY_HMAC_SHA256'])
+          end
+
+          def find_shop
+            @shop = DiscoApp::Shop.find_by_shopify_domain!(params[:shopify_domain])
+          end
+
+      end
+    end
+  end
+end

data/app/jobs/disco_app/flow/process_action_job.rb

@@ -0,0 +1,11 @@
+module DiscoApp
+  module Flow
+    class ProcessActionJob < DiscoApp::ShopJob
+
+      def perform(_shop, action)
+        ProcessAction.call(action: action)
+      end
+
+    end
+  end
+end

data/app/jobs/disco_app/flow/process_trigger_job.rb

@@ -0,0 +1,11 @@
+module DiscoApp
+  module Flow
+    class ProcessTriggerJob < DiscoApp::ShopJob
+
+      def perform(_shop, trigger)
+        ProcessTrigger.call(trigger: trigger)
+      end
+
+    end
+  end
+end

data/app/jobs/disco_app/shop_job.rb

@@ -1,10 +1,10 @@
+require 'appsignal'
+
 # The base class for all jobs that should be performed in the context of a
 # particular Shop's API session. The first argument to any job inheriting from
 # this class must be the domain of the relevant store, so that the appropriate
 # Shop model can be fetched and the temporary API session created.
 
-require 'rollbar'
-
 class DiscoApp::ShopJob < ApplicationJob
 
   queue_as :default
@@ -22,13 +22,12 @@ class DiscoApp::ShopJob < ApplicationJob
     end
 
     def shop_context(job, block)
-      Rollbar.scoped(rollbar_scope) do
-        @shop.with_api_context { block.call(job.arguments) }
-      end
-    end
-
-    def rollbar_scope
-      { person: { id: @shop.id, username: @shop.shopify_domain } }
+      Appsignal.tag_request(
+        shop_id: @shop.id,
+        shopify_domain: @shop.shopify_domain
+      )
+      
+      @shop.with_api_context { block.call(job.arguments) }
     end
 
 end

data/app/models/disco_app/concerns/renders_assets.rb

@@ -67,7 +67,7 @@ module DiscoApp::Concerns::RendersAssets
         assets: nil,
         triggered_by: nil,
         script_tags: nil,
-        minify: Rails.env.production?
+        minify: Rails.env.production? || Rails.env.staging?
       }
     end
 

data/app/models/disco_app/concerns/shop.rb

@@ -15,6 +15,10 @@ module DiscoApp::Concerns::Shop
     # Define relationship to sessions.
     has_many :sessions, class_name: 'DiscoApp::Session', dependent: :destroy
 
+    # Define relationship to Flow actions and triggers.
+    has_many :flow_actions, class_name: 'DiscoApp::Flow::Action', dependent: :destroy
+    has_many :flow_triggers, class_name: 'DiscoApp::Flow::Trigger', dependent: :destroy
+
     # Define possible installation statuses as an enum.
     enum status: {
       never_installed: 0,
@@ -73,11 +77,6 @@ module DiscoApp::Concerns::Shop
       "https://#{shopify_domain}/admin"
     end
 
-    # Convenience method to get the email of the shop's admin, to display in Rollbar.
-    def email
-      data[:email]
-    end
-
     def installed_duration
       distance_of_time_in_words_to_now(created_at.time)
     end

data/app/models/disco_app/flow/action.rb

@@ -0,0 +1,7 @@
+module DiscoApp
+  module Flow
+    class Action < ApplicationRecord
+      include DiscoApp::Flow::Concerns::Action
+    end
+  end
+end

data/app/models/disco_app/flow/concerns/action.rb

@@ -0,0 +1,24 @@
+module DiscoApp
+  module Flow
+    module Concerns
+      module Action
+        extend ActiveSupport::Concern
+
+        included do
+
+          belongs_to :shop
+
+          self.table_name = :disco_app_flow_actions
+
+          enum status: {
+            pending: 0,
+            succeeded: 1,
+            failed: 2
+          }
+
+        end
+
+      end
+    end
+  end
+end

data/app/models/disco_app/flow/concerns/trigger.rb

@@ -0,0 +1,24 @@
+module DiscoApp
+  module Flow
+    module Concerns
+      module Trigger
+        extend ActiveSupport::Concern
+
+        included do
+
+          belongs_to :shop
+
+          self.table_name = :disco_app_flow_triggers
+
+          enum status: {
+            pending: 0,
+            succeeded: 1,
+            failed: 2
+          }
+
+        end
+
+      end
+    end
+  end
+end

data/app/models/disco_app/flow/trigger.rb

@@ -0,0 +1,7 @@
+module DiscoApp
+  module Flow
+    class Trigger < ApplicationRecord
+      include DiscoApp::Flow::Concerns::Trigger
+    end
+  end
+end

data/app/services/disco_app/flow/create_action.rb

@@ -0,0 +1,35 @@
+require 'interactor'
+
+module DiscoApp
+  module Flow
+    class CreateAction
+
+      include Interactor
+
+      delegate :shop, :action_id, :action_run_id, :properties, to: :context
+      delegate :action, to: :context
+
+      def call
+        create_action
+        enqueue_process_action_job
+      end
+
+      private
+
+        def create_action
+          context.action = shop.flow_actions.create!(
+            action_id: action_id,
+            action_run_id: action_run_id,
+            properties: properties
+          )
+        rescue ActiveRecord::RecordNotUnique, PG::UniqueViolation
+          context.fail!
+        end
+
+        def enqueue_process_action_job
+          ProcessActionJob.perform_later(shop, action)
+        end
+
+    end
+  end
+end

data/app/services/disco_app/flow/create_trigger.rb

@@ -0,0 +1,34 @@
+require 'interactor'
+
+module DiscoApp
+  module Flow
+    class CreateTrigger
+
+      include Interactor
+
+      delegate :shop, :title, :resource_name, :resource_url, :properties, to: :context
+      delegate :trigger, to: :context
+
+      def call
+        create_trigger
+        enqueue_process_trigger_job
+      end
+
+      private
+
+        def create_trigger
+          context.trigger = shop.flow_triggers.create!(
+            title: title,
+            resource_name: resource_name,
+            resource_url: resource_url,
+            properties: properties
+          )
+        end
+
+        def enqueue_process_trigger_job
+          ProcessTriggerJob.perform_later(shop, trigger)
+        end
+
+    end
+  end
+end

data/app/services/disco_app/flow/process_action.rb

@@ -0,0 +1,50 @@
+require 'interactor'
+
+module DiscoApp
+  module Flow
+    class ProcessAction
+
+      include Interactor
+
+      delegate :action, to: :context
+      delegate :action_service_class, to: :context
+
+      def call
+        validate_action
+        find_action_service_class
+        execute_action_service_class
+      end
+
+      private
+
+        def validate_action
+          context.fail! unless action.pending?
+        end
+
+        def find_action_service_class
+          context.action_service_class =
+            action.action_id.classify.safe_constantize ||
+            %Q(Flow::Actions::#{("#{action.action_id}".classify)}).safe_constantize
+
+          if action_service_class.nil?
+            update_action(false, ["Could not find service class for #{action.action_id}"])
+            context.fail!
+          end
+        end
+
+        def execute_action_service_class
+          result = action_service_class.call(shop: action.shop, properties: action.properties)
+          update_action(result.success?, result.errors)
+        end
+
+        def update_action(success, errors)
+          action.update!(
+            status: success ? Action.statuses[:succeeded] : Action.statuses[:failed],
+            processing_errors: success ? [] : errors,
+            processed_at: Time.current
+          )
+        end
+
+    end
+  end
+end

data/app/services/disco_app/flow/process_trigger.rb

@@ -0,0 +1,52 @@
+require 'interactor'
+
+module DiscoApp
+  module Flow
+    class ProcessTrigger
+
+      include Interactor
+
+      delegate :trigger, to: :context
+      delegate :api_success, :api_errors, to: :context
+
+      def call
+        validate_trigger
+        make_api_request
+        update_trigger
+        fail_if_errors_present
+      end
+
+      private
+
+        def validate_trigger
+          context.fail! unless trigger.pending?
+        end
+
+        def make_api_request
+          context.api_success, context.api_errors = api_client.create_flow_trigger(
+            trigger.title,
+            trigger.resource_name,
+            trigger.resource_url,
+            trigger.properties
+          )
+        end
+
+        def update_trigger
+          trigger.update!(
+            status: api_success ? Trigger.statuses[:succeeded] : Trigger.statuses[:failed],
+            processing_errors: api_success ? [] : api_errors,
+            processed_at: Time.current
+          )
+        end
+
+        def fail_if_errors_present
+          context.fail! unless api_success
+        end
+
+        def api_client
+          @api_client ||= DiscoApp::GraphqlClient.new(trigger.shop)
+        end
+
+    end
+  end
+end

data/config/routes.rb

@@ -9,6 +9,12 @@ DiscoApp::Engine.routes.draw do
     post 'webhooks' => :process_webhook, as: :webhooks
   end
 
+  namespace :flow do
+    controller :actions do
+      post 'actions/:id' => :create_flow_action, as: :flow_actions
+    end
+  end
+
   resources :user_sessions, only: [:new, :create, :destroy]
   get 'auth/shopify_user/callback' => 'user_sessions#callback'
 
@@ -46,7 +52,7 @@ DiscoApp::Engine.routes.draw do
   end
 
   # Make the Sidekiq Web UI accessible using the same credentials as the admin.
-  if Rails.env.production?
+  if Rails.env.production? || Rails.env.staging?
     Sidekiq::Web.use Rack::Auth::Basic do |username, password|
       [
         ENV['ADMIN_APP_USERNAME'].present?,