fix(node-resolve): bump is-builtin-module version, imports with a trailing slash

[rsweeneydev]

Rollup Plugin Name: node-resolve

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes
• no

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

List any relevant issue numbers:
#1211

Description

Adding a trailing slash to a module name in an import statement is the convention for telling node to use the local module with that name, rather than the builtin module with the same name.

For an example, see the docs for punycode which is available as both a builtin and standalone module for node.

The is-builtin-module package did not respect this convention. A fix was made and released in is-builtin-modules v3.2.1. This change bumps the version of the dependency and adds a test case to verify that import statements with a trailing slash are not treated as builtin modules by the node-resolve package.

[shellscape]

Looks good, thanks for the additional test!

[shellscape]

@rsweeney21 I was all set to merge this but a prior PR has updated our lockfile. Please rebase to upstream/master and run pnpm install at the repo root to resolve the conflict. We'll merge as soon as you're able to do that.

[shellscape]

I was actually able to resolve the conflict using the web editor.

[shellscape]