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

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @mastra/pg
 
+## 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.
+
+- PostgresStore was setting `this.stores = {}` in the constructor and only populating it in the async `init()` method. This broke Memory because it checks `storage.stores.memory` synchronously in `getInputProcessors()` before `init()` is called. ([#10943](https://github.com/mastra-ai/mastra/pull/10943))
+
+  The fix moves domain instance creation to the constructor. This is safe because pg-promise creates database connections lazily when queries are executed.
+
+- 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

@@ -3737,15 +3737,15 @@ var ScoresPG = class extends storage.ScoresStorage {
     } catch (error$1) {
       throw new error.MastraError(
         {
-          id: storage.createStorageErrorId("PG", "SAVE_SCORE", "INVALID_PAYLOAD"),
+          id: storage.createStorageErrorId("PG", "SAVE_SCORE", "VALIDATION_FAILED"),
           domain: error.ErrorDomain.STORAGE,
           category: error.ErrorCategory.USER,
           details: {
-            scorer: score.scorer.id,
-            entityId: score.entityId,
-            entityType: score.entityType,
-            traceId: score.traceId || "",
-            spanId: score.spanId || ""
+            scorer: score.scorer?.id ?? "unknown",
+            entityId: score.entityId ?? "unknown",
+            entityType: score.entityType ?? "unknown",
+            traceId: score.traceId ?? "",
+            spanId: score.spanId ?? ""
           }
         },
         error$1
@@ -3753,6 +3753,7 @@ var ScoresPG = class extends storage.ScoresStorage {
     }
     try {
       const id = crypto.randomUUID();
+      const now = /* @__PURE__ */ new Date();
       const {
         scorer,
         preprocessStepResult,
@@ -3775,14 +3776,15 @@ var ScoresPG = class extends storage.ScoresStorage {
           scorer: scorer ? JSON.stringify(scorer) : null,
           preprocessStepResult: preprocessStepResult ? JSON.stringify(preprocessStepResult) : null,
           analyzeStepResult: analyzeStepResult ? JSON.stringify(analyzeStepResult) : null,
+          metadata: metadata ? JSON.stringify(metadata) : null,
+          additionalContext: additionalContext ? JSON.stringify(additionalContext) : null,
           requestContext: requestContext ? JSON.stringify(requestContext) : null,
           entity: entity ? JSON.stringify(entity) : null,
-          createdAt: (/* @__PURE__ */ new Date()).toISOString(),
-          updatedAt: (/* @__PURE__ */ new Date()).toISOString()
+          createdAt: now.toISOString(),
+          updatedAt: now.toISOString()
         }
       });
-      const scoreFromDb = await this.getScoreById({ id });
-      return { score: scoreFromDb };
+      return { score: { ...parsedScore, id, createdAt: now, updatedAt: now } };
     } catch (error$1) {
       throw new error.MastraError(
         {
@@ -4164,17 +4166,17 @@ var WorkflowsPG = class extends storage.WorkflowsStorage {
 var PostgresStore = class extends storage.MastraStorage {
   #db;
   #pgp;
-  #config;
   schema;
-  isConnected = false;
+  isInitialized = false;
   stores;
   constructor(config) {
     try {
       validateConfig("PostgresStore", config);
       super({ id: config.id, name: "PostgresStore", disableInit: config.disableInit });
       this.schema = config.schemaName || "public";
+      let pgConfig;
       if (isConnectionStringConfig(config)) {
-        this.#config = {
+        pgConfig = {
           id: config.id,
           connectionString: config.connectionString,
           max: config.max,
@@ -4182,14 +4184,14 @@ var PostgresStore = class extends storage.MastraStorage {
           ssl: config.ssl
         };
       } else if (isCloudSqlConfig(config)) {
-        this.#config = {
+        pgConfig = {
           ...config,
           id: config.id,
           max: config.max,
           idleTimeoutMillis: config.idleTimeoutMillis
         };
       } else if (isHostConfig(config)) {
-        this.#config = {
+        pgConfig = {
           id: config.id,
           host: config.host,
           port: config.port,
@@ -4205,7 +4207,20 @@ var PostgresStore = class extends storage.MastraStorage {
           "PostgresStore: invalid config. Provide either {connectionString}, {host,port,database,user,password}, or a pg ClientConfig (e.g., Cloud SQL connector with `stream`)."
         );
       }
-      this.stores = {};
+      this.#pgp = pgPromise__default.default();
+      this.#db = this.#pgp(pgConfig);
+      const operations = new StoreOperationsPG({ client: this.#db, schemaName: this.schema });
+      const scores = new ScoresPG({ client: this.#db, operations, schema: this.schema });
+      const workflows = new WorkflowsPG({ client: this.#db, operations, schema: this.schema });
+      const memory = new MemoryPG({ client: this.#db, schema: this.schema, operations });
+      const observability = new ObservabilityPG({ client: this.#db, operations, schema: this.schema });
+      this.stores = {
+        operations,
+        scores,
+        workflows,
+        memory,
+        observability
+      };
     } catch (e) {
       throw new error.MastraError(
         {
@@ -4218,33 +4233,19 @@ var PostgresStore = class extends storage.MastraStorage {
     }
   }
   async init() {
-    if (this.isConnected) {
+    if (this.isInitialized) {
       return;
     }
     try {
-      this.isConnected = true;
-      this.#pgp = pgPromise__default.default();
-      this.#db = this.#pgp(this.#config);
-      const operations = new StoreOperationsPG({ client: this.#db, schemaName: this.schema });
-      const scores = new ScoresPG({ client: this.#db, operations, schema: this.schema });
-      const workflows = new WorkflowsPG({ client: this.#db, operations, schema: this.schema });
-      const memory = new MemoryPG({ client: this.#db, schema: this.schema, operations });
-      const observability = new ObservabilityPG({ client: this.#db, operations, schema: this.schema });
-      this.stores = {
-        operations,
-        scores,
-        workflows,
-        memory,
-        observability
-      };
+      this.isInitialized = true;
       await super.init();
       try {
-        await operations.createAutomaticIndexes();
+        await this.stores.operations.createAutomaticIndexes();
       } catch (indexError) {
         console.warn("Failed to create indexes:", indexError);
       }
     } catch (error$1) {
-      this.isConnected = false;
+      this.isInitialized = false;
       throw new error.MastraError(
         {
           id: storage.createStorageErrorId("PG", "INIT", "FAILED"),
@@ -4256,15 +4257,9 @@ var PostgresStore = class extends storage.MastraStorage {
     }
   }
   get db() {
-    if (!this.#db) {
-      throw new Error(`PostgresStore: Store is not initialized, please call "init()" first.`);
-    }
     return this.#db;
   }
   get pgp() {
-    if (!this.#pgp) {
-      throw new Error(`PostgresStore: Store is not initialized, please call "init()" first.`);
-    }
     return this.#pgp;
   }
   get supports() {