Vouch request: Rushit #2142
Rushit
started this conversation in
Vouch Request
Replies: 1 comment
-
Before
After |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment


Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
What do you want to work on?
I'd like to fix #2120. I was using OpenShell to host a simple website out of a sandbox, I also bump into similar issue where TUI start throwing random logs and mine was trigger by network issue and loosing connection between gateway and terminal. I looked into it. The CLI logs to stderr for everything, and the TUI draws to that same terminal, so when a background call fails the warning gets printed right on TUI.
Draft - https://github.com/NVIDIA/OpenShell/compare/main...Rushit:OpenShell:fix/2120-tui-log-to-file?expand=1
Why this change?
I feel it's simple fix and it keeps me annoyed me enough on different issues when gateway not reachable or back end not accessible. Also, the approach to log into file -
• Fixes UI corruption: No more log spam breaking the layout. Also, TUI renders error in line, keeping stable UI and keepig user with right error message.
• Maintains consistency: Matches how OpenShell already handle logs / error across.
• Improves debugging: a TUI dev/user/agent can actually read them or tail -f them in another terminal.
Checklist
Beta Was this translation helpful? Give feedback.
All reactions