ruby_friendly_error 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a41f6d667be9cadbacddde143ff345d2d145e6d0e2d5c0c42249eceb1918b2d
-  data.tar.gz: 1901ec22809369187299906734c171be17ba0e3b4ff2e0a2597b151b1456638c
+  metadata.gz: 4f4b1bac560baef1e3575b2d65a0d0cd90c40f3daba2219b2d0b6dc6eef9b06e
+  data.tar.gz: d3baae55366188eb7df5d5cd9bbdadfc8273ac90e821cb31aa85404867277668
 SHA512:
-  metadata.gz: 8d68f65c5654d7584b6a4231345ad41f2146e2f8f09358b446dafe85fa82627e5ec7e3ab150b0c3a4c3b8496b0849b5bde1d3c48cf81baf36bab28b387861722
-  data.tar.gz: 565906dd0162aac5eb29094334519600354d527df8bc44d5811342ad5b1157fee14a9fed0a582aee73a4adbc354e388a1a22be4e1f153aa7822c403bee6bb6b7
+  metadata.gz: b47bafe16356f7ae7c875350f81c756a3f28cce4d7e58fdf0a7002b0a6688e7fc1824d562f5229fcf35f87441b083f1e18640a218d1ede235b2753ac30bc0df3
+  data.tar.gz: d5db46fa860dcaf072d0a9d726088222ce945e7ae10b6c26085d2ac62f94353df8a2a559361997997cd84274e1bf7f03ccdebc644c58ff5978cf074cc772b99b

data/.gitignore

@@ -6,5 +6,7 @@
 /tmp/
 /vendor/
 
+ruby_friendly_error-*.gem
+
 # rspec failure tracking
 .rspec_status

data/.rubocop_todo.yml

@@ -1,3 +1,3 @@
-Metrics/ModuleLength:
+Metrics/ClassLength:
   Exclude:
-    - lib/ruby_friendly_error.rb
+    - lib/ruby_friendly_error/renderers/exception_renderer.rb

data/Gemfile.lock

@@ -1,10 +1,10 @@
 PATH
   remote: .
   specs:
-    ruby_friendly_error (0.0.1)
+    ruby_friendly_error (0.0.2)
       colorize (~> 0.8)
       i18n (~> 1.1)
-      parser (~> 2.5.1)
+      parser (~> 2.5)
 
 GEM
   remote: https://rubygems.org/

data/README.adoc

@@ -6,7 +6,7 @@
 :sectnums:
 :source-highlighter: highlightjs
 :toc: left
-:toclevels: 1
+:toclevels: 2
 
 = RubyFriendlyError image:https://img.shields.io/badge/ruby-2.4.4-cc342d.svg["ruby 2.4.4", link="https://www.ruby-lang.org/en/news/2018/03/28/ruby-2-4-4-released/"] image:https://img.shields.io/badge/ruby-2.5.1-cc342d.svg["ruby 2.5.1", link="https://www.ruby-lang.org/en/news/2018/03/28/ruby-2-5-1-released/"]
 
@@ -24,6 +24,70 @@ gem 'ruby_friendly_error'
 $ bundle exec ruby_friendly_error your.rb
 ```
 
+== Samples
+
+=== miss spell
+
+[source,ruby]
+.sample.rb
+----
+# frozen_string_literal: true
+
+def hoge prayer_life = 100 , player_lifee = 200
+  puts 'hoge' if player_life > 0
+end
+
+hoge
+----
+
+```sh
+$ bundle exec ruby_friendly_error sample.rb
+```
+
+image:https://raw.githubusercontent.com/isuke/ruby_friendly_error/images/name_error_with_did_you_mean.png["name_error_with_did_you_mean", caption="output"]
+
+=== miss args num
+
+[source,ruby]
+.sample.rb
+----
+# frozen_string_literal: true
+
+def hoge arg1, arg2 = 'foobar'
+  puts arg1
+  puts arg2
+end
+
+hoge 'piyo', 'fuga', 'what!?'
+----
+
+```sh
+$ bundle exec ruby_friendly_error sample.rb
+```
+
+image:https://raw.githubusercontent.com/isuke/ruby_friendly_error/images/wrong_number_of_arguments_error.png["wrong_number_of_arguments_error", caption="output"]
+
+== Options
+
+[cols="1,1,1", options="header"]
+|===
+| key
+| description
+| values
+
+| RUBY_FRIENDLY_ERROR_LANG
+| message language.
+| en, ja
+|===
+
+=== RUBY_FRIENDLY_ERROR_LANG
+
+```sh
+$ RUBY_FRIENDLY_ERROR_LANG=ja bundle exec ruby_friendly_error sample.rb
+```
+
+image:https://raw.githubusercontent.com/isuke/ruby_friendly_error/images/wrong_number_of_arguments_error_ja.png["wrong_number_of_arguments_error_ja", caption="output"]
+
 == Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

data/exe/ruby_friendly_error

@@ -3,4 +3,4 @@
 
 require 'ruby_friendly_error'
 
-RubyFriendlyError.load ARGV[0]
+RubyFriendlyError.load ARGV[0], ENV['RUBY_FRIENDLY_ERROR_LANG']&.to_sym

data/lib/ruby_friendly_error/ast.rb

@@ -35,6 +35,20 @@ class Parser::AST::Node
     KEYWORD_REST_ARG_TYPE,
   ].freeze
 
+  REQURED_ARG_TYPES = [
+    ARG_TYPE,
+  ].freeze
+
+  OPTIONAL_ARG_TYPES = [
+    OPTION_ARG_TYPE,
+    KEYWORD_ARG_OPTION_TYPE,
+  ].freeze
+
+  DEF_TYPE  = :def
+  SEND_TYPE = :send
+
+  NOT_HAVE_LINE_TYPES = %i[args].freeze
+
   def find_by_variable_name variable_name
     to_a.each do |node|
       next unless node.is_a? Parser::AST::Node
@@ -42,9 +56,47 @@ class Parser::AST::Node
         return node if node.to_a[0].to_s.sub(/^@*/, '') == variable_name.to_s
       end
 
-      result = node.find_by_variable_name(variable_name)
+      result = node.find_by_variable_name variable_name
+      return result if result
+    end
+    nil
+  end
+
+  def find_by_line_number line_number
+    to_a.each do |node|
+      next unless node.is_a? Parser::AST::Node
+      next if NOT_HAVE_LINE_TYPES.include? node.type
+      return node if node.location.line == line_number
+      result = node.find_by_line_number line_number
+      return result if result
+    end
+    nil
+  end
+
+  def find_send_by_def_name def_name
+    to_a.each do |node|
+      next unless node.is_a? Parser::AST::Node
+
+      return node if node.type == SEND_TYPE && node.to_a[1].to_s == def_name.to_s
+
+      result = node.find_send_by_def_name def_name
       return result if result
     end
     nil
   end
+
+  def def_args_num_range
+    return nil unless type == DEF_TYPE
+
+    required_arg_num  = to_a[1].to_a.select { |n| REQURED_ARG_TYPES.include? n.type }.size
+    optionarl_arg_num = to_a[1].to_a.select { |n| OPTIONAL_ARG_TYPES.include? n.type }.size
+
+    (required_arg_num..(required_arg_num + optionarl_arg_num))
+  end
+
+  def send_args_num
+    return nil unless type == SEND_TYPE
+
+    to_a.size - 2
+  end
 end

data/lib/ruby_friendly_error/renderers/argument_error_renderer.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class ArgumentErrorRenderer < StandardErrorRenderer
+  private
+
+    def exception_i18n_name
+      I18n.t 'standard_error.argument_error.title'
+    end
+
+    def def_node
+      @def_node ||= ast.find_by_line_number error_line_number
+    end
+
+    def def_name
+      @def_name ||= def_node.to_a[0]
+    end
+
+    def send_node
+      @send_node ||= ast.find_send_by_def_name def_name
+    end
+
+    def error_message
+      args_num_range = def_node.def_args_num_range
+
+      args_num_str = args_num_range.size == 1 ? args_num_range.first.to_s : args_num_range.to_s
+
+      I18n.t 'standard_error.argument_error.wrong_number_of_arguments', def_name: def_name, arg_range: args_num_str, send_arg: send_node.send_args_num
+    end
+
+    def display_error_detail
+      display_error_line_string def_node.location.line, strong_pos_range: ((def_node.location.column + 1)..def_node.to_a[1].location.last_column)
+
+      display_error_line_string send_node.location.line, strong_pos_range: ((send_node.location.column + 1)..send_node.location.last_column)
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/exception_renderer.rb

@@ -0,0 +1,136 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class ExceptionRenderer
+    attr_reader :exception
+    attr_reader :eval_file_content
+
+    def initialize exception, eval_file_content
+      @exception         = exception
+      @eval_file_content = eval_file_content
+    end
+
+    def render
+      display_backtrace exception.backtrace unless eval?
+
+      display_main_message
+
+      display_error_detail
+
+      display_message
+    end
+
+  private
+
+    def file_name
+      @file_name ||= File.basename(file_path)
+    end
+
+    def file_path
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def file_expand_path
+      @file_expand_path ||= File.expand_path(file_path)
+    end
+
+    def file_content
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def eval?
+      file_name == '(eval)'
+    end
+
+    def exception_i18n_name
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def error_line_number
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def error_message
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def display_backtrace backtrace
+      fake_backtrace = backtrace
+        .dup
+        .delete_if { |b| b.match(/ruby_friendly_error/) }
+        .delete_if { |b| b.match(/`block in exec'/) }
+        .map do |b|
+          if b.match? Regexp.new("#{@file_name}.*`exec'")
+            b.sub("`exec'", "`<main>'")
+          else
+            b
+          end
+        end
+
+      fake_backtrace_size   = fake_backtrace.size
+      fake_backtrace_digits = fake_backtrace_size.to_s.length
+
+      fake_backtrace.reverse_each.with_index do |b, i|
+        num        = fake_backtrace_size - i
+        num_digits = num.to_s.length
+        color      = b.include?(@file_name) ? :light_red : :light_yellow
+        STDERR.puts "#{' ' * (fake_backtrace_digits - num_digits)}#{num}: #{b}".colorize(color)
+      end
+      STDERR.puts
+    end
+
+    def display_main_message
+      STDERR.puts "#{exception_i18n_name.underline}#{I18n.t('main_message')}: #{file_path}:#{error_line_number}".colorize(:light_red)
+      STDERR.puts
+    end
+
+    def display_error_detail
+      raise NotImplementedError, "You must implement #{__method__}"
+    end
+
+    def display_message
+      STDERR.puts "#{exception_i18n_name.underline}:".colorize(:light_red)
+      STDERR.puts format_line_strings(error_message) { |l, _i| "  #{l}" }.colorize(:light_red)
+    end
+
+    def display_error_line_string line_number = error_line_number, strong_pos_range: nil, window: RubyFriendlyError::WINDOW
+      line_strings   = file_content.split("\n", -1)
+      file_line_size = line_strings.size
+      window_start   = [line_number - window, 1].max
+      window_end     = [line_number + window, file_line_size].min
+
+      STDERR.puts RubyFriendlyError::DISPLAY_START.colorize(:light_yellow)
+
+      display_error_line_string_window line_strings, window_start, line_number - 1 if line_number > 1
+
+      lint_string =
+        if strong_pos_range
+          RubyFriendlyError::Utils.replace_string_at(line_strings[line_number - 1], strong_pos_range, &:underline)
+        else
+          line_strings[line_number - 1]
+        end
+      STDERR.puts "#{line_number}: #{lint_string}".rstrip.colorize(:light_red)
+
+      display_error_line_string_window line_strings, line_number + 1, window_end if line_number < file_line_size
+
+      STDERR.puts RubyFriendlyError::DISPLAY_END.colorize(:light_yellow)
+      STDERR.puts
+    end
+
+    def display_error_line_string_window line_strings, start_line_number, end_line_number
+      STDERR.puts format_line_strings(line_strings, start_line_number, end_line_number) { |l, i| "#{start_line_number + i}: #{l}" }&.colorize(:light_yellow)
+    end
+
+    def format_line_strings line_strings, start_line = 1, end_line = 0
+      line_strings =
+        case line_strings
+          when String then line_strings.split("\n", -1)
+          when Array  then line_strings
+        end
+      line_strings[(start_line - 1)..(end_line - 1)]
+        &.map
+        &.with_index { |l, i| block_given? ? yield(l, i).rstrip : l.rstrip }
+        &.join("\n")
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/missing_end_error_renderer.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class MissingEndErrorRenderer < SyntaxErrorRenderer
+  private
+
+    def error_message
+      I18n.t('syntax_error.missing_end')
+    end
+
+    def display_error_detail
+      display_error_line_string
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/name_error_renderer.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class NameEndErrorRenderer < StandardErrorRenderer
+  private
+
+    def exception_i18n_name
+      I18n.t 'standard_error.name_error.title'
+    end
+
+    def corrections
+      @corrections ||= exception.spell_checker.corrections
+    end
+
+    def error_message
+      var_name_str    = exception.spell_checker.name.underline
+      corrections_str = corrections.map { |c| "`#{c.to_s.underline}`" }.join(', ')
+      I18n.t('standard_error.name_error.with_did_you_mean', var_name: var_name_str, corrections: corrections_str)
+    end
+
+    def display_error_detail
+      corrections.each do |var_name|
+        node_location = ast.find_by_variable_name(var_name).location
+        display_error_line_string node_location.line, strong_pos_range: ((node_location.column + 1)..node_location.last_column)
+      end
+
+      display_error_line_string strong_pos_range: ((error_node.location.column + 1)..error_node.location.last_column)
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/standard_error_renderer.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class StandardErrorRenderer < ExceptionRenderer
+  private
+
+    def ast
+      @ast ||= RubyFriendlyError::Utils.suppress_error_display { Parser::CurrentRuby.parse file_content }
+    end
+
+    def error_node
+      @error_node ||= ast.find_by_line_number error_line_number
+    end
+
+    def file_path
+      @file_path ||= exception.backtrace.first.match(/(.+):[0-9]+:/)[1]
+    end
+
+    def file_content
+      @file_content ||= eval? ? eval_file_content : File.read(file_path)
+    end
+
+    def error_line_number
+      exception.backtrace.first.match(/:([0-9]+):/)[1].to_i
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/syntax_error_renderer.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class SyntaxErrorRenderer < ExceptionRenderer
+  private
+
+    def file_path
+      @file_path ||= exception.message.match(/(.+):[0-9]+:/)[1]
+    end
+
+    def file_content
+      @file_content ||= eval? ? eval_file_content : File.read(file_path)
+    end
+
+    def error_line_number
+      exception.message.match(/:([0-9]+):/)[1].to_i
+    end
+
+    def exception_i18n_name
+      I18n.t 'syntax_error.title'
+    end
+  end
+end

