[perf] Replace only-path-without-wrappers with sed

[mikeland73]

Summary

This replaces our existing only-path-without-wrappers wrapper implementation to use sed.

Context:

We use only-path-without-wrappers to remove bin wrappers in the environment of our binaries. This is particularly useful when a binary calls another binary. See #1160 for more context. Unfortunately this adds a perf hit to every single binary call. In my informal experiments this cost was ~40ms. For example:

Previously, time go version was taking ~60ms even if environment is fully up to date.
Now, time go version takes ~20ms.

@Lagoja I'm getting an error in the elixer example (since that was the motivation for the original fix) but it looks like nix elixer on Sonoma is broken: https://elixirforum.com/t/bus-error-after-upgrading-to-sonoma-beta/56354/33 (Our CICD runners will test on Ubuntu and older macOS, Monterrey I believe)

How was it tested?

• Tested time to call go version before and after
• Inspected PATH of shell to ensure .wrappers was still there.
• Copy pasted sed code from one of the bin wrappers

[savil]

nice!

[savil]

lets confirm this prints to Stderr and not Stdout, otherwise existing bin-wrappers may not work.

That said, I suppose we will re-generate the bin-wrappers due to the Devbox version change when this is released, so may be not a concern either way.

[mikeland73]

Good call. It's actually OutOrStdErr which is currently Stderr, but we can't guarantee that. Will remove.

[savil]

oh interesting. Availability was one of the concerns earlier.

[mikeland73]

@savil if it's a concern we can implement in pure bash. Here's an implementation that only uses bash builtins:

IFS=: read -ra dirs <<< "$PATH"  # Split PATH into an array using colon as a delimiter

new_PATH=""
for dir in "${dirs[@]}"; do
    if [[ "$dir" != "/path/to/remove" ]]; then
        if [[ -z "$new_PATH" ]]; then
            new_PATH="$dir"
        else
            new_PATH="$new_PATH:$dir"
        fi
    fi
done

export PATH="$new_PATH"

[savil]

oh not too bad. That's fairly understandable except for:

IFS=: read -ra dirs <<< "$PATH" 

:D

[savil]

but now that makes sense too...