fix: call the correct command identifier

Author: acke

plugin/src/main/java/io/snyk/eclipse/plugin/html/CodeHtmlProvider.java

@@ -70,7 +70,7 @@ public String replaceCssVariables(String html) {
 		htmlStyled = htmlStyled.replace("var(--generated-ai-fix-button-background-color)",
 				super.getColorAsHex("BUTTON_COLOR", "#375578"));
 		htmlStyled = htmlStyled.replace("var(--disabled-background-color)",
-				super.getColorAsHex("ADDITION_COLOR", "#6e6e6e"));
+				super.getColorAsHex("org.eclipse.ui.workbench.ACTIVE_TAB_OUTER_KEYLINE_COLOR", "#CCCCCC"));
 
 		String htmlWithScripts = replaceAIFixScripts(htmlStyled);
 

plugin/src/main/java/io/snyk/eclipse/plugin/views/snyktoolview/BrowserHandler.java

@@ -3,7 +3,7 @@
 import static org.apache.commons.lang3.StringUtils.isEmpty;
 
 import java.nio.file.Paths;
-import java.util.List;
+import java.util.Collections;
 import java.util.concurrent.CompletableFuture;
 
 import org.eclipse.core.commands.common.CommandException;
@@ -33,7 +33,6 @@
 import io.snyk.eclipse.plugin.views.snyktoolview.handlers.IHandlerCommands;
 import io.snyk.eclipse.plugin.wizards.SnykWizard;
 import io.snyk.languageserver.protocolextension.SnykExtendedLanguageClient;
-import io.snyk.languageserver.protocolextension.messageObjects.Fix;
 
 @SuppressWarnings("restriction")
 public class BrowserHandler {
@@ -102,33 +101,26 @@ public Object function(Object[] arguments) {
 		new BrowserFunction(browser, "ideGenAIFix") {
 			@Override
 			public Object function(Object[] arguments) {
-				// browser.execute("debugger;"); // Triggers a breakpoint for debugging
-				System.out.println("generateAIFix");
-				SnykLogger.logInfo("generateAIFix");
-
 				String params = (String) arguments[0];
 				String[] parts = params.split("@|@");
-
 				String folderURI = (String) parts[0];
 				String fileURI = (String) parts[2];
 				String issueID = (String) parts[4];
 
-				SnykExtendedLanguageClient.getInstance().sendCodeFixDiffsCommand(folderURI,
-						fileURI, issueID);
+				SnykExtendedLanguageClient.getInstance().sendCodeFixDiffsCommand(folderURI, fileURI, issueID);
 
-				return null;
+				return Collections.emptyList();
 			}
 		};
 
 		new BrowserFunction(browser, "ideApplyFix") {
 			@Override
 			public Object function(Object[] arguments) {
-				System.out.println("applyAIFix");
-				SnykLogger.logInfo("applyAIFix");
-
 				String fixId = (String) arguments[0];
+
 				SnykExtendedLanguageClient.getInstance().sendCodeApplyAiFixEditCommand(fixId);
-				return null;
+
+				return Collections.emptyList();
 			}
 		};
 

plugin/src/main/java/io/snyk/languageserver/LsConstants.java

@@ -17,7 +17,7 @@ private LsConstants() {
 	public static final String COMMAND_REPORT_ANALYTICS = "snyk.reportAnalytics";
 	public static final String COMMAND_GET_FEATURE_FLAG_STATUS = "snyk.getFeatureFlagStatus";
 	public static final String COMMAND_CODE_FIX_DIFFS = "snyk.code.fixDiffs";
-	public static final String COMMAND_CODE_FIX_APPLY_AI_EDIT = "$/snyk.code.fixApplyEdit";
+	public static final String COMMAND_CODE_FIX_APPLY_AI_EDIT = "snyk.code.fixApplyEdit";
 	public static final String COMMAND_CODE_SUBMIT_FIX_FEEDBACK = "snyk.code.submitFixFeedback";
 	public static final String COMMAND_SNYK_CLI = "snyk.executeCLI";
 	public static final String SNYK_HAS_AUTHENTICATED = "$/snyk.hasAuthenticated";

plugin/src/main/java/io/snyk/languageserver/protocolextension/SnykExtendedLanguageClient.java

@@ -311,10 +311,8 @@ public String getIssueDescription(String issueId) {
 		return String.valueOf(result);
 	}
 
-	public List<Fix> sendCodeFixDiffsCommand(String folderURI, String fileURI, String issueID) {
-		// TODO: capture and return results
+	public void sendCodeFixDiffsCommand(String folderURI, String fileURI, String issueID) {
 		executeCommand(LsConstants.COMMAND_CODE_FIX_DIFFS, List.of(folderURI, fileURI, issueID));
-		return null;
 	}
 
 	public void sendCodeApplyAiFixEditCommand(String fixId) {