Fix env race flake in cache_env scrub self-tests

[Mikola Lysenko (mikolalysenko)]

Problem

coverage (part of the ci-ok gate) failed on two PR runs in the last 24h with one cargo test target red and nothing else wrong:

• run 38194961803 (agent/fix-ci-test-helper-copies-5): -p socket-patch-cli --test in_process_edge_cases failed; attempt 2 on the same SHA passed, so it was a flake.
• run 38184196576 (agent/fix-go-selected-version-gate): -p socket-patch-cli --test vendor failed. That binary also links common/cache_env.rs and runs in-process tests. The failing test name isn't in the 5000-line log tail I could read, so I can't confirm this one is the same flake.

Running the in_process_edge_cases binary locally 60 times with --test-threads=16 failed 8 times. Every failure was one of two self-tests:

common::cache_env::cache_env_selftests::ambient_bun_scrub_removes_only_matching_parent_vars
  panicked at tests/common/cache_env.rs:392: SOCKET_OFFLINE is set in the parent env but was not scrubbed
common::cache_env::cache_env_selftests::ambient_vlt_scrub_removes_only_matching_parent_vars
  panicked at tests/common/cache_env.rs:451: SOCKET_OFFLINE kept

common/cache_env.rs is included through common/mod.rs in about 120 test binaries, so the flake can turn up in any of them that also runs in-process commands. In the merge queue, that evicts the entry.

Root cause

Each self-test reads std::env::vars_os() twice: once inside scrub_ambient_*_env and once to check that every matching parent var was removed. These self-tests are not #[serial], so they run at the same time as the binary's #[serial] in-process tests. Those tests call apply::run etc., and apply_env_toggles (src/args.rs) runs std::env::set_var(\"SOCKET_OFFLINE\", \"1\") for --offline. If the var appears between the two reads, the second read sees a matching var that the scrub never saw.

Fix

Read the parent env once (parent_var_names). Pass that one snapshot to both the scrub (scrub_parent_vars) and the self-test's assertions. The public scrub_ambient_bun_env / scrub_ambient_vlt_env behave exactly as before: they read the env once and remove the matching names. No assertion is weakened. The self-tests still check that the scrub removes exactly the matching names, now against a consistent snapshot.

Proof

• Before: in_process_edge_cases failed 8 of 60 runs with --test-threads=16.
• After: 0 of 200 runs failed with the same command.
• cargo test -p socket-patch-cli --test vendor --test cli_parse_get cache_env passes.
• rustfmt run on the touched file only. Clippy reports nothing new in cache_env.rs.

No tests were moved or removed.

🤖 Generated with Claude Code

https://claude.ai/code/session_018js6KeL55GXJTLwgF3bQWV

---

Generated by Claude Code

[Mikola Lysenko (mikolalysenko)]

bugbot run

---

Generated by Claude Code

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit 8f29f99. Configure here.}

[Mikola Lysenko (mikolalysenko)]

Ready for review (burn-down agent)

• Head: 8f29f99
• CI: 35/35 non-skipped checks green (19 skipped by path filters).
• Bugbot: reviewed 8f29f99, no findings; no open review threads.
• Mergeable, no conflicts; no CHANGELOG change.
• Slack announcement: not sent this run (Slack send tool unavailable in this session); next run retries.

---

Generated by Claude Code

[Mikola Lysenko (mikolalysenko)]

Final review brief

What it does. Fixes a test-only flake in tests/common/cache_env.rs (linked into ~120 CLI test binaries). The ambient bun/vlt scrub self-tests read the parent env twice, and in-process tests in the same binary can set SOCKET_OFFLINE between the reads. The scrub and its self-test assertions now share one snapshot of the parent env names.

Risk: low. Test helper only. No product code changes, no assertions weakened, and the public scrub_ambient_bun_env / scrub_ambient_vlt_env still read the env once and remove the same names.

Look here

• cache_env.rs:293: parent_var_names, the single env read
• cache_env.rs:302: scrub_parent_vars, shared by the scrubs and the self-tests
• cache_env.rs:387 and :450: the two self-tests now assert against the same snapshot

Verified. Read the full diff (one file). Checked the other self-tests in the file for the same double-read pattern; none of them read a var that in-process tests set. CHANGELOG.md is untouched. CI on 8f29f99: ci-ok and clippy green, 0 failed checks. Bugbot: no issues.

Changes I made. None.

Open questions. None.

Auto-merge is armed, so approving sends it straight to the merge queue.

---

Generated by Claude Code