@docverse-pdf/server 1.1.0 → 1.1.1

package/dist/unoServer.d.ts

@@ -1,6 +1,12 @@
 export interface UnoServerOptions {
-    /** TCP port the unoserver daemon listens on (default: 2003). */
+    /** TCP port the unoserver daemon listens on for its own RPC (default: 2003). */
     port?: number;
+    /**
+     * TCP port used by the underlying LibreOffice UNO socket. Must be unique per
+     * daemon — otherwise multiple daemons collide on the default 2002.
+     * (default: undefined = unoserver picks 2002, which only works for one daemon.)
+     */
+    unoPort?: number;
     /** Host/interface the daemon binds to (default: 127.0.0.1). */
     host?: string;
     /** Path to the `unoserver` executable (default: auto-detect). */
@@ -9,18 +15,27 @@ export interface UnoServerOptions {
     unoconvertPath?: string;
     /** Path to the LibreOffice `soffice` binary (passed to unoserver --executable). */
     libreOfficePath?: string;
-    /** Dedicated UNO user-profile dir for this daemon (passed to unoserver --user-installation). */
+    /**
+     * Dedicated UNO user-profile dir for this daemon (passed to unoserver
+     * --user-installation). **Must be an absolute filesystem path, not a
+     * `file://` URI** — unoserver converts it internally via `Path(...).as_uri()`.
+     */
     userInstallation?: string;
     /** Max time (ms) to wait for the daemon to become ready (default: 15000). */
     startTimeout?: number;
     /** Max time (ms) to wait for a single conversion (default: 30000). */
     convertTimeout?: number;
 }
-export interface UnoServerPoolOptions extends Omit<UnoServerOptions, 'port' | 'userInstallation'> {
+export interface UnoServerPoolOptions extends Omit<UnoServerOptions, 'port' | 'unoPort' | 'userInstallation'> {
     /** Number of parallel unoserver daemons (default: 2). */
     workers?: number;
-    /** First TCP port; each worker gets basePort+i (default: 2003). */
+    /** First daemon RPC port; each worker gets basePort+i (default: 2003). */
     basePort?: number;
+    /**
+     * First LibreOffice UNO-socket port; each worker gets unoBasePort+i
+     * (default: 2100). Must be in a non-overlapping range with `basePort`.
+     */
+    unoBasePort?: number;
     /** Base dir for per-worker user-profile dirs (default: os.tmpdir()/docverse_unoserver). */
     tempDir?: string;
 }

package/dist/unoServer.js

@@ -1,5 +1,6 @@
 import { spawn, execSync } from 'child_process';
 import { createConnection } from 'net';
+import { resolve as resolvePath } from 'path';
 // ═══════════════════════════════════════════════
 // HELPERS
 // ═══════════════════════════════════════════════
@@ -104,11 +105,14 @@ export class UnoServer {
     constructor(options = {}) {
         this.opts = {
             port: options.port ?? 2003,
+            unoPort: options.unoPort,
             host: options.host ?? '127.0.0.1',
             unoserverPath: options.unoserverPath ?? findUnoserver(),
             unoconvertPath: options.unoconvertPath ?? findUnoconvert(),
             libreOfficePath: options.libreOfficePath ?? findSoffice(),
-            userInstallation: options.userInstallation,
+            userInstallation: options.userInstallation
+                ? resolvePath(options.userInstallation)
+                : undefined,
             startTimeout: options.startTimeout ?? 15000,
             convertTimeout: options.convertTimeout ?? 30000,
         };
@@ -126,6 +130,9 @@ export class UnoServer {
         if (this.ready)
             return;
         const args = ['--interface', this.opts.host, '--port', String(this.opts.port)];
+        if (this.opts.unoPort !== undefined) {
+            args.push('--uno-port', String(this.opts.unoPort));
+        }
         if (this.opts.libreOfficePath) {
             args.push('--executable', this.opts.libreOfficePath);
         }
@@ -136,24 +143,52 @@ export class UnoServer {
             stdio: ['ignore', 'pipe', 'pipe'],
             detached: false,
         });
-        proc.on('error', (err) => {
-            this.ready = false;
-            // Let the caller see the launch error via waitForPort timeout.
-            // Emit to stderr so it's debuggable without swallowing silently.
-            process.stderr.write(`[unoserver] spawn error: ${err.message}\n`);
+        this.proc = proc;
+        // Capture stderr so startup failures surface the real reason, not just a
+        // generic "did not become ready" timeout.
+        const stderrChunks = [];
+        proc.stderr?.on('data', (c) => stderrChunks.push(c));
+        // Also drain stdout (unoserver is quiet here but we must consume it to
+        // avoid filling the pipe buffer on some platforms).
+        proc.stdout?.on('data', () => { });
+        const collectStderr = () => Buffer.concat(stderrChunks).toString('utf8').trim();
+        let earlyExit = null;
+        const earlyExitPromise = new Promise((_, reject) => {
+            proc.once('exit', (code, signal) => {
+                this.ready = false;
+                this.proc = null;
+                if (earlyExit === null) {
+                    earlyExit = { code, signal };
+                    reject(new Error(`unoserver exited during startup (code=${code}, signal=${signal})\n` +
+                        `stderr:\n${collectStderr() || '(empty)'}`));
+                }
+            });
         });
-        proc.on('exit', () => {
-            this.ready = false;
-            this.proc = null;
+        const spawnErrorPromise = new Promise((_, reject) => {
+            proc.once('error', (err) => {
+                this.ready = false;
+                reject(new Error(`unoserver spawn error: ${err.message}`));
+            });
         });
-        this.proc = proc;
         const deadline = Date.now() + this.opts.startTimeout;
         try {
-            await waitForPort(this.opts.host, this.opts.port, deadline);
+            await Promise.race([
+                waitForPort(this.opts.host, this.opts.port, deadline),
+                earlyExitPromise,
+                spawnErrorPromise,
+            ]);
         }
         catch (err) {
             await this.stop();
-            throw err;
+            if (earlyExit !== null) {
+                // Early-exit error is already descriptive; rethrow as-is.
+                throw err;
+            }
+            // Timeout path — attach whatever stderr we collected for debugging.
+            const stderr = collectStderr();
+            const message = `unoserver did not become ready on ${this.opts.host}:${this.opts.port} ` +
+                `within ${this.opts.startTimeout}ms\nstderr:\n${stderr || '(empty)'}`;
+            throw new Error(message);
         }
         this.ready = true;
     }
@@ -257,6 +292,7 @@ export class UnoServerPool {
         this.opts = {
             workers: options.workers ?? 2,
             basePort: options.basePort ?? 2003,
+            unoBasePort: options.unoBasePort ?? 2100,
             tempDir: options.tempDir ?? `${process.env.TMPDIR ?? '/tmp'}/docverse_unoserver`,
             startTimeout: options.startTimeout ?? 15000,
             convertTimeout: options.convertTimeout ?? 30000,
@@ -275,9 +311,12 @@ export class UnoServerPool {
         }
         const servers = [];
         for (let i = 0; i < this.opts.workers; i++) {
-            const userInstallation = `file://${this.opts.tempDir}/worker_${i}`;
+            // unoserver expects a raw absolute path for --user-installation; passing
+            // a `file://` URI trips its internal `Path(...).as_uri()` and crashes.
+            const userInstallation = resolvePath(this.opts.tempDir, `worker_${i}`);
             servers.push(new UnoServer({
                 port: this.opts.basePort + i,
+                unoPort: this.opts.unoBasePort + i,
                 host: this.opts.host,
                 unoserverPath: this.opts.unoserverPath,
                 unoconvertPath: this.opts.unoconvertPath,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@docverse-pdf/server",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "DocVerse Server SDK — server-side PDF processing, XFDF merge, Word to PDF (unoserver + LibreOffice), image extraction",
   "type": "module",
   "main": "dist/index.js",