Php-language-server: Workspace Symbols not including packages in vendor directory

[adevade]

OS: Windows 10
VS Code: 1.15.1 x64

I have a Laravel 5.5 app, and when I hit CTRL+T for Workspace Symbol search it doesn't include any files from the vendor directory except Composers own autoloader-files.

Here's some info from the Output->PHP Language Server tab

[Info  - 22:55:32] 74 Packages
[Info  - 22:55:32] Indexing laravel/framework for definitions and static 
..........
[Warn - 22:56:05] Could not compute cache key for laravel/framework
[Info  - 22:56:05] Restored nesbot/carbon:1.22.1 from cache
..........
[Info  - 22:56:07] All 4962 PHP files parsed in 36 seconds. 282 MiB allocated.

Here's without any search input. As you can see it loads my own files (in ./app and root directory) just fine.

Here's when I search for Laravels Illuminate packages.

Here's when I search for Carbon.

Here's when I search for Composer.

[duplicate-issues[bot]]

Hey @adevade,

We did a quick check and this issue looks very darn similar to

• #466 - how do i have it store intellisense data, so it doesnt get rerun on every program start
• #465 - Cpu load, memory leak
• #464 - Find reference error

This could be a coincidence, but if any of these issues solves your problem then I did a good job :smile:

If not, the maintainers will get to this issue shortly.

Cheers,
Your Friendly Neighborhood ProBot

[felixfbecker]

This is by design. workspace/symbol is designed to return the symbols defined in your own workspace. It would also be very slow to return all matches from dependencies.

[adamwathan]

This is by design. workspace/symbol is designed to return the symbols defined in your own workspace. It would also be very slow to return all matches from dependencies.

That's too bad 😕 Sublime handles this really well with no performance issues, would be awesome to have support for it in Code!

Thanks for all your hard work on these packages regardless, incredibly helpful stuff 👍

[hoangnamitc]

excuse me, but my project.
If "new Class()" inside one file it working good, but if "new Class()" in other file after include it not working definition.

after:

[giulianoriccio]

@felixfbecker could this be made optional?
We could use the built-in option to exclude the path if necessary.