fix(cli): --out writes only to the file, not also to stdout - #371

--out is meant to redirect output to a file. But every command that takes --out shares one helper — _write_or_print — which, despite its name, wrote the file and still printed the whole report to stdout:

def _write_or_print(text, out):
    if out is not None:
        out.parent.mkdir(parents=True, exist_ok=True)
        out.write_text(text, encoding="utf-8")
    print(text, end="")   # ran even when out was set

So patchrail ci explain --log ci.log --out report.md flooded the terminal with the report a first-timer had just asked to send to a file, and a CI script capturing stdout while using --out got the report duplicated into its pipe (having to add > /dev/null to work around it).

Fix: --out is now silent on stdout — matching curl -o, gcc -o, openssl … -out and the helper's own name (_write_or_print = write or print). Running without --out prints to stdout exactly as before.

Shared by every --out command (ci explain/classify/classes, redact, evidence …, and the rest).

Before / after

# before:  report written to file AND dumped to terminal
$ patchrail ci explain --log ci.log --out report.md
# PatchRail CI Report
- Root cause: `node_dependency_install`
...            # <-- whole report on stdout too

# after:  file written, stdout clean
$ patchrail ci explain --log ci.log --out report.md
$ cat report.md
# PatchRail CI Report
...

Tests

• New regression test test_ci_explain_out_writes_file_without_teeing_stdout: exit 0, stdout empty, file contains the report.
• Full suite green (947 passed, 739 subtests) + ruff clean.

Marca: Pablo Guillén · PatchRail · getpatchrail.com