Address review: rename bow_ws, move tests to TestReader, add tab support

- Rename bow_whitespace() to bow_ws() for shorter naming convention
- Move tests from separate TestBowWhitespace class into TestReader
- Add tab character to whitespace set (space, newline, tab)
- Add test_bow_ws_with_tabs for tab handling
- Fix link format to #146044
- Remove LLM-style long dashes from comments
- Update NEWS wording per review

Author: kimimgo

Lib/_pyrepl/commands.py

@@ -169,7 +169,7 @@ class unix_word_rubout(KillCommand):
     def do(self) -> None:
         r = self.reader
         for i in range(r.get_arg()):
-            self.kill_range(r.bow_whitespace(), r.pos)
+            self.kill_range(r.bow_ws(), r.pos)
 
 
 class kill_word(KillCommand):

Lib/_pyrepl/reader.py

@@ -417,19 +417,20 @@ def bow(self, p: int | None = None) -> int:
             p -= 1
         return p + 1
 
-    def bow_whitespace(self, p: int | None = None) -> int:
+    def bow_ws(self, p: int | None = None) -> int:
         """Return the 0-based index of the whitespace-delimited word break
         preceding p most immediately.
 
         p defaults to self.pos; only whitespace is considered a word
-        boundary, matching the behavior of unix-word-rubout in bash/readline."""
+        boundary, matching the behavior of unix-word-rubout in bash/readline.
+        See https://github.com/python/cpython/issues/146044"""
         if p is None:
             p = self.pos
         b = self.buffer
         p -= 1
-        while p >= 0 and b[p] in (" ", "\n"):
+        while p >= 0 and b[p] in " \n\t":
             p -= 1
-        while p >= 0 and b[p] not in (" ", "\n"):
+        while p >= 0 and b[p] not in " \n\t":
             p -= 1
         return p + 1
 

Lib/test/test_pyrepl/test_reader.py

@@ -358,6 +358,34 @@ def test_setpos_from_xy_for_non_printing_char(self):
         reader.setpos_from_xy(8, 0)
         self.assertEqual(reader.pos, 7)
 
+    def test_bow_ws_stops_at_whitespace(self):
+        # See https://github.com/python/cpython/issues/146044
+        reader = prepare_reader(prepare_console([]))
+        reader.buffer = list("foo.bar baz")
+        reader.pos = len(reader.buffer)
+        self.assertEqual(reader.bow_ws(), 8)
+
+    def test_bow_ws_includes_punctuation_in_word(self):
+        reader = prepare_reader(prepare_console([]))
+        reader.buffer = list("foo.bar(baz) qux")
+        reader.pos = 12
+        self.assertEqual(reader.bow_ws(), 0)
+
+    def test_bow_vs_bow_ws(self):
+        reader = prepare_reader(prepare_console([]))
+        reader.buffer = list("foo.bar")
+        reader.pos = len(reader.buffer)
+        # bow() stops at '.' so we return the index of 'b' in "bar"
+        self.assertEqual(reader.bow(), 4)
+        # bow_ws() treats entire "foo.bar" as one word
+        self.assertEqual(reader.bow_ws(), 0)
+
+    def test_bow_ws_with_tabs(self):
+        reader = prepare_reader(prepare_console([]))
+        reader.buffer = list("foo\tbar")
+        reader.pos = len(reader.buffer)
+        self.assertEqual(reader.bow_ws(), 4)
+
 @force_colorized_test_class
 class TestReaderInColor(ScreenEqualMixin, TestCase):
     def test_syntax_highlighting_basic(self):
@@ -560,45 +588,3 @@ def test_control_characters(self):
         self.assert_screen_equal(reader, 'flag {o}={z} {s}"🏳️\\u200d🌈"{z}'.format(**colors))
 
 
-class TestBowWhitespace(TestCase):
-    def test_bow_whitespace_stops_at_whitespace(self):
-        # GH#146044
-        # unix-word-rubout (ctrl-w) should use whitespace boundaries,
-        # not punctuation boundaries like bow() does
-        reader = prepare_reader(prepare_console([]))
-        reader.buffer = list("foo.bar baz")
-        reader.pos = len(reader.buffer)  # cursor at end
-
-        # bow_whitespace from end should jump to start of "baz"
-        result = reader.bow_whitespace()
-        self.assertEqual(result, 8)  # index of 'b' in "baz"
-
-    def test_bow_whitespace_includes_punctuation_in_word(self):
-        # GH#146044
-        reader = prepare_reader(prepare_console([]))
-        reader.buffer = list("foo.bar(baz) qux")
-        reader.pos = 12  # cursor after ")"
-
-        # bow_whitespace should treat "foo.bar(baz)" as one word
-        result = reader.bow_whitespace()
-        self.assertEqual(result, 0)
-
-    def test_bow_stops_at_punctuation(self):
-        # Verify existing bow() still uses syntax_table (punctuation boundary)
-        reader = prepare_reader(prepare_console([]))
-        reader.buffer = list("foo.bar baz")
-        reader.pos = len(reader.buffer)
-
-        result = reader.bow()
-        self.assertEqual(result, 8)  # same — "baz" is all word chars
-
-    def test_bow_vs_bow_whitespace_difference(self):
-        # The key difference: bow() stops at '.', bow_whitespace() does not
-        reader = prepare_reader(prepare_console([]))
-        reader.buffer = list("foo.bar")
-        reader.pos = len(reader.buffer)
-
-        # bow() stops at '.' → returns index of 'b' in "bar"
-        self.assertEqual(reader.bow(), 4)
-        # bow_whitespace() treats entire "foo.bar" as one word
-        self.assertEqual(reader.bow_whitespace(), 0)

Misc/NEWS.d/next/Library/2026-03-20-00-00-01.gh-issue-146044.WsCtrl.rst

@@ -1,3 +1,3 @@
 Fix ``unix-word-rubout`` (Ctrl-W) in the REPL to use whitespace-only word
-boundaries, matching bash/readline behavior. Previously it used
+boundaries, matching behavior of the basic REPL. Previously it used
 syntax-table boundaries which treated punctuation as word separators.