From 9dedaa9e393d451d19da8d9755a2562cdf48d8ce Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Sat, 14 Mar 2026 17:28:44 -0400 Subject: [PATCH] Fix doctor trycmd test: move env vars to [env.add] section MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit trycmd's Env struct requires env vars under [env.add], not directly under [env]. Vars placed directly under [env] are silently ignored by serde, so the subprocess ran with a fully cleared env. On CI this caused dirs::home_dir() to fall back to passwd, loading the real cli.toml (with app_id) but without GITHUB_APP_PRIVATE_KEY → partial config error. Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml b/lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml index d7fe8cb42..42757fac8 100644 --- a/lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml +++ b/lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml @@ -3,5 +3,7 @@ args = ["doctor", "--dry-run"] [env] inherit = false + +[env.add] HOME = "/tmp/fabro-trycmd-nonexistent" ANTHROPIC_API_KEY = "sk-test-dummy"