[INFO] cloning repository https://github.com/sonofnos/shell-t
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/sonofnos/shell-t" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsonofnos%2Fshell-t", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsonofnos%2Fshell-t'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 0a8358a0f67559885e310adc5c74777dd7ba8a14
[INFO] checking sonofnos/shell-t against try#53f0b05ecbcf011a1f77f1c627304e87ebd05801 for pr-154492
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsonofnos%2Fshell-t" "/workspace/builds/worker-3-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/sonofnos/shell-t
[INFO] finished tweaking git repo https://github.com/sonofnos/shell-t
[INFO] tweaked toml for git repo https://github.com/sonofnos/shell-t written to /workspace/builds/worker-3-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/sonofnos/shell-t on toolchain 53f0b05ecbcf011a1f77f1c627304e87ebd05801
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+53f0b05ecbcf011a1f77f1c627304e87ebd05801" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/sonofnos/shell-t already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+53f0b05ecbcf011a1f77f1c627304e87ebd05801" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+53f0b05ecbcf011a1f77f1c627304e87ebd05801" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] 14a22ebe62a113604447bf1345a1053e89830735780c76c7333fd32952a8fb9d
[INFO] running `Command { std: "docker" "start" "-a" "14a22ebe62a113604447bf1345a1053e89830735780c76c7333fd32952a8fb9d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "14a22ebe62a113604447bf1345a1053e89830735780c76c7333fd32952a8fb9d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "14a22ebe62a113604447bf1345a1053e89830735780c76c7333fd32952a8fb9d", kill_on_drop: false }`
[INFO] [stdout] 14a22ebe62a113604447bf1345a1053e89830735780c76c7333fd32952a8fb9d
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+53f0b05ecbcf011a1f77f1c627304e87ebd05801" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] e39909554efdc3dd417647cf8c1fe460182bb6e0f1769c68e6240e3b357cf1c4
[INFO] running `Command { std: "docker" "start" "-a" "e39909554efdc3dd417647cf8c1fe460182bb6e0f1769c68e6240e3b357cf1c4", kill_on_drop: false }`
[INFO] [stderr]    Compiling libc v0.2.175
[INFO] [stderr]    Compiling proc-macro2 v1.0.101
[INFO] [stderr]    Compiling unicode-ident v1.0.19
[INFO] [stderr]    Compiling signal-hook v0.3.18
[INFO] [stderr]     Checking cfg-if v1.0.3
[INFO] [stderr]    Compiling rustix v0.38.44
[INFO] [stderr]    Compiling serde v1.0.219
[INFO] [stderr]     Checking memchr v2.7.5
[INFO] [stderr]     Checking bitflags v2.9.4
[INFO] [stderr]     Checking linux-raw-sys v0.4.15
[INFO] [stderr]     Checking regex-syntax v0.8.6
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]     Checking either v1.15.0
[INFO] [stderr]     Checking home v0.5.11
[INFO] [stderr]     Checking bytes v1.10.1
[INFO] [stderr]    Compiling quote v1.0.40
[INFO] [stderr]     Checking aho-corasick v1.1.3
[INFO] [stderr]    Compiling syn v2.0.106
[INFO] [stderr]     Checking signal-hook-registry v1.4.6
[INFO] [stderr]     Checking parking_lot_core v0.9.11
[INFO] [stderr]     Checking mio v0.8.11
[INFO] [stderr]     Checking socket2 v0.6.0
[INFO] [stderr]     Checking mio v1.0.4
[INFO] [stderr]     Checking parking_lot v0.12.4
[INFO] [stderr]     Checking signal-hook-mio v0.2.4
[INFO] [stderr]     Checking crossterm v0.27.0
[INFO] [stderr]     Checking which v6.0.3
[INFO] [stderr]     Checking regex-automata v0.4.10
[INFO] [stderr]     Checking chrono v0.4.42
[INFO] [stderr]    Compiling tokio-macros v2.5.0
[INFO] [stderr]     Checking regex v1.11.2
[INFO] [stderr]     Checking tokio v1.47.1
[INFO] [stderr]     Checking shell-t v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused imports: `Color`, `Print`, `ResetColor`, and `SetForegroundColor`
[INFO] [stdout]  --> src/main.rs:1:24
[INFO] [stdout]   |
[INFO] [stdout] 1 | use crossterm::style::{Color, Print, ResetColor, SetForegroundColor};
[INFO] [stdout]   |                        ^^^^^  ^^^^^  ^^^^^^^^^^  ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crossterm::ExecutableCommand`
[INFO] [stdout]  --> src/main.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use crossterm::ExecutableCommand;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Write`
[INFO] [stdout]  --> src/main.rs:3:21
[INFO] [stdout]   |
[INFO] [stdout] 3 | use std::io::{self, Write};
[INFO] [stdout]   |                     ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::process::Command`
[INFO] [stdout]  --> src/main.rs:4:5
[INFO] [stdout]   |
[INFO] [stdout] 4 | use std::process::Command;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Path`
[INFO] [stdout]  --> src/security.rs:2:17
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::path::{Path, PathBuf};
[INFO] [stdout]   |                 ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::ffi::OsStr`
[INFO] [stdout]    --> src/security.rs:130:9
[INFO] [stdout]     |
[INFO] [stdout] 130 |     use std::ffi::OsStr;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::path::PathBuf`
[INFO] [stdout]  --> src/builtins.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::path::PathBuf;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::process`
[INFO] [stdout]  --> src/builtins.rs:3:5
[INFO] [stdout]   |
[INFO] [stdout] 3 | use std::process;
[INFO] [stdout]   |     ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ShellError`
[INFO] [stdout]  --> src/builtins.rs:8:33
[INFO] [stdout]   |
[INFO] [stdout] 8 | use crate::error::{ShellResult, ShellError};
[INFO] [stdout]   |                                 ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `Color`, `Print`, `ResetColor`, and `SetForegroundColor`
[INFO] [stdout]  --> src/main.rs:1:24
[INFO] [stdout]   |
[INFO] [stdout] 1 | use crossterm::style::{Color, Print, ResetColor, SetForegroundColor};
[INFO] [stdout]   |                        ^^^^^  ^^^^^  ^^^^^^^^^^  ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crossterm::ExecutableCommand`
[INFO] [stdout]  --> src/main.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use crossterm::ExecutableCommand;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Write`
[INFO] [stdout]  --> src/main.rs:3:21
[INFO] [stdout]   |
[INFO] [stdout] 3 | use std::io::{self, Write};
[INFO] [stdout]   |                     ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::process::Command`
[INFO] [stdout]  --> src/main.rs:4:5
[INFO] [stdout]   |
[INFO] [stdout] 4 | use std::process::Command;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Path`
[INFO] [stdout]  --> src/security.rs:2:17
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::path::{Path, PathBuf};
[INFO] [stdout]   |                 ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::ffi::OsStr`
[INFO] [stdout]    --> src/security.rs:130:9
[INFO] [stdout]     |
[INFO] [stdout] 130 |     use std::ffi::OsStr;
[INFO] [stdout]     |         ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::path::PathBuf`
[INFO] [stdout]  --> src/builtins.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::path::PathBuf;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::process`
[INFO] [stdout]  --> src/builtins.rs:3:5
[INFO] [stdout]   |
[INFO] [stdout] 3 | use std::process;
[INFO] [stdout]   |     ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `ShellError`
[INFO] [stdout]  --> src/builtins.rs:8:33
[INFO] [stdout]   |
[INFO] [stdout] 8 | use crate::error::{ShellResult, ShellError};
[INFO] [stdout]   |                                 ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0609]: no field `command_whitelist` on type `SecurityConfig`
[INFO] [stdout]    --> src/executor.rs:216:25
[INFO] [stdout]     |
[INFO] [stdout] 216 |         config.security.command_whitelist = Some(vec!["ls".to_string(), "pwd".to_string()]);
[INFO] [stdout]     |                         ^^^^^^^^^^^^^^^^^ unknown field
[INFO] [stdout]     |
[INFO] [stdout]     = note: available fields are: `enable_logging`, `enable_auditing`, `max_command_length`, `max_arg_count`, `allowed_commands` ... and 3 others
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0609]: no field `command_whitelist` on type `SecurityConfig`
[INFO] [stdout]    --> src/executor.rs:228:25
[INFO] [stdout]     |
[INFO] [stdout] 228 |         config.security.command_whitelist = Some(vec!["ls".to_string(), "pwd".to_string()]);
[INFO] [stdout]     |                         ^^^^^^^^^^^^^^^^^ unknown field
[INFO] [stdout]     |
[INFO] [stdout]     = note: available fields are: `enable_logging`, `enable_auditing`, `max_command_length`, `max_arg_count`, `allowed_commands` ... and 3 others
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0609]: no field `command_blacklist` on type `SecurityConfig`
[INFO] [stdout]    --> src/executor.rs:240:25
[INFO] [stdout]     |
[INFO] [stdout] 240 |         config.security.command_blacklist = Some(vec!["rm".to_string(), "sudo".to_string()]);
[INFO] [stdout]     |                         ^^^^^^^^^^^^^^^^^ unknown field
[INFO] [stdout]     |
[INFO] [stdout]     = note: available fields are: `enable_logging`, `enable_auditing`, `max_command_length`, `max_arg_count`, `allowed_commands` ... and 3 others
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `config`
[INFO] [stdout]   --> src/security.rs:55:47
[INFO] [stdout]    |
[INFO] [stdout] 55 |     pub fn check_rate_limit(&self, key: &str, config: &Config) -> ShellResult<()> {
[INFO] [stdout]    |                                               ^^^^^^ help: if this is intentional, prefix it with an underscore: `_config`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0061]: this function takes 3 arguments but 1 argument was supplied
[INFO] [stdout]    --> src/main.rs:170:22
[INFO] [stdout]     |
[INFO] [stdout] 170 |         let result = execute_commands(&commands);
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^----------- two arguments of type `&BuiltinManager` and `&executor::CommandExecutor` are missing
[INFO] [stdout]     |
[INFO] [stdout] note: function defined here
[INFO] [stdout]    --> src/main.rs:67:4
[INFO] [stdout]     |
[INFO] [stdout]  67 | fn execute_commands(
[INFO] [stdout]     |    ^^^^^^^^^^^^^^^^
[INFO] [stdout]  68 |     commands: &[parser::Command],
[INFO] [stdout]  69 |     builtin_manager: &builtins::BuiltinManager,
[INFO] [stdout]     |     ------------------------------------------
[INFO] [stdout]  70 |     executor: &executor::CommandExecutor,
[INFO] [stdout]     |     ------------------------------------
[INFO] [stdout] help: provide the arguments
[INFO] [stdout]     |
[INFO] [stdout] 170 |         let result = execute_commands(&commands, /* &BuiltinManager */, /* &executor::CommandExecutor */);
[INFO] [stdout]     |                                                +++++++++++++++++++++++++++++++++++++++++++++++++++++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `config`
[INFO] [stdout]   --> src/security.rs:55:47
[INFO] [stdout]    |
[INFO] [stdout] 55 |     pub fn check_rate_limit(&self, key: &str, config: &Config) -> ShellResult<()> {
[INFO] [stdout]    |                                               ^^^^^^ help: if this is intentional, prefix it with an underscore: `_config`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `path_obj`
[INFO] [stdout]   --> src/error.rs:87:13
[INFO] [stdout]    |
[INFO] [stdout] 87 |         let path_obj = Path::new(path);
[INFO] [stdout]    |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_path_obj`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `active_processes` and `rate_limiter` are never read
[INFO] [stdout]   --> src/security.rs:13:5
[INFO] [stdout]    |
[INFO] [stdout] 12 | pub struct SecurityManager {
[INFO] [stdout]    |            --------------- fields in this struct
[INFO] [stdout] 13 |     active_processes: AtomicUsize,
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] 14 |     command_history: Mutex<HashMap<String, CommandStats>>,
[INFO] [stdout] 15 |     rate_limiter: Mutex<HashMap<String, Vec<Instant>>>,
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `can_start_process`, `register_process`, `check_rate_limit`, and `validate_input` are never used
[INFO] [stdout]   --> src/security.rs:36:12
[INFO] [stdout]    |
[INFO] [stdout] 25 | impl SecurityManager {
[INFO] [stdout]    | -------------------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 36 |     pub fn can_start_process(&self, config: &Config) -> ShellResult<()> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 47 |     pub fn register_process(&self) -> ProcessGuard {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 55 |     pub fn check_rate_limit(&self, key: &str, config: &Config) -> ShellResult<()> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 88 |     pub fn validate_input(&self, input: &str) -> ShellResult<()> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `ProcessGuard` is never constructed
[INFO] [stdout]    --> src/security.rs:116:12
[INFO] [stdout]     |
[INFO] [stdout] 116 | pub struct ProcessGuard<'a> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `sanitize_input` is never used
[INFO] [stdout]    --> src/security.rs:133:12
[INFO] [stdout]     |
[INFO] [stdout] 133 |     pub fn sanitize_input(input: &str, config: &Config) -> ShellResult<String> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `check_suspicious_patterns` is never used
[INFO] [stdout]    --> src/security.rs:156:8
[INFO] [stdout]     |
[INFO] [stdout] 156 |     fn check_suspicious_patterns(input: &str) -> ShellResult<()> {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_file_path` is never used
[INFO] [stdout]    --> src/security.rs:180:12
[INFO] [stdout]     |
[INFO] [stdout] 180 |     pub fn validate_file_path(path: &str, config: &Config) -> ShellResult<PathBuf> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_arguments` is never used
[INFO] [stdout]    --> src/security.rs:212:12
[INFO] [stdout]     |
[INFO] [stdout] 212 |     pub fn validate_arguments(args: &[String], config: &Config) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `execute_with_monitoring` is never used
[INFO] [stdout]    --> src/security.rs:241:18
[INFO] [stdout]     |
[INFO] [stdout] 241 |     pub async fn execute_with_monitoring(
[INFO] [stdout]     |                  ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `sanitize_environment` is never used
[INFO] [stdout]    --> src/security.rs:291:12
[INFO] [stdout]     |
[INFO] [stdout] 291 |     pub fn sanitize_environment() -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_environment` is never used
[INFO] [stdout]    --> src/security.rs:312:12
[INFO] [stdout]     |
[INFO] [stdout] 312 |     pub fn validate_environment() -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `is_elevated_privileges` is never used
[INFO] [stdout]    --> src/security.rs:337:8
[INFO] [stdout]     |
[INFO] [stdout] 337 |     fn is_elevated_privileges() -> bool {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variant `Warning` is never constructed
[INFO] [stdout]   --> src/builtins.rs:65:5
[INFO] [stdout]    |
[INFO] [stdout] 61 | pub enum BuiltinResult {
[INFO] [stdout]    |          ------------- variant in this enum
[INFO] [stdout] ...
[INFO] [stdout] 65 |     Warning(String),
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `BuiltinResult` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `security` and `config` are never read
[INFO] [stdout]   --> src/builtins.rs:71:5
[INFO] [stdout]    |
[INFO] [stdout] 70 | pub struct BuiltinManager {
[INFO] [stdout]    |            -------------- fields in this struct
[INFO] [stdout] 71 |     security: Arc<SecurityManager>,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout] 72 |     config: Config,
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: multiple methods are never used
[INFO] [stdout]    --> src/ui.rs:59:12
[INFO] [stdout]     |
[INFO] [stdout]  13 | impl UiManager {
[INFO] [stdout]     | -------------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout]  59 |     pub fn display_success(&self, message: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout]  72 |     pub fn display_error(&self, message: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout]  85 |     pub fn display_warning(&self, message: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout]  98 |     pub fn display_info(&self, message: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 111 |     pub fn display_timestamped(&self, message: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 121 |     pub fn clear_screen(&self) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 128 |     pub fn move_cursor(&self, x: u16, y: u16) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 135 |     pub fn get_terminal_size(&self) -> ShellResult<(u16, u16)> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `ProgressIndicator` is never constructed
[INFO] [stdout]    --> src/ui.rs:142:12
[INFO] [stdout]     |
[INFO] [stdout] 142 | pub struct ProgressIndicator {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `new`, `start`, `update`, `complete`, and `fail` are never used
[INFO] [stdout]    --> src/ui.rs:149:12
[INFO] [stdout]     |
[INFO] [stdout] 147 | impl ProgressIndicator {
[INFO] [stdout]     | ---------------------- associated items in this implementation
[INFO] [stdout] 148 |     /// Create a new progress indicator
[INFO] [stdout] 149 |     pub fn new(message: String, ui: UiManager) -> Self {
[INFO] [stdout]     |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 154 |     pub fn start(&self) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 159 |     pub fn update(&self, progress: f32) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 167 |     pub fn complete(&self) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 173 |     pub fn fail(&self, error: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `TableFormatter` is never constructed
[INFO] [stdout]    --> src/ui.rs:180:12
[INFO] [stdout]     |
[INFO] [stdout] 180 | pub struct TableFormatter {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `new`, `add_row`, and `display` are never used
[INFO] [stdout]    --> src/ui.rs:188:12
[INFO] [stdout]     |
[INFO] [stdout] 186 | impl TableFormatter {
[INFO] [stdout]     | ------------------- associated items in this implementation
[INFO] [stdout] 187 |     /// Create a new table formatter
[INFO] [stdout] 188 |     pub fn new(headers: Vec<String>, ui: UiManager) -> Self {
[INFO] [stdout]     |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 197 |     pub fn add_row(&mut self, row: Vec<String>) {
[INFO] [stdout]     |            ^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 202 |     pub fn display(&self) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `InputReader` is never constructed
[INFO] [stdout]    --> src/ui.rs:249:12
[INFO] [stdout]     |
[INFO] [stdout] 249 | pub struct InputReader {
[INFO] [stdout]     |            ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `new`, `read_line`, `previous_command`, `next_command`, and `display_history` are never used
[INFO] [stdout]    --> src/ui.rs:257:12
[INFO] [stdout]     |
[INFO] [stdout] 255 | impl InputReader {
[INFO] [stdout]     | ---------------- associated items in this implementation
[INFO] [stdout] 256 |     /// Create a new input reader
[INFO] [stdout] 257 |     pub fn new(ui: UiManager) -> Self {
[INFO] [stdout]     |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 266 |     pub fn read_line(&mut self, prompt: &str) -> ShellResult<String> {
[INFO] [stdout]     |            ^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 284 |     pub fn previous_command(&mut self) -> Option<&String> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 294 |     pub fn next_command(&mut self) -> Option<&String> {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 304 |     pub fn display_history(&self) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `interpreters` is never read
[INFO] [stdout]   --> src/config.rs:12:9
[INFO] [stdout]    |
[INFO] [stdout]  8 | pub struct Config {
[INFO] [stdout]    |            ------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 12 |     pub interpreters: InterpreterConfig,
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Config` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `enable_logging`, `enable_auditing`, `max_command_length`, `max_arg_count`, `validate_paths`, and `sanitize_input` are never read
[INFO] [stdout]   --> src/config.rs:18:9
[INFO] [stdout]    |
[INFO] [stdout] 17 | pub struct SecurityConfig {
[INFO] [stdout]    |            -------------- fields in this struct
[INFO] [stdout] 18 |     pub enable_logging: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^
[INFO] [stdout] 19 |     pub enable_auditing: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^
[INFO] [stdout] 20 |     pub max_command_length: usize,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 21 |     pub max_arg_count: usize,
[INFO] [stdout]    |         ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 24 |     pub validate_paths: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^
[INFO] [stdout] 25 |     pub sanitize_input: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `SecurityConfig` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `max_background_processes`, `command_timeout`, and `max_memory_mb` are never read
[INFO] [stdout]   --> src/config.rs:31:9
[INFO] [stdout]    |
[INFO] [stdout] 30 | pub struct ResourceLimits {
[INFO] [stdout]    |            -------------- fields in this struct
[INFO] [stdout] 31 |     pub max_background_processes: usize,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 32 |     pub max_pipeline_length: usize,
[INFO] [stdout] 33 |     pub command_timeout: u64,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^
[INFO] [stdout] 34 |     pub max_memory_mb: usize,
[INFO] [stdout]    |         ^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `ResourceLimits` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `show_timestamps` and `enable_completion` are never read
[INFO] [stdout]   --> src/config.rs:43:9
[INFO] [stdout]    |
[INFO] [stdout] 40 | pub struct UiConfig {
[INFO] [stdout]    |            -------- fields in this struct
[INFO] [stdout] ...
[INFO] [stdout] 43 |     pub show_timestamps: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^
[INFO] [stdout] 44 |     pub enable_completion: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `UiConfig` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `python_path`, `ruby_path`, `node_path`, `enable_scripts`, and `allowed_extensions` are never read
[INFO] [stdout]   --> src/config.rs:50:9
[INFO] [stdout]    |
[INFO] [stdout] 49 | pub struct InterpreterConfig {
[INFO] [stdout]    |            ----------------- fields in this struct
[INFO] [stdout] 50 |     pub python_path: String,
[INFO] [stdout]    |         ^^^^^^^^^^^
[INFO] [stdout] 51 |     pub ruby_path: String,
[INFO] [stdout]    |         ^^^^^^^^^
[INFO] [stdout] 52 |     pub node_path: String,
[INFO] [stdout]    |         ^^^^^^^^^
[INFO] [stdout] 53 |     pub enable_scripts: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^
[INFO] [stdout] 54 |     pub allowed_extensions: HashSet<String>,
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `InterpreterConfig` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `load`, `parse_toml`, `load_from_env`, `validate`, and `save` are never used
[INFO] [stdout]    --> src/config.rs:135:12
[INFO] [stdout]     |
[INFO] [stdout] 133 | impl Config {
[INFO] [stdout]     | ----------- associated items in this implementation
[INFO] [stdout] 134 |     /// Load configuration from file and environment variables
[INFO] [stdout] 135 |     pub fn load() -> Result<Self, Box<dyn std::error::Error>> {
[INFO] [stdout]     |            ^^^^
[INFO] [stdout] ...
[INFO] [stdout] 148 |     fn parse_toml(_content: &str) -> Result<Self, Box<dyn std::error::Error>> {
[INFO] [stdout]     |        ^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 153 |     fn load_from_env(&mut self) {
[INFO] [stdout]     |        ^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 182 |     pub fn validate(&self) -> Result<(), String> {
[INFO] [stdout]     |            ^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 203 |     pub fn save(&self) -> Result<(), Box<dyn std::error::Error>> {
[INFO] [stdout]     |            ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_command` is never used
[INFO] [stdout]    --> src/config.rs:214:12
[INFO] [stdout]     |
[INFO] [stdout] 214 |     pub fn validate_command(config: &Config, command: &str) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_args` is never used
[INFO] [stdout]    --> src/config.rs:234:12
[INFO] [stdout]     |
[INFO] [stdout] 234 |     pub fn validate_args(config: &Config, args: &[String]) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `Parse` and `Config` are never constructed
[INFO] [stdout]   --> src/error.rs:9:5
[INFO] [stdout]    |
[INFO] [stdout]  6 | pub enum ShellError {
[INFO] [stdout]    |          ---------- variants in this enum
[INFO] [stdout] ...
[INFO] [stdout]  9 |     Parse(String),
[INFO] [stdout]    |     ^^^^^
[INFO] [stdout] 10 |     SecurityViolation(String),
[INFO] [stdout] 11 |     Config(String),
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `ShellError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variants `PathTraversal`, `DangerousCommand`, `InvalidInput`, `PermissionDenied`, and `ResourceLimitExceeded` are never constructed
[INFO] [stdout]   --> src/error.rs:19:5
[INFO] [stdout]    |
[INFO] [stdout] 18 | pub enum SecurityError {
[INFO] [stdout]    |          ------------- variants in this enum
[INFO] [stdout] 19 |     PathTraversal(String),
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout] 20 |     DangerousCommand(String),
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] 21 |     InvalidInput(String),
[INFO] [stdout]    |     ^^^^^^^^^^^^
[INFO] [stdout] 22 |     PermissionDenied(String),
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^
[INFO] [stdout] 23 |     ResourceLimitExceeded(String),
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `SecurityError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_path` is never used
[INFO] [stdout]   --> src/error.rs:86:12
[INFO] [stdout]    |
[INFO] [stdout] 86 |     pub fn validate_path(path: &str) -> ShellResult<()> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `is_allowed_absolute_path` is never used
[INFO] [stdout]    --> src/error.rs:107:8
[INFO] [stdout]     |
[INFO] [stdout] 107 |     fn is_allowed_absolute_path(path: &str) -> bool {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `validate_command_args` is never used
[INFO] [stdout]    --> src/error.rs:121:12
[INFO] [stdout]     |
[INFO] [stdout] 121 |     pub fn validate_command_args(args: &[String]) -> ShellResult<()> {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `sanitize_input` is never used
[INFO] [stdout]    --> src/error.rs:138:12
[INFO] [stdout]     |
[INFO] [stdout] 138 |     pub fn sanitize_input(input: &str) -> String {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `log_security_event` is never used
[INFO] [stdout]    --> src/error.rs:152:12
[INFO] [stdout]     |
[INFO] [stdout] 152 |     pub fn log_security_event(event: &str, details: &str) {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `log_command_execution` is never used
[INFO] [stdout]    --> src/error.rs:168:12
[INFO] [stdout]     |
[INFO] [stdout] 168 |     pub fn log_command_execution(command: &str, user: &str) {
[INFO] [stdout]     |            ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/security.rs:47:29
[INFO] [stdout]    |
[INFO] [stdout] 47 |     pub fn register_process(&self) -> ProcessGuard {
[INFO] [stdout]    |                             ^^^^^     ^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                             |
[INFO] [stdout]    |                             the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 47 |     pub fn register_process(&self) -> ProcessGuard<'_> {
[INFO] [stdout]    |                                                   ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `executor`
[INFO] [stdout]    --> src/executor.rs:172:13
[INFO] [stdout]     |
[INFO] [stdout] 172 |         let executor = create_test_executor();
[INFO] [stdout]     |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_executor`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `path_obj`
[INFO] [stdout]   --> src/error.rs:87:13
[INFO] [stdout]    |
[INFO] [stdout] 87 |         let path_obj = Path::new(path);
[INFO] [stdout]    |             ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_path_obj`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] Some errors have detailed explanations: E0061, E0609.
[INFO] [stdout] 
[INFO] [stdout] For more information about an error, try `rustc --explain E0061`.
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `shell-t` (bin "shell-t" test) due to 4 previous errors; 12 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "e39909554efdc3dd417647cf8c1fe460182bb6e0f1769c68e6240e3b357cf1c4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "e39909554efdc3dd417647cf8c1fe460182bb6e0f1769c68e6240e3b357cf1c4", kill_on_drop: false }`
[INFO] [stdout] e39909554efdc3dd417647cf8c1fe460182bb6e0f1769c68e6240e3b357cf1c4
