@devicecloud.dev/dcd 4.2.0 → 4.2.2

package/dist/commands/list.d.ts

@@ -0,0 +1,38 @@
+import { Command } from '@oclif/core';
+type UploadListItem = {
+    consoleUrl: string;
+    created_at: string;
+    id: string;
+    name: null | string;
+};
+type ListResponse = {
+    limit: number;
+    offset: number;
+    total: number;
+    uploads: UploadListItem[];
+};
+export default class List extends Command {
+    static description: string;
+    static enableJsonFlag: boolean;
+    static examples: string[];
+    static flags: {
+        apiKey: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        apiUrl: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        from: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        json: import("@oclif/core/lib/interfaces").BooleanFlag<boolean>;
+        limit: import("@oclif/core/lib/interfaces").OptionFlag<number, import("@oclif/core/lib/interfaces").CustomOptions>;
+        name: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+        offset: import("@oclif/core/lib/interfaces").OptionFlag<number, import("@oclif/core/lib/interfaces").CustomOptions>;
+        to: import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
+    };
+    run(): Promise<ListResponse | void>;
+    /**
+     * Detects if the provided name parameter likely underwent shell expansion
+     * Warns the user if shell expansion is detected
+     * @param name - The name parameter to check for shell expansion
+     * @returns void
+     */
+    private detectShellExpansion;
+    private displayResults;
+}
+export {};

package/dist/commands/list.js

