i18n-tasks 1.0.15 → 1.1.0

data/lib/i18n/tasks/reports/terminal.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/reports/base'
-require 'terminal-table'
+require "i18n/tasks/reports/base"
+require "terminal-table"
 module I18n
   module Tasks
     module Reports
@@ -10,17 +10,17 @@ module I18n
           forest = collapse_missing_tree! forest
           if forest.present?
             print_title missing_title(forest)
-            print_table headings: [Rainbow(I18n.t('i18n_tasks.common.locale')).cyan.bright,
-                                   Rainbow(I18n.t('i18n_tasks.common.key')).cyan.bright,
-                                   I18n.t('i18n_tasks.missing.details_title')] do |t|
+            print_table headings: [Rainbow(I18n.t("i18n_tasks.common.locale")).cyan.bright,
+              Rainbow(I18n.t("i18n_tasks.common.key")).cyan.bright,
+              I18n.t("i18n_tasks.missing.details_title")] do |t|
               t.rows = sort_by_attr!(forest_to_attr(forest)).map do |a|
-                [{ value: Rainbow(format_locale(a[:locale])).cyan, alignment: :center },
-                 format_key(a[:key], a[:data]),
-                 missing_key_info(a)]
+                [{value: Rainbow(format_locale(a[:locale])).cyan, alignment: :center},
+                  format_key(a[:key], a[:data]),
+                  missing_key_info(a)]
               end
             end
           else
-            print_success I18n.t('i18n_tasks.missing.none')
+            print_success I18n.t("i18n_tasks.missing.none")
           end
         end
 
@@ -29,7 +29,24 @@ module I18n
             print_title inconsistent_interpolations_title(forest)
             show_tree(forest)
           else
-            print_success I18n.t('i18n_tasks.inconsistent_interpolations.none')
+            print_success I18n.t("i18n_tasks.inconsistent_interpolations.none")
+          end
+        end
+
+        def reserved_interpolations(forest = task.reserved_interpolations)
+          if forest.present?
+            print_title reserved_interpolations_title(forest)
+            print_table headings: [Rainbow(I18n.t("i18n_tasks.common.locale")).cyan.bright,
+              Rainbow(I18n.t("i18n_tasks.common.key")).cyan.bright,
+              I18n.t("i18n_tasks.reserved_interpolations.details_title")] do |t|
+              t.rows = sort_by_attr!(forest_to_attr(forest)).map do |a|
+                [{value: Rainbow(format_locale(a[:locale])).cyan, alignment: :center},
+                  format_key(a[:key], a[:data]),
+                  a[:value].join(", ")]
+              end
+            end
+          else
+            print_success I18n.t("i18n_tasks.reserved_interpolations.none")
           end
         end
 
@@ -45,7 +62,7 @@ module I18n
               print_occurrences node, key
             end
           else
-            print_error I18n.t('i18n_tasks.usages.none')
+            print_error I18n.t("i18n_tasks.usages.none")
           end
         end
 
@@ -55,7 +72,7 @@ module I18n
             print_title unused_title(keys)
             print_locale_key_value_data_table keys
           else
-            print_success I18n.t('i18n_tasks.unused.none')
+            print_success I18n.t("i18n_tasks.unused.none")
           end
         end
 
@@ -65,7 +82,7 @@ module I18n
             print_title eq_base_title(keys)
             print_locale_key_value_data_table keys
           else
-            print_info Rainbow('No translations are the same as base value').cyan
+            print_info Rainbow("No translations are the same as base value").cyan
           end
         end
 
@@ -74,39 +91,39 @@ module I18n
         end
 
         def forest_stats(forest, stats = task.forest_stats(forest))
-          text  = if stats[:locale_count] == 1
-                    I18n.t('i18n_tasks.data_stats.text_single_locale', **stats)
-                  else
-                    I18n.t('i18n_tasks.data_stats.text', **stats)
-                  end
-          title = Rainbow(I18n.t('i18n_tasks.data_stats.title', **stats.slice(:locales))).bright
+          text = if stats[:locale_count] == 1
+            I18n.t("i18n_tasks.data_stats.text_single_locale", **stats)
+          else
+            I18n.t("i18n_tasks.data_stats.text", **stats)
+          end
+          title = Rainbow(I18n.t("i18n_tasks.data_stats.title", **stats.slice(:locales))).bright
           print_info "#{Rainbow(title).cyan} #{Rainbow(text).cyan}"
         end
 
         def mv_results(results)
           results.each do |(from, to)|
             if to
