bun-query-builder 0.2.13 → 0.2.15

package/dist/bin/cli.js

@@ -28419,7 +28419,7 @@ function detectTypeFromValidationRule(rule) {
       case "string":
       case "text": {
         const maxValue = extractMaxFromRules(ruleObj);
-        return maxValue && maxValue > 255 ? "text" : "string";
+        return maxValue && maxValue > MAX_BOUNDED_VARCHAR_LENGTH ? "text" : "string";
       }
       case "integer":
       case "int":
@@ -28569,7 +28569,7 @@ function buildMigrationPlan2(models, options) {
       inferred = normalizeAttributeType(attr.type) ?? detectTypeFromValidationRule(attr.validation?.rule);
       if (inferred === "string") {
         const maxLength = extractMaxFromRules(attr.validation?.rule);
-        if (maxLength && maxLength > 255)
+        if (maxLength && maxLength > MAX_BOUNDED_VARCHAR_LENGTH)
           inferred = "text";
       }
       if (!inferred || inferred === "enum") {
@@ -28608,7 +28608,7 @@ function buildMigrationPlan2(models, options) {
         hasDefault: typeof attr.default !== "undefined",
         defaultValue: normalizeDefaultValue(attr.default),
         enumValues,
-        maxLength: typeof maxLen === "number" && maxLen > 0 && maxLen <= 255 ? maxLen : undefined
+        maxLength: typeof maxLen === "number" && maxLen > 0 && maxLen <= MAX_BOUNDED_VARCHAR_LENGTH ? maxLen : undefined
       };
       if (columnName.endsWith("_id") && attr.foreignKey !== false && !polymorphicColumns.has(columnName)) {
         if (typeof attr.foreignKey === "object" && attr.foreignKey !== null) {
@@ -29296,14 +29296,20 @@ function generateDiffOperations(previous, next, opts = {}) {
       removedCols = detected.removed;
       addedCols = detected.added;
     }
-    if (preserveUnknownColumns && removedCols.length > 0) {
+    if (preserveUnknownColumns) {
       const preserved = removedCols.map((name) => prevCols[name]);
-      const preservedNames = new Set(removedCols);
-      const keptIndexes = prev.indexes.filter((idx) => idx.columns.some((c2) => preservedNames.has(c2)) && !currIdx[indexKey(idx, prev.table)]);
-      info2(`-- Keeping ${preserved.length} column(s) on "${curr.table}" no model declares: ${removedCols.join(", ")} \u2014 reconciled from the database, where an undeclared column is out of scope rather than removed`);
-      curr = { ...curr, columns: [...curr.columns, ...preserved], indexes: [...curr.indexes, ...keptIndexes] };
-      currCols = mapColumnsByName(curr.columns);
-      currIdx = mapIndexesByKey(curr.indexes, curr.table);
+      const keptIndexes = prev.indexes.filter((idx) => !currIdx[indexKey(idx, prev.table)]);
+      if (preserved.length > 0 || keptIndexes.length > 0) {
+        if (preserved.length > 0) {
+          info2(`-- Keeping ${preserved.length} column(s) on "${curr.table}" no model declares: ${removedCols.join(", ")} \u2014 reconciled from the database, where an undeclared column is out of scope rather than removed`);
+        }
+        if (keptIndexes.length > 0) {
+          info2(`-- Keeping ${keptIndexes.length} index(es) on "${curr.table}" no model declares: ${keptIndexes.map((index) => index.name).join(", ")} \u2014 reconciled from the database, where an undeclared index is out of scope rather than removed`);
+        }
+        curr = { ...curr, columns: [...curr.columns, ...preserved], indexes: [...curr.indexes, ...keptIndexes] };
+        currCols = mapColumnsByName(curr.columns);
+        currIdx = mapIndexesByKey(curr.indexes, curr.table);
+      }
       removedCols = [];
     }
     const modifiedCols = Object.keys(currCols).filter((n2) => prevCols[n2] && columnsAreDifferent(prevCols[n2], currCols[n2], dialect));
@@ -29453,7 +29459,7 @@ function generateDiffOperations(previous, next, opts = {}) {
 function generateDiffSql(previous, next, opts = {}) {
   return generateDiffOperations(previous, next, opts).statements;
 }
-var migrationCounter = 0, migrationsCreatedCount = 0, migrationsUpdatedCount = 0, useDeterministicNames = true, createdMigrationFiles, existingCorpusBodies = null, GENERATED_MARKER = "-- qb:generated";
+var migrationCounter = 0, migrationsCreatedCount = 0, migrationsUpdatedCount = 0, useDeterministicNames = true, createdMigrationFiles, existingCorpusBodies = null, GENERATED_MARKER = "-- qb:generated", MAX_BOUNDED_VARCHAR_LENGTH = 767;
 var init_migrations = __esm(() => {
   init_config();
   init_drivers();
@@ -31542,7 +31548,7 @@ function getPrefix() {
 }
 var prefix = getPrefix();
 // package.json
-var version2 = "0.2.13";
+var version2 = "0.2.15";
 
 // bin/cli.ts
 init_actions();

package/dist/src/index.js

@@ -28418,7 +28418,7 @@ function detectTypeFromValidationRule(rule) {
       case "string":
       case "text": {
         const maxValue = extractMaxFromRules(ruleObj);
-        return maxValue && maxValue > 255 ? "text" : "string";
+        return maxValue && maxValue > MAX_BOUNDED_VARCHAR_LENGTH ? "text" : "string";
       }
       case "integer":
       case "int":
@@ -28568,7 +28568,7 @@ function buildMigrationPlan2(models, options) {
       inferred = normalizeAttributeType(attr.type) ?? detectTypeFromValidationRule(attr.validation?.rule);
       if (inferred === "string") {
         const maxLength = extractMaxFromRules(attr.validation?.rule);
-        if (maxLength && maxLength > 255)
+        if (maxLength && maxLength > MAX_BOUNDED_VARCHAR_LENGTH)
           inferred = "text";
       }
       if (!inferred || inferred === "enum") {
@@ -28607,7 +28607,7 @@ function buildMigrationPlan2(models, options) {
         hasDefault: typeof attr.default !== "undefined",
         defaultValue: normalizeDefaultValue(attr.default),
         enumValues,
-        maxLength: typeof maxLen === "number" && maxLen > 0 && maxLen <= 255 ? maxLen : undefined
+        maxLength: typeof maxLen === "number" && maxLen > 0 && maxLen <= MAX_BOUNDED_VARCHAR_LENGTH ? maxLen : undefined
       };
       if (columnName.endsWith("_id") && attr.foreignKey !== false && !polymorphicColumns.has(columnName)) {
         if (typeof attr.foreignKey === "object" && attr.foreignKey !== null) {
@@ -29295,14 +29295,20 @@ function generateDiffOperations(previous, next, opts = {}) {
       removedCols = detected.removed;
       addedCols = detected.added;
     }
-    if (preserveUnknownColumns && removedCols.length > 0) {
+    if (preserveUnknownColumns) {
       const preserved = removedCols.map((name) => prevCols[name]);
-      const preservedNames = new Set(removedCols);
-      const keptIndexes = prev.indexes.filter((idx) => idx.columns.some((c2) => preservedNames.has(c2)) && !currIdx[indexKey(idx, prev.table)]);
-      info2(`-- Keeping ${preserved.length} column(s) on "${curr.table}" no model declares: ${removedCols.join(", ")} \u2014 reconciled from the database, where an undeclared column is out of scope rather than removed`);
-      curr = { ...curr, columns: [...curr.columns, ...preserved], indexes: [...curr.indexes, ...keptIndexes] };
-      currCols = mapColumnsByName(curr.columns);
-      currIdx = mapIndexesByKey(curr.indexes, curr.table);
+      const keptIndexes = prev.indexes.filter((idx) => !currIdx[indexKey(idx, prev.table)]);
+      if (preserved.length > 0 || keptIndexes.length > 0) {
+        if (preserved.length > 0) {
+          info2(`-- Keeping ${preserved.length} column(s) on "${curr.table}" no model declares: ${removedCols.join(", ")} \u2014 reconciled from the database, where an undeclared column is out of scope rather than removed`);
+        }
+        if (keptIndexes.length > 0) {
+          info2(`-- Keeping ${keptIndexes.length} index(es) on "${curr.table}" no model declares: ${keptIndexes.map((index) => index.name).join(", ")} \u2014 reconciled from the database, where an undeclared index is out of scope rather than removed`);
+        }
+        curr = { ...curr, columns: [...curr.columns, ...preserved], indexes: [...curr.indexes, ...keptIndexes] };
+        currCols = mapColumnsByName(curr.columns);
+        currIdx = mapIndexesByKey(curr.indexes, curr.table);
+      }
       removedCols = [];
     }
     const modifiedCols = Object.keys(currCols).filter((n2) => prevCols[n2] && columnsAreDifferent(prevCols[n2], currCols[n2], dialect));
@@ -29452,7 +29458,7 @@ function generateDiffOperations(previous, next, opts = {}) {
 function generateDiffSql(previous, next, opts = {}) {
   return generateDiffOperations(previous, next, opts).statements;
 }
-var migrationCounter = 0, migrationsCreatedCount = 0, migrationsUpdatedCount = 0, useDeterministicNames = true, createdMigrationFiles, existingCorpusBodies = null, GENERATED_MARKER = "-- qb:generated";
+var migrationCounter = 0, migrationsCreatedCount = 0, migrationsUpdatedCount = 0, useDeterministicNames = true, createdMigrationFiles, existingCorpusBodies = null, GENERATED_MARKER = "-- qb:generated", MAX_BOUNDED_VARCHAR_LENGTH = 767;
 var init_migrations = __esm(() => {
   init_config();
   init_drivers();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "bun-query-builder",
   "type": "module",
-  "version": "0.2.13",
+  "version": "0.2.15",
   "description": "A simple yet performant query builder for TypeScript. Built with Bun.",
   "author": "Chris Breuer <chris@stacksjs.org>",
   "license": "MIT",