python · gaogaotiantian · Mar 4, 2025 · Feb 24, 2025 · Feb 24, 2025 · Feb 24, 2025
@@ -215,10 +215,13 @@ def dispatch_opcode(self, frame, arg):
         If the debugger stops on the current opcode, invoke
         self.user_opcode(). Raise BdbQuit if self.quitting is set.
         Return self.trace_dispatch to continue tracing in this scope.
+
+        Opcode event will always trigger the user callback. For now the only
+        opcode event is from an inline set_trace() and we want to stop there
+        unconditionally.
         """
-        if self.stop_here(frame) or self.break_here(frame):
-            self.user_opcode(frame)
-            if self.quitting: raise BdbQuit
+        self.user_opcode(frame)
+        if self.quitting: raise BdbQuit
         return self.trace_dispatch
 
     # Normally derived classes don't override the following

@@ -4341,6 +4341,28 @@ def test_quit(self):
         # The quit prompt should be printed exactly twice
         self.assertEqual(stdout.count("Quit anyway"), 2)
 
+    def test_set_trace_with_skip(self):
+        """GH-82897
+        Inline set_trace() should break unconditionally. This example is a
+        bit oversimplified, but as `pdb.set_trace()` uses the previous Pdb
+        instance, it's possible that we had a previous pdb instance with
+        skip values when we use `pdb.set_trace()` - it would be confusing
+        to users when such inline breakpoints won't break immediately.
+        """
+        script = textwrap.dedent("""
+            import pdb
+            def foo():
+                x = 40 + 2
+                pdb.Pdb(skip=['__main__']).set_trace()
+            foo()
+        """)
+        commands = """
+            p x
+            c
+        """
+        stdout, _ = self._run_script(script, commands)
+        self.assertIn("42", stdout)
+
 
 @support.requires_subprocess()
 class PdbTestReadline(unittest.TestCase):

diff --git a/Misc/NEWS.d/next/Library/2025-02-24-01-49-11.gh-issue-82987.vHfQlG.rst b/Misc/NEWS.d/next/Library/2025-02-24-01-49-11.gh-issue-82987.vHfQlG.rst
@@ -0,0 +1 @@
+:mod:`pdb` will always stop on calling frames when inline breakpoints like :func:`breakpoint` or :func:`pdb.set_trace` are used, regardless of whether the module matches ``skip`` pattern.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		:mod:`pdb` will always stop on calling frames when inline breakpoints like :func:`breakpoint` or :func:`pdb.set_trace` are used, regardless of whether the module matches ``skip`` pattern.
Comment thread gaogaotiantian marked this conversation as resolved. Outdated Copy link Copy Markdown Member iritkatriel Mar 3, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. This is probably worth a what's new entry. Copy link Copy Markdown Member iritkatriel Mar 3, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. And a versionchange comment in the doc? Copy link Copy Markdown Member Author gaogaotiantian Mar 3, 2025 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others. Learn more. Done