-              print_info "#{Rainbow(from).cyan} #{Rainbow('⮕').yellow.bright} #{Rainbow(to).cyan}"
+              print_info "#{Rainbow(from).cyan} #{Rainbow("⮕").yellow.bright} #{Rainbow(to).cyan}"
             else
-              print_info "#{Rainbow(from).red}#{Rainbow(' 🗑').red.bright}"
+              print_info "#{Rainbow(from).red}#{Rainbow(" 🗑").red.bright}"
             end
           end
         end
 
         def cp_results(results)
           results.each do |(from, to)|
-            print_info "#{Rainbow(from).cyan} #{Rainbow('+').yellow.bright} #{Rainbow(to).green}"
+            print_info "#{Rainbow(from).cyan} #{Rainbow("+").yellow.bright} #{Rainbow(to).green}"
           end
         end
 
         def check_normalized_results(non_normalized)
           if non_normalized.empty?
-            print_success 'All data is normalized'
+            print_success "All data is normalized"
             return
           end
-          log_stderr Rainbow('The following data requires normalization:').yellow
+          log_stderr Rainbow("The following data requires normalization:").yellow
           puts non_normalized
-          log_stderr Rainbow('Run `i18n-tasks normalize` to fix').yellow
+          log_stderr Rainbow("Run `i18n-tasks normalize` to fix").yellow
         end
 
         private
@@ -116,7 +133,7 @@ module I18n
           when :missing_used
             first_occurrence leaf
           when :missing_plural
-            leaf[:data][:missing_keys].join(', ')
+            leaf[:data][:missing_keys].join(", ")
           else
             "#{Rainbow(leaf[:data][:missing_diff_locale]).cyan} " \
             "#{format_value(leaf[:value].is_a?(String) ? leaf[:value].strip : leaf[:value])}"
@@ -127,16 +144,16 @@ module I18n
           if data[:ref_info]
             from, to = data[:ref_info]
             resolved = key[0...to.length]
-            after    = key[to.length..]
+            after = key[to.length..]
             "  #{Rainbow(from).yellow}#{Rainbow(after).cyan}\n" \
-              "#{Rainbow('⮕').yellow.bright} #{Rainbow(resolved).yellow.bright}"
+              "#{Rainbow("⮕").yellow.bright} #{Rainbow(resolved).yellow.bright}"
           else
             Rainbow(key).cyan
           end
         end
 
         def format_value(val)
-          val.is_a?(Symbol) ? "#{Rainbow('⮕ ').yellow.bright}#{Rainbow(val).yellow}" : val.to_s.strip
+          val.is_a?(Symbol) ? "#{Rainbow("⮕ ").yellow.bright}#{Rainbow(val).yellow}" : val.to_s.strip
         end
 
         def format_reference_desc(node_data)
@@ -144,19 +161,19 @@ module I18n
 
           case node_data[:ref_type]
           when :reference_usage
-            Rainbow('(ref)').yellow.bright
+            Rainbow("(ref)").yellow.bright
           when :reference_usage_resolved
-            Rainbow('(resolved ref)').yellow.bright
+            Rainbow("(resolved ref)").yellow.bright
           when :reference_usage_key
-            Rainbow('(ref key)').yellow.bright
+            Rainbow("(ref key)").yellow.bright
           end
         end
 
         def print_occurrences(node, full_key = node.full_key)
           occurrences = node.data[:occurrences]
           puts [Rainbow(full_key).bright,
-                format_reference_desc(node.data),
-                (Rainbow(occurrences.size).green if occurrences.size > 1)].compact.join ' '
+            format_reference_desc(node.data),
+            (Rainbow(occurrences.size).green if occurrences.size > 1)].compact.join " "
           occurrences.each do |occurrence|
             puts "  #{key_occurrence full_key, occurrence}"
           end
@@ -164,25 +181,25 @@ module I18n
 
         def print_locale_key_value_data_table(locale_key_value_datas)
           if locale_key_value_datas.present?
-            print_table headings: [Rainbow(I18n.t('i18n_tasks.common.locale')).cyan.bright,
-                                   Rainbow(I18n.t('i18n_tasks.common.key')).cyan.bright,
-                                   I18n.t('i18n_tasks.common.value')] do |t|
+            print_table headings: [Rainbow(I18n.t("i18n_tasks.common.locale")).cyan.bright,
+              Rainbow(I18n.t("i18n_tasks.common.key")).cyan.bright,
+              I18n.t("i18n_tasks.common.value")] do |t|
               t.rows = locale_key_value_datas.map do |(locale, k, v, data)|
-                [{ value: Rainbow(locale).cyan, alignment: :center }, format_key(k, data), format_value(v)]
+                [{value: Rainbow(locale).cyan, alignment: :center}, format_key(k, data), format_value(v)]
               end
             end
           else
-            puts 'ø'
+            puts "ø"
           end
         end
 
         def print_title(title)
-          log_stderr "#{Rainbow(title.strip).bright} #{Rainbow('|').faint} " \
+          log_stderr "#{Rainbow(title.strip).bright} #{Rainbow("|").faint} " \
                      "#{"i18n-tasks v#{I18n::Tasks::VERSION}"}"
         end
 
         def print_success(message)
-          log_stderr Rainbow("✓ #{I18n.t('i18n_tasks.cmd.encourage').sample} #{message}").green.bright
+          log_stderr Rainbow("✓ #{I18n.t("i18n_tasks.cmd.encourage").sample} #{message}").green.bright
         end
 
         def print_error(message)
@@ -194,16 +211,16 @@ module I18n
         end
 
         def indent(txt, n = 2)
-          txt.gsub(/^/, ' ' * n)
+          txt.gsub(/^/, " " * n)
         end
 
-        def print_table(opts, &block)
-          puts ::Terminal::Table.new(opts, &block)
+        def print_table(opts, &)
+          puts ::Terminal::Table.new(opts, &)
         end
 
         def key_occurrence(full_key, occurrence)
           location = Rainbow("#{occurrence.path}:#{occurrence.line_num}").green
-          source   = highlight_key(occurrence.raw_key || full_key, occurrence.line, occurrence.line_pos..-1).strip
+          source = highlight_key(occurrence.raw_key || full_key, occurrence.line, occurrence.line_pos..-1).strip
           "#{location} #{source}"
         end
 
@@ -214,8 +231,8 @@ module I18n
           first = occurrences.first
           [
             Rainbow("#{first.path}:#{first.line_num}").green,
-            ("(#{I18n.t 'i18n_tasks.common.n_more', count: occurrences.length - 1})" if occurrences.length > 1)
-          ].compact.join(' ')
+            ("(#{I18n.t "i18n_tasks.common.n_more", count: occurrences.length - 1})" if occurrences.length > 1)
+          ].compact.join(" ")
         end
 
         def highlight_key(full_key, line, range = (0..-1))

data/lib/i18n/tasks/scanners/ast_matchers/base_matcher.rb

@@ -7,7 +7,7 @@ module I18n::Tasks::Scanners::AstMatchers
     end
 
     def convert_to_key_occurrences(send_node, _method_name, location: send_node.loc)
-      fail('Not implemented')
+      fail("Not implemented")
     end
 
     protected
@@ -32,7 +32,7 @@ module I18n::Tasks::Scanners::AstMatchers
       elsif %i[true false].include?(node.type)
         node.type.to_s
       elsif node.type == :nil
