@mastra/mysql 0.6.0 → 0.7.0-alpha.1

package/CHANGELOG.md

@@ -1,5 +1,59 @@
 # @mastra/mysql
 
+## 0.7.0-alpha.1
+
+### Patch Changes
+
+- Fixed resource-scoped message includes across storage adapters so included context cannot cross resource boundaries. ([#20984](https://github.com/mastra-ai/mastra/pull/20984))
+
+- Updated dependencies [[`6445eba`](https://github.com/mastra-ai/mastra/commit/6445eba6020abac681aba1cc9289f446cb400cbe), [`df31eb0`](https://github.com/mastra-ai/mastra/commit/df31eb0c7087d782a0d9346e467f9a4af4b0eef6), [`fcd0667`](https://github.com/mastra-ai/mastra/commit/fcd0667a4e378be35c9a1b1eb19cce78fbfd7282), [`bab06b1`](https://github.com/mastra-ai/mastra/commit/bab06b18923873a584bdfc71a6b4ec7fb4727fb7)]:
+  - @mastra/core@1.58.0-alpha.5
+
+## 0.7.0-alpha.0
+
+### Minor Changes
+
+- Added experiment provenance and grouping support to the LibSQL, MongoDB, MySQL, PostgreSQL, and Spanner storage adapters. These fields remain available for later grouping and filtering. ([#20645](https://github.com/mastra-ai/mastra/pull/20645))
+
+  ```ts
+  import { PostgresStore } from '@mastra/pg';
+
+  const storage = new PostgresStore({
+    id: 'postgres-storage',
+    connectionString: process.env.DATABASE_URL!,
+  });
+
+  await dataset.startExperiment({
+    task,
+    scorers,
+    provenance: { source: 'github', sourceVersion: 'abc123' },
+    grouping: { experimentSetId: 'benchmark-1', variantId: 'candidate', trialIndex: 0 },
+  });
+  ```
+
+- Memory list reads now surface database errors instead of silently returning empty results. ([#17910](https://github.com/mastra-ai/mastra/pull/17910))
+
+  Previously, the paginated memory reads (`listThreads`, `listMessages`, `listMessagesByResourceId`, and `listMessagesById`) caught backend failures, logged them, and returned an empty payload like `{ threads: [], total: 0, hasMore: false }`. A transient outage (locked table, dropped connection) was therefore indistinguishable from a genuinely empty result, so an agent reading conversation history during a brief failure would treat it as "no history" and could overwrite real state. These methods now re-throw the failure as a `MastraError`. Validation (USER) errors and genuinely empty results are unchanged.
+
+  **Behavior change**
+
+  Callers that previously received an empty result on a backend failure will now receive a thrown `MastraError`. If you call these read methods directly (rather than through an agent, which already surfaces errors), wrap them so a transient outage doesn't crash the caller:
+
+  ```ts
+  try {
+    const { threads } = await storage.listThreads({ resourceId });
+    // ...use threads
+  } catch (error) {
+    // a real backend failure. Decide whether to retry, surface, or degrade.
+    // An empty thread list no longer hides here; it only means "no threads".
+  }
+  ```
+
+### Patch Changes
+
+- Updated dependencies [[`e7109ee`](https://github.com/mastra-ai/mastra/commit/e7109ee6f731bacc79c885906f3c7dca8d8f013a), [`772c0c8`](https://github.com/mastra-ai/mastra/commit/772c0c897cec383258de2e6178147f8014767c7b), [`578bf2e`](https://github.com/mastra-ai/mastra/commit/578bf2e6a88e9d5b8bf502204e15a95dfbb679ae), [`06b2d87`](https://github.com/mastra-ai/mastra/commit/06b2d87e63bcdd0ed59215c6789692b9b12de376), [`ac01d63`](https://github.com/mastra-ai/mastra/commit/ac01d6355974aec73fdb8781449ed12bac582094), [`a810a05`](https://github.com/mastra-ai/mastra/commit/a810a058f62ad407cfc1701e0be36ae91145d7cf), [`f8da216`](https://github.com/mastra-ai/mastra/commit/f8da21633e7eb0e31c9ce0fc30567870d19416d3), [`6104347`](https://github.com/mastra-ai/mastra/commit/61043473ba6bfd0a25156824e853e13165562e6c), [`45bfb88`](https://github.com/mastra-ai/mastra/commit/45bfb88fd52f1dd3be20e2a38905777c96499c90), [`e3b9307`](https://github.com/mastra-ai/mastra/commit/e3b9307098daefbfae2a52ae2ef51bc9fc701190), [`d6834c5`](https://github.com/mastra-ai/mastra/commit/d6834c5a7866b16734d23900163c2414ed70d791), [`c52d346`](https://github.com/mastra-ai/mastra/commit/c52d3462ec831a5d95926ecd3d3373f5928ad2e5), [`0023e79`](https://github.com/mastra-ai/mastra/commit/0023e7919431078280abd11c89d1edeae35fcc69), [`c2ad51e`](https://github.com/mastra-ai/mastra/commit/c2ad51e2467f901eecba8c9f4a45e22a50bd7c18), [`3dc97ea`](https://github.com/mastra-ai/mastra/commit/3dc97ea415fad353b48a13095fad1835933cc12a), [`3d01cd3`](https://github.com/mastra-ai/mastra/commit/3d01cd387321b6f9c5cac31d487c84bf51b19c78), [`7bf3086`](https://github.com/mastra-ai/mastra/commit/7bf308663f0115ca74ad20554ade740f06640859), [`a8dd139`](https://github.com/mastra-ai/mastra/commit/a8dd1391a9fe9a6632c25809ef236980afa9a020), [`e5786be`](https://github.com/mastra-ai/mastra/commit/e5786be02bb903073082bd9d6da880ebaacc343f), [`2093fbd`](https://github.com/mastra-ai/mastra/commit/2093fbd53bb744bae19ec89f6d73db9a66fbe8a7), [`e7a5da4`](https://github.com/mastra-ai/mastra/commit/e7a5da4ef8e4dd452d2f232961b4e682a85ffe43), [`7b4393d`](https://github.com/mastra-ai/mastra/commit/7b4393d557411fdcf07b0e30e5acaf7cc85154ae)]:
+  - @mastra/core@1.58.0-alpha.1
+
 ## 0.6.0
 
 ### Minor Changes

package/dist/index.cjs

@@ -2838,15 +2838,28 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 	* Returns default index definitions for the experiments domain tables.
 	*/
 	static getDefaultIndexDefs(_prefix = "") {
-		return [{
-			name: "idx_experiments_org_project",
-			table: _mastra_core_storage.TABLE_EXPERIMENTS,
-			columns: ["organizationId", "projectId"]
-		}, {
-			name: "idx_experiment_results_org_project",
-			table: _mastra_core_storage.TABLE_EXPERIMENT_RESULTS,
-			columns: ["organizationId", "projectId"]
-		}];
+		return [
+			{
+				name: "idx_experiments_grouping",
+				table: _mastra_core_storage.TABLE_EXPERIMENTS,
+				columns: [
+					"experimentSetId",
+					"comparisonId",
+					"variantId",
+					"trialIndex"
+				]
+			},
+			{
+				name: "idx_experiments_org_project",
+				table: _mastra_core_storage.TABLE_EXPERIMENTS,
+				columns: ["organizationId", "projectId"]
+			},
+			{
+				name: "idx_experiment_results_org_project",
+				table: _mastra_core_storage.TABLE_EXPERIMENT_RESULTS,
+				columns: ["organizationId", "projectId"]
+			}
+		];
 	}
 	/**
 	* Exports DDL statements for all managed tables.
@@ -2902,7 +2915,13 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 			ifNotExists: [
 				"agentVersion",
 				"organizationId",
-				"projectId"
+				"projectId",
+				"provenance",
+				"runnerAttestation",
+				"experimentSetId",
+				"comparisonId",
+				"variantId",
+				"trialIndex"
 			]
 		});
 		await this.operations.alterTable({
@@ -2934,6 +2953,12 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 			name: row.name ?? void 0,
 			description: row.description ?? void 0,
 			metadata: parseJSON$2(row.metadata),
+			provenance: parseJSON$2(row.provenance) ?? null,
+			runnerAttestation: parseJSON$2(row.runnerAttestation) ?? null,
+			experimentSetId: row.experimentSetId ?? null,
+			comparisonId: row.comparisonId ?? null,
+			variantId: row.variantId ?? null,
+			trialIndex: row.trialIndex ?? null,
 			status: row.status,
 			totalItems: row.totalItems,
 			succeededCount: row.succeededCount,
@@ -2985,6 +3010,12 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 					name: input.name ?? null,
 					description: input.description ?? null,
 					metadata: input.metadata ? JSON.stringify(input.metadata) : null,
+					provenance: input.provenance ? JSON.stringify(input.provenance) : null,
+					runnerAttestation: input.runnerAttestation ? JSON.stringify(input.runnerAttestation) : null,
+					experimentSetId: input.experimentSetId ?? null,
+					comparisonId: input.comparisonId ?? null,
+					variantId: input.variantId ?? null,
+					trialIndex: input.trialIndex ?? null,
 					status: "pending",
 					totalItems: input.totalItems,
 					succeededCount: 0,
@@ -3008,6 +3039,12 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 				name: input.name,
 				description: input.description,
 				metadata: input.metadata,
+				provenance: input.provenance ?? null,
+				runnerAttestation: input.runnerAttestation ?? null,
+				experimentSetId: input.experimentSetId ?? null,
+				comparisonId: input.comparisonId ?? null,
+				variantId: input.variantId ?? null,
+				trialIndex: input.trialIndex ?? null,
 				status: "pending",
 				totalItems: input.totalItems,
 				succeededCount: 0,
@@ -3104,6 +3141,22 @@ var ExperimentsMySQL = class ExperimentsMySQL extends _mastra_core_storage.Exper
 				conditions.push(`${quoteIdentifier("status", "column name")} = ?`);
 				params.push(args.status);
 			}
+			if (args.experimentSetId !== void 0) {
+				conditions.push(`${quoteIdentifier("experimentSetId", "column name")} = ?`);
+				params.push(args.experimentSetId);
+			}
+			if (args.comparisonId !== void 0) {
+				conditions.push(`${quoteIdentifier("comparisonId", "column name")} = ?`);
+				params.push(args.comparisonId);
+			}
+			if (args.variantId !== void 0) {
+				conditions.push(`${quoteIdentifier("variantId", "column name")} = ?`);
+				params.push(args.variantId);
+			}
+			if (args.trialIndex !== void 0) {
+				conditions.push(`${quoteIdentifier("trialIndex", "column name")} = ?`);
+				params.push(args.trialIndex);
+			}
 			if (args.filters) {
 				const { organizationId, projectId } = args.filters;
 				if (organizationId !== void 0) {
@@ -4869,9 +4922,17 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 		if (row.type && row.type !== "v2") message.type = row.type;
 		return message;
 	}
-	async fetchMessagesForThread(threadId, limit) {
-		let sql = `SELECT id, thread_id, content, role, type, createdAt, resourceId FROM ${formatTableName(_mastra_core_storage.TABLE_MESSAGES)} WHERE ${quoteIdentifier("thread_id", "column name")} = ? ORDER BY ${quoteIdentifier("createdAt", "column name")} ASC`;
-		const params = [threadId];
+	/**
+	* Loads a thread's messages in chronological order.
+	*
+	* @param threadId - Thread to read.
+	* @param limit - Optional cap on the number of rows.
+	* @param resourceId - When set, returns only the rows owned by that resource.
+	*/
+	async fetchMessagesForThread(threadId, limit, resourceId) {
+		const resourceCondition = resourceId ? ` AND ${quoteIdentifier("resourceId", "column name")} = ?` : "";
+		let sql = `SELECT id, thread_id, content, role, type, createdAt, resourceId FROM ${formatTableName(_mastra_core_storage.TABLE_MESSAGES)} WHERE ${quoteIdentifier("thread_id", "column name")} = ?${resourceCondition} ORDER BY ${quoteIdentifier("createdAt", "column name")} ASC`;
+		const params = resourceId ? [threadId, resourceId] : [threadId];
 		if (limit && limit > 0) {
 			sql += ` LIMIT ?`;
 			params.push(limit);
@@ -4882,15 +4943,20 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 	/**
 	* Fetches included messages by ID, discovering their thread automatically.
 	* This handles cross-thread includes where the include item doesn't specify a threadId.
+	*
+	* @param include - Message ids to pin, each with an optional before/after window.
+	* @param resourceId - When set, restricts both the pinned messages and their context
+	* to that resource so an id from another resource returns nothing.
 	*/
-	async _getIncludedMessages({ include }) {
+	async _getIncludedMessages({ include, resourceId }) {
 		if (!include || include.length === 0) return null;
 		const tableName = formatTableName(_mastra_core_storage.TABLE_MESSAGES);
 		const selectColumns = `id, thread_id, content, role, type, createdAt, resourceId`;
+		const resourceCondition = resourceId ? ` AND m.${quoteIdentifier("resourceId", "column name")} = ?` : "";
 		const targetIds = include.map((inc) => inc.id).filter(Boolean);
 		if (targetIds.length === 0) return null;
 		const idPlaceholders = targetIds.map(() => "?").join(", ");
-		const [targetRows] = await this.pool.execute(`SELECT id, thread_id, createdAt FROM ${tableName} WHERE id IN (${idPlaceholders})`, targetIds);
+		const [targetRows] = await this.pool.execute(`SELECT id, thread_id, createdAt FROM ${tableName} WHERE id IN (${idPlaceholders})${resourceId ? ` AND ${quoteIdentifier("resourceId", "column name")} = ?` : ""}`, resourceId ? [...targetIds, resourceId] : targetIds);
 		if (!targetRows || targetRows.length === 0) return null;
 		const targetMap = new Map(targetRows.map((r) => [r.id, {
 			threadId: r.thread_id,
@@ -4908,21 +4974,23 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
         SELECT ${selectColumns}
         FROM ${tableName} m
         WHERE m.thread_id = ?
-          AND m.createdAt <= ?
+          AND m.createdAt <= ?${resourceCondition}
         ORDER BY m.createdAt DESC, m.id DESC
         LIMIT ${prevLimit}
       )`);
 			params.push(target.threadId, target.createdAt);
+			if (resourceId) params.push(resourceId);
 			if (nextLimit > 0) {
 				unionQueries.push(`(
           SELECT ${selectColumns}
           FROM ${tableName} m
           WHERE m.thread_id = ?
-            AND m.createdAt > ?
+            AND m.createdAt > ?${resourceCondition}
           ORDER BY m.createdAt ASC, m.id ASC
           LIMIT ${nextLimit}
         )`);
 				params.push(target.threadId, target.createdAt);
+				if (resourceId) params.push(resourceId);
 			}
 		}
 		if (unionQueries.length === 0) return null;
@@ -4930,7 +4998,17 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 		const [rows] = await this.pool.execute(finalQuery, params);
 		return rows;
 	}
-	async collectIncludeMessages({ threadId, include, messagesByThread }) {
+	/**
+	* Resolves include items against loaded threads, adding each pinned message and its
+	* before/after window.
+	*
+	* @param threadId - Thread used when an include item names no thread of its own.
+	* @param include - Message ids to pin, each with an optional before/after window.
+	* @param messagesByThread - Cache of thread snapshots, reused and filled as threads load.
+	* @param resourceId - When set, restricts both the pinned messages and their context
+	* to that resource so an id from another resource returns nothing.
+	*/
+	async collectIncludeMessages({ threadId, include, messagesByThread, resourceId }) {
 		if (!include?.length) return [];
 		const includeMessages = [];
 		const seenIds = /* @__PURE__ */ new Set();
@@ -4938,18 +5016,18 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 		const resolvedThreadIds = /* @__PURE__ */ new Map();
 		if (unresolvedIds.length > 0) {
 			const placeholders = unresolvedIds.map(() => "?").join(", ");
-			const [rows] = await this.pool.execute(`SELECT id, thread_id FROM ${formatTableName(_mastra_core_storage.TABLE_MESSAGES)} WHERE id IN (${placeholders})`, unresolvedIds);
+			const [rows] = await this.pool.execute(`SELECT id, thread_id FROM ${formatTableName(_mastra_core_storage.TABLE_MESSAGES)} WHERE id IN (${placeholders})${resourceId ? ` AND ${quoteIdentifier("resourceId", "column name")} = ?` : ""}`, resourceId ? [...unresolvedIds, resourceId] : unresolvedIds);
 			for (const row of rows) resolvedThreadIds.set(row.id, row.thread_id);
 		}
 		for (const inc of include) {
 			const targetThreadId = inc.threadId ?? resolvedThreadIds.get(inc.id) ?? threadId;
 			let threadMessages = messagesByThread.get(targetThreadId);
 			if (!threadMessages) {
-				threadMessages = (await this.fetchMessagesForThread(targetThreadId)).map((row) => this.mapMessage(row));
+				threadMessages = (await this.fetchMessagesForThread(targetThreadId, void 0, resourceId)).map((row) => this.mapMessage(row));
 				messagesByThread.set(targetThreadId, threadMessages);
 			}
 			if (!threadMessages.some((message) => message.id === inc.id) || (inc.withPreviousMessages ?? 0) > 0 || (inc.withNextMessages ?? 0) > 0 || threadMessages.length < (inc.withNextMessages ?? 0) + (inc.withPreviousMessages ?? 0) + 1) {
-				threadMessages = (await this.fetchMessagesForThread(targetThreadId)).map((row) => this.mapMessage(row));
+				threadMessages = (await this.fetchMessagesForThread(targetThreadId, void 0, resourceId)).map((row) => this.mapMessage(row));
 				messagesByThread.set(targetThreadId, threadMessages);
 			}
 			const targetIndex = threadMessages.findIndex((msg) => msg.id === inc.id);
@@ -5066,12 +5144,16 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 				hasMore: perPageInput === false ? false : offset + perPageNormalized < total
 			};
 		} catch (error) {
-			throw new _mastra_core_error.MastraError({
-				id: "MYSQL_MEMORY_LIST_THREADS_FAILED",
+			if (error instanceof _mastra_core_error.MastraError && error.category === _mastra_core_error.ErrorCategory.USER) throw error;
+			const mastraError = new _mastra_core_error.MastraError({
+				id: (0, _mastra_core_storage.createStorageErrorId)("MYSQL", "LIST_THREADS", "FAILED"),
 				domain: _mastra_core_error.ErrorDomain.STORAGE,
 				category: _mastra_core_error.ErrorCategory.THIRD_PARTY,
 				details: { filter: JSON.stringify(filter ?? {}) }
 			}, error);
+			this.logger?.error?.(mastraError.toString());
+			this.logger?.trackException?.(mastraError);
+			throw mastraError;
 		}
 	}
 	async getThreadsByResourceId(args) {
@@ -5539,7 +5621,10 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 				hasMore: false
 			};
 			if (perPage === 0 && include && include.length > 0) {
-				const includeRows = await this._getIncludedMessages({ include });
+				const includeRows = await this._getIncludedMessages({
+					include,
+					resourceId
+				});
 				if (!includeRows || includeRows.length === 0) return {
 					messages: [],
 					total: 0,
@@ -5577,7 +5662,8 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 			const includeMessages = await this.collectIncludeMessages({
 				threadId: primaryThreadId,
 				include,
-				messagesByThread
+				messagesByThread,
+				resourceId
 			});
 			const combinedMap = /* @__PURE__ */ new Map();
 			for (const msg of paginatedMain) combinedMap.set(msg.id, msg);
@@ -5597,8 +5683,9 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 				hasMore: metadataFilter || !include?.length || mainThreadMessageCount < total ? baseHasMore : false
 			};
 		} catch (error) {
+			if (error instanceof _mastra_core_error.MastraError && error.category === _mastra_core_error.ErrorCategory.USER) throw error;
 			const mastraError = new _mastra_core_error.MastraError({
-				id: "MYSQL_MEMORY_GET_MESSAGES_PAGINATED_FAILED",
+				id: (0, _mastra_core_storage.createStorageErrorId)("MYSQL", "LIST_MESSAGES", "FAILED"),
 				domain: _mastra_core_error.ErrorDomain.STORAGE,
 				category: _mastra_core_error.ErrorCategory.THIRD_PARTY,
 				details: {
@@ -5609,13 +5696,7 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 			}, error);
 			this.logger?.error?.(mastraError.toString());
 			this.logger?.trackException?.(mastraError);
-			return {
-				messages: [],
-				total: 0,
-				page,
-				perPage,
-				hasMore: false
-			};
+			throw mastraError;
 		}
 	}
 	async listMessagesByResourceId(args) {
@@ -5670,7 +5751,7 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 			if (include && include.length > 0) {
 				const validInclude = (await Promise.all(include.map(async (inc) => {
 					if (inc.threadId) return inc;
-					const [msgRows] = await this.pool.execute(`SELECT thread_id FROM ${tableName} WHERE id = ? LIMIT 1`, [inc.id]);
+					const [msgRows] = await this.pool.execute(`SELECT thread_id FROM ${tableName} WHERE id = ? AND ${quoteIdentifier("resourceId", "column name")} = ? LIMIT 1`, [inc.id, resourceId]);
 					const threadId = msgRows?.[0]?.thread_id;
 					return threadId ? {
 						...inc,
@@ -5682,7 +5763,8 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 					const includeMessages = await this.collectIncludeMessages({
 						threadId: validInclude[0].threadId,
 						include: validInclude,
-						messagesByThread
+						messagesByThread,
+						resourceId
 					});
 					for (const includeMsg of includeMessages) if (!messageIds.has(includeMsg.id)) {
 						messages.push(includeMsg);
@@ -5707,21 +5789,16 @@ var MemoryMySQL = class MemoryMySQL extends _mastra_core_storage.MemoryStorage {
 				hasMore
 			};
 		} catch (error) {
+			if (error instanceof _mastra_core_error.MastraError && error.category === _mastra_core_error.ErrorCategory.USER) throw error;
 			const mastraError = new _mastra_core_error.MastraError({
-				id: (0, _mastra_core_storage.createStorageErrorId)("MYSQL", "LIST_MESSAGES", "FAILED"),
+				id: (0, _mastra_core_storage.createStorageErrorId)("MYSQL", "LIST_MESSAGES_BY_RESOURCE_ID", "FAILED"),
 				domain: _mastra_core_error.ErrorDomain.STORAGE,
 				category: _mastra_core_error.ErrorCategory.THIRD_PARTY,
 				details: { resourceId }
 			}, error);
 			this.logger?.error?.(mastraError.toString());
 			this.logger?.trackException?.(mastraError);
-			return {
-				messages: [],
-				total: 0,
-				page,
-				perPage: perPageForResponse,
-				hasMore: false
-			};
+			throw mastraError;
 		}
 	}
 	async getResourceById({ resourceId }) {