i18n-tasks 0.9.33 → 1.0.12

data/lib/i18n/tasks/command/options/common.rb

@@ -28,6 +28,11 @@ module I18n::Tasks
             '--value VALUE',
             t('i18n_tasks.cmd.args.desc.value')
 
+        arg :config,
+            '-c',
+            '--config FILE',
+            t('i18n_tasks.cmd.args.desc.config')
+
         def arg_or_pos!(key, opts)
           opts[key] ||= opts[:arguments].try(:shift)
         end

data/lib/i18n/tasks/command/options/data.rb

@@ -9,7 +9,7 @@ module I18n::Tasks
         include Command::DSL
 
         DATA_FORMATS = %w[yaml json keys].freeze
-        OUT_FORMATS  = ['terminal-table', *DATA_FORMATS, 'inspect'].freeze
+        OUT_FORMATS  = ['terminal-table', *DATA_FORMATS, 'inspect', 'key-values'].freeze
 
         format_arg = proc do |type, values|
           default = values.first
@@ -64,6 +64,7 @@ module I18n::Tasks
         # @return [I18n::Tasks::Data::Tree::Siblings]
         def parse_forest(src, format)
           fail CommandError, I18n.t('i18n_tasks.cmd.errors.pass_forest') unless src
+
           if format == 'keys'
             ::I18n::Tasks::Data::Tree::Siblings.from_key_names parse_keys(src)
           else
@@ -84,6 +85,8 @@ module I18n::Tasks
             puts forest.inspect
           when 'keys'
             puts forest.key_names(root: true)
+          when 'key-values'
+            puts(forest.key_values(root: true).map { |kv| kv.join("\t") })
           when *DATA_FORMATS
             puts i18n.data.adapter_dump forest.to_hash(true), format
           end

data/lib/i18n/tasks/command/options/locales.rb

@@ -13,22 +13,22 @@ module I18n::Tasks
             '--locales en,es,ru',
             Array,
             t('i18n_tasks.cmd.args.desc.locales_filter'),
-            parser:             OptionParsers::Locale::ListParser,
-            default:            'all',
+            parser: OptionParsers::Locale::ListParser,
+            default: 'all',
             consume_positional: true
 
         arg :locale,
             '-l',
             '--locale en',
             t('i18n_tasks.cmd.args.desc.locale'),
-            parser:  OptionParsers::Locale::Parser,
+            parser: OptionParsers::Locale::Parser,
             default: 'base'
 
         arg :locale_to_translate_from,
             '-f',
             '--from en',
             t('i18n_tasks.cmd.args.desc.locale_to_translate_from'),
-            parser:  OptionParsers::Locale::Parser,
+            parser: OptionParsers::Locale::Parser,
             default: 'base'
 
         TRANSLATION_BACKENDS = %w[google deepl].freeze
@@ -36,7 +36,7 @@ module I18n::Tasks
             '-b',
             '--backend BACKEND',
             t('i18n_tasks.cmd.args.desc.translation_backend'),
-            parser:  OptionParsers::Locale::Parser,
+            parser: OptionParsers::Locale::Parser,
             default: TRANSLATION_BACKENDS[0]
       end
     end

data/lib/i18n/tasks/concurrent/cached_value.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/concurrent/cached_value'
-
 module I18n::Tasks::Concurrent
   # A thread-safe memoized value.
   # The given computation is guaranteed to be invoked at most once.
@@ -19,8 +17,10 @@ module I18n::Tasks::Concurrent
     # @return [Object] Result of the computation.
     def get
       return get_result_volatile unless get_result_volatile == NULL
+
       @mutex.synchronize do
         next unless get_result_volatile == NULL
+
         set_result_volatile @computation.call
         @computation = nil
       end

data/lib/i18n/tasks/configuration.rb

