microsoft / microsoft/TypeScript

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

Đang mở
#40,922 1 bình luận 8 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Domain: LS: Quick Fixes Suggestion
Ngôn ngữ chính
Go
Star
111k
Fork
14.4k
Merge trung bình
1 ngày 19 giờ
Pull request đã merge (30 ngày)
117

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu bằng cách đọc các lệnh TS Server hiện có và đường dẫn getCombinedCodeFix được đề cập trong issue, sau đó so sánh cách VS Code gọi source.removeUnused. Được xem là hoàn tất khi một lệnh chỉ xóa mã không thể truy cập an toàn, các import không được sử dụng và các khai báo không được sử dụng, đồng thời giữ nguyên các tham số và các biến có khả năng tạo ra side effect.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
typescript
Lĩnh vực
tooling
Loại issue
Tính năng
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.