sqlite-hub 0.12.0 → 0.17.0

package/server/services/sqlite/introspection.js

@@ -345,12 +345,20 @@ function getTableDetail(db, tableName, options = {}) {
   const foreignKeys = groupForeignKeys(
     db.prepare(`PRAGMA foreign_key_list(${quoteIdentifier(tableName)})`).all()
   );
+  const checkConstraints = extractCheckExpressions(entry.sql).map((expression, index) => ({
+    id: index,
+    expression: expression.trim(),
+  }));
 
   const indexList = db
     .prepare(`PRAGMA index_list(${quoteIdentifier(tableName)})`)
     .all()
     .map((indexEntry) => {
       let indexColumns = [];
+      const indexSql =
+        db
+          .prepare("SELECT sql FROM sqlite_master WHERE type = 'index' AND name = ?")
+          .get(indexEntry.name)?.sql ?? null;
 
       try {
         indexColumns = db
@@ -380,6 +388,7 @@ function getTableDetail(db, tableName, options = {}) {
         unique: Boolean(indexEntry.unique),
         origin: indexEntry.origin,
         partial: Boolean(indexEntry.partial),
+        sql: indexSql,
         columns: indexColumns,
       };
     });
@@ -402,6 +411,7 @@ function getTableDetail(db, tableName, options = {}) {
     withoutRowId,
     strict: Boolean(tableListEntry?.strict),
     columns,
+    checkConstraints,
     foreignKeys,
     indexes: indexList,
     indexCount: indexList.length,

package/server/services/sqlite/sqlExecutor.js

@@ -204,6 +204,13 @@ function mapEditableColumns(tableDetail, columnDefinitions) {
   const tableColumnsByName = new Map(
     tableDetail.columns.map((column) => [column.name, column])
   );
+  const foreignKeyColumnNames = new Set(
+    (tableDetail.foreignKeys ?? []).flatMap((foreignKey) =>
+      (foreignKey.mappings ?? [])
+        .map((mapping) => String(mapping.from ?? "").trim())
+        .filter(Boolean)
+    )
+  );
   const identityColumns = new Set(
     tableDetail.identityStrategy?.type === "primaryKey"
       ? tableDetail.identityStrategy.columns ?? []
@@ -223,12 +230,31 @@ function mapEditableColumns(tableDetail, columnDefinitions) {
       visible: isRowId ? true : Boolean(columnMeta?.visible),
       generated: Boolean(columnMeta?.generated),
       identity: identityColumns.has(definition.column),
+      declaredType: columnMeta?.declaredType ?? "",
+      affinity: columnMeta?.affinity ?? "",
+      primaryKeyPosition: Number(columnMeta?.primaryKeyPosition ?? 0),
+      foreignKey: foreignKeyColumnNames.has(definition.column),
       notNull: Boolean(columnMeta?.notNull),
       allowedValues: columnMeta?.allowedValues ?? [],
     };
   });
 }
 
+function buildTableMeta(tableDetail) {
+  return {
+    columns: (tableDetail.columns ?? []).map((column) => ({
+      name: column.name,
+      declaredType: column.declaredType,
+      affinity: column.affinity,
+      primaryKeyPosition: Number(column.primaryKeyPosition ?? 0),
+      foreignKey: (tableDetail.foreignKeys ?? []).some((foreignKey) =>
+        (foreignKey.mappings ?? []).some((mapping) => mapping.from === column.name)
+      ),
+    })),
+    foreignKeys: tableDetail.foreignKeys ?? [],
+  };
+}
+
 function hasRequiredIdentityColumns(tableDetail, editableColumns) {
   const availableSourceColumns = new Set(editableColumns.map((column) => column.sourceColumn));
 
@@ -301,6 +327,7 @@ function resolveEditableResult(db, columnDefinitions, serializedRows) {
       reason: getEditableResultReason(tableDetail),
       columns: editableColumns,
       identityStrategy: tableDetail.identityStrategy,
+      tableMeta: buildTableMeta(tableDetail),
     };
   }
 
@@ -319,6 +346,7 @@ function resolveEditableResult(db, columnDefinitions, serializedRows) {
     reason: "",
     columns: editableColumns,
     identityStrategy: tableDetail.identityStrategy,
+    tableMeta: buildTableMeta(tableDetail),
     rows,
   };
 }
@@ -378,6 +406,7 @@ class SqlExecutor {
                   reason: editableResult.reason ?? "",
                   columns: editableResult.columns ?? [],
                   identityStrategy: editableResult.identityStrategy ?? null,
+                  tableMeta: editableResult.tableMeta ?? null,
                 }
               : null,
           };

