From 66a94d727457781389adc40a9f48e87c82bbd0a9 Mon Sep 17 00:00:00 2001
From: Michael Panchenko <michael.panchenko@unternehmertum.de>
Date: Mon, 19 May 2025 16:18:59 +0200
Subject: [PATCH] PHP: try waiting before requesting references

Same fix was used for typescript. PHP tests don't fail on my system but do fail in CI
---
 .../intelephense/intelephense.py              | 26 ++++++++++---------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/src/multilspy/language_servers/intelephense/intelephense.py b/src/multilspy/language_servers/intelephense/intelephense.py
index 4533d590..7d976c2d 100644
--- a/src/multilspy/language_servers/intelephense/intelephense.py
+++ b/src/multilspy/language_servers/intelephense/intelephense.py
@@ -9,8 +9,8 @@ import logging
 import os
 import subprocess
 import pathlib
-import stat
 from contextlib import asynccontextmanager
+from time import sleep
 from typing import AsyncIterator
 
 from overrides import override
@@ -22,16 +22,6 @@ from multilspy.lsp_protocol_handler.lsp_types import InitializeParams
 from multilspy.multilspy_config import MultilspyConfig
 from multilspy.multilspy_utils import PlatformUtils, PlatformId
 
-# Platform-specific imports
-if os.name != 'nt':  # Unix-like systems
-    import pwd
-else:
-    # Dummy pwd module for Windows
-    class pwd:
-        @staticmethod
-        def getpwuid(uid):
-            return type('obj', (), {'pw_name': os.environ.get('USERNAME', 'unknown')})()
-
 class Intelephense(LanguageServer):
     """
     Provides PHP specific instantiation of the LanguageServer class using Intelephense.
@@ -91,6 +81,7 @@ class Intelephense(LanguageServer):
                     )
                 else:
                     # On Unix-like systems, run as non-root user
+                    import pwd
                     user = pwd.getpwuid(os.getuid()).pw_name
                     subprocess.run(
                         dependency["command"],
@@ -192,4 +183,15 @@ class Intelephense(LanguageServer):
             yield self
 
             await self.server.shutdown()
-            await self.server.stop()
\ No newline at end of file
+            await self.server.stop()
+            
+    @override
+    # For some reason, the LS may need longer to process this, so we just retry
+    async def _send_references_request(self, relative_file_path: str, line: int, column: int):
+        # TODO: The LS doesn't return references contained in other files if it doesn't sleep. This is
+        #   despite the LS having processed requests already. I don't know what causes this, but sleeping
+        #   one second helps. It may be that sleeping only once is enough but that's hard to reliably test.
+        #   It may be that even this 1sec is not enough in larger TS projects, at some point we should find what
+        #   causes this and solve it.
+        sleep(1)
+        return await super()._send_references_request(relative_file_path, line, column)
\ No newline at end of file