rename and add comments for clarity

Author: savil

devbox.go

@@ -74,6 +74,11 @@ func PrintEnvrcContent(w io.Writer) error {
 	return impl.PrintEnvrcContent(w)
 }
 
-func OnlyPathWithoutWrappers() string {
-	return impl.OnlyPathWithoutWrappers()
+// ExportedSystemPathWithoutWrappers reads $PATH, removes `virtenv/.wrappers/bin` paths,
+// and returns a string of the form `export PATH=....`
+//
+// This small utility function could have been inlined in the boxcli caller, but
+// needed the impl.exportify functionality. It does not depend on core-devbox.
+func ExportedSystemPathWithoutWrappers() string {
+	return impl.ExportedSystemPathWithoutWrappers()
 }

internal/boxcli/shellenv.go

@@ -89,7 +89,7 @@ func shellEnvOnlyPathWithoutWrappersCmd() *cobra.Command {
 	command := &cobra.Command{
 		Use:     "only-path-without-wrappers",
 		Hidden:  true,
-		Short:   "Print shell commands that export PATH without the bin-wrappers",
+		Short:   "[internal] Print shell command that exports the system $PATH without the bin-wrappers paths.",
 		Args:    cobra.ExactArgs(0),
 		PreRunE: ensureNixInstalled,
 		RunE: func(cmd *cobra.Command, args []string) error {
@@ -102,5 +102,5 @@ func shellEnvOnlyPathWithoutWrappersCmd() *cobra.Command {
 }
 
 func shellEnvOnlyPathWithoutWrappersFunc() string {
-	return devbox.OnlyPathWithoutWrappers()
+	return devbox.ExportedSystemPathWithoutWrappers()
 }

internal/impl/devbox.go

@@ -332,25 +332,6 @@ func (d *Devbox) PrintEnv(ctx context.Context, includeHooks bool) (string, error
 	return envStr, nil
 }
 
-// OnlyPathWithoutWrappers is a small utility to filter WrapperBin paths from PATH
-func OnlyPathWithoutWrappers() string {
-
-	path := []string{}
-	for _, p := range strings.Split(os.Getenv("PATH"), string(filepath.ListSeparator)) {
-		// Intentionally do not include projectDir with plugin.WrapperBinPath so that
-		// we filter out bin-wrappers for devbox-global and devbox-project.
-		if !strings.Contains(p, plugin.WrapperBinPath) {
-			path = append(path, p)
-		}
-	}
-
-	envs := map[string]string{
-		"PATH": strings.Join(path, string(filepath.ListSeparator)),
-	}
-
-	return exportify(envs)
-}
-
 func (d *Devbox) ShellEnvHash(ctx context.Context) (string, error) {
 	envs, err := d.nixEnv(ctx)
 	if err != nil {
@@ -1231,3 +1212,22 @@ func (d *Devbox) convertEnvToMap(currentEnv []string) (map[string]string, error)
 	}
 	return env, nil
 }
+
+// ExportedSystemPathWithoutWrappers is a small utility to filter WrapperBin paths from PATH
+func ExportedSystemPathWithoutWrappers() string {
+
+	path := []string{}
+	for _, p := range strings.Split(os.Getenv("PATH"), string(filepath.ListSeparator)) {
+		// Intentionally do not include projectDir with plugin.WrapperBinPath so that
+		// we filter out bin-wrappers for devbox-global and devbox-project.
+		if !strings.Contains(p, plugin.WrapperBinPath) {
+			path = append(path, p)
+		}
+	}
+
+	envs := map[string]string{
+		"PATH": strings.Join(path, string(filepath.ListSeparator)),
+	}
+
+	return exportify(envs)
+}