plazbot-cli 0.4.1 → 0.5.0

package/dist/commands/whatsapp/widget.js

@@ -1,68 +1,105 @@
 import { Command } 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 { addExamples, addNotes } from '../../utils/help.js';
 import { createSpinner } from '../../utils/ui.js';
+import { jsonOption, isJsonMode, printJson, fail, CliError } from '../../utils/output.js';
+import { phoneOption, normalizePhone } from '../../utils/phone.js';
+import { fetchFullAgent, getAgentClient, updateAgentMerged } from './shared.js';
 const widgetGroup = new Command('widget')
-    .description('Manage WhatsApp widget');
+    .description('Show or hide the WhatsApp click-to-chat button on an agent web widget');
 addExamples(widgetGroup, [
-    { description: 'Enable the WhatsApp click-to-chat widget on an agent',
-        command: 'plazbot whatsapp widget enable agt_AbcDef123 -u https://wa.me/51912345678' },
-    { description: 'Disable the widget',
-        command: 'plazbot whatsapp widget disable agt_AbcDef123' },
+    { description: 'Enable the WhatsApp button for a number',
+        command: 'plazbot whatsapp widget enable age_AbcDef123 -p +51912345678' },
+    { description: 'Disable it',
+        command: 'plazbot whatsapp widget disable age_AbcDef123' },
 ]);
 widgetGroup.command('enable')
