@mastra/mongodb 1.15.1 → 1.16.0-alpha.1

package/CHANGELOG.md

@@ -1,5 +1,50 @@
 # @mastra/mongodb
 
+## 1.16.0-alpha.1
+
+### Patch Changes
+
+- When MongoDB storage initialization fails because an existing non-unique index conflicts with Mastra's required unique index, the error now includes step-by-step migration commands instead of a generic failure message. ([#20486](https://github.com/mastra-ai/mastra/pull/20486))
+
+  **Before:** `Failed to create default index on collection "mastra_threads". Set skipDefaultIndexes to manage indexes yourself.`
+
+  **After:**
+
+  ```text
+  Index conflict on collection "mastra_threads": an existing non-unique index on { id: 1 }
+  conflicts with Mastra's required unique index.
+
+  To migrate:
+    1. Check for duplicates:  db.mastra_threads.aggregate([{ $group: { _id: "$id", n: { $sum: 1 } } }, { $match: { n: { $gt: 1 } } }])
+    2. Drop the old index:    db.mastra_threads.dropIndex("id_1")
+    3. Recreate as unique:    db.mastra_threads.createIndex({ id: 1 }, { unique: true })
+
+  Alternatively, set skipDefaultIndexes: true to manage indexes yourself.
+  ```
+
+- Updated dependencies [[`594f7b2`](https://github.com/mastra-ai/mastra/commit/594f7b28f5263fb9982fd50d95c471fb971ea984), [`311f943`](https://github.com/mastra-ai/mastra/commit/311f943bee60e8fdf5c84499ea50e884276c936c), [`0c89896`](https://github.com/mastra-ai/mastra/commit/0c8989673fb7d106837098398131e570c6023b68), [`23b4238`](https://github.com/mastra-ai/mastra/commit/23b423844ad0bcf2a502a68dd62866d6160f9f6d), [`e320a76`](https://github.com/mastra-ai/mastra/commit/e320a763feaf65c6be3cebecf746defcbde161b3), [`03b4918`](https://github.com/mastra-ai/mastra/commit/03b4918c80d188ce375334c393e131c6e94bd7eb), [`14ef73a`](https://github.com/mastra-ai/mastra/commit/14ef73a4bbd73e7808414816eb0628ce1d80b5d7), [`1d677d5`](https://github.com/mastra-ai/mastra/commit/1d677d5f99d7db403f7828585e8c25f299f72628), [`93e28ec`](https://github.com/mastra-ai/mastra/commit/93e28ecce9031c02397e0ae8406593e5c7a95883), [`729dab4`](https://github.com/mastra-ai/mastra/commit/729dab408faccfaef0cbb048e5a4338f9172847e), [`484003d`](https://github.com/mastra-ai/mastra/commit/484003d33ff59330c86b19863e4a38732d7e4155), [`933d291`](https://github.com/mastra-ai/mastra/commit/933d291146b789c19442ad206f94da3e4be90c64)]:
+  - @mastra/core@1.56.0-alpha.3
+
+## 1.16.0-alpha.0
+
+### Minor Changes
+
+- Added persistence for dataset item undeclared tool policies. ([#19643](https://github.com/mastra-ai/mastra/pull/19643))
+
+  ```typescript
+  await dataset.addItem({
+    input: 'What is the weather?',
+    unmockedToolPolicy: 'deny',
+  });
+  ```
+
+### Patch Changes
+
+- Added a comment column to experiment results so review comments persist. The column is added automatically and non-destructively on startup for existing databases (https://github.com/mastra-ai/mastra/issues/19857). ([#19865](https://github.com/mastra-ai/mastra/pull/19865))
+
+- Updated dependencies [[`c5e56ff`](https://github.com/mastra-ai/mastra/commit/c5e56ff3bcabdf062708f2d48744fec304df6792), [`4e35a56`](https://github.com/mastra-ai/mastra/commit/4e35a56cdf8d74a5ff6d5eda01f2c1deaf6cc7be)]:
+  - @mastra/core@1.56.0-alpha.1
+
 ## 1.15.1
 
 ### Patch Changes

package/dist/docs/SKILL.md

@@ -3,7 +3,7 @@ name: mastra-mongodb
 description: Documentation for @mastra/mongodb. Use when working with @mastra/mongodb APIs, configuration, or implementation.
 metadata:
   package: "@mastra/mongodb"
-  version: "1.15.1"
+  version: "1.16.0-alpha.1"
 ---
 
 ## When to use

package/dist/docs/assets/SOURCE_MAP.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.15.1",
+  "version": "1.16.0-alpha.1",
   "package": "@mastra/mongodb",
   "exports": {},
   "modules": {}

package/dist/index.cjs

@@ -10,7 +10,7 @@ let _mastra_core_agent = require("@mastra/core/agent");
 let _mastra_core_evals = require("@mastra/core/evals");
 let _mastra_core_storage_domains_skills = require("@mastra/core/storage/domains/skills");
 //#region package.json
-var version = "1.15.1";
+var version = "1.16.0-alpha.1";
 //#endregion
 //#region src/vector/filter.ts
 /**
@@ -2799,6 +2799,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 			groundTruth: typeof row.groundTruth === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.groundTruth) : row.groundTruth,
 			expectedTrajectory: typeof row.expectedTrajectory === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.expectedTrajectory) : row.expectedTrajectory,
 			toolMocks: (typeof row.toolMocks === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.toolMocks) : row.toolMocks) ?? void 0,
+			unmockedToolPolicy: row.unmockedToolPolicy ?? void 0,
 			requestContext: typeof row.requestContext === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.requestContext) : row.requestContext,
 			metadata: typeof row.metadata === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.metadata) : row.metadata,
 			source: typeof row.source === "string" ? (0, _mastra_core_storage.safelyParseJSON)(row.source) : row.source,
@@ -3041,6 +3042,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 					groundTruth: args.groundTruth ?? null,
 					expectedTrajectory: args.expectedTrajectory ?? null,
 					toolMocks: args.toolMocks ?? null,
+					unmockedToolPolicy: args.unmockedToolPolicy ?? null,
 					requestContext: args.requestContext ?? null,
 					metadata: args.metadata ?? null,
 					source: args.source ?? null,
@@ -3064,6 +3066,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 				groundTruth: args.groundTruth,
 				expectedTrajectory: args.expectedTrajectory,
 				toolMocks: args.toolMocks,
+				unmockedToolPolicy: args.unmockedToolPolicy,
 				requestContext: args.requestContext,
 				metadata: args.metadata,
 				source: args.source,
@@ -3098,13 +3101,14 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 					actualDatasetId: existing.datasetId
 				}
 			});
-			if (!(args.input !== void 0 || args.groundTruth !== void 0 || args.expectedTrajectory !== void 0 || args.toolMocks !== void 0 || args.requestContext !== void 0 || args.metadata !== void 0 || args.source !== void 0)) return existing;
+			if (!(args.input !== void 0 || args.groundTruth !== void 0 || args.expectedTrajectory !== void 0 || args.toolMocks !== void 0 || args.unmockedToolPolicy !== void 0 || args.requestContext !== void 0 || args.metadata !== void 0 || args.source !== void 0)) return existing;
 			const now = /* @__PURE__ */ new Date();
 			const versionId = (0, crypto$1.randomUUID)();
 			const mergedInput = args.input !== void 0 ? args.input : existing.input;
 			const mergedGroundTruth = args.groundTruth !== void 0 ? args.groundTruth : existing.groundTruth;
 			const mergedExpectedTrajectory = args.expectedTrajectory !== void 0 ? args.expectedTrajectory : existing.expectedTrajectory;
 			const mergedToolMocks = args.toolMocks !== void 0 ? args.toolMocks : existing.toolMocks;
+			const mergedUnmockedToolPolicy = args.unmockedToolPolicy !== void 0 ? args.unmockedToolPolicy : existing.unmockedToolPolicy;
 			const mergedRequestContext = args.requestContext !== void 0 ? args.requestContext : existing.requestContext;
 			const mergedMetadata = args.metadata !== void 0 ? args.metadata : existing.metadata;
 			const mergedSource = args.source !== void 0 ? args.source : existing.source;
@@ -3146,6 +3150,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 					groundTruth: mergedGroundTruth,
 					expectedTrajectory: mergedExpectedTrajectory ?? null,
 					toolMocks: mergedToolMocks ?? null,
+					unmockedToolPolicy: mergedUnmockedToolPolicy ?? null,
 					requestContext: mergedRequestContext,
 					metadata: mergedMetadata,
 					source: mergedSource,
@@ -3168,6 +3173,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 				groundTruth: mergedGroundTruth,
 				expectedTrajectory: mergedExpectedTrajectory,
 				toolMocks: mergedToolMocks,
+				unmockedToolPolicy: mergedUnmockedToolPolicy,
 				requestContext: mergedRequestContext,
 				metadata: mergedMetadata,
 				source: mergedSource,
@@ -3233,6 +3239,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 					groundTruth: existing.groundTruth,
 					expectedTrajectory: existing.expectedTrajectory ?? null,
 					toolMocks: existing.toolMocks ?? null,
+					unmockedToolPolicy: existing.unmockedToolPolicy ?? null,
 					requestContext: existing.requestContext,
 					metadata: existing.metadata,
 					source: existing.source,
@@ -3305,6 +3312,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 							groundTruth: insert.item.groundTruth,
 							expectedTrajectory: insert.item.expectedTrajectory,
 							toolMocks: insert.item.toolMocks,
+							unmockedToolPolicy: insert.item.unmockedToolPolicy,
 							requestContext: insert.item.requestContext,
 							metadata: insert.item.metadata,
 							source: insert.item.source,
@@ -3385,6 +3393,7 @@ var MongoDBDatasetsStorage = class extends _mastra_core_storage.DatasetsStorage
 					groundTruth: item.groundTruth,
 					expectedTrajectory: item.expectedTrajectory ?? null,
 					toolMocks: item.toolMocks ?? null,
+					unmockedToolPolicy: item.unmockedToolPolicy ?? null,
 					requestContext: item.requestContext,
 					metadata: item.metadata,
 					source: item.source,
@@ -3685,6 +3694,7 @@ function transformExperimentResultRow(row) {
 		traceId: row.traceId ?? null,
 		status: row.status ?? null,
 		tags: Array.isArray(row.tags) ? row.tags : parseJsonField(row.tags) ?? null,
+		comment: row.comment ?? null,
 		toolMockReport: parseJsonField(row.toolMockReport) ?? null,
 		createdAt: toDate(row.createdAt)
 	};
@@ -4101,6 +4111,7 @@ var MongoDBExperimentsStorage = class MongoDBExperimentsStorage extends _mastra_
 		const updateFields = {};
 		if (input.status !== void 0) updateFields.status = input.status;
 		if (input.tags !== void 0) updateFields.tags = input.tags;
+		if (input.comment !== void 0) updateFields.comment = input.comment;
 		if (Object.keys(updateFields).length === 0) {
 			const existing = await this.getExperimentResultById({ id: input.id });
 			if (!existing || input.experimentId && existing.experimentId !== input.experimentId) throw new _mastra_core_error.MastraError({
@@ -5318,11 +5329,15 @@ var MemoryStorageMongoDB = class MemoryStorageMongoDB extends _mastra_core_stora
 		for (const indexDef of this.getDefaultIndexDefinitions()) try {
 			await (await this.getCollection(indexDef.collection)).createIndex(indexDef.keys, indexDef.options);
 		} catch (error) {
+			const isUniqueConflict = error?.code === 85 && indexDef.options?.unique === true;
+			const field = Object.keys(indexDef.keys)[0] ?? "id";
+			const indexName = Object.entries(indexDef.keys).map(([k, v]) => `${k}_${v}`).join("_");
+			const text = isUniqueConflict ? `Index conflict on collection "${indexDef.collection}": an existing non-unique index on { ${field}: 1 } conflicts with Mastra's required unique index.\n\nTo migrate:\n  1. Check for duplicates:  db.${indexDef.collection}.aggregate([{ $group: { _id: "$${field}", n: { $sum: 1 } } }, { $match: { n: { $gt: 1 } } }])\n  2. Drop the old index:    db.${indexDef.collection}.dropIndex("${indexName}")\n  3. Recreate as unique:    db.${indexDef.collection}.createIndex({ ${field}: 1 }, { unique: true })\n\nAlternatively, set skipDefaultIndexes: true to manage indexes yourself.` : `Failed to create default index on collection "${indexDef.collection}". Set skipDefaultIndexes to manage indexes yourself.`;
 			throw new _mastra_core_error.MastraError({
 				id: (0, _mastra_core_storage.createStorageErrorId)("MONGODB", "CREATE_DEFAULT_INDEXES", "FAILED"),
 				domain: _mastra_core_error.ErrorDomain.STORAGE,
 				category: _mastra_core_error.ErrorCategory.THIRD_PARTY,
-				text: `Failed to create default index on collection "${indexDef.collection}". Set skipDefaultIndexes to manage indexes yourself.`,
+				text,
 				details: { collection: indexDef.collection }
 			}, error);
 		}