@@ -0,0 +1,143 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const core_1 = require("@oclif/core");
+const constants_1 = require("../constants");
+const api_gateway_1 = require("../gateways/api-gateway");
+const styling_1 = require("../utils/styling");
+class List extends core_1.Command {
+    static description = 'List recent flow uploads for your organization';
+    static enableJsonFlag = true;
+    static examples = [
+        '<%= config.bin %> <%= command.id %>',
+        '<%= config.bin %> <%= command.id %> --limit 10',
+        '<%= config.bin %> <%= command.id %> --name "nightly-*"  # Quote wildcards to prevent shell expansion!',
+        '<%= config.bin %> <%= command.id %> --from 2024-01-01 --to 2024-01-31',
+        '<%= config.bin %> <%= command.id %> --json',
+    ];
+    static flags = {
+        apiKey: constants_1.flags.apiKey,
+        apiUrl: constants_1.flags.apiUrl,
+        from: core_1.Flags.string({
+            description: 'Filter uploads created on or after this date (ISO 8601 format, e.g., 2024-01-01)',
+        }),
+        json: core_1.Flags.boolean({
+            description: 'Output in JSON format',
+        }),
+        limit: core_1.Flags.integer({
+            default: 20,
+            description: 'Maximum number of uploads to return',
+        }),
+        name: core_1.Flags.string({
+            description: 'Filter by upload name (supports * wildcard, e.g., "nightly-*"). IMPORTANT: Always quote wildcards to prevent shell expansion!',
+        }),
+        offset: core_1.Flags.integer({
+            default: 0,
+            description: 'Number of uploads to skip (for pagination)',
+        }),
+        to: core_1.Flags.string({
+            description: 'Filter uploads created on or before this date (ISO 8601 format, e.g., 2024-01-31)',
+        }),
+    };
+    async run() {
+        const { flags } = await this.parse(List);
+        const { apiKey: apiKeyFlag, apiUrl, from, json, limit, name, offset, to, } = flags;
+        const apiKey = apiKeyFlag || process.env.DEVICE_CLOUD_API_KEY;
+        if (!apiKey) {
+            this.error('API key is required. Please provide it via --api-key flag or DEVICE_CLOUD_API_KEY environment variable.');
+            return;
+        }
+        // Validate date formats if provided
+        if (from && Number.isNaN(Date.parse(from))) {
+            this.error('Invalid --from date format. Please use ISO 8601 format (e.g., 2024-01-01).');
+            return;
+        }
+        if (to && Number.isNaN(Date.parse(to))) {
+            this.error('Invalid --to date format. Please use ISO 8601 format (e.g., 2024-01-31).');
+            return;
+        }
+        // Detect potential shell expansion of wildcards
+        if (name) {
+            this.detectShellExpansion(name);
+        }
+        try {
+            const response = await api_gateway_1.ApiGateway.listUploads(apiUrl, apiKey, {
+                from,
+                limit,
+                name,
+                offset,
+                to,
+            });
+            if (json) {
+                return response;
+            }
+            this.displayResults(response);
+        }
+        catch (error) {
+            this.error(`Failed to list uploads: ${error.message}`);
+        }
+    }
+    /**
+     * Detects if the provided name parameter likely underwent shell expansion
+     * Warns the user if shell expansion is detected
+     * @param name - The name parameter to check for shell expansion
+     * @returns void
+     */
+    detectShellExpansion(name) {
+        const shellExpansionIndicators = [
+            // Contains file path separators (likely expanded to file paths)
+            name.includes('/') || name.includes('\\'),
+            // Contains file extensions (likely expanded to filenames)
+            /\.(yaml|yml|json|txt|md|ts|js|py|sh)$/i.test(name),
+            // Looks like multiple space-separated filenames (shell expanded glob to multiple files)
+            name.includes(' ') && !name.includes('*') && !name.includes('?'),
+        ];
+        if (shellExpansionIndicators.some(Boolean)) {
+            this.warn(`\nThe --name parameter appears to have been expanded by your shell: "${name}"\n` +
+                'Wildcards like * should be quoted to prevent shell expansion.\n' +
+                'Examples:\n' +
+                '  ✓ Correct:   dcd list --name "nightly-*"\n' +
+                '  ✓ Correct:   dcd list --name \'nightly-*\'\n' +
+                '  ✗ Incorrect: dcd list --name nightly-*\n');
+        }
+    }
+    displayResults(response) {
+        const { uploads, total, limit, offset } = response;
+        if (uploads.length === 0) {
+            this.log('\nNo uploads found matching your criteria.\n');
+            return;
+        }
+        this.log((0, styling_1.sectionHeader)('Recent Uploads'));
+        this.log(`   ${styling_1.colors.dim('Showing')} ${uploads.length} ${styling_1.colors.dim('of')} ${total} ${styling_1.colors.dim('uploads')}`);
+        if (offset > 0) {
+            this.log(`   ${styling_1.colors.dim('(offset:')} ${offset}${styling_1.colors.dim(')')}`);
+        }
+        this.log('');
+        for (const upload of uploads) {
+            const date = new Date(upload.created_at);
+            const formattedDate = date.toLocaleDateString('en-US', {
+                day: 'numeric',
+                hour: '2-digit',
+                minute: '2-digit',
+                month: 'short',
+                year: 'numeric',
+            });
+            // Upload name
+            const displayName = upload.name || styling_1.colors.dim('(unnamed)');
+            this.log(`   ${styling_1.colors.bold(displayName)}`);
+            // Upload ID and date
+            this.log(`      ${styling_1.colors.dim('ID:')} ${(0, styling_1.formatId)(upload.id)}`);
+            this.log(`      ${styling_1.colors.dim('Created:')} ${formattedDate}`);
+            // Console URL
+            this.log(`      ${styling_1.colors.dim('Console:')} ${(0, styling_1.formatUrl)(upload.consoleUrl)}`);
+            this.log('');
+        }
+        // Pagination hint
+        if (total > offset + uploads.length) {
+            const remaining = total - (offset + uploads.length);
+            this.log(`   ${styling_1.colors.dim('Use')} --offset ${offset + limit} ${styling_1.colors.dim('to see the next')} ${Math.min(remaining, limit)} ${styling_1.colors.dim('uploads')}\n`);
+        }
+        // Hint about getting detailed status
+        this.log(`   ${styling_1.colors.dim('Tip: Use')} dcd status --upload-id <id> ${styling_1.colors.dim('for detailed test results')}\n`);
+    }
+}
+exports.default = List;

package/dist/commands/status.d.ts

@@ -9,9 +9,12 @@ type StatusResponse = {
         message: string;
     };
     consoleUrl?: string;
+    createdAt?: string;
     error?: string;
+    name?: string;
     status: 'CANCELLED' | 'FAILED' | 'PASSED' | 'PENDING' | 'RUNNING';
     tests: {
+        createdAt?: string;
         durationSeconds?: number;
         failReason?: string;
         name: string;
@@ -31,5 +34,11 @@ export default class Status extends Command {
         'upload-id': import("@oclif/core/lib/interfaces").OptionFlag<string, import("@oclif/core/lib/interfaces").CustomOptions>;
     };
     run(): Promise<StatusResponse | void>;
+    /**
+     * Format an ISO date string to a human-readable local date/time
+     * @param isoString - ISO 8601 date string
+     * @returns Formatted local date/time string
+     */
+    private formatDateTime;
 }
 export {};

package/dist/commands/status.js

@@ -47,8 +47,10 @@ class Status extends core_1.Command {
         }
         let lastError = null;
         let status = null;
+        let attemptsMade = 0;
         for (let attempt = 1; attempt <= 5; attempt++) {
             try {
+                attemptsMade = attempt;
                 status = (await api_gateway_1.ApiGateway.getUploadStatus(apiUrl, apiKey, {
                     name,
                     uploadId,
@@ -57,57 +59,109 @@ class Status extends core_1.Command {
             }
             catch (error) {
                 lastError = error;
-                if (attempt < 5) {
+                // Check if this is a retryable error (network/timeout issues)
+                // Non-retryable errors: 4xx client errors (bad request, not found, unauthorized, forbidden)
+                const isNetworkError = lastError.name === 'NetworkError' ||
+                    (error instanceof TypeError && lastError.message === 'fetch failed');
+                const isClientError = lastError.message.includes('Invalid request:') ||
+                    lastError.message.includes('Resource not found') ||
+                    lastError.message.includes('Authentication failed') ||
+                    lastError.message.includes('Access denied') ||
+                    lastError.message.includes('Invalid API key') ||
+                    lastError.message.includes('Rate limit exceeded');
+                // Don't retry client errors - they won't succeed on retry
+                if (isClientError) {
+                    break;
+                }
+                // Only retry network errors
+                if (attempt < 5 && isNetworkError) {
                     this.log(`Network error on attempt ${attempt}/5. Retrying...`);
                     await new Promise((resolve) => {
                         setTimeout(resolve, 1000 * attempt);
                     });
                 }
+                else if (attempt < 5) {
+                    // For other errors (server errors), retry but with different message
+                    this.log(`Request failed on attempt ${attempt}/5. Retrying...`);
+                    await new Promise((resolve) => {
+                        setTimeout(resolve, 1000 * attempt);
+                    });
+                }
             }
         }
         if (!status) {
+            // Check if this was a client error (non-retryable)
+            const isClientError = lastError && (lastError.message.includes('Invalid request:') ||
+                lastError.message.includes('Resource not found') ||
+                lastError.message.includes('Authentication failed') ||
+                lastError.message.includes('Access denied') ||
+                lastError.message.includes('Invalid API key') ||
+                lastError.message.includes('Rate limit exceeded'));
+            if (isClientError) {
+                // For client errors, show the error immediately without connectivity check
+                const errorMessage = lastError?.message || 'Unknown error';
+                if (json) {
+                    return {
+                        status: 'FAILED',
+                        error: errorMessage,
+                        attempts: attemptsMade,
+                        tests: [],
+                    };
+                }
+                this.error(errorMessage);
+            }
             // Check if the failure is due to internet connectivity issues
             const connectivityCheck = await (0, connectivity_1.checkInternetConnectivity)();
             let errorMessage;
             if (connectivityCheck.connected) {
-                errorMessage = `Failed to get status after 5 attempts. Internet appears functional but unable to reach API. Last error: ${lastError?.message || 'Unknown error'}`;
+                errorMessage = `Failed to get status after ${attemptsMade} attempt${attemptsMade > 1 ? 's' : ''}. Internet appears functional but unable to reach API. Last error: ${lastError?.message || 'Unknown error'}`;
             }
             else {
                 // Build detailed error message with endpoint diagnostics
                 const endpointDetails = connectivityCheck.endpointResults
                     .map((r) => `  - ${r.endpoint}: ${r.error} (${r.latencyMs}ms)`)
                     .join('\n');
-                errorMessage = `Failed to get status after 5 attempts.\n\nInternet connectivity check failed - all test endpoints unreachable:\n${endpointDetails}\n\nPlease verify your network connection and DNS resolution.\nLast API error: ${lastError?.message || 'Unknown error'}`;
+                errorMessage = `Failed to get status after ${attemptsMade} attempt${attemptsMade > 1 ? 's' : ''}.\n\nInternet connectivity check failed - all test endpoints unreachable:\n${endpointDetails}\n\nPlease verify your network connection and DNS resolution.\nLast API error: ${lastError?.message || 'Unknown error'}`;
             }
             if (json) {
                 return {
-                    attempts: 5,
+                    status: 'FAILED',
+                    error: errorMessage,
+                    attempts: attemptsMade,
                     connectivityCheck: {
                         connected: connectivityCheck.connected,
                         endpointResults: connectivityCheck.endpointResults,
                         message: connectivityCheck.message,
                     },
-                    error: errorMessage,
-                    status: 'FAILED',
                     tests: [],
                 };
             }
             this.error(errorMessage);
-            return;
         }
         try {
             if (json) {
-                return status;
+                // Reconstruct object to ensure tests appears at the bottom
+                const { tests, ...rest } = status;
+                return {
+                    ...rest,
+                    tests,
+                };
             }
             this.log((0, styling_1.sectionHeader)('Upload Status'));
             // Display overall status
             this.log(`   ${(0, styling_1.formatStatus)(status.status)}`);
+            if (status.name) {
+                this.log(`   ${styling_1.colors.dim('Name:')} ${styling_1.colors.bold(status.name)}`);
+            }
             if (status.uploadId) {
                 this.log(`   ${styling_1.colors.dim('Upload ID:')} ${(0, styling_1.formatId)(status.uploadId)}`);
             }
             if (status.appBinaryId) {
                 this.log(`   ${styling_1.colors.dim('Binary ID:')} ${(0, styling_1.formatId)(status.appBinaryId)}`);
             }
+            if (status.createdAt) {
+                this.log(`   ${styling_1.colors.dim('Created:')} ${this.formatDateTime(status.createdAt)}`);
+            }
             if (status.consoleUrl) {
                 this.log(`   ${styling_1.colors.dim('Console:')} ${(0, styling_1.formatUrl)(status.consoleUrl)}`);
             }
@@ -121,6 +175,9 @@ class Status extends core_1.Command {
                     if (item.durationSeconds) {
                         this.log(`      ${styling_1.colors.dim('Duration:')} ${(0, methods_1.formatDurationSeconds)(item.durationSeconds)}`);
                     }
+                    if (item.createdAt) {
+                        this.log(`      ${styling_1.colors.dim('Created:')} ${this.formatDateTime(item.createdAt)}`);
+                    }
                     this.log('');
                 }
             }
@@ -129,5 +186,19 @@ class Status extends core_1.Command {
             this.error(`Failed to get status: ${error.message}`);
         }
     }
+    /**
+     * Format an ISO date string to a human-readable local date/time
+     * @param isoString - ISO 8601 date string
+     * @returns Formatted local date/time string
+     */
+    formatDateTime(isoString) {
+        try {
+            const date = new Date(isoString);
+            return date.toLocaleString();
+        }
+        catch {
+            return isoString;
+        }
+    }
 }
 exports.default = Status;

package/dist/config/flags/output.flags.js

@@ -30,7 +30,7 @@ exports.outputFlags = {
         description: 'Enable detailed debug logging for troubleshooting issues',
     }),
     'download-artifacts': core_1.Flags.string({
-        description: 'Download a zip containing the logs, screenshots and videos for each result in this run. You will be debited a $0.01 egress fee for each result. Options: ALL (everything), FAILED (failures only).',
+        description: 'Download a zip containing the logs, screenshots and videos for each result in this run. Options: ALL (everything), FAILED (failures only).',
         options: ['ALL', 'FAILED'],
     }),
     'dry-run': core_1.Flags.boolean({

package/dist/gateways/api-gateway.d.ts

@@ -1,5 +1,12 @@
 import { TAppMetadata } from '../types';
 export declare const ApiGateway: {
+    /**
+     * Enhances generic "fetch failed" errors with more specific diagnostic information
+     * @param error - The original TypeError from fetch
+     * @param url - The URL that was being fetched
+     * @returns Enhanced error with diagnostic information
+     */
+    enhanceFetchError(error: TypeError, url: string): Error;
     /**
      * Standardized error handling for API responses
      * @param res - The fetch response object
@@ -38,14 +45,34 @@ export declare const ApiGateway: {
         name?: string;
         uploadId?: string;
     }): Promise<{
+        createdAt?: string;
+        name?: string;
         status: "CANCELLED" | "FAILED" | "PASSED" | "PENDING";
         tests: Array<{
+            createdAt?: string;
             durationSeconds?: number;
             failReason?: string;
             name: string;
             status: "CANCELLED" | "FAILED" | "PASSED" | "PENDING";
         }>;
     }>;
+    listUploads(baseUrl: string, apiKey: string, options?: {
+        from?: string;
+        limit?: number;
+        name?: string;
+        offset?: number;
+        to?: string;
+    }): Promise<{
+        limit: number;
+        offset: number;
+        total: number;
+        uploads: Array<{
+            consoleUrl: string;
+            created_at: string;
+            id: string;
+            name: null | string;
+        }>;
+    }>;
     uploadFlow(baseUrl: string, apiKey: string, testFormData: FormData): Promise<{
         message?: string;
         results?: import("../types/generated/schema.types").components["schemas"]["IDBResult"][];