Fix to respect environment reporting setting in fish (microsoft#242234)

* fix to respect env setting in fish

* take the if clause outside of __vsc_update_env

Author: anthonykim1

src/vs/workbench/contrib/terminal/common/scripts/shellIntegration.fish

@@ -21,6 +21,8 @@ and ! set --query VSCODE_SHELL_INTEGRATION
 or exit
 
 set --global VSCODE_SHELL_INTEGRATION 1
+set --global __vscode_shell_env_reporting $VSCODE_SHELL_ENV_REPORTING
+set -e VSCODE_SHELL_ENV_REPORTING
 
 # Apply any explicit path prefix (see #99878)
 # On fish, '$fish_user_paths' is always prepended to the PATH, for both login and non-login shells, so we need
@@ -149,14 +151,16 @@ function __vsc_update_cwd --on-event fish_prompt
 	end
 end
 
-function __vsc_update_env --on-event fish_prompt
-	__vsc_esc EnvSingleStart 1
-	for line in (env)
-		set myVar (echo $line | awk -F= '{print $1}')
-		set myVal (echo $line | awk -F= '{print $2}')
-		__vsc_esc EnvSingleEntry $myVar (__vsc_escape_value "$myVal")
+if test "$__vscode_shell_env_reporting" = "1"
+	function __vsc_update_env --on-event fish_prompt
+		__vsc_esc EnvSingleStart 1
+		for line in (env)
+			set myVar (echo $line | awk -F= '{print $1}')
+			set myVal (echo $line | awk -F= '{print $2}')
+			__vsc_esc EnvSingleEntry $myVar (__vsc_escape_value "$myVal")
+		end
+		__vsc_esc EnvSingleEnd
 	end
-	__vsc_esc EnvSingleEnd
 end
 
 # Sent at the start of the prompt.