@samrose Thanks for taking the time to review and recmmend! We certainly could tell devs to run nix-shell --pure in the install guide (although it is nice to have most of your comfy environment still available to you). But the concern here is that a new bash -c is spawned for every step of the zome build process, and at least one of the steps captures STDOUT and tries to do meaningful things with it. If any command in .bashrc outputs to STDOUT it borks the build process. So it's less about the host environment leaking into those build steps.

Do you know if --pure causes 'inner' invocations of bash to be pure as well, or is it just the immediate shell created by nix-shell?

I presume the app-spec-tests-sim2h failure is a red herring... looks like it all happens after app-spec is already built.

Hey @samrose just checking in on your change request -- given the extra details I shared, does this seem like a decent solution?

@zippy bumping this PR -- I think it was languishing for a while because it was blocked by some requested changes. I don't know if those requested changes pertain to the issue this is trying to fix though.

I've updated this branch to master and the diff is still clean. It should be an easy merge. The only thing I don't know is whether this causes unintended consequences. Are there times when the inner bash that hc package spawns should actually re-run the .bashrc? Are there times when not running it could break things for the dev? Nothing comes to mind, but hey, these are computers we're talking about -- you never know.