fix(webpack): web app in default nx config not working with @aws-sdk

[owen26]

Current Behavior

web apps created via default NX configurations won't work with @aws-sdk

the reason is that all AWS SKD v3 packages define only partial entry via browser, and the rest via either module for browser, or main for node.js

with default NX withWeb settings, which inherit from withNX settings, the order will be browser, main, and module, in the case of AWS SDK, webpack will wrongly import most of the node.js packages instead of browser oriented packages.

I do not see a reason why this shouldn't be overwritten in withWeb config to let module takes priority. Thus the changes in this PR did it.

Expected Behavior

should work with AWS SDK (and potentially other open source libraries uses similar strategy as AWS SDK

Related Issue(s)

#14920

Fixes #14920

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
nx-dev	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 25, 2023 6:18pm

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 324ba24. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

🟥 Failed Commands
nx run-many -t e2e-macos --parallel=1 --exclude=e2e-detox

✅ Successfully ran 8 targets

• nx affected -t e2e-macos --parallel=1
• nx affected --target=e2e --base=5b0527bc45cdf8522f166d042bda9ee3090c9823 --head=324ba24a05207cd67f19e01d8aa65dbe8eb498ea --parallel=1
• nx affected --target=test --base=5b0527bc45cdf8522f166d042bda9ee3090c9823 --head=324ba24a05207cd67f19e01d8aa65dbe8eb498ea --parallel=1
• nx affected --target=build --base=5b0527bc45cdf8522f166d042bda9ee3090c9823 --head=324ba24a05207cd67f19e01d8aa65dbe8eb498ea --parallel=3
• nx run-many -t check-imports check-commit check-lock-files depcheck --parallel=1 --no-dte
• nx affected --target=lint --base=5b0527bc45cdf8522f166d042bda9ee3090c9823 --head=324ba24a05207cd67f19e01d8aa65dbe8eb498ea --parallel=3
• nx-cloud record -- nx format:check --base=5b0527bc45cdf8522f166d042bda9ee3090c9823 --head=324ba24a05207cd67f19e01d8aa65dbe8eb498ea
• nx documentation --no-dte

---

Sent with 💌 from NxCloud.

[github-actions]

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.