agrs-sequelize-sdk 1.3.12 → 1.3.13

package/migrations/change-adset-name-to-text.js

@@ -0,0 +1,79 @@
+"use strict";
+
+module.exports = {
+  up: async (queryInterface, Sequelize) => {
+    // Change AdSetName column from VARCHAR(255) to TEXT
+    // This is a SAFE operation - PostgreSQL can convert VARCHAR to TEXT without data loss
+    // All existing data will be preserved
+
+    // First, check if column exists and get its current type
+    const [columns] = await queryInterface.sequelize.query(`
+      SELECT column_name, data_type, character_maximum_length
+      FROM information_schema.columns
+      WHERE table_name = 'AdSet' AND column_name = 'AdSetName'
+    `);
+
+    if (columns.length === 0) {
+      console.log("⚠️  AdSetName column not found, skipping migration");
+      return;
+    }
+
+    const currentType = columns[0].data_type;
+    const currentMaxLength = columns[0].character_maximum_length;
+
+    console.log(
+      `Current AdSetName type: ${currentType}(${
+        currentMaxLength || "unlimited"
+      })`
+    );
+
+    // Only change if it's currently VARCHAR with length limit
+    if (currentType === "character varying" && currentMaxLength) {
+      console.log("Converting AdSetName from VARCHAR to TEXT...");
+
+      // Use direct SQL for more control
+      await queryInterface.sequelize.query(`
+        ALTER TABLE "AdSet" 
+        ALTER COLUMN "AdSetName" TYPE TEXT
+        USING "AdSetName"::TEXT
+      `);
+
+      console.log("✅ AdSetName successfully converted to TEXT");
+    } else if (currentType === "text") {
+      console.log("✅ AdSetName is already TEXT, no change needed");
+    } else {
+      console.log(`⚠️  AdSetName is ${currentType}, not changing`);
+    }
+  },
+
+  down: async (queryInterface, Sequelize) => {
+    // Revert back to VARCHAR(255)
+    // WARNING: This might truncate existing long values if any exist
+    console.log(
+      "⚠️  Reverting AdSetName to VARCHAR(255) - this may truncate long values!"
+    );
+
+    // Check for any values longer than 255 characters
+    const [longValues] = await queryInterface.sequelize.query(`
+      SELECT COUNT(*) as count
+      FROM "AdSet"
+      WHERE LENGTH("AdSetName") > 255
+    `);
+
+    if (longValues[0].count > 0) {
+      console.log(
+        `⚠️  WARNING: ${longValues[0].count} rows have AdSetName longer than 255 characters!`
+      );
+      console.log(
+        "⚠️  These will be truncated if you proceed with the down migration."
+      );
+    }
+
+    await queryInterface.changeColumn("AdSet", "AdSetName", {
+      type: Sequelize.STRING(255),
+      allowNull: true,
+    });
+
+    console.log("✅ AdSetName reverted to VARCHAR(255)");
+  },
+};

package/models/AdSet.js

@@ -151,7 +151,7 @@ module.exports = (sequelize, DataTypes) => {
         primaryKey: true,
       },
       AdSetName: {
-        type: DataTypes.STRING,
+        type: DataTypes.TEXT,
         allowNull: true,
       },
       Status: {
@@ -229,7 +229,7 @@ module.exports = (sequelize, DataTypes) => {
       target_roas: {
         type: DataTypes.FLOAT,
         allowNull: true,
-        comment: 'Target ROAS as decimal (0.9 = 90%)',
+        comment: "Target ROAS as decimal (0.9 = 90%)",
       },
       waiting: {
         type: DataTypes.BOOLEAN,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agrs-sequelize-sdk",
-  "version": "1.3.12",
+  "version": "1.3.13",
   "main": "index.js",
   "scripts": {
     "start": "node index.js",

package/services/sequelizeService.js

@@ -42,9 +42,56 @@ async function syncDatabase() {
     console.log("==============================\n");
 
     // Sync with alter: true to update schema
+    // NOTE: This is SAFE for VARCHAR->TEXT conversions (expanding type, no data loss)
+    // Sequelize will use ALTER COLUMN TYPE which preserves all existing data
+
+    // Safety check: Count existing AdSets before sync
+    let adSetCountBefore = null;
+    try {
+      const [result] = await db.sequelize.query(
+        'SELECT COUNT(*) as count FROM "AdSet"',
+        { type: db.sequelize.QueryTypes.SELECT }
+      );
+      adSetCountBefore = result.count;
+      console.log(`📊 Current AdSet count in database: ${adSetCountBefore}`);
+    } catch (error) {
+      console.log("⚠️  Could not count AdSets (table might not exist yet)");
+    }
+
     console.log("Syncing database schema...");
+    console.log(
+      "⚠️  Using alter: true - this will update column types to match models"
+    );
+    console.log(
+      "   Safe operations: VARCHAR->TEXT (expanding), adding columns"
+    );
+    console.log(
+      "   Unsafe operations: TEXT->VARCHAR (truncating), dropping columns"
+    );
+
     await db.sequelize.sync({ alter: true });
 
+    // Safety check: Verify AdSet count after sync
+    if (adSetCountBefore !== null) {
+      try {
+        const [result] = await db.sequelize.query(
+          'SELECT COUNT(*) as count FROM "AdSet"',
+          { type: db.sequelize.QueryTypes.SELECT }
+        );
+        const adSetCountAfter = result.count;
+        console.log(`📊 AdSet count after sync: ${adSetCountAfter}`);
+        if (adSetCountBefore !== adSetCountAfter) {
+          console.error(
+            `❌ WARNING: AdSet count changed from ${adSetCountBefore} to ${adSetCountAfter}!`
+          );
+        } else {
+          console.log("✅ AdSet count unchanged - no data loss confirmed");
+        }
+      } catch (error) {
+        console.log("⚠️  Could not verify AdSet count after sync");
+      }
+    }
+
     console.log("✅ Database synchronized successfully!");
     console.log("Schema has been updated to match model definitions.");