byebug 9.1.0 → 10.0.0

data/lib/byebug/helpers/var.rb

@@ -1,4 +1,6 @@
-require 'byebug/helpers/eval'
+# frozen_string_literal: true
+
+require "byebug/helpers/eval"
 
 module Byebug
   module Helpers
@@ -13,7 +15,7 @@ module Byebug
           [name, safe_inspect(silent_eval(name.to_s, binding))]
         end
 
-        puts prv(vars, 'instance')
+        puts prv(vars, "instance")
       end
 
       def var_global
@@ -25,7 +27,7 @@ module Byebug
       end
 
       def var_instance(str)
-        obj = warning_eval(str || 'self')
+        obj = warning_eval(str || "self")
 
         var_list(obj.instance_variables, obj.instance_eval { binding })
       end
@@ -33,8 +35,8 @@ module Byebug
       def var_local
         locals = context.frame.locals
         cur_self = context.frame._self
-        locals[:self] = cur_self unless cur_self.to_s == 'main'
-        puts prv(locals.keys.sort.map { |k| [k, locals[k]] }, 'instance')
+        locals[:self] = cur_self unless cur_self.to_s == "main"
+        puts prv(locals.keys.sort.map { |k| [k, locals[k]] }, "instance")
       end
 
       def var_args
@@ -45,7 +47,7 @@ module Byebug
         arg_values = args.map { |arg| arg[1] }
 
         locals = all_locals.select { |k, _| arg_values.include?(k) }
-        puts prv(locals.keys.sort.map { |k| [k, locals[k]] }, 'instance')
+        puts prv(locals.keys.sort.map { |k| [k, locals[k]] }, "instance")
       end
     end
   end

data/lib/byebug/history.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 begin
-  require 'readline'
+  require "readline"
 rescue LoadError
-  warn <<-EOW
+  warn <<-MESSAGE
     Sorry, you can't use byebug without Readline. To solve this, you need to
     rebuild Ruby with Readline support. If using Ubuntu, try `sudo apt-get
     install libreadline-dev` and then reinstall your Ruby.
-  EOW
+  MESSAGE
 
   raise
 end
@@ -43,7 +45,7 @@ module Byebug
     def save
       n_cmds = Setting[:histsize] > size ? size : Setting[:histsize]
 
-      open(Setting[:histfile], 'w') do |file|
+      open(Setting[:histfile], "w") do |file|
         n_cmds.times { file.puts(pop) }
       end
 
@@ -84,7 +86,7 @@ module Byebug
       commands = buffer.last(show_size)
 
       last_ids(show_size).zip(commands).map do |l|
-        format('%5d  %s', l[0], l[1])
+        format("%<position>5d  %<command>s", position: l[0], command: l[1])
       end.join("\n") + "\n"
     end
 

data/lib/byebug/interface.rb

@@ -1,6 +1,8 @@
-require 'byebug/setting'
-require 'byebug/history'
-require 'byebug/helpers/file'
+# frozen_string_literal: true
+
+require "byebug/setting"
+require "byebug/history"
+require "byebug/helpers/file"
 
 #
 # Namespace for all of byebug's code
@@ -20,7 +22,7 @@ module Byebug
     def initialize
       @command_queue = []
       @history = History.new
-      @last_line = ''
+      @last_line = ""
     end
 
     def last_if_empty(input)
@@ -97,7 +99,7 @@ module Byebug
     # Confirms user introduced an affirmative response to the input stream.
     #
     def confirm(prompt)
-      readline(prompt) == 'y'
+      readline(prompt) == "y"
     end
 
     def close
@@ -124,7 +126,7 @@ module Byebug
     # array of commands: [cmd1, cmd2, ..., cmdN]
     #
     def split_commands(cmd_line)
-      return [''] if cmd_line.empty?
+      return [""] if cmd_line.empty?
 
       cmd_line.split(/;/).each_with_object([]) do |v, m|
         if m.empty? || m.last[-1] != '\\'
@@ -132,13 +134,13 @@ module Byebug
           next
         end
 
-        m.last[-1, 1] = ''
-        m.last << ';' << v
+        m.last[-1, 1] = ""
+        m.last << ";" << v
       end
     end
   end
 end
 
-require 'byebug/interfaces/local_interface'
-require 'byebug/interfaces/script_interface'
-require 'byebug/interfaces/remote_interface'
+require "byebug/interfaces/local_interface"
+require "byebug/interfaces/script_interface"
+require "byebug/interfaces/remote_interface"

data/lib/byebug/interfaces/local_interface.rb

@@ -5,7 +5,7 @@ module Byebug
   # Interface class for standard byebug use.
   #
   class LocalInterface < Interface
-    EOF_ALIAS = 'continue'.freeze
+    EOF_ALIAS = "continue".freeze
 
     def initialize
       super()
@@ -32,13 +32,13 @@ module Byebug
     # @note Any external 'INT' traps are overriden during this method.
     #
     def with_repl_like_sigint
-      orig_handler = trap('INT') { raise Interrupt }
+      orig_handler = trap("INT") { raise Interrupt }
       yield
     rescue Interrupt
-      puts('^C')
+      puts("^C")
       retry
     ensure
-      trap('INT', orig_handler)
+      trap("INT", orig_handler)
     end
   end
 end

data/lib/byebug/interfaces/remote_interface.rb

@@ -1,4 +1,6 @@
-require 'byebug/history'
+# frozen_string_literal: true
+
+require "byebug/history"
 
 module Byebug
   #
@@ -14,25 +16,35 @@ module Byebug
 
     def read_command(prompt)
       super("PROMPT #{prompt}")
+    rescue Errno::EPIPE, Errno::ECONNABORTED
+      "continue"
     end
 
     def confirm(prompt)
       super("CONFIRM #{prompt}")
+    rescue Errno::EPIPE, Errno::ECONNABORTED
+      false
+    end
+
+    def print(message)
+      super(message)
+    rescue Errno::EPIPE, Errno::ECONNABORTED
+      nil
+    end
+
+    def puts(message)
+      super(message)
+    rescue Errno::EPIPE, Errno::ECONNABORTED
+      nil
     end
 
     def close
       output.close
-    rescue IOError
-      errmsg('Error closing the interface...')
     end
 
     def readline(prompt)
-      output.puts(prompt)
-
-      result = input.gets
-      raise IOError unless result
-
-      result.chomp
+      puts(prompt)
+      (input.gets || "continue").chomp
     end
   end
 end

data/lib/byebug/interfaces/script_interface.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Byebug
   #
   # Interface class for command execution from script files.

data/lib/byebug/interfaces/test_interface.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Byebug
   #
   # Custom interface for easier assertions
@@ -41,9 +43,9 @@ module Byebug
 
     def inspect
       [
-        'Input:', input.join("\n"),
-        'Output:', output.join("\n"),
-        'Error:', error.join("\n")
+        "Input:", input.join("\n"),
+        "Output:", output.join("\n"),
+        "Error:", error.join("\n")
       ].join("\n")
     end
 

data/lib/byebug/option_setter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Byebug
   #
   # Handles byebug's command line options
@@ -25,67 +27,67 @@ module Byebug
     private
 
     def debug
-      @opts.on '-d', '--debug', 'Set $DEBUG=true' do
+      @opts.on "-d", "--debug", "Set $DEBUG=true" do
         $DEBUG = true
       end
     end
 
     def include_flag
-      @opts.on '-I', '--include list', 'Add to paths to $LOAD_PATH' do |list|
-        $LOAD_PATH.push(list.split(':')).flatten!
+      @opts.on "-I", "--include list", "Add to paths to $LOAD_PATH" do |list|
+        $LOAD_PATH.push(list.split(":")).flatten!
       end
     end
 
     def post_mortem
-      @opts.on '-m', '--[no-]post-mortem', 'Use post-mortem mode' do |v|
+      @opts.on "-m", "--[no-]post-mortem", "Use post-mortem mode" do |v|
         Setting[:post_mortem] = v
       end
     end
 
     def quit
-      @opts.on '-q', '--[no-]quit', 'Quit when script finishes' do |v|
+      @opts.on "-q", "--[no-]quit", "Quit when script finishes" do |v|
         @runner.quit = v
       end
     end
 
     def rc
-      @opts.on '-x', '--[no-]rc', 'Run byebug initialization file' do |v|
+      @opts.on "-x", "--[no-]rc", "Run byebug initialization file" do |v|
         @runner.init_script = v
       end
     end
 
     def stop
-      @opts.on '-s', '--[no-]stop', 'Stop when script is loaded' do |v|
+      @opts.on "-s", "--[no-]stop", "Stop when script is loaded" do |v|
         @runner.stop = v
       end
     end
 
     def require_flag
-      @opts.on '-r', '--require file', 'Require library before script' do |lib|
+      @opts.on "-r", "--require file", "Require library before script" do |lib|
         require lib
       end
     end
 
     def remote
-      @opts.on '-R', '--remote [host:]port', 'Remote debug [host:]port' do |p|
+      @opts.on "-R", "--remote [host:]port", "Remote debug [host:]port" do |p|
         @runner.remote = p
       end
     end
 
     def trace
-      @opts.on '-t', '--[no-]trace', 'Turn on line tracing' do |v|
+      @opts.on "-t", "--[no-]trace", "Turn on line tracing" do |v|
         Setting[:linetrace] = v
       end
     end
 
     def version
-      @opts.on '-v', '--version', 'Print program version' do
+      @opts.on "-v", "--version", "Print program version" do
         @runner.version = Byebug::VERSION
       end
     end
 
     def help
-      @opts.on '-h', '--help', 'Display this message' do
+      @opts.on "-h", "--help", "Display this message" do
         @runner.help = @opts.help
       end
     end

data/lib/byebug/printers/base.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'yaml'
+require "yaml"
 
 module Byebug
   module Printers
@@ -11,17 +11,17 @@ module Byebug
       class MissedPath < StandardError; end
       class MissedArgument < StandardError; end
 
-      SEPARATOR = '.'.freeze
+      SEPARATOR = ".".freeze
 
       def type
-        self.class.name.split('::').last.downcase
+        self.class.name.split("::").last.downcase
       end
 
       private
 
       def locate(path)
         result = nil
-        contents.each do |_, contents|
+        contents.each_value do |contents|
           result = parts(path).reduce(contents) do |r, part|
             r && r.key?(part) ? r[part] : nil
           end
@@ -33,7 +33,7 @@ module Byebug
 
       def translate(string, args = {})
         # they may contain #{} string interpolation
-        string.gsub(/\|\w+$/, '').gsub(/([^#]?){([^}]*)}/) do
+        string.gsub(/\|\w+$/, "").gsub(/([^#]?){([^}]*)}/) do
           key = Regexp.last_match[2].to_s
           unless args.key?(key.to_sym)
             raise MissedArgument, "Missed argument #{key} for '#{string}'"
@@ -62,7 +62,7 @@ module Byebug
       end
 
       def contents_files
-        [File.join(__dir__, 'texts', 'base.yml')]
+        [File.join(__dir__, "texts", "base.yml")]
       end
     end
   end

data/lib/byebug/printers/plain.rb

@@ -1,4 +1,6 @@
-require 'byebug/printers/base'
+# frozen_string_literal: true
+
+require "byebug/printers/base"
 
 module Byebug
   module Printers
@@ -8,7 +10,7 @@ module Byebug
     class Plain < Base
       def print(path, args = {})
         message = translate(locate(path), args)
-        tail = parts(path).include?('confirmations') ? ' (y/n) ' : "\n"
+        tail = parts(path).include?("confirmations") ? " (y/n) " : "\n"
         message << tail
       end
 
@@ -21,11 +23,11 @@ module Byebug
       end
 
       def print_variables(variables, *_)
-        print_collection('variable.variable', variables) do |(key, value), _|
-          value = value.nil? ? 'nil' : value.to_s
+        print_collection("variable.variable", variables) do |(key, value), _|
+          value = value.nil? ? "nil" : value.to_s
           if "#{key} = #{value}".size > Setting[:width]
             key_size = "#{key} = ".size
-            value = value[0..Setting[:width] - key_size - 4] + '...'
+            value = value[0..Setting[:width] - key_size - 4] + "..."
           end
 
           { key: key, value: value }
@@ -35,7 +37,7 @@ module Byebug
       private
 
       def contents_files
-        [File.join(__dir__, 'texts', 'plain.yml')] + super
+        [File.join(__dir__, "texts", "plain.yml")] + super
       end
     end
   end

data/lib/byebug/printers/texts/base.yml

@@ -1,10 +1,10 @@
 base:
   errors:
-    only_local: 'Command is available only in local mode.'
+    only_local: "Command is available only in local mode."
 
 break:
   errors:
-    line: "Line {line} is not a valid breakpoint in file {file}"
+    line: "Line {line} is not a valid breakpoint in file {file}.\n\nValid break points are:\n{valid_breakpoints}"
     location: "Invalid breakpoint location"
     state: "We are not in a state that has an associated file"
     class: "Unknown class {klass}"
@@ -62,7 +62,7 @@ info:
 
 pry:
   errors:
-    not_installed: 'You need to install pry in order to run this command'
+    not_installed: "You need to install pry in order to run this command"
 
 quit:
   confirmations:

data/lib/byebug/processors/command_processor.rb

@@ -1,7 +1,9 @@
-require 'forwardable'
+# frozen_string_literal: true
 
-require 'byebug/helpers/eval'
-require 'byebug/errors'
+require "forwardable"
+
+require "byebug/helpers/eval"
+require "byebug/errors"
 
 module Byebug
   #
@@ -16,19 +18,16 @@ module Byebug
     include Helpers::EvalHelper
 
     attr_accessor :prev_line
-    attr_reader :context
+    attr_reader :context, :interface
 
-    def initialize(context)
+    def initialize(context, interface = LocalInterface.new)
       @context = context
+      @interface = interface
 
       @proceed = false
       @prev_line = nil
     end
 
-    def interface
-      @interface ||= Context.interface
-    end
-
     def printer
       @printer ||= Printers::Plain.new
     end
@@ -106,7 +105,7 @@ module Byebug
     # Prompt shown before reading a command.
     #
     def prompt
-      '(byebug) '
+      "(byebug) "
     end
 
     def before_repl
@@ -129,7 +128,7 @@ module Byebug
         cmd = interface.read_command(prompt)
         return if cmd.nil?
 
-        next if cmd == ''
+        next if cmd == ""
 
         run_cmd(cmd)
       end
@@ -167,7 +166,7 @@ module Byebug
 
     def safely
       yield
-    rescue => e
+    rescue StandardError => e
       errmsg(e.message)
     end
   end