@mastra/mssql 1.0.0-beta.4 → 1.0.0-beta.6

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 # @mastra/mssql
 
+## 1.0.0-beta.6
+
+### Patch Changes
+
+- Add delete workflow run API ([#10991](https://github.com/mastra-ai/mastra/pull/10991))
+
+  ```typescript
+  await workflow.deleteWorkflowRunById(runId);
+  ```
+
+- Updated dependencies [[`edb07e4`](https://github.com/mastra-ai/mastra/commit/edb07e49283e0c28bd094a60e03439bf6ecf0221), [`b7e17d3`](https://github.com/mastra-ai/mastra/commit/b7e17d3f5390bb5a71efc112204413656fcdc18d), [`261473a`](https://github.com/mastra-ai/mastra/commit/261473ac637e633064a22076671e2e02b002214d), [`5d7000f`](https://github.com/mastra-ai/mastra/commit/5d7000f757cd65ea9dc5b05e662fd83dfd44e932), [`4f0331a`](https://github.com/mastra-ai/mastra/commit/4f0331a79bf6eb5ee598a5086e55de4b5a0ada03), [`8a000da`](https://github.com/mastra-ai/mastra/commit/8a000da0c09c679a2312f6b3aa05b2ca78ca7393)]:
+  - @mastra/core@1.0.0-beta.10
+
+## 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

@@ -2,7 +2,7 @@
 
 var error = require('@mastra/core/error');
 var storage = require('@mastra/core/storage');
-var sql2 = require('mssql');
+var sql3 = require('mssql');
 var agent = require('@mastra/core/agent');
 var utils = require('@mastra/core/utils');
 var crypto = require('crypto');
@@ -10,7 +10,7 @@ var evals = require('@mastra/core/evals');
 
 function _interopDefault (e) { return e && e.__esModule ? e : { default: e }; }
 
-var sql2__default = /*#__PURE__*/_interopDefault(sql2);
+var sql3__default = /*#__PURE__*/_interopDefault(sql3);
 
 // src/storage/index.ts
 function getSchemaName(schema) {
@@ -233,7 +233,7 @@ var MemoryMSSQL = class extends storage.MemoryStorage {
       dataRequest.input("resourceId", resourceId);
       dataRequest.input("offset", offset);
       if (limitValue > 2147483647) {
-        dataRequest.input("perPage", sql2__default.default.BigInt, limitValue);
+        dataRequest.input("perPage", sql3__default.default.BigInt, limitValue);
       } else {
         dataRequest.input("perPage", limitValue);
       }
@@ -297,12 +297,12 @@ var MemoryMSSQL = class extends storage.MemoryStorage {
       req.input("title", thread.title);
       const metadata = thread.metadata ? JSON.stringify(thread.metadata) : null;
       if (metadata === null) {
-        req.input("metadata", sql2__default.default.NVarChar, null);
+        req.input("metadata", sql3__default.default.NVarChar, null);
       } else {
         req.input("metadata", metadata);
       }
-      req.input("createdAt", sql2__default.default.DateTime2, thread.createdAt);
-      req.input("updatedAt", sql2__default.default.DateTime2, thread.updatedAt);
+      req.input("createdAt", sql3__default.default.DateTime2, thread.createdAt);
+      req.input("updatedAt", sql3__default.default.DateTime2, thread.updatedAt);
       await req.query(mergeSql);
       return thread;
     } catch (error$1) {
@@ -598,7 +598,7 @@ var MemoryMSSQL = class extends storage.MemoryStorage {
           return result2.recordset || [];
         }
         request.input("offset", offset);
-        request.input("limit", perPage > 2147483647 ? sql2__default.default.BigInt : sql2__default.default.Int, perPage);
+        request.input("limit", perPage > 2147483647 ? sql3__default.default.BigInt : sql3__default.default.Int, perPage);
         const result = await request.query(
           `${baseQuery}${actualWhereClause} ${orderByStatement} OFFSET @offset ROWS FETCH NEXT @limit ROWS ONLY`
         );
@@ -723,7 +723,7 @@ var MemoryMSSQL = class extends storage.MemoryStorage {
             "content",
             typeof message.content === "string" ? message.content : JSON.stringify(message.content)
           );
-          request.input("createdAt", sql2__default.default.DateTime2, message.createdAt);
+          request.input("createdAt", sql3__default.default.DateTime2, message.createdAt);
           request.input("role", message.role);
           request.input("type", message.type || "v2");
           request.input("resourceId", message.resourceId);
@@ -742,7 +742,7 @@ var MemoryMSSQL = class extends storage.MemoryStorage {
           await request.query(mergeSql);
         }
         const threadReq = transaction.request();
-        threadReq.input("updatedAt", sql2__default.default.DateTime2, /* @__PURE__ */ new Date());
+        threadReq.input("updatedAt", sql3__default.default.DateTime2, /* @__PURE__ */ new Date());
         threadReq.input("id", threadId);
         await threadReq.query(`UPDATE ${tableThreads} SET [updatedAt] = @updatedAt WHERE id = @id`);
         await transaction.commit();
@@ -1443,7 +1443,7 @@ var StoreOperationsMSSQL = class extends storage.StoreOperations {
         const value = record[col];
         const preparedValue = this.prepareValue(value, col, tableName);
         if (preparedValue instanceof Date) {
-          request.input(`param${i}`, sql2__default.default.DateTime2, preparedValue);
+          request.input(`param${i}`, sql3__default.default.DateTime2, preparedValue);
         } else if (preparedValue === null || preparedValue === void 0) {
           request.input(`param${i}`, this.getMssqlType(tableName, col), null);
         } else {
@@ -1781,23 +1781,23 @@ ${columns}
     const col = storage.TABLE_SCHEMAS[tableName]?.[columnName];
     switch (col?.type) {
       case "text":
-        return sql2__default.default.NVarChar;
+        return sql3__default.default.NVarChar;
       case "timestamp":
-        return sql2__default.default.DateTime2;
+        return sql3__default.default.DateTime2;
       case "uuid":
-        return sql2__default.default.UniqueIdentifier;
+        return sql3__default.default.UniqueIdentifier;
       case "jsonb":
-        return sql2__default.default.NVarChar;
+        return sql3__default.default.NVarChar;
       case "integer":
-        return sql2__default.default.Int;
+        return sql3__default.default.Int;
       case "bigint":
-        return sql2__default.default.BigInt;
+        return sql3__default.default.BigInt;
       case "float":
-        return sql2__default.default.Float;
+        return sql3__default.default.Float;
       case "boolean":
-        return sql2__default.default.Bit;
+        return sql3__default.default.Bit;
       default:
-        return sql2__default.default.NVarChar;
+        return sql3__default.default.NVarChar;
     }
   }
   /**
@@ -2346,13 +2346,21 @@ var ScoresMSSQL = class extends storage.ScoresStorage {
         {
           id: storage.createStorageErrorId("MSSQL", "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
       );
     }
     try {
       const scoreId = crypto.randomUUID();
+      const now = /* @__PURE__ */ new Date();
       const {
         scorer,
         preprocessStepResult,
@@ -2379,12 +2387,11 @@ var ScoresMSSQL = class extends storage.ScoresStorage {
           requestContext: requestContext || null,
           entity: entity || null,
           scorer: scorer || null,
-          createdAt: (/* @__PURE__ */ new Date()).toISOString(),
-          updatedAt: (/* @__PURE__ */ new Date()).toISOString()
+          createdAt: now.toISOString(),
+          updatedAt: now.toISOString()
         }
       });
-      const scoreFromDb = await this.getScoreById({ id: scoreId });
-      return { score: scoreFromDb };
+      return { score: { ...validatedScore, id: scoreId, createdAt: now, updatedAt: now } };
     } catch (error$1) {
       throw new error.MastraError(
         {
@@ -2690,7 +2697,7 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
     const transaction = this.pool.transaction();
     try {
       await transaction.begin();
-      const selectRequest = new sql2__default.default.Request(transaction);
+      const selectRequest = new sql3__default.default.Request(transaction);
       selectRequest.input("workflow_name", workflowName);
       selectRequest.input("run_id", runId);
       const existingSnapshotResult = await selectRequest.query(
@@ -2718,12 +2725,12 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
       }
       snapshot.context[stepId] = result;
       snapshot.requestContext = { ...snapshot.requestContext, ...requestContext };
-      const upsertReq = new sql2__default.default.Request(transaction);
+      const upsertReq = new sql3__default.default.Request(transaction);
       upsertReq.input("workflow_name", workflowName);
       upsertReq.input("run_id", runId);
       upsertReq.input("snapshot", JSON.stringify(snapshot));
-      upsertReq.input("createdAt", sql2__default.default.DateTime2, /* @__PURE__ */ new Date());
-      upsertReq.input("updatedAt", sql2__default.default.DateTime2, /* @__PURE__ */ new Date());
+      upsertReq.input("createdAt", sql3__default.default.DateTime2, /* @__PURE__ */ new Date());
+      upsertReq.input("updatedAt", sql3__default.default.DateTime2, /* @__PURE__ */ new Date());
       await upsertReq.query(
         `MERGE ${table} AS target
          USING (SELECT @workflow_name AS workflow_name, @run_id AS run_id) AS src
@@ -2763,7 +2770,7 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
     const transaction = this.pool.transaction();
     try {
       await transaction.begin();
-      const selectRequest = new sql2__default.default.Request(transaction);
+      const selectRequest = new sql3__default.default.Request(transaction);
       selectRequest.input("workflow_name", workflowName);
       selectRequest.input("run_id", runId);
       const existingSnapshotResult = await selectRequest.query(
@@ -2791,11 +2798,11 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
         );
       }
       const updatedSnapshot = { ...snapshot, ...opts };
-      const updateRequest = new sql2__default.default.Request(transaction);
+      const updateRequest = new sql3__default.default.Request(transaction);
       updateRequest.input("snapshot", JSON.stringify(updatedSnapshot));
       updateRequest.input("workflow_name", workflowName);
       updateRequest.input("run_id", runId);
-      updateRequest.input("updatedAt", sql2__default.default.DateTime2, /* @__PURE__ */ new Date());
+      updateRequest.input("updatedAt", sql3__default.default.DateTime2, /* @__PURE__ */ new Date());
       await updateRequest.query(
         `UPDATE ${table} SET snapshot = @snapshot, [updatedAt] = @updatedAt WHERE workflow_name = @workflow_name AND run_id = @run_id`
       );
@@ -2835,8 +2842,8 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
       request.input("run_id", runId);
       request.input("resourceId", resourceId);
       request.input("snapshot", JSON.stringify(snapshot));
-      request.input("createdAt", sql2__default.default.DateTime2, new Date(now));
-      request.input("updatedAt", sql2__default.default.DateTime2, new Date(now));
+      request.input("createdAt", sql3__default.default.DateTime2, new Date(now));
+      request.input("updatedAt", sql3__default.default.DateTime2, new Date(now));
       const mergeSql = `MERGE INTO ${table} AS target
         USING (SELECT @workflow_name AS workflow_name, @run_id AS run_id) AS src
         ON target.workflow_name = src.workflow_name AND target.run_id = src.run_id
@@ -2933,6 +2940,35 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
       );
     }
   }
+  async deleteWorkflowRunById({ runId, workflowName }) {
+    const table = getTableName({ indexName: storage.TABLE_WORKFLOW_SNAPSHOT, schemaName: getSchemaName(this.schema) });
+    const transaction = this.pool.transaction();
+    try {
+      await transaction.begin();
+      const deleteRequest = new sql3__default.default.Request(transaction);
+      deleteRequest.input("workflow_name", workflowName);
+      deleteRequest.input("run_id", runId);
+      await deleteRequest.query(`DELETE FROM ${table} WHERE workflow_name = @workflow_name AND run_id = @run_id`);
+      await transaction.commit();
+    } catch (error$1) {
+      try {
+        await transaction.rollback();
+      } catch {
+      }
+      throw new error.MastraError(
+        {
+          id: storage.createStorageErrorId("MSSQL", "DELETE_WORKFLOW_RUN_BY_ID", "FAILED"),
+          domain: error.ErrorDomain.STORAGE,
+          category: error.ErrorCategory.THIRD_PARTY,
+          details: {
+            runId,
+            workflowName
+          }
+        },
+        error$1
+      );
+    }
+  }
   async listWorkflowRuns({
     workflowName,
     fromDate,
@@ -2976,7 +3012,7 @@ var WorkflowsMSSQL = class extends storage.WorkflowsStorage {
       const request = this.pool.request();
       Object.entries(paramMap).forEach(([key, value]) => {
         if (value instanceof Date) {
-          request.input(key, sql2__default.default.DateTime, value);
+          request.input(key, sql3__default.default.DateTime, value);
         } else {
           request.input(key, value);
         }
@@ -3039,7 +3075,7 @@ var MSSQLStore = class extends storage.MastraStorage {
         }
       }
       this.schema = config.schemaName || "dbo";
-      this.pool = "connectionString" in config ? new sql2__default.default.ConnectionPool(config.connectionString) : new sql2__default.default.ConnectionPool({
+      this.pool = "connectionString" in config ? new sql3__default.default.ConnectionPool(config.connectionString) : new sql3__default.default.ConnectionPool({
         server: config.server,
         database: config.database,
         user: config.user,
@@ -3230,6 +3266,9 @@ var MSSQLStore = class extends storage.MastraStorage {
   }) {
     return this.stores.workflows.getWorkflowRunById({ runId, workflowName });
   }
+  async deleteWorkflowRunById({ runId, workflowName }) {
+    return this.stores.workflows.deleteWorkflowRunById({ runId, workflowName });
+  }
   async close() {
     await this.pool.close();
   }
@@ -3308,8 +3347,8 @@ var MSSQLStore = class extends storage.MastraStorage {
       source: _source
     });
   }
-  async saveScore(_score) {
-    return this.stores.scores.saveScore(_score);
+  async saveScore(score) {
+    return this.stores.scores.saveScore(score);
   }
   async listScoresByRunId({
     runId: _runId,