Skip to content

perf(registry): bound component file cache with lru-cache and add cacheMaxSize config#1483

Open
ricardo-devis-agullo wants to merge 2 commits into
masterfrom
memory-efficient-registry-cache
Open

perf(registry): bound component file cache with lru-cache and add cacheMaxSize config#1483
ricardo-devis-agullo wants to merge 2 commits into
masterfrom
memory-efficient-registry-cache

Conversation

@ricardo-devis-agullo

Copy link
Copy Markdown
Collaborator

Replace the unbounded nice-cache in get-component with lru-cache so
compiled templates, data providers and env files are evicted after
reaching cacheMaxSize (default 100). Also avoid shallow-cloning every
component detail map in components-details when nothing has changed.

  • Add lru-cache@^10.4.3 dependency, remove nice-cache
  • Add cacheMaxSize config option with sanitiser default
  • Update tests for the new config default

…heMaxSize config

Replace the unbounded nice-cache in get-component with lru-cache so
compiled templates, data providers and env files are evicted after
reaching cacheMaxSize (default 100). Also avoid shallow-cloning every
component detail map in components-details when nothing has changed.

- Add lru-cache@^10.4.3 dependency, remove nice-cache
- Add cacheMaxSize config option with sanitiser default
- Update tests for the new config default
@ricardo-devis-agullo

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

…gistry-cache

# Conflicts:
#	package-lock.json
#	packages/oc/package.json

Copilot AI commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Done. Resolved the merge conflicts in packages/oc/package.json and package-lock.json by keeping lru-cache: ^10.4.3 from this branch and adopting morgan: ^1.11.0 from master. The merge commit is 5e56ef09.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants