Filter for Java (server) snippet

Fixes #265

Signed-off-by: azerr <azerr@redhat.com>

microprofile.jdt/com.redhat.microprofile.jdt.core/src/main/java/com/redhat/microprofile/commons/MicroProfileJavaProjectLabelsParams.java

@@ -11,6 +11,8 @@
 *******************************************************************************/
 package com.redhat.microprofile.commons;
 
+import java.util.List;
+
 /**
  * MicroProfile Java Project labels
  * 
@@ -21,6 +23,8 @@ public class MicroProfileJavaProjectLabelsParams {
 
 	private String uri;
 
+	private List<String> types;
+
 	/**
 	 * Returns the Java file uri.
 	 * 
@@ -38,4 +42,12 @@ public String getUri() {
 	public void setUri(String uri) {
 		this.uri = uri;
 	}
+
+	public List<String> getTypes() {
+		return types;
+	}
+
+	public void setTypes(List<String> types) {
+		this.types = types;
+	}
 }

microprofile.jdt/com.redhat.microprofile.jdt.core/src/main/java/com/redhat/microprofile/jdt/core/ProjectLabelManager.java

@@ -26,6 +26,7 @@
 import com.redhat.microprofile.commons.ProjectLabelInfoEntry;
 import com.redhat.microprofile.jdt.core.utils.IJDTUtils;
 import com.redhat.microprofile.jdt.core.utils.JDTMicroProfileUtils;
+import com.redhat.microprofile.jdt.core.utils.JDTTypeUtils;
 import com.redhat.microprofile.jdt.internal.core.ProjectLabelRegistry;
 
 /**
@@ -54,7 +55,7 @@ public List<ProjectLabelInfoEntry> getProjectLabelInfo() {
 		IProject[] projects = ResourcesPlugin.getWorkspace().getRoot().getProjects();
 
 		for (IProject project : projects) {
-			ProjectLabelInfoEntry info = getProjectLabelInfo(project);
+			ProjectLabelInfoEntry info = getProjectLabelInfo(project, null);
 			if (info != null) {
 				results.add(info);
 			}
@@ -66,12 +67,13 @@ public List<ProjectLabelInfoEntry> getProjectLabelInfo() {
 	 * Returns project label results for the given Eclipse project.
 	 *
 	 * @param project Eclipse project.
+	 * @param types
 	 * @return project label results for the given Eclipse project.
 	 */
-	private ProjectLabelInfoEntry getProjectLabelInfo(IProject project) {
+	private ProjectLabelInfoEntry getProjectLabelInfo(IProject project, List<String> types) {
 		String uri = JDTMicroProfileUtils.getProjectURI(project);
 		if (uri != null) {
-			return new ProjectLabelInfoEntry(uri, getProjectLabels(project));
+			return new ProjectLabelInfoEntry(uri, getProjectLabels(project, types));
 		}
 		return null;
 	}
@@ -91,10 +93,10 @@ public ProjectLabelInfoEntry getProjectLabelInfo(MicroProfileJavaProjectLabelsPa
 			// The uri doesn't belong to an Eclipse project
 			return ProjectLabelInfoEntry.EMPTY_PROJECT_INFO;
 		}
-		return getProjectLabelInfo(file.getProject());
+		return getProjectLabelInfo(file.getProject(), params.getTypes());
 	}
 
