plazbot-cli 0.4.1 → 0.5.0

package/dist/commands/agent/export.js

@@ -1,26 +1,43 @@
-import { Command } from 'commander';
+import { Command, Option } from 'commander';
 import { Agent } from 'plazbot';
 import { getStoredCredentials } from '../../utils/credentials.js';
-import { logger } from '../../utils/logger.js';
 import { addExamples } from '../../utils/help.js';
 import { createSpinner } from '../../utils/ui.js';
 import chalk from 'chalk';
 import fs from 'fs';
 import path from 'path';
-import inquirer from 'inquirer';
-// Campos validos segun el schema de agentValidation.ts (additionalProperties: false)
+import { jsonOption, isJsonMode, printJson, fail, CliError } from '../../utils/output.js';
+import { addYesOption, confirmAction } from '../../utils/confirm.js';
+function parseZone(value, flag) {
+    const zone = value.toUpperCase();
+    if (zone !== 'LA' && zone !== 'EU') {
+        throw new CliError(`${flag} must be LA or EU (got "${value}").`, 'invalid_zone');
+    }
+    return zone;
+}
+// Top-level fields of the agent schema the API validates on create/update
+// (Plazbot-Back-v2/Schemas/agent.config.schema.json, additionalProperties: false).
+// /api/agent/update replaces the whole document, so every field the agent can have must be
+// listed here or an update built from this config would wipe it. `version` is left out
+// (the API manages it).
 const VALID_AGENT_FIELDS = new Set([
     'name', 'description', 'prompt', 'zone', 'timezone', 'buffer',
     'color', 'question', 'enable', 'tags', 'showInChat',
     'instructions', 'person', 'fallbacks', 'rules',
     'examples', 'channels', 'services', 'actions', 'variables',
     'useToolCalling', 'customAIConfig', 'aiProviders',
-    'responseMode', 'batchWaitSeconds',
+    'responseMode', 'batchWaitSeconds', 'enableWebSearch', 'enableContingency',
+    'enableAudioInput', 'imagePrompt', 'voiceConfig', 'source',
     'enableWidget', 'enableWhatsappWidget', 'urlWhatsappWidget',
     'formWidget', 'darkWidget', 'nameWidget', 'initialShowWidget',
-    'fUseAutomationFlowWidget', 'iconWidget',
+    'fUseAutomationFlowWidget', 'iconWidget', 'colorWidget', 'positionWidget',
+    'typeWidget', 'formTypeWidget', 'formFieldsWidget', 'initialViewWidget',
+    'multianswerWidget', 'enablePoweredBy', 'advisorNameWidget', 'advisorNameModeWidget',
     'enableAvatarWidget', 'avatarGender', 'avatarVoiceId', 'avatarLanguage',
+    'avatarApiKey', 'avatarFaceId', 'avatarQuality', 'avatarBackground',
 ]);
