scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-08-28 05:27:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Fix doctor trycmd test: move env vars to [env.add] section

Browse source 
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) <noreply@anthropic.com>
This commit is contained in:
Bryan Helmkamp 2026-03-14 17:28:44 -04:00
parent 589a98ec6e
commit 9dedaa9e39
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/crates/fabro-cli/tests/cmd/doctor/dry-run-flag.toml
View file

@ -3,5 +3,7 @@ args = ["doctor", "--dry-run"]
[env]
inherit = false
[env.add]
HOME = "/tmp/fabro-trycmd-nonexistent"
ANTHROPIC_API_KEY = "sk-test-dummy"