-	private List<String> getProjectLabels(IProject project) {
+	private List<String> getProjectLabels(IProject project, List<String> types) {
 		IJavaProject javaProject = JavaCore.create(project);
 
 		if (javaProject == null) {
@@ -106,6 +108,13 @@ private List<String> getProjectLabels(IProject project) {
 		for (ProjectLabelDefinition definition : definitions) {
 			projectLabels.addAll(definition.getProjectLabels(javaProject));
 		}
+		if (types != null) {
+			for (String type : types) {
+				if (JDTTypeUtils.findType(javaProject, type) != null) {
+					projectLabels.add(type);
+				}
+			}
+		}
 
 		return projectLabels;
 	}

microprofile.jdt/com.redhat.microprofile.jdt.core/src/main/java/com/redhat/microprofile/jdt/internal/core/MicroProfilePropertiesListenerManager.java

@@ -1,11 +1,9 @@
 /*******************************************************************************
-* Copyright (c) 2019-2020 Red Hat Inc. and others.
+* Copyright (c) 2019 Red Hat Inc. and others.
 * All rights reserved. This program and the accompanying materials
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/legal/epl-v20.html
 *
-* SPDX-License-Identifier: EPL-2.0
-*
 * Contributors:
 *     Red Hat Inc. - initial API and implementation
 *******************************************************************************/
@@ -151,7 +149,7 @@ public boolean visit(IResourceDelta delta) throws CoreException {
 				return resource.isAccessible();
 			case IResource.FILE:
 				IFile file = (IFile) resource;
-				if (isJavaFile(file) && isFileContentChanged(delta)) {
+				if (isJavaFile(file)) {
 					// A Java file has been saved
 					MicroProfilePropertiesChangeEvent event = new MicroProfilePropertiesChangeEvent();
 					event.setType(MicroProfilePropertiesScope.ONLY_SOURCES);
@@ -186,10 +184,6 @@ private boolean isJavaFile(IFile file) {
 			return JAVA_FILE_EXTENSION.equals(file.getFileExtension());
 		}
 
-		private boolean isFileContentChanged(IResourceDelta delta) {
-			return (delta.getKind() == IResourceDelta.CHANGED && (delta.getFlags() & IResourceDelta.CONTENT) != 0);
-		}
-
 	}
 
 	private MicroProfileListener microprofileListener;

microprofile.jdt/com.redhat.microprofile.jdt.core/src/main/java/com/redhat/microprofile/jdt/internal/core/ls/JavaProjectDelegateCommandHandler.java

@@ -11,6 +11,7 @@
 
 import static com.redhat.microprofile.jdt.internal.core.ls.ArgumentUtils.getFirst;
 import static com.redhat.microprofile.jdt.internal.core.ls.ArgumentUtils.getString;
+import static com.redhat.microprofile.jdt.internal.core.ls.ArgumentUtils.getStringList;
 
 import java.util.List;
 import java.util.Map;
@@ -57,8 +58,10 @@ private static Object getProjectLabelInfo(List<Object> arguments, String command
 					"Command '%s' must be called with required MicroProfileJavaProjectLabelsParams.uri (java file URI)!",
 					commandId));
 		}
+		List<String> types = getStringList(obj, "types");
 		MicroProfileJavaProjectLabelsParams params = new MicroProfileJavaProjectLabelsParams();
 		params.setUri(javaFileUri);
+		params.setTypes(types);
 		return ProjectLabelManager.getInstance().getProjectLabelInfo(params, JDTUtilsLSImpl.getInstance(), monitor);
 	}
 }

microprofile.ls/com.redhat.microprofile.ls/src/main/java/com/redhat/microprofile/commons/MicroProfileJavaProjectLabelsParams.java

@@ -11,6 +11,8 @@
 *******************************************************************************/
 package com.redhat.microprofile.commons;
 
+import java.util.List;
+
 /**
  * MicroProfile Java Project labels
  * 
@@ -21,6 +23,8 @@ public class MicroProfileJavaProjectLabelsParams {
 
 	private String uri;
 
+	private List<String> types;
+
 	/**
 	 * Returns the Java file uri.
 	 * 
@@ -38,4 +42,12 @@ public String getUri() {
 	public void setUri(String uri) {
 		this.uri = uri;
 	}
+
+	public List<String> getTypes() {
+		return types;
+	}
+
+	public void setTypes(List<String> types) {
+		this.types = types;
+	}
 }

microprofile.ls/com.redhat.microprofile.ls/src/main/java/com/redhat/microprofile/ls/JavaTextDocumentService.java

@@ -54,7 +54,7 @@
 import com.redhat.microprofile.ls.commons.snippets.TextDocumentSnippetRegistry;
 import com.redhat.microprofile.settings.MicroProfileCodeLensSettings;
 import com.redhat.microprofile.settings.SharedSettings;
-import com.redhat.microprofile.snippets.LanguageId;
+import com.redhat.microprofile.snippets.SnippetContextForJava;
 
 /**
  * LSP text document service for Java file.
@@ -80,7 +80,8 @@ public JavaTextDocumentService(MicroProfileLanguageServer microprofileLanguageSe
 		this.documents = new JavaTextDocuments(microprofileLanguageServer);
 	}
 
-	// ------------------------------ did* for Java file ------------------------------
+	// ------------------------------ did* for Java file
+	// ------------------------------
 
 	@Override
 	public void didOpen(DidOpenTextDocumentParams params) {
@@ -110,32 +111,31 @@ public void didSave(DidSaveTextDocumentParams params) {
 
 	@Override
 	public CompletableFuture<Either<List<CompletionItem>, CompletionList>> completion(CompletionParams params) {
-		return CompletableFutures.computeAsync(cancel -> {
-			try {
-				// Returns java snippets
-				TextDocument document = documents.get(params.getTextDocument().getUri());
-				int completionOffset = document.offsetAt(params.getPosition());
-				boolean canSupportMarkdown = true;
-				CompletionList list = new CompletionList();
-				list.setItems(new ArrayList<>());
-				getSnippetRegistry().getCompletionItems(document, completionOffset, canSupportMarkdown, context -> {
-					return true;
-				}).forEach(item -> {
-					list.getItems().add(item);
-				});
-				return Either.forRight(list);
-			} catch (BadLocationException e) {
-				LOGGER.log(Level.SEVERE, "Error while getting java completions", e);
-				return Either.forRight(null);
-			}
-		});
-	}
-
-	private TextDocumentSnippetRegistry getSnippetRegistry() {
-		if (snippetRegistry == null) {
-			snippetRegistry = new TextDocumentSnippetRegistry(LanguageId.java.name());
-		}
-		return snippetRegistry;
+		JavaTextDocument document = documents.get(params.getTextDocument().getUri());
+		return document.executeIfInMicroProfileProject((projectInfo) -> {
+			return CompletableFutures.computeAsync(cancel -> {
+				try {
+					// Returns java snippets
+					int completionOffset = document.offsetAt(params.getPosition());
+					boolean canSupportMarkdown = true;
+					CompletionList list = new CompletionList();
+					list.setItems(new ArrayList<>());
+					documents.getSnippetRegistry()
+							.getCompletionItems(document, completionOffset, canSupportMarkdown, context -> {
+								if (context != null && context instanceof SnippetContextForJava) {
+									return ((SnippetContextForJava) context).isMatch(projectInfo);
+								}
+								return true;
+							}).forEach(item -> {
+								list.getItems().add(item);
+							});
+					return Either.forRight(list);
+				} catch (BadLocationException e) {
+					LOGGER.log(Level.SEVERE, "Error while getting java completions", e);
+					return Either.forRight(null);
+				}
+			});
+		}, Either.forRight(null));
 	}
 
 	// ------------------------------ Code Lens ------------------------------
@@ -152,7 +152,7 @@ public CompletableFuture<List<? extends CodeLens>> codeLens(CodeLensParams param
 			return CompletableFuture.completedFuture(Collections.emptyList());
 		}
 		JavaTextDocument document = documents.get(params.getTextDocument().getUri());
-		return document.executeIfInMicroProfileProject(() -> {
+		return document.executeIfInMicroProfileProject((projectInfo) -> {
 			MicroProfileJavaCodeLensParams javaParams = new MicroProfileJavaCodeLensParams(
 					params.getTextDocument().getUri());
 			if (sharedSettings.getCommandCapabilities().isCommandSupported(CommandKind.COMMAND_OPEN_URI)) {
@@ -171,7 +171,7 @@ public CompletableFuture<List<? extends CodeLens>> codeLens(CodeLensParams param
 	@Override
 	public CompletableFuture<List<Either<Command, CodeAction>>> codeAction(CodeActionParams params) {
 		JavaTextDocument document = documents.get(params.getTextDocument().getUri());
-		return document.executeIfInMicroProfileProject(() -> {
+		return document.executeIfInMicroProfileProject((projectInfo) -> {
 			MicroProfileJavaCodeActionParams javaParams = new MicroProfileJavaCodeActionParams();
 			javaParams.setTextDocument(params.getTextDocument());
 			javaParams.setRange(params.getRange());
@@ -195,7 +195,7 @@ public CompletableFuture<List<Either<Command, CodeAction>>> codeAction(CodeActio
 	@Override
 	public CompletableFuture<Hover> hover(HoverParams params) {
 		JavaTextDocument document = documents.get(params.getTextDocument().getUri());
-		return document.executeIfInMicroProfileProject(() -> {
+		return document.executeIfInMicroProfileProject((projectinfo) -> {
 			boolean markdownSupported = sharedSettings.getHoverSettings().isContentFormatSupported(MarkupKind.MARKDOWN);
 			DocumentFormat documentFormat = markdownSupported ? DocumentFormat.Markdown : DocumentFormat.PlainText;
 			MicroProfileJavaHoverParams javaParams = new MicroProfileJavaHoverParams(params.getTextDocument().getUri(),
@@ -212,7 +212,7 @@ public CompletableFuture<Hover> hover(HoverParams params) {
 	 * @param document the opened Java file.
 	 */
 	private void triggerValidationFor(JavaTextDocument document) {
-		document.executeIfInMicroProfileProject(() -> {
+		document.executeIfInMicroProfileProject((projectinfo) -> {
 			String uri = document.getUri();
 			triggerValidationFor(Arrays.asList(uri));
 			return null;

microprofile.ls/com.redhat.microprofile.ls/src/main/java/com/redhat/microprofile/ls/JavaTextDocumentSnippetRegistry.java

@@ -0,0 +1,42 @@
+package com.redhat.microprofile.ls;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import com.redhat.microprofile.ls.commons.snippets.Snippet;
+import com.redhat.microprofile.ls.commons.snippets.TextDocumentSnippetRegistry;
+import com.redhat.microprofile.snippets.LanguageId;
+import com.redhat.microprofile.snippets.SnippetContextForJava;
+
+public class JavaTextDocumentSnippetRegistry extends TextDocumentSnippetRegistry {
+
+	private List<String> types;
+
+	public JavaTextDocumentSnippetRegistry() {
+		super(LanguageId.java.name());
+	}
+
+	public List<String> getTypes() {
+		if (types != null) {
+			return types;
+		}
+		types = collectTypes();
+		return types;
+	}
+
+	private synchronized List<String> collectTypes() {
+		if (types != null) {
+			return types;
+		}
+		List<String> types = new ArrayList<>();
+		for (Snippet snippet : getSnippets()) {
+			if (snippet.getContext() != null && snippet.getContext() instanceof SnippetContextForJava) {
+				List<String> t = ((SnippetContextForJava) snippet.getContext()).getTypes();
+				if (t != null) {
+					types.addAll(t);
+				}
+			}
+		}
+		return types;
+	}
+}

microprofile.ls/com.redhat.microprofile.ls/src/main/java/com/redhat/microprofile/ls/JavaTextDocuments.java

@@ -16,7 +16,7 @@
 import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.function.Supplier;
+import java.util.function.Function;
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
@@ -49,6 +49,8 @@ class JavaTextDocuments extends TextDocuments<JavaTextDocument> {
 
 	private final MicroProfileJavaProjectLabelsProvider provider;
 
+	private JavaTextDocumentSnippetRegistry snippetRegistry;
+
 	/**
 	 * Opened Java file.
 	 *
@@ -80,13 +82,13 @@ public void setProjectURI(String projectURI) {
 		 * @return the given code only if the Java file belongs to a MicroProfile
 		 *         project.
 		 */
-		public <T> CompletableFuture<T> executeIfInMicroProfileProject(Supplier<CompletableFuture<T>> code,
-				T defaultValue) {
+		public <T> CompletableFuture<T> executeIfInMicroProfileProject(
+				Function<ProjectLabelInfoEntry, CompletableFuture<T>> code, T defaultValue) {
 			return getProjectInfo(this).thenComposeAsync(projectInfo -> {
 				if (!isMicroProfileProject(projectInfo)) {
 					return CompletableFuture.completedFuture(defaultValue);
 				}
-				return code.get();
+				return code.apply(projectInfo);
 			});
 		}
 
@@ -150,6 +152,7 @@ CompletableFuture<ProjectLabelInfoEntry> getProjectInfoFromCache(JavaTextDocumen
 			// not found in the cache, load the project info from the JDT LS Extension
 			MicroProfileJavaProjectLabelsParams params = new MicroProfileJavaProjectLabelsParams();
 			params.setUri(documentURI);
+			params.setTypes(getSnippetRegistry().getTypes());
 			final CompletableFuture<ProjectLabelInfoEntry> future = provider.getJavaProjectlabels(params);
 			future.thenApply(entry -> {
 				if (entry != null) {
@@ -200,4 +203,11 @@ private void classpathChanged(Set<String> projectURIs) {
 	private static boolean isMicroProfileProject(ProjectLabelInfoEntry projectInfo) {
 		return projectInfo != null && projectInfo.hasLabel(MICROPROFILE_PROJECT_LABEL);
 	}
+
+	public JavaTextDocumentSnippetRegistry getSnippetRegistry() {
+		if (snippetRegistry == null) {
+			snippetRegistry = new JavaTextDocumentSnippetRegistry();
+		}
+		return snippetRegistry;
+	}
 }