@restforgejs/platform 5.0.1 → 5.0.8

package/generators/cli/processor/create.js

@@ -879,7 +879,7 @@ module.exports = {
             type: 'string',
             required: false,
             default: null,
-            description: 'Tipe database (postgres|mysql|oracle). Default: postgres'
+            description: 'Tipe database (postgres|mysql|oracle|sqlite). Default: postgres'
         },
         force: {
             type: 'boolean',

package/generators/lib/dbschema-kit/apply-executor.js

@@ -59,7 +59,21 @@ async function connectMysql(driver, config) {
     multipleStatements: false
   });
   return {
-    exec: (sql) => conn.query(sql),
+    exec: async (sql) => {
+      try {
+        return await conn.query(sql);
+      } catch (err) {
+        // Idempotensi migrate: MySQL tidak punya `CREATE INDEX IF NOT EXISTS`
+        // (lihat statement-modifier.js) sehingga re-run pada index yang sudah
+        // ada melempar ER_DUP_KEYNAME (errno 1061). Abaikan khusus untuk
+        // CREATE INDEX — setara dengan suppression ORA-00955 di jalur Oracle.
+        const isDupIndex = err && (err.errno === 1061 || err.code === 'ER_DUP_KEYNAME');
+        if (isDupIndex && /^\s*CREATE\s+(?:UNIQUE\s+)?INDEX\b/i.test(sql)) {
+          return [{ affectedRows: 0, suppressed: 'ER_DUP_KEYNAME' }];
+        }
+        throw err;
+      }
+    },
     close: () => conn.end()
   };
 }

package/generators/lib/dbschema-kit/dialect/mysql.js

@@ -82,6 +82,8 @@ function formatReferentialAction(action) {
 module.exports = {
   name: 'mysql',
   maxIdentifierLength: 64,
+  // Boolean disimpan sebagai VARCHAR(5) 'true'/'false'; bukan native.
+  nativeBoolean: false,
   mapType,
   formatDefault,
   quoteIdentifier,

package/generators/lib/dbschema-kit/dialect/oracle.js

@@ -87,6 +87,8 @@ module.exports = {
   // RESTForge target Oracle 19c+ (default COMPATIBLE >= 12.2). Override via
   // --max-name-length=30 if deploying to legacy database.
   maxIdentifierLength: 128,
+  // Boolean disimpan sebagai VARCHAR2(5) 'true'/'false'; bukan native.
+  nativeBoolean: false,
   mapType,
   formatDefault,
   quoteIdentifier,

package/generators/lib/dbschema-kit/dialect/postgres.js

@@ -81,6 +81,9 @@ function formatReferentialAction(action) {
 module.exports = {
   name: 'postgres',
   maxIdentifierLength: 63,
+  // PostgreSQL punya tipe BOOLEAN native; dialect lain menyimpan boolean
+  // sebagai VARCHAR 'true'/'false' dan butuh CHECK sebagai penanda boolean.
+  nativeBoolean: true,
   mapType,
   formatDefault,
   quoteIdentifier,

package/generators/lib/dbschema-kit/dialect/sqlite.js

@@ -85,6 +85,8 @@ module.exports = {
   name: 'sqlite',
   // SQLite has no hard identifier limit; cap at 63 to stay portable with PG/MySQL.
   maxIdentifierLength: 63,
+  // Boolean disimpan sebagai VARCHAR(5) 'true'/'false'; bukan native.
+  nativeBoolean: false,
   mapType,
   formatDefault,
   quoteIdentifier,

package/generators/lib/dbschema-kit/emitters/alter-table.js

@@ -54,6 +54,13 @@ function emitAddColumn(tableIR, fieldName, field, dialect) {
         parts.push(`DEFAULT ${dialect.formatDefault(field.default)}`);
     }
 
+    // Penanda boolean untuk dialect varchar (selaras dengan create-table emitter):
+    // CHECK inline pada kolom agar introspeksi dapat mengenalinya kembali sebagai
+    // boolean. PostgreSQL (nativeBoolean) memakai tipe BOOLEAN native, dikecualikan.
+    if (field.type === 'boolean' && dialect.nativeBoolean !== true) {
+        parts.push(`CHECK (${column} IN ('true', 'false'))`);
+    }
+
     if (dialect.name === 'oracle') {
         // Oracle membungkus column definition di dalam parentheses untuk ADD,
         // konvensi yang juga mendukung multi-column add bila diperlukan nanti.

package/generators/lib/dbschema-kit/emitters/create-table.js

@@ -44,9 +44,14 @@ function emitCreateTable(ir, dialect) {
     lines.push(emitForeignKey(tableName, relName, rel, dialect));
   }
 
-  const checkNames = resolveCheckNames(tableName, checks, dialect.maxIdentifierLength);
-  for (let i = 0; i < checks.length; i++) {
-    const check = checks[i];
+  // Gabungkan CHECK eksplisit (dari SDF) dengan CHECK boolean yang di-derive
+  // otomatis untuk dialect non-native-boolean (mysql/oracle/sqlite). CHECK ini
+  // menjadi penanda deterministik agar introspeksi dapat mengenali kembali kolom
+  // VARCHAR sebagai boolean (lihat database-introspector.getBooleanColumns).
+  const allChecks = (Array.isArray(checks) ? checks : []).concat(deriveBooleanChecks(fields, dialect));
+  const checkNames = resolveCheckNames(tableName, allChecks, dialect.maxIdentifierLength);
+  for (let i = 0; i < allChecks.length; i++) {
+    const check = allChecks[i];
     if (check._invalid) continue;
     lines.push(emitCheck(check, checkNames[i], dialect));
   }
@@ -54,6 +59,29 @@ function emitCreateTable(ir, dialect) {
   return `CREATE TABLE ${dialect.formatTableIdentifier(ir)} (\n${lines.join(',\n')}\n)`;
 }
 
+/**
+ * Derive CHECK constraint penanda boolean untuk dialect yang menyimpan boolean
+ * sebagai VARCHAR (mysql/oracle/sqlite). Bentuk: `col IN ('true','false')`.
+ *
+ * PostgreSQL (nativeBoolean === true) memakai tipe BOOLEAN native sehingga tidak
+ * memerlukan penanda dan dikecualikan. CHECK ini TIDAK ditulis user di SDF;
+ * kata kunci `boolean` di field sudah cukup, generator yang menyisipkannya.
+ *
+ * @param {Object} fields - Map fieldName -> field IR (punya .type)
+ * @param {Object} dialect - Dialect aktif
+ * @returns {Array<{field:string, op:string, value:string[]}>}
+ */
+function deriveBooleanChecks(fields, dialect) {
+  if (dialect.nativeBoolean === true) return [];
+  const derived = [];
+  for (const [fieldName, field] of Object.entries(fields)) {
+    if (field && field.type === 'boolean') {
+      derived.push({ field: fieldName, op: 'in', value: ['true', 'false'] });
+    }
+  }
+  return derived;
+}
+
 function emitColumn(fieldName, field, dialect) {
   const parts = [`${COLUMN_INDENT}${dialect.quoteIdentifier(fieldName)}`, dialect.mapType(field)];
 

package/generators/lib/dbschema-kit/statement-modifier.js

@@ -13,8 +13,12 @@ const RE_BEGIN = /^\s*BEGIN\b/i;
  * `--drop=false` (statement re-run idempotent).
  *
  * Rules per dialect:
- *   - postgres / mysql / sqlite: insert `IF NOT EXISTS ` after `CREATE TABLE `
- *     or `CREATE INDEX ` (also `CREATE UNIQUE INDEX `).
+ *   - postgres / sqlite: insert `IF NOT EXISTS ` after `CREATE TABLE ` or
+ *     `CREATE INDEX ` (also `CREATE UNIQUE INDEX `).
+ *   - mysql: insert `IF NOT EXISTS ` after `CREATE TABLE ` saja. MySQL TIDAK
+ *     mendukung `CREATE INDEX IF NOT EXISTS` (syntax error), jadi CREATE INDEX
+ *     dibiarkan apa adanya — valid pada first-run; re-run index yang sudah ada
+ *     akan gagal duplicate (keterbatasan MySQL, gunakan --drop untuk recreate).
  *   - oracle: wrap CREATE TABLE / CREATE INDEX in a PL/SQL block that suppresses
  *     SQLCODE -955 (ORA-00955: name is already used by an existing object).
  *   - DROP statements pass through unchanged (the emitter already uses IF EXISTS).
@@ -53,6 +57,12 @@ function applyIfNotExistsModifier(statement, dialect) {
     return statement.replace(RE_CREATE_TABLE, '$1CREATE TABLE IF NOT EXISTS ');
   }
 
+  // MySQL tidak mendukung `CREATE INDEX IF NOT EXISTS` (parser error). Biarkan
+  // statement CREATE INDEX apa adanya; valid pada first-run.
+  if (dialect === 'mysql') {
+    return statement;
+  }
+
   return statement.replace(RE_CREATE_INDEX, (match, leading, unique) => {
     const uniquePart = unique ? 'UNIQUE ' : '';
     return `${leading}CREATE ${uniquePart}INDEX IF NOT EXISTS `;