Intercept stdout of plugins to add a timestamp to each line

[subatiq]

Expected plugins output:

• Output formatting for Python plugins #14

[12-12-2023T12:12:12] dropper   > Doing some stuff... 
[12-12-2023T12:12:12] deathloop > Deathlooping like crazy

[subatiq]

@ilya-siluyanov let's collect every option for the solution here.

It must be platform-independent and compatible with any type of plugin.

Also, plugins themselves (the model) should be decoupled from the output formatter

[subatiq]

There is no way to do this in the standard library1. The gag crate allows redirecting stderr or stdout either to a file or to nothing, but it only works on *nix systems.

In a different view of the problem, I'd encourage you to not use stdout or stderr directly at all. Instead, use dependency injection to pass down values that can be written to. Instead of using println, use writeln.

https://stackoverflow.com/questions/54429389/how-to-redirect-stderr-from-inside-the-process-in-rust

[subatiq]

We can add logging implementation for every type of plugin

PyO3/pyo3#1918 (comment)

[ilya-siluyanov]

Seems like an old thread - https://doc.rust-lang.org/std/process/struct.Command.html#method.stderr
I don't see any problems to use stderr in a similar manner as here

It must be platform-independent

That seems legit, my solution doesn't use any method marked as *nix only

Compatible with any type of plugin

If I print a symbol without \n, should I assign it a timestamp? what if I will get another symbol then? I need to rewrite the previous timestamp? I think there should be an agreement like "assign a timestamp only to a line", because supporting some features might be too costly

[subatiq]

Also, there are methods for styling the output in stdout_styling.rs