Allow hinting `DebugProtocol.Source` to prefer `path` over `sourceReference`
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript, nodejs, typescript
- Lĩnh vực
- api
Hướng nghiên cứu
Bắt đầu với định nghĩa DebugProtocol.Source và quy tắc hiện tại rằng một sourceReference dương được ưu tiên hơn path. Xác định rõ cơ chế ưu tiên, việc xác thực checksum và ngữ nghĩa fallback, sau đó cập nhật định nghĩa giao thức và các ví dụ để các client có thể nhất quán lựa chọn giữa một đường dẫn có thể chỉnh sửa và một snapshot sourceReference.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
It'd be nice if a new field (e.g. preference) gets added to DebugProtocol.Source to tell clients (IDEs) that if the Source can be mapped to a file, it should open that file instead of using sourceReference. Perhaps it's even worth making this the default, especially when combined with my item of checknotes below.
Details
- It's not uncommon to end up debugging a program for which the input source has changed.
- As in, the debugger still knows the original input source that the program is currently running.
- A good example is NodeJS with JavaScript, and even TypeScript if the (loaded) source maps also contain the source.
- Currently the protocol states that a
sourceReferencebigger than 0 means that should be used, regardless ofpath.- This would either be a "breaking" change (although it won't break anything, IDEs just wouldn't show "modern" behavior).
- We can specify to only prefer
pathifchecksumsis present to validate whether it results in the correct file (contents).
- It is more user friendly in IDEs if a
Sourcecan be mapped to the actual source file, instead of a snapshot in the debugger.- The user might already have the file open, and focusing it instead of opening a cached file is preferred.
- The user can now also directly edit the file, without having to first look for the correct file based on the read-only snapshot.
- The IDE might find the right file, but maybe its contents have changed:
- We can use
checksumsto validate whether it is (the correct version of) the correct file. If not, revert tosourceReference.
- We can use
- We have the
sourcesfield but this isn't (meant as) a 1-on-1 mapping of the source itself. - Having the debug adapter only provide
pathis not enough:- If the file isn't found, the IDE has nothing to show.
- If the file changed (and
checksumsisn't present), the IDE might display an outdated (or completely wrong) file. - If
checksumsis present, the IDE can validate whether the file (content) is correct, but not fallback to anything if it isn't.
This is mostly a big quality-of-life improvement towards IDE users, as these would now prefer the actual source files over cached versions provided by the debug adapter, assuming they're the same (using checksums). While the file content will be the exact same, mapping to an editable source file or mapping to a read-only "debug content file" is quite a large difference.
Hotreload example
Since this would still allow different Sources with the same path but a different sourceReference/checksums (and the IDE automatically picking between showing the path file or the read-only sourceReference viewer), we can also easily allow debug adapters to differentiate different versions of the same (path-specified) file, e.g. because of hotreloading:
- Program loads
a.jswithAAAas checksum. - Debugger attaches, adapter tells the IDE about
{ path: 'a.js', sourceReference: 1, checksum: ['AAA'] }.- IDE finds the properly mapped
a.js, validates the checksum and opens it.
- IDE finds the properly mapped
- User edits file and saves it, causing the program/debugger to hotreload it one way or another.
- Let's say the new content results in
BBBas checksum.
- Let's say the new content results in
- Debug adapter tells the IDE about
{ path: 'a.js', sourceReference: 2, checksum: ['BBB'] }.- IDE finds the file again (which might still be open), sees the checksum matches the (new) content, so opens it.
- Debug adapter might tell IDE again about the previous file (ref 1, checksum
AAA).- IDE finds the file but notices the different checksum. Falls back to using the
sourceReferencewith value1.
- IDE finds the file but notices the different checksum. Falls back to using the
- Debug adapter might tell IDE again about the current file (ref 2, checksum
BBB) after it was saved, but before the hotreload.- IDE finds the file but notices the different checksum. Falls back to using the
sourceReferencewith value2.
- IDE finds the file but notices the different checksum. Falls back to using the
Of course, this hotreload example is an advanced use case this change would allow. Just the QoL improvement to open the true (editable) source file instead of a snapshot if possible is already a very nice use case this allows.
- 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
Hướng dẫn đóng góp
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 ·
Tất cả issue của microsoft/debug-adapter-protocol
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
ArrayCollection::matching() throws a TypeError for composite expressions over array elements Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
doctrine/collections#554 · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
receptron/mulmoterminal#2264 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
openfoodfacts/robotoff#1957 ·
-
Replace biplist with plistlib Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
django-tastypie/django-tastypie#1702 ·