From 4c3beb56ca7f3df64d5dc4b2b7ced3d16bee09cf Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 5 Dec 2025 19:18:29 +0000 Subject: [PATCH 1/2] Initial plan From 58a304b2c9ee467c9d784b37a8d3d653ce9ec9d5 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 5 Dec 2025 19:25:11 +0000 Subject: [PATCH 2/2] Remove VSCode install extension workaround and update pipelines to use insiders - Remove the retry loop workaround for https://github.com/microsoft/vscode/issues/256031 in test/vscodeLauncher.ts since the issue has been fixed in vscode-insiders - Update azure-pipelines.yml to always use insiders version instead of conditionally using stable/insiders - Update azure-pipelines/profiling.yml to use insiders version Co-authored-by: dibarbet <5749229+dibarbet@users.noreply.github.com> --- azure-pipelines.yml | 5 +---- azure-pipelines/profiling.yml | 2 +- test/vscodeLauncher.ts | 19 +------------------ 3 files changed, 3 insertions(+), 23 deletions(-) diff --git a/azure-pipelines.yml b/azure-pipelines.yml index 62f00b16cf..053fd60db6 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -56,10 +56,7 @@ schedules: variables: - name: testVSCodeVersion - ${{ if eq( variables['Build.Reason'], 'Schedule' ) }}: - value: insiders - ${{ else }}: - value: stable + value: insiders stages: - template: azure-pipelines/build-vsix.yml diff --git a/azure-pipelines/profiling.yml b/azure-pipelines/profiling.yml index c55328c124..190b1ad275 100644 --- a/azure-pipelines/profiling.yml +++ b/azure-pipelines/profiling.yml @@ -56,5 +56,5 @@ extends: parameters: installDotNet: true npmCommand: profiling - testVSCodeVersion: stable + testVSCodeVersion: insiders isIntegration: true \ No newline at end of file diff --git a/test/vscodeLauncher.ts b/test/vscodeLauncher.ts index a4fdbf3dd6..8627507a89 100644 --- a/test/vscodeLauncher.ts +++ b/test/vscodeLauncher.ts @@ -69,24 +69,7 @@ export async function prepareVSCodeAndExecuteTests( async function installExtensions(extensionIds: string[], vscodeCli: string, vscodeArgs: string[]): Promise { for (const extensionId of extensionIds) { - // Workaround for https://github.com/microsoft/vscode/issues/256031 to retry installing the extension with a delay. - let installError: any | undefined = undefined; - let installSucceeded = false; - for (let attempts = 0; attempts < 5; attempts++) { - try { - await installExtension(extensionId, vscodeCli, vscodeArgs); - installSucceeded = true; - break; - } catch (error) { - console.warn(`Failed to install extension ${extensionId}; retrying: ${error}`); - installError = error; - await new Promise((resolve) => setTimeout(resolve, 2000)); - } - } - - if (!installSucceeded) { - throw installError; - } + await installExtension(extensionId, vscodeCli, vscodeArgs); } console.log();