node-plantuml-2 1.1.2 → 1.1.3

package/lib/node-plantuml.js

@@ -224,8 +224,16 @@ function joinOptions (argv, options) {
   }
 
   if (options.dot) {
+    // Ensure dot path is absolute - PlantUML Java process needs absolute path
+    var absoluteDotPath = path.resolve(options.dot)
+    // Normalize path separators
+    if (process.platform === 'win32') {
+      absoluteDotPath = absoluteDotPath.replace(/\//g, '\\')
+    } else {
+      absoluteDotPath = absoluteDotPath.replace(/\\/g, '/')
+    }
     argv.push(DOT)
-    argv.push(options.dot)
+    argv.push(absoluteDotPath)
   }
 
   if (options.charset) {

package/lib/plantuml-executor.js

@@ -92,10 +92,12 @@ function execWithSpawn (argv, cwd, options, cb) {
 
   // Check if we're using bundled Graphviz and need to set library path
   var dotPath = null
+  var dotPathIndex = -1
   // Extract dot path from argv if present
   for (var i = 0; i < argv.length; i++) {
     if (argv[i] === '-graphvizdot' && i + 1 < argv.length) {
       dotPath = argv[i + 1]
+      dotPathIndex = i + 1
       break
     }
   }
@@ -105,17 +107,55 @@ function execWithSpawn (argv, cwd, options, cb) {
     dotPath = dotResolver.resolveDotExecutable({ dotPath: null })
   }
 
+  // CRITICAL FIX: Ensure dot path is absolute and properly formatted
+  // PlantUML Java process needs absolute path to launch Graphviz subprocess
+  if (dotPath) {
+    // Convert to absolute path
+    var absoluteDotPath = path.resolve(dotPath)
+    
+    // Normalize path separators for the platform
+    if (process.platform === 'win32') {
+      // Windows: Use backslashes and ensure .exe extension
+      absoluteDotPath = absoluteDotPath.replace(/\//g, '\\')
+      if (!absoluteDotPath.toLowerCase().endsWith('.exe')) {
+        // If path doesn't end with .exe, check if it's a directory and add dot.exe
+        var fs = require('fs')
+        try {
+          var stat = fs.statSync(absoluteDotPath)
+          if (stat.isDirectory()) {
+            absoluteDotPath = path.join(absoluteDotPath, 'dot.exe')
+          }
+        } catch (e) {
+          // Ignore errors
+        }
+      }
+    } else {
+      // Unix: Use forward slashes
+      absoluteDotPath = absoluteDotPath.replace(/\\/g, '/')
+    }
+    
+    // Update argv with absolute path if it was found in argv
+    if (dotPathIndex >= 0) {
+      argv[dotPathIndex] = absoluteDotPath
+    }
+    
+    // Use absolute path for environment variable setup
+    dotPath = absoluteDotPath
+  }
+
   // Get library path for bundled Graphviz
   var libPath = dotResolver.getBundledGraphvizLibPath(dotPath)
   var platform = require('os').platform()
   
-  // Only create new env object if we need to modify it
-  var envModified = false
-  var env = null
+  // Always create env object to ensure environment variables are properly set
+  // This is critical for Java subprocess to inherit correct environment
+  var env = Object.assign({}, process.env)
+  var envModified = true
 
   if (libPath) {
-    env = Object.assign({}, process.env)
-    envModified = true
+    // Ensure libPath is absolute
+    libPath = path.resolve(libPath)
+    
     if (platform === 'linux') {
       // Set LD_LIBRARY_PATH for Linux
       var existingLibPath = env.LD_LIBRARY_PATH || ''
@@ -130,12 +170,10 @@ function execWithSpawn (argv, cwd, options, cb) {
   // For Windows, add Graphviz bin directory to PATH
   // This is needed for both bundled and system-installed Graphviz to find DLLs
   if (platform === 'win32' && dotPath) {
-    if (!env) {
-      env = Object.assign({}, process.env)
-      envModified = true
-    }
-    
     var binDir = path.dirname(dotPath)
+    // Normalize to Windows path format
+    binDir = binDir.replace(/\//g, '\\')
+    
     // On Windows, PATH might be Path (case-insensitive but Node.js preserves case)
     var pathKey = 'PATH'
     for (var key in process.env) {
@@ -147,14 +185,11 @@ function execWithSpawn (argv, cwd, options, cb) {
     
     var existingPath = env[pathKey] || env.PATH || ''
     
-    // Normalize binDir to Windows path format
-    binDir = binDir.replace(/\//g, '\\')
-    
     // Check if bin directory is already in PATH
     var pathEntries = existingPath.split(';')
     var alreadyInPath = false
-    for (var i = 0; i < pathEntries.length; i++) {
-      var normalizedEntry = pathEntries[i].replace(/\//g, '\\').toLowerCase().trim()
+    for (var j = 0; j < pathEntries.length; j++) {
+      var normalizedEntry = pathEntries[j].replace(/\//g, '\\').toLowerCase().trim()
       if (normalizedEntry && normalizedEntry === binDir.toLowerCase()) {
         alreadyInPath = true
         break

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-plantuml-2",
-  "version": "1.1.2",
+  "version": "1.1.3",
   "plantumlVersion": "1.2026.1",
   "description": "Node.js PlantUML renderer with Java backend. Multiple output formats (PNG, SVG, EPS, ASCII, Unicode) with full UTF-8 support",
   "main": "index.js",
@@ -26,13 +26,13 @@
     "plantuml-encoder": "^1.2.5"
   },
   "optionalDependencies": {
-    "@node-plantuml-2/jre-win32-x64": "^1.1.2",
-    "@node-plantuml-2/jre-darwin-arm64": "^1.1.2",
-    "@node-plantuml-2/jre-linux-x64": "^1.1.2",
-    "@node-plantuml-2/graphviz-win32-x64": "^1.1.2",
-    "@node-plantuml-2/graphviz-darwin-arm64": "^1.1.2",
-    "@node-plantuml-2/graphviz-darwin-x64": "^1.1.2",
-    "@node-plantuml-2/graphviz-linux-x64": "^1.1.2"
+    "@node-plantuml-2/jre-win32-x64": "^1.1.3",
+    "@node-plantuml-2/jre-darwin-arm64": "^1.1.3",
+    "@node-plantuml-2/jre-linux-x64": "^1.1.3",
+    "@node-plantuml-2/graphviz-win32-x64": "^1.1.3",
+    "@node-plantuml-2/graphviz-darwin-arm64": "^1.1.3",
+    "@node-plantuml-2/graphviz-darwin-x64": "^1.1.3",
+    "@node-plantuml-2/graphviz-linux-x64": "^1.1.3"
   },
   "devDependencies": {
     "chai": "^4.x",