Skip to content

Suppress non-actionable webpack warnings in trydotnet editor build - #450

Draft
BenjaminMichaelis with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-critical-dependency-warnings
Draft

Suppress non-actionable webpack warnings in trydotnet editor build#450
BenjaminMichaelis with Copilot wants to merge 1 commit into
mainfrom
copilot/fix-critical-dependency-warnings

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown

The editor webpack build was surfacing repeated Critical dependency warnings from @microsoft/polyglot-notebooks/dist/setup.js, plus asset/entrypoint size warnings that do not indicate a regression in this bundle. This change trims that noise so build output highlights actionable problems.

  • Warning filtering

    • Adds a targeted ignoreWarnings rule in src/microsoft-trydotnet-editor/webpack.config.js
    • Suppresses only the known require function is used in a way in which dependencies cannot be statically extracted warnings emitted from @microsoft/polyglot-notebooks/dist/setup.js
  • Performance hint cleanup

    • Disables webpack performance.hints for this bundle
    • Removes repetitive size-limit warnings for the Monaco/editor worker assets, which are expected for this artifact shape
  • Scope

    • Leaves bundling behavior unchanged
    • Does not mute unrelated webpack warnings
ignoreWarnings: [
  (warning) =>
    warning.module?.resource?.includes('@microsoft/polyglot-notebooks/dist/setup.js') &&
    warning.message?.includes('require function is used in a way in which dependencies cannot be statically extracted')
],
performance: {
  hints: false
}

Co-authored-by: BenjaminMichaelis <22186029+BenjaminMichaelis@users.noreply.github.com>
Copilot AI changed the title Suppress known webpack warnings in trydotnet editor build Suppress non-actionable webpack warnings in trydotnet editor build Aug 20, 2026
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.

2 participants