Skip to content

feat: make cache defaults runtime-configurable - #201

Merged
O2sa merged 1 commit into
O2sa:mainfrom
snowyukitty:fix/runtime-cache-defaults
Aug 15, 2026
Merged

feat: make cache defaults runtime-configurable#201
O2sa merged 1 commit into
O2sa:mainfrom
snowyukitty:fix/runtime-cache-defaults

Conversation

@snowyukitty

Copy link
Copy Markdown
Contributor

Fixes #161

Summary

  • Parse cache TTL settings strictly and fall back safely for malformed, non-positive, unsafe, or over-limit values.
  • Support the documented CACHE_TTL_SECONDS and CACHE_NAMESPACE aliases while preserving REDIS_* precedence.
  • Document the defaults and one-year TTL ceiling.
  • Add focused coverage for aliases, precedence, invalid values, bounds, and namespace handling.

Testing

  • pnpm exec vitest run (106 tests)
  • pnpm exec tsc --noEmit
  • pnpm exec eslint .
  • pnpm exec next build --webpack

Additionally, at the time of submission of this PR:

  • The referred issue is not blocked currently
  • All unittests passed after changes were made

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

@snowyukitty is attempting to deploy a commit to the osama's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thank you for the pull request! ✅

A maintainer will review this soon. Please be patient while we take a look. 🙌

@vercel

vercel Bot commented Aug 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dev-impact Ready Ready Preview Aug 15, 2026 6:51pm

@O2sa
O2sa merged commit dcfc366 into O2sa:main Aug 15, 2026
4 checks passed
@O2sa

O2sa commented Aug 15, 2026

Copy link
Copy Markdown
Owner

Thank you, @snowyukitty! Another great contribution merged! 🚀

We truly appreciate your continued support! If you find this tool useful, please consider giving us a ⭐ star on GitHub—it helps more developers find our work and motivates us to keep improving!

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.

chore: Make cache defaults configurable via runtime env

2 participants