@mastra/libsql 1.17.0-alpha.3 → 1.17.0-alpha.4

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @mastra/libsql
 
+## 1.17.0-alpha.4
+
+### Patch Changes
+
+- Fix factory storage error classification and schema drift handling. ([#20002](https://github.com/mastra-ai/mastra/pull/20002))
+
+  - `isUniqueViolation` in the LibSQL factory storage now only matches real unique-index violations (`SQLITE_CONSTRAINT_UNIQUE` / `SQLITE_CONSTRAINT_PRIMARYKEY`). Previously any `SQLITE_CONSTRAINT` failure — including NOT NULL violations — was reported as a "Unique constraint violation", which masked the real error (seen as `Unique constraint violation on collection 'factory_rule_evaluations'` when polled ingestion inserted nullable columns into a stale table).
+  - `ensureCollections` in both the LibSQL and Postgres factory storage adapters now relaxes stale NOT NULL constraints when a column becomes nullable in the schema. Postgres uses `ALTER COLUMN ... DROP NOT NULL`; LibSQL rebuilds the table in place since SQLite cannot drop NOT NULL directly. Existing rows and unique indexes are preserved.
+
+- Updated dependencies:
+  - @mastra/core@1.52.0-alpha.13
+
 ## 1.17.0-alpha.3
 
 ### Minor Changes

package/dist/docs/SKILL.md

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

package/dist/docs/assets/SOURCE_MAP.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.17.0-alpha.3",
+  "version": "1.17.0-alpha.4",
   "package": "@mastra/libsql",
   "exports": {},
   "modules": {}

package/dist/docs/references/docs-agent-builder-overview.md

@@ -88,6 +88,8 @@ Start Mastra's development server:
 npx mastra dev
 ```
 
+For Agent Builder development, you can also use `mastra factory dev`, which uses the same dev runtime and flags as `mastra dev` and writes to the same `.mastra/output` directory. Both commands share a dev lock and can't run simultaneously in the same project. See the [CLI reference](https://mastra.ai/reference/cli/mastra) for details.
+
 The Agent Builder is mounted at `http://localhost:4111/agent-builder`.
 
 ## Disabling the Builder

package/dist/index.cjs

@@ -13365,8 +13365,10 @@ function assertIdentifier(kind, name) {
   }
 }
 function isUniqueViolation(error) {
+  const code = typeof error === "object" && error !== null ? error.code : void 0;
+  if (code === "SQLITE_CONSTRAINT_UNIQUE" || code === "SQLITE_CONSTRAINT_PRIMARYKEY") return true;
   const message = error instanceof Error ? error.message : String(error);
-  return message.includes("UNIQUE constraint failed") || message.includes("SQLITE_CONSTRAINT");
+  return message.includes("UNIQUE constraint failed") || message.includes("SQLITE_CONSTRAINT_UNIQUE") || message.includes("SQLITE_CONSTRAINT_PRIMARYKEY");
 }
 function primaryKeyOf(schema) {
   const pks = Object.entries(schema.columns).filter(([, spec]) => spec.type === "uuid-pk" || spec.primaryKey);
@@ -13710,6 +13712,35 @@ var LibSQLFactoryStorage = class extends storage.FactoryStorage {
     if (spec.default !== void 0) ddl += ` DEFAULT ${serializeDefault(spec.default)}`;
     return ddl;
   }
+  /**
+   * A table created by an older schema may still say NOT NULL on a column the
+   * current schema declares nullable — inserts of null then fail on databases
+   * that predate the change. SQLite has no `ALTER COLUMN DROP NOT NULL`, so
+   * relaxation swaps in a table rebuilt from the current schema (create shadow
+   * → copy → drop → rename) atomically.
+   */
+  async #relaxDriftedNotNulls(schema) {
+    const info = await this.#client.execute(`PRAGMA table_info("${schema.name}")`);
+    const hasDrift = info.rows.some((row) => {
+      const spec = schema.columns[String(row.name)];
+      if (!spec || spec.type === "uuid-pk" || spec.primaryKey) return false;
+      return spec.nullable === true && Number(row.notnull) === 1;
+    });
+    if (!hasDrift) return;
+    const ddl = Object.entries(schema.columns).map(([name, spec]) => this.#columnDdl(name, spec));
+    const shadow = `${schema.name}__nullable_rebuild`;
+    const cols = Object.keys(schema.columns).map((name) => `"${name}"`).join(", ");
+    await this.#client.batch(
+      [
+        `DROP TABLE IF EXISTS "${shadow}"`,
+        `CREATE TABLE "${shadow}" (${ddl.join(", ")})`,
+        `INSERT INTO "${shadow}" (${cols}) SELECT ${cols} FROM "${schema.name}"`,
+        `DROP TABLE "${schema.name}"`,
+        `ALTER TABLE "${shadow}" RENAME TO "${schema.name}"`
+      ],
+      "write"
+    );
+  }
   async #ensureCollection(schema) {
     assertIdentifier("collection", schema.name);
     primaryKeyOf(schema);
@@ -13721,6 +13752,7 @@ var LibSQLFactoryStorage = class extends storage.FactoryStorage {
       if (existing.has(name)) continue;
       await this.#client.execute(`ALTER TABLE "${schema.name}" ADD COLUMN ${this.#columnDdl(name, spec)}`);
     }
+    await this.#relaxDriftedNotNulls(schema);
     for (const index of schema.uniqueIndexes ?? []) {
       assertIdentifier("index", index.name);
       index.columns.forEach((column) => assertIdentifier("column", column));