rany2
rany2
aiohttp doesn't support this out of the box, I seem to need something like https://pypi.org/project/aiohttp-socks/ as well.
I don't see anywhere in your change where you escape shell input. I'd much prefer if the prompt was in a file instead
@bernatvadell There is no way your change made a difference. Try with a single-quote to get it to obviously not work. If you end up closing the quote it does...
If you need a description, $(ls) ended up just running and the output of ls got sent to the model. This is not what should happen under any circumstances...
IMHO the most proper way to get around this is to put the prompt in some temp file and have the model read from it (using LLAMA's -f). I recommend...
@bernatvadell Yes that would be ideal, no idea why it is spawning in a shell...
Why do you escape when you could just call exec directly? You don't need to exec it in a shell ..
Anyway this proposal makes me a bit uneasy because in different shells you need to escape different things. For example if user was using fish, then `(ls)` would execute also....
Is executing it in a shell still needed? I think you could call node's spawn directly now.
Fantastic, thank you. That's a fast pace!