refactor: serve is read-only — remove auto-enrich, use CLI pipeline

Pipeline is now strictly:
  code-iq index  → H2 (memory-efficient batched scanning)
  code-iq enrich → Neo4j (linkers, layers, services, bulk-load)
  code-iq serve  → read-only server (API, MCP, UI)

serve no longer runs analysis or loads data into Neo4j. It expects
enrich to have already populated Neo4j. Reports graph status on startup
and warns if Neo4j is empty.

This matches the deployment model: index+enrich on developer machine,
serve on remote server from pre-built graph.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: aksOps

src/main/java/io/github/randomcodespace/iq/cli/ServeCommand.java

@@ -1,14 +1,10 @@
 package io.github.randomcodespace.iq.cli;
 
-import io.github.randomcodespace.iq.analyzer.AnalysisResult;
-import io.github.randomcodespace.iq.analyzer.Analyzer;
-import io.github.randomcodespace.iq.cache.AnalysisCache;
 import io.github.randomcodespace.iq.config.CodeIqConfig;
 import io.github.randomcodespace.iq.graph.GraphStore;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.cache.CacheManager;
 import org.springframework.stereotype.Component;
 import picocli.CommandLine.Command;
 import picocli.CommandLine.Option;
@@ -17,19 +13,15 @@
 import java.nio.file.Path;
 import java.text.NumberFormat;
 import java.util.Locale;
