Skip to content

Commit 9d4fcf8

Browse files
committed
use language client 8.0.0-next.4
Signed-off-by: Shi Chen <chenshi@microsoft.com>
1 parent 6765a6e commit 9d4fcf8

6 files changed

+50
-56
lines changed

package-lock.json

Lines changed: 34 additions & 44 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1414,7 +1414,7 @@
14141414
"htmlparser2": "6.0.1",
14151415
"jdk-utils": "^0.4.4",
14161416
"semver": "^7.3.5",
1417-
"vscode-languageclient": "7.1.0-next.5",
1417+
"vscode-languageclient": "8.0.0-next.4",
14181418
"winreg-utf8": "^0.1.1",
14191419
"winston": "^3.2.1",
14201420
"winston-daily-rotate-file": "^3.10.0"

src/extension.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -174,8 +174,8 @@ export function activate(context: ExtensionContext): Promise<ExtensionAPI> {
174174
},
175175
middleware: {
176176
workspace: {
177-
didChangeConfiguration: () => {
178-
standardClient.getClient().sendNotification(DidChangeConfigurationNotification.type, {
177+
didChangeConfiguration: async () => {
178+
await standardClient.getClient().sendNotification(DidChangeConfigurationNotification.type, {
179179
settings: {
180180
java: getJavaConfig(requirements.java_home),
181181
}
@@ -208,7 +208,7 @@ export function activate(context: ExtensionContext): Promise<ExtensionAPI> {
208208
const codeActionContext: CodeActionContext = {
209209
diagnostics: allDiagnostics,
210210
only: context.only,
211-
triggerKind: CodeActionTriggerKind.Invoke,
211+
triggerKind: context.triggerKind,
212212
};
213213
params.context = client.code2ProtocolConverter.asCodeActionContext(codeActionContext);
214214
}

src/standardLanguageClient.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -323,15 +323,19 @@ export class StandardLanguageClient {
323323
commands.executeCommand(Commands.SHOW_REFERENCES, Uri.parse(uri), this.languageClient.protocol2CodeConverter.asPosition(position), locations.map(this.languageClient.protocol2CodeConverter.asLocation));
324324
}));
325325

326-
context.subscriptions.push(commands.registerCommand(Commands.CONFIGURATION_UPDATE, uri => projectConfigurationUpdate(this.languageClient, uri)));
326+
context.subscriptions.push(commands.registerCommand(Commands.CONFIGURATION_UPDATE, async (uri) => {
327+
await projectConfigurationUpdate(this.languageClient, uri);
328+
}));
327329

328330
context.subscriptions.push(commands.registerCommand(Commands.IGNORE_INCOMPLETE_CLASSPATH, () => setIncompleteClasspathSeverity('ignore')));
329331

330332
context.subscriptions.push(commands.registerCommand(Commands.IGNORE_INCOMPLETE_CLASSPATH_HELP, () => {
331333
commands.executeCommand(Commands.OPEN_BROWSER, Uri.parse('https://github.com/redhat-developer/vscode-java/wiki/%22Classpath-is-incomplete%22-warning'));
332334
}));
333335

334-
context.subscriptions.push(commands.registerCommand(Commands.PROJECT_CONFIGURATION_STATUS, (uri, status) => setProjectConfigurationUpdate(this.languageClient, uri, status)));
336+
context.subscriptions.push(commands.registerCommand(Commands.PROJECT_CONFIGURATION_STATUS, async (uri, status) => {
337+
await setProjectConfigurationUpdate(this.languageClient, uri, status);
338+
}));
335339

336340
context.subscriptions.push(commands.registerCommand(Commands.NULL_ANALYSIS_SET_MODE, (status) => setNullAnalysisStatus(status)));
337341

@@ -673,7 +677,7 @@ function setIncompleteClasspathSeverity(severity: string) {
673677
);
674678
}
675679

676-
function setProjectConfigurationUpdate(languageClient: LanguageClient, uri: Uri, status: FeatureStatus) {
680+
async function setProjectConfigurationUpdate(languageClient: LanguageClient, uri: Uri, status: FeatureStatus) {
677681
const config = getJavaConfiguration();
678682
const section = 'configuration.updateBuildConfiguration';
679683

@@ -683,7 +687,7 @@ function setProjectConfigurationUpdate(languageClient: LanguageClient, uri: Uri,
683687
(error) => logger.error(error)
684688
);
685689
if (status !== FeatureStatus.disabled) {
686-
projectConfigurationUpdate(languageClient, uri);
690+
await projectConfigurationUpdate(languageClient, uri);
687691
}
688692
}
689693

src/standardLanguageClientUtils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,11 @@ export async function projectConfigurationUpdate(languageClient: LanguageClient,
3333
}
3434

3535
if (resources.length === 1) {
36-
languageClient.sendNotification(ProjectConfigurationUpdateRequest.type, {
36+
await languageClient.sendNotification(ProjectConfigurationUpdateRequest.type, {
3737
uri: resources[0].toString(),
3838
});
3939
} else if (resources.length > 1) {
40-
languageClient.sendNotification(ProjectConfigurationUpdateRequest.typeV2, {
40+
await languageClient.sendNotification(ProjectConfigurationUpdateRequest.typeV2, {
4141
identifiers: resources.map(r => {
4242
return { uri: r.toString() };
4343
}),

src/syntaxLanguageClient.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,8 @@ export class SyntaxLanguageClient {
2424
const newClientOptions: LanguageClientOptions = Object.assign({}, clientOptions, {
2525
middleware: {
2626
workspace: {
27-
didChangeConfiguration: () => {
28-
this.languageClient.sendNotification(DidChangeConfigurationNotification.type, {
27+
didChangeConfiguration: async () => {
28+
await this.languageClient.sendNotification(DidChangeConfigurationNotification.type, {
2929
settings: {
3030
java: getJavaConfig(requirements.java_home),
3131
}

0 commit comments

Comments
 (0)