Terminal User Interface Issues When Running Windows Executable from PowerShell Desktop (Version 0.1.33)

[sevehub]

When running the Windows exe directly from PowerShell desktop, I encounter issues with the terminal user interface. I'm unsure if this is related to Rust's Ratatui, Windows Terminal, PowerShell itself, or another factor. Interestingly, it works fine when executed from the command prompt. I'm using version 0.1.33.

[yassinebridi]

Thanks for the report.
There are some issues with windows but i'm not sure where is the problem exactly, i'm using a macos right now.
Once i get my hands on a windows computer i will fix these issues.

[sevehub]

Hope this will help you in debugging your app. When running the precompiled Windows executable (version 0.3.1 - 2024-07-17), executing 'serp.exe -V' results in an incorrect configuration path: 'Config directory: C:\Users\user\AppData\Roaming\yassinebridi\serpl\config'.

I had a glance at the source code, but I couldn't locate the problem. This issue may also occur with precompiled binaries on other platforms.

[yassinebridi]

Thanks, i didn't test the config on windows, the path is weird, but this shouldn't have been produced by a wrong config file, since the config file isn't necessary to run the application, the default config will be used.

[sevehub]

On Windows, what is the expected directory where the app sources its configuration file? %appdata%\serpl\config ?

[yassinebridi]

I'm using this package to manage config files:
https://github.com/lunacookies/etcetera

From their documentation, it seems that that the config should be inside ~\AppData\Roaming\serpl\config.json

Though from what you have posted earlier:

C:\Users\user\AppData\Roaming\yassinebridi\serpl\config

This might be the correct path too, windows might use org name(yassinebridi).
I will try to introduce an argument to serpl to supply your config file path.

[sevehub]

I will try to introduce an argument to serpl to supply your config file path.

This may be unnecessary if the application can correctly source the config file from ~\AppData\Roaming\serpl\config.json

[Eragra3]

Seems like RipGrep returns windows line ends \r\n and your TUI library doesn't like it.
If I remove it (.replace("\\r", "") here https://github.com/yassinebridi/serpl/blob/2a195169e14d3a871f31cac89bfef68c3524c048/src/redux/thunk/process_search.rs#L149C68-L149C72 it seems to work fine.

There seems to be another call here https://github.com/yassinebridi/serpl/blob/2a195169e14d3a871f31cac89bfef68c3524c048/src/redux/thunk/process_search.rs#L69C58-L69C64 but I haven't investigated very well.