microsoft / microsoft/TypeScript
Add TS Server command for 'add all missing imports'
Personne n'a encore pris cette issue.
- Langage dominant
- Go
- Étoiles
- 111k
- Forks
- 14.3k
- Merge moyen
- 2 j 4 h
- PR mergées (30 j)
- 132
Description
Search terms
- code action
- source action
- Add missing imports
- getCombinedCodeFix
Feature request
Add a new TS Server command that triggers 'add all missing imports' for a given file.
Use case
VS Code allows users to configure code actions that are run on save. One of the supported code actions is source.addMissingImports which attempts to add all missing imports in a file. Users can also setup keybindings for these specific actions
To implement this, right now we request all code actions for the file and then try fetching the 'add all missing imports' combined fix for one of them. This is complicated and results in the TS Server doing extra work.
It would be better for us if we could ask the Typescript server: give me the edits that add the missing imports in this file
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par la gestion existante de getCombinedCodeFix dans le serveur TypeScript pour l’action source.addMissingImports. Suivez la manière dont les commandes et les modifications de fichiers sont exposées, puis vérifiez que la nouvelle commande renvoie les modifications permettant d’ajouter tous les imports manquants dans un fichier donné sans nécessiter le flux plus large de requête code-action.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- tooling
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100