Add TODO comments.

Co-authored-by: Copilot <copilot@github.com>

Author: ethanyhou

com.microsoft.copilot.eclipse.core/src/com/microsoft/copilot/eclipse/core/lsp/CopilotLanguageClient.java

@@ -136,11 +136,11 @@ public CompletableFuture<Object[]> getConversationContext(ConversationContextPar
   public CompletableFuture<Object> invokeClientTool(InvokeClientToolParams params) {
     return CompletableFuture.supplyAsync(() -> {
       try {
-        CompletableFuture<LanguageModelToolResult[]> toolFuture =
-            CopilotCore.getPlugin().getChatEventsManager().invokeAgentTool(params);
+        CompletableFuture<LanguageModelToolResult[]> toolFuture = CopilotCore.getPlugin().getChatEventsManager()
+            .invokeAgentTool(params);
         if (toolFuture == null) {
-          CopilotCore.LOGGER.error(
-              new IllegalStateException("invokeAgentTool returned null for tool: " + params.getName()));
+          CopilotCore.LOGGER
+              .error(new IllegalStateException("invokeAgentTool returned null for tool: " + params.getName()));
           LanguageModelToolResult errorResult = new LanguageModelToolResult();
           errorResult.addContent("Failed to invoke the tool: tool invocation returned null");
           errorResult.setStatus(ToolInvocationStatus.error);
@@ -178,6 +178,8 @@ public CompletableFuture<Object[]> confirmClientTool(InvokeClientToolConfirmatio
     });
   }
 
+  // TODO: Should remove workspace-root folder as the projects are not directly under it in Eclipse, and can cause
+  // confusion in CLS.
   @Override
   public CompletableFuture<List<WorkspaceFolder>> workspaceFolders() {
     // Ideally, we should return each IProject as a workspace folder, but given that when
@@ -277,9 +279,8 @@ private void notifyCustomizationFilesChanged() {
   }
 
   /**
-   * Handles the Dynamic OAuth request for MCP.
-   * Shows a dialog with multiple input fields and returns the user's input values.
-   * Returns null if the user cancels the request.
+   * Handles the Dynamic OAuth request for MCP. Shows a dialog with multiple input fields and returns the user's input
+   * values. Returns null if the user cancels the request.
    */
   @JsonRequest("copilot/dynamicOAuth")
   public CompletableFuture<Map<String, String>> mcpOauth(McpOauthRequest request) {
@@ -325,13 +326,11 @@ public void onDidChangePolicy(DidChangePolicyParams params) {
       }
       if (flags.isSubAgentPolicyEnabled() != params.isSubAgentEnabled()) {
         flags.setSubAgentPolicyEnabled(params.isSubAgentEnabled());
-        eventBroker.post(CopilotEventConstants.TOPIC_DID_CHANGE_SUB_AGENT_POLICY,
-            params.isSubAgentEnabled());
+        eventBroker.post(CopilotEventConstants.TOPIC_DID_CHANGE_SUB_AGENT_POLICY, params.isSubAgentEnabled());
       }
       if (flags.isCustomAgentPolicyEnabled() != params.isCustomAgentEnabled()) {
         flags.setCustomAgentPolicyEnabled(params.isCustomAgentEnabled());
-        eventBroker.post(CopilotEventConstants.TOPIC_DID_CHANGE_CUSTOM_AGENT_POLICY,
-            params.isCustomAgentEnabled());
+        eventBroker.post(CopilotEventConstants.TOPIC_DID_CHANGE_CUSTOM_AGENT_POLICY, params.isCustomAgentEnabled());
       }
     }
   }

com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/services/ChatCompletionService.java

@@ -52,8 +52,7 @@ public class ChatCompletionService implements CopilotAuthStatusListener {
   private volatile Set<String> allCommands = Set.of();
   // Exclude intelliJ sepcific slash commands
   private static final Set<String> EXCLUDED_COMMANDS = Set.of("help", "feedback");
-  public static final String REFRESH_JOB_FAMILY =
-      "com.microsoft.copilot.eclipse.chat.services.SlashCommandService.refreshJob";
+  public static final String REFRESH_JOB_FAMILY = "com.microsoft.copilot.eclipse.chat.services.SlashCommandService.refreshJob";
   private CopilotLanguageServerConnection lsConnection;
   private AuthStatusManager authStatusManager;
   private IResourceChangeListener skillFileListener;
@@ -263,6 +262,9 @@ public void dispose() {
   /**
    * Listens for workspace resource changes involving SKILL.md or .prompt.md files and triggers a template refresh when
    * such files are added, removed, or changed.
+   * 
+   * TODO: Remove this listener once workspace-root is removed from workspaceFolders in CopilotLanguageClient as CLS can
+   * watch the project prompt file change directly.
    */
   private class SkillFileChangeListener implements IResourceChangeListener {
     @Override