-        ''
+        ""
       elsif node.type == :array && array_join_with
         extract_array_as_string(
           node,
@@ -109,7 +109,7 @@ module I18n::Tasks::Scanners::AstMatchers
       if array_reject_blank
         children_strings.reject! do |x|
           # empty strings and nils in the scope argument are ignored by i18n
-          x == ''
+          x == ""
         end
       end
       children_strings.join(array_join_with)

data/lib/i18n/tasks/scanners/ast_matchers/default_i18n_subject_matcher.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/ast_matchers/base_matcher'
-require 'i18n/tasks/scanners/results/occurrence'
+require "i18n/tasks/scanners/ast_matchers/base_matcher"
+require "i18n/tasks/scanners/results/occurrence"
 
 module I18n::Tasks::Scanners::AstMatchers
   class DefaultI18nSubjectMatcher < BaseMatcher
@@ -10,7 +10,7 @@ module I18n::Tasks::Scanners::AstMatchers
       return unless children[1] == :default_i18n_subject
 
       key = @scanner.absolute_key(
-        '.subject',
+        ".subject",
         location.expression.source_buffer.name,
         calling_method: method_name
       )

data/lib/i18n/tasks/scanners/ast_matchers/message_receivers_matcher.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/ast_matchers/base_matcher'
-require 'i18n/tasks/scanners/results/occurrence'
+require "i18n/tasks/scanners/ast_matchers/base_matcher"
+require "i18n/tasks/scanners/results/occurrence"
 
 module I18n::Tasks::Scanners::AstMatchers
   class MessageReceiversMatcher < BaseMatcher
@@ -64,25 +64,25 @@ module I18n::Tasks::Scanners::AstMatchers
     def process_options(node:, key:)
       return [key, nil] if node&.type != :hash
 
-      scope_node = extract_hash_pair(node, 'scope')
+      scope_node = extract_hash_pair(node, "scope")
 
       if scope_node
         scope = extract_string(
           scope_node.children[1],
-          array_join_with: '.',
+          array_join_with: ".",
           array_flatten: true,
           array_reject_blank: true
         )
         return nil if scope.nil? && scope_node.type != :nil
 
-        key = [scope, key].join('.') unless scope == ''
+        key = [scope, key].join(".") unless scope == ""
       end
-      if default_arg_node = extract_hash_pair(node, 'default')
+      if (default_arg_node = extract_hash_pair(node, "default"))
         default_arg = if default_arg_node.children[1]&.type == :hash
-                        extract_hash(default_arg_node.children[1])
-                      else
-                        extract_string(default_arg_node.children[1])
-                      end
+          extract_hash(default_arg_node.children[1])
+        else
+          extract_string(default_arg_node.children[1])
+        end
       end
 
       [key, default_arg]

data/lib/i18n/tasks/scanners/ast_matchers/rails_model_matcher.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/results/occurrence'
+require "i18n/tasks/scanners/results/occurrence"
 
 module I18n::Tasks::Scanners::AstMatchers
   class RailsModelMatcher < BaseMatcher

data/lib/i18n/tasks/scanners/erb_ast_scanner.rb

@@ -1,16 +1,21 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/ruby_ast_scanner'
-require 'i18n/tasks/scanners/local_ruby_parser'
+require "i18n/tasks/scanners/ruby_scanner"
+require "i18n/tasks/scanners/local_ruby_parser"
+require "i18n/tasks/scanners/occurrence_from_position"
+require "prism"
 
 module I18n::Tasks::Scanners
-  # Scan for I18n.translate calls in ERB-file better-html and ASTs
-  class ErbAstScanner < RubyAstScanner
-    DEFAULT_REGEXP = /<%(={1,2}|-|\#|%)?(.*?)([-=])?%>/m.freeze
+  # Scan for I18n.translate calls in ERB-file using regexp and Parser/Prism
+  class ErbAstScanner < RubyScanner
+    include OccurrenceFromPosition
 
-    def initialize(**args)
-      super(**args)
-      @ruby_parser = LocalRubyParser.new(ignore_blocks: true)
+    DEFAULT_REGEXP = /<%(={1,2}|-|\#-?|%)?(.*?)([-=])?%>/m
+
+    # Parser scanner, method called in RubyScanner
+    def ast_parser_parse_file(path)
+      @ruby_parser ||= LocalRubyParser.new(ignore_blocks: true)
+      super
     end
 
     private
@@ -34,11 +39,11 @@ module I18n::Tasks::Scanners
           stop = match.end(0) - 2 - (tailch&.size || 0)
 
           case character
-          when '=', nil, '-'
+          when "=", nil, "-"
             parsed, parsed_comments = handle_code(buffer, code, start, stop)
             comments.concat(parsed_comments)
             children << parsed unless parsed.nil?
-          when '#', '#-'
+          when "#", "#-"
             comments << handle_comment(buffer, start, stop)
           end
         end
@@ -74,5 +79,59 @@ module I18n::Tasks::Scanners
         )
       ::Parser::AST::Node.new(:erb, children, location: location)
     end
+
+    # Prism scanner, method called in RubyScanner
+    def prism_parse_file(path)
+      occurrences = []
+      content = File.read(path)
+
+      content.scan(DEFAULT_REGEXP) do |indicator, code, _tailch, _rspace|
+        match = Regexp.last_match
+        character = indicator ? indicator[0] : nil
+        start = match.begin(0) + 2 + (character&.size || 0)
+
+        case character
+        when "=", nil, "-"
+          occurrences += process_code(path, code, content, start)
+        when "#", "#-"
+          occurrences += process_comments(path, code, content, start)
+        end
+      end
+
+      occurrences
+    end
+
+    def process_code(path, code, content, start)
+      return [] if code.strip.empty? # skip empty ERB tags
+
+      process_prism_results(path, Prism.parse(code)).map do |key, occurrence|
+        [
+          key,
+          occurrence_from_position(
+            path,
+            content,
+            start + occurrence.pos,
+            raw_key: occurrence.raw_key
+          )
+        ]
+      end
+    end
+
+    def process_comments(path, code, content, start)
+      return [] if code.strip.empty?
+
+      parsed = Prism.parse(code.gsub("i18n-tasks-use ", "#i18n-tasks-use "))
+      process_prism_results(path, parsed).map do |key, occurrence|
+        [
+          key,
+          occurrence_from_position(
+            path,
+            content,
+            start + (code.index(key) || occurrence.pos),
+            raw_key: occurrence.raw_key
+          )
+        ]
+      end
+    end
   end
 end

data/lib/i18n/tasks/scanners/file_scanner.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/scanner'
+require "i18n/tasks/scanners/scanner"
 
 module I18n::Tasks::Scanners
   # A base class for a scanner that analyses files.
@@ -16,7 +16,7 @@ module I18n::Tasks::Scanners
       file_reader: Files::CachingFileReader.new
     )
       super()
-      @config      = config
+      @config = config
       @file_reader = file_reader
       @file_finder = file_finder_provider.get(**config.slice(:paths, :only, :exclude))
     end
@@ -36,7 +36,7 @@ module I18n::Tasks::Scanners
     #
     # @return [Array<[key, Results::KeyOccurrence]>] each occurrence found in the file
     def scan_file(_path)
-      fail 'Unimplemented'
+      fail "Unimplemented"
     end
 
     # Read a file. Reads of the same path are cached.
@@ -53,8 +53,8 @@ module I18n::Tasks::Scanners
     # @param (see FileFinder#traverse_files)
     # @yieldparam (see FileFinder#traverse_files)
     # @return (see FileFinder#traverse_files)
-    def traverse_files(&block)
-      @file_finder.traverse_files(&block)
+    def traverse_files(&)
+      @file_finder.traverse_files(&)
     end
 
     # @note This method is cached, it will only access the filesystem on the first invocation.

data/lib/i18n/tasks/scanners/files/caching_file_finder.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/concurrent/cached_value'
-require 'i18n/tasks/scanners/files/file_finder'
+require "i18n/tasks/concurrent/cached_value"
+require "i18n/tasks/scanners/files/file_finder"
 
 module I18n::Tasks::Scanners::Files
   # Finds the files in the specified search paths with support for exclusion / inclusion patterns.
@@ -23,7 +23,7 @@ module I18n::Tasks::Scanners::Files
     # @yieldparam (see FileFinder#traverse_files)
     # @return (see FileFinder#traverse_files)
 
-    alias uncached_find_files find_files
+    alias_method :uncached_find_files, :find_files
     private :uncached_find_files
 
     # @note This method is cached, it will only access the filesystem on the first invocation.

data/lib/i18n/tasks/scanners/files/caching_file_finder_provider.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/concurrent/cache'
-require 'i18n/tasks/scanners/files/caching_file_finder'
+require "i18n/tasks/concurrent/cache"
+require "i18n/tasks/scanners/files/caching_file_finder"
 
 module I18n::Tasks::Scanners::Files
   # Finds the files and provides their contents.
@@ -12,7 +12,7 @@ module I18n::Tasks::Scanners::Files
     # @param exclude [Array<String>]
     def initialize(exclude: [])
       @cache = ::I18n::Tasks::Concurrent::Cache.new
-      @defaults = { exclude: exclude }
+      @defaults = {exclude: exclude}
     end
 
     # Initialize a {CachingFileFinder} or get one from cache based on the constructor arguments.

data/lib/i18n/tasks/scanners/files/caching_file_reader.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/concurrent/cache'
-require 'i18n/tasks/scanners/files/file_reader'
+require "i18n/tasks/concurrent/cache"
+require "i18n/tasks/scanners/files/file_reader"
 
 module I18n::Tasks::Scanners::Files
   # Reads the files in 'rb' mode and UTF-8 encoding.

data/lib/i18n/tasks/scanners/files/file_finder.rb

@@ -13,10 +13,10 @@ module I18n::Tasks::Scanners::Files
     #     Files not matching any of the inclusion patterns will be excluded.
     # @param exclude [Arry<String>] {File.fnmatch}-compatible patterns of files to exclude.
     #     Files matching any of the exclusion patterns will be excluded even if they match an inclusion pattern.
-    def initialize(paths: ['.'], only: nil, exclude: [])
-      fail 'paths argument is required' if paths.nil?
+    def initialize(paths: ["."], only: nil, exclude: [])
+      fail "paths argument is required" if paths.nil?
 
-      @paths   = paths
+      @paths = paths
       @include = only
       @exclude = exclude || []
     end
@@ -26,8 +26,8 @@ module I18n::Tasks::Scanners::Files
     # @yield [path]
     # @yieldparam path [String] the path of the found file.
     # @return [Array<of block results>]
-    def traverse_files(&block)
-      find_files.map(&block)
+    def traverse_files(&)
+      find_files.map(&)
     end
 
     # @return [Array<String>] found files
@@ -36,10 +36,10 @@ module I18n::Tasks::Scanners::Files
       paths = @paths.select { |p| File.exist?(p) }
       log_warn "None of the search.paths exist #{@paths.inspect}" if paths.empty?
       Find.find(*paths) do |path|
-        is_dir   = File.directory?(path)
-        hidden   = File.basename(path).start_with?('.') && !%w[. ./].include?(path)
+        is_dir = File.directory?(path)
+        hidden = File.basename(path).start_with?(".") && !%w[. ./].include?(path)
         not_incl = @include && !path_fnmatch_any?(path, @include)
-        excl     = path_fnmatch_any?(path, @exclude)
+        excl = path_fnmatch_any?(path, @exclude)
         if is_dir || hidden || not_incl || excl
           Find.prune if is_dir && (hidden || excl)
         else

data/lib/i18n/tasks/scanners/files/file_reader.rb

@@ -12,7 +12,7 @@ module I18n::Tasks::Scanners::Files
     # @return [String] file contents
     def read_file(path)
       result = nil
-      File.open(path, 'rb', encoding: 'UTF-8') { |f| result = f.read }
+      File.open(path, "rb", encoding: "UTF-8") { |f| result = f.read }
       result
     end
   end

data/lib/i18n/tasks/scanners/local_ruby_parser.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require 'i18n/tasks/scanners/ruby_parser_factory'
+require "i18n/tasks/scanners/ruby_parser_factory"
 
 module I18n::Tasks::Scanners
   class LocalRubyParser
     # ignore_blocks feature inspired by shopify/better-html
     # https://github.com/Shopify/better-html/blob/087943ffd2a5877fa977d71532010b0c91239519/lib/better_html/test_helper/ruby_node.rb#L24
-    BLOCK_EXPR = /\s*((\s+|\))do|\{)(\s*\|[^|]*\|)?\s*\Z/.freeze
+    BLOCK_EXPR = /\s*((\s+|\))do|\{)(\s*\|[^|]*\|)?\s*\Z/
 
     def initialize(ignore_blocks: false)
       @parser = RubyParserFactory.create_parser
@@ -15,12 +15,12 @@ module I18n::Tasks::Scanners
 
     # Parse string and normalize location
     def parse(source, location: nil)
-      buffer = ::Parser::Source::Buffer.new('(string)')
+      buffer = ::Parser::Source::Buffer.new("(string)")
       buffer.source = if @ignore_blocks
-                        source.sub(BLOCK_EXPR, '')
-                      else
-                        source
-                      end
+        source.sub(BLOCK_EXPR, "")
+      else
+        source
+      end
 
       @parser.reset
       ast, comments = @parser.parse_with_comments(buffer)
@@ -42,7 +42,7 @@ module I18n::Tasks::Scanners
       node.updated(
         nil,
         node.children.map { |child| normalize_location(child, location) },
-        { location: updated_location(location, node.location) }
+        {location: updated_location(location, node.location)}
       )
     end
 

data/lib/i18n/tasks/scanners/occurrence_from_position.rb

@@ -13,7 +13,7 @@ module I18n
         # @return [Results::Occurrence]
         def occurrence_from_position(path, contents, position, raw_key: nil)
           line_begin = contents.rindex(/^/, position - 1)
-          line_end   = contents.index(/.(?=\r?\n|$)/, position)
+          line_end = contents.index(/.(?=\r?\n|$)/, position)
           Results::Occurrence.new(
             path: path,
             pos: position,