Skip to content

Debug Failure. False expression. in findPrecedingToken during getQuickInfoAtPosition #20560

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

Closed
aozgaa opened this issue Dec 8, 2017 · 4 comments
Labels
Bug A bug in TypeScript Duplicate An existing issue was already created Source: Telemetry The issue relates to the telemetry in editors

Comments

@aozgaa
Copy link
Contributor

aozgaa commented Dec 8, 2017

tsserver version: 2.6.1
hits: at least 570381
stack:

Error: Debug Failure. False expression.
    at findPrecedingToken (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62860:18)
    at getTokenAtPositionWorker (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62822:41)
    at getTouchingToken (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62794:16)
    at Object.getTouchingPropertyName (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:62790:16)
    at Object.getQuickInfoAtPosition (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:79791:27)
    at IOSession.Session.getQuickInfoWorker (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:86102:62)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:85266:61)
    at /Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:86669:88
    at IOSession.Session.executeWithRequestId (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:86660:28)
    at IOSession.Session.executeCommand (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:86669:33)
    at IOSession.Session.onMessage (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:86689:35)
    at Interface.<anonymous> (/Applications/Visual Studio Code.app/Contents/Resources/app/extensions/node_modules/typescript/lib/tsserver.js:87881:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)
@ghost
Copy link

ghost commented Dec 11, 2017

@aozgaa It looks like that assertion was added in #16385.

@aozgaa aozgaa changed the title Debug Failure in findPrecedingToken during getQuickInfoAtPosition Debug Failure. False expression. in findPrecedingToken during getQuickInfoAtPosition Dec 19, 2017
@DanielRosenwasser DanielRosenwasser added the Source: Telemetry The issue relates to the telemetry in editors label Dec 19, 2017
@mhegazy mhegazy added the Bug A bug in TypeScript label Jan 20, 2018
@mhegazy mhegazy added this to the TypeScript 2.8 milestone Jan 20, 2018
@mhegazy
Copy link
Contributor

mhegazy commented Jan 20, 2018

The Source: Telemetry have limited information. we only have the call stack from the crash report. We need to either mitigate the crash, or add additional asserts to get more information in future crash reports.

@aozgaa
Copy link
Contributor Author

aozgaa commented Jan 30, 2018

repro's in 2.7.0 with the following statistics:

TypeScript Version:
2.7.0
VSCode Version
1.20.0-insider
command requested
quickinfo
hitting sessions
1812
proportion of all sessions
0.0319047786738036

@mhegazy
Copy link
Contributor

mhegazy commented Feb 9, 2018

Closing in favor of #21799

@mhegazy mhegazy closed this as completed Feb 9, 2018
@mhegazy mhegazy added the Duplicate An existing issue was already created label Feb 9, 2018
@mhegazy mhegazy removed this from the TypeScript 2.8 milestone Feb 9, 2018
@microsoft microsoft locked and limited conversation to collaborators Jul 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A bug in TypeScript Duplicate An existing issue was already created Source: Telemetry The issue relates to the telemetry in editors
Projects
None yet
Development

No branches or pull requests

3 participants