ionic-team / ionic-team/rollup-plugin-node-polyfills

import/require with trailing slash not getting resolved

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
142
Forks
65
PR merge metrics
No merged PRs in 30d

Description

https://github.com/ionic-team/rollup-plugin-node-polyfills/blob/9b5fe1a9cafffd4871e6d65613ed224f807ea251/src/modules.ts#L65

I believe the intent was to remove the trailing slash and check in the map if the Polyfill exists
`importee === importee.slice(0, -1);` should have been `importee = importee.slice(0, -1);`

Contributor guide

No contributing guide indexed for this repository

Research direction

Open src/modules.ts at line 65 and inspect how an import or require ending in a slash is handled before the polyfill map lookup. Verify the trailing slash is removed and that the resulting module resolves correctly; the issue does not mention a specific test file, so confirm the behavior with the project’s available checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, rollup
Domain
build-system, tooling
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.