Skip to content

fix(@schematics/angular): remove default workspace vscode mcp.json configuration#33402

Merged
clydin merged 1 commit into
angular:mainfrom
clydin:refactor/remove-default-workspace-mcp
Jun 17, 2026
Merged

fix(@schematics/angular): remove default workspace vscode mcp.json configuration#33402
clydin merged 1 commit into
angular:mainfrom
clydin:refactor/remove-default-workspace-mcp

Conversation

@clydin

@clydin clydin commented Jun 17, 2026

Copy link
Copy Markdown
Member

The workspace schematic previously generated a default .vscode/mcp.json file when initializing a new workspace. However, this configuration is now managed and optionally generated via the ai-config schematic (either during ng new tool setup or through direct schematic invocation).

…nfiguration

The workspace schematic previously generated a default `.vscode/mcp.json` file when initializing a new workspace. However, this configuration is now managed and optionally generated via the `ai-config` schematic (either during `ng new` tool setup or through direct schematic invocation).
@clydin clydin added the target: patch This PR is targeted for the next patch release label Jun 17, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the MCP configuration file (.vscode/mcp.json) and deletes its corresponding assertions from the workspace schematic tests. There are no review comments to evaluate, and I have no feedback to provide.

@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jun 17, 2026
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 17, 2026
@clydin clydin merged commit 01008f1 into angular:main Jun 17, 2026
69 of 70 checks passed
@clydin

clydin commented Jun 17, 2026

Copy link
Copy Markdown
Member Author

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

@clydin clydin deleted the refactor/remove-default-workspace-mcp branch June 17, 2026 23:10
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: @schematics/angular target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants