dbtasker 3.1.1 → 3.1.2

package/README.md

@@ -29,13 +29,13 @@ Property | Description
 | **host** | MySQL Host (e.g., `localhost`) |
 | **user** | Database Username |
 | **password** | Database Password |
-| **port** | Connection Port (Default: `3306`) |
+| **port** | Connection Port (e.g., `3306`) |
 | **drop database** | `Boolean`: If true, allows dropping databases. |
 | **drop table** | `Boolean`: If true, allows dropping tables. |
 | **drop column** | `Boolean`: If true, allows dropping columns. |
 | **do not touch** | `Array`: List of database names protected from deletion. |
 | **force delete column** | `Boolean`: If true, allow dropping column even if referanced by any other column. |
-| **force update column** | `Boolean`: It is set `default: false` If true, allow updating column even if referanced by any other column. This one is very powerful. If you turn it on it will remove any foreign key that is referanced to the column is being updated then update the column then add the foreign key to those column again and if the column is set to `UNIQUE` it will remove all the column value that are simillar. |
+| **force update column** | `Boolean`: It is set `default: false` If true, allow updating column even if referanced by any other column. This one is very powerful. If you turn it on it will remove any foreign key that is referanced to the column is being updated then update the column then add the foreign key to those column again and if the column is set to `UNIQUE` it will remove all the column value that are same. |
 
 ### Configuration Example
 `JavaScript`

package/altercolumn.js

@@ -310,7 +310,9 @@ async function alterColumnQuery(dbConfig, columndata, columnName, tableName, dat
                 }
                 if (hasdupes === true) {
                     if (forceupdatecolumn) {
+                        console.log(cstyler.bold.underline.yellow("********************************************************"));
                         console.log(cstyler.bold.underline.yellow("***Cleaning duplicate rows before adding UNIQUE index***"));
+                        console.log(cstyler.bold.underline.yellow("********************************************************"));
                         const cleancol = await fncs.cleanDuplicateRows(dbConfig, database, tableName, columnName);
                         if (cleancol === null) {
                             console.error("Having server connection problem cleaning duplicate rows from database");
@@ -404,6 +406,7 @@ async function addForeignKeyWithIndexQuery(config, databaseName, tableName, colu
 async function alterColumnIfNeeded(config, jsondata, forceupdatecolumn, separator) {
     try {
         console.log(cstyler.bold.yellow("Let's initiate Alter Column to table if needed..."));
+        console.log(cstyler.bold.blue("Please wait..."));
         let count = 0;
         for (const jsondb of Object.keys(jsondata)) {
             const loopdb = fncs.perseTableNameWithLoop(jsondb, separator);
@@ -550,10 +553,8 @@ async function alterColumnIfNeeded(config, jsondata, forceupdatecolumn, separato
                             }
                         }
                     } else {
-                        console.log(cstyler.blue("Database:"), cstyler.hex("#00d9ffff")(databaseName), cstyler.blue("Table:"), cstyler.hex("#00d9ffff")(tableName), cstyler.blue("Column Name:"), cstyler.hex("#00d9ffff")(jsoncolumn), cstyler.yellow("- Given column data and server data are same."));
                         // lets check if foreign key need update
                         if (fkdetails === false && !columndata.hasOwnProperty("foreign_key")) {
-                            console.log(cstyler.underline("Column do not have foreign key."));
                             continue;
                         } else if (fkdetails === false && columndata.hasOwnProperty("foreign_key")) {
                             console.log(cstyler.yellow("Foreign key need to be added for "), cstyler.blue("Database: "), cstyler.hex("#00d9ffff")(databaseName), cstyler.blue(" Table: "), cstyler.hex("#00d9ffff")(tableName), cstyler.blue(" Column Name: "), cstyler.hex("#00d9ffff")(jsoncolumn));
@@ -575,7 +576,6 @@ async function alterColumnIfNeeded(config, jsondata, forceupdatecolumn, separato
                                 fkdetails.column === fkdt.column &&
                                 fkdetails.deleteOption === fkdt.deleteOption &&
                                 (fkdt.updateOption === undefined || fkdetails.updateOption === fkdt.updateOption)) {
-                                console.log(cstyler.underline("Foreign key details are matched. No changes needed."))
                                 continue;
                             }
                             console.log(cstyler.blue("Database: "), cstyler.hex("#00d9ffff")(databaseName), cstyler.blue(" Table: "), cstyler.hex("#00d9ffff")(tableName), cstyler.blue(" Column Name: "), cstyler.hex("#00d9ffff")(jsoncolumn), cstyler.yellow("- Foreign key details are different, updating foreign key."));

package/dropcolumn.js

@@ -17,11 +17,7 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
         for (const jsDb of Object.keys(tableJson)) {
             const parsedDb = fncs.perseDatabaseNameWithLoop(jsDb, separator);
             if (!parsedDb) {
-                console.error(
-                    cstyler.bold.red("Cannot parse database name."),
-                    jsDb,
-                    parsedDb
-                );
+                console.error(cstyler.bold.red("Cannot parse database name."), jsDb, parsedDb);
                 return null;
             }
 
@@ -34,20 +30,13 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
 
                 const parsedTable = fncs.perseTableNameWithLoop(jsTable, separator);
                 if (!parsedTable) {
-                    console.error(
-                        cstyler.bold.red("Cannot parse table name."),
-                        jsTable
-                    );
+                    console.error(cstyler.bold.red("Cannot parse table name."), jsTable);
                     return null;
                 }
 
                 const tableName = parsedTable.loopname;
 
-                const existingColumns = await fncs.getColumnNames(
-                    config,
-                    databaseName,
-                    tableName
-                );
+                const existingColumns = await fncs.getColumnNames(config, databaseName, tableName);
 
                 if (!existingColumns) {
                     console.error(
@@ -60,14 +49,10 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
                 }
 
                 for (const column of existingColumns) {
-                    const definedInJson = Object.prototype.hasOwnProperty.call(
-                        tableDef,
-                        column
-                    );
-
-                    const isValidDefinition =
-                        definedInJson && fncs.isJsonObject(tableDef[column]);
+                    const definedInJson = Object.prototype.hasOwnProperty.call(tableDef, column);
 
+                    const isValidDefinition = definedInJson && fncs.isJsonObject(tableDef[column]);
+                    
                     if (isValidDefinition) continue;
 
                     console.log(
@@ -79,19 +64,10 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
                         cstyler.yellow(column)
                     );
 
-                    const referencingColumns =
-                        await fncs.findReferencingFromColumns(
-                            config,
-                            databaseName,
-                            tableName,
-                            column
-                        );
+                    const referencingColumns = await fncs.findReferencingFromColumns(config, databaseName, tableName, column);
 
                     if (referencingColumns === null) {
-                        console.error(
-                            cstyler.bold.red("Failed to resolve FK references for column:"),
-                            cstyler.hex("#00d9ffff")(column)
-                        );
+                        console.error(cstyler.bold.red("Failed to resolve FK references for column:"), cstyler.hex("#00d9ffff")(column));
                         return null;
                     }
 
@@ -108,12 +84,7 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
                         let allFkRemoved = true;
 
                         for (const ref of referencingColumns) {
-                            const removed = await fncs.removeForeignKeyFromColumn(
-                                config,
-                                ref.child_schema,
-                                ref.child_table,
-                                ref.child_columns[0]
-                            );
+                            const removed = await fncs.removeForeignKeyFromColumn(config, ref.child_schema, ref.child_table, ref.child_columns[0]);
 
                             if (removed === null) {
                                 console.error(
@@ -127,21 +98,12 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
                         }
 
                         if (!allFkRemoved) {
-                            console.error(
-                                cstyler.bold.red("Aborting column drop due to FK failures:"),
-                                cstyler.hex("#00d9ffff")(column)
-                            );
+                            console.error(cstyler.bold.red("Aborting column drop due to FK failures:"), cstyler.hex("#00d9ffff")(column));
                             continue;
                         }
                     }
 
-                    const dropped = await fncs.dropColumn(
-                        config,
-                        databaseName,
-                        tableName,
-                        column
-                    );
-
+                    const dropped = await fncs.dropColumn(config, databaseName, tableName, column);
                     if (dropped === null) {
                         console.error(
                             cstyler.bold.red("Failed to drop column:"),
@@ -151,17 +113,11 @@ async function dropcolumn(config, tableJson, forceDropColumn, separator = "_") {
                     }
                     console.log(cstyler.blue("Database: "), cstyler.hex("#00d9ffff")(databaseName), cstyler.blue(" Table: "), cstyler.hex("#00d9ffff")(tableName), cstyler.blue("Column: "), cstyler.hex("#00d9ffff")(column), cstyler.green("- dropped successfully."))
                     count += 1;
-                    console.log(
-                        cstyler.bold.green("Successfully dropped column:"),
-                        cstyler.hex("#00d9ffff")(column),
-                        cstyler.bold.green("from table:"),
-                        cstyler.hex("#00d9ffff")(tableName)
-                    );
                 }
             }
         }
         if (count > 0) {
-            console.log(cstyler.green("Successfully dropped ", count, " columns."));
+            console.log(cstyler.bold.green("Successfully dropped " + count + " columns."));
         } else {
             console.log("There is not column found to be dropped.");
         }

package/function.js

@@ -289,35 +289,33 @@ async function dropTable(config, databaseName, tableName) {
 async function dropColumn(config, databaseName, tableName, columnName) {
   let connection;
   try {
-    config.database = databaseName;
-    connection = await mysql.createConnection(config);
+    // Ensure the database is selected in the config
+    const dbConfig = { ...config, database: databaseName };
+    connection = await mysql.createConnection(dbConfig);
 
-    // 1️⃣ Check if column exists
+    // 1️⃣ Check if column exists and get its metadata
     const [columns] = await connection.query(
-      `SELECT COLUMN_NAME, COLUMN_KEY
-       FROM INFORMATION_SCHEMA.COLUMNS
+      `SELECT COLUMN_NAME, COLUMN_KEY, COLUMN_TYPE, EXTRA 
+       FROM INFORMATION_SCHEMA.COLUMNS 
        WHERE TABLE_SCHEMA = ? AND TABLE_NAME = ? AND COLUMN_NAME = ?`,
       [databaseName, tableName, columnName]
     );
 
-    // ❌ Column does not exist → return false
     if (columns.length === 0) {
-      console.log(
-        `Column '${columnName}' does not exist in ${databaseName}.${tableName}`
-      );
+      console.log(`Column '${columnName}' does not exist in ${tableName}.`);
       return false;
     }
 
-    const columnKey = columns[0].COLUMN_KEY;
+    const columnKey = columns[0].COLUMN_KEY; // "PRI", "UNI", or ""
+    const columnType = columns[0].COLUMN_TYPE; // e.g., "int(11)"
+    const isAutoIncrement = columns[0].EXTRA.toLowerCase().includes('auto_increment');
 
-    // 2️⃣ Drop foreign key constraints
+    // 2️⃣ Drop Foreign Key constraints first
     const [fkConstraints] = await connection.query(
       `SELECT CONSTRAINT_NAME
        FROM INFORMATION_SCHEMA.KEY_COLUMN_USAGE
-       WHERE TABLE_SCHEMA = ?
-         AND TABLE_NAME = ?
-         AND COLUMN_NAME = ?
-         AND REFERENCED_TABLE_NAME IS NOT NULL`,
+       WHERE TABLE_SCHEMA = ? AND TABLE_NAME = ? AND COLUMN_NAME = ?
+       AND REFERENCED_TABLE_NAME IS NOT NULL`,
       [databaseName, tableName, columnName]
     );
 
@@ -327,23 +325,28 @@ async function dropColumn(config, databaseName, tableName, columnName) {
       );
     }
 
-    // 3️⃣ Drop primary key if needed
-    if (columnKey === "PRI") {
+    // 3️⃣ Handle Primary Key and Auto-Increment Conflict
+    // If it's Auto-Increment, we MUST remove that attribute before dropping the key
+    if (isAutoIncrement) {
+      // Modifying without the 'AUTO_INCREMENT' keyword
       await connection.query(
-        `ALTER TABLE \`${tableName}\` DROP PRIMARY KEY`
+        `ALTER TABLE \`${tableName}\` MODIFY COLUMN \`${columnName}\` ${columnType}`
       );
     }
 
-    // 4️⃣ Drop column
+    // Now it is safe to drop the Primary Key constraint if it was one
+    if (columnKey === "PRI") {
+      await connection.query(`ALTER TABLE \`${tableName}\` DROP PRIMARY KEY`);
+    }
+
+    // 4️⃣ Final Step: Drop the column
     await connection.query(
       `ALTER TABLE \`${tableName}\` DROP COLUMN \`${columnName}\``
     );
 
-    console.log(
-      `Column '${columnName}' dropped successfully from ${databaseName}.${tableName}`
-    );
-
+    console.log(`Column '${columnName}' dropped successfully from ${tableName}`);
     return true;
+
   } catch (err) {
     console.error("Error dropping column:", err.message);
     return null;
@@ -1873,7 +1876,7 @@ async function checkDuplicates(config, databaseName, tableName, columnName) {
       ) AS dup_query`;
 
     const [rows] = await connection.execute(sql);
-    
+
     // Returns true if duplicates exist, false if not
     return rows[0].duplicateCount > 0;
 
@@ -1901,7 +1904,7 @@ async function cleanDuplicateRows(config, databaseName, tableName, columnName) {
       AND t1.id > t2.id`;
 
     const [result] = await connection.execute(sql);
-    
+
     // Returns the number of deleted rows
     return result.affectedRows;
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dbtasker",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "funding": {
     "type": "patreon",
     "url": "https://www.patreon.com/kormoi/gift"