web_translate_it 3.2.2 → 3.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2344f55bed19e435f8334c66229e58266ff1efe3cf54e9e81f4d17287f42a93d
-  data.tar.gz: 89c6ea08bcc87a43741c8419d5de32e2441f896f12e8338762c15cf84759ab5e
+  metadata.gz: 660b650bae69f2f4a4282acb69b7abc72296b7e352660adaf3c1047a3bac04ce
+  data.tar.gz: acf8cc2ef71f6a897d400cd99793b86c33d28672b8aa6049b49d1f3b3a2f3c16
 SHA512:
-  metadata.gz: c4a9b5b8f2874c7779eedb0d413d826a1d9bccbd8e9db33834ab82a0b8b22b2fb719bf412e7339cb05880018ddc072ee6e70a370824f3fc96f7d6982a2f8396f
-  data.tar.gz: ad5b96143c805efa324ea6119af80dd24517c2e41691b451920934a3deea7115385c13f859ccff7d314c034680f64b95033aba839b1bc4728ff09aa885395a00
+  metadata.gz: dc29fee26eb5eb394705c81975adcf21f03dbe3ca7918d0e9a622a791ef0163ee9c7aa00f88d9c608eea462b5a75e8b9e4c7d530af276cfa3449ab7824ef53ec
+  data.tar.gz: 148e6238f00cb1cd2030a8a9f13c1b155ed82c26baac67f6933d86da68463998a8f91f144339f8d538512fe7eee0ec47cab15b23a2de8bdab27df48f77176b76

data/bin/wti

@@ -42,11 +42,12 @@ when 'pull'
       wti pull [filename] - Pull target language file(s)
       [options] are:
     BANNER
-    opt :locale, 'ISO code of locale(s) to pull, space-separated', type: :string
-    opt :all,    'Pull all files'
-    opt :force,  'Force pull (bypass conditional requests to WTI)'
-    opt :config, 'Path to a configuration file', short: '-c', default: '.wti'
-    opt :debug,  'Display debug information'
+    opt :locale,  'ISO code of locale(s) to pull, space-separated', type: :string
+    opt :all,     'Pull all files'
+    opt :force,   'Force pull (bypass conditional requests to WTI)'
+    opt :threads, 'Number of threads for parallel downloads', type: :integer, default: 10
+    opt :config,  'Path to a configuration file', short: '-c', default: '.wti'
+    opt :debug,   'Display debug information'
   end
 when 'push'
   Optimist.options do

data/history.md

@@ -1,3 +1,9 @@
+## Version 3.2.3 / 2026-04-09
+
+* Handle HTTP 429 (rate limit) errors: add `RateLimitError` class, retry with backoff respecting the `Retry-After` header, and fix garbled error messages when the response body is not JSON.
+* Add `--threads N` option to `wti pull` to control the number of concurrent download threads. Defaults to 10. Use `--threads 1` for sequential pulls.
+* Modernize code for Ruby 3.0+: use `match?` instead of `!~`, `start_with?`/`delete_prefix` instead of index slicing, array difference instead of `reject`, and remove redundant `|| nil` fallbacks.
+
 ## Version 3.2.2 / 2026-03-03
 
 * Replace O(n²) string concatenation loop in `StringUtil#backward_truncate` with `String#ljust`.

data/lib/web_translate_it/api_resource.rb

@@ -9,9 +9,9 @@ module WebTranslateIt
     def initialize(params = {}, connection: nil)
       params = params.transform_keys(&:to_s)
       self.connection   = connection
-      self.id           = params['id'] || nil
-      self.created_at   = params['created_at'] || nil
-      self.updated_at   = params['updated_at'] || nil
+      self.id           = params['id']
+      self.created_at   = params['created_at']
+      self.updated_at   = params['updated_at']
       self.translations = params['translations'] || []
       self.new_record   = true
       assign_attributes(params)

data/lib/web_translate_it/auto_fetch.rb

@@ -29,7 +29,7 @@ module WebTranslateIt
     end
 
     def valid_request?(env)
-      env['PATH_INFO'] !~ /\.(js|css|jpeg|jpg|gif|png|woff)$/
+      !env['PATH_INFO'].match?(/\.(js|css|jpeg|jpg|gif|png|woff)$/)
     end
 
   end

data/lib/web_translate_it/commands/pull.rb

@@ -29,7 +29,7 @@ module WebTranslateIt
 
       def pull_files(files) # rubocop:todo Metrics/AbcSize, Metrics/MethodLength, Naming/PredicateMethod
         time = Time.now
-        results, n_threads = Concurrency.concurrent_batch(files) do |batch|
+        results, n_threads = Concurrency.concurrent_batch(files, max_threads: command_options.threads) do |batch|
           with_connection do |conn|
             batch.map do |file|
               result = file.fetch(conn, command_options.force)

data/lib/web_translate_it/commands/push.rb

@@ -37,7 +37,7 @@ module WebTranslateIt
         if command_options.locale
           warn_unknown_locales(command_options.locale.split)
         elsif command_options.target
-          configuration.target_locales.reject { |locale| locale == configuration.source_locale }
+          configuration.target_locales - [configuration.source_locale]
         else
           [configuration.source_locale]
         end

data/lib/web_translate_it/string.rb

@@ -16,16 +16,16 @@ module WebTranslateIt
 
     protected
 
-    def assign_attributes(params) # rubocop:todo Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-      self.key          = params['key'] || nil
-      self.plural       = params['plural'] || nil
-      self.type         = params['type'] || nil
-      self.dev_comment  = params['dev_comment'] || nil
-      self.word_count   = params['word_count'] || nil
-      self.status       = params['status'] || nil
-      self.category     = params['category'] || nil
-      self.labels       = params['labels'] || nil
-      self.file         = params['file'] || nil
+    def assign_attributes(params) # rubocop:todo Metrics/AbcSize
+      self.key          = params['key']
+      self.plural       = params['plural']
+      self.type         = params['type']
+      self.dev_comment  = params['dev_comment']
+      self.word_count   = params['word_count']
+      self.status       = params['status']
+      self.category     = params['category']
+      self.labels       = params['labels']
+      self.file         = params['file']
     end
 
     def parse_translation_response(json)

data/lib/web_translate_it/term.rb

@@ -13,8 +13,8 @@ module WebTranslateIt
     protected
 
     def assign_attributes(params)
-      self.text         = params['text'] || nil
-      self.description  = params['description'] || nil
+      self.text         = params['text']
+      self.description  = params['description']
     end
 
     def parse_translation_response(json)

data/lib/web_translate_it/util/concurrency.rb

@@ -4,18 +4,27 @@ module WebTranslateIt
 
   module Concurrency
 
-    # Execute a block with automatic retry on Timeout::Error.
+    # Execute a block with automatic retry on Timeout::Error and RateLimitError.
     # Returns the block's return value on success, or re-raises after retries are exhausted.
     def self.with_retries(retries: 3, delay: 5)
       yield
     rescue Timeout::Error
-      puts "Request timeout. Will retry in #{delay} seconds."
-      if (retries -= 1).positive?
-        sleep(delay)
-        retry
-      end
-      raise
+      raise unless (retries -= 1).positive?
+
+      log_retry('Request timeout', delay)
+      retry
+    rescue RateLimitError => e
+      raise unless (retries -= 1).positive?
+
+      log_retry('Rate limited', e.retry_after || delay)
+      retry
+    end
+
+    def self.log_retry(message, wait)
+      puts "#{message}. Will retry in #{wait} seconds."
+      sleep(wait)
     end
+    private_class_method :log_retry
 
     # Process items in parallel using a thread pool.
     # Yields each batch (array of items) to the block; collects return values.

data/lib/web_translate_it/util/http_response.rb

@@ -2,6 +2,17 @@
 
 module WebTranslateIt
 
+  class RateLimitError < StandardError
+
+    attr_reader :retry_after
+
+    def initialize(retry_after: nil)
+      @retry_after = retry_after
+      super("Rate limited#{", retry after #{retry_after}s" if retry_after}")
+    end
+
+  end
+
   module HttpResponse
 
     STATUS_LABELS = {
@@ -31,15 +42,24 @@ module WebTranslateIt
 
     def self.raise_on_error!(response)
       code = response.code.to_i
-      if code >= 400 && code < 500
-        raise "Error: #{MultiJson.load(response.body)['error']}"
-      elsif code == 500
-        raise 'Error: Server temporarily unavailable (Error 500).'
-      elsif code == 503
-        raise 'Error: Locked (another import in progress)'
-      end
+      raise_on_rate_limit!(response) if code == 429
+      raise "Error: #{error_message(response)}" if code >= 400 && code < 500
+      raise 'Error: Server temporarily unavailable (Error 500).' if code == 500
+      raise 'Error: Locked (another import in progress)' if code == 503
     end
-    private_class_method :raise_on_error!
+
+    def self.raise_on_rate_limit!(response)
+      retry_after = response['Retry-After']&.to_i
+      raise RateLimitError.new(retry_after: retry_after)
+    end
+
+    def self.error_message(response)
+      MultiJson.load(response.body)['error']
+    rescue StandardError
+      response.body.to_s
+    end
+
+    private_class_method :raise_on_error!, :raise_on_rate_limit!, :error_message
 
   end
 

data/lib/web_translate_it/util/string_util.rb

@@ -25,8 +25,8 @@ class StringUtil
   end
 
   def self.array_to_columns(array)
-    if array[0][0] == '*'
-      "*#{backward_truncate(array[0][1..])} | #{array[1]}  #{array[2]}\n"
+    if array[0].start_with?('*')
+      "*#{backward_truncate(array[0].delete_prefix('*'))} | #{array[1]}  #{array[2]}\n"
     else
       " #{backward_truncate(array[0])} | #{array[1]}  #{array[2]}\n"
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: web_translate_it
 version: !ruby/object:Gem::Version
-  version: 3.2.2
+  version: 3.2.3
 platform: ruby
 authors:
 - Edouard Briere
@@ -113,7 +113,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 4.0.6
 specification_version: 4
 summary: A CLI tool to sync locale files with WebTranslateIt.com.
 test_files: []