@mastra/spanner 1.4.0 → 1.5.0-alpha.0

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @mastra/spanner
 
+## 1.5.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.4.0
 
 ### Minor Changes

package/dist/docs/SKILL.md

@@ -3,7 +3,7 @@ name: mastra-spanner
 description: Documentation for @mastra/spanner. Use when working with @mastra/spanner APIs, configuration, or implementation.
 metadata:
   package: "@mastra/spanner"
-  version: "1.4.0"
+  version: "1.5.0-alpha.0"
 ---
 
 ## When to use

package/dist/docs/assets/SOURCE_MAP.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.4.0",
+  "version": "1.5.0-alpha.0",
   "package": "@mastra/spanner",
   "exports": {},
   "modules": {}

package/dist/docs/references/reference-storage-spanner.md

@@ -2,7 +2,7 @@
 
 # Google Cloud Spanner storage
 
-The Google Cloud Spanner storage implementation provides a horizontally scalable, strongly consistent storage backend for Mastra. It targets the GoogleSQL dialect of Cloud Spanner.
+The Google Cloud Spanner storage implementation provides a horizontally high-capacity, strongly consistent storage backend for Mastra. It targets the GoogleSQL dialect of Cloud Spanner.
 
 ## Installation
 
@@ -133,14 +133,14 @@ The storage adapter creates the following tables, all using the GoogleSQL dialec
 
 Tables are created with `STRING(MAX)` for text and JSON payloads, `INT64`, `FLOAT64`, `BOOL`, and `TIMESTAMP`.
 
-Two tables also carry Spanner-specific `STORED` generated columns that the adapter populates from JSON payloads so common filters can use a regular secondary index instead of a `JSON_VALUE` scan:
+The following tables carry Spanner-specific `STORED` generated columns that the adapter populates from JSON payloads so common filters can use a regular secondary index instead of a `JSON_VALUE` scan:
 
-- `mastra_workflow_snapshot.snapshotStatus` — extracts `$.status` from `snapshot`; backs `listWorkflowRuns({ status })`.
-- `mastra_schedules.target_workflow_id` — extracts `$.workflowId` from `target`; backs `listSchedules({ workflowId })`.
+- `mastra_workflow_snapshot.snapshotStatus`: Extracts `$.status` from `snapshot`. Backs `listWorkflowRuns({ status })`.
+- `mastra_schedules.target_workflow_id`: Extracts `$.workflowId` from `target`. Backs `listSchedules({ workflowId })`.
 
 Both are added via `ALTER TABLE ... ADD COLUMN IF NOT EXISTS` during `init()` and skipped under `initMode: 'validate'` (where the schema is owned externally). When the column is absent, the adapter falls back to a `JSON_VALUE` filter at runtime.
 
-The adapter doesn't create or use named schemas; use a dedicated database for isolation.
+The adapter doesn't create or use schemas. Use a dedicated database for isolation.
 
 ### Initialization
 
@@ -184,11 +184,11 @@ const thread = await memory?.getThreadById({ threadId: '...' })
 A few behaviors differ from other relational adapters:
 
 - Upserts use `INSERT OR UPDATE`. Spanner doesn't provide a `RETURNING` clause for upserts, so callers needing the post-write state must read it back.
-- No `TRUNCATE` exists; `dangerouslyClearAll()` issues `DELETE WHERE TRUE`.
+- No `TRUNCATE` exists. `dangerouslyClearAll()` issues `DELETE WHERE TRUE`.
 - Identifiers are quoted with backticks.
 - DDL is applied through `database.updateSchema(...)`, which is asynchronous (long-running operation).
 - `NULLS FIRST/LAST` isn't supported. Ordering with NULL handling is emulated through an `IS NULL` ordering key.
-- JSON containment isn't supported natively. `listTraces` `metadata` and `scope` filters compile to per-key `JSON_VALUE(...) = @v` equality checks, and `tags` filters compile to `EXISTS` over `JSON_QUERY_ARRAY(...)`. This differs from Postgres' `@>` containment operator (which can match nested structure in a single index scan) — most one-shot lookups still work but deeply nested structural matches aren't expressible.
+- JSON containment isn't supported natively. `listTraces` `metadata` and `scope` filters compile to per-key `JSON_VALUE(...) = @v` equality checks, and `tags` filters compile to `EXISTS` over `JSON_QUERY_ARRAY(...)`. This differs from Postgres' `@>` containment operator (which can match nested structure in a single index scan): most one-shot lookups still work but deeply nested structural matches aren't expressible.
 
 ### Direct database access
 
@@ -217,4 +217,4 @@ gcloud spanner instances create test-instance --config=emulator-config --nodes=1
 gcloud spanner databases create test-db --instance=test-instance
 ```
 
-Then connect with the same env var set in your Node.js process; the `@google-cloud/spanner` client detects the emulator automatically.
+Then connect with the same env var set in your Node.js process. The `@google-cloud/spanner` client detects the emulator automatically.

package/dist/index.cjs

@@ -2788,6 +2788,7 @@ function rowToItem(row) {
 		groundTruth: t.groundTruth ?? void 0,
 		expectedTrajectory: t.expectedTrajectory ?? void 0,
 		toolMocks: t.toolMocks ?? void 0,
+		unmockedToolPolicy: t.unmockedToolPolicy ?? void 0,
 		requestContext: t.requestContext ?? void 0,
 		metadata: t.metadata ?? void 0,
 		source: t.source ?? void 0,
@@ -2885,7 +2886,8 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 			ifNotExists: [
 				"organizationId",
 				"projectId",
-				"externalId"
+				"externalId",
+				"unmockedToolPolicy"
 			]
 		});
 		await this.createDefaultIndexes();
@@ -3350,6 +3352,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 							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,
@@ -3370,6 +3373,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 						groundTruth: args.groundTruth,
 						expectedTrajectory: args.expectedTrajectory,
 						toolMocks: args.toolMocks,
+						unmockedToolPolicy: args.unmockedToolPolicy,
 						requestContext: args.requestContext,
 						metadata: args.metadata,
 						source: args.source,
@@ -3419,6 +3423,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 				groundTruth: args.groundTruth !== void 0 ? args.groundTruth : existing.groundTruth,
 				expectedTrajectory: args.expectedTrajectory !== void 0 ? args.expectedTrajectory : existing.expectedTrajectory,
 				toolMocks: args.toolMocks !== void 0 ? args.toolMocks : existing.toolMocks,
+				unmockedToolPolicy: args.unmockedToolPolicy !== void 0 ? args.unmockedToolPolicy : existing.unmockedToolPolicy,
 				requestContext: args.requestContext !== void 0 ? args.requestContext : existing.requestContext,
 				metadata: args.metadata !== void 0 ? args.metadata : existing.metadata,
 				source: args.source !== void 0 ? args.source : existing.source
@@ -3444,6 +3449,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 							groundTruth: merged.groundTruth ?? null,
 							expectedTrajectory: merged.expectedTrajectory ?? null,
 							toolMocks: merged.toolMocks ?? null,
+							unmockedToolPolicy: merged.unmockedToolPolicy ?? null,
 							requestContext: merged.requestContext ?? null,
 							metadata: merged.metadata ?? null,
 							source: merged.source ?? null,
@@ -3465,6 +3471,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 						groundTruth: merged.groundTruth,
 						expectedTrajectory: merged.expectedTrajectory,
 						toolMocks: merged.toolMocks,
+						unmockedToolPolicy: merged.unmockedToolPolicy,
 						requestContext: merged.requestContext,
 						metadata: merged.metadata,
 						source: merged.source,
@@ -3523,6 +3530,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 							groundTruth: existing.groundTruth ?? null,
 							expectedTrajectory: existing.expectedTrajectory ?? null,
 							toolMocks: existing.toolMocks ?? null,
+							unmockedToolPolicy: existing.unmockedToolPolicy ?? null,
 							requestContext: existing.requestContext ?? null,
 							metadata: existing.metadata ?? null,
 							source: existing.source ?? null,
@@ -3839,6 +3847,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 								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,
@@ -3907,6 +3916,7 @@ var DatasetsSpanner = class DatasetsSpanner extends _mastra_core_storage.Dataset
 								groundTruth: existing.groundTruth ?? null,
 								expectedTrajectory: existing.expectedTrajectory ?? null,
 								toolMocks: existing.toolMocks ?? null,
+								unmockedToolPolicy: existing.unmockedToolPolicy ?? null,
 								requestContext: existing.requestContext ?? null,
 								metadata: existing.metadata ?? null,
 								source: existing.source ?? null,
@@ -3991,6 +4001,7 @@ function rowToExperimentResult(row) {
 		traceId: t.traceId ?? null,
 		status: t.status ?? null,
 		tags: t.tags ?? null,
+		comment: t.comment ?? null,
 		toolMockReport: t.toolMockReport ?? null,
 		createdAt: toDate(t.createdAt)
 	};
@@ -4035,7 +4046,11 @@ var ExperimentsSpanner = class ExperimentsSpanner extends _mastra_core_storage.E
 		await this.db.alterTable({
 			tableName: _mastra_core_storage.TABLE_EXPERIMENT_RESULTS,
 			schema: _mastra_core_storage.TABLE_SCHEMAS[_mastra_core_storage.TABLE_EXPERIMENT_RESULTS],
-			ifNotExists: ["organizationId", "projectId"]
+			ifNotExists: [
+				"comment",
+				"organizationId",
+				"projectId"
+			]
 		});
 		await this.createDefaultIndexes();
 		await this.createCustomIndexes();
@@ -4428,7 +4443,7 @@ var ExperimentsSpanner = class ExperimentsSpanner extends _mastra_core_storage.E
 	}
 	async updateExperimentResult(input) {
 		try {
-			if (input.status === void 0 && input.tags === void 0) {
+			if (input.status === void 0 && input.tags === void 0 && input.comment === void 0) {
 				const existing = await this.getExperimentResultById({ id: input.id });
 				if (!existing || input.experimentId !== void 0 && existing.experimentId !== input.experimentId) throw new _mastra_core_error.MastraError({
 					id: (0, _mastra_core_storage.createStorageErrorId)("SPANNER", "UPDATE_EXPERIMENT_RESULT", "NOT_FOUND"),
@@ -4452,6 +4467,11 @@ var ExperimentsSpanner = class ExperimentsSpanner extends _mastra_core_storage.E
 				params.tags = input.tags === null ? null : JSON.stringify(input.tags);
 				types.tags = "json";
 			}
+			if (input.comment !== void 0) {
+				setClauses.push(`${quoteIdent("comment", "column name")} = @comment`);
+				params.comment = input.comment;
+				if (input.comment === null) types.comment = "string";
+			}
 			const whereClauses = [`${quoteIdent("id", "column name")} = @id`];
 			if (input.experimentId !== void 0) {
 				whereClauses.push(`${quoteIdent("experimentId", "column name")} = @experimentId`);