Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Starting in v2024.4.1, extension mangles vscode integrated terminal path #23236

Closed
djsavvy opened this issue Apr 15, 2024 · 4 comments
Closed
Assignees
Labels
triage-needed Needs assignment to the proper sub-team

Comments

@djsavvy
Copy link

djsavvy commented Apr 15, 2024

Type: Bug

Starting in v2024.4.1, this extension started mangling my integrated terminal path. Now, for instance, I can't run yarn dev because the dependencies in the scripts are no longer found.

Extension version: 2024.4.1
VS Code version: Code 1.88.1 (e170252f762678dec6ca2cc69aba1570769a5d39, 2024-04-10T17:41:02.734Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info
Item Value
CPUs Intel(R) Xeon(R) W-1370P @ 3.60GHz (16 x 3600)
GPU Status 2d_canvas: enabled
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: enabled
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
Load (avg) undefined
Memory (System) 31.75GB (11.41GB free)
Process Argv --crash-reporter-id 26bf80c7-ebf8-4932-96de-41d3d273eedc
Screen Reader no
VM 0%
A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
vsaa593:30376534
py29gd2263:30899288
c4g48928:30535728
azure-dev_surveyone:30548225
962ge761:30959799
pythongtdpath:30769146
welcomedialogc:30910334
pythonidxpt:30866567
pythonnoceb:30805159
asynctok:30898717
pythontestfixt:30902429
pythonregdiag2:30936856
pyreplss1:30897532
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30995554
dsvsc016:30899300
dsvsc017:30899301
dsvsc018:30899302
cppperfnew:31000557
8082a590:30971561
fegfb526:30981948
bg6jg535:30979843
ccp2r3:30993541
dsvsc020:30976470
pythonait:31006305
jchc7451:30973076
gee8j676:31009558
dsvsc021:30996838
g1icg217:30999571
jg8ic977:31013176

@github-actions github-actions bot added the triage-needed Needs assignment to the proper sub-team label Apr 15, 2024
@karthiknadig
Copy link
Member

@djsavvy Can you provide more details? is the PATH env variable not right or is it something else? Also, can you share the logs from Output > Python.

@github-actions github-actions bot added the info-needed Issue requires more information from poster label Apr 15, 2024
@karthiknadig karthiknadig self-assigned this Apr 16, 2024
@djsavvy
Copy link
Author

djsavvy commented Apr 17, 2024

Yeah the PATH env variable is not right. If I copy the PATH env variable from Windows Terminal and set it in the integrated terminal, then my yarn commands work. I also used Extension bisect to narrow it down to this extension specifically.

Strangely, when the PATH variable is messed up, it seems to be a superset of the good PATH. A lot of the directories seem to be duplicated.

I tried messing around with it a bit more today with various combinations of disabling/enabling the extension, and installing/updating/uninstalling. It seems like the issue was being triggered intermittently. I was able to see it in 2024.4.0, but also managed to get a good state with 2024.4.1 after a reinstall.

Is it possible that reinstalling resets the a/b testing, and that the issue is actually because an A/B test? In that case it might be a specific combination of some A/B test and the 2024.4.1 version that triggers the bug.

@github-actions github-actions bot removed the info-needed Issue requires more information from poster label Apr 17, 2024
@djsavvy
Copy link
Author

djsavvy commented Apr 17, 2024

To be honest, I'm suspecting that one of the A/B tests in my issue above is causing the bug. I opted out of all tests in my settings and reinstalled the plugin and am not seeing the issue anymore.

@karthiknadig
Copy link
Member

This may have been caused by pythonTerminalEnvVarActivation experiment. We are working on closing that soon, as this seems to break env variables and shell aliases in weird ways.

@karthiknadig karthiknadig closed this as not planned Won't fix, can't repro, duplicate, stale Dec 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage-needed Needs assignment to the proper sub-team
Projects
None yet
Development

No branches or pull requests

2 participants