fix(grpc): 🐛 Fix gRPC doc generation for proto files organized in packages

- Closes #1116

- Fixes the core issue: Resolves the problem where grpc-xxx goals fail when .proto files are organized using packages (e.g., base/enum.proto, file/fileops.proto). The root cause was an incorrect --proto_path calculation.
- Enhances path handling: Refactored getUniqueParentDirectories to correctly find the deepest common ancestor directory, ensuring --proto_path points to the actual root of the .proto files.
- Improves command robustness: Uses absolute paths for protoc, protoc-gen-doc, and all input .proto files to guarantee reliable execution across different environments.
- Adds comprehensive logging: Logs the final protoc command for debugging and adds [protoc] prefix to plugin output, making it easier to distinguish between smart-doc logs and protoc errors.

Author: linwumingshi

src/main/java/com/ly/doc/model/grpc/ProtoInfo.java

@@ -87,7 +87,7 @@ private ProtoInfo() {
 		String os = System.getProperty("os.name").toLowerCase();
 		String arch = System.getProperty("os.arch").toLowerCase();
 
-		log.info("The [os.name] is:" + os + ";[os.arch] is: " + arch);
+		log.info("The [os.name] is: " + os + ";[os.arch] is: " + arch);
 
 		this.setTargetJsonDirectoryPath(targetJsonPath);
 		this.setJsonName("combined.json");

src/main/java/com/ly/doc/template/GRpcDocBuildTemplate.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2018-2024 smart-doc
+ * Copyright (C) 2018-2025 smart-doc
  *
  * Licensed to the Apache Software Foundation (ASF) under one
  * or more contributor license agreements.  See the NOTICE file
@@ -34,18 +34,35 @@
 import com.ly.doc.model.grpc.GrpcApiDoc;
 import com.ly.doc.model.grpc.GrpcJavaMethod;
 import com.ly.doc.model.grpc.ProtoInfo;
-import com.ly.doc.model.grpc.proto.*;
+import com.ly.doc.model.grpc.proto.EnumDefinition;
+import com.ly.doc.model.grpc.proto.EnumValue;
+import com.ly.doc.model.grpc.proto.Message;
+import com.ly.doc.model.grpc.proto.MessageField;
+import com.ly.doc.model.grpc.proto.ProtoJson;
+import com.ly.doc.model.grpc.proto.Service;
+import com.ly.doc.model.grpc.proto.ServiceMethod;
 import com.ly.doc.utils.DocUtil;
 import com.power.common.util.FileUtil;
 import com.thoughtworks.qdox.model.JavaClass;
 
-import java.io.*;
+import java.io.BufferedReader;
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.io.InputStreamReader;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
 import java.nio.file.StandardCopyOption;
 import java.nio.file.attribute.PosixFilePermissions;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Map;
+import java.util.Objects;
+import java.util.Set;
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.function.Consumer;
 import java.util.function.Function;
@@ -344,7 +361,8 @@ private void executeCommand(List<String> command, ProtoInfo protoInfo) {
 		try {
 			Process process = processBuilder.start();
 
-			StreamGobbler outputGobbler = new StreamGobbler(process.getInputStream(), log::warning);
+			StreamGobbler outputGobbler = new StreamGobbler(process.getInputStream(),
+					line -> log.warning("[protoc] " + line));
 			Thread outputThread = new Thread(outputGobbler);
 			outputThread.start();
 
@@ -354,7 +372,7 @@ private void executeCommand(List<String> command, ProtoInfo protoInfo) {
 			outputThread.join();
 
 			if (exitCode != 0) {
-				log.warning("Error executing command for files");
+				log.warning("protoc command failed with exit code: " + exitCode);
 			}
 		}
 		catch (IOException | InterruptedException e) {
@@ -370,23 +388,75 @@ private void executeCommand(List<String> command, ProtoInfo protoInfo) {
 	 */
 	private List<String> buildProtocCommand(Set<String> protoFiles, ProtoInfo protoInfo) {
 		List<String> command = new ArrayList<>();
-		command.add(protoInfo.getProtocPath());
-		command.add("--proto_path=" + String.join(";", this.getUniqueParentDirectories(protoFiles)));
-		command.add("--doc_out=" + protoInfo.getTargetJsonDirectoryPath());
+
+		// Add the protoc executable (using absolute path)
+		command.add(Paths.get(protoInfo.getProtocPath()).toAbsolutePath().toString());
+
+		// Calculate the common root directory for all .proto files
+		Set<String> commonRoots = this.getUniqueParentDirectories(protoFiles);
+		if (commonRoots.isEmpty()) {
+			log.warning("No common root directory found for proto files.");
+			return Collections.emptyList();
+		}
+		String commonRootPath = commonRoots.iterator().next();
+
+		// Set --proto_path to the common root directory
+		command.add("--proto_path=" + commonRootPath);
+
+		// Set output options
+		command.add("--doc_out=" + Paths.get(protoInfo.getTargetJsonDirectoryPath()).toAbsolutePath());
 		command.add("--doc_opt=json," + protoInfo.getJsonName());
-		command.addAll(protoFiles);
-		command.add("--plugin=protoc-gen-doc=" + protoInfo.getProtocGenDocPath());
+
+		// Set the plugin path (using absolute path)
+		command.add("--plugin=protoc-gen-doc=" + Paths.get(protoInfo.getProtocGenDocPath()).toAbsolutePath());
+
+		// Add the absolute paths of the .proto files
+		for (String filePath : protoFiles) {
+			Path file = Paths.get(filePath).toAbsolutePath().normalize();
+			command.add(file.toString());
+		}
+
 		return command;
 	}
 
 	/**
-	 * Get the unique parent directories of the given set of files.
-	 * @param files The set of files.
-	 * @return The set of unique parent directories.
+	 * Get the unique parent directories of the given set of files. This method finds the
+	 * deepest common ancestor directory among all proto files.
+	 * @param files The set of .proto file paths.
+	 * @return The set containing the common root directory path.
 	 */
 	private Set<String> getUniqueParentDirectories(Set<String> files) {
+		if (files.isEmpty()) {
+			return Collections.emptySet();
+		}
+
+		// Convert all paths to Path objects
+		List<Path> filePaths = files.stream()
+			.map(Paths::get)
+			.map(Path::toAbsolutePath)
+			.map(Path::normalize)
+			.collect(Collectors.toList());
+
+		// Use the first path as the initial common root
+		Path commonRoot = filePaths.get(0);
+
+		// Compare with all other paths, gradually shorten commonRoot until it is a prefix
+		// of all paths
+		for (Path path : filePaths) {
+			while (!path.startsWith(commonRoot)) {
+				commonRoot = commonRoot.getParent();
+				if (commonRoot == null) {
+					// If we've gone up to null, there's no common parent, return the root
+					// of the path
+					commonRoot = path.getRoot();
+					break;
+				}
+			}
+		}
+
+		// Return the found common root directory
 		Set<String> directories = new HashSet<>();
-		files.forEach(file -> directories.add(new File(file).getParent()));
+		directories.add(commonRoot.toString());
 		return directories;
 	}