[typescript] avoid activation on startup if there is no other activation event

[AlexTugarev]

Fixes #6163

How to test

• start a workspace containing ts/js files and see the language service is working as usual
• start a workspace without any ts/js files contained and see there is no tsserver running

Signed-off-by: Alex Tugarev alex.tugarev@typefox.io

[akosyakov]

Should not we generally fix restart method? that it does not do anything if nothin is running?

[AlexTugarev]

Should not we generally fix restart method? that it does not do anything if nothin is running?

really good question. is there a chance that activation might fail, but a second try would be successful? in that case, calling restart manually would stop working at all, no?

[akosyakov]

really good question. is there a chance that activation might fail, but a second try would be successful? in that case, calling restart manually would stop working at all, no?

restart is not available if the server is not running: https://github.com/theia-ide/theia/blob/732ef15589b668c456590276bebba9fcfa37f409/packages/languages/src/browser/languages-frontend-contribution.ts#L75-L82

I think it is safe to add a guard in restart method as well.

[AlexTugarev]

@akosyakov I changed again and added the guard in restart method.

[akosyakov]

@AlexTugarev check boxes are for reviewers or for you? :)

[AlexTugarev]

@AlexTugarev check boxes are for reviewers or for you? :)

for reviewers ;-)

[akosyakov]

tested that ts support is there if a project contains tsconfig and is not there if not

[akosyakov]

@AlexTugarev please rebase and if the ci is green please merge