--test-cmd is broken

[Thanatermesis]

Issue

Simple test:

echo foo > test.txt
cecli --yes-always --test-cmd "true" --auto-test --message "change foo to bar" test.txt

For some reason it tries to run the cmd using my shell or something like that, I know that because it shows my PS prompt, but the command should be simply run as a command, not from a shell and even less from a "login mode shell", this leads also to issues making the feature broken:

And for some reason it detects that the test has failed (not listening to return status? only sees if there's a returned message? which on the second case is confused by my shell PS)

so instead of continuing ("true" returns no message and the successful exit status) it just try to fix the (non) errors in a loop

Note: this bug may affect too to --lint-cmd

Version and model info

No response

[dwash96]

Hmm, so the test and lint commands are meant to be ran by the terminal and, in powershell, "true" is not a valid command the way it would be in a linux shell. I think what you'd want to do is

cecli --yes-always --test-cmd "{valid command or path to ps1 script}" --auto-test --message "change foo to bar" test.txt

I personally in my .cecli.conf.yml file have

auto-test: true
test-cmd: pre-commit run --all-files

[Thanatermesis]

So the test always "fail" no matter what, running again the tool over and over trying to fix "something that is not needed to fix"

Note: when I want to use the --test-cmd feature I need to switch to aider, on which it works correctly (proof that something is happening in cecli that doesnt works well)