@aifabrix/builder 2.8.0 → 2.10.0

package/lib/external-system-download.js

@@ -0,0 +1,431 @@
+/**
+ * External System Download Module
+ *
+ * Downloads external systems from dataplane to local development structure.
+ * Supports downloading system configuration and datasources for local development.
+ *
+ * @fileoverview External system download functionality for AI Fabrix Builder
+ * @author AI Fabrix Team
+ * @version 2.0.0
+ */
+
+const fs = require('fs').promises;
+const path = require('path');
+const os = require('os');
+const yaml = require('js-yaml');
+const chalk = require('chalk');
+const { authenticatedApiCall } = require('./utils/api');
+const { getDeploymentAuth } = require('./utils/token-manager');
+const { getDataplaneUrl } = require('./datasource-deploy');
+const { getConfig } = require('./config');
+const { detectAppType } = require('./utils/paths');
+const logger = require('./utils/logger');
+
+/**
+ * Validates system type from downloaded application
+ * @param {Object} application - External system configuration
+ * @returns {string} System type (openapi, mcp, custom)
+ * @throws {Error} If system type is invalid
+ */
+function validateSystemType(application) {
+  if (!application || typeof application !== 'object') {
+    throw new Error('Application configuration is required');
+  }
+
+  const validTypes = ['openapi', 'mcp', 'custom'];
+  const systemType = application.type;
+
+  if (!systemType || !validTypes.includes(systemType)) {
+    throw new Error(`Invalid system type: ${systemType}. Must be one of: ${validTypes.join(', ')}`);
+  }
+
+  return systemType;
+}
+
+/**
+ * Validates downloaded data structure before writing files
+ * @param {Object} application - External system configuration
+ * @param {Array} dataSources - Array of datasource configurations
+ * @throws {Error} If validation fails
+ */
+function validateDownloadedData(application, dataSources) {
+  if (!application || typeof application !== 'object') {
+    throw new Error('Application configuration is required');
+  }
+
+  if (!application.key || typeof application.key !== 'string') {
+    throw new Error('Application key is required');
+  }
+
+  if (!Array.isArray(dataSources)) {
+    throw new Error('DataSources must be an array');
+  }
+
+  // Validate each datasource has required fields
+  for (const datasource of dataSources) {
+    if (!datasource.key || typeof datasource.key !== 'string') {
+      throw new Error('Datasource key is required for all datasources');
+    }
+    if (!datasource.systemKey || typeof datasource.systemKey !== 'string') {
+      throw new Error('Datasource systemKey is required for all datasources');
+    }
+    if (datasource.systemKey !== application.key) {
+      throw new Error(`Datasource systemKey (${datasource.systemKey}) does not match application key (${application.key})`);
+    }
+  }
+}
+
+/**
+ * Handles partial download errors gracefully
+ * @param {string} systemKey - System key
+ * @param {Object} systemData - System data that was successfully downloaded
+ * @param {Array<Error>} datasourceErrors - Array of errors from datasource downloads
+ * @throws {Error} Aggregated error message
+ */
+function handlePartialDownload(systemKey, systemData, datasourceErrors) {
+  if (datasourceErrors.length === 0) {
+    return;
+  }
+
+  const errorMessages = datasourceErrors.map(err => err.message).join('\n  - ');
+  throw new Error(
+    `Partial download completed for system '${systemKey}', but some datasources failed:\n  - ${errorMessages}\n\n` +
+    'System configuration was downloaded successfully. You may need to download datasources separately.'
+  );
+}
+
+/**
+ * Extracts environment variables from authentication configuration
+ * @param {Object} application - External system configuration
+ * @returns {string} Environment variables template content
+ */
+function generateEnvTemplate(application) {
+  const lines = ['# Environment variables for external system'];
+  lines.push(`# System: ${application.key || 'unknown'}`);
+  lines.push('');
+
+  if (!application.authentication) {
+    return lines.join('\n');
+  }
+
+  const auth = application.authentication;
+
+  // OAuth2 configuration
+  if (auth.type === 'oauth2' && auth.oauth2) {
+    if (auth.oauth2.clientId && auth.oauth2.clientId.includes('{{')) {
+      const key = auth.oauth2.clientId.replace(/[{}]/g, '').trim();
+      lines.push(`${key}=kv://secrets/${application.key}/client-id`);
+    }
+    if (auth.oauth2.clientSecret && auth.oauth2.clientSecret.includes('{{')) {
+      const key = auth.oauth2.clientSecret.replace(/[{}]/g, '').trim();
+      lines.push(`${key}=kv://secrets/${application.key}/client-secret`);
+    }
+  }
+
+  // API Key configuration
+  if (auth.type === 'apikey' && auth.apikey) {
+    if (auth.apikey.key && auth.apikey.key.includes('{{')) {
+      const key = auth.apikey.key.replace(/[{}]/g, '').trim();
+      lines.push(`${key}=kv://secrets/${application.key}/api-key`);
+    }
+  }
+
+  // Basic Auth configuration
+  if (auth.type === 'basic' && auth.basic) {
+    if (auth.basic.username && auth.basic.username.includes('{{')) {
+      const key = auth.basic.username.replace(/[{}]/g, '').trim();
+      lines.push(`${key}=kv://secrets/${application.key}/username`);
+    }
+    if (auth.basic.password && auth.basic.password.includes('{{')) {
+      const key = auth.basic.password.replace(/[{}]/g, '').trim();
+      lines.push(`${key}=kv://secrets/${application.key}/password`);
+    }
+  }
+
+  return lines.join('\n');
+}
+
+/**
+ * Generates variables.yaml with externalIntegration block
+ * @param {string} systemKey - System key
+ * @param {Object} application - External system configuration
+ * @param {Array} dataSources - Array of datasource configurations
+ * @returns {Object} Variables YAML object
+ */
+function generateVariablesYaml(systemKey, application, dataSources) {
+  const systemFileName = `${systemKey}-deploy.json`;
+  const datasourceFiles = dataSources.map(ds => {
+    // Extract entity key from datasource key or use entityKey
+    const entityKey = ds.entityKey || ds.key.split('-').pop();
+    return `${systemKey}-deploy-${entityKey}.json`;
+  });
+
+  return {
+    name: systemKey,
+    displayName: application.displayName || systemKey,
+    description: application.description || `External system integration for ${systemKey}`,
+    externalIntegration: {
+      schemaBasePath: './',
+      systems: [systemFileName],
+      dataSources: datasourceFiles,
+      autopublish: false,
+      version: application.version || '1.0.0'
+    }
+  };
+}
+
+/**
+ * Generates README.md with setup instructions
+ * @param {string} systemKey - System key
+ * @param {Object} application - External system configuration
+ * @param {Array} dataSources - Array of datasource configurations
+ * @returns {string} README.md content
+ */
+function generateReadme(systemKey, application, dataSources) {
+  const displayName = application.displayName || systemKey;
+  const description = application.description || `External system integration for ${systemKey}`;
+  const systemType = application.type || 'unknown';
+
+  const lines = [
+    `# ${displayName}`,
+    '',
+    description,
+    '',
+    '## System Information',
+    '',
+    `- **System Key**: \`${systemKey}\``,
+    `- **System Type**: \`${systemType}\``,
+    `- **Datasources**: ${dataSources.length}`,
+    '',
+    '## Files',
+    '',
+    '- `variables.yaml` - Application configuration with externalIntegration block',
+    `- \`${systemKey}-deploy.json\` - External system definition`
+  ];
+
+  dataSources.forEach(ds => {
+    const entityKey = ds.entityKey || ds.key.split('-').pop();
+    lines.push(`- \`${systemKey}-deploy-${entityKey}.json\` - Datasource: ${ds.displayName || ds.key}`);
+  });
+
+  lines.push(
+    '- `env.template` - Environment variables template',
+    '',
+    '## Setup Instructions',
+    '',
+    '1. Review and update configuration files as needed',
+    '2. Set up environment variables in `env.template`',
+    '3. Run unit tests: `aifabrix test ${systemKey}`',
+    '4. Run integration tests: `aifabrix test-integration ${systemKey}`',
+    '5. Deploy: `aifabrix deploy ${systemKey} --environment dev`',
+    '',
+    '## Testing',
+    '',
+    '### Unit Tests',
+    'Run local validation without API calls:',
+    '```bash',
+    `aifabrix test ${systemKey}`,
+    '```',
+    '',
+    '### Integration Tests',
+    'Run integration tests via dataplane:',
+    '```bash',
+    `aifabrix test-integration ${systemKey} --environment dev`,
+    '```',
+    '',
+    '## Deployment',
+    '',
+    'Deploy to dataplane via miso-controller:',
+    '```bash',
+    `aifabrix deploy ${systemKey} --environment dev`,
+    '```'
+  );
+
+  return lines.join('\n');
+}
+
+/**
+ * Downloads external system from dataplane to local development structure
+ * @async
+ * @function downloadExternalSystem
+ * @param {string} systemKey - System key or ID
+ * @param {Object} options - Download options
+ * @param {string} [options.environment] - Environment (dev, tst, pro)
+ * @param {string} [options.controller] - Controller URL
+ * @param {boolean} [options.dryRun] - Show what would be downloaded without actually downloading
+ * @returns {Promise<void>} Resolves when download completes
+ * @throws {Error} If download fails
+ */
+async function downloadExternalSystem(systemKey, options = {}) {
+  if (!systemKey || typeof systemKey !== 'string') {
+    throw new Error('System key is required and must be a string');
+  }
+
+  // Validate system key format (alphanumeric, hyphens, underscores)
+  if (!/^[a-z0-9-_]+$/.test(systemKey)) {
+    throw new Error('System key must contain only lowercase letters, numbers, hyphens, and underscores');
+  }
+
+  try {
+    logger.log(chalk.blue(`\n📥 Downloading external system: ${systemKey}`));
+
+    // Get authentication
+    const config = await getConfig();
+    const environment = options.environment || 'dev';
+    const controllerUrl = options.controller || config.deployment?.controllerUrl || 'http://localhost:3000';
+    const authConfig = await getDeploymentAuth(controllerUrl, environment, systemKey);
+
+    if (!authConfig.token && !authConfig.clientId) {
+      throw new Error('Authentication required. Run "aifabrix login" or "aifabrix app register" first.');
+    }
+
+    // Get dataplane URL from controller
+    logger.log(chalk.blue('🌐 Getting dataplane URL from controller...'));
+    const dataplaneUrl = await getDataplaneUrl(controllerUrl, systemKey, environment, authConfig);
+    logger.log(chalk.green(`✓ Dataplane URL: ${dataplaneUrl}`));
+
+    // Download system configuration
+    // Note: Verify this endpoint exists. Alternative: GET /api/v1/pipeline/{systemIdOrKey}
+    const downloadEndpoint = `${dataplaneUrl}/api/v1/external/systems/${systemKey}/config`;
+    logger.log(chalk.blue(`📡 Downloading from: ${downloadEndpoint}`));
+
+    if (options.dryRun) {
+      logger.log(chalk.yellow('🔍 Dry run mode - would download from:'));
+      logger.log(chalk.gray(`  ${downloadEndpoint}`));
+      logger.log(chalk.yellow('\nWould create:'));
+      logger.log(chalk.gray(`  integration/${systemKey}/`));
+      logger.log(chalk.gray(`  integration/${systemKey}/variables.yaml`));
+      logger.log(chalk.gray(`  integration/${systemKey}/${systemKey}-deploy.json`));
+      logger.log(chalk.gray(`  integration/${systemKey}/env.template`));
+      logger.log(chalk.gray(`  integration/${systemKey}/README.md`));
+      return;
+    }
+
+    const response = await authenticatedApiCall(
+      downloadEndpoint,
+      {
+        method: 'GET'
+      },
+      authConfig.token
+    );
+
+    if (!response.success || !response.data) {
+      throw new Error(`Failed to download system configuration: ${response.error || response.formattedError || 'Unknown error'}`);
+    }
+
+    const downloadData = response.data.data || response.data;
+    const application = downloadData.application;
+    const dataSources = downloadData.dataSources || [];
+
+    if (!application) {
+      throw new Error('Application configuration not found in download response');
+    }
+
+    // Validate downloaded data
+    logger.log(chalk.blue('🔍 Validating downloaded data...'));
+    validateDownloadedData(application, dataSources);
+    const systemType = validateSystemType(application);
+    logger.log(chalk.green(`✓ System type: ${systemType}`));
+    logger.log(chalk.green(`✓ Found ${dataSources.length} datasource(s)`));
+
+    // Create temporary folder for validation
+    const tempDir = path.join(os.tmpdir(), `aifabrix-download-${systemKey}-${Date.now()}`);
+    await fs.mkdir(tempDir, { recursive: true });
+
+    try {
+      // Generate files in temporary folder first
+      const systemFileName = `${systemKey}-deploy.json`;
+      const systemFilePath = path.join(tempDir, systemFileName);
+      await fs.writeFile(systemFilePath, JSON.stringify(application, null, 2), 'utf8');
+
+      // Generate datasource files
+      const datasourceErrors = [];
+      const datasourceFiles = [];
+      for (const datasource of dataSources) {
+        try {
+          const entityKey = datasource.entityKey || datasource.key.split('-').pop();
+          const datasourceFileName = `${systemKey}-deploy-${entityKey}.json`;
+          const datasourceFilePath = path.join(tempDir, datasourceFileName);
+          await fs.writeFile(datasourceFilePath, JSON.stringify(datasource, null, 2), 'utf8');
+          datasourceFiles.push(datasourceFilePath);
+        } catch (error) {
+          datasourceErrors.push(new Error(`Failed to write datasource ${datasource.key}: ${error.message}`));
+        }
+      }
+
+      // Handle partial downloads
+      if (datasourceErrors.length > 0) {
+        handlePartialDownload(systemKey, application, datasourceErrors);
+      }
+
+      // Generate variables.yaml
+      const variables = generateVariablesYaml(systemKey, application, dataSources);
+      const variablesPath = path.join(tempDir, 'variables.yaml');
+      await fs.writeFile(variablesPath, yaml.dump(variables, { indent: 2, lineWidth: 120, noRefs: true }), 'utf8');
+
+      // Generate env.template
+      const envTemplate = generateEnvTemplate(application);
+      const envTemplatePath = path.join(tempDir, 'env.template');
+      await fs.writeFile(envTemplatePath, envTemplate, 'utf8');
+
+      // Generate README.md
+      const readme = generateReadme(systemKey, application, dataSources);
+      const readmePath = path.join(tempDir, 'README.md');
+      await fs.writeFile(readmePath, readme, 'utf8');
+
+      // Determine final destination (integration folder)
+      const { appPath } = await detectAppType(systemKey);
+      const finalPath = appPath || path.join(process.cwd(), 'integration', systemKey);
+
+      // Create final directory
+      await fs.mkdir(finalPath, { recursive: true });
+
+      // Move files from temp to final location
+      logger.log(chalk.blue(`📁 Creating directory: ${finalPath}`));
+      const filesToMove = [
+        { from: systemFilePath, to: path.join(finalPath, systemFileName) },
+        { from: variablesPath, to: path.join(finalPath, 'variables.yaml') },
+        { from: envTemplatePath, to: path.join(finalPath, 'env.template') },
+        { from: readmePath, to: path.join(finalPath, 'README.md') }
+      ];
+
+      for (const dsFile of datasourceFiles) {
+        const fileName = path.basename(dsFile);
+        filesToMove.push({ from: dsFile, to: path.join(finalPath, fileName) });
+      }
+
+      for (const file of filesToMove) {
+        await fs.copyFile(file.from, file.to);
+        logger.log(chalk.green(`✓ Created: ${path.relative(process.cwd(), file.to)}`));
+      }
+
+      // Clean up temporary folder
+      await fs.rm(tempDir, { recursive: true, force: true });
+
+      logger.log(chalk.green('\n✅ External system downloaded successfully!'));
+      logger.log(chalk.blue(`Location: ${finalPath}`));
+      logger.log(chalk.blue(`System: ${systemKey}`));
+      logger.log(chalk.blue(`Datasources: ${dataSources.length}`));
+    } catch (error) {
+      // Clean up temporary folder on error
+      try {
+        await fs.rm(tempDir, { recursive: true, force: true });
+      } catch {
+        // Ignore cleanup errors
+      }
+      throw error;
+    }
+  } catch (error) {
+    throw new Error(`Failed to download external system: ${error.message}`);
+  }
+}
+
+module.exports = {
+  downloadExternalSystem,
+  validateSystemType,
+  validateDownloadedData,
+  generateVariablesYaml,
+  generateEnvTemplate,
+  generateReadme,
+  handlePartialDownload
+};

package/lib/external-system-generator.js

@@ -44,10 +44,10 @@ async function generateExternalSystemTemplate(appPath, systemKey, config) {
     };
 
     const rendered = template(context);
-    const schemasDir = path.join(appPath, 'schemas');
-    await fs.mkdir(schemasDir, { recursive: true });
 
-    const outputPath = path.join(schemasDir, `${systemKey}.json`);
+    // Generate in same folder as variables.yaml (new structure)
+    // Use naming: <app-name>-deploy.json
+    const outputPath = path.join(appPath, `${systemKey}-deploy.json`);
     await fs.writeFile(outputPath, rendered, 'utf8');
 
     return outputPath;
@@ -83,10 +83,10 @@ async function generateExternalDataSourceTemplate(appPath, datasourceKey, config
     };
 
     const rendered = template(context);
-    const schemasDir = path.join(appPath, 'schemas');
-    await fs.mkdir(schemasDir, { recursive: true });
 
-    const outputPath = path.join(schemasDir, `${datasourceKey}.json`);
+    // Generate in same folder as variables.yaml (new structure)
+    // Use naming: <app-name>-deploy-<datasource-key>.json
+    const outputPath = path.join(appPath, `${datasourceKey}-deploy.json`);
     await fs.writeFile(outputPath, rendered, 'utf8');
 
     return outputPath;
