Bracket REPL error overlay messages

Author: johnslavik

Lib/_pyrepl/reader.py

@@ -882,6 +882,7 @@ def debug(self, msg: str) -> None:
         pass
 
     def error(self, msg: str = "none") -> None:
+        msg = f"[ {msg} ]"
         if self.can_colorize:
             colors = _colorize.ANSIColors
             msg = f"{colors.RED}{msg}{colors.RESET}"

Lib/test/test_pyrepl/test_reader.py

@@ -469,7 +469,7 @@ def test_colorized_error_message(self):
 
         self.assertEqual(
             reader.msg,
-            f"{ANSIColors.RED}nothing to yank{ANSIColors.RESET}",
+            f"{ANSIColors.RED}[ nothing to yank ]{ANSIColors.RESET}",
         )
         self.assertEqual(reader.screen[-1], reader.msg)
 
@@ -495,69 +495,69 @@ def _test_error_message(
         console.beep.assert_called_once()
 
     def test_invalid_key_message(self):
-        self._test_error_message("\x1bo", "no command is bound to this key")
+        self._test_error_message("\x1bo", "[ no command is bound to this key ]")
 
     def test_unimplemented_command_message(self):
         self._test_error_message(
             "x",
-            "command 'missing-command' is not implemented",
+            "[ command 'missing-command' is not implemented ]",
             command="missing-command",
         )
 
     def test_nothing_to_yank_message(self):
-        self._test_error_message("\x19", "nothing to yank")
+        self._test_error_message("\x19", "[ nothing to yank ]")
 
     def test_yank_pop_with_empty_kill_ring_message(self):
-        self._test_error_message("\x1by", "nothing to yank")
+        self._test_error_message("\x1by", "[ nothing to yank ]")
 
     def test_yank_pop_after_other_command_message(self):
         self._test_error_message(
             "x\x01\x0b\x1by",
-            "previous command was not a yank",
+            "[ previous command was not a yank ]",
         )
 
     def test_start_of_history_message(self):
-        self._test_error_message("\x10", "start of history list")
+        self._test_error_message("\x10", "[ start of history list ]")
 
     def test_end_of_history_message(self):
-        self._test_error_message("\x0e", "end of history list")
+        self._test_error_message("\x0e", "[ end of history list ]")
 
     def test_yank_arg_at_beginning_of_history_message(self):
         self._test_error_message(
             "\x1b.\x1b.\x1b.",
-            "beginning of history list",
+            "[ beginning of history list ]",
             history=["one"],
             historyi=1,
         )
 
     def test_yank_arg_with_no_argument_message(self):
         self._test_error_message(
             "\x1b.",
-            "no such arg",
+            "[ no such arg ]",
             history=[""],
             historyi=1,
         )
 
     def test_empty_incremental_search_backspace_message(self):
         self._test_error_message(
             "x",
-            "nothing to rubout",
+            "[ nothing to rubout ]",
             command="isearch-backspace",
         )
 
     def test_history_search_not_found_message(self):
         self._test_error_message(
             "missingx",
-            "not found",
+            "[ not found ]",
             command="history-search-backward",
             binding="x",
         )
 
     def test_incremental_search_not_found_message(self):
-        self._test_error_message("\x12x", "not found")
+        self._test_error_message("\x12x", "[ not found ]")
 
     def test_no_completion_matches_message(self):
-        self._test_error_message("does_not_exist\t", "no matches")
+        self._test_error_message("does_not_exist\t", "[ no matches ]")
 
 
 @force_colorized_test_class

Misc/NEWS.d/next/Library/2026-10-20-20-00-00.gh-issue-148228.X7mQp2.rst

@@ -1,2 +1,2 @@
-Display error overlays in the new interactive shell without the ``!`` prefix
+Display error overlays in the new interactive shell in square brackets
 and in non-bold red when color is enabled. Patch by Bartosz Sławecki.