Skip to content

feat: Integrated CLIProgressManager and SummaryManager in export #2037

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
aman19K merged 5 commits into from
Aug 1, 2025
Merged

feat: Integrated CLIProgressManager and SummaryManager in export #2037

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d62857e
feat: Integrated CLIProgressManager and SummaryManager in export
aman19K Aug 1, 2025
ab042ed
updated talismanrc
aman19K Aug 1, 2025
044595a
refactor: updated test cases
aman19K Aug 1, 2025
3978d53
fix: PR comments
aman19K Aug 1, 2025
cc738c5
fix: padding issue in progress bar UX
aman19K Aug 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix: padding issue in progress bar UX
  • Loading branch information
@aman19K
aman19K committed Aug 1, 2025
commit cc738c531a000576b93cd7fb18311f2f192cb183
36 changes: 27 additions & 9 deletions packages/contentstack-utilities/src/progress-summary/cli-progress-manager.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -170,6 +170,13 @@ export default class CLIProgressManager {
this.callbacks = { ...this.callbacks, ...callbacks };
}

/**
* Convert module name from UPPERCASE to PascalCase
*/
private formatModuleName(name: string): string {
return name.charAt(0).toUpperCase() + name.slice(1).toLowerCase();
}

private initializeProgress(): void {
if (this.showConsoleLogs) {
return;
Expand Down Expand Up @@ -202,8 +209,10 @@ export default class CLIProgressManager {
barIncompleteChar: '\u2591',
hideCursor: true,
});
const formattedName = this.formatModuleName(this.moduleName);
const displayName = formattedName.length > 20 ? formattedName.substring(0, 17) + '...' : formattedName;
this.progressBar.start(this.total, 0, {
label: chalk.gray(` └─ ${this.moduleName}`.padEnd(18)),
label: chalk.gray(` └─ ${displayName}`.padEnd(25)),
status: chalk.gray('Starting...'),
percentage: '0',
});
Expand All @@ -229,7 +238,8 @@ export default class CLIProgressManager {
};

if (!this.showConsoleLogs) {
const indentedLabel = ` ├─ ${processName}`.padEnd(18);
const truncatedName = processName.length > 20 ? processName.substring(0, 17) + '...' : processName;
const indentedLabel = ` ├─ ${truncatedName}`.padEnd(25);
process.progressBar = this.multiBar.create(total, 0, {
label: chalk.gray(indentedLabel),
status: chalk.gray('Pending'),
Expand All @@ -251,10 +261,11 @@ export default class CLIProgressManager {
if (process) {
process.status = 'active';
if (!this.showConsoleLogs && process.progressBar) {
const indentedLabel = ` ├─ ${processName}`.padEnd(18);
const truncatedName = processName.length > 20 ? processName.substring(0, 17) + '...' : processName;
const indentedLabel = ` ├─ ${truncatedName}`.padEnd(25);
process.progressBar.update(0, {
label: chalk.yellow(indentedLabel),
status: chalk.yellow(this.moduleName),
status: chalk.yellow('Processing'),
percentage: '0',
});
}
Expand All @@ -277,7 +288,8 @@ export default class CLIProgressManager {
const statusText = success
? chalk.green(`✓ Complete (${process.successCount}/${process.current})`)
: chalk.red(`✗ Failed (${process.successCount}/${process.current})`);
const indentedLabel = ` ├─ ${processName}`.padEnd(18);
const truncatedName = processName.length > 20 ? processName.substring(0, 17) + '...' : processName;
const indentedLabel = ` ├─ ${truncatedName}`.padEnd(25);
process.progressBar.update(process.total, {
label: success ? chalk.green(indentedLabel) : chalk.red(indentedLabel),
status: statusText,
Expand All @@ -297,7 +309,8 @@ export default class CLIProgressManager {
const process = this.processes.get(processName);
if (process && process.progressBar) {
const percentage = Math.round((process.current / process.total) * 100);
const indentedLabel = ` ├─ ${processName}`.padEnd(18);
const truncatedName = processName.length > 20 ? processName.substring(0, 17) + '...' : processName;
const indentedLabel = ` ├─ ${truncatedName}`.padEnd(25);
process.progressBar.update(process.current, {
label: chalk.yellow(indentedLabel),
status: chalk.yellow(message),
Expand All @@ -306,8 +319,10 @@ export default class CLIProgressManager {
}
} else if (this.progressBar) {
const percentage = Math.round(this.progressBar.getProgress() * 100);
const formattedName = this.formatModuleName(this.moduleName);
const displayName = formattedName.length > 20 ? formattedName.substring(0, 17) + '...' : formattedName;
this.progressBar.update(this.progressBar.getProgress() * this.total, {
label: chalk.yellow(` └─ ${this.moduleName}`.padEnd(18)),
label: chalk.yellow(` └─ ${displayName}`.padEnd(25)),
status: chalk.yellow(message),
percentage: percentage.toString(),
});
Expand Down Expand Up @@ -355,7 +370,8 @@ export default class CLIProgressManager {
const percentage = Math.round((process.current / process.total) * 100);
const statusText = `${process.successCount}✓ ${process.failureCount}✗`;

const indentedLabel = ` ├─ ${targetProcess}`.padEnd(18);
const truncatedName = targetProcess.length > 20 ? targetProcess.substring(0, 17) + '...' : targetProcess;
const indentedLabel = ` ├─ ${truncatedName}`.padEnd(25);
process.progressBar.increment(1, {
label: chalk.cyan(indentedLabel),
status: chalk.cyan(statusText),
Expand All @@ -381,8 +397,10 @@ export default class CLIProgressManager {
const labelColor =
totalProcessed >= this.total ? (this.failureCount === 0 ? chalk.green : chalk.yellow) : chalk.cyan;

const formattedName = this.formatModuleName(this.moduleName);
const displayName = formattedName.length > 20 ? formattedName.substring(0, 17) + '...' : formattedName;
this.progressBar.increment(1, {
label: labelColor(` └─ ${this.moduleName}`.padEnd(18)),
label: labelColor(` └─ ${displayName}`.padEnd(25)),
status: statusText,
percentage: percentage.toString(),
});
Expand Down