-import java.util.Optional;
 import java.util.concurrent.Callable;
 
 /**
  * Start the web UI + REST API + MCP server.
- *
- * This command signals to the application that it should keep running
- * (the web server is already started by Spring Boot when the "serving" profile
- * is active). The serve command simply prints server info and blocks until
- * the Spring context shuts down.
- *
- * On startup, if the Neo4j graph is empty but an H2 analysis cache exists,
- * the command auto-loads data from H2 into Neo4j (like a mini enrich).
+ * <p>
+ * This command expects Neo4j to already be populated by {@code enrich}.
+ * It simply starts the Spring Boot web server and blocks until shutdown.
+ * <p>
+ * Pipeline: {@code index} → {@code enrich} → {@code serve}
  */
 @Component
 @Command(name = "serve", mixinStandardHelpOptions = true,
@@ -38,7 +30,6 @@ public class ServeCommand implements Callable<Integer> {
 
     private static final Logger log = LoggerFactory.getLogger(ServeCommand.class);
 
-    /** Marker flag — checked by CodeIqApplication to activate serving profile. */
     public static final String COMMAND_NAME = "serve";
 
     @Parameters(index = "0", defaultValue = ".", description = "Path to analyzed codebase")
@@ -56,40 +47,42 @@ public class ServeCommand implements Callable<Integer> {
     @Autowired
     private CodeIqConfig config;
 
-    @Autowired
-    private Analyzer analyzer;
-
     @Autowired(required = false)
     private GraphStore graphStore;
 
-    @Autowired(required = false)
-    private CacheManager cacheManager;
-
     @Override
     public Integer call() {
         Path root = path.toAbsolutePath().normalize();
-
-        // Update config with the root path so controllers can find it
         config.setRootPath(root.toString());
+        NumberFormat nf = NumberFormat.getIntegerInstance(Locale.US);
 
-        // Auto-enrich: if Neo4j is empty, run full analysis to populate it
-        autoEnrich(root);
+        // Report Neo4j graph status
+        if (graphStore != null) {
+            try {
+                long nodeCount = graphStore.count();
+                long edgeCount = graphStore.countEdges();
+                if (nodeCount > 0) {
+                    CliOutput.success("Neo4j graph: " + nf.format(nodeCount) + " nodes, "
+                            + nf.format(edgeCount) + " edges");
+                } else {
+                    CliOutput.warn("Neo4j graph is empty. Run 'code-iq enrich " + root + "' to populate.");
+                }
+            } catch (Exception e) {
+                log.debug("Could not check Neo4j state", e);
+                CliOutput.warn("Could not read Neo4j graph. Run 'code-iq enrich' first.");
+            }
+        }
 
         CliOutput.step("\uD83D\uDE80", "@|bold,green Server started|@");
         System.out.println();
         CliOutput.info("  URL:       http://" + host + ":" + port);
         CliOutput.info("  REST API:  http://" + host + ":" + port + "/api");
         CliOutput.info("  MCP:       http://" + host + ":" + port + "/mcp");
         CliOutput.info("  Health:    http://" + host + ":" + port + "/actuator/health");
-        CliOutput.info("  API Docs:  http://" + host + ":" + port + "/docs");
         CliOutput.info("  Codebase:  " + root);
         System.out.println();
         CliOutput.info("Press Ctrl+C to stop.");
 
-        // The Spring Boot web server is already running. We block here
-        // to prevent the CommandLineRunner from returning (which would
-        // trigger application shutdown). The JVM shutdown hook will
-        // handle cleanup.
         try {
             Thread.currentThread().join();
         } catch (InterruptedException e) {
@@ -99,79 +92,8 @@ public Integer call() {
         return 0;
     }
 
-    /**
-     * Auto-enrich Neo4j on startup: if the graph is empty, run the full analysis
-     * pipeline to populate Neo4j from the codebase. This ensures the API/MCP
-     * endpoints return data immediately without requiring a manual POST /api/analyze.
-     */
-    private void autoEnrich(Path root) {
-        // Check if Neo4j already has data
-        boolean neo4jEmpty = true;
-        if (graphStore != null) {
-            try {
-                neo4jEmpty = graphStore.count() == 0;
-            } catch (Exception e) {
-                log.debug("Could not check Neo4j state", e);
-            }
-        }
-
-        if (!neo4jEmpty) {
-            long nodeCount = graphStore.count();
-            long edgeCount = graphStore.countEdges();
-            CliOutput.success("Neo4j graph loaded: " + nodeCount + " nodes, " + edgeCount + " edges");
-            return;
-        }
-
-        // Neo4j is empty — run analysis to populate it
-        NumberFormat nf = NumberFormat.getIntegerInstance(Locale.US);
-        CliOutput.step("\u26A1", "Auto-enriching Neo4j graph...");
-
-        try {
-            AnalysisResult result = analyzer.run(root, null, true, msg -> {
-                if (msg.startsWith("Discovering") || msg.startsWith("Found") || msg.startsWith("Analyzing")
-                        || msg.startsWith("Building") || msg.startsWith("Linking") || msg.startsWith("Classifying")
-                        || msg.startsWith("Analysis complete")) {
-                    CliOutput.info("  " + msg);
-                }
-            });
-
-            // Persist to Neo4j
-            if (graphStore != null && result.nodes() != null && !result.nodes().isEmpty()) {
-                CliOutput.step("\uD83D\uDCBE", "Persisting " + nf.format(result.nodes().size()) + " nodes to Neo4j...");
-                graphStore.bulkSave(result.nodes());
-            }
-
-            // Evict Spring caches
-            if (cacheManager != null) {
-                cacheManager.getCacheNames().forEach(name -> {
-                    var cache = cacheManager.getCache(name);
-                    if (cache != null) cache.clear();
-                });
-            }
-
-            CliOutput.success("Graph ready: " + nf.format(result.nodeCount()) + " nodes, "
-                    + nf.format(result.edgeCount()) + " edges from "
-                    + nf.format(result.filesAnalyzed()) + " files");
-        } catch (Exception e) {
-            log.error("Auto-enrich failed", e);
-            CliOutput.error("Auto-enrich failed: " + e.getMessage());
-            CliOutput.info("  You can manually trigger analysis via POST /api/analyze");
-        }
-    }
-
-    public Path getPath() {
-        return path;
-    }
-
-    public int getPort() {
-        return port;
-    }
-
-    public String getHost() {
-        return host;
-    }
-
-    public Path getGraphPath() {
-        return graphPath;
-    }
+    public Path getPath() { return path; }
+    public int getPort() { return port; }
+    public String getHost() { return host; }
+    public Path getGraphPath() { return graphPath; }
 }