Skip to content

fix(vscode-extension): prompt for confirmation before loading workspa…#68864

Merged
alxhub merged 1 commit into
angular:21.2.xfrom
atscott:tsdkharden212
May 22, 2026
Merged

fix(vscode-extension): prompt for confirmation before loading workspa…#68864
alxhub merged 1 commit into
angular:21.2.xfrom
atscott:tsdkharden212

Conversation

@atscott
Copy link
Copy Markdown
Contributor

@atscott atscott commented May 21, 2026

…ce tsdk

Harden the typescript.tsdk and js/ts.tsdk.path settings loading in the VS Code extension client.

This change hardens tsdk loading by:

  1. Ignoring workspace-level tsdk paths in untrusted workspaces.
  2. Prompting the user for explicit confirmation before loading a workspace-level tsdk path in trusted workspaces, and saving the approval state in a secure, local workspaceState.

…ce tsdk

Harden the typescript.tsdk and js/ts.tsdk.path settings loading
in the VS Code extension client.

This change hardens tsdk loading by:
1. Ignoring workspace-level tsdk paths in untrusted workspaces.
2. Prompting the user for explicit confirmation before loading a
   workspace-level tsdk path in trusted workspaces, and saving the
   approval state in a secure, local workspaceState.
@atscott atscott added the target: patch This PR is targeted for the next patch release label May 21, 2026
@angular-robot angular-robot Bot added the area: vscode-extension Issues related to the Angular Language Service VsCode extension label May 21, 2026
@ngbot ngbot Bot added this to the Backlog milestone May 21, 2026
@atscott atscott requested a review from alan-agius4 May 21, 2026 18:11
@atscott atscott added the action: merge The PR is ready for merge by the caretaker label May 22, 2026
@alxhub alxhub merged commit 82cf38a into angular:21.2.x May 22, 2026
23 checks passed
@alxhub
Copy link
Copy Markdown
Member

alxhub commented May 22, 2026

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: vscode-extension Issues related to the Angular Language Service VsCode extension target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants