@playdrop/playdrop-cli 0.3.4-build.1 → 0.3.5-build.1

package/dist/commands/feedback.js

@@ -3,321 +3,189 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.submitFeedback = submitFeedback;
-exports.listFeedback = listFeedback;
-exports.deleteFeedback = deleteFeedback;
+exports.sendFeedback = sendFeedback;
+exports.browseFeedback = browseFeedback;
+exports.deleteFeedbackEntry = deleteFeedbackEntry;
 const node_fs_1 = require("node:fs");
 const node_path_1 = __importDefault(require("node:path"));
-const types_1 = require("@playdrop/types");
 const commandContext_1 = require("../commandContext");
+const errors_1 = require("../errors");
 const messages_1 = require("../messages");
-const http_1 = require("../http");
+const output_1 = require("../output");
 const clientInfo_1 = require("../clientInfo");
 const MAX_FEEDBACK_TITLE_LENGTH = 70;
 const MAX_FEEDBACK_COMMENT_LENGTH = 10000;
 const MAX_FEEDBACK_SOURCE_LENGTH = 40;
-const DEFAULT_SOURCE = 'cli';
-function isNetworkError(error) {
-    if (!error || typeof error !== 'object') {
-        return false;
-    }
-    const candidate = error;
-    if (candidate instanceof TypeError) {
-        return true;
+function parsePositiveInteger(raw, label, command, options = {}) {
+    if (raw === undefined) {
+        return null;
     }
-    const code = typeof candidate.code === 'string' ? candidate.code : undefined;
-    return code === 'ECONNREFUSED' || code === 'ENOTFOUND';
-}
-function parsePositiveInteger(value, context, command) {
-    const parsed = Number.parseInt(value, 10);
-    if (!Number.isInteger(parsed) || parsed <= 0) {
-        (0, messages_1.printErrorWithHelp)(`${context} must be a positive integer.`, [], { command });
+    const parsed = typeof raw === 'number' ? raw : Number.parseInt(String(raw), 10);
+    if (!Number.isInteger(parsed) || parsed < 0 || (!options.zeroAllowed && parsed === 0)) {
+        (0, messages_1.printErrorWithHelp)(`${label} must be ${options.zeroAllowed ? 'zero or a positive integer' : 'a positive integer'}.`, [], { command });
         process.exitCode = 1;
         return null;
     }
     return parsed;
 }
-async function loadCommentFromFile(filePath, command) {
+async function loadCommentFromFile(filePath) {
     const absolute = node_path_1.default.resolve(filePath);
     try {
         const contents = await node_fs_1.promises.readFile(absolute, 'utf8');
         const trimmed = contents.trim();
-        if (!trimmed) {
-            (0, messages_1.printErrorWithHelp)(`File "${filePath}" is empty.`, ['Add feedback text to the file or remove the --file flag.'], { command });
-            process.exitCode = 1;
-            return null;
-        }
-        return trimmed;
-    }
-    catch (error) {
-        const code = error?.code;
-        if (code === 'ENOENT') {
-            (0, messages_1.printErrorWithHelp)(`File "${filePath}" was not found.`, ['Provide a valid path or remove the --file flag.'], { command });
-            process.exitCode = 1;
-            return null;
-        }
-        throw error;
-    }
-}
-function printSubmitValidationError(message) {
-    (0, messages_1.printErrorWithHelp)(message, [
-        'Provide --title and either --comment or --file.',
-        'Example: playdrop-cli feedback --title "Bug report" --comment "Details here."',
-    ], { command: 'feedback' });
-    process.exitCode = 1;
-}
-function parseOutputFormat(raw) {
-    if (!raw || raw.trim().length === 0) {
-        return 'text';
+        return trimmed.length > 0 ? trimmed : null;
     }
-    const normalized = raw.trim().toLowerCase();
-    if (normalized === 'text' || normalized === 'json') {
-        return normalized;
+    catch {
+        return null;
     }
-    return null;
 }
-async function submitFeedback(options = {}) {
+async function sendFeedback(options = {}) {
     const title = typeof options.title === 'string' ? options.title.trim() : '';
     if (!title) {
-        printSubmitValidationError('Feedback title is required.');
+        (0, messages_1.printErrorWithHelp)('Feedback title is required.', ['Use --title <title>.'], { command: 'feedback send' });
+        process.exitCode = 1;
         return;
     }
     if (title.length > MAX_FEEDBACK_TITLE_LENGTH) {
-        printSubmitValidationError(`Feedback title must be ${MAX_FEEDBACK_TITLE_LENGTH} characters or fewer.`);
+        (0, messages_1.printErrorWithHelp)(`Feedback title must be ${MAX_FEEDBACK_TITLE_LENGTH} characters or fewer.`, [], { command: 'feedback send' });
+        process.exitCode = 1;
         return;
     }
     const commentParts = [];
-    if (typeof options.comment === 'string') {
-        const trimmed = options.comment.trim();
-        if (trimmed) {
-            commentParts.push(trimmed);
-        }
+    if (typeof options.comment === 'string' && options.comment.trim().length > 0) {
+        commentParts.push(options.comment.trim());
     }
-    if (typeof options.file === 'string' && options.file.trim()) {
-        const fromFile = await loadCommentFromFile(options.file.trim(), 'feedback');
-        if (!fromFile) {
+    if (typeof options.file === 'string' && options.file.trim().length > 0) {
+        const fileComment = await loadCommentFromFile(options.file.trim());
+        if (!fileComment) {
+            (0, messages_1.printErrorWithHelp)(`Could not read feedback text from "${options.file}".`, ['Provide a readable file or use --comment <text>.'], { command: 'feedback send' });
+            process.exitCode = 1;
             return;
         }
-        commentParts.push(fromFile);
+        commentParts.push(fileComment);
     }
-    const combinedComment = commentParts.join('\n\n').trim();
-    if (!combinedComment) {
-        printSubmitValidationError('Feedback comment text is required.');
+    const comment = commentParts.join('\n\n').trim();
+    if (!comment) {
+        (0, messages_1.printErrorWithHelp)('Feedback text is required.', ['Use --comment <text> or --file <path>.'], { command: 'feedback send' });
+        process.exitCode = 1;
         return;
     }
-    if (combinedComment.length > MAX_FEEDBACK_COMMENT_LENGTH) {
-        printSubmitValidationError(`Feedback comment must be ${MAX_FEEDBACK_COMMENT_LENGTH} characters or fewer.`);
+    if (comment.length > MAX_FEEDBACK_COMMENT_LENGTH) {
+        (0, messages_1.printErrorWithHelp)(`Feedback text must be ${MAX_FEEDBACK_COMMENT_LENGTH} characters or fewer.`, [], { command: 'feedback send' });
+        process.exitCode = 1;
         return;
     }
-    const runtime = (0, clientInfo_1.getRuntimeInfo)();
-    const source = (typeof runtime.client === 'string' && runtime.client.trim().length > 0)
-        ? runtime.client.trim()
-        : DEFAULT_SOURCE;
-    const versionRaw = (0, clientInfo_1.getCliVersion)();
-    const buildRaw = String((0, clientInfo_1.getCliBuild)());
-    const version = versionRaw && versionRaw.trim().length > 0 ? versionRaw.trim() : null;
-    const buildLabel = buildRaw && buildRaw.trim().length > 0 ? buildRaw.trim() : null;
-    try {
-        await (0, commandContext_1.withEnvironment)('feedback', 'Submitting feedback', async ({ client }) => {
-            try {
-                const response = await client.submitFeedback({
-                    title,
-                    comment: combinedComment,
-                    source,
-                    version,
-                    build: buildLabel,
-                });
-                console.log('Feedback recorded, thank you!');
+    await (0, commandContext_1.withEnvironment)('feedback send', 'Sending feedback', async ({ client }) => {
+        try {
+            const runtime = (0, clientInfo_1.getRuntimeInfo)();
+            const source = runtime.client.trim().slice(0, MAX_FEEDBACK_SOURCE_LENGTH) || 'cli';
+            const response = await client.submitFeedback({
+                title,
+                comment,
+                source,
+                version: (0, clientInfo_1.getCliVersion)(),
+                build: String((0, clientInfo_1.getCliBuild)()),
+            });
+            if (options.json) {
+                (0, output_1.printJson)({ success: true, response });
+                return;
             }
-            catch (error) {
-                if (error instanceof http_1.CLIUnsupportedClientError) {
-                    return;
-                }
-                if (error instanceof types_1.UnsupportedClientError) {
-                    (0, http_1.handleUnsupportedError)(error, 'Feedback');
-                    return;
-                }
-                if (error instanceof types_1.ApiError) {
-                    (0, messages_1.printErrorWithHelp)(`Submitting feedback failed with status ${error.status}.`, [
-                        'Double check the provided fields and try again.',
-                    ], { command: 'feedback' });
-                    process.exitCode = 1;
-                    return;
-                }
-                if (isNetworkError(error)) {
-                    (0, messages_1.printNetworkIssue)('Could not reach the Playdrop API to submit feedback.', 'feedback');
-                    process.exitCode = 1;
-                    return;
-                }
+            (0, output_1.printSuccess)('Feedback sent.', ['Next: run "playdrop feedback browse" if you need to review feedback as an admin.']);
+        }
+        catch (error) {
+            const handled = (0, errors_1.handleCommandFailure)(error, 'feedback send', 'Feedback send', {
+                apiMessage: (apiError) => ({
+                    problem: `Feedback send failed with status ${apiError.status}.`,
+                    suggestions: ['Check the provided fields and retry.', 'Run "playdrop auth login" if your session may be stale.'],
+                }),
+            });
+            if (!handled) {
                 throw error;
             }
-        });
-    }
-    catch (error) {
-        if (error instanceof http_1.CLIUnsupportedClientError) {
-            return;
         }
-        throw error;
-    }
+    });
 }
-async function listFeedback(options = {}) {
-    let limit = null;
-    if (typeof options.limit === 'string') {
-        limit = parsePositiveInteger(options.limit, '--limit', 'feedback list');
-        if (limit === null) {
-            return;
-        }
-        if (limit > 100) {
-            (0, messages_1.printErrorWithHelp)('--limit must not exceed 100.', [], { command: 'feedback list' });
-            process.exitCode = 1;
-            return;
-        }
-    }
-    let offset = null;
-    if (typeof options.offset === 'string') {
-        const parsed = Number.parseInt(options.offset, 10);
-        if (!Number.isInteger(parsed) || parsed < 0) {
-            (0, messages_1.printErrorWithHelp)('--offset must be zero or a positive integer.', [], { command: 'feedback list' });
-            process.exitCode = 1;
-            return;
-        }
-        offset = parsed;
+async function browseFeedback(options = {}) {
+    const limit = options.limit !== undefined ? parsePositiveInteger(options.limit, '--limit', 'feedback browse') : null;
+    if (options.limit !== undefined && limit === null) {
+        return;
     }
-    const source = typeof options.source === 'string' && options.source.trim().length > 0
-        ? options.source.trim()
-        : undefined;
-    if (source && source.length > MAX_FEEDBACK_SOURCE_LENGTH) {
-        (0, messages_1.printErrorWithHelp)(`Source filter must be ${MAX_FEEDBACK_SOURCE_LENGTH} characters or fewer.`, [], { command: 'feedback list' });
-        process.exitCode = 1;
+    const offset = options.offset !== undefined ? parsePositiveInteger(options.offset, '--offset', 'feedback browse', { zeroAllowed: true }) : null;
+    if (options.offset !== undefined && offset === null) {
         return;
     }
-    let userId;
-    if (typeof options.userId === 'string' && options.userId.trim().length > 0) {
-        const parsed = parsePositiveInteger(options.userId.trim(), '--user-id', 'feedback list');
-        if (parsed === null) {
-            return;
-        }
-        userId = parsed;
+    const userId = options.userId !== undefined ? parsePositiveInteger(options.userId, '--user-id', 'feedback browse') : null;
+    if (options.userId !== undefined && userId === null) {
+        return;
     }
-    const format = parseOutputFormat(options.format);
-    if (!format) {
-        (0, messages_1.printErrorWithHelp)(`Format "${options.format ?? ''}" is not supported.`, [
-            'Use "--format text" (default) or "--format json".',
-        ], { command: 'feedback list' });
+    const source = typeof options.source === 'string' && options.source.trim().length > 0 ? options.source.trim() : undefined;
+    if (source && source.length > MAX_FEEDBACK_SOURCE_LENGTH) {
+        (0, messages_1.printErrorWithHelp)(`Source filter must be ${MAX_FEEDBACK_SOURCE_LENGTH} characters or fewer.`, [], { command: 'feedback browse' });
         process.exitCode = 1;
         return;
     }
-    try {
-        await (0, commandContext_1.withEnvironment)('feedback list', 'Listing feedback', async ({ client }) => {
-            try {
-                const response = await client.listFeedback({
-                    limit: limit ?? undefined,
-                    offset: offset ?? undefined,
-                    source,
-                    userId,
-                });
-                const entries = Array.isArray(response.feedback) ? response.feedback : [];
-                if (format === 'json') {
-                    console.log(JSON.stringify({ feedback: entries, pagination: response.pagination }, null, 2));
-                    return;
-                }
-                if (entries.length === 0) {
-                    console.log('No feedback entries found.');
-                    return;
-                }
-                for (const entry of entries) {
-                    console.log(`#${entry.id} ${entry.createdAt} ${entry.username}`);
-                    console.log(` Title: ${entry.title}`);
-                    console.log(` Comment: ${entry.comment}`);
-                }
-                const meta = response.pagination;
-                if (meta?.hasMore) {
-                    console.log(`More results available. Next offset: ${meta.offset + entries.length}`);
-                }
+    await (0, commandContext_1.withEnvironment)('feedback browse', 'Browsing feedback', async ({ client }) => {
+        try {
+            const response = await client.listFeedback({
+                limit: limit ?? undefined,
+                offset: offset ?? undefined,
+                source,
+                userId: userId ?? undefined,
+            });
+            const items = response.feedback ?? [];
+            if (options.json) {
+                (0, output_1.printJson)({ items, pagination: response.pagination });
+                return;
+            }
+            if (items.length === 0) {
+                console.log('No feedback entries found.');
+                return;
             }
-            catch (error) {
-                if (error instanceof http_1.CLIUnsupportedClientError) {
-                    return;
-                }
-                if (error instanceof types_1.UnsupportedClientError) {
-                    (0, http_1.handleUnsupportedError)(error, 'Feedback list');
-                    return;
-                }
-                if (error instanceof types_1.ApiError) {
-                    (0, messages_1.printErrorWithHelp)(`Listing feedback failed with status ${error.status}.`, [
-                        'Ensure you are logged in with an admin account.',
-                    ], { command: 'feedback list' });
-                    process.exitCode = 1;
-                    return;
-                }
-                if (isNetworkError(error)) {
-                    (0, messages_1.printNetworkIssue)('Could not reach the Playdrop API to list feedback.', 'feedback list');
-                    process.exitCode = 1;
-                    return;
-                }
+            console.log('Feedback:\n');
+            for (const item of items) {
+                console.log(`#${item.id} | ${item.username} | ${(0, output_1.formatTimestamp)(item.createdAt)}`);
+                console.log(`  ${item.title}`);
+                console.log(`  ${item.comment}`);
+            }
+            console.log('\nNext: run "playdrop feedback delete <id>" if you need to remove one entry.');
+        }
+        catch (error) {
+            const handled = (0, errors_1.handleCommandFailure)(error, 'feedback browse', 'Feedback browse', {
+                apiMessage: (apiError) => ({
+                    problem: `Feedback browse failed with status ${apiError.status}.`,
+                    suggestions: ['Ensure you are logged in with an account that can review feedback.'],
+                }),
+            });
+            if (!handled) {
                 throw error;
             }
-        });
-    }
-    catch (error) {
-        if (error instanceof http_1.CLIUnsupportedClientError) {
-            return;
         }
-        throw error;
-    }
+    });
 }
-async function deleteFeedback(id) {
-    if (!id || !id.trim()) {
-        (0, messages_1.printErrorWithHelp)('Feedback id is required.', ['Usage: playdrop-cli feedback delete <id>'], { command: 'feedback delete' });
-        process.exitCode = 1;
+async function deleteFeedbackEntry(rawId, options = {}) {
+    const id = parsePositiveInteger(rawId, 'Feedback id', 'feedback delete');
+    if (id === null) {
         return;
     }
-    const numericId = parsePositiveInteger(id.trim(), 'Feedback id', 'feedback delete');
-    if (numericId === null) {
-        return;
-    }
-    try {
-        await (0, commandContext_1.withEnvironment)('feedback delete', 'Deleting feedback', async ({ client }) => {
-            try {
-                const response = await client.deleteFeedback(numericId);
-                if (response?.success) {
-                    console.log(`Feedback #${numericId} deleted.`);
-                }
-                else {
-                    console.log(`Feedback #${numericId} deleted.`);
-                }
+    await (0, commandContext_1.withEnvironment)('feedback delete', 'Deleting feedback', async ({ client }) => {
+        try {
+            const response = await client.deleteFeedback(id);
+            if (options.json) {
+                (0, output_1.printJson)({ success: Boolean(response.success), id });
+                return;
             }
-            catch (error) {
-                if (error instanceof http_1.CLIUnsupportedClientError) {
-                    return;
-                }
-                if (error instanceof types_1.UnsupportedClientError) {
-                    (0, http_1.handleUnsupportedError)(error, 'Feedback delete');
-                    return;
-                }
-                if (error instanceof types_1.ApiError) {
-                    if (error.status === 404) {
-                        (0, messages_1.printErrorWithHelp)(`Feedback #${numericId} was not found.`, [], { command: 'feedback delete' });
-                    }
-                    else {
-                        (0, messages_1.printErrorWithHelp)(`Deleting feedback failed with status ${error.status}.`, [], { command: 'feedback delete' });
-                    }
-                    process.exitCode = 1;
-                    return;
-                }
-                if (isNetworkError(error)) {
-                    (0, messages_1.printNetworkIssue)('Could not reach the Playdrop API to delete feedback.', 'feedback delete');
-                    process.exitCode = 1;
-                    return;
-                }
+            (0, output_1.printSuccess)(`Feedback #${id} deleted.`, ['Next: run "playdrop feedback browse" to review the remaining entries.']);
+        }
+        catch (error) {
+            const handled = (0, errors_1.handleCommandFailure)(error, 'feedback delete', 'Feedback delete', {
+                apiMessage: (apiError) => ({
+                    problem: `Feedback delete failed with status ${apiError.status}.`,
+                    suggestions: ['Check the id and your permissions, then retry.'],
+                }),
+            });
+            if (!handled) {
                 throw error;
             }
-        });
-    }
-    catch (error) {
-        if (error instanceof http_1.CLIUnsupportedClientError) {
-            return;
         }
-        throw error;
-    }
+    });
 }

package/dist/commands/format.js

@@ -8,7 +8,7 @@ const validate_1 = require("../apps/validate");
 async function format(pathOrName) {
     const selection = (0, taskSelection_1.selectTasks)(pathOrName);
     if (selection.errors.length > 0) {
-        (0, taskSelection_1.reportTaskErrors)(selection.errors, 'format');
+        (0, taskSelection_1.reportTaskErrors)(selection.errors, 'project format');
         process.exitCode = 1;
         return;
     }
@@ -23,28 +23,21 @@ async function format(pathOrName) {
         try {
             if (task.kind === 'app') {
                 const ran = await (0, validate_1.runFormatScript)(task);
+                if (!ran) {
+                    throw new Error(`project format requires a package.json "format" script for app "${task.name}".`);
+                }
                 const entry = {
                     action: 'format',
-                    status: ran ? 'success' : 'skipped',
+                    status: 'success',
                     entityType: 'app',
                     entityId: task.name,
                     catalogue: task.cataloguePath,
-                    detail: ran ? undefined : 'package.json missing "format" script',
                 };
                 results.push(entry);
                 console.log((0, uploadLog_1.formatTaskLogLine)(entry));
             }
             else {
-                const entry = {
-                    action: 'format',
-                    status: 'skipped',
-                    entityType: task.kind,
-                    entityId: task.name,
-                    catalogue: task.cataloguePath,
-                    detail: `no format step for ${task.kind}`,
-                };
-                results.push(entry);
-                console.log((0, uploadLog_1.formatTaskLogLine)(entry));
+                throw new Error(`project format does not support ${task.kind} "${task.name}".`);
             }
         }
         catch (error) {

package/dist/commands/generation.d.ts

@@ -21,6 +21,7 @@ type AiGenerateOptions = {
     texture?: boolean;
     timeoutSeconds?: string | number;
     pollIntervalMs?: string | number;
+    json?: boolean;
 };
 type AiListOptions = {
     type?: string;
@@ -29,5 +30,15 @@ type AiListOptions = {
     format?: string;
 };
 export declare function generate(typeInput: string, promptInput: string, options?: AiGenerateOptions): Promise<void>;
+type AiJobBrowseOptions = {
+    type?: string;
+    limit?: string | number;
+    offset?: string | number;
+    json?: boolean;
+};
+export declare function browseGenerationJobs(options?: AiJobBrowseOptions): Promise<void>;
+export declare function showGenerationJob(id: string | undefined, options?: {
+    json?: boolean;
+}): Promise<void>;
 export declare function generations(options?: AiListOptions): Promise<void>;
 export {};