Skip to content

DX-9992 | Revert console transport override for error/warn levels - #2713

Open
harshitha-cstk wants to merge 1 commit into
developmentfrom
fix/DX-10224-bulk-error-visibility
Open

DX-9992 | Revert console transport override for error/warn levels#2713
harshitha-cstk wants to merge 1 commit into
developmentfrom
fix/DX-10224-bulk-error-visibility

Conversation

@harshitha-cstk

Copy link
Copy Markdown
Contributor

Summary

Reverts the logger override in cli-utilities that forced error/warn levels to always reach the console (bypassing showConsoleLogs). Console transport is now added solely based on showConsoleLogs, restoring the original behavior.

Change

packages/contentstack-utilities/src/logger/logger.ts — the console transport is pushed only when showConsoleLogs is true, dropping the isErrorOrWarn bypass.

Context

The dropped override was the utilities-level approach to keeping failures visible in progress mode. That responsibility is instead handled at the plugin layer — see the companion bulk-operations fix (contentstack/cli-plugins#352), which surfaces failures via catch() without changing cli-utilities.

🤖 Generated with Claude Code

@harshitha-cstk
harshitha-cstk requested a review from a team as a code owner August 18, 2026 08:07
@snyk-io

snyk-io Bot commented Aug 18, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 0 25 ✅ Passed
🟡 Medium Severity 0 2 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 0
  • Medium without fixes: 2
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant