Skip to content

Commit ccb6935

Browse files
committed
feat: enable LSP (Language Server Protocol) infrastructure
This commit enables the complete LSP infrastructure that was previously disabled in Phoenix. The LSP system provides a foundation for language server integration supporting features like code completion, parameter hints, jump to definition, find references, and diagnostics. Changes: - Enabled LSP initialization in ClientLoader.js by uncommenting initDomainAndHandleNodeCrash() call - Enabled LSP unit tests in UnitTestSuite.js The LSP infrastructure includes: - Complete Language Client implementation with Node.js bridge - Protocol adapter for LSP communication (JSON-RPC over IPC/stdio) - Default providers for code hints, parameter hints, and navigation - Support for all standard LSP features defined in ToolingInfo.json - Test suite with mock language servers This enables the platform for JavaScript/TypeScript and other language server integrations. Language-specific clients can now be registered using LanguageTools.initiateToolingService() API.
1 parent b10c829 commit ccb6935

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/languageTools/ClientLoader.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ define(function (require, exports, module) {
171171
console.error("ClientInfo domain could not be loaded: ", err);
172172
});
173173
}
174-
//initDomainAndHandleNodeCrash();
174+
initDomainAndHandleNodeCrash();
175175

176176

177177
exports.initiateLanguageClient = initiateLanguageClient;

test/UnitTestSuite.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,6 @@ define(function (require, exports, module) {
143143
// Node Tests
144144
require("spec/NodeConnection-test");
145145
// todo TEST_MODERN
146-
// require("spec/LanguageTools-test"); LSP tests. disabled for now
146+
require("spec/LanguageTools-test");
147147
// require("spec/Menu-native-integ-test"); evaluate after we have native menus in os installed builds
148148
});

0 commit comments

Comments
 (0)