-
Notifications
You must be signed in to change notification settings - Fork 29.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
debug requirements for TypeScript #88857
Comments
Lower priority:
|
@mjbvz Thanks a lot! I'm very interested in that information, but the link does not work for me. |
You need to be on corpnet for this. The specific call to the TS Server is: var result = await this.taskHandler.InvokeTSServerAsync(CommandNames.NameOrDottedNameSpan, document, TaskPriority.High, cancellationToken,
(proxy, endpoint, operationToken) => proxy.GetNameOrDottedNameSpan(endpoint, document, endPoint.Value.Position, operationToken)); We don't use this command currently |
Request for an InlineValueProvider: #119489 (comment) |
@hediet brought up the state of const enums. This can partially be solved by sourcemap renames (which we will support some day, #12066) but not very well. There's also been the request to transpile typescript in the REPL, which @eamodio asked for. Both of these would be solved if I could ask the TS server to transpile this bit of code as if it was written at a certain position/scope in a file, though that may be easier said than done. |
THis is now done with the sourcemap "names", which are now pretty good following microsoft/vscode-js-debug#1827
Work is happening in that in https://github.com/tc39/source-map-rfc
Covered by #18058, merging |
from #84044:
E.g. for any character position in the string "doc.foo[a+b].x > foo" return the "evaluatable" expression that should be shown in the hover.
The JS debugger only sees JS variable names and the debug extension needs to have a way of mapping JS names to TS names (and vice versa). Source maps provide a mechanism for that but the maps were not complete and aren't used in our debuggers I think.
The text was updated successfully, but these errors were encountered: