befly 3.15.19 → 3.15.20

package/dist/befly.js

@@ -10227,6 +10227,8 @@ class SyncTable {
         throw new Error("\u540C\u6B65\u8868\uFF1A\u8BF7\u4F20\u5165\u591A\u4E2A\u8868\u5B9A\u4E49\u7EC4\u6210\u7684\u6570\u7EC4");
       }
       await SyncTable.ensureDbVersion(this.db);
+      const tableTasks = [];
+      const incompatibleTypeChanges = [];
       for (const item of items) {
         if (!item || item.type !== "table") {
           continue;
@@ -10237,9 +10239,98 @@ class SyncTable {
           SyncTable.normalizeFieldDefinitionInPlace(fieldDef);
         }
         const existsTable = await SyncTable.tableExists(this.db, this.dbName, tableName);
+        const task = {
+          item,
+          tableName,
+          tableFields,
+          existsTable,
+          existingColumns: null,
+          existingIndexes: null,
+          plan: null,
+          planSummary: null,
+          planDetails: null
+        };
+        if (existsTable) {
+          const existingColumns = await SyncTable.getTableColumns(this.db, this.dbName, tableName);
+          const existingIndexes = await SyncTable.getTableIndexes(this.db, this.dbName, tableName);
+          const changes = SyncTable.collectIncompatibleTypeChanges(tableName, existingColumns, tableFields);
+          for (const change of changes) {
+            incompatibleTypeChanges.push(change);
+          }
+          task.existingColumns = existingColumns;
+          task.existingIndexes = existingIndexes;
+        }
+        tableTasks.push(task);
+      }
+      if (incompatibleTypeChanges.length > 0) {
+        const lines = [];
+        for (const change of incompatibleTypeChanges) {
+          lines.push(`- ${change.tableName}.${change.dbFieldName}: ${change.currentType} -> ${change.expectedType}`);
+        }
+        const msgLines = [];
+        msgLines.push("\u7981\u6B62\u5B57\u6BB5\u7C7B\u578B\u53D8\u66F4\uFF08\u68C0\u6D4B\u5230\u4E0D\u517C\u5BB9/\u6536\u7F29\u53D8\u66F4\uFF09:");
+        for (const line of lines) {
+          msgLines.push(line);
+        }
+        msgLines.push("\u8BF4\u660E: \u4EC5\u5141\u8BB8\u540C\u7C7B\u578B\u7684\u5BBD\u5316\u53D8\u66F4\uFF08\u5982 TINYINT->SMALLINT->INT->BIGINT\uFF09\uFF0C\u4EE5\u53CA\u90E8\u5206\u517C\u5BB9\u53D8\u66F4\uFF08\u5982 VARCHAR->TEXT\u3001CHAR/VARCHAR \u4E92\u8F6C\u3001float->double\uFF09\u3002");
+        msgLines.push("\u63D0\u793A: \u82E5\u786E\u9700\u6536\u7F29\uFF0C\u8BF7\u5148\u624B\u5DE5\u8FC1\u79FB/\u6E05\u6D17\u6570\u636E\u540E\u518D\u6267\u884C\u540C\u6B65\u3002");
+        throw new Error(msgLines.join(`
+`));
+      }
+      for (const task of tableTasks) {
+        if (!task.existsTable) {
+          continue;
+        }
+        if (!task.existingColumns || !task.existingIndexes) {
+          throw new Error(`\u540C\u6B65\u8868\uFF1A\u5185\u90E8\u9519\u8BEF\uFF1A\u9884\u68C0\u9636\u6BB5\u7F3A\u5931\u8868\u5143\u4FE1\u606F\uFF08\u8868=${task.tableName}\uFF09`);
+        }
+        const built = SyncTable.buildTablePlan({
+          tableName: task.tableName,
+          fields: task.tableFields,
+          existingColumns: task.existingColumns,
+          existingIndexes: task.existingIndexes
+        });
+        task.plan = built.plan;
+        task.planSummary = built.summary;
+        task.planDetails = built.details;
+      }
+      for (const task of tableTasks) {
+        const item = task.item;
+        const tableName = task.tableName;
+        const tableFields = task.tableFields;
+        const existsTable = task.existsTable;
         try {
           if (existsTable) {
-            await SyncTable.modifyTable(this.db, this.dbName, tableName, tableFields);
+            const plan = task.plan;
+            const summary = task.planSummary;
+            const details = task.planDetails;
+            if (plan && plan.changed && summary && details) {
+              const msg = `[\u8868 ${tableName}] \u53D8\u66F4\u6C47\u603B\uFF0C\u65B0\u589E\u5B57\u6BB5=${summary.addedBusiness}\uFF0C\u65B0\u589E\u7CFB\u7EDF\u5B57\u6BB5=${summary.addedSystem}\uFF0C\u4FEE\u6539\u5B57\u6BB5=${summary.modified}\uFF0C\u7D22\u5F15\u53D8\u66F4=${summary.indexChanges}`;
+              const detailLines = details.fieldChanges.flatMap((d) => d.changes.map((change) => {
+                const current = String(change.current ?? "");
+                const expected = String(change.expected ?? "");
+                return `- ${d.dbFieldName}.${change.type}: ${current} -> ${expected}`;
+              }));
+              const indexLines = details.indexChanges.map((change) => {
+                const indexLabel = `idx_${change.fieldName}` === change.indexName ? change.indexName : change.indexName;
+                if (change.action === "create") {
+                  return `- index.${indexLabel}: \u65E0 -> ${change.indexName}(${change.fieldName})`;
+                }
+                return `- index.${indexLabel}: ${change.indexName}(${change.fieldName}) -> \u65E0`;
+              });
+              const allLines = [];
+              for (const line of detailLines) {
+                allLines.push(line);
+              }
+              for (const line of indexLines) {
+                allLines.push(line);
+              }
+              Logger.debug(msg);
+              for (const line of allLines) {
+                Logger.debug(`[\u8868 ${tableName}] \u53D8\u66F4\u660E\u7EC6 ${line}`);
+              }
+              await SyncTable.applyTablePlan(this.db, tableName, plan);
+            }
           } else {
             await SyncTable.createTable(this.db, tableName, tableFields);
           }
@@ -10594,6 +10685,32 @@ class SyncTable {
       throw new Error([`\u7981\u6B62\u5B57\u6BB5\u7C7B\u578B\u53D8\u66F4: ${tableName}.${dbFieldName}`, `\u5F53\u524D\u7C7B\u578B: ${typeChange.current}`, `\u76EE\u6807\u7C7B\u578B: ${typeChange.expected}`, "\u8BF4\u660E: \u4EC5\u5141\u8BB8\u5BBD\u5316\u578B\u53D8\u66F4\uFF08\u5982 INT->BIGINT, VARCHAR->TEXT\uFF09\uFF0C\u4EE5\u53CA CHAR/VARCHAR \u4E92\u8F6C\uFF1BDATETIME \u4E0E BIGINT \u4E0D\u5141\u8BB8\u4E92\u8F6C\uFF08\u9700\u8981\u624B\u52A8\u8FC1\u79FB\u6570\u636E\uFF09"].join(`
 `));
   }
+  static collectIncompatibleTypeChanges(tableName, existingColumns, fields) {
+    const out = [];
+    for (const [fieldKey, fieldDef] of Object.entries(fields)) {
+      const dbFieldName = snakeCase(fieldKey);
+      const existing = existingColumns[dbFieldName];
+      if (!existing)
+        continue;
+      const comparison = SyncTable.compareFieldDefinition(existing, fieldDef);
+      const typeChange = comparison.find((c) => c.type === "datatype");
+      if (!typeChange)
+        continue;
+      const currentType = String(typeChange.current || "").toLowerCase();
+      const expectedType = String(typeChange.expected || "").toLowerCase();
+      const currentBase = currentType.replace(/\s*unsigned/gi, "").replace(/\([^)]*\)/g, "").trim();
+      const expectedBase = expectedType.replace(/\s*unsigned/gi, "").replace(/\([^)]*\)/g, "").trim();
+      if (currentBase !== expectedBase && !SyncTable.isCompatibleTypeChange(currentType, expectedType)) {
+        out.push({
+          tableName,
+          dbFieldName,
+          currentType: String(typeChange.current ?? ""),
+          expectedType: String(typeChange.expected ?? "")
+        });
+      }
+    }
+    return out;
+  }
   static truncateForLog(input, maxLen) {
     const s = String(input);
     if (maxLen <= 0)