Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Oct 20, 2025

The in-memory cache for the package manager abstraction did not properly include the registry URL in its cache key. This could lead to cache collisions if the same package was requested from two different registries. This commit corrects the cache key to be a composite of the package specifier and the registry URL.

Additionally, a new index.ts barrel file has been added to create a single, clear public entry point for the package manager feature, improving code organization.

…rel file

The in-memory cache for the package manager abstraction did not properly include the registry URL in its cache key. This could lead to cache collisions if the same package was requested from two different registries. This commit corrects the cache key to be a composite of the package specifier and the registry URL.

Additionally, a new `index.ts` barrel file has been added to create a single, clear public entry point for the package manager feature, improving code organization.
@clydin clydin added the target: major This PR is targeted for the next major release label Oct 20, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 20, 2025
@alan-agius4 alan-agius4 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 Oct 21, 2025
@clydin clydin merged commit 4aba526 into angular:main Oct 21, 2025
35 checks passed
@clydin clydin deleted the package-managers/index-updates branch October 21, 2025 10:49
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: @angular/cli target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants