From 7b7259d7159bc7144c385c1fd6a13e16dd196001 Mon Sep 17 00:00:00 2001
From: Koichi Sasada <ko1@atdot.net>
Date: Thu, 22 Dec 2022 20:50:37 +0900
Subject: [PATCH] skip <internal:...> file

For the `<internal:...>` files does not have a file and `absolute_path`
is nil, so it is assumed as eval'ed code and `skip_location?` add prefix
`!eval:` to the path name. So it also cheks with this prefix too.

fix https://github.com/ruby/debug/issues/866
---
 lib/debug/thread_client.rb                 |  2 +-
 test/console/control_flow_commands_test.rb | 14 ++++++++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/lib/debug/thread_client.rb b/lib/debug/thread_client.rb
index bfb2c3ee4..cd79a3205 100644
--- a/lib/debug/thread_client.rb
+++ b/lib/debug/thread_client.rb
@@ -29,7 +29,7 @@ def skip_config_skip_path?(path)
     end
 
     def skip_internal_path?(path)
-      path.start_with?(__dir__) || path.start_with?('<internal:')
+      path.start_with?(__dir__) || path.delete_prefix('!eval:').start_with?('<internal:')
     end
 
     def skip_location?(loc)
diff --git a/test/console/control_flow_commands_test.rb b/test/console/control_flow_commands_test.rb
index dd46301b8..93b00f4e0 100644
--- a/test/console/control_flow_commands_test.rb
+++ b/test/console/control_flow_commands_test.rb
@@ -493,6 +493,20 @@ def test_unit_method_in_the_same_line
     end
   end
 
+  class SkipStepOnInternalCodeTest < ConsoleTestCase
+    def test_skip_step_on_internal_code
+      code = <<~RUBY
+      1| _a = 1.abs # step here and it should not step into Integer#abs's internal source
+      2| _b = _a.abs
+      RUBY
+      debug_code code do
+        type 's'
+        assert_line_num 2
+        type 'c'
+      end
+    end
+  end
+
   #
   # Tests that next/finish work for a deep call stack.
   # We use different logic for computing frame depth when the call stack is above/below 4096.