debug 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5e9851059a728e477461cc318c9b94932fa0aeb19215f936ab6c25cf62f67cd0
-  data.tar.gz: b050eee3dbf0afecfda2c1221fe38ab678808405c10f89d5832a1950e2728aa8
+  metadata.gz: d6eba8a00eaa711138e43036c5adb97834f1a87e586f8a5a3f580875e4d78e5c
+  data.tar.gz: 26ed8ac77bb93ab2a9f67fac6c4ef4481fbcdd227f116aaff78e808a2343b13b
 SHA512:
-  metadata.gz: a15549ee06b890a32d0960dd24a99c4c925d66b5f6512a7e5ff3052f10ca3a255038d44eaa2d10046cbe2b68bca121c813db3a8e5fea68def5dcd2e1797864d7
-  data.tar.gz: '058b5ef8c6e40a8ccfa12b62b09c8808c1b524c398e37a25ffd790b8a8e0364026911478019d06100b7a6d8fde878dd7695f2f2630f3f35f37f12695b84b2ddf'
+  metadata.gz: b459267a95928388d1bf53bab8bf51fc8d5a6d0db0bc2fbaa55f740b1b446aad2294e45cc2b70471ccf8bb1daada458f239f08c7ceb7588774f08796a7838620
+  data.tar.gz: 86d06a4ec76943e26af39a26120c8a3a906d3fcecca195a662e63b9be2b7b9a1ba82bd2fba177205b436cde89be9214988e0f2cccc0f7fb2dcbf7226e92886e9

data/README.md

@@ -372,6 +372,7 @@ config set no_color true
 
 * BOOT
   * `RUBY_DEBUG_NONSTOP` (`nonstop`): Nonstop mode
+  * `RUBY_DEBUG_STOP_AT_LOAD` (`stop_at_load`): Stop at just loading location
   * `RUBY_DEBUG_INIT_SCRIPT` (`init_script`): debug command script path loaded at first stop
   * `RUBY_DEBUG_COMMANDS` (`commands`): debug commands invoked at first stop. commands should be separated by ';;'
   * `RUBY_DEBUG_NO_RC` (`no_rc`): ignore loading ~/.rdbgrc(.rb)
@@ -436,6 +437,9 @@ The `<...>` notation means the argument.
   * Stop the debuggee process with `Kernal#exit!`.
 * `kill!`
   * Same as kill but without the confirmation prompt.
+* `sigint`
+  * Execute SIGINT handler registerred by the debuggee.
+  * Note that this command should be used just after stop by `SIGINT`.
 
 ### Breakpoint
 
@@ -728,6 +732,7 @@ Attach mode:
 Other options:
     -h, --help                       Print help
         --util=NAME                  Utility mode (used by tools)
+        --stop-at-load               Stop immediately when the debugging feature is loaded.
 
 NOTE
   All messages communicated between a debugger and a debuggee are *NOT* encrypted.

data/ext/debug/debug.c

@@ -122,6 +122,11 @@ Init_debug(void)
 {
     rb_mDebugger = rb_const_get(rb_cObject, rb_intern("DEBUGGER__"));
     rb_cFrameInfo = rb_const_get(rb_mDebugger, rb_intern("FrameInfo"));
+
+    // Debugger and FrameInfo were defined in Ruby. We need to register them
+    // as mark objects so they are automatically pinned.
+    rb_gc_register_mark_object(rb_mDebugger);
+    rb_gc_register_mark_object(rb_cFrameInfo);
     rb_define_singleton_method(rb_mDebugger, "capture_frames", capture_frames, 1);
     rb_define_singleton_method(rb_mDebugger, "frame_depth", frame_depth, 0);
     rb_define_singleton_method(rb_mDebugger, "create_method_added_tracker", create_method_added_tracker, 0);

data/lib/debug/breakpoint.rb

@@ -288,6 +288,7 @@ module DEBUGGER__
       @tp = TracePoint.new(:line){|tp|
         next if tp.path.start_with? __dir__
         next if tp.path.start_with? '<internal:'
+        next if ThreadClient.current.management?
 
         if safe_eval tp.binding, @expr
           suspend
@@ -430,17 +431,24 @@ module DEBUGGER__
         if @sig_op == '#'
           @cond_class = @klass if @method.owner != @klass
         else # '.'
-          @cond_class = @klass.singleton_class if @method.owner != @klass.singleton_class
+          begin
+            @cond_class = @klass.singleton_class if @method.owner != @klass.singleton_class
+          rescue TypeError
+          end
         end
 
-      rescue ArgumentError
+      rescue ArgumentError => e
         raise if retried
         retried = true
 
         # maybe C method
         case @sig_op
         when '.'
-          override @klass.singleton_class
+          begin
+            override @klass.singleton_class
+          rescue TypeError
+            override @klass.class
+          end
         when '#'
           override @klass
         end
@@ -450,7 +458,7 @@ module DEBUGGER__
         @override_method = true if @method
         retry
       end
-    rescue Exception
+    rescue Exception => e
       raise unless added
     end
 
@@ -460,7 +468,8 @@ module DEBUGGER__
 
     def to_s
       if @method
-        "#{generate_label("Method")} #{sig} at #{@method.source_location.join(':')}"
+        loc = @method.source_location || []
+        "#{generate_label("Method")} #{sig} at #{loc.join(':')}"
       else
         "#{generate_label("Method (pending)")} #{sig}"
       end + super

data/lib/debug/color.rb

@@ -19,7 +19,7 @@ module DEBUGGER__
     if defined? IRB::Color.colorize
       def colorize str, color
         if !CONFIG[:no_color]
-          IRB::Color.colorize str, color
+          IRB::Color.colorize str, color, colorable: true
         else
           str
         end
@@ -64,7 +64,7 @@ module DEBUGGER__
 
     if defined? IRB::Color.colorize_code
       def colorize_code code
-        IRB::Color.colorize_code(code)
+        IRB::Color.colorize_code(code, colorable: true)
       end
     else
       def colorize_code code

data/lib/debug/config.rb

@@ -29,6 +29,7 @@ module DEBUGGER__
 
     # boot setting
     nonstop:        ['RUBY_DEBUG_NONSTOP',     "BOOT: Nonstop mode",                                                :bool],
+    stop_at_load:   ['RUBY_DEBUG_STOP_AT_LOAD',"BOOT: Stop at just loading location",                               :bool],
     init_script:    ['RUBY_DEBUG_INIT_SCRIPT', "BOOT: debug command script path loaded at first stop"],
     commands:       ['RUBY_DEBUG_COMMANDS',    "BOOT: debug commands invoked at first stop. commands should be separated by ';;'"],
     no_rc:          ['RUBY_DEBUG_NO_RC',       "BOOT: ignore loading ~/.rdbgrc(.rb)",                               :bool],
@@ -110,7 +111,9 @@ module DEBUGGER__
       end
 
       if_updated old_conf, conf, :postmortem do |_, new_p|
-        SESSION.postmortem = new_p
+        if defined?(SESSION)
+          SESSION.postmortem = new_p
+        end
       end
 
       if_updated old_conf, conf, :sigdump_sig do |old_sig, new_sig|
@@ -314,6 +317,10 @@ module DEBUGGER__
           exit
         end
 
+        o.on('--stop-at-load', 'Stop immediately when the debugging feature is loaded.') do
+          config[:stop_at_load] = true
+        end
+
         o.separator ''
         o.separator 'NOTE'
         o.separator '  All messages communicated between a debugger and a debuggee are *NOT* encrypted.'

data/lib/debug/console.rb

@@ -6,11 +6,19 @@ module DEBUGGER__
       require 'reline'
 
       # reline 0.2.7 or later is required.
-      raise LoadError if Reline::VERSION < '0.2.6'
+      raise LoadError if Reline::VERSION < '0.2.7'
 
       require_relative 'color'
       include Color
 
+      begin
+        prev = trap(:SIGWINCH, nil)
+        trap(:SIGWINCH, prev)
+        SIGWINCH_SUPPORTED = true
+      rescue ArgumentError
+        SIGWINCH_SUPPORTED = false
+      end
+
       # 0.2.7 has SIGWINCH issue on non-main thread
       class ::Reline::LineEditor
         m = Module.new do
@@ -20,7 +28,7 @@ module DEBUGGER__
           end
         end
         prepend m
-      end
+      end if SIGWINCH_SUPPORTED
 
       def readline_setup prompt
         commands = DEBUGGER__.commands

data/lib/debug/frame_info.rb

@@ -27,6 +27,7 @@ module DEBUGGER__
     end
 
     def pretty_path
+      return '#<none>' unless path = self.path
       use_short_path = CONFIG[:use_short_path]
 
       case
@@ -137,10 +138,9 @@ module DEBUGGER__
       if lvars = self._local_variables
         lvars
       elsif b = self.binding
-        lvars = b.local_variables.map{|var|
+        b.local_variables.map{|var|
           [var, b.local_variable_get(var)]
         }.to_h
-        self._local_variables = lvars
       end
     end
 

data/lib/debug/local.rb

@@ -7,32 +7,27 @@ module DEBUGGER__
   class UI_LocalConsole < UI_Base
     def initialize
       @console = Console.new
-
-      unless CONFIG[:no_sigint_hook]
-        @prev_handler = trap(:SIGINT){
-          if SESSION.active?
-            ThreadClient.current.on_trap :SIGINT
-          end
-        }
-      end
-    end
-
-    def close
-      if @prev_handler
-        trap(:SIGINT, @prev_handler)
-      end
     end
 
     def remote?
       false
     end
 
-    def activate on_fork: false
-      # Do nothing
+    def activate session, on_fork: false
+      unless CONFIG[:no_sigint_hook]
+        prev_handler = trap(:SIGINT){
+          if session.active?
+            ThreadClient.current.on_trap :SIGINT
+          end
+        }
+        session.intercept_trap_sigint_start prev_handler
+      end
     end
 
     def deactivate
-      # Do nothing
+      if SESSION.intercept_trap_sigint?
+        trap(:SIGINT, SESSION.intercepted_sigint_cmd)
+      end
     end
 
     def width
@@ -76,15 +71,17 @@ module DEBUGGER__
     end
 
     def setup_interrupt
-      current_thread = Thread.current # should be session_server thread
+      SESSION.intercept_trap_sigint false do
+        current_thread = Thread.current # should be session_server thread
 
-      prev_handler = trap(:INT){
-        current_thread.raise Interrupt
-      }
+        prev_handler = trap(:INT){
+          current_thread.raise Interrupt
+        }
 
-      yield
-    ensure
-      trap(:INT, prev_handler)
+        yield
+      ensure
+        trap(:INT, prev_handler)
+      end
     end
   end
 end

data/lib/debug/server.rb

@@ -15,8 +15,6 @@ module DEBUGGER__
       @q_ans = nil
       @unsent_messages = []
       @width = 80
-
-      activate
     end
 
     class Terminate < StandardError
@@ -37,7 +35,7 @@ module DEBUGGER__
       end
     end
 
-    def activate on_fork: false
+    def activate session, on_fork: false
       @reader_thread = Thread.new do
         # An error on this thread should break the system.
         Thread.current.abort_on_exception = true
@@ -138,9 +136,9 @@ module DEBUGGER__
     end
 
     def setup_interrupt
-      prev_handler = trap(:SIGINT) do
+      prev_handler = trap(:SIGURG) do
         # $stderr.puts "trapped SIGINT"
-        ThreadClient.current.on_trap :SIGINT
+        ThreadClient.current.on_trap :SIGURG
 
         case prev_handler
         when Proc
@@ -150,9 +148,12 @@ module DEBUGGER__
         end
       end
 
+      if prev_handler != "SYSTEM_DEFAULT"
+        DEBUGGER__.warn "SIGURG handler is overriddend by the debugger."
+      end
       yield
     ensure
-      trap(:SIGINT, prev_handler)
+      trap(:SIGURG, prev_handler)
     end
 
     attr_reader :reader_thread
@@ -230,7 +231,7 @@ module DEBUGGER__
 
     def pause
       # $stderr.puts "DEBUG: pause request"
-      Process.kill(:SIGINT, Process.pid)
+      Process.kill(:SIGURG, Process.pid)
     end
 
     def quit n

data/lib/debug/session.rb

@@ -86,7 +86,7 @@ module DEBUGGER__
                 #   [:watch, ivar] => WatchIVarBreakpoint
                 #   [:check, expr] => CheckBreakpoint
       #
-      @tracers = []
+      @tracers = {}
       @th_clients = nil # {Thread => ThreadClient}
       @q_evt = Queue.new
       @displays = []
@@ -96,17 +96,22 @@ module DEBUGGER__
       @postmortem_hook = nil
       @postmortem = false
       @thread_stopper = nil
+      @intercept_trap_sigint = false
+      @intercepted_sigint_cmd = 'DEFAULT'
 
       @frame_map = {} # {id => [threadId, frame_depth]} for DAP
       @var_map   = {1 => [:globals], } # {id => ...} for DAP
       @src_map   = {} # {id => src}
 
+      @tp_thread_begin = nil
       @tp_load_script = TracePoint.new(:script_compiled){|tp|
         ThreadClient.current.on_load tp.instruction_sequence, tp.eval_script
       }
       @tp_load_script.enable
 
       activate
+
+      self.postmortem = CONFIG[:postmortem]
     end
 
     def active?
@@ -117,40 +122,41 @@ module DEBUGGER__
       @bps.has_key? [file, line]
     end
 
-    def check_forked
-      unless active?
-        # TODO: Support it
-        raise 'DEBUGGER: stop at forked process is not supported yet.'
+    def activate on_fork: false
+      @tp_thread_begin&.disable
+      @tp_thread_begin = nil
+
+      if on_fork
+        @ui.activate self, on_fork: true
+      else
+        @ui.activate self, on_fork: false
       end
-    end
 
-    def activate on_fork: false
+      q = Queue.new
       @session_server = Thread.new do
         Thread.current.name = 'DEBUGGER__::SESSION@server'
         Thread.current.abort_on_exception = true
-        session_server_main
-      end
 
-      setup_threads
+        # Thread management
+        setup_threads
+        thc = thread_client Thread.current
+        thc.is_management
 
-      thc = thread_client @session_server
-      thc.is_management
+        if @ui.respond_to?(:reader_thread) && thc = thread_client(@ui.reader_thread)
+          thc.is_management
+        end
 
-      if on_fork
-        @tp_thread_begin.disable
-        @tp_thread_begin = nil
-        @ui.activate on_fork: true
-      end
+        @tp_thread_begin = TracePoint.new(:thread_begin) do |tp|
+          thread_client
+        end
+        @tp_thread_begin.enable
 
-      if @ui.respond_to?(:reader_thread) && thc = thread_client(@ui.reader_thread)
-        thc.is_management
+        # session start
+        q << true
+        session_server_main
       end
 
-      @tp_thread_begin = TracePoint.new(:thread_begin){|tp|
-        th = Thread.current
-        ThreadClient.current.on_thread_begin th
-      }
-      @tp_thread_begin.enable
+      q.pop
     end
 
     def deactivate
@@ -160,14 +166,14 @@ module DEBUGGER__
       @tp_thread_begin.disable
       @bps.each{|k, bp| bp.disable}
       @th_clients.each{|th, thc| thc.close}
-      @tracers.each{|t| t.disable}
+      @tracers.values.each{|t| t.disable}
       @q_evt.close
       @ui&.deactivate
       @ui = nil
     end
 
     def reset_ui ui
-      @ui.close
+      @ui.deactivate
       @ui = ui
     end
 
@@ -182,6 +188,13 @@ module DEBUGGER__
         output.each{|str| @ui.puts str}
 
         case ev
+
+        when :thread_begin # special event, tc is nil
+          th = ev_args.shift
+          q = ev_args.shift
+          on_thread_begin th
+          q << true
+
         when :init
           wait_command_loop tc
 
@@ -193,24 +206,23 @@ module DEBUGGER__
 
         when :trace
           trace_id, msg = ev_args
-          if t = @tracers.find{|t| t.object_id == trace_id}
+          if t = @tracers.values.find{|t| t.object_id == trace_id}
             t.puts msg
           end
           tc << :continue
 
-        when :thread_begin
-          th = ev_args.shift
-          on_thread_begin th
-          @ui.event :thread_begin, th
-          tc << :continue
-
         when :suspend
           case ev_args.first
           when :breakpoint
             bp, i = bp_index ev_args[1]
             @ui.event :suspend_bp, i, bp
           when :trap
-            @ui.event :suspend_trap, ev_args[1]
+            @ui.event :suspend_trap, sig = ev_args[1]
+
+            if sig == :SIGINT && (@intercepted_sigint_cmd.kind_of?(Proc) || @intercepted_sigint_cmd.kind_of?(String))
+              @ui.puts "#{@intercepted_sigint_cmd.inspect} is registerred as SIGINT handler."
+              @ui.puts "`sigint` command execute it."
+            end
           else
             @ui.event :suspended
           end
@@ -246,8 +258,7 @@ module DEBUGGER__
             obj_id = ev_args[1]
             obj_inspect = ev_args[2]
             opt = ev_args[3]
-            @tracers << t = ObjectTracer.new(@ui, obj_id, obj_inspect, **opt)
-            @ui.puts "Enable #{t.to_s}"
+            add_tracer ObjectTracer.new(@ui, obj_id, obj_inspect, **opt)
           else
             # ignore
           end
@@ -432,6 +443,29 @@ module DEBUGGER__
       when 'kill!'
         exit! (arg || 1).to_i
 
+      # * `sigint`
+      #   * Execute SIGINT handler registerred by the debuggee.
+      #   * Note that this command should be used just after stop by `SIGINT`.
+      when 'sigint'
+        begin
+          case cmd = @intercepted_sigint_cmd
+          when nil, 'IGNORE', :IGNORE, 'DEFAULT', :DEFAULT
+            # ignore
+          when String
+            eval(cmd)
+          when Proc
+            cmd.call
+          end
+
+          @tc << :continue
+          restart_all_threads
+
+        rescue Exception => e
+          @ui.puts "Exception: #{e}"
+          @ui.puts e.backtrace.map{|line| "  #{e}"}
+          return :retry
+        end
+
       ### Breakpoint
 
       # * `b[reak]`
@@ -740,7 +774,7 @@ module DEBUGGER__
           @ui.puts "not supported on the remote console."
           return :retry
         end
-        @tc << [:eval, :call, 'binding.irb']
+        @tc << [:eval, :irb]
 
         # don't repeat irb command
         @repl_prev_line = nil
@@ -778,32 +812,29 @@ module DEBUGGER__
         case arg
         when nil
           @ui.puts 'Tracers:'
-          @tracers.each_with_index{|t, i|
+          @tracers.values.each_with_index{|t, i|
             @ui.puts "* \##{i} #{t}"
           }
           @ui.puts
           return :retry
 
         when /\Aline\z/
-          @tracers << t = LineTracer.new(@ui, pattern: pattern, into: into)
-          @ui.puts "Enable #{t.to_s}"
+          add_tracer LineTracer.new(@ui, pattern: pattern, into: into)
           return :retry
 
         when /\Acall\z/
-          @tracers << t = CallTracer.new(@ui, pattern: pattern, into: into)
-          @ui.puts "Enable #{t.to_s}"
+          add_tracer CallTracer.new(@ui, pattern: pattern, into: into)
           return :retry
 
         when /\Aexception\z/
-          @tracers << t = ExceptionTracer.new(@ui, pattern: pattern, into: into)
-          @ui.puts "Enable #{t.to_s}"
+          add_tracer ExceptionTracer.new(@ui, pattern: pattern, into: into)
           return :retry
 
         when /\Aobject\s+(.+)/
           @tc << [:trace, :object, $1.strip, {pattern: pattern, into: into}]
 
         when /\Aoff\s+(\d+)\z/
-          if t = @tracers[$1.to_i]
+          if t = @tracers.values[$1.to_i]
             t.disable
             @ui.puts "Disable #{t.to_s}"
           else
@@ -812,7 +843,7 @@ module DEBUGGER__
           return :retry
 
         when /\Aoff(\s+(line|call|exception|object))?\z/
-          @tracers.each{|t|
+          @tracers.values.each{|t|
             if $2.nil? || t.type == $2
               t.disable
               @ui.puts "Disable #{t.to_s}"
@@ -1190,6 +1221,20 @@ module DEBUGGER__
       @ui.puts e.message
     end
 
+    # tracers
+
+    def add_tracer tracer
+      # don't repeat commands that add tracers
+      @repl_prev_line = nil
+      if @tracers.has_key? tracer.key
+        tracer.disable
+        @ui.puts "Duplicated tracer: #{tracer}"
+      else
+        @tracers[tracer.key] = tracer
+        @ui.puts "Enable #{tracer}"
+      end
+    end
+
     # threads
 
     def update_thread_list
@@ -1242,10 +1287,6 @@ module DEBUGGER__
       thread_list
     end
 
-    def thread_client_create th
-      @th_clients[th] = ThreadClient.new((@tc_id += 1), @q_evt, Queue.new, th)
-    end
-
     def setup_threads
       @th_clients = {}
 
@@ -1256,18 +1297,38 @@ module DEBUGGER__
 
     def on_thread_begin th
       if @th_clients.has_key? th
-        # OK
-      else
         # TODO: NG?
+      else
         thread_client_create th
       end
     end
 
-    def thread_client thr = Thread.current
-      if @th_clients.has_key? thr
-        @th_clients[thr]
+    private def thread_client_create th
+      # TODO: Ractor support
+      raise "Only session_server can create thread_client" unless Thread.current == @session_server
+      @th_clients[th] = ThreadClient.new((@tc_id += 1), @q_evt, Queue.new, th)
+    end
+
+    private def ask_thread_client th = Thread.current
+      # TODO: Ractor support
+      q2 = Queue.new
+      # tc, output, ev, @internal_info, *ev_args = evt
+      @q_evt << [nil, [], :thread_begin, nil, th, q2]
+      q2.pop
+
+      @th_clients[th] or raise "unexpected error"
+    end
+
+    # can be called by other threads
+    def thread_client th = Thread.current
+      if @th_clients.has_key? th
+        @th_clients[th]
       else
-        @th_clients[thr] = thread_client_create(thr)
+        if Thread.current == @session_server
+          thread_client_create th
+        else
+          ask_thread_client th
+        end
       end
     end
 
@@ -1386,7 +1447,10 @@ module DEBUGGER__
       end
     end
 
-    def enter_postmortem_session frames
+    def enter_postmortem_session exc
+      return unless exc.instance_variable_defined? :@__debugger_postmortem_frames
+
+      frames = exc.instance_variable_get(:@__debugger_postmortem_frames)
       @postmortem = true
       ThreadClient.current.suspend :postmortem, postmortem_frames: frames
     ensure
@@ -1399,7 +1463,7 @@ module DEBUGGER__
           @postmortem_hook = TracePoint.new(:raise){|tp|
             exc = tp.raised_exception
             frames = DEBUGGER__.capture_frames(__dir__)
-            exc.instance_variable_set(:@postmortem_frames, frames)
+            exc.instance_variable_set(:@__debugger_postmortem_frames, frames)
           }
           at_exit{
             @postmortem_hook.disable
@@ -1411,7 +1475,7 @@ module DEBUGGER__
                 @ui.puts exc.backtrace.map{|e| '  ' + e}
                 @ui.puts "\n"
 
-                enter_postmortem_session exc.instance_variable_get(:@postmortem_frames)
+                enter_postmortem_session exc
               rescue SystemExit
                 exit!
               rescue Exception => e
@@ -1431,6 +1495,30 @@ module DEBUGGER__
         end
       end
     end
+
+    def save_int_trap cmd
+      prev, @intercepted_sigint_cmd = @intercepted_sigint_cmd, cmd
+      prev
+    end
+
+    attr_reader :intercepted_sigint_cmd
+
+    def intercept_trap_sigint?
+      @intercept_trap_sigint
+    end
+
+    def intercept_trap_sigint flag, &b
+      prev = @intercept_trap_sigint
+      @intercept_trap_sigint = flag
+      yield
+    ensure
+      @intercept_trap_sigint = prev
+    end
+
+    def intercept_trap_sigint_start prev
+      @intercept_trap_sigint = true
+      @intercepted_sigint_cmd = prev
+    end
   end
 
   class UI_Base
@@ -1526,6 +1614,9 @@ module DEBUGGER__
   def self.setup_initial_suspend
     if !CONFIG[:nonstop]
       case
+      when CONFIG[:stop_at_load]
+        add_line_breakpoint __FILE__, __LINE__ + 1, oneshot: true, hook_call: false
+        nil # stop here
       when path = ENV['RUBY_DEBUG_INITIAL_SUSPEND_PATH']
         add_line_breakpoint path, 0, oneshot: true, hook_call: false
       when loc = ::DEBUGGER__.require_location
@@ -1643,7 +1734,18 @@ module DEBUGGER__
 
           at_exit{
             trap(:SIGINT, :IGNORE)
-            Process.waitpid(child_pid)
+
+            # only check child process from its parent
+            if Process.pid == parent_pid
+              begin
+                # sending a null signal to see if the child is still alive
+                Process.kill(0, child_pid)
+                # if the child is still alive, wait for it
+                Process.waitpid(child_pid)
+              rescue Errno::ESRCH
+                # if the child process has died, do nothing
+              end
+            end
           }
         }
         child_hook = -> {
@@ -1675,8 +1777,36 @@ module DEBUGGER__
     end
   end
 
-  class ::Object
-    include ForkInterceptor
+  module TrapInterceptor
+    def trap sig, *command, &command_proc
+      case sig&.to_sym
+      when :INT, :SIGINT
+        if defined?(SESSION) && SESSION.active? && SESSION.intercept_trap_sigint?
+          return SESSION.save_int_trap(command.empty? ? command_proc : command.first)
+        end
+      end
+
+      super
+    end
+  end
+
+  if RUBY_VERSION >= '3.0.0'
+    module ::Kernel
+      prepend ForkInterceptor
+      prepend TrapInterceptor
+    end
+  else
+    class ::Object
+      include ForkInterceptor
+      include TrapInterceptor
+    end
+  end
+
+  module ::Kernel
+    class << self
+      prepend ForkInterceptor
+      prepend TrapInterceptor
+    end
   end
 
   module ::Process
@@ -1684,10 +1814,16 @@ module DEBUGGER__
       prepend ForkInterceptor
     end
   end
+
+  module ::Signal
+    class << self
+      prepend TrapInterceptor
+    end
+  end
 end
 
-class Binding
-  def break pre: nil, do: nil
+module Kernel
+  def debugger pre: nil, do: nil
     return if !defined?(::DEBUGGER__::SESSION) || !::DEBUGGER__::SESSION.active?
 
     if pre || (do_expr = binding.local_variable_get(:do))
@@ -1697,20 +1833,10 @@ class Binding
     ::DEBUGGER__.add_line_breakpoint __FILE__, __LINE__ + 1, oneshot: true, command: cmds
     self
   end
-  alias b break
 end
 
-module Kernel
-  if RUBY_VERSION >= '2.7.0'
-    eval <<~RUBY, binding, __FILE__, __LINE__
-      def debugger(...)
-        binding.break(...)
-      end
-    RUBY
-  else
-    def debugger pre: nil, do: nil
-      b = binding
-      b.break pre: pre, do: b.local_variable_get(:do)
-    end
-  end
+class Binding
+  alias break debugger
+  alias b debugger
 end
+

data/lib/debug/thread_client.rb

@@ -20,9 +20,11 @@ module DEBUGGER__
 
   class ThreadClient
     def self.current
-      Thread.current[:DEBUGGER__ThreadClient] || begin
-        tc = ::DEBUGGER__::SESSION.thread_client
-        Thread.current[:DEBUGGER__ThreadClient] = tc
+      if thc = Thread.current[:DEBUGGER__ThreadClient]
+        thc
+      else
+        thc = SESSION.thread_client
+        Thread.current[:DEBUGGER__ThreadClient] = thc
       end
     end
 
@@ -115,7 +117,7 @@ module DEBUGGER__
         # TODO: there is waiting -> waiting
         # raise "#{mode} is given, but #{mode}" unless self.running?
       else
-        raise
+        raise "unknown mode: #{mode}"
       end
 
       @mode = mode
@@ -138,7 +140,11 @@ module DEBUGGER__
     end
 
     def inspect
-      "#<DBG:TC #{self.id}:#{@mode}@#{@thread.backtrace[-1]}>"
+      if bt = @thread.backtrace
+        "#<DBG:TC #{self.id}:#{@mode}@#{bt[-1]}>"
+      else # bt can be nil
+        "#<DBG:TC #{self.id}:#{@mode}>"
+      end
     end
 
     def to_s
@@ -194,10 +200,6 @@ module DEBUGGER__
       wait_next_action
     end
 
-    def on_thread_begin th
-      wait_reply [:thread_begin, th]
-    end
-
     def on_load iseq, eval_src
       wait_reply [:load, iseq, eval_src]
     end
@@ -340,7 +342,8 @@ module DEBUGGER__
       begin
         @success_last_eval = false
 
-        b = current_frame.eval_binding
+        b = current_frame&.eval_binding || TOPLEVEL_BINDING
+
         result = if b
                    f, _l = b.source_location
                    b.eval(src, "(rdbg)/#{f}")
@@ -659,18 +662,17 @@ module DEBUGGER__
 
     def wait_next_action_
       # assertions
-      raise "@mode is #{@mode}" unless @mode == :waiting
+      raise "@mode is #{@mode}" if !waiting?
 
       unless SESSION.active?
         pp caller
         set_mode :running
         return
       end
-      # SESSION.check_forked
 
       while true
         begin
-          set_mode :waiting if @mode != :waiting
+          set_mode :waiting if !waiting?
           cmds = @q_cmd.pop
           # pp [self, cmds: cmds]
           break unless cmds
@@ -779,6 +781,13 @@ module DEBUGGER__
             end
           when :call
             result = frame_eval(eval_src)
+          when :irb
+            begin
+              result = frame_eval('binding.irb')
+            ensure
+              # workaround: https://github.com/ruby/debug/issues/308
+              Reline.prompt_proc = nil if defined? Reline
+            end
           when :display, :try_display
             failed_results = []
             eval_src.each_with_index{|src, i|

data/lib/debug/tracer.rb

@@ -14,7 +14,7 @@ module DEBUGGER__
       end
     end
 
-    attr_reader :type
+    attr_reader :type, :key
 
     def initialize ui, pattern: nil, into: nil
       if /\ADEBUGGER__::(([A-Z][a-z]+?)[A-Z][a-z]+)/ =~ self.class.name
@@ -37,6 +37,8 @@ module DEBUGGER__
         @output = ui
       end
 
+      @key = [@type, @pattern, @into].freeze
+
       enable
     end
 
@@ -95,6 +97,8 @@ module DEBUGGER__
     end
 
     def minfo tp
+      return "block{}" if tp.event == :b_call
+
       klass = tp.defined_class
 
       if klass.singleton_class?
@@ -172,6 +176,7 @@ module DEBUGGER__
       @obj_id = obj_id
       @obj_inspect = obj_inspect
       super(ui, **kw)
+      @key = [@type, @obj_id, @pattern, @into].freeze
     end
 
     def description
@@ -201,7 +206,7 @@ module DEBUGGER__
             end
 
           out tp, " #{colorized_obj_inspect} receives #{colorize_blue(method_info)}"
-        else
+        elsif !tp.parameters.empty?
           b = tp.binding
           method_info = colorize_blue(minfo(tp))
 

data/lib/debug/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DEBUGGER__
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: debug
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Koichi Sasada
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-09-14 00:00:00.000000000 Z
+date: 2021-09-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: irb