Skip to content

feat: add concurrent tag revalidation checks - #58

Open
toiim wants to merge 1 commit into
mrjasonroy:mainfrom
toiim:parallelize-tag-staleness-checks
Open

feat: add concurrent tag revalidation checks#58
toiim wants to merge 1 commit into
mrjasonroy:mainfrom
toiim:parallelize-tag-staleness-checks

Conversation

@toiim

@toiim toiim commented Aug 18, 2026

Copy link
Copy Markdown

Description

Cache tag lookups were sequential. Given there can be up to 128 per cache entry this can become an unnecessary bottleneck. Uses Promise.all() instead of awaiting in the loop.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Chore (dependencies, CI, etc.)

Testing

  • All existing tests pass (pnpm test)
  • Added new tests for new functionality
  • Tested with memory cache handler
  • Tested with Redis cache handler (if applicable)
  • E2E tests pass (pnpm test:e2e)

Checklist

  • My code follows the project's code style (pnpm lint passes)
  • I have run pnpm format to format my code
  • Type checking passes (pnpm typecheck)
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Related Issues

Additional Context

@toiim
toiim marked this pull request as ready for review August 19, 2026 01:16
@toiim
toiim requested a review from mrjasonroy as a code owner August 19, 2026 01:16
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