grape-slack-bot 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5a6e6b020caecc8ee5a8520f7f6f76107d006ca88070189e0cf23428bc10d2e7
-  data.tar.gz: 02aa101bb61f5ad8fc96c9d295e75fe032e1d088d90d3d49b008e09551f53b8c
+  metadata.gz: 4d79cd991f4efaa9ae3319b25d73e0addde71aa04d59369236225f1bb97e0b4e
+  data.tar.gz: 402ede62474860b3009c44107f7e54fba9de531c992315eba021bc84a3a2c21e
 SHA512:
-  metadata.gz: 3217a763efe6f6b101067c422029450e59efee9ea47d611bef1bd06a166a99404c1243955a3e42a5e9f31a78e9c3cbdd051d324fcd3b0b2414812da3d84f7336
-  data.tar.gz: 3faca964608cb59b26ab6712c4eec82356f0ce761adfb4811d282e7d0051f9cdc58ac6958634feef2e24563b800a38259756a5847e0c6b0d8a3b2b6c62cb74b5
+  metadata.gz: c1bf71eedfc65e15481d6f06ed038bfd637f1ef0784c8040a482386f20087c98c53c435bd43f3b62eed9ac82482c36a35c70f87920e40372eefca45d2f783243
+  data.tar.gz: daaded041bc84bac15c0c20bc04709df914ab895b222244d05d1aef9f568e7b63ed1ce6475a0a027a8601bcf597a91cb1a97ef1a7e840456f49c3e5104e4549c

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # CHANGELOG
 
+## 2.0.1 (2026-05-31)
+
+- Allow `grape` 3.x by expanding the runtime dependency to `< 4.0`
+- Replace SimpleCov and JUnit formatter test wiring with Polyrun coverage and failure reporting
+- Move lint and type-check execution into Polyrun `before_suite` hooks
+- Refresh development and release tooling, CI, and repository maintenance configuration
+
 ## 2.0.0 (2025-11-06)
 
 - Fix status code handling for empty/false responses - ensure 200 OK instead of 204 No Content

data/README.md

@@ -1,6 +1,6 @@
 # grape-slack-bot.rb
 
