multilspy icon indicating copy to clipboard operation
multilspy copied to clipboard

Php support

Open v4rgas opened this issue 7 months ago • 5 comments

Closes #106

I tried using phpactor since it is open source but it was not working properly, every time I ran it It found more and more repeated references for no reason at all.

Inteliphense worked out of the box, the only bad thing is that you need to install the dependency with npm and that is it closed source, maybe we should add a warning when running multilspy with intelephense?

anyways, ill add a few more test but for now ill open the pr anyways

pd: I also fixed the readme since it was missing some languages

v4rgas avatar May 01 '25 02:05 v4rgas

its only finding the references within the same file, would u mind helping me debug that? i cannot understand why isnt indexing the entire project

i tried modifying the initialization params but i had no luck, am i missing something?

v4rgas avatar May 01 '25 04:05 v4rgas

We added php support via intelephense to our multilspy copy, could be useful for you as reference. Since there's quite some deviation from the original in our version at the moment, the code would need to be a bit adapted for merging back to multilspy. But it's a straightforward task.

The functionality of the new LS is tested in CI on windows, ubuntu and macos - everything seems to work fine. https://github.com/oraios/serena/pull/119

MischaPanch avatar May 19 '25 18:05 MischaPanch

its only finding the references within the same file, would u mind helping me debug that? i cannot understand why isnt indexing the entire project

I also had to face an fix this issue. I think it's a bug in the LS, we "fixed" it by just waiting a bit before requesting references and definitions. You'll see it in the PR I mentioned above

MischaPanch avatar May 19 '25 23:05 MischaPanch

thanks for letting me know, ill check it out

v4rgas avatar May 20 '25 20:05 v4rgas

What about this one? https://github.com/felixfbecker/php-language-server

Edit: hmm, that one is not maintained. :/

jkuchar avatar Aug 17 '25 07:08 jkuchar