@@ -2,10 +2,10 @@
 
 module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
   DEFAULTS = {
-    base_locale:     'en',
+    base_locale: 'en',
     internal_locale: 'en',
-    search:          ::I18n::Tasks::UsedKeys::SEARCH_DEFAULTS,
-    data:            ::I18n::Tasks::Data::DATA_DEFAULTS
+    search: ::I18n::Tasks::UsedKeys::SEARCH_DEFAULTS,
+    data: ::I18n::Tasks::Data::DATA_DEFAULTS
   }.freeze
 
   # i18n-tasks config (defaults + config/i18n-tasks.yml)
@@ -20,7 +20,7 @@ module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
   ].freeze
 
   def file_config
-    file   = CONFIG_FILES.detect { |f| File.exist?(f) }
+    file   = @config_override || CONFIG_FILES.detect { |f| File.exist?(f) }
     # rubocop:disable Security/Eval
     config = file && YAML.load(eval(Erubi::Engine.new(File.read(file, encoding: 'UTF-8')).src))
     # rubocop:enable Security/Eval
@@ -30,6 +30,13 @@ module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
           warn_deprecated 'Please move relative_roots under search in config/i18n-tasks.yml.'
           c[:search][:relative_roots] = c.delete(:relative_roots)
         end
+
+        if c.dig(:search, :exclude_method_name_paths)
+          warn_deprecated(
+            'Please rename exclude_method_name_paths to relative_exclude_method_name_paths in config/i18n-tasks.yml.'
+          )
+          c[:search][:relative_exclude_method_name_paths] = c[:search].delete(:exclude_method_name_paths)
+        end
       end
     else
       {}.with_indifferent_access
@@ -44,12 +51,10 @@ module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
   # data config
   #  @return [Hash<adapter: String, options: Hash>]
   def data_config
-    @config_sections[:data] ||= begin
-      {
-        adapter: data.class.name,
-        config:  data.config
-      }
-    end
+    @config_sections[:data] ||= {
+      adapter: data.class.name,
+      config: data.config
+    }
   end
 
   # translation config
@@ -59,6 +64,8 @@ module I18n::Tasks::Configuration # rubocop:disable Metrics/ModuleLength
       conf = (config[:translation] || {}).with_indifferent_access
       conf[:google_translate_api_key] = ENV['GOOGLE_TRANSLATE_API_KEY'] if ENV.key?('GOOGLE_TRANSLATE_API_KEY')
       conf[:deepl_api_key] = ENV['DEEPL_AUTH_KEY'] if ENV.key?('DEEPL_AUTH_KEY')
+      conf[:deepl_host] = ENV['DEEPL_HOST'] if ENV.key?('DEEPL_HOST')
+      conf[:deepl_version] = ENV['DEEPL_VERSION'] if ENV.key?('DEEPL_VERSION')
       conf[:yandex_api_key] = ENV['YANDEX_API_KEY'] if ENV.key?('YANDEX_API_KEY')
       conf
     end

data/lib/i18n/tasks/console_context.rb

@@ -34,7 +34,7 @@ module I18n::Tasks
       end
 
       def guide
-        Rainbow('i18n-tasks IRB Quick Start guide').green.bright + "\n" + <<~TEXT
+        "#{Rainbow('i18n-tasks IRB Quick Start guide').green.bright}\n#{<<~TEXT}"
           #{Rainbow('Data as trees').yellow}
             tree(locale)
             used_tree(key_filter: nil, strict: nil)

data/lib/i18n/tasks/data/adapter/yaml_adapter.rb

@@ -5,11 +5,13 @@ module I18n::Tasks
   module Data
     module Adapter
       module YamlAdapter
+        EMOJI_REGEX = /\\u[\da-f]{8}/i.freeze
+
         class << self
           # @return [Hash] locale tree
           def parse(str, options)
             if YAML.method(:load).arity.abs == 2
-              YAML.load(str, **(options || {}))
+              YAML.safe_load(str, **(options || {}), permitted_classes: [Symbol], aliases: true)
             else
               # older jruby and rbx 2.2.7 do not accept options
               YAML.load(str)
