plazbot-cli 0.4.1 → 0.5.0

package/dist/commands/agent/set.js

@@ -3,130 +3,278 @@ import { Agent } from 'plazbot';
 import inquirer from 'inquirer';
 import { getStoredCredentials } from '../../utils/credentials.js';
 import { logger } from '../../utils/logger.js';
-import { addExamples } from '../../utils/help.js';
+import { addExamples, addNotes } from '../../utils/help.js';
 import { createSpinner, section } from '../../utils/ui.js';
+import { jsonOption, isJsonMode, printJson, fail, CliError } from '../../utils/output.js';
+import { patchAgent } from '../../utils/agent-patch.js';
+// Valores que guarda la plataforma (se mantienen tal cual) con su etiqueta en ingles.
+// Los flags aceptan tambien valores libres.
+const TONES = [
+    { name: 'Professional', value: 'profesional' },
+    { name: 'Friendly', value: 'amigable' },
+    { name: 'Formal', value: 'formal' },
+    { name: 'Casual', value: 'casual' },
+    { name: 'Technical', value: 'tecnico' },
+    { name: 'Empathetic', value: 'empatico' },
+];
+const STYLES = [
+    { name: 'Concise', value: 'conciso' },
+    { name: 'Detailed', value: 'detallado' },
+    { name: 'Conversational', value: 'conversacional' },
+    { name: 'Direct', value: 'directo' },
+];
+const LANGUAGES = [
+    { name: 'Spanish', value: 'Espanol' },
+    { name: 'English', value: 'English' },
+    { name: 'Portuguese', value: 'Portugues' },
+    { name: 'French', value: 'Frances' },
+];
+/** Choices for a list prompt, keeping a custom current value selectable. */
+function withCurrent(choices, current) {
+    if (current && !choices.some((c) => c.value === current)) {
+        return [{ name: `${current} (current)`, value: current }, ...choices];
+    }
+    return choices;
+}
+/** Parses true/false style flag values. */
+function parseBool(value, flag) {
+    if (value === undefined)
+        return undefined;
+    const v = value.trim().toLowerCase();
+    if (['true', 'yes', 'y', '1', 'on'].includes(v))
+        return true;
+    if (['false', 'no', 'n', '0', 'off'].includes(v))
+        return false;
+    throw new CliError(`Invalid value "${value}" for ${flag}.`, 'invalid_boolean', `Use ${flag} true or ${flag} false.`);
+}
+function canPrompt() {
+    return Boolean(process.stdin.isTTY) && !isJsonMode();
+}
+function buildAgent(credentials, dev) {
+    return new Agent({
+        workspaceId: credentials.workspace,
+        apiKey: credentials.apiKey,
+        zone: credentials.zone,
+        ...(dev && { customUrl: "http://localhost:5090" })
+    });
+}
+async function loadAgent(agent, agentId) {
+    const spinner = createSpinner('Loading agent...');
+    spinner.start();
+    try {
+        const res = await agent.getAgentById({ id: agentId });
+        const data = res?.agent || res?.data || res;
+        if (!data || !data.name) {
+            throw new CliError(`Agent ${agentId} was not found in the active workspace.`, 'not_found', "Check the ID with 'plazbot agent list'.");
+        }
+        return data;
+    }
+    finally {
+        spinner.stop();
+    }
+}
+/**
+ * Guarda cambios parciales del agente. Los endpoints set-greeting/set-instructions/set-persona
+ * que usa el SDK no existen en la API, y POST /api/agent/update REEMPLAZA el documento completo:
+ * por eso se parte de la config actual (limpia, como en export), se aplica el parche y se envia todo.
+ */
+async function saveAgentPatch(credentials, dev, agentId, current, patch) {
+    await patchAgent({ apiKey: credentials.apiKey, workspace: credentials.workspace, zone: credentials.zone, dev }, agentId, patch, current);
+}
 const setGroup = new Command('set')
-    .description('Quickly configure agent properties');
+    .description('Quickly change the greeting, instructions or persona of an agent');
 addExamples(setGroup, [
     { description: 'Replace the agent greeting',
-        command: 'plazbot agent set greeting agt_AbcDef123 "Hi! How can I help you today?"' },
-    { description: 'Configure tone, style, language, etc. interactively',
-        command: 'plazbot agent set instructions agt_AbcDef123' },
-    { description: 'Define the agent persona (name, role, voice)',
-        command: 'plazbot agent set persona agt_AbcDef123' },
+        command: 'plazbot agent set greeting age_AbcDef123 "Hi! How can I help you today?"' },
+    { description: 'Change tone and language without prompts',
+        command: 'plazbot agent set instructions age_AbcDef123 --tone amigable --language English' },
+    { description: 'Define the agent persona',
+        command: 'plazbot agent set persona age_AbcDef123 --name "Sofia" --role "Sales advisor" --human false' },
 ]);
-// Set greeting
-setGroup.command('greeting')
-    .description('Change the agent greeting')
+// ── greeting ─────────────────────────────────────────────────────────
+const greetingCmd = setGroup.command('greeting')
+    .description('Change the agent greeting (first message of a conversation)')
     .argument('<agentId>', 'Agent ID')
-    .argument('<greeting>', 'New greeting')
+    .argument('<greeting>', 'New greeting text (quote it)')
+    .addOption(jsonOption('Print the result as JSON ({"id","updated","greeting"})'))
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, greeting, options) => {
     try {
+        if (!greeting || !greeting.trim()) {
+            throw new CliError('The greeting cannot be empty.', 'missing_greeting');
+        }
         const credentials = await getStoredCredentials();
-        const agent = new Agent({
-            workspaceId: credentials.workspace,
-            apiKey: credentials.apiKey,
-            zone: credentials.zone,
-            ...(options.dev && { customUrl: "http://localhost:5090" })
-        });
+        const agent = buildAgent(credentials, options.dev);
+        const current = await loadAgent(agent, agentId);
         const spinner = createSpinner('Updating greeting...');
         spinner.start();
-        await agent.setGreeting(agentId, greeting);
+        try {
+            await saveAgentPatch(credentials, options.dev, agentId, current, {
+                instructions: { ...(current.instructions || {}), greeting },
+            });
+        }
+        catch (err) {
+            spinner.fail('Could not update the greeting');
+            throw err;
+        }
         spinner.succeed('Greeting updated');
+        if (isJsonMode()) {
+            printJson({ id: agentId, updated: true, greeting });
+            return;
+        }
         logger.label('New greeting', greeting);
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
-// Set instructions (wizard)
-setGroup.command('instructions')
-    .description('Configure agent instructions')
+addExamples(greetingCmd, [
+    { description: 'Set a new greeting',
+        command: 'plazbot agent set greeting age_AbcDef123 "Hi! How can I help you today?"' },
+]);
+const instructionsCmd = setGroup.command('instructions')
+    .description('Configure tone, style, personality, objective, language and emojis')
     .argument('<agentId>', 'Agent ID')
+    .option('--tone <tone>', 'Tone: profesional, amigable, formal, casual, tecnico, empatico, or any custom value')
+    .option('--style <style>', 'Style: conciso, detallado, conversacional, directo, or any custom value')
+    .option('--personality <text>', 'Personality in one sentence')
+    .option('--objective <text>', 'Main objective of the agent')
+    .option('--language <language>', 'Language: Espanol, English, Portugues, Frances, or any custom value')
+    .option('--emojis <true|false>', 'Use emojis in answers')
+    .addOption(jsonOption('Print the result as JSON ({"id","updated","instructions"})'))
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, options) => {
     try {
+        const emojisFlag = parseBool(options.emojis, '--emojis');
+        const flags = {
+            tone: options.tone,
+            style: options.style,
+            personality: options.personality,
+            objective: options.objective,
+            language: options.language,
+            emojis: emojisFlag,
+        };
+        const given = Object.fromEntries(Object.entries(flags).filter(([, v]) => v !== undefined));
+        const hasFlags = Object.keys(given).length > 0;
+        if (!hasFlags && !canPrompt()) {
+            throw new CliError('No instruction values were given and there is no interactive terminal.', 'missing_values', `Pass at least one flag, e.g. plazbot agent set instructions ${agentId} --tone amigable --emojis false`);
+        }
         const credentials = await getStoredCredentials();
-        const agent = new Agent({
-            workspaceId: credentials.workspace,
-            apiKey: credentials.apiKey,
-            zone: credentials.zone,
-            ...(options.dev && { customUrl: "http://localhost:5090" })
-        });
-        // Cargar instrucciones actuales
-        const loadSpinner = createSpinner('Loading...');
-        loadSpinner.start();
-        const _res1 = await agent.getAgentById({ id: agentId });
-        const agentData = _res1.agent || _res1.data || _res1;
-        loadSpinner.stop();
+        const agent = buildAgent(credentials, options.dev);
+        const agentData = await loadAgent(agent, agentId);
         const current = agentData.instructions || {};
-        console.log(section('Instructions - ' + (agentData.name || agentId)));
-        const answers = await inquirer.prompt([
-            { type: 'list', name: 'tone', message: 'Tone:', choices: ['profesional', 'amigable', 'formal', 'casual', 'tecnico', 'empatico'], default: current.tone || 'profesional' },
-            { type: 'list', name: 'style', message: 'Style:', choices: ['conciso', 'detallado', 'conversacional', 'directo'], default: current.style || 'conciso' },
-            { type: 'input', name: 'personality', message: 'Personality:', default: current.personality || '' },
-            { type: 'input', name: 'objective', message: 'Objective:', default: current.objective || '' },
-            { type: 'list', name: 'language', message: 'Language:', choices: ['Espanol', 'English', 'Portugues'], default: current.language || 'Espanol' },
-            { type: 'confirm', name: 'emojis', message: 'Use emojis?', default: current.emojis === true },
-        ]);
-        const instructions = {
-            ...current,
-            tone: answers.tone,
-            style: answers.style,
-            personality: answers.personality,
-            objective: answers.objective,
-            language: answers.language,
-            emojis: answers.emojis,
-        };
+        let changes = given;
+        if (!hasFlags) {
+            // Sin flags y con terminal: asistente con los valores actuales por defecto
+            console.log(section('Instructions - ' + (agentData.name || agentId)));
+            changes = await inquirer.prompt([
+                { type: 'list', name: 'tone', message: 'Tone:', choices: withCurrent(TONES, current.tone), default: current.tone || 'profesional' },
+                { type: 'list', name: 'style', message: 'Style:', choices: withCurrent(STYLES, current.style), default: current.style || 'conciso' },
+                { type: 'input', name: 'personality', message: 'Personality:', default: current.personality || '' },
+                { type: 'input', name: 'objective', message: 'Objective:', default: current.objective || '' },
+                { type: 'list', name: 'language', message: 'Language:', choices: withCurrent(LANGUAGES, current.language), default: current.language || 'Espanol' },
+                { type: 'confirm', name: 'emojis', message: 'Use emojis?', default: current.emojis === true },
+            ]);
+        }
+        const instructions = { ...current, ...changes };
         const spinner = createSpinner('Saving instructions...');
         spinner.start();
-        await agent.setInstructions(agentId, instructions);
+        try {
+            await saveAgentPatch(credentials, options.dev, agentId, agentData, { instructions });
+        }
+        catch (err) {
+            spinner.fail('Could not save the instructions');
+            throw err;
+        }
         spinner.succeed('Instructions updated');
+        if (isJsonMode()) {
+            printJson({ id: agentId, updated: true, fields: Object.keys(changes), instructions });
+            return;
+        }
+        Object.entries(changes).forEach(([key, value]) => logger.label(key, String(value)));
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
-// Set persona (wizard)
-setGroup.command('persona')
-    .description('Configure the agent persona')
+addExamples(instructionsCmd, [
+    { description: 'Edit every value interactively',
+        command: 'plazbot agent set instructions age_AbcDef123' },
+    { description: 'Change only tone and emojis (no prompts)',
+        command: 'plazbot agent set instructions age_AbcDef123 --tone amigable --emojis false' },
+    { description: 'Set a custom objective and language',
+        command: 'plazbot agent set instructions age_AbcDef123 --objective "Book demos" --language English' },
+]);
+addNotes(instructionsCmd, [
+    'With flags, only the given values change and nothing is asked. Without flags, an interactive editor opens.',
+]);
+const personaCmd = setGroup.command('persona')
+    .description('Configure the persona of the agent (name, role, voice)')
     .argument('<agentId>', 'Agent ID')
+    .option('--name <name>', 'Character name, e.g. "Sofia"')
+    .option('--role <role>', 'Role, e.g. "Sales advisor"')
+    .option('--first-person <true|false>', 'Speak in first person')
+    .option('--human <true|false>', 'Present itself as a human instead of a virtual assistant')
+    .addOption(jsonOption('Print the result as JSON ({"id","updated","persona"})'))
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, options) => {
     try {
+        const flags = {
+            name: options.name,
+            role: options.role,
+            speaksInFirstPerson: parseBool(options.firstPerson, '--first-person'),
+            isHuman: parseBool(options.human, '--human'),
+        };
+        const given = Object.fromEntries(Object.entries(flags).filter(([, v]) => v !== undefined));
+        const hasFlags = Object.keys(given).length > 0;
+        if (!hasFlags && !canPrompt()) {
+            throw new CliError('No persona values were given and there is no interactive terminal.', 'missing_values', `Pass at least one flag, e.g. plazbot agent set persona ${agentId} --name "Sofia" --role "Sales advisor"`);
+        }
         const credentials = await getStoredCredentials();
-        const agent = new Agent({
-            workspaceId: credentials.workspace,
-            apiKey: credentials.apiKey,
-            zone: credentials.zone,
-            ...(options.dev && { customUrl: "http://localhost:5090" })
-        });
-        const loadSpinner = createSpinner('Loading...');
-        loadSpinner.start();
-        const _res2 = await agent.getAgentById({ id: agentId });
-        const agentData = _res2.agent || _res2.data || _res2;
-        loadSpinner.stop();
+        const agent = buildAgent(credentials, options.dev);
+        const agentData = await loadAgent(agent, agentId);
         const current = agentData.person || {};
-        console.log(section('Persona - ' + (agentData.name || agentId)));
-        const answers = await inquirer.prompt([
-            { type: 'input', name: 'name', message: 'Character name:', default: current.name || '' },
-            { type: 'input', name: 'role', message: 'Role:', default: current.role || '' },
-            { type: 'confirm', name: 'speaksInFirstPerson', message: 'Speak in first person?', default: current.speaksInFirstPerson !== false },
-        ]);
+        let changes = given;
+        if (!hasFlags) {
+            console.log(section('Persona - ' + (agentData.name || agentId)));
+            changes = await inquirer.prompt([
+                { type: 'input', name: 'name', message: 'Character name:', default: current.name || '' },
+                { type: 'input', name: 'role', message: 'Role:', default: current.role || '' },
+                { type: 'confirm', name: 'speaksInFirstPerson', message: 'Speak in first person?', default: current.speaksInFirstPerson !== false },
+                { type: 'confirm', name: 'isHuman', message: 'Present itself as a human?', default: current.isHuman === true },
+            ]);
+        }
+        // Se parte de la persona actual para no perder campos que no se tocan
+        const persona = { ...current, ...changes };
         const spinner = createSpinner('Saving persona...');
         spinner.start();
-        await agent.setPersona(agentId, answers);
+        try {
+            await saveAgentPatch(credentials, options.dev, agentId, agentData, { person: persona });
+        }
+        catch (err) {
+            spinner.fail('Could not save the persona');
+            throw err;
+        }
         spinner.succeed('Persona updated');
+        if (isJsonMode()) {
+            printJson({ id: agentId, updated: true, fields: Object.keys(changes), persona });
+            return;
+        }
+        Object.entries(changes).forEach(([key, value]) => logger.label(key, String(value)));
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
+addExamples(personaCmd, [
+    { description: 'Edit the persona interactively',
+        command: 'plazbot agent set persona age_AbcDef123' },
+    { description: 'Set name and role (no prompts)',
+        command: 'plazbot agent set persona age_AbcDef123 --name "Sofia" --role "Sales advisor"' },
+    { description: 'Make the agent speak as a human in first person',
+        command: 'plazbot agent set persona age_AbcDef123 --first-person true --human true' },
+]);
+addNotes(personaCmd, [
+    'With flags, only the given values change and nothing is asked. Without flags, an interactive editor opens.',
+]);
 export const setCommand = setGroup;