build(@angular/cli): update @modelcontextprotocol/sdk to v1.24.1#32002
build(@angular/cli): update @modelcontextprotocol/sdk to v1.24.1#32002hlovdal wants to merge 1 commit intoangular:mainfrom
Conversation
Version 1.24.0 fixes GHSA-w48q-cv73-mx4w.
|
In my initial angular project repository where @angular/cli is a dependency I added the following as a workaround "overrides": {
"@angular/cli": {
"@modelcontextprotocol/sdk": "1.24.0"
}
}but it would be nice to have this fixed upstream. |
|
Thank you for the contribution. However, Renovate will automatically update the dependency once the minimum age threshold has been reached. Also of note is that the linked report does not affect the Angular CLI since it uses a stdio-based MCP server. Unfortunately, there is no mechanism to mark it as such. |
Hello. Is there any prediction on when the update will be done, and the release with the fix? Thanks. |
|
This will be released later today. |
|
@jpmartins-ca This was done in commit f1a7116 ("fix(@angular/cli): update |
Thank you for pointing this out. This change achieves the desired outcome. Well done! However, I did notice two minor issues. Firstly, the latest version (1.24.1) was not used. Secondly, the new version was unusable for about half an hour after its release. It appeared to be synchronising with the repository. During this process, there was a brief inconsistency with a dependency that could not be found because it was still synchronising. |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Version 1.24.0 fixes GHSA-w48q-cv73-mx4w.
PR Checklist
Please check to confirm your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
npm auditcomplains due to issue with@modelcontextprotocol/sdkdependency.What is the new behavior?
npm auditdoes not complain due to issue with@modelcontextprotocol/sdkdependency.Does this PR introduce a breaking change?
Other information
pnpm installrejects to install the new version due tominimumReleaseAgepolicy, so the lock file is not updated in this PR and this is not tested. So this PR is not complete but more a heads up on dependency update required. Feel free to come with updates to the branch.