Skip to content

Conversation

peteski22
Copy link
Contributor

@peteski22 peteski22 commented Aug 20, 2025

  • Add internal/cache module with functional options pattern
  • Add --cache-dir, --cache-ttl, --no-cache, --refresh-cache CLI flags for add + search cmds
  • Follow XDG Base Directory Specification for default cache location
  • Only create cache directory when caching is enabled
  • Refactor UserSpecific{Config/Cache}Dir to shared helper with XDG validation
  • Add comprehensive unit and integration tests for all cache behaviors
  • Add Registry Caching documentation with examples and troubleshooting

Closes: #33

image

* Add internal/cache module with functional options pattern
* Add --cache-dir, --cache-ttl, --no-cache, --refresh-cache CLI flags for add + search cmds
* Follow XDG Base Directory Specification for default cache location
* Only create cache directory when caching is enabled
* Refactor UserSpecific{Config/Cache}Dir to shared helper with XDG validation
* Add comprehensive unit and integration tests for all cache behaviors
* Add Registry Caching documentation with examples and troubleshooting
Co-authored-by: Davide Eynard <davide.eynard@gmail.com>
Copy link
Member

@aittalam aittalam left a comment

Choose a reason for hiding this comment

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

Thanks Peter! I left a few comments/questions, but overall this LGTM. I pre-approve it but feel free to ask more questions if you want to delve deeper into any comment.

@peteski22 peteski22 merged commit 44a6515 into main Aug 20, 2025
2 checks passed
@peteski22 peteski22 deleted the peteski22/33/cache-registry-data branch August 20, 2025 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli enhancement New feature or request registry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add caching for registry manifest data
2 participants