@aifabrix/builder 2.44.3 → 2.44.4

package/lib/validation/validate-display.js

@@ -1,18 +1,30 @@
-const { formatBlockingError, formatSuccessLine, formatSuccessParagraph } = require('../utils/cli-test-layout-chalk');
+const {
+  headerKeyValue,
+  metadata,
+  formatDatasourceListRow,
+  formatBlockingError,
+  formatSuccessLine,
+  formatSuccessParagraph,
+  successGlyph,
+  failureGlyph
+} = require('../utils/cli-test-layout-chalk');
 /**
- * Validation Display Utilities
- *
- * Display functions for validation results output.
- *
- * @fileoverview Validation display utilities for AI Fabrix Builder
+ * @fileoverview Validation display for `aifabrix validate` (tty-summary: cli-layout.mdc, layout.md).
  * @author AI Fabrix Team
- * @version 2.0.0
  */
 
 const path = require('path');
 const chalk = require('chalk');
 const logger = require('../utils/logger');
 const { loadConfigFile } = require('../utils/config-format');
+const { flattenRootDimensionsForDisplay } = require('./dimension-display-helpers');
+const {
+  logSectionTitle,
+  logOkRowRest,
+  logWarnRow,
+  logDimLine,
+  logErrorDetail
+} = require('./validate-display-log-helpers');
 
 /**
  * Displays application validation results
@@ -24,26 +36,27 @@ function displayApplicationValidation(application) {
     return;
   }
 
-  logger.log(chalk.blue('\nApplication:'));
+  logSectionTitle('Application');
   if (application.valid) {
-    logger.log(chalk.green('  ✔ Application configuration is valid'));
+    logOkRowRest('Application configuration is valid');
   } else {
-    logger.log(chalk.red('  ✖ Application configuration has errors:'));
+    logger.log(`  ${failureGlyph()} ${chalk.red('Application configuration has errors:')}`);
     if (application.errors && application.errors.length > 0) {
       application.errors.forEach(error => {
-        logger.log(chalk.red(`    • ${error}`));
+        logErrorDetail(error);
       });
     }
   }
   if (application.warnings && application.warnings.length > 0) {
     application.warnings.forEach(warning => {
-      logger.log(chalk.yellow(`    ⚠ ${warning}`));
+      logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
     });
   }
 }
 
 /**
- * Extracts dimensions from a datasource file
+ * Extracts dimensions from a datasource file for CLI display.
+ * Local bindings become `metadata.<field>`; FK bindings become `fk:<fk>→<dimension>...` plus optional `(actor: ...)`.
  * @function extractDimensionsFromDatasource
  * @param {string} filePath - Path to datasource file
  * @returns {Object} Dimensions info { dimensions: Object, dimensionKeys: string[], hasDimensions: boolean }
@@ -52,15 +65,7 @@ function extractDimensionsFromDatasource(filePath) {
   try {
     const parsed = loadConfigFile(filePath);
 
-    const rootFlat = {};
-    const root = parsed.dimensions;
-    if (root && typeof root === 'object' && !Array.isArray(root)) {
-      for (const [dimKey, binding] of Object.entries(root)) {
-        if (binding && typeof binding === 'object' && typeof binding.field === 'string') {
-          rootFlat[dimKey] = `metadata.${binding.field}`;
-        }
-      }
-    }
+    const rootFlat = flattenRootDimensionsForDisplay(parsed.dimensions);
     const abacDimensions = parsed.abac?.dimensions || {};
     const allDimensions = { ...rootFlat, ...abacDimensions };
     const dimensionKeys = Object.keys(allDimensions);
@@ -85,19 +90,19 @@ function displayExternalFilesValidation(externalFiles) {
     return;
   }
 
-  logger.log(chalk.blue('\nExternal Integration Files:'));
+  logSectionTitle('External integration files');
   externalFiles.forEach(file => {
     if (file.valid) {
-      logger.log(chalk.green(`  ✔ ${file.file} (${file.type})`));
+      logger.log(formatDatasourceListRow('ok', file.file, file.type));
     } else {
-      logger.log(chalk.red(`  ✖ ${file.file} (${file.type}):`));
+      logger.log(formatDatasourceListRow('fail', file.file, file.type));
       file.errors.forEach(error => {
-        logger.log(chalk.red(`    • ${error}`));
+        logErrorDetail(error);
       });
     }
     if (file.warnings && file.warnings.length > 0) {
       file.warnings.forEach(warning => {
-        logger.log(chalk.yellow(`    ⚠ ${warning}`));
+        logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
       });
     }
   });
@@ -122,7 +127,7 @@ function displayDimensionsValidation(externalFiles) {
     return [];
   }
 
-  logger.log(chalk.blue('\nDimensions (ABAC):'));
+  logSectionTitle('Dimensions (ABAC)');
 
   const warnings = [];
   let anyDatasourceHasDimensions = false;
@@ -132,19 +137,19 @@ function displayDimensionsValidation(externalFiles) {
 
     if (dimensionsInfo.hasDimensions) {
       anyDatasourceHasDimensions = true;
-      logger.log(chalk.green(`  ✔ ${file.file}`));
+      logger.log(formatDatasourceListRow('ok', file.file, file.type));
       dimensionsInfo.dimensionKeys.forEach(key => {
         const mapping = dimensionsInfo.dimensions[key];
-        logger.log(chalk.gray(`      ${key} → ${mapping}`));
+        logDimLine(`${key} → ${mapping}`);
       });
     } else {
-      logger.log(chalk.yellow(`  ⚠ ${file.file} - no dimensions configured`));
+      logWarnRow(`${file.file} — no dimensions configured`);
       warnings.push(`${file.file} - no dimensions configured, ABAC filtering disabled`);
     }
   });
 
   if (!anyDatasourceHasDimensions) {
-    logger.log(chalk.yellow('    ⚠ No dimensions configured - ABAC filtering disabled for all datasources'));
+    logWarnRow('No dimensions configured — ABAC filtering disabled for all datasources');
   }
 
   return warnings;
@@ -160,18 +165,18 @@ function displayRbacValidation(rbac) {
     return;
   }
 
-  logger.log(chalk.blue('\nRBAC Configuration:'));
+  logSectionTitle('RBAC configuration');
   if (rbac.valid) {
-    logger.log(chalk.green('  ✔ RBAC configuration is valid'));
+    logOkRowRest('RBAC configuration is valid');
   } else {
-    logger.log(chalk.red('  ✖ RBAC configuration has errors:'));
+    logger.log(`  ${failureGlyph()} ${chalk.red('RBAC configuration has errors:')}`);
     rbac.errors.forEach(error => {
-      logger.log(chalk.red(`    • ${error}`));
+      logErrorDetail(error);
     });
   }
   if (rbac.warnings && rbac.warnings.length > 0) {
     rbac.warnings.forEach(warning => {
-      logger.log(chalk.yellow(`    ⚠ ${warning}`));
+      logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
     });
   }
 }
@@ -186,19 +191,20 @@ function displayFileValidation(result) {
     return;
   }
 
-  logger.log(chalk.blue(`\nFile: ${result.file}`));
-  logger.log(chalk.blue(`Type: ${result.type}`));
+  logSectionTitle('File');
+  logger.log(`  ${headerKeyValue('Path:', result.file)}`);
+  logger.log(`  ${headerKeyValue('Type:', String(result.type))}`);
   if (result.valid) {
-    logger.log(chalk.green('  ✔ File is valid'));
+    logOkRowRest('File is valid');
   } else {
-    logger.log(chalk.red('  ✖ File has errors:'));
+    logger.log(`  ${failureGlyph()} ${chalk.red('File has errors:')}`);
     result.errors.forEach(error => {
-      logger.log(chalk.red(`    • ${error}`));
+      logErrorDetail(error);
     });
   }
   if (result.warnings && result.warnings.length > 0) {
     result.warnings.forEach(warning => {
-      logger.log(chalk.yellow(`    ⚠ ${warning}`));
+      logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
     });
   }
 }
@@ -213,9 +219,9 @@ function displayAggregatedWarnings(warnings) {
     return;
   }
 
-  logger.log(chalk.yellow('\nWarnings:'));
+  logSectionTitle('Warnings');
   warnings.forEach(warning => {
-    logger.log(chalk.yellow(`  • ${warning}`));
+    logger.log(`  ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
   });
 }
 
@@ -226,18 +232,18 @@ function displayAggregatedWarnings(warnings) {
  * @returns {void}
  */
 function displayApplicationStep(application) {
-  logger.log(chalk.blue('\nApplication:'));
+  logSectionTitle('Application');
   if (application.valid) {
-    logger.log(chalk.green('  ✔ Application configuration is valid'));
+    logOkRowRest('Application configuration is valid');
   } else {
-    logger.log(chalk.red('  ✖ Application configuration has errors:'));
+    logger.log(`  ${failureGlyph()} ${chalk.red('Application configuration has errors:')}`);
     application.errors.forEach(error => {
-      logger.log(chalk.red(`    • ${error}`));
+      logErrorDetail(error);
     });
   }
   if (application.warnings && application.warnings.length > 0) {
     application.warnings.forEach(warning => {
-      logger.log(chalk.yellow(`    ⚠ ${warning}`));
+      logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
     });
   }
 }
@@ -255,19 +261,19 @@ function displayComponentsStep(components) {
     return;
   }
 
-  logger.log(chalk.blue('\nExternal Integration Files:'));
+  logSectionTitle('External integration files');
   if (hasFiles) {
     components.files.forEach(file => {
       if (file.valid) {
-        logger.log(chalk.green(`  ✔ ${file.file} (${file.type})`));
+        logger.log(formatDatasourceListRow('ok', file.file, file.type));
       } else {
-        logger.log(chalk.red(`  ✖ ${file.file} (${file.type})`));
+        logger.log(formatDatasourceListRow('fail', file.file, file.type));
       }
     });
   }
   if (hasErrors) {
     components.errors.forEach(error => {
-      logger.log(chalk.red(`  • ${error}`));
+      logger.log(formatBlockingError(error));
     });
   }
 }
@@ -283,18 +289,18 @@ function displayDimensionsStep(datasourceFiles) {
     return;
   }
 
-  logger.log(chalk.blue('\nDimensions (ABAC):'));
+  logSectionTitle('Dimensions (ABAC)');
   datasourceFiles.forEach(file => {
     try {
       const dimensionsInfo = extractDimensionsFromDatasource(file.path || file.file);
       if (dimensionsInfo.hasDimensions) {
-        logger.log(chalk.green(`  ✔ ${file.file}`));
+        logger.log(formatDatasourceListRow('ok', file.file, file.type));
         dimensionsInfo.dimensionKeys.forEach(key => {
           const mapping = dimensionsInfo.dimensions[key];
-          logger.log(chalk.gray(`      ${key} → ${mapping}`));
+          logDimLine(`${key} → ${mapping}`);
         });
       } else {
-        logger.log(chalk.yellow(`  ⚠ ${file.file} - no dimensions configured`));
+        logWarnRow(`${file.file} — no dimensions configured`);
       }
     } catch {
       // Skip if file path not available
@@ -310,32 +316,32 @@ function displayDimensionsStep(datasourceFiles) {
  * @returns {void}
  */
 function displayManifestStep(manifest, componentFiles) {
-  logger.log(chalk.blue('\nDeployment Manifest:'));
+  logSectionTitle('Deployment manifest');
   if (manifest.skipped) {
-    logger.log(chalk.yellow('  ⊘ Skipped (fix errors above first)'));
+    logger.log(`  ${chalk.gray('⏭')} ${chalk.white('Skipped (fix errors above first)')}`);
   } else if (manifest.valid) {
-    logger.log(chalk.green('  ✔ Full deployment manifest is valid'));
+    logOkRowRest('Full deployment manifest is valid');
     if (componentFiles) {
       const datasourceFiles = componentFiles.filter(f => f.type === 'datasource' || f.type === 'external-datasource');
-      logger.log(chalk.green('    ✔ System configuration valid'));
-      logger.log(chalk.green(`    ✔ ${datasourceFiles.length} datasource(s) valid`));
-      logger.log(chalk.green('    ✔ Schema validation passed'));
+      logger.log(`    ${successGlyph()} ${chalk.white('System configuration valid')}`);
+      logger.log(`    ${successGlyph()} ${chalk.white(`${datasourceFiles.length} datasource(s) valid`)}`);
+      logger.log(`    ${successGlyph()} ${chalk.white('Schema validation passed')}`);
     }
   } else {
-    logger.log(chalk.red('  ✖ Full deployment manifest validation failed:'));
+    logger.log(`  ${failureGlyph()} ${chalk.red('Full deployment manifest validation failed:')}`);
     const errs = manifest.errors && manifest.errors.length > 0 ? manifest.errors : [];
     if (errs.length > 0) {
       errs.forEach(error => {
         const msg = typeof error === 'string' ? error : String(error);
-        logger.log(chalk.red(`    • ${msg}`));
+        logErrorDetail(msg);
       });
     } else {
-      logger.log(chalk.red('    • No error details available (check schema and manifest structure).'));
+      logErrorDetail('No error details available (check schema and manifest structure).');
     }
   }
   if (manifest.warnings && manifest.warnings.length > 0) {
     manifest.warnings.forEach(warning => {
-      logger.log(chalk.yellow(`    ⚠ ${warning}`));
+      logger.log(`    ${chalk.yellow('⚠')} ${chalk.white(warning)}`);
     });
   }
 }
@@ -349,7 +355,7 @@ function displayStepByStepValidation(result) {
   if (result.valid) {
     logger.log(formatSuccessParagraph('Validation passed!'));
   } else {
-    logger.log(chalk.red('\n✖ Validation failed!'));
+    logger.log(`\n${formatBlockingError('Validation failed!')}`);
   }
 
   displayApplicationStep(result.steps.application);
@@ -375,7 +381,7 @@ function displayStepByStepValidation(result) {
 
   displayOverallStatus(result);
   if (result.appPath) {
-    logger.log(chalk.gray(`\n${path.resolve(result.appPath)}`));
+    logger.log(`\n${metadata(path.resolve(result.appPath))}`);
   }
 }
 
@@ -392,9 +398,9 @@ function displayBatchValidationResults(batchResult) {
 
   const results = batchResult.results;
   results.forEach(item => {
-    logger.log(chalk.blue(`\n--- ${item.appName} ---`));
+    logger.log(`\n${metadata('──')} ${chalk.white.bold(item.appName)} ${metadata('──')}`);
     if (item.error) {
-      logger.log(chalk.red(`  ✖ ${item.error}`));
+      logger.log(`  ${failureGlyph()} ${chalk.red(item.error)}`);
     } else if (item.result) {
       displayValidationResults(item.result);
     }
@@ -402,13 +408,13 @@ function displayBatchValidationResults(batchResult) {
 
   const passed = results.filter(r => r.result && r.result.valid).length;
   const failed = results.length - passed;
-  logger.log(chalk.blue('\n--- Summary ---'));
+  logSectionTitle('Summary');
   if (failed === 0) {
-    logger.log(formatSuccessLine(`${passed} passed, 0 failed`));
-    logger.log(chalk.green('Overall: Passed'));
+    logger.log(`  ${formatSuccessLine(`${passed} passed, 0 failed`)}`);
+    logger.log(`  ${headerKeyValue('Overall:', chalk.green('Passed'))}`);
   } else {
-    logger.log(formatBlockingError(`${passed} passed, ${failed} failed`));
-    logger.log(chalk.red('Overall: Failed'));
+    logger.log(`  ${formatBlockingError(`${passed} passed, ${failed} failed`)}`);
+    logger.log(`  ${headerKeyValue('Overall:', chalk.red('Failed'))}`);
   }
 }
 
@@ -427,12 +433,13 @@ function displayOverallStatus(result) {
     }
   }
   const hasWarnings = result.warnings && result.warnings.length > 0;
+  logger.log('');
   if (hasErrors) {
-    logger.log(chalk.red('\nOverall: Failed'));
+    logger.log(headerKeyValue('Overall:', chalk.red('Failed')));
   } else if (hasWarnings) {
-    logger.log(chalk.yellow('\nOverall: Passed with warnings'));
+    logger.log(headerKeyValue('Overall:', chalk.yellow('Passed with warnings')));
   } else {
-    logger.log(chalk.green('\nOverall: Passed'));
+    logger.log(headerKeyValue('Overall:', chalk.green('Passed')));
   }
 }
 
@@ -453,7 +460,7 @@ function displayValidationResults(result) {
   if (result.valid) {
     logger.log(formatSuccessParagraph('Validation passed!'));
   } else {
-    logger.log(chalk.red('\n✖ Validation failed!'));
+    logger.log(`\n${formatBlockingError('Validation failed!')}`);
   }
 
   displayApplicationValidation(result.application);
@@ -471,10 +478,9 @@ function displayValidationResults(result) {
 
   displayOverallStatus(result);
   if (result.appPath) {
-    logger.log(chalk.gray(`\n${path.resolve(result.appPath)}`));
+    logger.log(`\n${metadata(path.resolve(result.appPath))}`);
   }
 }
-
 module.exports = {
   displayValidationResults,
   displayBatchValidationResults,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aifabrix/builder",
-  "version": "2.44.3",
+  "version": "2.44.4",
   "description": "AI Fabrix Local Fabric & Deployment SDK",
   "main": "lib/index.js",
   "bin": {

package/templates/external-system/README.md.hbs

@@ -144,8 +144,7 @@ Options:
   -e, --env <env>                    Environment: dev, tst, or pro
   -v, --verbose                      Show detailed step output and poll progress
   --debug                            Include debug output and write log to integration/{{appName}}/logs/
-  --test-crud                        Enable CRUD lifecycle test (body testCrud: true)
-  --record-id <id>                   Record ID for test (body recordId)
+  --no-run-scenarios                 Skip expanding testPayload.scenarios in capacity step
   --no-cleanup                       Disable cleanup after test (body cleanup: false)
   --primary-key-value <value|@path>  Primary key value or path to JSON file (e.g. @pk.json) for body primaryKeyValue
   --no-async                         Use sync mode (no polling); single POST, no asyncRun