Skip to content

Feat/dx 2951 v2 logger integration #2008

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.

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

Draft
wants to merge 73 commits into
base: development
Choose a base branch
from

Conversation

aman19K
Copy link
Contributor

@aman19K aman19K commented Jul 18, 2025

No description provided.

aman19K and others added 30 commits June 5, 2025 23:08
…gration-export

feat: integrated v2Logger in export plugin
feat: Integrated Logger and cliErrorHandler in auth plugin
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
feat: Added Support for config get & config set for Logging Configuration
feat: added log config support in utilities
feat:added debug log in export, variant(export) & auth plugin
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request introduces comprehensive v2 logger integration across the Contentstack CLI ecosystem, implementing structured logging with context tracking and enhanced error handling throughout multiple packages.

  • Enhanced logger with configurable levels, file/console output, and comprehensive error classification
  • Updated dependencies to support the new v2 logger and logging configuration commands
  • Integrated structured logging with context tracking across contentstack-variants and contentstack-import packages

Reviewed Changes

Copilot reviewed 131 out of 133 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
packages/contentstack/package.json Updated CLI dependencies to newer versions supporting v2 logger
packages/contentstack/README.md Added new config commands for log level and path configuration
packages/contentstack-variants/ Comprehensive v2 logger integration with context tracking
packages/contentstack-utilities/ Enhanced Logger class with improved error handling and configuration
packages/contentstack-import/ Migrated from legacy logging to v2 logger with structured context
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)

packages/contentstack-utilities/src/logger/logger.ts:5

  • [nitpick] The import name levelColors should be logLevelColors to be more descriptive and consistent with naming conventions.
import { levelColors, logLevels } from '../constants/logging';

cs-raj
cs-raj previously approved these changes Jul 18, 2025
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.

3 participants