Extend `StackFrame` to support longer description than `name`
@connor4312 đang làm issue này rồi.
Từ ngày 15/9/2023.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Many DAP clients use the values in name when rendering info about a stack frame.
interface StackFrame {
...
/**
* The name of the stack frame, typically a method name.
*/
name: string;
...
}
There is no length limit on name, but empirically we found long strings don't work well in some DAP clients (VS Code). See the screenshots included in https://github.com/go-delve/delve/issues/3323#issuecomment-1499639519. When the go debug adapter includes both package path (corresponding to "module" in other languages) and the method name in the name, it can produce a very long string.
StackFrameFormat was brought up while we were discussing how to surface both package and method name in the current VS Code UI (https://github.com/microsoft/vscode/issues/193153). According to the conversation in https://github.com/microsoft/debug-adapter-protocol/issues/411, the debug adapter is responsible for formatting and producing the string based on StackFrameFormat. Now where should this potentially long formatted string go?
In the current spec, I cannot find any place other than name, and we are again back to the original problem. Some DAP clients cannot efficiently present a long stack frame name.
What do you think about having an optional longer description field?
I think VS Code can potentially present the longer description in tooltip.
Other DAP clients can also choose to select name and the new longer description based on the need, instead of arbitrarily hiding or truncating name.
- Ngôn ngữ chính
- HTML
- Star
- 1.8k
- Fork
- 173
- Merge trung bình
- 7 ngày 7 giờ
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của microsoft/debug-adapter-protocol
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
microsoft/debug-adapter-protocol#633 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
microsoft/debug-adapter-protocol#599 · 8 reaction ·
-
under-discussion
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 48/100
microsoft/debug-adapter-protocol#596 · 9 bình luận ·