@dbcube/cli 5.2.1 → 5.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dbcube/cli",
-  "version": "5.2.1",
+  "version": "5.2.2",
   "main": "src/index.js",
   "scripts": {
     "dbcube": "node src/index.js"
@@ -28,11 +28,11 @@
     "access": "public"
   },
   "dependencies": {
-    "@dbcube/schema-builder": "^5.2.1",
+    "@dbcube/schema-builder": "^5.2.2",
     "@inquirer/prompts": "^8.5.2",
     "alwait": "^1.0.0",
     "chalk": "4.1.2",
-    "dbcube": "^5.2.1",
+    "dbcube": "^5.2.2",
     "dotenv": "^17.4.2",
     "fs-extra": "^11.3.5",
     "glob": "^13.0.6",

package/src/commands/run/pull.js

@@ -170,7 +170,11 @@ function renderCube(dbName, tableName, table) {
     out += `@meta({\n    name: "${tableName}";\n    description: "Imported from existing database by dbcube pull";\n});\n\n`;
     out += `@columns({\n`;
 
-    for (const col of table.columns) {
+    // uuid la genera y gestiona DBCube automáticamente: declararla en un
+    // .cube es un error de validación, así que se excluye del cube generado
+    const columns = table.columns.filter(c => c.name !== 'uuid' && c.name !== '_id');
+
+    for (const col of columns) {
         const cubeType = SQL_TYPE_MAP[col.dataType] || 'varchar';
         out += `    ${col.name}: {\n`;
         out += `        type: "${cubeType === 'enum' && !col.enumValues ? 'varchar' : cubeType}";\n`;

package/src/commands/run/table/alter.js

@@ -32,15 +32,25 @@ async function main() {
     const targetNames = targetFiles.map(f => path.basename(f));
     const configuredDatabases = await ConfigFileUtils.getConfiguredDatabases();
 
+    let totalErrors = 0;
     for (const config of configuredDatabases) {
       const schema = new Schema(config.name);
       try {
-        await schema.executeAlters(targetNames, { dryRun });
+        const result = await schema.executeAlters(targetNames, { dryRun });
+        totalErrors += result?.errors ?? 0;
       } catch (e) {
         if (!String(e.message).includes('no .alter.cube files')) throw e;
       }
     }
 
+    // Una migración que falló NO puede quedar registrada como aplicada:
+    // jamás se reintentaría y la DB quedaría desincronizada del historial
+    if (totalErrors > 0) {
+      console.error(`\n❌ ${totalErrors} error(s) applying migrations — batch NOT recorded.`);
+      console.error('   Fix the .alter.cube files and run table:alter again.\n');
+      process.exit(1);
+    }
+
     // Record history (skip on dry-run)
     if (!dryRun) {
       const batch = history.lastBatch + 1;