data/lib/ruby_friendly_error/renderers/unnecessary_end_error_renderer.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError::Renderers
+  class UnnecessaryEndErrorRenderer < SyntaxErrorRenderer
+  private
+
+    def error_message
+      I18n.t('syntax_error.unnecessary_end')
+    end
+
+    def display_error_detail
+      display_error_line_string
+    end
+  end
+end

data/lib/ruby_friendly_error/utils.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module RubyFriendlyError
+  class Utils
+    class << self
+      def suppress_error_display
+        original_stdout = $stderr
+        $stderr         = StringIO.new
+
+        yield
+      ensure
+        $stderr = original_stdout
+      end
+
+      def replace_string_at string, range
+        regexp          = Regexp.new "\\A(.{#{range.first - 1}})(.{#{range.size}})(.*)\\Z"
+        match_data      = string.match regexp
+        replaced_string = yield match_data[2]
+        match_data[1] + replaced_string + match_data[3]
+      end
+    end
+  end
+end

data/lib/ruby_friendly_error/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RubyFriendlyError
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 end

data/lib/ruby_friendly_error.rb

@@ -5,8 +5,17 @@ require 'i18n'
 require 'parser/current'
 
 require 'ruby_friendly_error/ast'
+require 'ruby_friendly_error/utils'
 require 'ruby_friendly_error/version'
 
