tleunen/babel-plugin-module-resolver

handle resolving duplicated files

Open

#122 opened on Feb 13, 2017

View on GitHub
 (4 comments) (0 reactions) (0 assignees)JavaScript (3,417 stars) (217 forks)batch import
enhancementhelp wanted

Description

I have these two files

app/component/home/Trader.js

app/component/contact/Trader.js

Now I import that second Trader in this component app/component/sent/Sent.js

import Trader from 'Trader'
export default class Sent extends React.Component{
  render(){ 
    return <div>hello <Trader /></div>
  }
}

Problem: I will have the first trader file instead second one.

I understand that babel will resolve as soon as it find a match. However, I think we can do a warning. If babel found case like this. Give a warning, then users will know they have to change the file name in the import. They can change to something like below

import Trader from 'contact/Trader'
export default class Sent extends React.Component{
  render(){ 
    return <div>hello <Trader /></div>
  }
}

Contributor guide

Tech stack
javascript
Domain
developer experiencetooling
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Understanding of Babel pluginsKnowledge of module resolution
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
Investigate the module resolution logic in the plugin's source code, likely in the `src/index.js` or `src/resolve.js` file. Identify where file paths are matched and where duplicate file names could cause ambiguity. Implement a warning when multiple files with the same basename match the import path, referencing the existing logging mechanism. Test with the provided example to ensure the warning triggers correctly.