fix(cache): use stale value by default when swr is enabled

[atinux]

🔗 Linked issue

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

To be properly tested.

So far, when swr: true, we should return the stale data for all incoming request until the cache is updated in background.

Also, I am not sure to understand the (opts.staleMaxAge || 0) >= 0 condition here.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[codecov]

Codecov Report

Merging #1038 (97e8e28) into main (a322405) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1038   +/-   ##
=======================================
  Coverage   67.59%   67.59%           
=======================================
  Files          62       62           
  Lines        6259     6259           
  Branches      707      707           
=======================================
  Hits         4231     4231           
  Misses       2015     2015           
  Partials       13       13           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[pi0]

Thanks for the PR.

staleMaxAge is the time when we should stop responding to stale responses.

#857 supported -1 value as a quick solution to allow setting stale time to inifnity.

I think we can set it as default.

[pi0]

I think this is a good hotfix to be back to previous behavior until handling staleMaxAge properly