TSServer: navto command should be supported without a file

[dbaeumer]

Currently the file is only used to detect the virtual project. Now that we have tsconfig.json support we can use the language service bound to a tsconfig.json file to find navto items. I would propose the following handling in case of the absense of a file:

• no tsconfig project. Command return no content message
• all projects are tsconfig project: return the nav to items (filter duplicates)
• there is a mix: the response contains a flag indicating that the result might be incomplete.

[mhegazy]

I find it strange that you would get results from all projects even if you have a file open which points you to a single project. Can you explain the scenario you want to enable?

I think we have a related issue (#2531), if that fixed, given a file you would be getting all results from related projects, but not unrelated. would that satisfy your scenario?

[dbaeumer]

If I think about a workspace with n components I would have a tsconfig.json file per component. Now if I want to navigate in that workspace I would like to see a list of all navto items accross all components. For me that is comparable to a VS solution having n projects where project === tsconfig.json

[dbaeumer]

So to answer your question: no.

I would still like to see a mode where I can see all items.

[mhegazy]

mmm. this is interesting, i think this is the a good scenario to support; but how would you make this not noisy in the general case? imagine a few files open, each with a few .d.ts files, now you get navigate to results, and you get a lot of results that may or may not be what you are looking for.

[steveluc]

I think the only way to make this not confusing is to have some way of expressing project relationships. So far we have chosen not to do this, leaving that level of specification to project systems such as grunt and gulp.

I agree with Mohamed this is a good scenario and we should support it in some way. We could have two different commands in Sublime, one that just searches through every open project and one that searches only the projects that include the current file. This could be supported in the protocol, as Dirk suggests, by making the file optional.

Either way, I think this would be more helpful and less confusing if we implement a way to explore the set of currently active projects, as Mohamed has previously suggested.

Steve

From: Mohamed Hegazy [mailto:notifications@github.com]
Sent: Monday, March 30, 2015 11:02 AM
To: Microsoft/TypeScript
Subject: Re: [TypeScript] TSServer: navto command should be supported without a file (#2530)

mmm. this is interesting, i think this is the a good scenario to support; but how would you make this not noisy in the general case? imagine a few files open, each with a few .d.ts files, now you get navigate to results, and you get a lot of results that may or may not be what you are looking for.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/2530#issuecomment-87774495.

[dbaeumer]

I gave it some more thoughts and simply providing no file will not do the trick. It will force the TS service to search for all tsconfig.json files and open them. And that might not be what the user wants.

So if a workspace has more than one tsconfig.json file I think we need something like a tsglobal.json file which describes the valid tsconfig.json files and there dependencies. The TS service could then open the tsglobal.json file and act appropriately.

And I like the idea of being able to explore the set of active projects. It would help the user to understand the behavior of some global actions better.