microsoft / microsoft/TypeScript
Make automatic adding missing imports smarter
Nessuno ha ancora preso questa issue.
- Lingua principale
- Go
- Stelle
- 111k
- Fork
- 14.3k
- Merge medio
- 2g 4h
- PR unite (30g)
- 132
Descrizione
Scenario 1: Fuzzy Logic
I have a several files containing import {connect} from "react-redux";. If I use the add missing imports context menu to add the missing import for connect, the system should realize "react-redux" is far more probable module than "net", "tls", and "http2", which are all recommended higher than "react-redux", but none of which are used anywhere in my project.
Scenario 2: Respect node conventions
If I have a file ./myfolder/index.ts, I can import the file with import "./myfolder". The system currently resolves missing imports from ./myfolder as ./myfolder/index.ts, when it should actually use ./myfolder.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Riproduci entrambi gli scenari con i file TypeScript descritti: un import connect mancante da react-redux e un import da ./myfolder, dove index.ts è il punto di ingresso. Inizia individuando i percorsi add-missing-imports e module-resolution, quindi verifica che l’utilizzo del progetto influisca sull’ordinamento e che lo specificatore di directory venga preservato. Il lavoro è completato quando entrambi gli scenari scelgono le forme di import previste senza introdurre regressioni nei suggerimenti esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- developer-experience, tooling
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100