gh-120221: Support KeyboardInterrupt in asyncio REPL#123795
Uh oh!
There was an error while loading. Please reload this page.
Conversation
This switches the main pyrepl event loop to always be non-blocking so that it can listen to incoming interruptions from other threads. This also resolves invalid display of exceptions from other threads (pythongh-123178).
ambv requested review from 1st1, asvetlov, gvanrossum, kumaraditya303, lysnikolaou, pablogsal and willingc as code owners
| @@ -0,0 +1,2 @@ | |||
| asyncio REPL is now again properly recognizing KeyboardInterrupts. Display | |||
There was a problem hiding this comment.
| asyncio REPL is now again properly recognizing KeyboardInterrupts. Display | |
| asyncio REPL is now again properly recognizing :exc:`KeyboardInterrupt`. Display |
There was a problem hiding this comment.
That's not the same word. Let's do stuff like that in a subsequent PR please, so we don't waste time on CI now.
There was a problem hiding this comment.
Agreed. The upcoming release is more important now.
There was a problem hiding this comment.
FTR, to put the s, you would do :exc:`KeyboardInterrupt`\s
ambv commented Sep 6, 2024
 |
| handler=ExceptHookHandler() | ||
| reader.threading_hook=handler | ||
| threading.excepthook=handler.exception |
There was a problem hiding this comment.
This except hook essentially queues tracebacks for nicer display whenever the REPL is actually ready to do it.
| defrun_hooks(self) ->None: | ||
| threading_hook=self.threading_hook | ||
| ifthreading_hookisNoneand'threading'insys.modules: |
There was a problem hiding this comment.
This automatic installation is such that when people create threads in the REPL directly or indirectly (via libraries), they get correctly formatted tracebacks.
It just so happens that asyncio REPL uses threads so it triggers this condition, too.
| return ( | ||
| notself.event_queue.empty() | ||
| orself.more_in_buffer() | ||
| orbool(self.pollob.poll(timeout)) | ||
| ) |
There was a problem hiding this comment.
The previous behavior caused pyrepl to lose reads in non-blocking mode (i.e. with an input hook) if more than 1 character was read previously to the input_buffer (only 1 character is turned into an event at a time). This made tests flaky. Now they pass every time.
| from_pyrepl.simple_interactimport_get_reader | ||
| r=_get_reader() | ||
| ifr.threading_hookisnotNone: | ||
| r.threading_hook.add("") # type: ignore |
There was a problem hiding this comment.
This is how we add a KeyboardInterrupt to the other thread. We don't want a traceback, and the "KeyboardInterrupt" message will already appear.
| ifmsvcrt.kbhit(): # type: ignore[attr-defined] | ||
| returnTrue | ||
| iftimeoutandtime.time() -start_time>timeout: | ||
| iftimeoutandtime.time() -start_time>timeout/1000: |
| definstall_threading_hook(reader: Reader) ->None: | ||
| importthreading |
There was a problem hiding this comment.
I put this import here so that pyrepl will continue to work in thread-free environments like iOS and the browser.
There was a problem hiding this comment.
FYI: iOS and Android do support threads, but not subprocesses. The only platform that doesn't support threads is WASM.
willingc left a comment
willingc left a comment There was a problem hiding this comment.
Looks great @ambv. Thanks for making the PR easy to review with explanatory comments. So excited for the new REPL @ambv and @pablogsal. 🎉
Thanks @ambv for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13. |
…123795) This switches the main pyrepl event loop to always be non-blocking so that it can listen to incoming interruptions from other threads. This also resolves invalid display of exceptions from other threads (pythongh-123178). This also fixes freezes with pasting and an active input hook. (cherry picked from commit 033510e) Co-authored-by: Łukasz Langa <lukasz@langa.pl>
GH-123799 is a backport of this pull request to the 3.13 branch. |
… (#123799) This switches the main pyrepl event loop to always be non-blocking so that it can listen to incoming interruptions from other threads. This also resolves invalid display of exceptions from other threads (gh-123178). This also fixes freezes with pasting and an active input hook. (cherry picked from commit 033510e) Co-authored-by: Łukasz Langa <lukasz@langa.pl>
6 participants
This switches the main pyrepl event loop to always be non-blocking so that it can listen to incoming interruptions from other threads.
This also resolves invalid display of exceptions from other threads (gh-123178).
This also fixes freezes with pasting and an active input hook.