microsoft / microsoft/TypeScript
Add TS server command for removing all unused code in a file
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Go
- Estrellas
- 111k
- Forks
- 14.3k
- Merge medio
- 2 d 4 h
- PR fusionados (30 d)
- 132
Descripción
Search terms
- code action
- source action
- getCombinedCodeFix
Feature request
Add a new TS Server command that removes all unused code in a file. This should only perform 'safe' removals, such as:
- Unreachable code
- Unused imports
- Unused declarations
We should likely not remove unused parameters. Not sure about removing unused variable since removing them can also have side effects
Use case
VS Code allows users to configure code actions that are run on save. One of the supported code actions is source.removeUnused which attempts to remove unused code in a file
On the VS Code side, we can't implement this very well since unused code often ends up overlapping with other unused code and we don't have much insight into what is "safe" to remove and what is not
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza leyendo los comandos existentes de TS Server y la ruta getCombinedCodeFix mencionada en el issue; después, compara cómo VS Code invoca source.removeUnused. Se considera terminado cuando un comando elimina únicamente código inalcanzable seguro, imports no utilizados y declaraciones no utilizadas, dejando intactos los parámetros y las variables que podrían tener efectos secundarios.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- tooling
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100