Skip to content

Add persistent EphemeralStore adaptors#18

Merged
zxch3n merged 5 commits intomainfrom
codex/create-non-ephemeral-adaptors-for-ephemeralstore
Nov 5, 2025
Merged

Add persistent EphemeralStore adaptors#18
zxch3n merged 5 commits intomainfrom
codex/create-non-ephemeral-adaptors-for-ephemeralstore

Conversation

@zxch3n
Copy link
Copy Markdown
Member

@zxch3n zxch3n commented Nov 5, 2025

Summary

  • add a new CRDT type for persisted Loro EphemeralStore payloads
  • provide client and server adaptors that use the persisted store flag
  • register the persisted store in the websocket server and extend tests
  • document why the persisted ephemeral store type exists and how to use the adaptor

Testing

  • pnpm --filter loro-adaptors test
  • pnpm --filter loro-websocket test -- --runInBand

https://chatgpt.com/codex/tasks/task_e_690b0346b400832e9c7e3979e3a59a07

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/loro-websocket/src/server/crdt-doc.ts Outdated
@zxch3n zxch3n merged commit db6bfab into main Nov 5, 2025
2 checks passed
@zxch3n zxch3n deleted the codex/create-non-ephemeral-adaptors-for-ephemeralstore branch November 5, 2025 12:13
@github-actions github-actions Bot mentioned this pull request Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant