Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clio should have a generic panic recovery function #84

Open
willmurphyscode opened this issue Dec 16, 2024 · 0 comments
Open

clio should have a generic panic recovery function #84

willmurphyscode opened this issue Dec 16, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@willmurphyscode
Copy link
Contributor

What would you like to be added:

A Clio CLI, when it encounters a panic, should recover and print out the stack trace in good order, get the terminal out of raw mode, tear down any TUI as far as possible, and explain to the user to have a bug report.

I think this would require adding a parameter like "bugReportURL" to the App constructor, and then clio could template out a nice panic / recovery screen.

Why is this needed:

Right now, when apps using Clio panic, the TUI isn't torn down and the tty is left in raw mode, leaving the user to figure out what to do with a broken-looking terminal and a weirdly indented stack trace.

Additional context:

@willmurphyscode willmurphyscode added the enhancement New feature or request label Dec 16, 2024
@willmurphyscode willmurphyscode moved this to Ready in OSS Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Ready
Development

No branches or pull requests

1 participant