feat: add response caching to minimize upstream API calls #3

Open

elia wants to merge 3 commits from feature/cache into master

pull from: feature/cache

merge into: elia:master

elia:master

Conversation 0 Commits 3 Files changed 7 +907 -12

elia commented 2026-07-18 14:27:29 +00:00

Owner

Copy link

• Cache weather responses for 1 hour (3600 seconds)
• Store cache in system-appropriate location (XDG on Linux, Library/Caches on macOS, LocalAppData on Windows)
• Use SHA256 hash of URL as cache filename for uniqueness
• Add --refresh flag to bypass cache and force fresh API call
• Add --quiet-warnings flag to suppress cache fallback notifications
• Add GO_CMW_QUIET_WARNINGS env var for multiplexer use cases
• Fallback to cache on API failure with warning notification
• Add comprehensive unit and integration tests with httptest mocks
• Update README with caching documentation and examples

Cache flow:

1. Check cache first (unless --refresh is set)
2. If cache hit and valid (< 1 hour), return cached data
3. If cache miss/expired, fetch from upstream API
4. On success, cache response and return
5. On failure, fallback to cache if available, else fatal error

- Cache weather responses for 1 hour (3600 seconds) - Store cache in system-appropriate location (XDG on Linux, Library/Caches on macOS, LocalAppData on Windows) - Use SHA256 hash of URL as cache filename for uniqueness - Add --refresh flag to bypass cache and force fresh API call - Add --quiet-warnings flag to suppress cache fallback notifications - Add GO_CMW_QUIET_WARNINGS env var for multiplexer use cases - Fallback to cache on API failure with warning notification - Add comprehensive unit and integration tests with httptest mocks - Update README with caching documentation and examples Cache flow: 1. Check cache first (unless --refresh is set) 2. If cache hit and valid (< 1 hour), return cached data 3. If cache miss/expired, fetch from upstream API 4. On success, cache response and return 5. On failure, fallback to cache if available, else fatal error

elia added the

Priority

Medium

Kind/Feature

Kind/Enhancement

Kind/Documentation

labels 2026-07-18 14:27:29 +00:00

elia added 1 commit 2026-07-18 14:27:29 +00:00

feat: add response caching to minimize upstream API calls ... deec15f4b2

- Cache weather responses for 1 hour (3600 seconds)
- Store cache in system-appropriate location (XDG on Linux, Library/Caches on macOS, LocalAppData on Windows)
- Use SHA256 hash of URL as cache filename for uniqueness
- Add --refresh flag to bypass cache and force fresh API call
- Add --quiet-warnings flag to suppress cache fallback notifications
- Add GO_CMW_QUIET_WARNINGS env var for multiplexer use cases
- Fallback to cache on API failure with warning notification
- Add comprehensive unit and integration tests with httptest mocks
- Update README with caching documentation and examples

Cache flow:
1. Check cache first (unless --refresh is set)
2. If cache hit and valid (< 1 hour), return cached data
3. If cache miss/expired, fetch from upstream API
4. On success, cache response and return
5. On failure, fallback to cache if available, else fatal error

elia force-pushed feature/cache from deec15f4b2 to 7145568729 2026-07-18 14:55:42 +00:00 Compare

elia added 1 commit 2026-07-18 15:22:11 +00:00

fix(): Don't exit 1 if upstream is unavailable 2858403125

elia added 1 commit 2026-07-18 15:32:33 +00:00

ci: split test-build into prepare/build/cleanup jobs with shared cache volumes cd40ea4849

All checks were successful

Hide all checks

Test Build / prepare (pull_request) Successful in 9s

Details

Test Build / build (cmw-darwin-amd64, amd64, darwin) (pull_request) Successful in 2m10s

Details

Test Build / build (cmw-darwin-arm64, arm64, darwin) (pull_request) Successful in 2m11s

Details

Test / test (pull_request) Successful in 2m37s

Details

Test Build / build (cmw-linux-arm64, arm64, linux) (pull_request) Successful in 2m18s

Details

Test Build / build (cmw-linux-amd64, amd64, linux) (pull_request) Successful in 2m18s

Details

Test Build / build (cmw-windows-amd64, amd64, windows) (pull_request) Successful in 2m10s

Details

Test Build / cleanup (pull_request) Successful in 4s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin feature/cache:feature/cache

git switch feature/cache

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master

git merge --no-ff feature/cache

git switch feature/cache

git rebase master

git switch master

git merge --ff-only feature/cache

git switch feature/cache

git rebase master

git switch master

git merge --no-ff feature/cache

git switch master

git merge --squash feature/cache

git switch master

git merge --ff-only feature/cache

git switch master

git merge feature/cache

git push origin master

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

Kind/Breaking

Breaking change that won't be backward compatible

  

Kind/Bug

Something is not working

  

Kind/Documentation

Documentation changes

  

Kind/Enhancement

Improve existing functionality

  

Kind/Feature

New functionality

  

Kind/Security

This is security issue

  

Kind/Testing

Issue or pull request related to testing

  

Priority

Critical

The priority is critical

  

Priority

High

The priority is high

  

Priority

Low

The priority is low

  

Priority

Medium

The priority is medium

  

Reviewed

Confirmed

Issue has been confirmed

  

Reviewed

Duplicate

This issue or pull request already exists

  

Reviewed

Invalid

Invalid issue

  

Reviewed

Won't Fix

This issue won't be fixed

  

Status

Abandoned

Somebody has started to work on this but abandoned work

  

Status

Blocked

Something is blocking this issue or pull request

  

Status

Need More Info

Feedback is required to reproduce issue or to continue work

  

renovate

Created by Renovate

No labels

Kind/Breaking

Kind/Bug

Kind/Documentation

Kind/Enhancement

Kind/Feature

Kind/Security

Kind/Testing

Priority

Critical

Priority

High

Priority

Low

Priority

Medium

Reviewed

Confirmed

Reviewed

Duplicate

Reviewed

Invalid

Reviewed

Won't Fix

Status

Abandoned

Status

Blocked

Status

Need More Info

renovate

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

elia/go-cmw!3

No description provided.