smart_todo 1.7.0 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7bbca316fba93242bc5ea22479579407ed9ce3ed93a6bd618978223f96f0626d
-  data.tar.gz: c91cf35475f5a32da647f6ed649ac7d50d85a929908aa29f1768a9b1d6db75da
+  metadata.gz: a7cac40289c5fb2fdf4bb143ee7cca9b1c74ce41c7e1fa426f7efc17364a2683
+  data.tar.gz: c93946f4131f64068e9c249c760829a33f3e88ae669f322bb00ea0647f68eb20
 SHA512:
-  metadata.gz: 1668111d5578d6c64f2ff7df6071f9ede6ccc4aeb01bf1c5cdccea8d514cf29caad0b2b576bb3bb60eaaaa309f44b723e066c62b39b50ab17e418372a29c4dad
-  data.tar.gz: 25636e700c032b2bc0509791f8884f9e9f6cb78ae09103db503d69739e48507c07c8a8fc6579f4f441c1f08caf09f19274ad19061305b9ad9b82efc5c41e9711
+  metadata.gz: a6a84fd8bd731f7b31fe0033be57a50ba018af724ae29cfa16a4c9a0a96766e6335ee1c1145976002519aff1f69d6a28a3bf220963e42f55c28b051f02c85d80
+  data.tar.gz: 562151dd8c4fc2b8d55d72c004a701ef2c10f01d3e84f632b1a945ee969469e7dffd5473c7f109548ca105175b1a893f1e039a9ee091b3cb8034af7e47116082

data/.github/workflows/build.yml

@@ -12,7 +12,7 @@ jobs:
     name: Ruby ${{ matrix.version }}
     strategy:
       matrix:
-        version: [3.0, 3.1, 3.2]
+        version: [3.0, 3.1, 3.2, 3.3]
 
     steps:
       - uses: actions/checkout@v2

data/.github/workflows/rubocop.yml

@@ -8,10 +8,9 @@ jobs:
 
     steps:
     - uses: actions/checkout@v2
-    - name: Set up Ruby 3.0
+    - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:
-        ruby-version: 3.0
         bundler-cache: true
     - name: Install gems
       run: |

data/.rubocop.yml

@@ -2,7 +2,6 @@ inherit_gem:
   rubocop-shopify: rubocop.yml
 
 AllCops:
-  TargetRubyVersion: 3.0
   NewCops: disable
   SuggestExtensions: false
   Exclude:

data/.ruby-version

@@ -0,0 +1 @@
+3.3.0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    smart_todo (1.7.0)
+    smart_todo (1.8.0)
       prism (~> 0.15)
 
 GEM
@@ -13,35 +13,37 @@ GEM
     crack (0.4.5)
       rexml
     hashdiff (1.0.1)
-    json (2.6.3)
+    json (2.7.1)
+    language_server-protocol (3.17.0.3)
     minitest (5.14.4)
-    parallel (1.23.0)
-    parser (3.2.2.3)
+    parallel (1.24.0)
+    parser (3.3.0.5)
       ast (~> 2.4.1)
       racc
     prism (0.17.0)
     public_suffix (4.0.6)
-    racc (1.7.0)
+    racc (1.7.3)
     rainbow (3.1.1)
     rake (13.0.6)
-    regexp_parser (2.8.1)
-    rexml (3.2.5)
-    rubocop (1.52.1)
+    regexp_parser (2.9.0)
+    rexml (3.2.6)
+    rubocop (1.62.1)
       json (~> 2.3)
+      language_server-protocol (>= 3.17.0)
       parallel (~> 1.10)
-      parser (>= 3.2.2.3)
+      parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.28.0, < 2.0)
+      rubocop-ast (>= 1.31.1, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.29.0)
-      parser (>= 3.2.1.0)
+    rubocop-ast (1.31.2)
+      parser (>= 3.3.0.4)
     rubocop-shopify (2.14.0)
       rubocop (~> 1.51)
     ruby-progressbar (1.13.0)
-    unicode-display_width (2.4.2)
+    unicode-display_width (2.5.0)
     webmock (3.11.2)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
@@ -59,4 +61,4 @@ DEPENDENCIES
   webmock
 
 BUNDLED WITH
-   2.2.25
+   2.5.7

data/dev.yml

@@ -4,7 +4,7 @@ type:
   - ruby
 
 up:
-  - ruby: 3.0.2
+  - ruby
   - bundler
 
 console:

data/lib/smart_todo/slack_client.rb

@@ -39,9 +39,9 @@ module SmartTodo
     #
     # @see https://api.slack.com/methods/users.lookupByEmail
     def lookup_user_by_email(email)
-      headers = { "Content-Type" => "application/x-www-form-urlencoded" }
-
-      request(:get, "/api/users.lookupByEmail?email=#{CGI.escape(email)}", nil, headers)
+      headers = default_headers.merge("Content-Type" => "application/x-www-form-urlencoded")
+      request = Net::HTTP::Get.new("/api/users.lookupByEmail?email=#{CGI.escape(email)}", headers)
+      dispatch(request)
     end
 
     # Send a message to a Slack channel or to a user
@@ -55,7 +55,10 @@ module SmartTodo
     #
     # @see https://api.slack.com/methods/chat.postMessage
     def post_message(channel, text)
-      request(:post, "/api/chat.postMessage", JSON.dump(channel: channel, text: text))
+      headers = default_headers
+      request = Net::HTTP::Post.new("/api/chat.postMessage", headers)
+      request.body = JSON.dump(channel: channel, text: text)
+      dispatch(request)
     end
 
     private
@@ -67,27 +70,19 @@ module SmartTodo
     #
     # @raise [Net::HTTPError] in case the request to Slack failed
     # @raise [SlackClient::Error] in case Slack returns a { ok: false } in the body
-    def request(method, endpoint, data = nil, headers = {})
-      response = case method
-      when :post, :patch
-        @client.public_send(method, endpoint, data, default_headers.merge(headers))
-      else
-        @client.public_send(method, endpoint, default_headers.merge(headers))
-      end
+    def dispatch(request, max_attempts = 5)
+      response = @client.request(request)
+      attempts = 1
 
-      slack_response!(response)
-    end
+      while response.is_a?(Net::HTTPTooManyRequests) && attempts < max_attempts
+        sleep([Integer(response["Retry-After"]), 600].min)
+        response = @client.request(request)
+        attempts += 1
+      end
 
-    # Check if the response to Slack was a 200 and the Slack API request was successful
-    #
-    # @param response [Net::HTTPResponse] a net Net::HTTPResponse subclass
-    #   (Net::HTTPOK, Net::HTTPNotFound ...)
-    # @return [Hash]
-    #
-    # @raise [Net::HTTPError] in case the request to Slack failed
-    # @raise [SlackClient::Error] in case Slack returns a { ok: false } in the body
-    def slack_response!(response)
-      raise(Net::HTTPError.new("Request to slack failed", response)) unless response.code_type < Net::HTTPSuccess
+      unless response.code_type < Net::HTTPSuccess
+        raise(Net::HTTPError.new("Request to slack failed", response))
+      end
 
       body = JSON.parse(response.body)
 

data/lib/smart_todo/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SmartTodo
-  VERSION = "1.7.0"
+  VERSION = "1.8.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smart_todo
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 1.8.0
 platform: ruby
 authors:
 - Shopify
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-12-08 00:00:00.000000000 Z
+date: 2024-06-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: prism
@@ -97,6 +97,7 @@ files:
 - ".github/workflows/rubocop.yml"
 - ".gitignore"
 - ".rubocop.yml"
+- ".ruby-version"
 - CODE_OF_CONDUCT.md
 - Gemfile
 - Gemfile.lock
@@ -147,7 +148,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.22
+rubygems_version: 3.5.13
 signing_key: 
 specification_version: 4
 summary: Enhance todo's comments in your codebase.