Skip to content

fix(directus-client): harden refresh coordination - #280

Merged
remihuigen merged 11 commits into
mainfrom
feat/coordinator
Sep 6, 2026
Merged

fix(directus-client): harden refresh coordination#280
remihuigen merged 11 commits into
mainfrom
feat/coordinator

Conversation

@remihuigen

@remihuigen remihuigen commented Sep 5, 2026

Copy link
Copy Markdown
Member

Summary

  • Harden Directus refresh coordination for memory and Redis backends.
  • Prevent Redis acquire/result races and stale-token retries.
  • Align owner/shared result semantics and follower cookie adoption.
  • Simplify coordinator dependencies, Redis control flow, and refresh execution.
  • Add deterministic concurrency, lease, publication, and TTL coverage.
  • Document the coordinator architecture in an ADR.

Validation

  • corepack pnpm typecheck
  • corepack pnpm test — 120 files, 679 tests
  • corepack pnpm build
  • corepack pnpm validate:packages
  • corepack pnpm validate:docs

Closes #257
Closes #258
Closes #260
Closes #287
Closes #290
Closes #291

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

🦋 Changeset detected

Latest commit: 1cd3964

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@onderwijsin/nuxt-directus-client Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-code-quality

github-code-quality Bot commented Sep 6, 2026

Copy link
Copy Markdown

Code Coverage Overview

Languages: JavaScript

JavaScript / code-coverage/vitest

The overall line coverage in commit 1cd3964 in the feat/coordinator branch is 69%. Line coverage data for the main branch is not yet available.

Show a line coverage summary of the most covered files.
File main feat/coordinator 1cd3964 +/-
modules/directu...irectus-auth.ts 100%
modules/directu...sitemap-urls.ts 100%
modules/simple-...untime/index.ts 98%
modules/theme-c...e-customizer.ts 96%
modules/redirec...tils/storage.ts 95%
modules/directu...inator/redis.ts 95%
modules/directu...rver/refresh.ts 92%
modules/directu...t/src/module.ts 85%
modules/healthc...utils/health.ts 84%
modules/directu...nfig/typegen.ts 83%

Updated September 06, 2026 20:23 UTC

@remihuigen
remihuigen added this pull request to the merge queue Sep 6, 2026
Merged via the queue into main with commit 20ada0c Sep 6, 2026
14 checks passed
@remihuigen
remihuigen deleted the feat/coordinator branch September 6, 2026 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment