Refactor: Remove all comments from source code

Author: farmer0010

demo-web/src/main/java/com/jpyrust/JPyRustBridge.java

@@ -79,7 +79,6 @@ public synchronized static void initialize(String workDirectory, String sourceSc
                 Files.copy(src, dst, StandardCopyOption.REPLACE_EXISTING);
             }
 
-            // Copy plugins directory
             Path pluginsSrc = Paths.get(sourceScriptDir, "plugins");
             Path pluginsDst = Paths.get(workDir, "plugins");
             if (Files.exists(pluginsSrc)) {
@@ -184,7 +183,6 @@ public String processNlp(String text) {
     }
 
     public String processRegression(String jsonPoints) {
-        // Input: "[[1, 2], [2, 4]]"
         String requestId = UUID.randomUUID().toString();
         byte[] inputBytes = jsonPoints.getBytes(StandardCharsets.UTF_8);
         ByteBuffer buffer = ByteBuffer.allocateDirect(inputBytes.length);
@@ -220,14 +218,8 @@ public byte[] processImage(ByteBuffer data, int length, int width, int height, i
         return processImage(workDir, data, length, width, height, channels);
     }
 
-    /**
-     * Generic method to send any task to the Python daemon.
-     * Useful for Status checks and Plugins.
-     */
     public String sendTask(String taskType, String metadata) {
         String requestId = UUID.randomUUID().toString();
-        // Send dummy input (1 byte) as some tasks might expect it,
-        // though STATUS/PLUGINS might primarily use metadata.
         byte[] dummyInput = "{}".getBytes(StandardCharsets.UTF_8);
 
         ByteBuffer buffer = ByteBuffer.allocateDirect(dummyInput.length);

demo-web/src/main/java/com/jpyrust/demo/AIController.java

@@ -14,7 +14,6 @@ public class AIController {
 
     @GetMapping("/chat")
     public Map<String, Object> chat(@RequestParam String message, @RequestParam int id) {
-        // ... legacy code ...
         return Map.of("status", "success", "python_response", "Text AI is under maintenance.");
     }
 
@@ -26,7 +25,6 @@ public Map<String, Object> analyzeSentiment(@RequestBody String text) {
 
     @PostMapping("/regression")
     public Map<String, Object> performRegression(@RequestBody String jsonPoints) {
-        // jsonPoints example: "[[1, 2], [2, 4], [3, 6]]"
         String result = bridge.processRegression(jsonPoints);
         return Map.of("result", result);
     }

demo-web/src/main/java/com/jpyrust/demo/AIImageController.java

@@ -47,7 +47,6 @@ public void init() {
 
     @PostMapping(value = "/process-image", consumes = MediaType.MULTIPART_FORM_DATA_VALUE, produces = MediaType.IMAGE_JPEG_VALUE)
     public ResponseEntity<byte[]> processImage(@RequestParam("file") MultipartFile file) {
-        // ... (keep existing YOLO logic) ...
         return processImageInternal(file, "YOLO");
     }
 
@@ -60,10 +59,11 @@ private ResponseEntity<byte[]> processImageInternal(MultipartFile file, String m
         String requestId = UUID.randomUUID().toString();
         try {
             BufferedImage inputImage = ImageIO.read(file.getInputStream());
-            if (inputImage == null) return ResponseEntity.badRequest().build();
+            if (inputImage == null)
+                return ResponseEntity.badRequest().build();
 
-            // Convert to BGR (OpenCV standard)
-            BufferedImage bgrImage = new BufferedImage(inputImage.getWidth(), inputImage.getHeight(), BufferedImage.TYPE_3BYTE_BGR);
+            BufferedImage bgrImage = new BufferedImage(inputImage.getWidth(), inputImage.getHeight(),
+                    BufferedImage.TYPE_3BYTE_BGR);
             bgrImage.getGraphics().drawImage(inputImage, 0, 0, null);
 
             byte[] pixelData = ((DataBufferByte) bgrImage.getRaster().getDataBuffer()).getData();
@@ -74,14 +74,16 @@ private ResponseEntity<byte[]> processImageInternal(MultipartFile file, String m
 
             JPyRustBridge bridge = new JPyRustBridge();
             byte[] resultData;
-            
+
             if ("EDGE".equals(mode)) {
                 resultData = bridge.processEdgeDetection(pixelData, bgrImage.getWidth(), bgrImage.getHeight(), 3);
             } else {
-                resultData = bridge.processImage(workDir, directBuffer, pixelData.length, bgrImage.getWidth(), bgrImage.getHeight(), 3, requestId);
+                resultData = bridge.processImage(workDir, directBuffer, pixelData.length, bgrImage.getWidth(),
+                        bgrImage.getHeight(), 3, requestId);
             }
 
-            if (resultData == null) return ResponseEntity.internalServerError().build();
+            if (resultData == null)
+                return ResponseEntity.internalServerError().build();
             return ResponseEntity.ok(resultData);
 
         } catch (Exception e) {

demo-web/src/main/java/com/jpyrust/demo/DemoApplication.java

@@ -16,8 +16,6 @@ public static void main(String[] args) {
     @PostConstruct
     public void init() {
         System.out.println("Scheduling JPyRust initialization (non-blocking)...");
-        // Run Python initialization in background thread so Tomcat can start
-        // immediately
         CompletableFuture.runAsync(() -> {
             try {
                 System.out.println("[Async] Starting JPyRust initialization...");

demo-web/src/main/java/com/jpyrust/demo/PluginController.java

@@ -13,13 +13,8 @@ public class PluginController {
     @PostMapping("/{taskType}")
     public Map<String, Object> executePlugin(@PathVariable String taskType, @RequestBody Map<String, Object> payload) {
         try {
-            // Convert payload values to simple metadata string if possible
-            // For this sample, we assume payload has "args" list or we just send values
-            // Example input: {"args": [1, 2]}
-
             String metadata = "NONE";
             if (payload.containsKey("args")) {
-                // simple space-joined args
                 Object args = payload.get("args");
                 if (args instanceof Iterable) {
                     StringBuilder sb = new StringBuilder();

demo-web/src/main/java/com/jpyrust/demo/StatusController.java

@@ -19,16 +19,13 @@ public Map<String, Object> getStatus() {
         try {
             String jsonResult = bridge.sendTask("STATUS", "NONE");
 
-            // If result starts with "ERROR", return error map
             if (jsonResult.startsWith("ERROR")) {
                 return Map.of("status", "DOWN", "error", jsonResult);
             }
 
-            // Parse JSON string from Python
             try {
                 return mapper.readValue(jsonResult, Map.class);
             } catch (Exception e) {
-                // If not valid JSON (e.g. raw string), return as raw
                 return Map.of("status", "UNKNOWN", "raw", jsonResult);
             }
 

java-api/src/main/java/com/jpyrust/JPyRustBridge.java

@@ -26,7 +26,7 @@ public class JPyRustBridge {
             System.out.println("[JPyRustBridge] Native library loaded successfully via NativeLoader");
         } catch (Throwable e) {
             System.out.println("DEBUG: Failed to load native library!");
-            e.printStackTrace(System.out); // Print to stdout to ensure visibility
+            e.printStackTrace(System.out);
             throw new RuntimeException("Fatal: Native library load failed", e);
         }
     }
@@ -35,14 +35,12 @@ public class JPyRustBridge {
     private static Path pythonExe;
 
     public synchronized static void initialize() {
-        // Default to user home directory
         String userHome = System.getProperty("user.home");
         Path defaultWorkDir = Paths.get(userHome, ".jpyrust");
         initialize(defaultWorkDir.toString(), null);
     }
 
     public synchronized static void initialize(String workDirectory, String ignoredSourceScript) {
-        // Generate a unique session key for Shared Memory
         String memoryKey = "JPyRust_" + java.util.UUID.randomUUID().toString();
         System.out.println("[JPyRust] Generated Session Key (Default): " + memoryKey);
         initialize(workDirectory, ignoredSourceScript, memoryKey);
@@ -53,11 +51,7 @@ private static void setupEmbeddedPython(Path targetDir) throws Exception {
         Path markerFile = pythonDistDir.resolve(".installed");
 
         if (!Files.exists(markerFile)) {
-            // Robustness: Check if it's actually installed but marker is missing (e.g.
-            // crash during last boot)
-            // We check for a key package like 'ultralytics' or 'torch'
-            Path sitePackages = pythonDistDir.resolve("Lib/site-packages"); // Standard for embedded layout usually, or
-                                                                            // just Lib/site-packages
+            Path sitePackages = pythonDistDir.resolve("Lib/site-packages");
             if (Files.exists(pythonDistDir.resolve("python.exe")) &&
                     (Files.exists(sitePackages.resolve("ultralytics"))
                             || Files.exists(sitePackages.resolve("torch")))) {
@@ -69,48 +63,25 @@ private static void setupEmbeddedPython(Path targetDir) throws Exception {
             System.out.println("[Init] Extracting user embedded python to: " + pythonDistDir);
             NativeLoader.extractZip("/python_dist.zip", pythonDistDir);
 
-            // --- [Patched] Auto-fix for pip support ---
             Path pthFile = pythonDistDir.resolve("python311._pth");
             if (Files.exists(pthFile)) {
                 System.out.println("[Init] Patching python311._pth to enable 'import site'...");
                 Files.write(pthFile, "python311.zip\n.\nimport site".getBytes());
             }
 
-            // --- [Patched] Manual Dependency Installation ---
             System.out.println("[Init] Installing dependencies via pip...");
             Path pyExe = pythonDistDir.resolve("python.exe");
             Path requirements = targetDir.resolve("requirements.txt");
 
-            // Generate requirements.txt if missing (fallback)
-            if (!Files.exists(requirements)) {
-                // Copy from classpath or creating a minimal one if needed,
-                // but for now we assume it exists or the user provided it.
-                // Actually, let's copy the one from project root if we can or skip.
-                // Better: JPyRustBridge is usually run where requirements.txt is available or
-                // provided.
-            }
-
-            // We need to assume requirements.txt is in the targetDir or provided.
-            // The previous logic didn't explicitly move it, but our manual steps did.
-            // Let's ensure requirements.txt is present.
-            // Since we can't easily access the project root from here dynamically in all
-            // cases,
-            // we will rely on the fact that if it exists, we install.
-
             if (Files.exists(requirements)) {
-                Path wheelsDir = pythonDistDir.resolve("../wheels"); // Based on our manual copy structure
-                // Adjust logic: The wheels were in build/python_staging/wheels.
-                // We need to be careful. If wheels are not there, try online install or skip?
-                // For now, let's just try to run pip.
+                Path wheelsDir = pythonDistDir.resolve("../wheels");
 
                 ProcessBuilder pipPb = new ProcessBuilder(
                         pyExe.toString(),
                         "-m", "pip", "install",
                         "--no-index",
-                        "--find-links=wheels", // Relative to working dir? No, let's use absolute if possible or
-                                               // relative
+                        "--find-links=wheels",
                         "-r", requirements.toString());
-                // We assume wheels are in targetDir/wheels
                 pipPb.directory(targetDir.toFile());
                 pipPb.redirectErrorStream(true);
                 Process pipProc = pipPb.start();
@@ -124,7 +95,6 @@ private static void setupEmbeddedPython(Path targetDir) throws Exception {
                 pipProc.waitFor();
             }
 
-            // Create marker
             Files.createFile(markerFile);
 
         } else {
@@ -135,40 +105,28 @@ private static void setupEmbeddedPython(Path targetDir) throws Exception {
         pythonExe = pythonDistDir.resolve("python.exe");
     }
 
-    /**
-     * [Native Callback] Rust 프로세스에서 발생하는 로그를 Java로 전달받는 메서드입니다.
-     * 주의: Native 코드(Rust)에서 이 메서드 시그니처를 참조하므로 절대 삭제하거나 변경하면 안 됩니다.
-     *
-     * @param level 로그 레벨 (INFO, ERROR, DEBUG 등)
-     * @param msg   로그 메시지 내용
-     */
     public static void log(String level, String msg) {
-        // 기본적으로 System.out으로 출력 (추후 SLF4J 등으로 확장 가능)
         System.out.println("[JPyRust-Native] [" + level + "] " + msg);
     }
 
     private static native void initNative(String workDir, String sourceScriptDir, String modelPath, float confidence,
             String memoryKey);
 
-    // 4-param initialize overload for AIImageController compatibility
     public static void initialize(String workDirectory, String sourceScript, String modelPath, float confidence) {
         System.out.println("[JPyRust] Init with model: " + modelPath + ", confidence: " + confidence);
 
-        // Generate a unique session key for Shared Memory
         String memoryKey = "JPyRust_" + java.util.UUID.randomUUID().toString();
         System.out.println("[JPyRust] Generated Session Key: " + memoryKey);
 
         initialize(workDirectory, sourceScript, memoryKey);
     }
 
-    // Internal initialize with key
     private synchronized static void initialize(String workDirectory, String ignoredSourceScript, String memoryKey) {
         if (initialized) {
             return;
         }
 
         workDir = workDirectory;
-        // sourceScriptDir is no longer needed as we use the embedded one
 
         System.out.println("=== JPyRust IPC Initialization ===");
         System.out.println("[Init] Work Directory: " + workDir);
@@ -179,10 +137,8 @@ private synchronized static void initialize(String workDirectory, String ignored
                 Files.createDirectories(workPath);
             }
 
-            // 1. Setup Embedded Python
             setupEmbeddedPython(workPath);
 
-            // 2. Initialize Native
             initNative(workDir, workDir, "yolov8n.pt", 0.5f, memoryKey);
 
             System.out.println("=== Initialization Complete ===");
@@ -194,18 +150,15 @@ private synchronized static void initialize(String workDirectory, String ignored
         }
     }
 
-    // Native executeTask declaration matching Rust signature
     private native byte[] executeTask(String workDir, String taskType, String requestId, String metadata,
             ByteBuffer data, int length);
 
     public byte[] processImage(String workDirectory, ByteBuffer data, int length, int width, int height, int channels) {
         String requestId = java.util.UUID.randomUUID().toString();
-        // Construct metadata directly as simple string "width height channels"
         String metadata = width + " " + height + " " + channels;
         return executeTask(workDirectory, "YOLO", requestId, metadata, data, length);
     }
 
-    // 7-param processImage overload for AIImageController compatibility
     public byte[] processImage(String workDirectory, ByteBuffer data, int length, int width, int height, int channels,
             String requestId) {
         System.out.println("[JPyRust] Processing request: " + requestId);
@@ -217,7 +170,6 @@ public byte[] processImage(ByteBuffer data, int length, int width, int height, i
         return processImage(workDir, data, length, width, height, channels);
     }
 
-    // Edge detection implementation
     public byte[] processEdgeDetection(byte[] imageData, int width, int height, int channels) {
         System.out.println("[JPyRust] Edge detection called (Native)");
         try {
@@ -236,7 +188,6 @@ public byte[] processEdgeDetection(byte[] imageData, int width, int height, int
         }
     }
 
-    // NLP processing implementation
     public String processNlp(String text) {
         System.out.println("[JPyRust] NLP processing: " + text);
         try {
@@ -246,7 +197,6 @@ public String processNlp(String text) {
             directBuffer.flip();
 
             String requestId = java.util.UUID.randomUUID().toString();
-            // Metadata empty for NLP, or specialized if needed
             String metadata = "TEXT";
 
             byte[] resultBytes = executeTask(workDir, "NLP_TEXTBLOB", requestId, metadata, directBuffer,
@@ -262,7 +212,6 @@ public String processNlp(String text) {
         }
     }
 
-    // Regression processing implementation
     public String processRegression(String jsonPoints) {
         System.out.println("[JPyRust] Regression processing: " + jsonPoints);
         try {
@@ -287,13 +236,6 @@ public String processRegression(String jsonPoints) {
         }
     }
 
-    // Legacy runPythonProcess declaration removed/replaced by executeTask usage
-    /*
-     * private native byte[] runPythonProcess(String workDir, ByteBuffer data, int
-     * length, int width, int height,
-     * int channels, String requestId);
-     */
-
     public String runPythonRaw(ByteBuffer data, int length, int width, int height, int channels) {
         String inputFilePath = workDir + "/input_image.dat";
         String outputFilePath = workDir + "/output_image.dat";

java-api/src/main/java/com/jpyrust/NativeLoader.java

@@ -25,7 +25,6 @@ public static void load(String libName) {
 
             String filename = libName + extension;
 
-            // Try multiple resource paths for Spring Boot compatibility
             String[] resourcePaths = {
                     "/natives/" + filename,
                     "natives/" + filename,
@@ -36,7 +35,6 @@ public static void load(String libName) {
             InputStream is = null;
             String foundPath = null;
 
-            // Try class classloader first
             for (String path : resourcePaths) {
                 System.err.println("[NativeLoader] Checking path: " + path);
                 is = NativeLoader.class.getResourceAsStream(path);
@@ -47,7 +45,6 @@ public static void load(String libName) {
                 }
             }
 
-            // Fallback to thread context classloader
             if (is == null) {
                 ClassLoader contextCL = Thread.currentThread().getContextClassLoader();
                 if (contextCL != null) {

java-api/src/test/java/com/jpyrust/TestLoader.java

@@ -7,7 +7,6 @@ public class TestLoader {
     public void testLoading() {
         System.out.println("Testing JPyRustBridge Loading...");
         try {
-            // Force class loading to trigger static block
             Class.forName("com.jpyrust.JPyRustBridge");
             System.out.println("JPyRustBridge loaded successfully!");