Repository navigation
Adding LSP: PHP Intelephense - #4504
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for the PHP Intelephense language server to OpenCode. Intelephense is a freemium LSP that provides PHP language features like autocomplete, diagnostics, and code intelligence.
Key changes:
- Added PHP Intelephense LSP server configuration with auto-installation support
- Updated documentation to include the new LSP server in the built-in servers table
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| packages/opencode/src/lsp/server.ts | Implements the PHPIntelephense LSP server with auto-installation of the intelephense npm package for PHP projects |
| packages/web/src/content/docs/lsp.mdx | Adds PHP Intelephense entry to the built-in LSP servers table |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Yeah we should defo document |
|
Awesome, done 👍 |
|
@dbpolito you can ignore the test failure, but what happens fi they dont have the license? Does it error or will it just not do anything |
|
It's freemium, it just works, but with license it enables some premium features: https://intelephense.com/ This is the most used LSP for php: https://marketplace.visualstudio.com/items?itemName=bmewburn.vscode-intelephense-client 16M |
|
awesome just wanted to validate that |
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Adding LSP: PHP Intelephense
Intelephense is a freemium LSP, which means you can also add a license to enable the some premium features, probably who buys it will know how to setup but in order to make it work with opencode we need some extra config, which you can see at https://intelephense.com/docs:
Wondering if we should mention that on the docs, or just leave for the user to find out.