package/server/services/sqlite/tableDesigner/changeAnalysis.js

@@ -26,6 +26,17 @@ function hasColumnChanged(originalColumn, draftColumn) {
   );
 }
 
+function hasConstraintChanged(constraint) {
+  return (
+    normalizeComparableValue(constraint.name) !==
+      normalizeComparableValue(constraint.originalName ?? constraint.name) ||
+    normalizeComparableValue(constraint.expression || constraint.sql) !==
+      normalizeComparableValue(
+        constraint.originalExpression ?? constraint.originalSql ?? constraint.expression ?? constraint.sql
+      )
+  );
+}
+
 function hasMeaningfulNewDraftContent(draft) {
   return (
     Boolean(String(draft.tableName ?? "").trim()) ||
@@ -149,7 +160,7 @@ function analyzeEditDraft(draft, originalDraft) {
           warnings.push(
             buildRiskyChangeWarning(
               "Column Rename Requires Rebuild",
-              `Renaming column ${originalColumn.name} to ${column.name} is intentionally blocked in Table Designer v1.`
+              `Renaming column ${originalColumn.name} to ${column.name} is intentionally blocked in Table Designer v2.`
             )
           );
         }
@@ -265,6 +276,19 @@ function analyzeEditDraft(draft, originalDraft) {
     );
   });
 
