microsoft / microsoft/TypeScript

Add TS server command for removing all unused code in a file

Abierto
#40,922 1 comentario 8 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Domain: LS: Quick Fixes Suggestion
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

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. 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

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.