plazbot-cli 0.4.1 → 0.5.1

package/dist/commands/workers/logs.js

@@ -1,17 +1,26 @@
-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 { addExamples, addNotes } from '../../utils/help.js';
 import { createSpinner, createTable, theme, formatDate } from '../../utils/ui.js';
+import { jsonOption, isJsonMode, printJson, fail, CliError, describeError } from '../../utils/output.js';
+const FOLLOW_INTERVAL_MS = 3000;
+const FOLLOW_BATCH = 50;
 export const logsCommand = new Command('logs')
-    .description('View execution logs for a worker')
+    .description('View the execution logs of a worker')
     .argument('<name>', 'Worker name')
-    .option('-l, --limit <number>', 'Number of logs to display', '20')
-    .option('-s, --status <status>', 'Filter by status (success, error, timeout)')
+    .option('-l, --limit <number>', 'Number of logs to show, newest first (1-1000)', '20')
+    .addOption(new Option('-s, --status <status>', 'Filter by status').choices(['success', 'error', 'timeout']))
+    .option('-f, --follow', 'Keep polling every 3s and print new executions (Ctrl+C to stop)', false)
+    .addOption(jsonOption())
     .option('--dev', 'Use development environment', false)
     .action(async (name, options) => {
     try {
+        const limit = Number(options.limit ?? 20);
+        if (!Number.isInteger(limit) || limit < 1 || limit > 1000) {
+            throw new CliError('--limit must be a whole number between 1 and 1000.', 'invalid_limit', 'Example: plazbot workers logs my-tool -l 100');
+        }
         const credentials = await getStoredCredentials();
         const api = createApiClient({
             apiKey: credentials.apiKey,
@@ -21,62 +30,156 @@ export const logsCommand = new Command('logs')
         });
         const spinner = createSpinner(`Fetching logs for "${name}"...`);
         spinner.start();
-        const params = {
-            limit: String(options.limit || 20),
-        };
-        if (options.status)
-            params.status = options.status;
-        const response = await api.get(`/api/worker/${encodeURIComponent(name)}/logs`, { params });
-        const logs = response.data;
-        spinner.stop();
-        if (!logs || logs.length === 0) {
+        let logs;
+        try {
+            logs = await fetchLogs(api, name, limit, options.status);
+        }
+        finally {
+            spinner.stop();
+        }
+        if (options.follow) {
+            await follow(api, name, logs, options.status);
+            return;
+        }
+        if (isJsonMode()) {
+            printJson(logs.map(toJson));
+            return;
+        }
+        if (logs.length === 0) {
             logger.info(`\n  No logs found for "${name}"`);
+            logger.dim(`Check the worker name with: plazbot workers list`);
             console.log();
             return;
         }
         logger.title(`Logs for "${name}" (${logs.length})`);
-        const rows = logs.map(log => [
+        console.log(createTable(['Date', 'Status', 'Duration', 'Source', 'Error'], logs.map(log => [
             formatDate(log.createdAt),
-            log.status === 'success' ? theme.success('ok')
-                : log.status === 'error' ? theme.error('error')
-                    : theme.warning('timeout'),
-            `${log.durationMs}ms`,
+            statusLabel(log.status),
+            `${log.durationMs ?? 0}ms`,
             log.triggerSource || '—',
             log.error ? theme.error(truncate(log.error, 40)) : '—',
-        ]);
-        console.log(createTable(['Date', 'Status', 'Duration', 'Source', 'Error'], rows));
-        // Resumen
+        ])));
         const successCount = logs.filter(l => l.status === 'success').length;
         const errorCount = logs.filter(l => l.status === 'error').length;
-        const avgDuration = Math.round(logs.reduce((sum, l) => sum + l.durationMs, 0) / logs.length);
+        const timeoutCount = logs.filter(l => l.status === 'timeout').length;
+        const avgDuration = Math.round(logs.reduce((sum, l) => sum + (l.durationMs ?? 0), 0) / logs.length);
         console.log();
         logger.label('Successful', theme.success(`${successCount}`));
         logger.label('Errors', errorCount > 0 ? theme.error(`${errorCount}`) : theme.muted('0'));
+        logger.label('Timeouts', timeoutCount > 0 ? theme.warning(`${timeoutCount}`) : theme.muted('0'));
         logger.label('Average duration', `${avgDuration}ms`);
+        if (logs.some(l => l.error && l.error.length > 40)) {
+            logger.dim('Errors are truncated; use --json to see the full message.');
+        }
         console.log();
         if (options.dev) {
             logger.warning('Environment: development');
         }
     }
     catch (error) {
-        if (error.response?.status === 404) {
-            logger.error(`Worker "${name}" not found`);
-        }
-        else {
-            const message = error instanceof Error ? error.message : 'Unknown error';
-            logger.error(message);
-        }
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(logsCommand, [
     { description: 'Show the last 20 executions of a worker',
-        command: 'plazbot workers logs my-tool' },
-    { description: 'Get the last 100 executions',
-        command: 'plazbot workers logs my-tool -l 100' },
-    { description: 'Only show failed executions',
-        command: 'plazbot workers logs my-tool -s error' },
+        command: 'plazbot workers logs get-stock' },
+    { description: 'Show the last 100 failed executions',
+        command: 'plazbot workers logs get-stock -l 100 -s error' },
+    { description: 'Follow new executions in real time',
+        command: 'plazbot workers logs get-stock --follow' },
+    { description: 'Get logs as JSON with full error messages',
+        command: 'plazbot workers logs get-stock --json' },
+]);
+addNotes(logsCommand, [
+    'The table truncates errors to 40 characters; --json includes the full message.',
+    'With --follow and --json, each execution is printed as one JSON object per line (NDJSON).',
 ]);
+async function fetchLogs(api, name, limit, status) {
+    const params = { limit: String(limit) };
+    if (status)
+        params.status = status;
+    const response = await api.get(`/api/worker/${encodeURIComponent(name)}/logs`, { params });
+    return Array.isArray(response.data) ? response.data : [];
+}
+// Polls the logs endpoint (ordered by created_at DESC) and prints entries not seen yet.
+async function follow(api, name, initial, status) {
+    const seen = new Set();
+    const emit = (entries) => {
+        // API returns newest first; print oldest first
+        for (const log of [...entries].reverse()) {
+            const key = logKey(log);
+            if (seen.has(key))
+                continue;
+            seen.add(key);
+            if (isJsonMode()) {
+                process.stdout.write(JSON.stringify(toJson(log)) + '\n');
+            }
+            else {
+                console.log(formatLine(log));
+            }
+        }
+    };
+    if (!isJsonMode()) {
+        logger.title(`Following logs for "${name}"`);
+        logger.dim('Press Ctrl+C to stop.');
+        console.log();
+    }
+    emit(initial);
+    process.on('SIGINT', () => {
+        if (!isJsonMode())
+            console.log();
+        process.exit(0);
+    });
+    let failing = false;
+    for (;;) {
+        await new Promise(resolve => setTimeout(resolve, FOLLOW_INTERVAL_MS));
+        try {
+            emit(await fetchLogs(api, name, FOLLOW_BATCH, status));
+            if (failing)
+                logger.dim('Connection restored.');
+            failing = false;
+        }
+        catch (error) {
+            const info = describeError(error);
+            if (info.status === 401 || info.status === 403 || info.status === 404)
+                fail(error);
+            if (!failing)
+                logger.warning(`Could not fetch logs (${info.message}). Retrying...`);
+            failing = true;
+        }
+    }
+}
+function logKey(log) {
+    return log.id != null ? String(log.id) : `${log.createdAt}|${log.status}|${log.durationMs}`;
+}
+function toJson(log) {
+    return {
+        id: log.id,
+        workerName: log.workerName,
+        workerType: log.workerType,
+        status: log.status,
+        durationMs: log.durationMs ?? 0,
+        triggerSource: log.triggerSource ?? null,
+        error: log.error ?? null,
+        createdAt: log.createdAt,
+    };
+}
+function formatLine(log) {
+    const parts = [
+        theme.muted(formatDate(log.createdAt)),
+        statusLabel(log.status),
+        `${log.durationMs ?? 0}ms`,
+        theme.muted(log.triggerSource || '—'),
+    ];
+    if (log.error)
+        parts.push(theme.error(log.error));
+    return '  ' + parts.join('  ');
+}
+function statusLabel(status) {
+    return status === 'success' ? theme.success('ok')
+        : status === 'error' ? theme.error('error')
+            : theme.warning(status || 'timeout');
+}
 function truncate(text, max) {
     if (text.length <= max)
         return text;

package/dist/commands/workers/remove.js

@@ -4,13 +4,15 @@ import { createApiClient } from '../../utils/api.js';
 import { logger } from '../../utils/logger.js';
 import { addExamples } from '../../utils/help.js';
 import { createSpinner } from '../../utils/ui.js';
-import readline from 'readline';
+import { jsonOption, isJsonMode, printJson, fail, CliError } from '../../utils/output.js';
+import { addYesOption, confirmAction } from '../../utils/confirm.js';
 export const removeCommand = new Command('remove')
     .description('Delete a worker from the workspace')
     .argument('<name>', 'Name of the worker to delete')
-    .option('-f, --force', 'Delete without confirmation', false)
-    .option('--dev', 'Use development environment', false)
-    .action(async (name, options) => {
+    .addOption(jsonOption())
+    .option('--dev', 'Use development environment', false);
+addYesOption(removeCommand);
+removeCommand.action(async (name, options) => {
     try {
         const credentials = await getStoredCredentials();
         const api = createApiClient({
@@ -19,71 +21,67 @@ export const removeCommand = new Command('remove')
             zone: credentials.zone,
             dev: options.dev,
         });
-        // Verificar que el worker existe
         const spinner = createSpinner(`Looking up worker "${name}"...`);
         spinner.start();
         let workerInfo;
         try {
             const response = await api.get(`/api/worker/${encodeURIComponent(name)}`);
             workerInfo = response.data;
-            spinner.stop();
         }
         catch (error) {
-            spinner.stop();
-            if (error.response?.status === 404) {
-                logger.error(`Worker "${name}" not found`);
-            }
-            else {
-                logger.error(error.response?.data?.message || error.message);
+            if (error?.response?.status === 404) {
+                throw new CliError(`Worker "${name}" not found.`, 'not_found', 'List your workers with: plazbot workers list');
             }
-            process.exit(1);
+            throw error;
         }
-        // Mostrar info del worker
-        logger.warning(`\n  You are about to delete the following worker:`);
-        logger.divider();
-        logger.label('Name', workerInfo.name);
-        logger.label('Type', workerInfo.type);
-        logger.label('Status', workerInfo.status);
-        logger.label('Executions', `${workerInfo.executionCount || 0}`);
-        logger.divider();
-        // Confirmacion
-        if (!options.force) {
-            const confirmed = await askConfirmation(`Delete worker "${name}"? This action cannot be undone (y/N): `);
-            if (!confirmed) {
-                logger.info('\n  Operation cancelled');
-                process.exit(0);
-            }
+        finally {
+            spinner.stop();
+        }
+        if (!isJsonMode()) {
+            logger.warning('You are about to delete this worker:');
+            logger.divider();
+            logger.label('Name', workerInfo?.name ?? name);
+            logger.label('Type', workerInfo?.type ?? '—');
+            logger.label('Status', workerInfo?.status ?? '—');
+            logger.label('Executions', `${workerInfo?.executionCount ?? 0}`);
+            logger.divider();
+        }
+        const ok = await confirmAction(`Delete worker "${name}"? This cannot be undone.`, options);
+        if (!ok) {
+            logger.dim('Aborted.');
+            process.exit(1);
         }
         const deleteSpinner = createSpinner(`Deleting "${name}"...`);
         deleteSpinner.start();
-        await api.delete(`/api/worker/${encodeURIComponent(name)}`);
+        try {
+            const response = await api.delete(`/api/worker/${encodeURIComponent(name)}`);
+            if (response.data?.success === false) {
+                throw new CliError(response.data.message || `Could not delete worker "${name}".`, 'delete_failed');
+            }
+        }
+        catch (error) {
+            deleteSpinner.stop();
+            throw error;
+        }
         deleteSpinner.succeed(`Worker "${name}" deleted`);
+        if (isJsonMode()) {
+            printJson({ name, removed: true });
+            return;
+        }
         console.log();
         if (options.dev) {
             logger.warning('Environment: development');
         }
     }
     catch (error) {
-        const message = error instanceof Error ? error.message : 'Unknown error';
-        logger.error(message);
-        process.exit(1);
+        fail(error);
     }
 });
 addExamples(removeCommand, [
-    { description: 'Delete a worker with confirmation prompt',
-        command: 'plazbot workers remove my-tool' },
+    { description: 'Delete a worker (asks for confirmation)',
+        command: 'plazbot workers remove get-stock' },
     { description: 'Delete without confirmation (scripts/CI)',
-        command: 'plazbot workers remove my-tool -f' },
+        command: 'plazbot workers remove get-stock --yes' },
+    { description: 'Delete and get the result as JSON',
+        command: 'plazbot workers remove get-stock --yes --json' },
 ]);
-function askConfirmation(question) {
-    return new Promise((resolve) => {
-        const rl = readline.createInterface({
-            input: process.stdin,
-            output: process.stdout,
-        });
-        rl.question(question, (answer) => {
-            rl.close();
-            resolve(answer.toLowerCase() === 'y');
-        });
-    });
-}