+/** Fields that hold third-party credentials: export/cross-copy only include them with --ai. */
+export const SECRET_AGENT_FIELDS = ['aiProviders', 'avatarApiKey'];
 // Campos que el schema espera como string (no pueden ser null)
 const STRING_FIELDS = new Set([
     'name', 'description', 'prompt', 'zone', 'timezone', 'color',
@@ -31,11 +48,27 @@ const STRING_FIELDS = new Set([
 const ARRAY_FIELDS = new Set([
     'tags', 'examples', 'channels', 'services', 'actions', 'aiProviders', 'variables',
 ]);
+// Removes null/undefined recursively. Nested objects of the schema accept extra keys, so
+// everything else the agent has is kept (an update built from this config must not lose data).
+function dropNulls(value) {
+    if (Array.isArray(value))
+        return value.filter((v) => v !== null && v !== undefined).map(dropNulls);
+    if (value && typeof value === 'object') {
+        const out = {};
+        for (const [k, v] of Object.entries(value)) {
+            if (v !== null && v !== undefined)
+                out[k] = dropNulls(v);
+        }
+        return out;
+    }
+    return value;
+}
 function cleanRequiredFields(fields) {
     if (!fields || !Array.isArray(fields))
         return [];
     return fields.map((f) => {
-        const clean = {};
+        const clean = dropNulls(f);
+        delete clean.properties;
         if (f.name)
             clean.name = f.name;
         if (f.description)
@@ -81,7 +114,7 @@ function cleanServices(services) {
             clean.responseMessage = svc.responseMessage;
         if (svc.responseConditions && Array.isArray(svc.responseConditions)) {
             clean.responseConditions = svc.responseConditions.map((rc) => {
-                const c = { condition: rc.condition || '' };
+                const c = { ...dropNulls(rc), condition: rc.condition || '' };
                 if (typeof rc.message === 'string')
                     c.message = rc.message;
                 if (typeof rc.nextService === 'string')
@@ -103,6 +136,7 @@ function cleanActions(actions) {
             reference: act.reference || '',
             enabled: act.enabled ?? true,
             action: Array.isArray(act.action) ? act.action.map((a) => ({
+                ...dropNulls(a),
                 type: a.type || '',
                 value: a.value ?? '',
             })) : [],
@@ -114,6 +148,8 @@ function cleanActions(actions) {
             clean.responseMessage = act.responseMessage;
         if (typeof act.responseJson === 'boolean')
             clean.responseJson = act.responseJson;
+        if (typeof act.responseExact === 'boolean')
+            clean.responseExact = act.responseExact;
         return clean;
     });
 }
@@ -137,7 +173,7 @@ function cleanVariables(variables) {
         return clean;
     });
 }
-function cleanAgentForExport(agentData) {
+export function cleanAgentForExport(agentData) {
     const cleaned = {};
     for (const key of Object.keys(agentData)) {
         if (!VALID_AGENT_FIELDS.has(key))
@@ -207,17 +243,22 @@ export const exportCommand = new Command('export')
     .argument('<agentId>', 'Agent ID to export')
     .option('-w, --workspace <id>', 'Source workspace ID (overrides local config)')
     .option('-z, --zone <zone>', 'Zone LA or EU (overrides local config)')
-    .option('-o, --output <path>', 'Output file path (.json)')
-    .option('--no-channels', 'Exclude channels from export')
-    .option('--no-ai', 'Exclude custom AI configuration')
+    .option('-o, --output <path>', 'Write the JSON to this file instead of stdout')
+    .option('--channels', 'Include channels (workspace-specific)', false)
+    .option('--ai', 'Include the custom AI configuration and the avatar key (contains third-party API keys)', false)
+    // Compatibilidad: antes canales e IA se exportaban por defecto y se excluian con --no-*
+    .addOption(new Option('--no-channels').hideHelp())
+    .addOption(new Option('--no-ai').hideHelp())
+    .addOption(jsonOption('With -o, print {"output": "<path>"}; without -o the agent JSON is printed anyway'))
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, options) => {
     try {
         const credentials = await getStoredCredentials();
         const effectiveWorkspace = options.workspace || credentials.workspace;
-        const effectiveZone = (options.zone?.toUpperCase() === 'EU' ? 'EU' : options.zone?.toUpperCase() === 'LA' ? 'LA' : credentials.zone);
+        const effectiveZone = options.zone ? parseZone(options.zone, '--zone') : credentials.zone;
         if (options.workspace || options.zone) {
-            console.log(chalk.hex('#FFA726')(`\n  Support mode: workspace=${effectiveWorkspace} zone=${effectiveZone}`));
+            // stderr: stdout queda solo para el JSON
+            console.error(chalk.hex('#FFA726')(`\n  Support mode: workspace=${effectiveWorkspace} zone=${effectiveZone}`));
         }
         const agent = new Agent({
             workspaceId: effectiveWorkspace,
@@ -227,51 +268,72 @@ export const exportCommand = new Command('export')
         });
         const spinner = createSpinner('Fetching agent...');
         spinner.start();
-        const response = await agent.getAgentById({ id: agentId });
-        const agentData = response.agent || response.data || response;
-        spinner.succeed('Agent fetched');
+        let agentData;
+        try {
+            const response = await agent.getAgentById({ id: agentId });
+            agentData = response?.agent || response?.data || response;
+        }
+        catch (err) {
+            spinner.fail('Could not fetch the agent');
+            throw err;
+        }
+        if (!agentData || !agentData.name) {
+            spinner.fail('Agent not found');
+            throw new CliError(`Agent ${agentId} was not found in workspace ${effectiveWorkspace}.`, 'not_found', "Check the ID with 'plazbot agent list'.");
+        }
+        spinner.succeed(`Agent fetched: ${agentData.name}`);
         // Limpiar para exportacion
         const cleaned = cleanAgentForExport(agentData);
-        // Aplicar opciones de exclusion
+        // Canales e IA solo con --channels / --ai: la config de IA lleva los apiToken de los
+        // proveedores. Se omiten las claves (no se ponen en false) para que un
+        // export -> editar -> update no borre los canales ni la IA del agente.
         if (!options.channels) {
             delete cleaned.channels;
         }
         if (!options.ai) {
-            cleaned.customAIConfig = false;
-            delete cleaned.aiProviders;
+            delete cleaned.customAIConfig;
+            SECRET_AGENT_FIELDS.forEach((field) => delete cleaned[field]);
         }
         const jsonOutput = JSON.stringify(cleaned, null, 2);
         // Guardar en archivo o mostrar en consola
         if (options.output) {
             const outputPath = path.resolve(options.output);
-            fs.writeFileSync(outputPath, jsonOutput, 'utf-8');
+            fs.writeFileSync(outputPath, jsonOutput + '\n', 'utf-8');
+            if (isJsonMode()) {
+                printJson({ output: outputPath, id: agentId, name: cleaned.name ?? null });
+                return;
+            }
             console.log(chalk.hex('#4ade80')(`\n  Exported to: ${outputPath}`));
         }
         else {
-            console.log();
+            // stdout solo lleva el JSON, para poder redirigirlo (export > agent.json)
             console.log(jsonOutput);
+            if (isJsonMode())
+                return;
         }
-        console.log();
-        console.log(chalk.gray('  This JSON is compatible with:'));
-        console.log(chalk.gray('  - plazbot agent create <file.json>'));
-        console.log(chalk.gray('  - Importing JSON in the platform (AgentTraining)'));
-        console.log();
+        console.error();
+        console.error(chalk.gray('  This JSON is compatible with:'));
+        console.error(chalk.gray('  - plazbot agent create <file.json>'));
+        console.error(chalk.gray('  - plazbot agent update <agentId> <file.json>'));
+        console.error(chalk.gray('  - Importing JSON in the platform (AgentTraining)'));
+        if (!options.channels || !options.ai) {
+            console.error(chalk.gray(`  Not included: ${[!options.channels && 'channels (--channels)', !options.ai && 'AI configuration (--ai)'].filter(Boolean).join(', ')}`));
+        }
+        console.error();
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(exportCommand, [
     { description: 'Print the cleaned JSON to stdout',
-        command: 'plazbot agent export agt_AbcDef123' },
+        command: 'plazbot agent export age_AbcDef123' },
     { description: 'Write the cleaned JSON to a file',
-        command: 'plazbot agent export agt_AbcDef123 -o ./agent.json' },
-    { description: 'Export without channels nor AI credentials (portable JSON)',
-        command: 'plazbot agent export agt_AbcDef123 -o ./agent.json --no-channels --no-ai' },
+        command: 'plazbot agent export age_AbcDef123 -o ./agent.json' },
+    { description: 'Include channels and the AI configuration (contains provider API tokens)',
+        command: 'plazbot agent export age_AbcDef123 -o ./agent.json --channels --ai' },
     { description: 'Export from a different workspace (support mode)',
-        command: 'plazbot agent export agt_AbcDef123 -w wok_Other123 -z LA -o ./agent.json' },
+        command: 'plazbot agent export age_AbcDef123 -w wok_Other123 -z LA -o ./agent.json' },
 ]);
 export const crossCopyCommand = new Command('cross-copy')
     .description('Copy an agent from one workspace to another')
@@ -280,23 +342,29 @@ export const crossCopyCommand = new Command('cross-copy')
     .requiredOption('--to-workspace <id>', 'Target workspace ID')
     .option('--from-zone <zone>', 'Source workspace zone (LA or EU)', 'LA')
     .option('--to-zone <zone>', 'Target workspace zone (LA or EU)', 'LA')
-    .option('--no-channels', 'Do not copy channels (recommended)')
-    .option('--no-ai', 'Do not copy custom AI configuration')
-    .option('--dev', 'Use development environment', false)
-    .action(async (agentId, options) => {
+    .option('--channels', 'Also copy channels (usually wrong: channels belong to the source workspace)', false)
+    .option('--ai', 'Also copy the custom AI configuration and the avatar key (contains third-party API keys)', false)
+    .addOption(new Option('--no-channels').hideHelp())
+    .addOption(new Option('--no-ai').hideHelp())
+    .addOption(jsonOption('Print the result as JSON ({"id","sourceId","fromWorkspace","toWorkspace"})'))
+    .option('--dev', 'Use development environment', false);
+addYesOption(crossCopyCommand);
+crossCopyCommand.action(async (agentId, options) => {
     try {
         const credentials = await getStoredCredentials();
-        const fromZone = (options.fromZone.toUpperCase() === 'EU' ? 'EU' : 'LA');
-        const toZone = (options.toZone.toUpperCase() === 'EU' ? 'EU' : 'LA');
-        console.log();
-        console.log(chalk.hex('#4CAF50')('  ┌' + '─'.repeat(58) + '┐'));
-        console.log(chalk.hex('#4CAF50')('  │') + chalk.bold('  Copy agent between workspaces').padEnd(68) + chalk.hex('#4CAF50')('│'));
-        console.log(chalk.hex('#4CAF50')('  └' + '─'.repeat(58) + '┘'));
-        console.log();
-        console.log(chalk.gray(`  Source: ${options.fromWorkspace} (${fromZone})`));
-        console.log(chalk.gray(`  Target: ${options.toWorkspace} (${toZone})`));
-        console.log(chalk.gray(`  Agent:  ${agentId}`));
-        console.log();
+        // En --json todo lo informativo va a stderr
+        const out = (message = '') => (isJsonMode() ? console.error : console.log)(message);
+        const fromZone = parseZone(options.fromZone, '--from-zone');
+        const toZone = parseZone(options.toZone, '--to-zone');
+        out();
+        out(chalk.hex('#4CAF50')('  ┌' + '─'.repeat(58) + '┐'));
+        out(chalk.hex('#4CAF50')('  │') + chalk.bold('  Copy agent between workspaces').padEnd(68) + chalk.hex('#4CAF50')('│'));
+        out(chalk.hex('#4CAF50')('  └' + '─'.repeat(58) + '┘'));
+        out();
+        out(chalk.gray(`  Source: ${options.fromWorkspace} (${fromZone})`));
+        out(chalk.gray(`  Target: ${options.toWorkspace} (${toZone})`));
+        out(chalk.gray(`  Agent:  ${agentId}`));
+        out();
         // 1. Obtener agente del workspace origen
         const spinnerGet = createSpinner('Fetching agent from source workspace...');
         spinnerGet.start();
@@ -306,11 +374,18 @@ export const crossCopyCommand = new Command('cross-copy')
             zone: fromZone,
             ...(options.dev && { customUrl: 'http://localhost:5090' }),
         });
-        const response = await sourceAgent.getAgentById({ id: agentId });
-        const agentData = response.agent || response.data || response;
+        let agentData;
+        try {
+            const response = await sourceAgent.getAgentById({ id: agentId });
+            agentData = response?.agent || response?.data || response;
+        }
+        catch (err) {
+            spinnerGet.fail('Could not fetch the agent from the source workspace');
+            throw err;
+        }
         if (!agentData || !agentData.name) {
-            spinnerGet.fail('Agent not found in source workspace');
-            process.exit(1);
+            spinnerGet.fail('Agent not found in the source workspace');
+            throw new CliError(`Agent ${agentId} was not found in workspace ${options.fromWorkspace} (${fromZone}).`, 'not_found', 'Check --from-workspace and --from-zone.');
         }
         spinnerGet.succeed(`Agent fetched: ${agentData.name}`);
         // 2. Limpiar JSON
@@ -324,29 +399,24 @@ export const crossCopyCommand = new Command('cross-copy')
         // Limpiar AI config si se solicita
         if (!options.ai) {
             cleaned.customAIConfig = false;
-            delete cleaned.aiProviders;
+            SECRET_AGENT_FIELDS.forEach((field) => delete cleaned[field]);
         }
         // Desactivar widget (seguridad)
         cleaned.enableWidget = false;
         // 3. Mostrar resumen
-        console.log();
-        console.log(chalk.bold('  Configuration to copy:'));
-        console.log(chalk.gray(`  Name:        ${cleaned.name}`));
-        console.log(chalk.gray(`  Target zone: ${cleaned.zone}`));
-        console.log(chalk.gray(`  Channels:    ${cleaned.channels?.length || 0}`));
-        console.log(chalk.gray(`  Services:    ${cleaned.services?.length || 0}`));
-        console.log(chalk.gray(`  Actions:     ${cleaned.actions?.length || 0}`));
-        console.log(chalk.gray(`  Custom AI:   ${cleaned.customAIConfig ? 'Yes' : 'No'}`));
-        console.log();
-        const { confirm } = await inquirer.prompt([{
-                type: 'confirm',
-                name: 'confirm',
-                message: 'Create this agent in the target workspace?',
-                default: true,
-            }]);
-        if (!confirm) {
-            console.log(chalk.gray('  Operation cancelled.'));
-            return;
+        out();
+        out(chalk.bold('  Configuration to copy:'));
+        out(chalk.gray(`  Name:        ${cleaned.name}`));
+        out(chalk.gray(`  Target zone: ${cleaned.zone}`));
+        out(chalk.gray(`  Channels:    ${cleaned.channels?.length || 0}`));
+        out(chalk.gray(`  Services:    ${cleaned.services?.length || 0}`));
+        out(chalk.gray(`  Actions:     ${cleaned.actions?.length || 0}`));
+        out(chalk.gray(`  Custom AI:   ${cleaned.customAIConfig ? 'Yes' : 'No'}`));
+        out();
+        const ok = await confirmAction(`Create "${cleaned.name}" in workspace ${options.toWorkspace} (${toZone})?`, options);
+        if (!ok) {
+            out(chalk.gray('  Aborted.'));
+            process.exit(1);
         }
         // 4. Crear en workspace destino
         const spinnerCreate = createSpinner('Creating agent in target workspace...');
@@ -357,32 +427,53 @@ export const crossCopyCommand = new Command('cross-copy')
             zone: toZone,
             ...(options.dev && { customUrl: 'http://localhost:5090' }),
         });
-        const result = await targetAgent.addAgent(cleaned);
+        let result;
+        try {
+            result = await targetAgent.addAgent(cleaned);
+        }
+        catch (err) {
+            spinnerCreate.fail('Could not create the agent in the target workspace');
+            throw err;
+        }
         if (result?.success === false) {
-            spinnerCreate.fail(`Error: ${result.message || 'Could not create the agent'}`);
-            process.exit(1);
+            spinnerCreate.fail('Could not create the agent in the target workspace');
+            throw new CliError(result.message || 'The API rejected the agent configuration.', 'create_failed');
         }
         const newId = result?.agentId || result?.id || result?.data?.agentId;
         spinnerCreate.succeed('Agent created successfully');
-        console.log();
+        if (isJsonMode()) {
+            printJson({
+                id: newId ?? null,
+                sourceId: agentId,
+                name: cleaned.name ?? null,
+                fromWorkspace: options.fromWorkspace,
+                toWorkspace: options.toWorkspace,
+                fromZone,
+                toZone,
+            });
+            return;
+        }
+        out();
         if (newId) {
-            console.log(chalk.hex('#4ade80')(`  New ID: ${newId}`));
+            out(chalk.hex('#4ade80')(`  New ID: ${newId}`));
+        }
+        out(chalk.gray('  Widget disabled by default.'));
+        if (!options.channels) {
+            out(chalk.gray('  Channels were not copied: connect them in the target workspace.'));
         }
-        console.log(chalk.gray('  Widget disabled by default.'));
-        console.log(chalk.gray('  Channels must be configured manually in the target workspace.'));
-        console.log();
+        out();
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(crossCopyCommand, [
-    { description: 'Copy an agent between two LA workspaces (skip channels)',
-        command: 'plazbot agent cross-copy agt_AbcDef123 --from-workspace wok_Source123 --to-workspace wok_Target456' },
+    { description: 'Copy an agent between two LA workspaces (without channels nor AI config)',
+        command: 'plazbot agent cross-copy age_AbcDef123 --from-workspace wok_Source123 --to-workspace wok_Target456' },
     { description: 'Copy across zones (LA → EU)',
-        command: 'plazbot agent cross-copy agt_AbcDef123 --from-workspace wok_Source123 --from-zone LA --to-workspace wok_Target456 --to-zone EU' },
+        command: 'plazbot agent cross-copy age_AbcDef123 --from-workspace wok_Source123 --from-zone LA --to-workspace wok_Target456 --to-zone EU' },
+    { description: 'Copy without the confirmation prompt (scripts and CI)',
+        command: 'plazbot agent cross-copy age_AbcDef123 --from-workspace wok_Source123 --to-workspace wok_Target456 --yes --json' },
     { description: 'Copy preserving channels and AI providers',
-        command: 'plazbot agent cross-copy agt_AbcDef123 --from-workspace wok_Source123 --to-workspace wok_Target456 --channels --ai' },
+        command: 'plazbot agent cross-copy age_AbcDef123 --from-workspace wok_Source123 --to-workspace wok_Target456 --channels --ai' },
 ]);