Merge pull request #51308 from phillip-kruger/dev-mcp-installable-ext

Dev MCP: Reduce the payload for the extension list call

Author: phillip-kruger

extensions/devui/deployment/src/main/java/io/quarkus/devui/deployment/menu/ExtensionsProcessor.java

@@ -3,6 +3,7 @@
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Set;
@@ -71,7 +72,8 @@ void createBuildTimeActions(BuildProducer<BuildTimeActionBuildItem> buildTimeAct
 
             BuildTimeActionBuildItem buildTimeActions = new BuildTimeActionBuildItem(NAMESPACE);
             getCategories(buildTimeActions);
-            getInstallableExtensions(buildTimeActions);
+            getInstallableExtensions(buildTimeActions); // For Dev UI
+            listInstallableExtensions(buildTimeActions); // For Dev MCP
             getInstalledNamespaces(buildTimeActions);
             removeExtension(buildTimeActions);
             addExtension(buildTimeActions);
@@ -104,34 +106,59 @@ private void getCategories(BuildTimeActionBuildItem buildTimeActions) {
     }
 
     private void getInstallableExtensions(BuildTimeActionBuildItem buildTimeActions) {
+        buildTimeActions
+                .actionBuilder().methodName(new Object() {
+                }.getClass().getEnclosingMethod().getName())
+                .function(ignored -> {
+                    return CompletableFuture.supplyAsync(() -> {
+                        return listExtensionInQuarkusProject();
+                    });
+                })
+                .build();
+    }
+
+    private void listInstallableExtensions(BuildTimeActionBuildItem buildTimeActions) {
         buildTimeActions
                 .actionBuilder().methodName(new Object() {
                 }.getClass().getEnclosingMethod().getName())
                 .description(
                         "Get all extensions that can be added to the current project (i.e it's not currently added to the pom)")
                 .function(ignored -> {
                     return CompletableFuture.supplyAsync(() -> {
-                        try {
-                            QuarkusCommandOutcome outcome = new ListExtensions(getQuarkusProject())
-                                    .installed(false)
-                                    .all(false)
-                                    .format("object")
-                                    .execute();
 
-                            if (outcome.isSuccess()) {
-                                return outcome.getResult();
-                            }
-
-                            return null;
-                        } catch (QuarkusCommandException e) {
-                            throw new RuntimeException(e);
+                        List<Map<String, String>> filtered = new ArrayList<>();
+                        for (io.quarkus.registry.catalog.Extension e : listExtensionInQuarkusProject()) {
+                            Map<String, String> entry = new HashMap<>();
+                            entry.put("name", e.getName());
+                            entry.put("description", e.getDescription());
+                            entry.put("extensionArtifactId", e.getArtifact().toCompactCoords());
+                            filtered.add(entry);
                         }
+                        return filtered;
                     });
                 })
                 .enableMcpFuctionByDefault()
                 .build();
     }
 
+    private List<io.quarkus.registry.catalog.Extension> listExtensionInQuarkusProject() throws RuntimeException {
+        try {
+            QuarkusCommandOutcome outcome = new ListExtensions(getQuarkusProject())
+                    .installed(false)
+                    .all(false)
+                    .format("object")
+                    .execute();
+
+            if (outcome.isSuccess()) {
+                return (List) outcome.getResult();
+            }
+
+            return null;
+        } catch (QuarkusCommandException e) {
+            throw new RuntimeException(e);
+        }
+    }
+
     private void getInstalledNamespaces(BuildTimeActionBuildItem buildTimeActions) {
         buildTimeActions
                 .actionBuilder().methodName(new Object() {
@@ -188,12 +215,7 @@ private void removeExtension(BuildTimeActionBuildItem buildTimeActions) {
                             QuarkusCommandOutcome outcome = new RemoveExtensions(getQuarkusProject())
                                     .extensions(Set.of(extensionArtifactId))
                                     .execute();
-
-                            if (outcome.isSuccess()) {
-                                return true;
-                            } else {
-                                return false;
-                            }
+                            return outcome.isSuccess();
                         } catch (QuarkusCommandException e) {
                             throw new RuntimeException(e);
                         }