+require 'ruby_friendly_error/renderers/exception_renderer'
+require 'ruby_friendly_error/renderers/syntax_error_renderer'
+require 'ruby_friendly_error/renderers/missing_end_error_renderer'
+require 'ruby_friendly_error/renderers/unnecessary_end_error_renderer'
+require 'ruby_friendly_error/renderers/standard_error_renderer'
+require 'ruby_friendly_error/renderers/name_error_renderer'
+require 'ruby_friendly_error/renderers/argument_error_renderer'
+
 Bundler.require(:development)
 
 Parser::Builders::Default.emit_lambda   = true
@@ -15,138 +24,55 @@ Parser::Builders::Default.emit_encoding = true
 Parser::Builders::Default.emit_index    = true
 
 module RubyFriendlyError
-  ROOT_PATH = Pathname.new(__FILE__).dirname.parent.to_s
-  WINDOW    = 2
+  module Renderers; end
+
+  ROOT_PATH     = Pathname.new(__FILE__).dirname.parent.to_s
+  WINDOW        = 2
+  DISPLAY_START = '<' * 80
+  DISPLAY_END   = '>' * 80
 
   I18n.load_path = Dir[File.join(ROOT_PATH, 'locales', '*.yml')]
   I18n.backend.load_translations
-  I18n.backend.store_translations(:en , YAML.load_file(File.join(ROOT_PATH, 'locales', 'en.yml')))
 
   class << self