-[![Gem Version](https://badge.fury.io/rb/grape-slack-bot.svg)](https://badge.fury.io/rb/grape-slack-bot) [![Test Status](https://github.com/amkisko/grape-slack-bot.rb/actions/workflows/test.yml/badge.svg)](https://github.com/amkisko/grape-slack-bot.rb/actions/workflows/test.yml) [![codecov](https://codecov.io/gh/amkisko/grape-slack-bot.rb/graph/badge.svg?token=VIZ94XFOR3)](https://codecov.io/gh/amkisko/grape-slack-bot.rb)
+[![Gem Version](https://badge.fury.io/rb/grape-slack-bot.svg)](https://badge.fury.io/rb/grape-slack-bot) [![Test Status](https://github.com/amkisko/grape-slack-bot.rb/actions/workflows/test.yml/badge.svg)](https://github.com/amkisko/grape-slack-bot.rb/actions/workflows/test.yml)
 
 Extensible Slack bot implementation gem for [ruby-grape](https://github.com/ruby-grape/grape) with support for slash commands, interactive components, events, and views.
 
@@ -509,11 +509,16 @@ The gem implements Slack's signature verification with the following security fe
 
 ## Development
 
-```bash
+```sh
 bundle install
-bundle exec rspec
-bundle exec rbs validate
-bundle exec standardrb --fix
+bundle exec polyrun hook run before_suite
+bundle exec polyrun parallel-rspec --workers 1 --merge-failures
+```
+
+To generate a local coverage report with Polyrun:
+
+```sh
+POLYRUN_COVERAGE=1 bundle exec polyrun parallel-rspec --workers 1 --merge-failures
 ```
 
 For development and testing purposes you can use [Cloudflare Argo Tunnel](https://developers.cloudflare.com/cloudflare-one/connections/connect-apps) to expose your local development environment to the internet.
@@ -528,7 +533,7 @@ For easiness of getting information, most of endpoints have `SlackBot::DevConsol
 
 ### Code Quality
 
-The gem uses [StandardRB](https://github.com/standardrb/standard) for consistent code style. Run `bundle exec standardrb --fix` to automatically fix style issues.
+The gem uses a StandardRB-compatible RuboCop configuration for consistent code style, with Polyrun orchestrating the lint and type-check hooks. Run `bundle exec polyrun hook run before_suite` to execute the configured checks.
 
 The gem includes [RBS](https://github.com/ruby/rbs) type signatures in the `sig/` directory for better type checking and IDE support. Type signatures are included in the gem package.
 
@@ -557,7 +562,7 @@ gem push grape-slack-bot-*.gem
 Or use the release script:
 
 ```sh
-usr/bin/release.sh
+usr/bin/release.rb
 ```
 
 ## License

data/grape-slack-bot.gemspec

@@ -30,19 +30,22 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
 
   gem.add_runtime_dependency "rack", "~> 3.0"
-  gem.add_runtime_dependency "grape", ">= 1.6", "< 3.0"
+  gem.add_runtime_dependency "grape", ">= 1.6", "< 4.0"
   gem.add_runtime_dependency "faraday", "~> 2.0"
   gem.add_runtime_dependency "activesupport", ">= 6.1", "< 9.0"
 
-  gem.add_development_dependency "rspec", "~> 3.12"
+  gem.add_development_dependency "rspec", "~> 3"
+  gem.add_development_dependency "polyrun", "~> 1.4.2"
   gem.add_development_dependency "webmock", "~> 3"
-  gem.add_development_dependency "rake", "~> 13.0"
-  gem.add_development_dependency "simplecov", "~> 0.21"
-  gem.add_development_dependency "rspec_junit_formatter", "~> 0.6"
-  gem.add_development_dependency "simplecov-cobertura", "~> 3"
-  gem.add_development_dependency "standard", "~> 1.0"
-  gem.add_development_dependency "appraisal", "~> 2.4"
-  gem.add_development_dependency "memory_profiler", "~> 1.0"
-  gem.add_development_dependency "rbs", "~> 3.0"
-  gem.add_development_dependency "rack-test", "~> 2.0"
+  gem.add_development_dependency "rake", "~> 13"
+  gem.add_development_dependency "standard", "~> 1.52"
+  gem.add_development_dependency "standard-custom", "~> 1.0"
+  gem.add_development_dependency "standard-performance", "~> 1.8"
+  gem.add_development_dependency "standard-rspec", "~> 0.3"
+  gem.add_development_dependency "rubocop-rspec", "~> 3.8"
+  gem.add_development_dependency "rubocop-thread_safety", "~> 0.7"
+  gem.add_development_dependency "appraisal", "~> 2"
+  gem.add_development_dependency "memory_profiler", "~> 1"
+  gem.add_development_dependency "rbs", "~> 3"
+  gem.add_development_dependency "rack-test", "~> 2"
 end

data/lib/slack_bot/api_client.rb

@@ -47,32 +47,13 @@ module SlackBot
     REQUEST_TIMEOUT = 30
     # Connection timeout in seconds
     CONNECTION_TIMEOUT = 10
+    TOKEN_FORMAT = /\Axox[bpa]-/
 
     attr_reader :client
-    def initialize(authorization_token: ENV["SLACK_BOT_API_TOKEN"])
-      authorization_token_available = !authorization_token.nil? && authorization_token.is_a?(String) && !authorization_token.empty?
-      unless authorization_token_available
-        raise SlackBot::Errors::SlackApiError.new("Slack bot API token is not set")
-      end
-
-      # Validate token format
-      # Bot tokens: xoxb-, User tokens: xoxp-, App tokens: xoxa-
-      # For this gem, we primarily expect bot tokens (xoxb-)
-      # Allow test tokens (starting with "test_") for testing purposes
-      unless authorization_token.start_with?("test_") || authorization_token.match?(/\Axox[bpa]-/)
-        raise SlackBot::Errors::SlackApiError.new("Invalid Slack API token format")
-      end
 
-      @client =
-        Faraday.new do |conn|
-          conn.request :url_encoded
-          conn.adapter Faraday.default_adapter
-          conn.url_prefix = SLACK_API_BASE_URL
-          conn.headers["Content-Type"] = "application/json; charset=utf-8"
-          conn.headers["Authorization"] = "Bearer #{authorization_token}"
-          conn.options.timeout = REQUEST_TIMEOUT
-          conn.options.open_timeout = CONNECTION_TIMEOUT
-        end
+    def initialize(authorization_token: ENV["SLACK_BOT_API_TOKEN"])
+      validate_authorization_token!(authorization_token)
+      @client = build_client(authorization_token)
     end
 
     def views_open(trigger_id:, view:)
@@ -189,11 +170,7 @@ module SlackBot
     end
 
     def users_list(cursor: nil, limit: 200, include_locale: nil, team_id: nil)
-      args = {}
-      args[:cursor] = cursor if cursor
-      args[:limit] = limit if limit
-      args[:include_locale] = include_locale if include_locale
-      args[:team_id] = team_id if team_id
+      args = compact_payload(cursor: cursor, limit: limit, include_locale: include_locale, team_id: team_id)
       ApiResponse.new do
         client.post("users.list", args.to_json)
       rescue Faraday::Error => e
@@ -202,24 +179,54 @@ module SlackBot
     end
 
     def chat_post_ephemeral(channel:, user:, text:, as_user: nil, attachments: nil, blocks: nil, icon_emoji: nil, icon_url: nil, link_names: nil, parse: nil, thread_ts: nil, username: nil)
-      args = {}
-      args[:channel] = channel
-      args[:user] = user
-      args[:text] = text if text
-      args[:as_user] = as_user if as_user
-      args[:attachments] = attachments if attachments
-      args[:blocks] = blocks if blocks
-      args[:icon_emoji] = icon_emoji if icon_emoji
-      args[:icon_url] = icon_url if icon_url
-      args[:link_names] = link_names if link_names
-      args[:parse] = parse if parse
-      args[:thread_ts] = thread_ts if thread_ts
-      args[:username] = username if username
+      args = compact_payload(
+        channel: channel,
+        user: user,
+        text: text,
+        as_user: as_user,
+        attachments: attachments,
+        blocks: blocks,
+        icon_emoji: icon_emoji,
+        icon_url: icon_url,
+        link_names: link_names,
+        parse: parse,
+        thread_ts: thread_ts,
+        username: username
+      )
       ApiResponse.new do
         client.post("chat.postEphemeral", args.to_json)
       rescue Faraday::Error => e
         raise SlackBot::Errors::SlackApiError.new("Network error: #{e.message}")
       end
     end
+
+    private
+
+    def validate_authorization_token!(authorization_token)
+      raise SlackBot::Errors::SlackApiError.new("Slack bot API token is not set") unless valid_authorization_token?(authorization_token)
+      return if authorization_token.start_with?("test_") || authorization_token.match?(TOKEN_FORMAT)
+
+      raise SlackBot::Errors::SlackApiError.new("Invalid Slack API token format")
+    end
+
+    def valid_authorization_token?(authorization_token)
+      authorization_token.is_a?(String) && !authorization_token.empty?
+    end
+
+    def build_client(authorization_token)
+      Faraday.new do |conn|
+        conn.request :url_encoded
+        conn.adapter Faraday.default_adapter
+        conn.url_prefix = SLACK_API_BASE_URL
+        conn.headers["Content-Type"] = "application/json; charset=utf-8"
+        conn.headers["Authorization"] = "Bearer #{authorization_token}"
+        conn.options.timeout = REQUEST_TIMEOUT
+        conn.options.open_timeout = CONNECTION_TIMEOUT
+      end
+    end
+
+    def compact_payload(**payload)
+      payload.compact
+    end
   end
 end

data/lib/slack_bot/grape_extension.rb

@@ -19,49 +19,12 @@ module SlackBot
 
     def verify_slack_signature!
       slack_signing_secret = ENV["SLACK_SIGNING_SECRET"]
-      timestamp = request.headers["x-slack-request-timestamp"] || request.headers["X-Slack-Request-Timestamp"]
-      slack_signature = request.headers["x-slack-signature"] || request.headers["X-Slack-Signature"]
-      if slack_signing_secret.blank? || timestamp.blank? || slack_signature.blank?
-        raise SlackBot::Errors::SignatureAuthenticationError.new("Missing signature headers")
-      end
-
-      # Validate signing secret format (allow test secrets for testing)
-      unless slack_signing_secret.start_with?("test_") || slack_signing_secret.length >= MIN_SIGNING_SECRET_LENGTH
-        raise SlackBot::Errors::SignatureAuthenticationError.new("Invalid signing secret format")
-      end
+      timestamp = slack_request_header("x-slack-request-timestamp", "X-Slack-Request-Timestamp")
+      slack_signature = slack_request_header("x-slack-signature", "X-Slack-Signature")
 
-      # Validate timestamp to prevent replay attacks (Slack recommends 5 minutes)
-      request_timestamp = timestamp.to_i
-      current_timestamp = Time.now.to_i
-      if (current_timestamp - request_timestamp).abs > TIMESTAMP_TOLERANCE_SECONDS
-        raise SlackBot::Errors::SignatureAuthenticationError.new("Request timestamp too old")
-      end
-
-      request_body = if request.body
-        request.body.rewind if request.body.respond_to?(:rewind)
-        body_content = request.body.read
-        request.body.rewind if request.body.respond_to?(:rewind)
-        body_content
-      else
-        ""
-      end
-
-      sig_basestring = "v0:#{timestamp}:#{request_body}"
-      my_signature =
-        "v0=" +
-        OpenSSL::HMAC.hexdigest(
-          OpenSSL::Digest.new("sha256"),
-          slack_signing_secret,
-          sig_basestring
-        )
-      if ActiveSupport::SecurityUtils.secure_compare(
-        my_signature,
-        slack_signature
-      )
-        true
-      else
-        raise SlackBot::Errors::SignatureAuthenticationError.new("Signature mismatch")
-      end
+      validate_signature_headers!(slack_signing_secret, timestamp, slack_signature)
+      validate_request_timestamp!(timestamp)
+      verify_signature_match!(slack_signing_secret, timestamp, slack_signature)
     end
 
     def verify_slack_team!
@@ -112,41 +75,117 @@ module SlackBot
     end
 
     def handle_block_actions_view(view:, user:, params:)
-      callback_id = view&.dig("callback_id")
+      callback = find_callback!(view: view, user: user)
+      log_callback_check(callback, user)
+      validate_callback_user!(callback, user)
+
+      interaction_klass = callback_interaction_klass(callback)
+      return false if interaction_klass.blank?
+
+      interaction_klass.new(current_user: user, params: params, callback: callback, config: config).call
+    end
+
+    private
+
+    def slack_request_header(*names)
+      names.each do |name|
+        header = request.headers[name]
+        return header if header
+      end
+
+      nil
+    end
+
+    def validate_signature_headers!(slack_signing_secret, timestamp, slack_signature)
+      raise SlackBot::Errors::SignatureAuthenticationError.new("Missing signature headers") if slack_signing_secret.blank? || timestamp.blank? || slack_signature.blank?
+      return if slack_signing_secret.start_with?("test_") || slack_signing_secret.length >= MIN_SIGNING_SECRET_LENGTH
+
+      raise SlackBot::Errors::SignatureAuthenticationError.new("Invalid signing secret format")
+    end
+
+    def validate_request_timestamp!(timestamp)
+      request_timestamp = timestamp.to_i
+      current_timestamp = Time.now.to_i
+      return if (current_timestamp - request_timestamp).abs <= TIMESTAMP_TOLERANCE_SECONDS
+
+      raise SlackBot::Errors::SignatureAuthenticationError.new("Request timestamp too old")
+    end
+
+    def verify_signature_match!(slack_signing_secret, timestamp, slack_signature)
+      return true if ActiveSupport::SecurityUtils.secure_compare(computed_signature(slack_signing_secret, timestamp), slack_signature)
 
-      callback = SlackBot::Callback.find(callback_id, user: user, config: config)
+      raise SlackBot::Errors::SignatureAuthenticationError.new("Signature mismatch")
+    end
+
+    def computed_signature(slack_signing_secret, timestamp)
+      sig_basestring = "v0:#{timestamp}:#{request_body_content}"
+      "v0=" + OpenSSL::HMAC.hexdigest(OpenSSL::Digest.new("sha256"), slack_signing_secret, sig_basestring)
+    end
+
+    def request_body_content
+      return "" unless request.body
+
+      request.body.rewind if request.body.respond_to?(:rewind)
+      body_content = request.body.read
+      request.body.rewind if request.body.respond_to?(:rewind)
+      body_content
+    end
+
+    def find_callback!(view:, user:)
+      callback = SlackBot::Callback.find(view&.dig("callback_id"), user: user, config: config)
       raise SlackBot::Errors::CallbackNotFound.new if callback.blank?
 
-      SlackBot::DevConsole.log_check "SlackApi::Interactions##{__method__}: #{callback.id} #{callback.payload} #{callback.user_id} #{user&.id}"
+      callback
+    end
 
-      validate_callback_user!(callback, user)
+    def log_callback_check(callback, user)
+      SlackBot::DevConsole.log_check "SlackApi::Interactions##{__method__}: #{callback.id} #{callback.payload} #{callback.user_id} #{user&.id}"
+    end
 
+    def callback_interaction_klass(callback)
       handler_class_obj = callback.handler_class
-      interaction_klass = handler_class_obj&.interaction_klass if handler_class_obj&.respond_to?(:interaction_klass)
-      return false if interaction_klass.blank?
+      handler_class_obj&.interaction_klass if handler_class_obj&.respond_to?(:interaction_klass)
+    end
 
-      interaction_klass.new(current_user: user, params: params, callback: callback, config: config).call
+    def parse_interaction_payload!(raw_payload)
+      JSON.parse(raw_payload)
+    rescue JSON::ParserError => e
+      raise SlackBot::Errors::InvalidPayloadError.new("Invalid JSON payload: #{e.message}")
+    end
+
+    def resolve_action_user(payload)
+      resolve_user_session(payload.dig("user", "team_id"), payload.dig("user", "id"))
+    end
+
+    def blank_slack_response!
+      body false
+      status 200
     end
   end
 
   module GrapeExtension
     def self.included(base)
+      configure_base!(base)
+      add_commands_resource!(base)
+      add_interactions_resource!(base)
+      add_events_resource!(base)
+      add_menu_options_resource!(base)
+    end
+
+    def self.configure_base!(base)
       base.format :json
       base.content_type :json, "application/json"
       base.use ActionDispatch::RemoteIp if defined?(ActionDispatch::RemoteIp)
       base.helpers SlackBot::GrapeHelpers
-
-      # Handle custom errors
-      # Slack API requires 200 OK responses to avoid retries
-      # Errors should be returned as 200 OK with error information in the response body
       base.rescue_from SlackBot::Error do |e|
         error!({error: e.message}, 200)
       end
-
       base.before do
         verify_slack_signature!
       end
+    end
 
+    def self.add_commands_resource!(base)
       base.resource :commands do
         post ":url_token" do
           command_config = config.find_slash_command_config(params[:url_token], params[:command], params[:text])
@@ -156,101 +195,68 @@ module SlackBot
           args = params[:text].gsub(/^#{command_config.full_token}\s?/, "")
           SlackBot::DevConsole.log_input "SlackApi::SlashCommands#post: #{command_config.url_token} | #{command_config.full_token} | #{args}"
 
-          action =
-            command_klass.new(
-              current_user: current_user,
-              params: params,
-              args: args,
-              config: config
-            )
+          action = command_klass.new(current_user: current_user, params: params, args: args, config: config)
           verify_slack_team! if action.only_slack_team?
           verify_direct_message_channel! if action.only_direct_message?
           verify_current_user! if action.only_user?
 
           result = action.call
-          if !result
-            body false
-            status 200
-            return
-          end
+          return blank_slack_response! unless result
 
           result
         end
       end
+    end
 
+    def self.add_interactions_resource!(base)
       base.resource :interactions do
         post do
-          begin
-            payload = JSON.parse(params[:payload])
-          rescue JSON::ParserError => e
-            raise SlackBot::Errors::InvalidPayloadError.new("Invalid JSON payload: #{e.message}")
-          end
-
-          action_user_session =
-            resolve_user_session(
-              payload.dig("user", "team_id"),
-              payload.dig("user", "id")
-            )
-          action_user = action_user_session&.user
+          payload = parse_interaction_payload!(params[:payload])
+          action_user = resolve_action_user(payload)&.user
 
-          action_type = payload["type"]
-          result = case action_type
+          result = case payload["type"]
           when "block_actions", "view_submission"
-            handle_block_actions_view(
-              view: payload["view"],
-              user: action_user,
-              params: params
-            )
+            handle_block_actions_view(view: payload["view"], user: action_user, params: params)
           else
-            raise SlackBot::Errors::UnknownActionTypeError.new(action_type)
+            raise SlackBot::Errors::UnknownActionTypeError.new(payload["type"])
           end
 
-          if result.blank? || result == false
-            body false
-            status 200
-            return
-          end
+          return blank_slack_response! if result.blank? || result == false
 
           result
         end
       end
+    end
 
+    def self.add_events_resource!(base)
       base.resource :events do
         post do
-          result =
-            case params[:type]
-            when "url_verification"
-              url_verification(params)
-            when "event_callback"
-              events_callback(params)
-            else
-              raise SlackBot::Errors::UnknownActionTypeError.new(params[:type])
-            end
-
-          if result.blank? || result == false
-            body false
-            status 200
-            return
+          result = case params[:type]
+          when "url_verification"
+            url_verification(params)
+          when "event_callback"
+            events_callback(params)
+          else
+            raise SlackBot::Errors::UnknownActionTypeError.new(params[:type])
           end
 
+          return blank_slack_response! if result.blank? || result == false
+
           result
         end
       end
+    end
 
+    def self.add_menu_options_resource!(base)
       base.resource :menu_options do
         get do
           SlackBot::DevConsole.log_input "SlackApi::MenuOptions#get: #{params.inspect}"
 
-          action_id = params[:action_id]
-          menu_options_klass = config.find_menu_options(action_id)
+          menu_options_klass = config.find_menu_options(params[:action_id])
           raise SlackBot::Errors::MenuOptionsNotImplemented.new if menu_options_klass.blank?
 
           menu_options = menu_options_klass.new(current_user: current_user, params: params, config: config).call
-          if menu_options.blank?
-            body false
-            status 200
-            return
-          end
+          return blank_slack_response! if menu_options.blank?
 
           menu_options
         end

data/lib/slack_bot/interaction.rb

@@ -10,55 +10,20 @@ module SlackBot
     include SlackBot::Concerns::ViewKlass
 
     def self.open_modal(callback:, trigger_id:, view:)
-      view = view.merge({type: "modal", callback_id: callback&.id})
-      response =
-        SlackBot::ApiClient.new.views_open(trigger_id: trigger_id, view: view)
-
-      if !response.ok?
-        raise SlackBot::Errors::OpenModalError.new(response.error, data: response.data, payload: view)
-      end
-
-      view_id = response.data.dig("view", "id")
-      if callback.present? && view_id.present?
-        callback.view_id = view_id
-        callback.save
-      end
-      SlackViewsReply.new(callback&.id, view_id)
+      view = modal_payload(callback, view)
+      response = SlackBot::ApiClient.new.views_open(trigger_id: trigger_id, view: view)
+      build_view_reply(response: response, callback: callback, payload: view, error_class: SlackBot::Errors::OpenModalError)
     end
 
     def self.update_modal(callback:, view_id:, view:)
-      view = view.merge({type: "modal", callback_id: callback&.id})
-      response =
-        SlackBot::ApiClient.new.views_update(view_id: view_id, view: view)
-
-      if !response.ok?
-        raise SlackBot::Errors::UpdateModalError.new(response.error, data: response.data, payload: view)
-      end
-
-      view_id = response.data.dig("view", "id")
-      if callback.present? && view_id.present?
-        callback.view_id = view_id
-        callback.save
-      end
-      SlackViewsReply.new(callback&.id, view_id)
+      view = modal_payload(callback, view)
+      response = SlackBot::ApiClient.new.views_update(view_id: view_id, view: view)
+      build_view_reply(response: response, callback: callback, payload: view, error_class: SlackBot::Errors::UpdateModalError)
     end
 
     def self.publish_view(user_id:, view:, callback: nil, metadata: nil)
-      view = view.merge(callback_id: callback.id) if callback.present?
-      view = view.merge(private_metadata: metadata) if metadata.present?
-      response =
-        SlackBot::ApiClient.new.views_publish(user_id: user_id, view: view)
-
-      if !response.ok?
-        raise SlackBot::Errors::PublishViewError.new(response.error, data: response.data, payload: view)
-      end
-
-      view_id = response.data.dig("view", "id")
-      if callback.present? && view_id.present?
-        callback.view_id = view_id
-        callback.save
-      end
-      SlackViewsReply.new(callback&.id, view_id)
+      response = SlackBot::ApiClient.new.views_publish(user_id: user_id, view: publish_payload(callback, metadata, view))
+      build_view_reply(response: response, callback: callback, payload: view, error_class: SlackBot::Errors::PublishViewError)
     end
 
     attr_reader :current_user, :params, :callback, :config
@@ -73,6 +38,31 @@ module SlackBot
       nil
     end
 
+    def self.modal_payload(callback, view)
+      view.merge(type: "modal", callback_id: callback&.id)
+    end
+
+    def self.publish_payload(callback, metadata, view)
+      view = view.merge(callback_id: callback.id) if callback.present?
+      view = view.merge(private_metadata: metadata) if metadata.present?
+      view
+    end
+
+    def self.build_view_reply(response:, callback:, payload:, error_class:)
+      raise error_class.new(response.error, data: response.data, payload: payload) unless response.ok?
+
+      view_id = response.data.dig("view", "id")
+      persist_view_id(callback, view_id)
+      SlackViewsReply.new(callback&.id, view_id)
+    end
+
+    def self.persist_view_id(callback, view_id)
+      return unless callback.present? && view_id.present?
+
+      callback.view_id = view_id
+      callback.save
+    end
+
     private
 
     def interaction_type

data/lib/slack_bot.rb

@@ -25,5 +25,5 @@ require "slack_bot/pager"
 require "slack_bot/grape_extension"
 
 module SlackBot
-  VERSION = "2.0.0".freeze
+  VERSION = "2.0.1".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: grape-slack-bot
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Andrei Makarov
@@ -32,7 +32,7 @@ dependencies:
         version: '1.6'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -42,7 +42,7 @@ dependencies:
         version: '1.6'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
@@ -83,14 +83,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.12'
+        version: '3'
+- !ruby/object:Gem::Dependency
+  name: polyrun
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.4.2
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 1.4.2
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -111,126 +125,154 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: '13'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '13.0'
+        version: '13'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: standard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '1.52'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '1.52'
 - !ruby/object:Gem::Dependency
-  name: rspec_junit_formatter
+  name: standard-custom
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '1.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.6'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
-  name: simplecov-cobertura
+  name: standard-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '1.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3'
+        version: '1.8'
 - !ruby/object:Gem::Dependency
-  name: standard
+  name: standard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0.3'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.8'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.8'
+- !ruby/object:Gem::Dependency
+  name: rubocop-thread_safety
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.7'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '2'
 - !ruby/object:Gem::Dependency
   name: memory_profiler
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '1'
 - !ruby/object:Gem::Dependency
   name: rbs
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '2'
 description: Slack bot implementation for ruby-grape
 email:
 - contact@kiskolabs.com