Skip to content

Fix adapted cache deserialization recovery - #129

Open
Fryuni wants to merge 1 commit into
masterfrom
adated-deserialize-error
Open

Fix adapted cache deserialization recovery#129
Fryuni wants to merge 1 commit into
masterfrom
adated-deserialize-error

Conversation

@Fryuni

@Fryuni Fryuni commented Jul 16, 2026

Copy link
Copy Markdown
Member

Summary

  • validate freshly loaded values before they can enter cached-value recovery
  • evict malformed cached values and retry once through the normal cache miss path
  • cover direct loads, background revalidation, and shared in-flight requests

Verification

  • npx jest -c jest.config.js test/adapted.test.ts --runInBand (15 passed)
  • npx eslint src/adapted.ts test/adapted.test.ts
  • npm run validate
  • end-to-end malformed JSON cache smoke scenario

@Fryuni
Fryuni requested review from a team as code owners July 16, 2026 20:50
@github-actions

Copy link
Copy Markdown

👋 @Fryuni

Thanks for your contribution!

The approval and merge process is almost fully automated 🧙

Here's how it works:

  1. You open a new pull request
  2. Automated tests check the code
  3. Maintainers review the code
  4. Once approved, the PR is ready to merge.

👉 Omit the extended description
Please remove the commit body before merging the pull request.
Instead, include the pull request number in the title to provide the full context
about the change.

☝️ Lastly, the title for the commit will come from the pull request title. So please provide a descriptive title that summarizes the changes in 50 characters or less using the imperative mood.

Happy coding! 🎉

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