-    .description('Enable the WhatsApp widget on an agent')
-    .argument('<agentId>', 'Agent ID')
-    .requiredOption('-u, --url <url>', 'WhatsApp URL (e.g.: https://wa.me/51912345678)')
+    .description('Enable the WhatsApp button on an agent web widget')
+    .argument('<agentId>', 'Agent ID (e.g. age_AbcDef123)')
+    .addOption(phoneOption('WhatsApp number the button opens, e.g. +51912345678 (builds https://wa.me/<number>)'))
+    .option('-u, --url <url>', 'Custom click-to-chat URL instead of --phone (e.g. https://wa.me/51912345678?text=Hi)')
+    .addOption(jsonOption())
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, options) => {
     try {
-        const credentials = await getStoredCredentials();
-        const agent = new Agent({
-            workspaceId: credentials.workspace,
-            apiKey: credentials.apiKey,
-            zone: credentials.zone,
-            ...(options.dev && { customUrl: "http://localhost:5090" })
-        });
-        const spinner = createSpinner('Enabling WhatsApp widget...');
-        spinner.start();
-        await agent.updateAgent(agentId, {
-            enableWhatsappWidget: true,
-            urlWhatsappWidget: options.url,
-        });
-        spinner.succeed('WhatsApp widget enabled');
-        logger.label('URL', options.url);
+        if (options.phone && options.url) {
+            throw new CliError('Use either --phone or --url, not both.', 'conflicting_options');
+        }
+        let url;
+        if (options.phone) {
+            url = `https://wa.me/${normalizePhone(options.phone)}`;
+        }
+        else if (options.url) {
+            url = options.url.trim();
+            if (!/^https?:\/\/\S+$/i.test(url)) {
+                throw new CliError(`Invalid URL "${options.url}".`, 'invalid_url', 'Use a full URL, e.g. https://wa.me/51912345678');
+            }
+        }
+        else {
+            throw new CliError('Missing WhatsApp number.', 'missing_phone', 'Pass --phone +51912345678, or a custom link with --url <url>.');
+        }
+        const { agent, credentials } = await getAgentClient(options.dev);
+        const spinner = createSpinner('Enabling WhatsApp button...').start();
+        try {
+            const current = await fetchFullAgent(agent, agentId);
+            await updateAgentMerged(credentials, options.dev, agentId, current, { enableWhatsappWidget: true, urlWhatsappWidget: url });
+        }
+        catch (error) {
+            spinner.stop();
+            throw error;
+        }
+        spinner.succeed('WhatsApp button enabled');
+        if (isJsonMode())
+            printJson({ id: agentId, enableWhatsappWidget: true, url });
+        else
+            logger.label('URL', url);
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 widgetGroup.command('disable')
-    .description('Disable the WhatsApp widget on an agent')
-    .argument('<agentId>', 'Agent ID')
+    .description('Disable the WhatsApp button on an agent web widget')
+    .argument('<agentId>', 'Agent ID (e.g. age_AbcDef123)')
+    .addOption(jsonOption())
     .option('--dev', 'Use development environment', false)
     .action(async (agentId, options) => {
     try {
-        const credentials = await getStoredCredentials();
-        const agent = new Agent({
-            workspaceId: credentials.workspace,
-            apiKey: credentials.apiKey,
-            zone: credentials.zone,
-            ...(options.dev && { customUrl: "http://localhost:5090" })
-        });
-        const spinner = createSpinner('Disabling WhatsApp widget...');
-        spinner.start();
-        await agent.updateAgent(agentId, { enableWhatsappWidget: false });
-        spinner.succeed('WhatsApp widget disabled');
+        const { agent, credentials } = await getAgentClient(options.dev);
+        const spinner = createSpinner('Disabling WhatsApp button...').start();
+        try {
+            const current = await fetchFullAgent(agent, agentId);
+            await updateAgentMerged(credentials, options.dev, agentId, current, { enableWhatsappWidget: false });
+        }
+        catch (error) {
+            spinner.stop();
+            throw error;
+        }
+        spinner.succeed('WhatsApp button disabled');
+        if (isJsonMode())
+            printJson({ id: agentId, enableWhatsappWidget: false });
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
+const enableCmd = widgetGroup.commands.find(c => c.name() === 'enable');
+const disableCmd = widgetGroup.commands.find(c => c.name() === 'disable');
+addExamples(enableCmd, [
+    { description: 'Button that opens a chat with your number',
+        command: 'plazbot whatsapp widget enable age_AbcDef123 -p +51912345678' },
+    { description: 'Custom link with a pre-filled message',
+        command: 'plazbot whatsapp widget enable age_AbcDef123 -u "https://wa.me/51912345678?text=Hi"' },
+    { description: 'Machine-readable output',
+        command: 'plazbot whatsapp widget enable age_AbcDef123 -p +51912345678 --json' },
+]);
+addNotes(enableCmd, [
+    'Pass one of --phone or --url. Only the widget fields change; the rest of the agent is kept.',
+]);
+addExamples(disableCmd, [
+    { description: 'Hide the WhatsApp button',
+        command: 'plazbot whatsapp widget disable age_AbcDef123' },
+    { description: 'Machine-readable output',
+        command: 'plazbot whatsapp widget disable age_AbcDef123 --json' },
+]);
 export const widgetCommand = widgetGroup;

package/dist/commands/workers/deploy.js

@@ -1,28 +1,31 @@
-import { Command } from 'commander';
+import { Command, Option } from 'commander';
 import fs from 'fs/promises';
 import path from 'path';
 import { getStoredCredentials } from '../../utils/credentials.js';
 import { createApiClient } from '../../utils/api.js';
 import { logger } from '../../utils/logger.js';
-import { addExamples } from '../../utils/help.js';
+import { addExamples, addNotes } from '../../utils/help.js';
 import { createSpinner, createTable, theme } from '../../utils/ui.js';
+import { jsonOption, isJsonMode, printJson, fail, CliError, describeError } from '../../utils/output.js';
 export const deployCommand = new Command('deploy')
-    .description('Deploy a worker to the Plazbot workspace')
-    .argument('[file]', 'Worker TypeScript file (e.g.: ./workers/my-tool.ts)')
-    .option('-n, --name <name>', 'Worker name (overrides the one in the file)')
-    .option('-d, --description <desc>', 'Worker description')
+    .description('Deploy a worker file, or every worker in ./workers')
+    .argument('[file]', 'Worker TypeScript file (default: every *.ts file in ./workers)')
+    .option('-n, --name <name>', 'Worker name, overrides the one in the file (single file only)')
+    .option('--description <text>', 'Worker description (single file only)')
+    .addOption(new Option('-d <text>', 'Alias of --description').hideHelp())
+    .addOption(jsonOption())
     .option('--dev', 'Use development environment', false)
     .action(async (file, options) => {
     try {
-        const credentials = await getStoredCredentials();
-        // Buscar archivos de workers
+        const description = options.description ?? options.d;
         const files = await resolveWorkerFiles(file);
         if (files.length === 0) {
-            logger.error('No worker files found to deploy');
-            logger.dim('Usage: plazbot workers deploy ./workers/my-tool.ts');
-            logger.dim('   or: plazbot workers deploy (searches ./workers/ automatically)');
-            process.exit(1);
+            throw new CliError('No worker files found to deploy.', 'no_worker_files', 'Pass a file (plazbot workers deploy ./workers/my-tool.ts) or add *.ts files to ./workers.');
+        }
+        if (files.length > 1 && (options.name || description)) {
+            throw new CliError(`--name and --description only apply to a single file, but ${files.length} files were found in ./workers.`, 'name_with_multiple_files', 'Deploy one file at a time: plazbot workers deploy ./workers/my-tool.ts -n "my-tool"');
         }
+        const credentials = await getStoredCredentials();
         const api = createApiClient({
             apiKey: credentials.apiKey,
             workspace: credentials.workspace,
@@ -30,103 +33,122 @@ export const deployCommand = new Command('deploy')
             dev: options.dev,
         });
         logger.title(`Deploying ${files.length} worker${files.length > 1 ? 's' : ''}`);
-        console.log();
         const results = [];
         for (const filePath of files) {
             const fileName = path.basename(filePath);
             const spinner = createSpinner(`Deploying ${fileName}...`);
             spinner.start();
             try {
-                // Leer archivo fuente
                 const sourceCode = await fs.readFile(filePath, 'utf-8');
-                // Enviar al API
                 const response = await api.post('/api/worker/deploy', {
                     sourceCode,
                     fileName,
                     name: files.length === 1 ? options.name : undefined,
-                    description: files.length === 1 ? options.description : undefined,
+                    description: files.length === 1 ? description : undefined,
                 });
                 const data = response.data;
+                if (data?.success === false || !data?.name) {
+                    throw new CliError(data?.message || 'The API did not return the deployed worker.', 'deploy_failed');
+                }
                 spinner.succeed(`${fileName} → ${data.name} (${data.type})`);
-                results.push({ file: fileName, result: data });
+                results.push({
+                    file: fileName,
+                    name: data.name,
+                    type: data.type,
+                    status: 'deployed',
+                    workerStatus: data.status,
+                    webhookUrl: data.webhookUrl || undefined,
+                    cronExpression: data.cronExpression || undefined,
+                });
             }
             catch (error) {
-                const msg = error.response?.data?.message || error.message || 'Unknown error';
-                spinner.fail(`${fileName}: ${msg}`);
-                results.push({ file: fileName, error: msg });
+                const { message } = describeError(error);
+                spinner.fail(`${fileName}: ${message}`);
+                results.push({ file: fileName, status: 'failed', error: message });
             }
         }
-        // Resumen
-        console.log();
-        const success = results.filter(r => r.result);
-        const failed = results.filter(r => r.error);
+        const success = results.filter(r => r.status === 'deployed');
+        const failed = results.filter(r => r.status === 'failed');
+        if (isJsonMode()) {
+            printJson(results);
+            if (failed.length > 0)
+                process.exit(1);
+            return;
+        }
         if (success.length > 0) {
             logger.success(`${success.length} worker${success.length > 1 ? 's' : ''} deployed`);
-            const rows = success.map(r => [
-                r.result.name,
-                r.result.type,
-                r.result.status,
-                r.result.webhookUrl || r.result.cronExpression || '—',
-            ]);
-            console.log(createTable(['Name', 'Type', 'Status', 'URL/Cron'], rows));
+            console.log(createTable(['File', 'Name', 'Type', 'Status', 'URL/Cron'], success.map(r => [
+                r.file,
+                r.name,
+                r.type || '—',
+                r.workerStatus || '—',
+                r.webhookUrl || r.cronExpression || '—',
+            ])));
         }
-        if (failed.length > 0) {
-            logger.warning(`${failed.length} worker${failed.length > 1 ? 's' : ''} failed`);
-            failed.forEach(f => logger.dim(`  ${f.file}: ${f.error}`));
-        }
-        // Mostrar webhook URLs si hay
-        const webhooks = success.filter(r => r.result.webhookUrl);
+        const webhooks = success.filter(r => r.webhookUrl);
         if (webhooks.length > 0) {
-            console.log();
             logger.title('Webhook URLs');
-            webhooks.forEach(w => {
-                logger.label(w.result.name, theme.secondary(w.result.webhookUrl));
-            });
+            webhooks.forEach(w => logger.label(w.name, theme.secondary(w.webhookUrl)));
+        }
+        if (failed.length > 0) {
+            logger.warning(`${failed.length} of ${results.length} worker${results.length > 1 ? 's' : ''} failed to deploy`);
+            failed.forEach(f => logger.dim(`  ${f.file}: ${f.error}`));
         }
         console.log();
         if (options.dev) {
             logger.warning('Environment: development');
         }
+        if (failed.length > 0)
+            process.exit(1);
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(deployCommand, [
     { description: 'Deploy a single worker file',
-        command: 'plazbot workers deploy ./workers/my-tool.ts' },
-    { description: 'Deploy every TypeScript file inside ./workers automatically',
+        command: 'plazbot workers deploy ./workers/get-stock.ts' },
+    { description: 'Deploy every *.ts file inside ./workers',
         command: 'plazbot workers deploy' },
     { description: 'Deploy with a custom name and description',
-        command: 'plazbot workers deploy ./workers/cron.ts -n "Daily Report" -d "Generates daily KPIs"' },
-    { description: 'Deploy against the local backend',
-        command: 'plazbot workers deploy ./workers/test.ts --dev' },
+        command: 'plazbot workers deploy ./workers/cron.ts -n "daily-report" --description "Generates daily KPIs"' },
+    { description: 'Deploy everything and get per-file results as JSON (CI)',
+        command: 'plazbot workers deploy --json' },
+]);
+addNotes(deployCommand, [
+    'Without [file], every *.ts file in ./workers is deployed (*.d.ts files are skipped).',
+    '--name and --description only work when deploying a single file.',
+    'Only the single source file is uploaded: relative imports and npm dependencies are not bundled.',
+    'The TypeScript is compiled on the Plazbot server; compile errors show up as a failed file.',
+    'Exits with code 1 if any file fails to deploy.',
 ]);
 // Resuelve los archivos a desplegar
 async function resolveWorkerFiles(file) {
     if (file) {
-        // Archivo especifico
         const resolved = path.resolve(file);
+        let isFile = false;
         try {
-            await fs.access(resolved);
-            return [resolved];
+            isFile = (await fs.stat(resolved)).isFile();
         }
         catch {
-            throw new Error(`File not found: ${file}`);
+            isFile = false;
+        }
+        if (!isFile) {
+            throw new CliError(`File not found: ${file}.`, 'file_not_found', 'Check the path to your worker file.');
         }
+        return [resolved];
     }
     // Buscar en ./workers/ automaticamente
     const workersDir = path.resolve('workers');
+    let entries;
     try {
-        const entries = await fs.readdir(workersDir);
-        const tsFiles = entries
-            .filter(e => e.endsWith('.ts') && !e.endsWith('.d.ts'))
-            .map(e => path.join(workersDir, e));
-        return tsFiles;
+        entries = await fs.readdir(workersDir);
     }
     catch {
         return [];
     }
+    return entries
+        .filter(e => e.endsWith('.ts') && !e.endsWith('.d.ts'))
+        .sort()
+        .map(e => path.join(workersDir, e));
 }

package/dist/commands/workers/index.js

@@ -7,7 +7,7 @@ import { removeCommand } from './remove.js';
 import { secretCommand } from './secret.js';
 import { testCommand } from './test.js';
 export const workersCommands = new Command('workers')
-    .description('Commands to manage workers (tools, syncs, schedules, webhooks)')
+    .description('Manage workers (tools, syncs, schedules, webhooks)')
     .addCommand(deployCommand)
     .addCommand(listCommand)
     .addCommand(logsCommand)
@@ -15,14 +15,14 @@ export const workersCommands = new Command('workers')
     .addCommand(secretCommand)
     .addCommand(testCommand);
 addExamples(workersCommands, [
-    { description: 'Deploy every worker file in ./workers automatically',
+    { description: 'Deploy every worker file in ./workers',
         command: 'plazbot workers deploy' },
     { description: 'List active workers',
         command: 'plazbot workers list -s active' },
-    { description: 'Run a worker with a JSON payload',
-        command: 'plazbot workers test my-tool -p \'{"product":"iPhone 15"}\'' },
-    { description: 'Save a secret for workers (plz.env.KEY)',
-        command: 'plazbot workers secret set STRIPE_API_KEY sk_live_xxxxx' },
-    { description: 'View the last 100 executions of a worker',
-        command: 'plazbot workers logs my-tool -l 100' },
+    { description: 'Run a worker with JSON data',
+        command: 'plazbot workers test get-stock -d \'{"product":"iPhone 15"}\'' },
+    { description: 'Save a secret from stdin (available as plz.env.STRIPE_API_KEY)',
+        command: 'echo "$STRIPE_KEY" | plazbot workers secret set STRIPE_API_KEY' },
+    { description: 'Follow the executions of a worker in real time',
+        command: 'plazbot workers logs get-stock --follow' },
 ]);

package/dist/commands/workers/list.js

@@ -1,13 +1,17 @@
-import { Command } from 'commander';
+import { Command, Option } from 'commander';
 import { getStoredCredentials } from '../../utils/credentials.js';
 import { createApiClient } from '../../utils/api.js';
 import { logger } from '../../utils/logger.js';
 import { addExamples } from '../../utils/help.js';
 import { createSpinner, createTable, theme, formatDate } from '../../utils/ui.js';
+import { jsonOption, isJsonMode, printJson, fail } from '../../utils/output.js';
+const WORKER_TYPES = ['tool', 'worker', 'sync', 'schedule', 'webhook'];
+const WORKER_STATUSES = ['active', 'inactive', 'error'];
 export const listCommand = new Command('list')
-    .description('List all workers in the workspace')
-    .option('-t, --type <type>', 'Filter by type (tool, worker, sync, schedule, webhook)')
-    .option('-s, --status <status>', 'Filter by status (active, inactive, error)')
+    .description('List the workers in the workspace')
+    .addOption(new Option('-t, --type <type>', 'Filter by type').choices(WORKER_TYPES))
+    .addOption(new Option('-s, --status <status>', 'Filter by status').choices(WORKER_STATUSES))
+    .addOption(jsonOption())
     .option('--dev', 'Use development environment', false)
     .action(async (options) => {
     try {
@@ -25,12 +29,36 @@ export const listCommand = new Command('list')
             params.type = options.type;
         if (options.status)
             params.status = options.status;
-        const response = await api.get('/api/worker', { params });
-        const workers = response.data;
-        spinner.stop();
-        if (!workers || workers.length === 0) {
+        let workers;
+        try {
+            const response = await api.get('/api/worker', { params });
+            workers = Array.isArray(response.data) ? response.data : [];
+        }
+        finally {
+            spinner.stop();
+        }
+        if (isJsonMode()) {
+            printJson(workers.map(w => ({
+                id: w.id,
+                name: w.name,
+                type: w.type,
+                status: w.status,
+                description: w.description ?? null,
+                cronExpression: w.cronExpression ?? null,
+                webhookUrl: w.webhookUrl ?? null,
+                executionCount: w.executionCount ?? 0,
+                errorCount: w.errorCount ?? 0,
+                avgDurationMs: w.avgDurationMs ?? 0,
+                lastExecutedAt: w.lastExecutedAt ?? null,
+                lastError: w.lastError ?? null,
+                createdAt: w.createdAt,
+                updatedAt: w.updatedAt,
+            })));
+            return;
+        }
+        if (workers.length === 0) {
             logger.info('\n  No workers found in this workspace');
-            logger.dim('  Deploy your first worker: plazbot workers deploy ./workers/my-tool.ts');
+            logger.dim('Deploy your first worker: plazbot workers deploy ./workers/my-tool.ts');
             console.log();
             return;
         }
@@ -41,20 +69,16 @@ export const listCommand = new Command('list')
             w.status === 'active' ? theme.success('active')
                 : w.status === 'error' ? theme.error('error')
                     : theme.muted('inactive'),
-            `${w.executionCount}`,
+            `${w.executionCount ?? 0}`,
             w.errorCount > 0 ? theme.error(`${w.errorCount}`) : theme.muted('0'),
             w.avgDurationMs > 0 ? `${w.avgDurationMs}ms` : '—',
             w.lastExecutedAt ? formatDate(w.lastExecutedAt) : '—',
         ]);
         console.log(createTable(['Name', 'Type', 'Status', 'Executions', 'Errors', 'Duration', 'Last execution'], rows));
-        // Mostrar workers con error
         const errored = workers.filter(w => w.status === 'error');
         if (errored.length > 0) {
-            console.log();
             logger.warning('Workers with errors:');
-            errored.forEach(w => {
-                logger.dim(`  ${w.name}: ${w.lastError || 'no details'}`);
-            });
+            errored.forEach(w => logger.dim(`  ${w.name}: ${w.lastError || 'no details'}`));
         }
         console.log();
         if (options.dev) {
@@ -62,9 +86,7 @@ export const listCommand = new Command('list')
         }
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(listCommand, [
@@ -74,8 +96,8 @@ addExamples(listCommand, [
         command: 'plazbot workers list -t schedule' },
     { description: 'Show only workers with errors',
         command: 'plazbot workers list -s error' },
-    { description: 'Filter active webhooks',
-        command: 'plazbot workers list -t webhook -s active' },
+    { description: 'Get active webhooks as JSON',
+        command: 'plazbot workers list -t webhook -s active --json' },
 ]);
 function typeLabel(type) {
     const labels = {