+  [...(draft.uniqueConstraints ?? []), ...(draft.checkConstraints ?? [])].forEach((constraint) => {
+    if (!hasConstraintChanged(constraint)) {
+      return;
+    }
+
+    warnings.push(
+      buildRiskyChangeWarning(
+        "Constraint Change Requires Rebuild",
+        `Changing ${constraint.originalName || constraint.name || "a table constraint"} requires a SQLite table rebuild.`
+      )
+    );
+  });
+
   const dirty = Boolean(executableStatements.length || warnings.length);
 
   return {

package/server/services/sqlite/tableDesigner/schemaMapping.js

@@ -74,6 +74,95 @@ function buildSimpleForeignKeyMap(foreignKeys = []) {
   return map;
 }
 
+function buildUniqueConstraintExpression(index = {}) {
+  const columns = (index.columns ?? [])
+    .map((column) => column?.name)
+    .filter(Boolean);
+
+  if (!columns.length) {
+    return "UNIQUE constraint";
+  }
+
+  return `UNIQUE (${columns.map((columnName) => quoteIdentifier(columnName)).join(", ")})`;
+}
+
+function mapComplexUniqueConstraint(index = {}) {
+  const sql = String(index.sql ?? "").trim();
+  const columns = (index.columns ?? [])
+    .map((column) => ({
+      name: column.name ?? "",
+      descending: Boolean(column.descending),
+      collation: column.collation ?? "",
+    }))
+    .filter((column) => column.name);
+
+  return {
+    id: `unique:${index.name}`,
+    name: index.name ?? "",
+    originalName: index.name ?? "",
+    columns,
+    partial: Boolean(index.partial),
+    origin: index.origin ?? "",
+    sql,
+    originalSql: sql,
+    expression: sql || buildUniqueConstraintExpression(index),
+    originalExpression: sql || buildUniqueConstraintExpression(index),
+    editable: true,
+    preserved: true,
+  };
+}
+
+function buildComplexUniqueConstraints(indexes = []) {
+  return indexes
+    .filter((index) => index?.unique && (index.partial || (index.columns?.length ?? 0) !== 1))
+    .map(mapComplexUniqueConstraint);
+}
+
+function expressionMentionsColumn(expression = "", columnName = "") {
+  const normalizedExpression = String(expression ?? "").toLowerCase();
+  const normalizedColumn = String(columnName ?? "").toLowerCase();
+
+  if (!normalizedColumn) {
+    return false;
+  }
+
+  return (
+    new RegExp(`(^|[^a-z0-9_$])${normalizedColumn.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}([^a-z0-9_$]|$)`).test(
+      normalizedExpression
+    ) ||
+    normalizedExpression.includes(`"${normalizedColumn.replaceAll('"', '""')}"`) ||
+    normalizedExpression.includes(`\`${normalizedColumn.replaceAll("`", "``")}\``) ||
+    normalizedExpression.includes(`[${normalizedColumn}]`)
+  );
+}
+
+function buildCheckConstraints(tableDetail) {
+  const visibleColumns = (tableDetail.columns ?? []).filter(
+    (column) => column.visible !== false && !column.generated
+  );
+
+  return (tableDetail.checkConstraints ?? []).map((constraint, index) => {
+    const expression = String(constraint.expression ?? "").trim();
+    const columns = visibleColumns
+      .filter((column) => expressionMentionsColumn(expression, column.name))
+      .map((column) => ({
+        name: column.name,
+        allowedValues: Array.isArray(column.allowedValues) ? column.allowedValues : [],
+      }));
+
+    return {
+      id: `check:${constraint.id ?? index}`,
+      name: `CHECK ${index + 1}`,
+      originalName: `CHECK ${index + 1}`,
+      columns,
+      expression: expression ? `CHECK (${expression})` : "CHECK constraint",
+      originalExpression: expression ? `CHECK (${expression})` : "CHECK constraint",
+      editable: true,
+      preserved: true,
+    };
+  });
+}
+
 function buildSchemaWarnings(tableDetail) {
   const warnings = [];
   const generatedColumns = (tableDetail.columns ?? []).filter((column) => column.generated);
@@ -83,9 +172,7 @@ function buildSchemaWarnings(tableDetail) {
   const complexForeignKeys = (tableDetail.foreignKeys ?? []).filter(
     (foreignKey) => (foreignKey.mappings?.length ?? 0) !== 1
   );
-  const complexUniqueIndexes = (tableDetail.indexes ?? []).filter(
-    (index) => index.unique && (index.partial || (index.columns?.length ?? 0) !== 1)
-  );
+  const complexUniqueConstraints = buildComplexUniqueConstraints(tableDetail.indexes);
 
   if (generatedColumns.length) {
     warnings.push(
@@ -93,7 +180,7 @@ function buildSchemaWarnings(tableDetail) {
         code: "GENERATED_COLUMNS_PRESENT",
         title: "Generated Columns Detected",
         message:
-          "Generated or hidden columns are not editable in Table Designer v1. Safe operations like table rename or adding simple columns still work.",
+          "Generated or hidden columns are inspectable but not editable in Table Designer v2. Safe operations like table rename or adding simple columns still work.",
       })
     );
   }
@@ -104,7 +191,7 @@ function buildSchemaWarnings(tableDetail) {
         code: "COMPOSITE_PRIMARY_KEY_PRESENT",
         title: "Composite Primary Key Detected",
         message:
-          "This table uses more than one primary key column. Table Designer v1 preserves it, but changing primary key structure requires a manual table rebuild.",
+          "This table uses more than one primary key column. Table Designer v2 preserves it, but changing primary key structure requires a manual table rebuild.",
       })
     );
   }
@@ -115,18 +202,19 @@ function buildSchemaWarnings(tableDetail) {
         code: "COMPLEX_FOREIGN_KEYS_PRESENT",
         title: "Complex Foreign Keys Detected",
         message:
-          "Composite or multi-mapping foreign keys cannot be edited directly in Table Designer v1. They are preserved until a rebuild is done manually.",
+          "Composite or multi-mapping foreign keys cannot be edited directly in Table Designer v2. They are preserved until a rebuild is done manually.",
       })
     );
   }
 
-  if (complexUniqueIndexes.length) {
+  if (complexUniqueConstraints.length) {
     warnings.push(
       createDesignerWarning({
         code: "COMPLEX_UNIQUE_CONSTRAINTS_PRESENT",
-        title: "Complex Unique Constraints Detected",
+        title: "Table Designer v2 Unique Constraints",
         message:
-          "Multi-column or partial UNIQUE constraints are outside Table Designer v1. They remain untouched unless you rebuild the table manually.",
+          "Multi-column and partial UNIQUE constraints are detected, shown in the v2 constraints panel, and preserved by SQLite-safe saves. Editing them still requires manual SQL review.",
+        tone: "muted",
       })
     );
   }
@@ -149,7 +237,7 @@ function buildSchemaWarnings(tableDetail) {
         code: "WITHOUT_ROWID_PRESENT",
         title: "WITHOUT ROWID Table",
         message:
-          "WITHOUT ROWID tables can be inspected here, but rebuild-style changes are intentionally blocked in v1.",
+          "WITHOUT ROWID tables can be inspected here, but rebuild-style changes are intentionally blocked in v2.",
         tone: "muted",
       })
     );
@@ -189,6 +277,8 @@ function mapTableColumnToDraft(column, { uniqueColumns, foreignKeyMap }) {
 function buildTableDesignerDraft(tableDetail) {
   const uniqueColumns = buildSingleColumnUniqueSet(tableDetail.indexes);
   const foreignKeyMap = buildSimpleForeignKeyMap(tableDetail.foreignKeys);
+  const uniqueConstraints = buildComplexUniqueConstraints(tableDetail.indexes);
+  const checkConstraints = buildCheckConstraints(tableDetail);
   const columns = (tableDetail.columns ?? [])
     .filter((column) => column.visible !== false && !column.generated)
     .map((column) => mapTableColumnToDraft(column, { uniqueColumns, foreignKeyMap }));
@@ -199,6 +289,9 @@ function buildTableDesignerDraft(tableDetail) {
     originalTableName: tableDetail.name,
     tableName: tableDetail.name,
     columns,
+    uniqueConstraints,
+    checkConstraints,
+    designerVersion: 2,
     dirty: false,
     schemaWarnings,
     warnings: [...schemaWarnings],
@@ -227,6 +320,8 @@ function listDesignerTables(db) {
 module.exports = {
   SUPPORTED_TABLE_DESIGNER_TYPES,
   buildTableDesignerDraft,
+  buildComplexUniqueConstraints,
+  buildCheckConstraints,
   createDesignerWarning,
   listDesignerTables,
   normalizeDesignerType,

package/server/services/sqlite/tableDesigner/validation.js

@@ -66,6 +66,53 @@ function normalizeColumnPayload(column = {}, index = 0) {
   };
 }
 
+function normalizeUniqueConstraintPayload(constraint = {}, index = 0) {
+  return {
+    id: String(constraint.id ?? `unique:${index}`),
+    name: String(constraint.name ?? "").trim(),
+    originalName: String(constraint.originalName ?? constraint.name ?? "").trim(),
+    columns: Array.isArray(constraint.columns)
+      ? constraint.columns
+          .map((column) => ({
+            name: String(column?.name ?? "").trim(),
+            descending: normalizeBoolean(column?.descending),
+            collation: String(column?.collation ?? "").trim(),
+          }))
+          .filter((column) => column.name)
+      : [],
+    partial: normalizeBoolean(constraint.partial),
+    origin: String(constraint.origin ?? "").trim(),
+    sql: String(constraint.sql ?? "").trim(),
+    originalSql: String(constraint.originalSql ?? constraint.sql ?? "").trim(),
+    expression: String(constraint.expression ?? "").trim(),
+    originalExpression: String(constraint.originalExpression ?? constraint.expression ?? constraint.sql ?? "").trim(),
+    editable: normalizeBoolean(constraint.editable),
+    preserved: constraint.preserved === undefined ? true : normalizeBoolean(constraint.preserved),
+  };
+}
+
+function normalizeCheckConstraintPayload(constraint = {}, index = 0) {
+  return {
+    id: String(constraint.id ?? `check:${index}`),
+    name: String(constraint.name ?? `CHECK ${index + 1}`).trim(),
+    originalName: String(constraint.originalName ?? constraint.name ?? `CHECK ${index + 1}`).trim(),
+    columns: Array.isArray(constraint.columns)
+      ? constraint.columns
+          .map((column) => ({
+            name: String(column?.name ?? "").trim(),
+            allowedValues: Array.isArray(column?.allowedValues)
+              ? column.allowedValues.map((value) => String(value ?? ""))
+              : [],
+          }))
+          .filter((column) => column.name)
+      : [],
+    expression: String(constraint.expression ?? "").trim(),
+    originalExpression: String(constraint.originalExpression ?? constraint.expression ?? "").trim(),
+    editable: normalizeBoolean(constraint.editable),
+    preserved: constraint.preserved === undefined ? true : normalizeBoolean(constraint.preserved),
+  };
+}
+
 function normalizeDraftPayload(payload = {}) {
   const mode = String(payload.mode ?? "create").trim() === "edit" ? "edit" : "create";
 
@@ -76,6 +123,17 @@ function normalizeDraftPayload(payload = {}) {
     columns: Array.isArray(payload.columns)
       ? payload.columns.map((column, index) => normalizeColumnPayload(column, index))
       : [],
+    uniqueConstraints: Array.isArray(payload.uniqueConstraints)
+      ? payload.uniqueConstraints.map((constraint, index) =>
+          normalizeUniqueConstraintPayload(constraint, index)
+        )
+      : [],
+    checkConstraints: Array.isArray(payload.checkConstraints)
+      ? payload.checkConstraints.map((constraint, index) =>
+          normalizeCheckConstraintPayload(constraint, index)
+        )
+      : [],
+    designerVersion: Number(payload.designerVersion) || 1,
     schemaWarnings: Array.isArray(payload.schemaWarnings) ? payload.schemaWarnings : [],
     warnings: Array.isArray(payload.warnings) ? payload.warnings : [],
     fillImportedRows: normalizeBoolean(payload.fillImportedRows),
@@ -112,7 +170,7 @@ function validatePrimaryKeys(draft, originalDraft) {
 
   if (!originalDraft) {
     throw new ValidationError(
-      "Table Designer v1 supports a single primary key column when creating a table."
+      "Table Designer v2 supports a single primary key column when creating a table."
     );
   }
 
@@ -134,7 +192,7 @@ function validatePrimaryKeys(draft, originalDraft) {
 
   if (!isUnchangedCompositePrimaryKey) {
     throw new ValidationError(
-      "Table Designer v1 does not support editing composite primary keys."
+      "Table Designer v2 does not support editing composite primary keys."
     );
   }
 }

package/server/services/sqlite/tableDesignerService.js

@@ -109,7 +109,7 @@ class TableDesignerService {
 
     if (!analysis.executable) {
       throw new ValidationError(
-        "This schema change set would require a SQLite table rebuild. Table Designer v1 keeps the SQL preview available but will not execute those changes automatically.",
+        "This schema change set would require a SQLite table rebuild. Table Designer v2 keeps the SQL preview available but will not execute those changes automatically.",
         {
           code: "TABLE_DESIGNER_REBUILD_REQUIRED",
           warnings: analysis.warnings,