From 88f30ec3087627d9d4180af2d98c7ffbd708fee4 Mon Sep 17 00:00:00 2001 From: Ben King <9087625+benfdking@users.noreply.github.com> Date: Thu, 19 Jun 2025 23:14:36 +0200 Subject: [PATCH] fix(vscode): include process path env variable in passed path --- vscode/extension/src/utilities/sqlmesh/sqlmesh.ts | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/vscode/extension/src/utilities/sqlmesh/sqlmesh.ts b/vscode/extension/src/utilities/sqlmesh/sqlmesh.ts index 9285f20c7c..06f05f7792 100644 --- a/vscode/extension/src/utilities/sqlmesh/sqlmesh.ts +++ b/vscode/extension/src/utilities/sqlmesh/sqlmesh.ts @@ -83,7 +83,7 @@ export const getTcloudBin = async (): Promise ...envVariables.value, PYTHONPATH: interpreterDetails.path[0], VIRTUAL_ENV: path.dirname(interpreterDetails.binPath!), - PATH: interpreterDetails.binPath!, + PATH: `${interpreterDetails.binPath!}${path.delimiter}${process.env.PATH || ''}`, }, args: [], }) @@ -308,7 +308,7 @@ export const sqlmeshExec = async (): Promise< ...envVariables.value, PYTHONPATH: interpreterDetails.path?.[0], VIRTUAL_ENV: path.dirname(path.dirname(interpreterDetails.binPath!)), // binPath now points to bin dir - PATH: interpreterDetails.binPath!, + PATH: `${interpreterDetails.binPath!}${path.delimiter}${process.env.PATH || ''}`, }, args: [], }) @@ -459,11 +459,11 @@ export const sqlmeshLspExec = async (): Promise< bin: binPath, workspacePath, env: { - PYTHONPATH: interpreterDetails.path?.[0], - VIRTUAL_ENV: path.dirname(path.dirname(interpreterDetails.binPath!)), // binPath now points to bin dir - PATH: interpreterDetails.binPath!, // binPath already points to the bin directory ...process.env, ...envVariables.value, + PYTHONPATH: interpreterDetails.path?.[0], + VIRTUAL_ENV: path.dirname(path.dirname(interpreterDetails.binPath!)), // binPath now points to bin dir + PATH: `${interpreterDetails.binPath!}${path.delimiter}${process.env.PATH || ''}`, // binPath already points to the bin directory }, args: [], })