-    def load file_path
-      exec File.read(file_path), file_path
+    def load file_path, lang = :en
+      exec File.read(file_path), File.expand_path(file_path), lang
     end
 
-    def exec file_content, file_name = '(eval)'
-      eval file_content, nil, file_name # rubocop:disable Security/Eval
-    rescue Exception => ex # rubocop:disable Lint/RescueException
-      exception_file_name    = file_name == '(eval)' ? file_name : ex.backtrace.first.match(/(.+):[0-9]+:/)[1]
-      exception_file_content = file_name == '(eval)' ? file_content : File.read(exception_file_name)
-      exception_handling ex, exception_file_name, exception_file_content, file_name != '(eval)'
+    def exec file_content, file_name = '(eval)', lang = :en
+      load_i18n lang
+
+      eval file_content, nil, file_name, 1 # rubocop:disable Security/Eval
+    rescue Exception => exception # rubocop:disable Lint/RescueException
+      renderer_class = renderer_class(exception)
+
+      raise exception unless renderer_class
+
+      renderer_class.new(exception, file_content).render
+      exit false
     end
 
   private
 
-    def exception_handling exception, file_name, file_content, displayed_backtrace
-      display_backtrace file_name, exception.backtrace if displayed_backtrace
+    def load_i18n lang
+      I18n.backend.store_translations(lang, YAML.load_file(File.join(ROOT_PATH, 'locales', "#{lang}.yml")))
+      I18n.locale = lang
+    end
 
+    def renderer_class exception
       case exception
         when SyntaxError
           case exception.message
             when /unnecessary `end`/
-              render_unnecessary_end_error file_content, exception
+              RubyFriendlyError::Renderers::UnnecessaryEndErrorRenderer
             when /unexpected end-of-input/
-              render_missing_end_error file_content, exception
+              RubyFriendlyError::Renderers::MissingEndErrorRenderer
           end
         when NameError
-          ast = suppress_error_display { Parser::CurrentRuby.parse file_content }
-          render_name_error_with_did_you_mean file_content, exception, ast
-      end
-
-      raise exception
-    end
-
-    def render_missing_end_error file_content, ex
-      line = ex.message.match(/:([0-9]+):/)[1].to_i
-      display_error_line file_content, line
-      STDERR.puts I18n.t('syntax_error.title').colorize(:light_red) + ':'
-      STDERR.puts format_lines(I18n.t('syntax_error.missing_end')) { |l, _i| "  #{l}" }.colorize(:light_red)
-    end
-
-    def render_unnecessary_end_error file_content, ex
-      line = ex.message.match(/:([0-9]+):/)[1].to_i
-      display_error_line file_content, line
-      STDERR.puts I18n.t('syntax_error.title').colorize(:light_red) + ':'
-      STDERR.puts format_lines(I18n.t('syntax_error.unnecessary_end')) { |l, _i| "  #{l}" }.colorize(:light_red)
-    end
-
-    def render_name_error_with_did_you_mean file_content, ex, ast
-      corrections = ex.spell_checker.corrections
-      corrections.each do |var_name|
-        node = ast.find_by_variable_name var_name
-        display_error_line file_content, node.loc.line
-      end
-
-      STDERR.puts I18n.t('name_error.title').colorize(:light_red) + ':'
-      var_name_str    = ex.spell_checker.name.underline
-      corrections_str = corrections.map { |c| "`#{c.to_s.underline}`" }.join(', ')
-      message         = I18n.t('name_error.with_did_you_mean', var_name: var_name_str, corrections: corrections_str)
-      STDERR.puts format_lines(message) { |l, _i| "  #{l}" }.colorize(:light_red)
-    end
-
-    def suppress_error_display
-      original_stdout = $stderr
-      $stderr         = StringIO.new
-
-      yield
-    ensure
-      $stderr = original_stdout
-    end
-
-    def display_error_line file_content, error_line, window = WINDOW
-      lines        = file_content.split("\n", -1)
-      line_size    = lines.size
-      window_start = [error_line - window, 1].max
-      window_end   = [error_line + window, line_size].min
-
-      if error_line > 1
-        start_line = window_start
-        end_line   = error_line - 1
-        STDERR.puts format_lines(lines, start_line, end_line) { |l, i| "#{start_line + i}: #{l}" }&.colorize(:light_yellow)
-      end
-      STDERR.puts "#{error_line}: #{lines[error_line - 1]}".rstrip.colorize(:light_red)
-      if error_line < line_size
-        start_line = error_line + 1
-        end_line   = window_end
-        STDERR.puts format_lines(lines, error_line + 1, end_line) { |l, i| "#{start_line + i}: #{l}" }&.colorize(:light_yellow)
-      end
-      STDERR.puts
-    end
-
-    def format_lines lines, start_line = 1, end_line = 0
-      lines =
-        case lines
-          when String then lines.split("\n", -1)
-          when Array  then lines
-        end
-      lines[(start_line - 1)..(end_line - 1)]
-        &.map
-        &.with_index { |l, i| block_given? ? yield(l, i).rstrip : l.rstrip }
-        &.join("\n")
-    end
-
-    def display_backtrace file_name, backtrace
-      fake_backtrace = backtrace
-        .dup
-        .delete_if { |b| b.match(/ruby_friendly_error/) }
-        .delete_if { |b| b.match(/`block in exec'/) }
-        .map do |b|
-          if b.match? Regexp.new("#{file_name}.*`exec'")
-            b.sub("`exec'", "`<main>'")
-          else
-            b
-          end
-        end
-
-      fake_backtrace_size   = fake_backtrace.size
-      fake_backtrace_digits = fake_backtrace_size.to_s.length
-
-      fake_backtrace.reverse_each.with_index do |b, i|
-        num        = fake_backtrace_size - i
-        num_digits = num.to_s.length
-        STDERR.puts "#{' ' * (fake_backtrace_digits - num_digits)}#{num}:#{b}".colorize(:light_red)
+          RubyFriendlyError::Renderers::NameEndErrorRenderer
+        when ArgumentError
+          RubyFriendlyError::Renderers::ArgumentErrorRenderer
       end
-      STDERR.puts
     end
   end
 end

data/locales/en.yml

@@ -1,4 +1,5 @@
 en:
+  main_message: " occurred"
   syntax_error:
     title: "syntax error"
     missing_end: |
@@ -7,8 +8,15 @@ en:
     unnecessary_end: |
       exist unnecessary `end`.
       Please remove unnecessary `end`.
-  name_error:
-    title: "name error"
-    with_did_you_mean: |
-      undefined local variable or method `%{var_name}`
-      Did you mean? %{corrections}
+  standard_error:
+    name_error:
+      title: "name error"
+      with_did_you_mean: |
+        undefined local variable or method `%{var_name}`.
+        Did you mean? %{corrections}.
+    argument_error:
+      title: "argument error"
+      wrong_number_of_arguments: |
+        wrong number of arguments.
+        `%{def_name}` expected %{arg_range}.
+        but given %{send_arg}.

data/locales/ja.yml

@@ -0,0 +1,22 @@
+ja:
+  main_message: "が発生しました。"
+  syntax_error:
+    title: "構文エラー"
+    missing_end: |
+      `end` が欠けています。
+      おそらくこの行より前に問題があります。
+    unnecessary_end: |
+      `end` が余分に存在しています。
+      不要な `end` を取り除いてください。
+  standard_error:
+    name_error:
+      title: "名前エラー"
+      with_did_you_mean: |
+        `%{var_name}` という名前の変数、もしくはメソッドが定義されていません。
+        もしかして? %{corrections}。
+    argument_error:
+      title: "引数エラー"
+      wrong_number_of_arguments: |
+        引数の数が誤っています。
+        `%{def_name}` は %{arg_range} 個の引数を期待しています。
+        しかし %{send_arg} 個の引数を渡しています。

data/ruby_friendly_error.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |spec|
   spec.email         = ['isuke770@gmail.com']
 
   spec.summary       = 'make to ruby error messages friendly.'
-  spec.description   = 'make to ruby error messages friendly.'
+  spec.description   = 'make to ruby error messages friendly. Display multilingual message and error lines'
   spec.homepage      = 'https://github.com/isuke/ruby_friendly_error'
   spec.license       = 'MIT'
 
@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency 'colorize', '~> 0.8'
   spec.add_dependency 'i18n'    , '~> 1.1'
-  spec.add_dependency 'parser'  , '~> 2.5.1'
-  spec.add_development_dependency 'bundler'      , '~> 1.16'
-  spec.add_development_dependency 'rake'         , '~> 10.0'
+  spec.add_dependency 'parser'  , '~> 2.5'
+  spec.add_development_dependency 'bundler', '~> 1.16'
+  spec.add_development_dependency 'rake'   , '~> 10.0'
 end

data/samples/arg_error.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+def hoge arg1, arg2 = 'foobar'
+  puts arg1
+  puts arg2
+end
+
+hoge 'piyo', 'fuga', 'what!?'

data/samples/miss_spell1.rb

@@ -1,20 +1,9 @@
 # frozen_string_literal: true
 
-# 1: prayer_life = 100
-# 2: player_lifee = 200
-# 3:
-# 4: puts 'hoge' if player_life > 0
-
-# 1: prayer_life = 100
-# 2: player_lifee = 200
-# 3:
-
-# name error:
-#   undefined local variable or method `player_life`
-#   Did you mean? `player_lifee`, `prayer_life`
-
 prayer_life = 100
 
 player_lifee = 100
 
-puts 'hoge' if player_life > 0
+3.times do
+  puts 'hoge' if player_life > 0
+end

data/samples/not_error.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative 'miss_spell1'
+
 player_life = 100
 
 puts 'hoge' if player_life > 0

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_friendly_error
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - isuke
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2018-09-08 00:00:00.000000000 Z
+date: 2018-09-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.1
+        version: '2.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.5.1
+        version: '2.5'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -80,7 +80,8 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '10.0'
-description: make to ruby error messages friendly.
+description: make to ruby error messages friendly. Display multilingual message and
+  error lines
 email:
 - isuke770@gmail.com
 executables:
@@ -108,9 +109,19 @@ files:
 - exe/ruby_friendly_error
 - lib/ruby_friendly_error.rb
 - lib/ruby_friendly_error/ast.rb
+- lib/ruby_friendly_error/renderers/argument_error_renderer.rb
+- lib/ruby_friendly_error/renderers/exception_renderer.rb
+- lib/ruby_friendly_error/renderers/missing_end_error_renderer.rb
+- lib/ruby_friendly_error/renderers/name_error_renderer.rb
+- lib/ruby_friendly_error/renderers/standard_error_renderer.rb
+- lib/ruby_friendly_error/renderers/syntax_error_renderer.rb
+- lib/ruby_friendly_error/renderers/unnecessary_end_error_renderer.rb
+- lib/ruby_friendly_error/utils.rb
 - lib/ruby_friendly_error/version.rb
 - locales/en.yml
+- locales/ja.yml
 - ruby_friendly_error.gemspec
+- samples/arg_error.rb
 - samples/mismatch_args.rb
 - samples/miss_spell1.rb
 - samples/miss_spell2.rb