TypeError: Cannot read property 'startRecordingFilesWithChangedResolutions' of undefined

[amcasey]

From 3.1.3:

Error processing request. Cannot read property 'startRecordingFilesWithChangedResolutions' of undefined
TypeError: Cannot read property 'startRecordingFilesWithChangedResolutions' of undefined
    at InferredProject.Project.updateGraph (tsserver.js:118761:38)
    at updateProjectIfDirty (tsserver.js:119449:45)
    at updateGraph (tsserver.js:119734:34)
    at Array.forEach (<anonymous>)
    at ProjectService.ensureProjectStructuresUptoDate (tsserver.js:119738:39)
    at ProjectService.doEnsureDefaultProjectForFile (tsserver.js:119722:22)
    at ProjectService.ensureDefaultProjectForFile (tsserver.js:119719:75)
    at IOSession.Session.getFileAndProjectWorker (tsserver.js:122451:87)
    at IOSession.Session.getFileAndProject (tsserver.js:122436:29)
    at IOSession.Session.getCompletions (tsserver.js:122605:31)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:121665:61)
    at tsserver.js:123086:88
    at IOSession.Session.executeWithRequestId (tsserver.js:123077:28)
    at IOSession.Session.executeCommand (tsserver.js:123086:33)
    at IOSession.Session.onMessage (tsserver.js:123106:35)
    at Interface.<anonymous> (tsserver.js:124369:27)
    at emitOne (events.js:116:13)
    at Interface.emit (events.js:211:7)
    at Interface._onLine (readline.js:282:10)
    at Interface._normalWrite (readline.js:424:12)
    at Socket.ondata (readline.js:141:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

[amcasey]

Command was completionInfo.

[sheetalkamat]

OK this assert is possible only when the project closed (all the internal datastructures are set to undefined) is not removed from the list of inferred projects (that's the project it was asserting on). This is possible only when assert to verify all the file info's attached to the project are detached from this project (https://github.com/Microsoft/TypeScript/blob/master/src/server/editorServices.ts#L1036). So do we have any hits on that assert?

[amcasey]

I'll look but, for future reference, it's much easier to search for asserts that have message strings.

[amcasey]

I don't see any assert stacks containing "removeProject", which it seems like they'd have to for that assert.

[sheetalkamat]

Even though there were not stacks for remove project I realized that 94df516 fixed lot of issues with incorrect script info still being attached to the project and then not being removed correctly. I think that should suffice. but as @amcasey suggests will add more info to assert for future.

[sheetalkamat]

Closing this to be re-opened if there is more info