bktide 1.0.1755568192 → 1.0.1755639164

package/dist/formatters/build-detail/PlainTextFormatter.js

@@ -140,10 +140,17 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         // Header line
         lines.push(this.formatHeader(build));
         lines.push(this.formatCommitInfo(build));
+        lines.push(''); // Blank line after commit info
         // Show annotations summary if present
         if (build.annotations?.edges?.length > 0) {
-            lines.push('');
             lines.push(this.formatAnnotationSummary(build.annotations.edges));
+        }
+        // Jobs summary
+        if (build.jobs?.edges?.length > 0) {
+            if (build.annotations?.edges?.length > 0) {
+                lines.push(''); // Add space between annotations and steps
+            }
+            lines.push(this.formatJobSummary(build.jobs, build.state));
             if (!options?.annotations) {
                 lines.push('');
                 const tips = formatTips(['Use --annotations to view annotation details'], TipStyle.GROUPED);
@@ -163,18 +170,18 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         lines.push(this.formatHeader(build));
         lines.push(this.formatCommitInfo(build));
         lines.push('');
-        // Failed jobs summary
-        const failedJobs = this.getFailedJobs(build.jobs?.edges);
         const allHints = [];
-        if (failedJobs.length > 0) {
-            const { summary, hints } = this.formatFailedJobsSummaryWithHints(failedJobs, options);
-            lines.push(summary);
-            allHints.push(...hints);
-        }
-        // Annotation summary
+        // Annotation summary (first, as it appears first in UI)
         if (build.annotations?.edges?.length > 0) {
             lines.push(this.formatAnnotationSummary(build.annotations.edges));
         }
+        // Jobs summary
+        if (build.jobs?.edges?.length > 0) {
+            if (build.annotations?.edges?.length > 0) {
+                lines.push(''); // Add space between annotations and steps
+            }
+            lines.push(this.formatJobSummary(build.jobs, build.state));
+        }
         // Show detailed job info if requested
         if (options?.jobs || options?.failed) {
             lines.push('');
@@ -186,12 +193,17 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             lines.push(this.formatAnnotationDetails(build.annotations.edges));
         }
         // Collect all hints for more info
+        const failedJobs = this.getFailedJobs(build.jobs?.edges);
         if (!options?.failed && failedJobs.length > 0) {
             allHints.push('Use --failed to show failure details');
         }
         if (!options?.annotations && build.annotations?.edges?.length > 0) {
             allHints.push('Use --annotations to view annotation details');
         }
+        // Add hint about incomplete step data if truncated
+        if (!options?.jobs && build.jobs?.pageInfo?.hasNextPage) {
+            allHints.push('Use --jobs to fetch all step data (currently showing first 100 only)');
+        }
         // Display all hints together
         if (allHints.length > 0) {
             lines.push('');
@@ -205,9 +217,17 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         lines.push(this.formatHeader(build));
         lines.push(this.formatCommitInfo(build));
         lines.push('');
-        // Progress information
-        const jobStats = this.getJobStats(build.jobs?.edges);
-        lines.push(`Progress: ${SEMANTIC_COLORS.count(String(jobStats.completed))}/${jobStats.total} complete, ${SEMANTIC_COLORS.info(String(jobStats.running))} running, ${jobStats.queued} queued`);
+        // Annotations first (if any)
+        if (build.annotations?.edges?.length > 0) {
+            lines.push(this.formatAnnotationSummary(build.annotations.edges));
+        }
+        // Jobs summary with progress
+        if (build.jobs?.edges?.length > 0) {
+            if (build.annotations?.edges?.length > 0) {
+                lines.push(''); // Add space between annotations and steps
+            }
+            lines.push(this.formatJobSummary(build.jobs, build.state));
+        }
         // Show running jobs
         const runningJobs = this.getRunningJobs(build.jobs?.edges);
         if (runningJobs.length > 0) {
@@ -237,20 +257,22 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         lines.push(this.formatHeader(build));
         lines.push(this.formatCommitInfo(build));
         lines.push('');
+        // Annotations first (if any)
+        if (build.annotations?.edges?.length > 0) {
+            lines.push(this.formatAnnotationSummary(build.annotations.edges));
+        }
+        // Jobs summary
+        if (build.jobs?.edges?.length > 0) {
+            if (build.annotations?.edges?.length > 0) {
+                lines.push(''); // Add space between annotations and steps
+            }
+            lines.push(this.formatJobSummary(build.jobs, build.state));
+        }
         // Blocked information
         const blockedJobs = this.getBlockedJobs(build.jobs?.edges);
         if (blockedJobs.length > 0) {
-            lines.push(`${getProgressIcon('BLOCKED_MESSAGE')} Blocked: "${blockedJobs[0].node.label}" (manual unblock required)`);
-        }
-        // Show jobs summary
-        const jobStats = this.getJobStats(build.jobs?.edges);
-        if (jobStats.completed > 0) {
-            lines.push(`${getStateIcon('PASSED')} ${jobStats.completed} jobs passed before block`);
-        }
-        // Annotation summary
-        if (build.annotations?.edges?.length > 0) {
             lines.push('');
-            lines.push(this.formatAnnotationSummary(build.annotations.edges));
+            lines.push(`${getProgressIcon('BLOCKED_MESSAGE')} Blocked: "${blockedJobs[0].node.label}" (manual unblock required)`);
         }
         // Show job details if requested
         if (options?.jobs) {
@@ -270,14 +292,23 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         lines.push(this.formatHeader(build));
         lines.push(this.formatCommitInfo(build));
         lines.push('');
+        // Annotations first (if any)
+        if (build.annotations?.edges?.length > 0) {
+            lines.push(this.formatAnnotationSummary(build.annotations.edges));
+        }
+        // Jobs summary
+        if (build.jobs?.edges?.length > 0) {
+            if (build.annotations?.edges?.length > 0) {
+                lines.push(''); // Add space between annotations and steps
+            }
+            lines.push(this.formatJobSummary(build.jobs, build.state));
+        }
         // Canceled information
         if (build.createdBy) {
+            lines.push('');
             const creator = build.createdBy.name || build.createdBy.email;
             lines.push(`Canceled by: ${creator}`);
         }
-        // Show jobs summary
-        const jobStats = this.getJobStats(build.jobs?.edges);
-        lines.push(`Completed: ${jobStats.completed}/${jobStats.total} jobs before cancellation`);
         // Show job details if requested
         if (options?.jobs) {
             lines.push('');
@@ -323,8 +354,8 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             lines.push(`  Triggered from: ${build.triggeredFrom.pipeline?.name} #${build.triggeredFrom.number}`);
         }
         lines.push('');
-        // Jobs section
-        lines.push('Jobs:');
+        // Steps section
+        lines.push('Steps:');
         lines.push(this.formatJobDetails(build.jobs?.edges, { ...options, full: true }));
         // Annotations section
         if (build.annotations?.edges?.length > 0) {
@@ -340,15 +371,31 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         const coloredIcon = this.colorizeStatusIcon(statusIcon, build.state);
         const stateFormatted = formatBuildStatus(build.state, { useSymbol: false });
         const duration = this.formatDuration(build);
-        const age = this.formatAge(build.createdAt);
-        const branch = SEMANTIC_COLORS.identifier(build.branch);
-        return `${coloredIcon} ${SEMANTIC_COLORS.label(`#${build.number}`)} ${stateFormatted} • ${duration} • ${branch} • ${age}`;
+        // Get first line of commit message
+        const message = build.message || 'No commit message';
+        const firstLineMessage = message.split('\n')[0];
+        const truncatedMessage = firstLineMessage.length > 80 ? firstLineMessage.substring(0, 77) + '...' : firstLineMessage;
+        return `${coloredIcon} ${stateFormatted} ${truncatedMessage} ${SEMANTIC_COLORS.dim(`#${build.number}`)} ${SEMANTIC_COLORS.dim(duration)}`;
     }
     formatCommitInfo(build) {
         const shortSha = build.commit ? build.commit.substring(0, 7) : 'unknown';
-        const message = build.message || 'No commit message';
-        const truncatedMessage = message.length > 60 ? message.substring(0, 57) + '...' : message;
-        return `   "${truncatedMessage}" (${shortSha})`;
+        const branch = SEMANTIC_COLORS.identifier(build.branch);
+        const age = this.formatAge(build.createdAt);
+        // Get author information
+        const author = build.createdBy?.name || build.createdBy?.email || 'Unknown';
+        // Calculate indentation to align with commit message
+        // Map each state to its proper indentation (icon + space + state text + space)
+        const indentMap = {
+            'PASSED': 9, // ✓ PASSED 
+            'FAILED': 9, // ✗ FAILED 
+            'RUNNING': 10, // ⟳ RUNNING 
+            'BLOCKED': 10, // ◼ BLOCKED 
+            'CANCELED': 11, // ⊘ CANCELED 
+            'SCHEDULED': 12, // ⏱ SCHEDULED 
+            'SKIPPED': 10, // ⊙ SKIPPED 
+        };
+        const indent = ' '.repeat(indentMap[build.state] || 9);
+        return `${indent}${author} • ${branch} • ${shortSha} • ${SEMANTIC_COLORS.dim(`Created ${age}`)}`;
     }
     formatAnnotationSummary(annotations) {
         if (!annotations || annotations.length === 0) {
@@ -383,6 +430,73 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         }
         return lines.join('\n');
     }
+    formatJobSummary(jobsData, buildState) {
+        const jobs = jobsData?.edges;
+        if (!jobs || jobs.length === 0) {
+            return '';
+        }
+        const lines = [];
+        const jobStats = this.getJobStats(jobs);
+        // Build summary parts based on job states
+        const countParts = [];
+        if (jobStats.failed > 0)
+            countParts.push(SEMANTIC_COLORS.error(`${jobStats.failed} failed`));
+        if (jobStats.passed > 0)
+            countParts.push(SEMANTIC_COLORS.success(`${jobStats.passed} passed`));
+        if (jobStats.running > 0)
+            countParts.push(SEMANTIC_COLORS.info(`${jobStats.running} running`));
+        if (jobStats.blocked > 0)
+            countParts.push(SEMANTIC_COLORS.warning(`${jobStats.blocked} blocked`));
+        // Don't show skipped jobs
+        if (jobStats.canceled > 0)
+            countParts.push(SEMANTIC_COLORS.muted(`${jobStats.canceled} canceled`));
+        // Use appropriate icon based on build state
+        const icon = buildState === 'FAILED' ? getStateIcon('FAILED') :
+            buildState === 'RUNNING' ? getStateIcon('RUNNING') :
+                buildState === 'PASSED' ? getStateIcon('PASSED') :
+                    buildState === 'BLOCKED' ? getStateIcon('BLOCKED') : '•';
+        // Check if we have partial data
+        const hasMorePages = jobsData?.pageInfo?.hasNextPage;
+        const totalCount = jobsData?.count;
+        if (hasMorePages) {
+            const showing = jobs.length;
+            const total = totalCount || `${showing}+`;
+            lines.push(`${icon} Showing ${SEMANTIC_COLORS.count(String(showing))} of ${SEMANTIC_COLORS.count(String(total))} steps: ${countParts.join(', ')}`);
+            lines.push(SEMANTIC_COLORS.warning('⚠️  Showing first 100 steps only (more available)'));
+            lines.push(SEMANTIC_COLORS.dim('  → Use --jobs to fetch all step data and see accurate statistics'));
+        }
+        else {
+            lines.push(`${icon} ${SEMANTIC_COLORS.count(String(jobStats.total))} step${jobStats.total > 1 ? 's' : ''}: ${countParts.join(', ')}`);
+        }
+        // For failed builds, show the specific failed job names
+        if (buildState === 'FAILED') {
+            const failedJobs = this.getFailedJobs(jobs);
+            const jobGroups = this.groupJobsByLabel(failedJobs);
+            // Show up to 3 failed job types
+            const displayGroups = jobGroups.slice(0, 3);
+            for (const group of displayGroups) {
+                const label = this.parseEmoji(group.label);
+                const icon = getStateIcon('FAILED');
+                // Get duration for display
+                const duration = group.count === 1 && group.jobs[0]?.node
+                    ? ` ${SEMANTIC_COLORS.dim(`- ran ${this.formatJobDuration(group.jobs[0].node)}`)}`
+                    : '';
+                if (group.parallelTotal > 0) {
+                    lines.push(`   ${icon} ${label} ${SEMANTIC_COLORS.dim(`(${group.stateCounts.failed || 0}/${group.parallelTotal} failed)`)}`);
+                }
+                else if (group.count > 1) {
+                    lines.push(`   ${icon} ${label} ${SEMANTIC_COLORS.dim(`(${group.stateCounts.failed || 0} failed)`)}`);
+                }
+                else {
+                    lines.push(`   ${icon} ${label}${duration}`);
+                }
+            }
+            if (jobGroups.length > 3) {
+                lines.push(`   ${SEMANTIC_COLORS.muted(`...and ${jobGroups.length - 3} more`)}`);
+            }
+        }
+        return lines.join('\n');
+    }
     formatAnnotationDetails(annotations) {
         const lines = [];
         // Group annotations by style
@@ -409,7 +523,7 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
     }
     formatJobDetails(jobs, options) {
         if (!jobs || jobs.length === 0) {
-            return 'No jobs found';
+            return 'No steps found';
         }
         const lines = [];
         const jobStats = this.getJobStats(jobs);
@@ -423,45 +537,83 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             parts.push(`${getStateIcon('RUNNING')} ${jobStats.running} running`);
         if (jobStats.blocked > 0)
             parts.push(`${getStateIcon('BLOCKED')} ${jobStats.blocked} blocked`);
-        if (jobStats.skipped > 0)
-            parts.push(`${getStateIcon('SKIPPED')} ${jobStats.skipped} skipped`);
-        lines.push(`Jobs: ${parts.join('  ')}`);
+        // Don't show skipped jobs in summary
+        lines.push(`Steps: ${parts.join('  ')}`);
         lines.push('');
         // Filter jobs based on options
         let filteredJobs = jobs;
         if (options?.failed) {
             filteredJobs = this.getFailedJobs(jobs);
         }
-        // Group jobs by state
+        // Group jobs by state first
         const grouped = this.groupJobsByState(filteredJobs);
         for (const [state, stateJobs] of Object.entries(grouped)) {
             if (stateJobs.length === 0)
                 continue;
             const icon = this.getJobStateIcon(state);
             const stateColored = this.colorizeJobState(state);
+            // Collapse parallel jobs with same label
+            const collapsedGroups = this.collapseParallelJobs(stateJobs);
             lines.push(`${icon} ${stateColored} (${SEMANTIC_COLORS.count(String(stateJobs.length))}):`);
-            for (const job of stateJobs) {
-                const label = this.parseEmoji(job.node.label);
-                const duration = this.formatJobDuration(job.node);
-                // Basic job line
-                lines.push(`  ${label} (${duration})`);
-                // Show additional details if --jobs or --full
-                if (options?.jobs || options?.full) {
-                    // Timing details
-                    if (job.node.startedAt) {
-                        const startTime = new Date(job.node.startedAt).toLocaleTimeString();
-                        const endTime = job.node.finishedAt
-                            ? new Date(job.node.finishedAt).toLocaleTimeString()
-                            : 'still running';
-                        lines.push(`    ${SEMANTIC_COLORS.dim(`${getProgressIcon('TIMING')}  ${startTime} → ${endTime}`)}`);
+            for (const group of collapsedGroups) {
+                if (group.isParallelGroup && group.jobs.length > 1) {
+                    // Collapsed parallel group display
+                    const label = this.parseEmoji(group.label);
+                    const total = group.parallelTotal || group.jobs.length;
+                    const passedCount = group.jobs.filter(j => this.isJobPassed(j.node)).length;
+                    const failedCount = group.jobs.filter(j => this.isJobFailed(j.node)).length;
+                    // Show summary line for parallel group
+                    if (failedCount > 0) {
+                        // If there are failures, show breakdown
+                        // Apply state color to label
+                        const coloredLabel = state === 'Failed' ? SEMANTIC_COLORS.error(label) :
+                            state === 'Passed' ? SEMANTIC_COLORS.success(label) :
+                                state === 'Running' ? SEMANTIC_COLORS.info(label) :
+                                    state === 'Blocked' ? SEMANTIC_COLORS.warning(label) : label;
+                        lines.push(`  ${coloredLabel} ${SEMANTIC_COLORS.dim(`(${passedCount}/${total} passed, ${failedCount} failed)`)}`);
+                        // Show failed steps individually
+                        const failedJobs = group.jobs.filter(j => this.isJobFailed(j.node));
+                        for (const job of failedJobs) {
+                            const duration = this.formatJobDuration(job.node);
+                            const parallelInfo = job.node.parallelGroupIndex !== undefined
+                                ? ` ${SEMANTIC_COLORS.dim(`[Parallel: ${job.node.parallelGroupIndex + 1}/${job.node.parallelGroupTotal}]`)}`
+                                : '';
+                            lines.push(`    ${SEMANTIC_COLORS.error('↳ Failed')}: ${SEMANTIC_COLORS.dim(duration)}${parallelInfo}`);
+                        }
                     }
-                    // Parallel group info
-                    if (job.node.parallelGroupIndex !== undefined && job.node.parallelGroupTotal) {
-                        lines.push(`    ${SEMANTIC_COLORS.dim(`${getProgressIcon('PARALLEL')}  Parallel: ${job.node.parallelGroupIndex + 1}/${job.node.parallelGroupTotal}`)}`);
+                    else {
+                        // All passed/running/blocked - just show summary
+                        const avgDuration = this.calculateAverageDuration(group.jobs);
+                        // Apply state color to label
+                        const coloredLabel = state === 'Passed' ? SEMANTIC_COLORS.success(label) :
+                            state === 'Failed' ? SEMANTIC_COLORS.error(label) :
+                                state === 'Running' ? SEMANTIC_COLORS.info(label) :
+                                    state === 'Blocked' ? SEMANTIC_COLORS.warning(label) : label;
+                        lines.push(`  ${coloredLabel} ${SEMANTIC_COLORS.dim(`(${total} parallel steps, avg: ${avgDuration})`)}`);
                     }
-                    // Retry info
-                    if (job.node.retried) {
-                        lines.push(`    ${SEMANTIC_COLORS.warning(`${getProgressIcon('RETRY')} Retried`)}`);
+                }
+                else {
+                    // Single job or non-parallel group - display as before
+                    const job = group.jobs[0];
+                    const label = this.parseEmoji(job.node.label);
+                    const duration = this.formatJobDuration(job.node);
+                    // Apply state color to label
+                    const coloredLabel = state === 'Passed' ? SEMANTIC_COLORS.success(label) :
+                        state === 'Failed' ? SEMANTIC_COLORS.error(label) :
+                            state === 'Running' ? SEMANTIC_COLORS.info(label) :
+                                state === 'Blocked' ? SEMANTIC_COLORS.warning(label) : label;
+                    // Add parallel info inline if present
+                    const parallelInfo = (job.node.parallelGroupIndex !== undefined && job.node.parallelGroupTotal)
+                        ? ` ${SEMANTIC_COLORS.dim(`[Parallel: ${job.node.parallelGroupIndex + 1}/${job.node.parallelGroupTotal}]`)}`
+                        : '';
+                    // Basic step line with optional parallel info
+                    lines.push(`  ${coloredLabel} ${SEMANTIC_COLORS.dim(`(${duration})`)}${parallelInfo}`);
+                    // Show additional details if --jobs or --full and single step
+                    if ((options?.jobs || options?.full) && !group.isParallelGroup) {
+                        // Retry info
+                        if (job.node.retried) {
+                            lines.push(`    ${SEMANTIC_COLORS.warning(`${getProgressIcon('RETRY')} Retried`)}`);
+                        }
                     }
                 }
             }
@@ -469,67 +621,6 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
         }
         return lines.join('\n').trim();
     }
-    formatFailedJobsSummaryWithHints(failedJobs, options) {
-        const hints = [];
-        const summary = this.formatFailedJobsSummary(failedJobs, options, hints);
-        return { summary, hints };
-    }
-    formatFailedJobsSummary(failedJobs, options, hints) {
-        const lines = [];
-        // Group identical jobs by label
-        const jobGroups = this.groupJobsByLabel(failedJobs);
-        // Show all groups if --all-jobs, otherwise limit to 10
-        const displayGroups = options?.allJobs
-            ? jobGroups
-            : jobGroups.slice(0, 10);
-        for (const group of displayGroups) {
-            const label = this.parseEmoji(group.label);
-            if (group.count === 1) {
-                const duration = this.formatJobDuration(group.jobs[0].node);
-                lines.push(`   ${SEMANTIC_COLORS.error('Failed')}: ${label} - ran ${duration}`);
-            }
-            else {
-                // Multiple jobs with same label - show detailed breakdown
-                const statusParts = [];
-                if (group.stateCounts.failed > 0) {
-                    statusParts.push(`${group.stateCounts.failed} failed`);
-                }
-                if (group.stateCounts.broken > 0) {
-                    statusParts.push(`${group.stateCounts.broken} broken`);
-                }
-                if (group.stateCounts.notStarted > 0) {
-                    statusParts.push(`${group.stateCounts.notStarted} not started`);
-                }
-                if (group.stateCounts.passed > 0) {
-                    statusParts.push(`${group.stateCounts.passed} passed`);
-                }
-                if (group.stateCounts.other > 0) {
-                    statusParts.push(`${group.stateCounts.other} other`);
-                }
-                const statusInfo = statusParts.join(', ') || 'various states';
-                // Show parallel info if it's a parallel job group
-                const parallelInfo = group.parallelTotal > 0 ? ` (${group.count}/${group.parallelTotal} parallel)` : ` (${SEMANTIC_COLORS.count(String(group.count))} jobs)`;
-                lines.push(`   ${SEMANTIC_COLORS.error('Failed')}: ${label}${parallelInfo}: ${statusInfo}`);
-            }
-        }
-        // Add summary if there are more job types and not showing all
-        if (!options?.allJobs) {
-            const remaining = jobGroups.length - displayGroups.length;
-            if (remaining > 0) {
-                lines.push(`   ${SEMANTIC_COLORS.muted(`...and ${remaining} more job types`)}`);
-                // If hints array is provided, add hint there; otherwise format inline
-                if (hints) {
-                    hints.push('Use --all-jobs to show all jobs');
-                }
-                else {
-                    lines.push('');
-                    const tips = formatTips(['Use --all-jobs to show all jobs'], TipStyle.GROUPED);
-                    lines.push(tips);
-                }
-            }
-        }
-        return lines.join('\n');
-    }
     groupJobsByLabel(jobs) {
         const groups = new Map();
         for (const job of jobs) {
@@ -688,6 +779,7 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             running: 0,
             blocked: 0,
             skipped: 0,
+            canceled: 0,
             queued: 0,
             completed: 0
         };
@@ -714,7 +806,12 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             else if (state === 'BLOCKED') {
                 stats.blocked++;
             }
-            else if (state === 'SKIPPED' || state === 'CANCELED') {
+            else if (state === 'CANCELED' || state === 'CANCELLED') {
+                stats.canceled++;
+                stats.completed++;
+            }
+            else if (state === 'SKIPPED' || state === 'BROKEN') {
+                // BROKEN jobs are functionally skipped - they don't run due to conditions not matching
                 stats.skipped++;
                 stats.completed++;
             }
@@ -736,7 +833,7 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
                 stats.passed++;
                 stats.completed++;
             }
-            else if (state === 'FAILED' || state === 'BROKEN' || job.node.passed === false) {
+            else if (state === 'FAILED' || job.node.passed === false) {
                 stats.failed++;
                 stats.completed++;
             }
@@ -748,14 +845,22 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             return [];
         return jobs.filter(job => {
             const state = job.node.state?.toUpperCase();
+            // BROKEN jobs are skipped/not run, not failed
+            if (state === 'BROKEN' || state === 'SKIPPED') {
+                return false;
+            }
             // If we have an exit status, use that as the source of truth
             // Note: exitStatus comes as a string from Buildkite API
             if (job.node.exitStatus !== null && job.node.exitStatus !== undefined) {
                 const exitCode = parseInt(job.node.exitStatus, 10);
                 return exitCode !== 0;
             }
-            // Otherwise fall back to state
-            return state === 'FAILED' || state === 'BROKEN' || job.node.passed === false;
+            // For FINISHED jobs, check the passed field
+            if (state === 'FINISHED') {
+                return job.node.passed === false;
+            }
+            // Otherwise check if explicitly failed
+            return state === 'FAILED';
         });
     }
     getRunningJobs(jobs) {
@@ -773,8 +878,8 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             'Failed': [],
             'Passed': [],
             'Running': [],
-            'Blocked': [],
-            'Skipped': []
+            'Blocked': []
+            // Don't include Skipped - we don't display them
         };
         if (!jobs)
             return grouped;
@@ -797,8 +902,9 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             else if (state === 'BLOCKED') {
                 grouped['Blocked'].push(job);
             }
-            else if (state === 'SKIPPED' || state === 'CANCELED') {
-                grouped['Skipped'].push(job);
+            else if (state === 'SKIPPED' || state === 'CANCELED' || state === 'BROKEN') {
+                // Don't display skipped/broken/canceled jobs - they're not shown in Buildkite UI
+                // Skip these entirely
             }
             else if (state === 'FINISHED' || state === 'COMPLETED') {
                 // For finished jobs without exit status, check passed field
@@ -812,12 +918,88 @@ export class PlainTextFormatter extends BaseBuildDetailFormatter {
             else if (state === 'PASSED' || job.node.passed === true) {
                 grouped['Passed'].push(job);
             }
-            else if (state === 'FAILED' || state === 'BROKEN' || job.node.passed === false) {
+            else if (state === 'FAILED') {
                 grouped['Failed'].push(job);
             }
         }
         return grouped;
     }
+    collapseParallelJobs(jobs) {
+        const groups = new Map();
+        // Group jobs by label
+        for (const job of jobs) {
+            const label = job.node.label || 'Unnamed';
+            if (!groups.has(label)) {
+                groups.set(label, []);
+            }
+            groups.get(label).push(job);
+        }
+        // Convert to array and determine if each group is a parallel group
+        const result = [];
+        for (const [label, groupJobs] of groups.entries()) {
+            // Check if this is a parallel group (multiple jobs with same label and parallel info)
+            const hasParallelInfo = groupJobs.some(j => j.node.parallelGroupIndex !== undefined && j.node.parallelGroupTotal !== undefined);
+            const isParallelGroup = hasParallelInfo && groupJobs.length > 1;
+            // Get the total from the first job if available
+            const parallelTotal = groupJobs[0]?.node?.parallelGroupTotal;
+            result.push({
+                label,
+                jobs: groupJobs,
+                isParallelGroup,
+                parallelTotal
+            });
+        }
+        return result;
+    }
+    isJobPassed(job) {
+        const state = job.state?.toUpperCase();
+        if (job.exitStatus !== null && job.exitStatus !== undefined) {
+            return parseInt(job.exitStatus, 10) === 0;
+        }
+        if (state === 'PASSED')
+            return true;
+        if (state === 'FINISHED' || state === 'COMPLETED') {
+            return job.passed === true;
+        }
+        return job.passed === true;
+    }
+    isJobFailed(job) {
+        const state = job.state?.toUpperCase();
+        if (job.exitStatus !== null && job.exitStatus !== undefined) {
+            return parseInt(job.exitStatus, 10) !== 0;
+        }
+        if (state === 'FAILED')
+            return true;
+        if (state === 'FINISHED' || state === 'COMPLETED') {
+            return job.passed === false;
+        }
+        return false;
+    }
+    calculateAverageDuration(jobs) {
+        const durationsMs = jobs
+            .filter(j => j.node.startedAt && j.node.finishedAt)
+            .map(j => {
+            const start = new Date(j.node.startedAt).getTime();
+            const end = new Date(j.node.finishedAt).getTime();
+            return end - start;
+        });
+        if (durationsMs.length === 0) {
+            return 'unknown';
+        }
+        const avgMs = durationsMs.reduce((a, b) => a + b, 0) / durationsMs.length;
+        const avgSeconds = Math.floor(avgMs / 1000);
+        if (avgSeconds < 60) {
+            return `${avgSeconds}s`;
+        }
+        const minutes = Math.floor(avgSeconds / 60);
+        const seconds = avgSeconds % 60;
+        if (minutes < 60) {
+            return seconds > 0 ? `${minutes}m ${seconds}s` : `${minutes}m`;
+        }
+        const hours = Math.floor(minutes / 60);
+        const remainingMinutes = minutes % 60;
+        return remainingMinutes > 0 ? `${hours}h ${remainingMinutes}m` : `${hours}h`;
+    }
     countAnnotationsByStyle(annotations) {
         const counts = {
             ERROR: 0,