[ybai08]

atac request send --console builds its output from (post_request_output, post_request_output) instead of (pre_request_output, post_request_output). So the pre-request script output is never printed, and when there is a post-request script its output is printed twice:

$ atac -d <dir> request send console/scripts --console --hide-content --dry-run
post ran

post ran

This PR changes the first element of the tuple to pre_request_output. The TUI's console tab (tui_logic/result_tabs.rs) already uses the right pair.

Test: tests/cli_console.rs runs the atac binary (CARGO_BIN_EXE_atac) on a collection in a temporary directory. The request has a pre- and a post-request script that both console.log, and it goes to a one-shot local TcpListener. The test checks that each output appears exactly once. On main it fails with left: 0, right: 1 and stdout "post-request output\n\npost-request output\n\n".

What was run:

• cargo test: 37 unit tests plus the new integration test pass. The integration test was run 3 more times and passed each time.
• cargo clippy --all-targets: same warning count as main.