Skip to content

Refactor: add local cache infrastructure with configurable directory#5

Merged
soumyaray merged 1 commit intomainfrom
feature-reverseproxy-caching
Nov 30, 2025
Merged

Refactor: add local cache infrastructure with configurable directory#5
soumyaray merged 1 commit intomainfrom
feature-reverseproxy-caching

Conversation

@soumyaray
Copy link
Contributor

Add LocalCache class to manage file-based caching operations, mirroring the Remote (Redis) cache interface. Simplify cache configuration by using direct string interpolation in environment.rb rather than class methods.

Changes:

  • Add Cache::Local class for disk cache operations (keys, wipe)
  • Rename Cache::Client to Cache::Remote for consistency
  • Update environment.rb to use LOCAL_CACHE config for cache directory
  • Refactor Rakefile cache tasks to use cache infrastructure classes
  • Add comprehensive unit tests for LocalCache (5 tests, all passing)

Architecture decisions:

  • Config files use direct strings (not infrastructure methods) for simplicity
  • Cache classes provide utility methods for rake tasks only
  • Both Local and Remote caches maintain consistent interface

All tests pass (40 runs, 146 assertions, 96.23% coverage)

🤖 Generated with Claude Code

Add LocalCache class to manage file-based caching operations, mirroring
the Remote (Redis) cache interface. Simplify cache configuration by using
direct string interpolation in environment.rb rather than class methods.

Changes:
- Add Cache::Local class for disk cache operations (keys, wipe)
- Rename Cache::Client to Cache::Remote for consistency
- Update environment.rb to use LOCAL_CACHE config for cache directory
- Refactor Rakefile cache tasks to use cache infrastructure classes
- Add comprehensive unit tests for LocalCache (5 tests, all passing)

Architecture decisions:
- Config files use direct strings (not infrastructure methods) for simplicity
- Cache classes provide utility methods for rake tasks only
- Both Local and Remote caches maintain consistent interface

All tests pass (40 runs, 146 assertions, 96.23% coverage)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@soumyaray soumyaray merged commit 171dc44 into main Nov 30, 2025
2 checks passed
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.

1 participant