@@ -18,7 +20,12 @@ module I18n::Tasks
 
           # @return [String]
           def dump(tree, options)
-            tree.to_yaml(options || {})
+            restore_emojis(tree.to_yaml(options || {}))
+          end
+
+          # @return [String]
+          def restore_emojis(yaml)
+            yaml.gsub(EMOJI_REGEX) { |m| [m[-8..].to_i(16)].pack('U') }
           end
         end
       end

data/lib/i18n/tasks/data/file_formats.rb

@@ -53,12 +53,14 @@ module I18n
           content = adapter_dump(hash, adapter)
           # Ignore unchanged data
           return if File.file?(path) && content == read_file(path)
+
           ::FileUtils.mkpath(File.dirname(path))
-          ::File.open(path, 'w') { |f| f.write content }
+          ::File.write(path, content)
         end
 
         def normalized?(path, tree)
           return false unless File.file?(path)
+
           read_file(path) == adapter_dump(tree.to_hash(true), self.class.adapter_name_for_path(path))
         end
 

data/lib/i18n/tasks/data/file_system_base.rb

@@ -12,11 +12,12 @@ module I18n::Tasks
       include ::I18n::Tasks::Data::FileFormats
       include ::I18n::Tasks::Logging
 
-      attr_reader :config, :base_locale, :locales
-      attr_writer :locales
+      attr_accessor :locales
+      attr_reader :config, :base_locale
+      attr_writer :router
 
       DEFAULTS = {
-        read:  ['config/locales/%{locale}.yml'],
+        read: ['config/locales/%{locale}.yml'],
         write: ['config/locales/%{locale}.yml']
       }.freeze
 
@@ -131,11 +132,12 @@ module I18n::Tasks
       def t(key, locale)
         tree = self[locale.to_s]
         return unless tree
+
         tree[locale][key].try(:value_or_children_hash)
       end
 
       def config=(config)
-        @config = DEFAULTS.deep_merge((config || {}).reject { |_k, v| v.nil? })
+        @config = DEFAULTS.deep_merge((config || {}).compact)
         reload
       end
 
@@ -159,7 +161,6 @@ module I18n::Tasks
           router_class.new(self, @config.merge(base_locale: base_locale, locales: locales))
         end
       end
-      attr_writer :router
 
       protected
 
@@ -188,7 +189,7 @@ module I18n::Tasks
               The following unquoted YAML keys result in a nil key:
                 #{%w[null Null NULL ~].join(', ')}
               See http://yaml.org/type/null.html
-TEXT
+            TEXT
           elsif value.is_a?(Hash)
             filter_nil_keys! path, value, suffix + [key]
           end

data/lib/i18n/tasks/data/router/conservative_router.rb

@@ -13,8 +13,9 @@ module I18n::Tasks
         super
       end
 
-      def route(locale, forest, &block)
+      def route(locale, forest, &block) # rubocop:disable Metrics/AbcSize
         return to_enum(:route, locale, forest) unless block
+
         out = Hash.new { |hash, key| hash[key] = Set.new }
         not_found = Set.new
         forest.keys do |key, _node|

data/lib/i18n/tasks/data/router/pattern_router.rb

@@ -10,6 +10,7 @@ module I18n::Tasks
       include ::I18n::Tasks::KeyPatternMatching
 
       attr_reader :routes
+
       # @option data_config write [Array] of routes
       # @example
       #   {write:
@@ -29,6 +30,7 @@ module I18n::Tasks
       # @return [Hash] mapping of destination => [ [key, value], ... ]
       def route(locale, forest, &block)
         return to_enum(:route, locale, forest) unless block
+
         locale = locale.to_s
         out = {}
         forest.keys do |key, _node|
@@ -36,7 +38,7 @@ module I18n::Tasks
           if pattern
             key_match = $~
             path      = format(path, locale: locale)
-            path.gsub!(/\\\d+/) { |m| key_match[m[1..-1].to_i] }
+            path.gsub!(/\\\d+/) { |m| key_match[m[1..].to_i] }
             (out[path] ||= Set.new) << "#{locale}.#{key}"
           else
             fail CommandError, "Cannot route key #{key}. Routes are #{@routes_config.inspect}"

data/lib/i18n/tasks/data/tree/node.rb

@@ -18,7 +18,7 @@ module I18n::Tasks::Data::Tree
       @data = data
       @parent = parent
       @warn_about_add_children_to_leaf = warn_about_add_children_to_leaf
-      self.children = (children if children)
+      self.children = children if children
     end
     # rubocop:enable Metrics/ParameterLists
 
@@ -48,6 +48,7 @@ module I18n::Tasks::Data::Tree
 
     def each(&block)
       return to_enum(:each) { 1 } unless block
+
       block.yield(self)
       self
     end
@@ -112,6 +113,7 @@ module I18n::Tasks::Data::Tree
 
     def walk_to_root(&visitor)
       return to_enum(:walk_to_root) unless visitor
+
       visitor.yield self
       parent.walk_to_root(&visitor) if parent?
     end
@@ -124,6 +126,7 @@ module I18n::Tasks::Data::Tree
 
     def walk_from_root(&visitor)
       return to_enum(:walk_from_root) unless visitor
+
       walk_to_root.reverse_each do |node|
         visitor.yield node
       end
@@ -165,11 +168,11 @@ module I18n::Tasks::Data::Tree
       label = if key.nil?
                 Rainbow('∅').faint
               else
-                [Rainbow(key).color(1 + level % 15),
+                [Rainbow(key).color(1 + (level % 15)),
                  (": #{format_value_for_inspect(value)}" if leaf?),
                  (" #{data}" if data?)].compact.join
               end
-      ['  ' * level, label, ("\n" + children.map { |c| c.inspect(level + 1) }.join("\n") if children?)].compact.join
+      ['  ' * level, label, ("\n#{children.map { |c| c.inspect(level + 1) }.join("\n")}" if children?)].compact.join
     end
 
     def format_value_for_inspect(value)

data/lib/i18n/tasks/data/tree/nodes.rb

@@ -31,13 +31,11 @@ module I18n::Tasks::Data::Tree
     end
 
     def to_hash(sort = false)
-      (@hash ||= {})[sort] ||= begin
-        if sort
-          sort_by(&:key)
-        else
-          self
-        end.map { |node| node.to_hash(sort) }.reduce({}, :deep_merge!)
-      end
+      (@hash ||= {})[sort] ||= if sort
+                                 sort_by(&:key)
+                               else
+                                 self
+                               end.map { |node| node.to_hash(sort) }.reduce({}, :deep_merge!)
     end
 
     delegate :to_json, to: :to_hash
@@ -80,6 +78,7 @@ module I18n::Tasks::Data::Tree
 
     def children(&block)
       return to_enum(:children) { map { |c| c.children ? c.children.size : 0 }.reduce(:+) } unless block
+
       each do |node|
         node.children.each(&block) if node.children?
       end

data/lib/i18n/tasks/data/tree/siblings.rb

@@ -3,7 +3,6 @@
 require 'set'
 require 'i18n/tasks/split_key'
 require 'i18n/tasks/data/tree/nodes'
-require 'i18n/tasks/data/tree/node'
 
 module I18n::Tasks::Data::Tree
   # Siblings represents a subtree sharing a common parent
@@ -50,7 +49,7 @@ module I18n::Tasks::Data::Tree
             next
           end
           match = $~
-          new_key = to_pattern.gsub(/\\\d+/) { |m| match[m[1..-1].to_i] }
+          new_key = to_pattern.gsub(/\\\d+/) { |m| match[m[1..].to_i] }
           old_key_to_new_key[full_key] = new_key
           moved_forest.merge!(Siblings.new.tap do |forest|
             forest[[(node.root.try(:key) unless root), new_key].compact.join('.')] =
@@ -62,6 +61,7 @@ module I18n::Tasks::Data::Tree
       # TODO: support nested references better
       nodes do |node|
         next unless node.reference?
+
         old_target = [(node.root.key if root), node.value.to_s].compact.join('.')
         new_target = old_key_to_new_key[old_target]
         if new_target
@@ -92,6 +92,7 @@ module I18n::Tasks::Data::Tree
     # add or replace node by full key
     def set(full_key, node)
       fail 'value should be a I18n::Tasks::Data::Tree::Node' unless node.is_a?(Node)
+
       key_part, rest = split_key(full_key, 2)
       child          = key_to_node[key_part]
 
@@ -131,6 +132,7 @@ module I18n::Tasks::Data::Tree
     def append!(nodes)
       nodes = nodes.map do |node|
         fail "already has a child with key '#{node.key}'" if key_to_node.key?(node.key)
+
         key_to_node[node.key] = (node.parent == parent ? node : node.derive(parent: parent))
       end
       super(nodes)
@@ -172,16 +174,18 @@ module I18n::Tasks::Data::Tree
 
     def set_root_key!(new_key, data = nil)
       return self if empty?
+
       rename_key first.key, new_key
       leaves { |node| node.data.merge! data } if data
       self
     end
 
     # @param on_leaves_merge [Proc] invoked when a leaf is merged with another leaf
-    def merge_node!(node, on_leaves_merge: nil) # rubocop:disable Metrics/AbcSize,Metrics/PerceivedComplexity
+    def merge_node!(node, on_leaves_merge: nil) # rubocop:disable Metrics/AbcSize
       if key_to_node.key?(node.key)
         our = key_to_node[node.key]
         return if our == node
+
         our.value = node.value if node.leaf?
         our.data.merge!(node.data) if node.data?
         if node.children?
@@ -262,7 +266,7 @@ module I18n::Tasks::Data::Tree
         build_forest(warn_about_add_children_to_leaf: false) do |forest|
           key_occurrences.each do |key_occurrence|
             forest[key_occurrence.key] = ::I18n::Tasks::Data::Tree::Node.new(
-              key:  split_key(key_occurrence.key).last,
+              key: split_key(key_occurrence.key).last,
               data: { occurrences: key_occurrence.occurrences }
             )
           end
@@ -273,6 +277,7 @@ module I18n::Tasks::Data::Tree
         build_forest(opts) do |forest|
           key_attrs.each do |(full_key, attr)|
             fail "Invalid key #{full_key.inspect}" if full_key.end_with?('.')
+
             node = ::I18n::Tasks::Data::Tree::Node.new(**attr.merge(key: split_key(full_key).last))
             yield(full_key, node) if block
             forest[full_key] = node
@@ -295,6 +300,7 @@ module I18n::Tasks::Data::Tree
       def from_nested_hash(hash, opts = {})
         parse_parent_opt!(opts)
         fail I18n::Tasks::CommandError, "invalid tree #{hash.inspect}" unless hash.respond_to?(:map)
+
         opts[:nodes] = hash.map { |key, value| Node.from_key_value key, value }
         Siblings.new(opts)
       end

data/lib/i18n/tasks/data/tree/traversal.rb

@@ -12,6 +12,7 @@ module I18n::Tasks
 
       def leaves(&visitor)
         return to_enum(:leaves) unless visitor
+
         nodes do |node|
           visitor.yield(node) if node.leaf?
         end
@@ -20,6 +21,7 @@ module I18n::Tasks
 
       def levels(&block)
         return to_enum(:levels) unless block
+
         nodes = to_nodes
         unless nodes.empty?
           block.yield nodes
@@ -35,6 +37,7 @@ module I18n::Tasks
 
       def breadth_first(&visitor)
         return to_enum(:breadth_first) unless visitor
+
         levels do |nodes|
           nodes.each { |node| visitor.yield(node) }
         end
@@ -43,9 +46,11 @@ module I18n::Tasks
 
       def depth_first(&visitor)
         return to_enum(:depth_first) unless visitor
+
         each do |node|
           visitor.yield node
           next unless node.children?
+
           node.children.each do |child|
             child.depth_first(&visitor)
           end
@@ -56,6 +61,7 @@ module I18n::Tasks
       # @option root include root in full key
       def keys(root: false, &visitor)
         return to_enum(:keys, root: root) unless visitor
+
         leaves { |node| visitor.yield(node.full_key(root: root), node) }
         self
       end
@@ -70,13 +76,13 @@ module I18n::Tasks
 
       def root_key_values(sort = false)
         result = keys(root: false).map { |key, node| [node.root.key, key, node.value] }
-        result.sort! { |a, b| a[0] != b[0] ? a[0] <=> b[0] : a[1] <=> b[1] } if sort
+        result.sort! { |a, b| a[0] == b[0] ? a[1] <=> b[1] : a[0] <=> b[0] } if sort
         result
       end
 
       def root_key_value_data(sort = false)
         result = keys(root: false).map { |key, node| [node.root.key, key, node.value, node.data] }
-        result.sort! { |a, b| a[0] != b[0] ? a[0] <=> b[0] : a[1] <=> b[1] } if sort
+        result.sort! { |a, b| a[0] == b[0] ? a[1] <=> b[1] : a[0] <=> b[0] } if sort
         result
       end
 
@@ -88,6 +94,7 @@ module I18n::Tasks
         tree = Siblings.new
         each do |node|
           next unless block.yield(node)
+
           tree.append! node.derive(
             parent: tree.parent,
             children: (node.children.select_nodes(&block).to_a if node.children)
@@ -163,22 +170,38 @@ module I18n::Tasks
         value_proc ||= proc do |node|
           node_value = node.value
           next node_value if node.reference?
+
           human_key = ActiveSupport::Inflector.humanize(node.key.to_s)
           full_key = node.full_key
           default = (node.data[:occurrences] || []).detect { |o| o.default_arg.presence }.try(:default_arg)
-          StringInterpolation.interpolate_soft(
-            val_pattern,
-            value: node_value,
-            human_key: human_key,
-            key: full_key,
-            default: default,
-            value_or_human_key: node_value.presence || human_key,
-            value_or_default_or_human_key: node_value.presence || default || human_key
-          )
+          if default.is_a?(Hash)
+            default.each_with_object({}) do |(k, v), h|
+              h[k] = StringInterpolation.interpolate_soft(
+                val_pattern,
+                value: node_value,
+                human_key: human_key,
+                key: full_key,
+                default: v,
+                value_or_human_key: node_value.presence || human_key,
+                value_or_default_or_human_key: node_value.presence || v || human_key
+              )
+            end
+          else
+            StringInterpolation.interpolate_soft(
+              val_pattern,
+              value: node_value,
+              human_key: human_key,
+              key: full_key,
+              default: default,
+              value_or_human_key: node_value.presence || human_key,
+              value_or_default_or_human_key: node_value.presence || default || human_key
+            )
+          end
         end
         pattern_re = I18n::Tasks::KeyPatternMatching.compile_key_pattern(key_pattern) if key_pattern.present?
         keys.each do |key, node|
           next if pattern_re && key !~ pattern_re
+
           node.value = value_proc.call(node)
         end
         self

data/lib/i18n/tasks/html_keys.rb

@@ -2,15 +2,13 @@
 
 module I18n::Tasks
   module HtmlKeys
-    HTML_KEY_PATTERN = /[.\-_]html\z/
-    MAYBE_PLURAL_HTML_KEY_PATTERN = /[.\-_]html\.[^.]+\z/
+    HTML_KEY_PATTERN = /[.\-_]html\z/.freeze
+    MAYBE_PLURAL_HTML_KEY_PATTERN = /[.\-_]html\.[^.]+\z/.freeze
 
     def html_key?(full_key, locale)
-      # rubocop:disable Style/DoubleNegation
       !!(full_key =~ HTML_KEY_PATTERN ||
-          full_key =~ MAYBE_PLURAL_HTML_KEY_PATTERN &&
-              depluralize_key(split_key(full_key, 2)[1], locale) =~ HTML_KEY_PATTERN)
-      # rubocop:enable Style/DoubleNegation
+          (full_key =~ MAYBE_PLURAL_HTML_KEY_PATTERN &&
+              depluralize_key(split_key(full_key, 2)[1], locale) =~ HTML_KEY_PATTERN))
     end
   end
 end

data/lib/i18n/tasks/ignore_keys.rb

@@ -17,12 +17,13 @@ module I18n::Tasks::IgnoreKeys
     @ignore_patterns[type][locale] ||= begin
       global = ignore_config.presence || []
       type_ignore = ignore_config(type).presence || []
-      patterns = if type_ignore.is_a?(Array)
+      patterns = case type_ignore
+                 when Array
                    global + type_ignore
-                 elsif type_ignore.is_a?(Hash)
+                 when Hash
                    # ignore per locale
                    global + (type_ignore['all'] || []) +
-                     type_ignore.select { |k, _v| k.to_s =~ /\b#{locale}\b/ }.values.flatten(1).compact
+                   type_ignore.select { |k, _v| k.to_s =~ /\b#{locale}\b/ }.values.flatten(1).compact
                  end
       compile_patterns_re patterns
     end

data/lib/i18n/tasks/interpolations.rb

@@ -2,20 +2,26 @@
 
 module I18n::Tasks
   module Interpolations
-    VARIABLE_REGEX = /%{[^}]+}/
+    class << self
+      attr_accessor :variable_regex
+    end
+    @variable_regex = /%{[^}]+}/.freeze
 
     def inconsistent_interpolations(locales: nil, base_locale: nil) # rubocop:disable Metrics/AbcSize
       locales ||= self.locales
       base_locale ||= self.base_locale
       result = empty_forest
+      variable_regex = I18n::Tasks::Interpolations.variable_regex
 
       data[base_locale].key_values.each do |key, value|
         next if !value.is_a?(String) || ignore_key?(key, :inconsistent_interpolations)
-        base_vars = Set.new(value.scan(VARIABLE_REGEX))
+
+        base_vars = Set.new(value.scan(variable_regex))
         (locales - [base_locale]).each do |current_locale|
           node = data[current_locale].first.children[key]
-          next unless node&.value&.is_a?(String)
-          if base_vars != Set.new(node.value.scan(VARIABLE_REGEX))
+          next unless node&.value.is_a?(String)
+
+          if base_vars != Set.new(node.value.scan(variable_regex))
             result.merge!(node.walk_to_root.reduce(nil) { |c, p| [p.derive(children: c)] })
           end
         end

data/lib/i18n/tasks/key_pattern_matching.rb

@@ -5,7 +5,7 @@ require 'strscan'
 module I18n::Tasks::KeyPatternMatching
   extend self # rubocop:disable Style/ModuleFunction
 
-  MATCH_NOTHING = /\z\A/
+  MATCH_NOTHING = /\z\A/.freeze
 
   # one regex to match any
   def compile_patterns_re(key_patterns)
@@ -13,7 +13,7 @@ module I18n::Tasks::KeyPatternMatching
       # match nothing
       MATCH_NOTHING
     else
-      /(?:#{ key_patterns.map { |p| compile_key_pattern p } * '|' })/m
+      /(?:#{key_patterns.map { |p| compile_key_pattern p } * '|'})/m
     end
   end
 
@@ -24,6 +24,7 @@ module I18n::Tasks::KeyPatternMatching
   #   { a, b.c } match any in set, can use : and *, match is captured
   def compile_key_pattern(key_pattern)
     return key_pattern if key_pattern.is_a?(Regexp)
+
     /\A#{key_pattern_re_body(key_pattern)}\z/
   end