From 5b222f94b3e65be5da2ca6982583363b36de428c Mon Sep 17 00:00:00 2001 From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com> Date: Fri, 21 Aug 2026 00:25:12 +0000 Subject: [PATCH] [Refactor] Simplify isVSCode implementation Pass `.vscode` directly as the search target to `findPathUp` with `{cwd: root}` option, eliminating unnecessary path joining and removing redundant `fileExists` check. --- packages/cli-kit/src/public/node/vscode.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/cli-kit/src/public/node/vscode.ts b/packages/cli-kit/src/public/node/vscode.ts index fa9846c6b8e..05c3c76eb62 100644 --- a/packages/cli-kit/src/public/node/vscode.ts +++ b/packages/cli-kit/src/public/node/vscode.ts @@ -14,13 +14,9 @@ export async function isVSCode(root = cwd()): Promise { root, )} or any of its parents has a .vscode directory... `, ) - const config = await findPathUp(joinPath(root, '.vscode'), {type: 'directory'}) + const config = await findPathUp('.vscode', {cwd: root, type: 'directory'}) - if (!config) { - return false - } - - return fileExists(config) + return Boolean(config) } /**