Skip to content

Conversation

@gagik
Copy link
Contributor

@gagik gagik commented Dec 3, 2025

See: https://github.com/mongodb-js/mongosh/actions/runs/19892733974
I'm not sure why this broke and if this is the right fix but it works?

@gagik gagik requested a review from a team as a code owner December 3, 2025 12:48
Copilot AI review requested due to automatic review settings December 3, 2025 12:48
Copy link

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 fixes a prepublish issue in the node-runtime-worker-thread package by adding webpack external configurations for package.json imports. The changes ensure that package.json files from external dependencies are properly handled as external modules rather than being bundled.

Key changes:

  • Added webpack external configuration for mongodb-client-encryption/package.json
  • Added webpack external configuration for kerberos/package.json

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@addaleax addaleax left a comment

Choose a reason for hiding this comment

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

LGTM

@gagik gagik added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Dec 3, 2025
@gagik gagik merged commit b8a0a4c into main Dec 3, 2025
18 of 20 checks passed
@gagik gagik deleted the gagik/pre-publish-fix branch December 3, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants