Skip to content

refactor: improve debug log messages for better clarity#5902

Merged
chenjiahan merged 3 commits intomainfrom
debug_logs_0821
Aug 21, 2025
Merged

refactor: improve debug log messages for better clarity#5902
chenjiahan merged 3 commits intomainfrom
debug_logs_0821

Conversation

@chenjiahan
Copy link
Copy Markdown
Member

@chenjiahan chenjiahan commented Aug 21, 2025 •
edited
Loading

Summary

  • update debug log messages across core modules to be more descriptive and consistent
  • update documentation examples to reflect new debug log format
➜  DEBUG=rsbuild pnpm dev

  rsbuild 10:49:23 configuration loaded from: /path/to/rsbuild.config.ts
  rsbuild 10:49:23 registering default plugins
  rsbuild 10:49:23 default plugins registered
  rsbuild 10:49:23 initializing plugins
  rsbuild 10:49:23 plugins initialized
  rsbuild 10:49:23 applying modifyRsbuildConfig hook
  rsbuild 10:49:23 applied modifyRsbuildConfig hook
  rsbuild 10:49:23 applying modifyEnvironmentConfig hook (web)
  rsbuild 10:49:23 applied modifyEnvironmentConfig hook (web)
  rsbuild 10:49:23 create dev server
  rsbuild 10:49:23 creating compiler
  rsbuild 10:49:23 applying modifyBundlerChain hook
  rsbuild 10:49:23 applied modifyBundlerChain hook
  rsbuild 10:49:23 applying modifyRspackConfig hook
  rsbuild 10:49:23 applied modifyRspackConfig hook

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings August 21, 2025 02:53
@netlify
Copy link
Copy Markdown

netlify bot commented Aug 21, 2025 •
edited
Loading

✅ Deploy Preview for rsbuild ready!

Name Link
🔨 Latest commit 908d5d5
🔍 Latest deploy log https://app.netlify.com/projects/rsbuild/deploys/68a68be3f31a79000837392e
😎 Deploy Preview https://deploy-preview-5902--rsbuild.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 65 (🔴 down 11 from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI reviewed Aug 21, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

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 PR refactors debug log messages across Rsbuild's core modules to improve clarity and consistency. The changes standardize debug log format to use present participle/past tense patterns and include timestamps in documentation examples.

Key changes:

  • Updated debug log messages to use more descriptive language (e.g., "applying/applied hook" instead of "modify config")
  • Enhanced documentation examples to show the new debug log format with timestamps
  • Improved consistency across all core modules for better debugging experience

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/core/src/loadConfig.ts Updated config loading debug message to be more descriptive
packages/core/src/createRsbuild.ts Improved default plugin registration debug messages
packages/core/src/pluginManager.ts Enhanced plugin initialization debug messages
packages/core/src/provider/initConfigs.ts Standardized hook application debug messages
packages/core/src/provider/rspackConfig.ts Updated Rspack config modification debug messages
packages/core/src/provider/createCompiler.ts Improved compiler creation debug messages
packages/core/src/configChain.ts Updated bundler chain modification debug messages
packages/compat/webpack/src/createCompiler.ts Synchronized webpack compiler debug messages
website/docs/en/guide/debug/debug-mode.mdx Updated English documentation with new debug format
website/docs/en/guide/faq/features.mdx Updated English FAQ with new debug format
website/docs/zh/guide/debug/debug-mode.mdx Updated Chinese documentation with new debug format
website/docs/zh/guide/faq/features.mdx Updated Chinese FAQ with new debug format

@chenjiahan chenjiahan merged commit 6ffb3a2 into main Aug 21, 2025
11 checks passed
@chenjiahan chenjiahan deleted the debug_logs_0821 branch August 21, 2025 03:07
@chenjiahan chenjiahan mentioned this pull request Aug 21, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

release: refactor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chenjiahan