@mastra/upstash 1.0.0-beta.4 → 1.0.0-beta.5

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @mastra/upstash
 
+## 1.0.0-beta.5
+
+### Patch Changes
+
+- Fix saveScore not persisting ID correctly, breaking getScoreById retrieval ([#10915](https://github.com/mastra-ai/mastra/pull/10915))
+
+  **What Changed**
+  - saveScore now correctly returns scores that can be retrieved with getScoreById
+  - Validation errors now include contextual information (scorer, entity, trace details) for easier debugging
+
+  **Impact**
+  Previously, calling getScoreById after saveScore would return null because the generated ID wasn't persisted to the database. This is now fixed across all store implementations, ensuring consistent behavior and data integrity.
+
+- Updated dependencies [[`0d41fe2`](https://github.com/mastra-ai/mastra/commit/0d41fe245355dfc66d61a0d9c85d9400aac351ff), [`6b3ba91`](https://github.com/mastra-ai/mastra/commit/6b3ba91494cc10394df96782f349a4f7b1e152cc), [`7907fd1`](https://github.com/mastra-ai/mastra/commit/7907fd1c5059813b7b870b81ca71041dc807331b)]:
+  - @mastra/core@1.0.0-beta.8
+
 ## 1.0.0-beta.4
 
 ### Minor Changes

package/dist/index.cjs

@@ -5,7 +5,7 @@ var redis = require('@upstash/redis');
 var agent = require('@mastra/core/agent');
 var error = require('@mastra/core/error');
 var evals = require('@mastra/core/evals');
-var crypto = require('crypto');
+var crypto$1 = require('crypto');
 var vector = require('@mastra/core/vector');
 var vector$1 = require('@upstash/vector');
 var filter = require('@mastra/core/vector/filter');
@@ -34,7 +34,7 @@ function processRecord(tableName, record) {
       ...record.resourceId ? { resourceId: record.resourceId } : {}
     });
   } else if (tableName === storage.TABLE_SCORERS) {
-    key = getKey(tableName, { runId: record.runId });
+    key = getKey(tableName, { id: record.id });
   } else {
     key = getKey(tableName, { id: record.id });
   }
@@ -1125,22 +1125,39 @@ var ScoresUpstash = class extends storage.ScoresStorage {
         {
           id: storage.createStorageErrorId("UPSTASH", "SAVE_SCORE", "VALIDATION_FAILED"),
           domain: error.ErrorDomain.STORAGE,
-          category: error.ErrorCategory.THIRD_PARTY
+          category: error.ErrorCategory.USER,
+          details: {
+            scorer: score.scorer?.id ?? "unknown",
+            entityId: score.entityId ?? "unknown",
+            entityType: score.entityType ?? "unknown",
+            traceId: score.traceId ?? "",
+            spanId: score.spanId ?? ""
+          }
         },
         error$1
       );
     }
-    const { key, processedRecord } = processRecord(storage.TABLE_SCORERS, validatedScore);
+    const now = /* @__PURE__ */ new Date();
+    const id = crypto.randomUUID();
+    const createdAt = now;
+    const updatedAt = now;
+    const scoreWithId = {
+      ...validatedScore,
+      id,
+      createdAt,
+      updatedAt
+    };
+    const { key, processedRecord } = processRecord(storage.TABLE_SCORERS, scoreWithId);
     try {
       await this.client.set(key, processedRecord);
-      return { score };
+      return { score: { ...validatedScore, id, createdAt, updatedAt } };
     } catch (error$1) {
       throw new error.MastraError(
         {
           id: storage.createStorageErrorId("UPSTASH", "SAVE_SCORE", "FAILED"),
           domain: error.ErrorDomain.STORAGE,
           category: error.ErrorCategory.THIRD_PARTY,
-          details: { id: score.id }
+          details: { id }
         },
         error$1
       );
@@ -1907,7 +1924,7 @@ var UpstashVector = class extends vector.MastraVector {
     ids,
     sparseVectors
   }) {
-    const generatedIds = ids || vectors.map(() => crypto.randomUUID());
+    const generatedIds = ids || vectors.map(() => crypto$1.randomUUID());
     const points = vectors.map((vector, index) => ({
       id: generatedIds[index],
       vector,