@@ -120,7 +120,8 @@ async function generateExternalSystemFiles(appPath, appName, config) {
 
     for (let i = 0; i < datasourceCount; i++) {
       const entityKey = `entity${i + 1}`;
-      const datasourceKey = `${systemKey}-${entityKey}`;
+      // For datasource key, use just the entity key (will be prefixed with app-name-deploy-)
+      const datasourceKey = entityKey;
       const resourceType = resourceTypes[i % resourceTypes.length];
 
       const datasourceConfig = {
@@ -132,7 +133,8 @@ async function generateExternalSystemFiles(appPath, appName, config) {
         datasourceDescription: `External datasource for ${entityKey} entity`
       };
 
-      const datasourcePath = await generateExternalDataSourceTemplate(appPath, datasourceKey, datasourceConfig);
+      // Generate with full naming: <app-name>-deploy-<entity-key>.json
+      const datasourcePath = await generateExternalDataSourceTemplate(appPath, `${systemKey}-deploy-${datasourceKey}`, datasourceConfig);
       datasourcePaths.push(datasourcePath);
       logger.log(chalk.green(`✓ Generated datasource: ${path.basename(datasourcePath)}`));
     }
@@ -165,9 +167,10 @@ async function updateVariablesYamlWithExternalIntegration(appPath, systemKey, da
     const variables = yaml.load(variablesContent);
 
     // Add externalIntegration block
+    // Files are in same folder, so schemaBasePath is './'
     variables.externalIntegration = {
-      schemaBasePath: './schemas',
-      systems: [`${systemKey}.json`],
+      schemaBasePath: './',
+      systems: [`${systemKey}-deploy.json`],
       dataSources: datasourcePaths.map(p => path.basename(p)),
       autopublish: true,
       version: '1.0.0'