Feat/cloudflare kv support #2642
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
🦋 Changeset detectedLatest commit: b550643 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
@Mrinank-Bhowmick is attempting to deploy a commit to the Mastra Team on Vercel. A member of the Team first needs to authorize it. |
|
📝 Documentation updates detected! You can review documentation updates here |
Thanks for the review! I will make those changes, but I need some time as I am currently busy with my exams. Once they are over, I will resume work aggressively. |
|
Thanks a lot for this! |
|
@NikAiyer /examples/memory-with-libsql Cloudflare KV |
|
📝 Documentation updates detected! You can review documentation updates here |
There was a problem hiding this comment.
Looking good! Needs tests though :) Cursor or windsurf can write them. Copy the upstash ones from stores/upstash/src/storage/upstash.test.ts and tell cursor that you copied those tests from upstash and it needs to update them to test cloudflare/src/storage/index.ts
Fixes #2504
Some changes that needs to be reviewed -
I imported
{ TABLE_THREADS } from '@mastra/core/storage'directly, becauseMastraStorage.TABLE_THREADSwas showing deprecated.VSCode Warning
Some helper functions that were done because of REST API implementation.
getKV(key)redis.get(key)putKV(key, value)redis.set(key, value)deleteKV(key)redis.del(key)listKV(prefix)redis.keys(pattern)zadd,zrank,zrangeputKV,getKV(JSON)redis.set,redis.getNote: I took some help from chatgpt to structure this code for clarity, but I ensured the correctness of the mappings based on my understanding.