feat: libreoffice v25 support

Author: vladholubiev

.gitignore

@@ -26,10 +26,7 @@ yarn.lock
 .idea/.gitignore
 !.husky/_/husky.sh
 
-test/cid
-test/lib
-test/node_modules
-test/dist/
-test/test-bundled.js
+dist/
+tests/integration/lambda/__fixtures__/documents/**/*.pdf
 
 .DS_Store

eslint.config.mjs

@@ -2,15 +2,22 @@ import rules from '@shelf/eslint-config/typescript.js';
 
 export default [
   ...rules,
-  {files: ['**/*.js', '**/*.jsx', '**/*.ts', '**/*.tsx', '**/*.json']},
+  {files: ['**/*.js', '**/*.mjs', '**/*.jsx', '**/*.ts', '**/*.tsx', '**/*.json']},
   {
     ignores: [
       '**/node_modules/',
       '**/coverage/',
       '**/lib/',
+      '**/dist/',
       'renovate.json',
       'tsconfig.json',
       '.pnpm-store/',
     ],
   },
+  {
+    files: ['tests/**/*.{js,jsx,ts,tsx,mjs}'],
+    rules: {
+      'multiline-ternary': 'off',
+    },
+  },
 ];

jest.config.mjs

@@ -3,6 +3,9 @@
  */
 const config = {
   extensionsToTreatAsEsm: ['.ts'],
+  roots: ['<rootDir>/tests'],
+  testMatch: ['**/*.test.ts'],
+  setupFilesAfterEnv: ['<rootDir>/tests/setup.ts'],
   transform: {
     '^.+\\.(t|j)sx?$': '@swc/jest',
   },

package.json

@@ -29,7 +29,12 @@
     "lint": "eslint . --fix",
     "lint:ci": "eslint .",
     "prepack": "pnpm build",
-    "test": "jest src",
+    "test": "pnpm run test:unit",
+    "test:unit": "jest --runInBand --testPathPatterns tests/unit",
+    "test:integration": "jest --runInBand --testPathPatterns tests/integration",
+    "test:e2e": "pnpm run test:integration",
+    "build:lambda": "node scripts/build-lambda.mjs",
+    "test:lambda": "node scripts/run-lambda-integration.mjs",
     "type-check": "tsc --noEmit",
     "type-check:watch": "pnpm run type-check -- --watch"
   },

readme.md

@@ -93,31 +93,32 @@ module.exports.handler = async () => {
 
 ## Test
 
-Beside unit tests that could be run via `pnpm test`, there are integration tests.
+The repository now ships two Jest suites and a standalone Lambda harness.
 
-### Running Tests
+### Jest suites
+
+- `pnpm test:unit` (or `pnpm test`) – runs unit tests in `tests/unit`
+- `pnpm test:integration` – spins up the Lambda container through Jest; automatically skips if Podman or the base image is unavailable
+- `pnpm test:e2e` – alias for the current integration suite
+
+### Lambda harness
+
+Use `pnpm test:lambda` to exercise the runtime end-to-end. The script bundles the handler, builds a disposable image on top of `libreoffice-lambda-base:local`, starts the Lambda runtime with Podman, invokes it once, and streams the conversion summary plus the temp output directory.
+
+Requirements:
+
+- `podman` installed locally
+- A base image tagged `libreoffice-lambda-base:local` (override with `LIBREOFFICE_LAMBDA_BASE_IMAGE`)
+- `pnpm build` completed beforehand (generates `lib/index.js` consumed by the bundler)
+
+Handy workflow:
 
 ```sh
-# Unit tests
-pnpm test
-
-# Integration test with Docker/Podman
-cd test
-./test.sh
-
-# The test will:
-# 1. Build the ESM code and transpile to CommonJS for Lambda compatibility
-# 2. Process all files in test-data/ directory
-# 3. Generate PDFs in the same test-data/ directory
-# 4. Show conversion summary
+pnpm build
+pnpm test:lambda -- --cleanup
 ```
 
-The test setup includes:
-
-- Automatic ESM to CommonJS transpilation using esbuild
-- Batch conversion of multiple file types (DOCX, HTML, etc.)
-- Volume mounting for easy PDF retrieval
-- Sample test files in `test/test-data/`
+`--cleanup` deletes the generated fixture output once you are done inspecting it. Additional flags like `--bundle`, `--fixtures`, and `--port` are available for custom runs; see `scripts/run-lambda-integration.mjs` for the full list.
 
 ## Publish
 

scripts/build-lambda.mjs

@@ -0,0 +1,44 @@
+#!/usr/bin/env node
+import {fileURLToPath} from 'node:url';
+import path from 'node:path';
+import {buildLambdaBundle} from './utils/lambda.js';
+
+const parseArgs = argv => {
+  const args = {};
+
+  for (let index = 0; index < argv.length; index += 1) {
+    const value = argv[index];
+
+    if (value === '--outfile' || value === '-o') {
+      args.outfile = argv[index + 1];
+      index += 1;
+      continue;
+    }
+
+    if (value === '--entry') {
+      args.entry = argv[index + 1];
+      index += 1;
+      continue;
+    }
+  }
+
+  return args;
+};
+
+const main = async () => {
+  const args = parseArgs(process.argv.slice(2));
+
+  try {
+    const outfile = await buildLambdaBundle(args);
+    console.log(`Lambda bundle generated at ${outfile}`);
+  } catch (error) {
+    console.error(error instanceof Error ? error.message : error);
+    process.exitCode = 1;
+  }
+};
+
+const executedFile = fileURLToPath(import.meta.url);
+
+if (process.argv[1] && path.resolve(process.argv[1]) === executedFile) {
+  await main();
+}

scripts/run-lambda-integration.mjs

@@ -0,0 +1,91 @@
+#!/usr/bin/env node
+import {fileURLToPath} from 'node:url';
+import path from 'node:path';
+import {isPodmanAvailable, removeTempDir, runLambdaContainer} from './utils/lambda.js';
+
+const parseArgs = argv => {
+  const args = {};
+
+  for (let index = 0; index < argv.length; index += 1) {
+    const value = argv[index];
+
+    if (value === '--image-tag') {
+      args.imageTag = argv[index + 1];
+      index += 1;
+      continue;
+    }
+
+    if (value === '--base-image') {
+      args.baseImage = argv[index + 1];
+      index += 1;
+      continue;
+    }
+
+    if (value === '--bundle') {
+      args.bundlePath = argv[index + 1];
+      index += 1;
+      continue;
+    }
+
+    if (value === '--fixtures') {
+      args.documentsDir = argv[index + 1];
+      index += 1;
+      continue;
+    }
+
+    if (value === '--port') {
+      args.port = Number(argv[index + 1]);
+      index += 1;
+      continue;
+    }
+
+    if (value === '--cleanup') {
+      args.cleanup = true;
+      continue;
+    }
+  }
+
+  return args;
+};
+
+const main = async () => {
+  const args = parseArgs(process.argv.slice(2));
+
+  if (!(await isPodmanAvailable())) {
+    console.error('podman is not available. Install it to run the Lambda integration harness.');
+    process.exitCode = 1;
+
+    return;
+  }
+
+  let cleanupDir;
+
+  try {
+    const result = await runLambdaContainer(args);
+    cleanupDir = result.outputDir;
+
+    console.log('Lambda invocation response:', result.response);
+
+    if (result.results) {
+      console.log('Conversion results parsed from logs:', result.results);
+    } else {
+      console.log('Raw container logs:\n', result.logs);
+    }
+
+    console.log('Generated files:', result.outputFiles);
+    console.log(`Output directory: ${result.outputDir}`);
+  } catch (error) {
+    console.error(error instanceof Error ? error.message : error);
+    process.exitCode = 1;
+  } finally {
+    if (args.cleanup && cleanupDir) {
+      await removeTempDir(cleanupDir);
+    }
+  }
+};
+
+const executedFile = fileURLToPath(import.meta.url);
+
+if (process.argv[1] && path.resolve(process.argv[1]) === executedFile) {
+  await main();
+}

scripts/utils/bundler.js

@@ -0,0 +1,31 @@
+import {existsSync} from 'node:fs';
+import path from 'node:path';
+import esbuild from 'esbuild';
+import {ensureDirectory} from './io.js';
+import {lambdaBundlePath, lambdaSourceEntry, projectRoot} from './paths.js';
+
+export const buildLambdaBundle = async ({
+  entry = lambdaSourceEntry,
+  outfile = lambdaBundlePath,
+} = {}) => {
+  const buildOutputDir = path.dirname(outfile);
+
+  if (!existsSync(path.join(projectRoot, 'lib/index.js'))) {
+    throw new Error('Missing lib/index.js. Run `pnpm build` before building the Lambda bundle.');
+  }
+
+  ensureDirectory(buildOutputDir);
+
+  await esbuild.build({
+    entryPoints: [entry],
+    bundle: true,
+    platform: 'node',
+    target: 'node20',
+    format: 'cjs',
+    outfile,
+    logLevel: 'info',
+    external: ['child_process', 'fs', 'fs/promises', 'path', 'util', 'node:path'],
+  });
+
+  return outfile;
+};

scripts/utils/io.js

@@ -0,0 +1,22 @@
+import {existsSync, mkdirSync, mkdtempSync} from 'node:fs';
+import {promises as fsPromises} from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+
+export const ensureDirectory = directory => {
+  if (!existsSync(directory)) {
+    mkdirSync(directory, {recursive: true});
+  }
+};
+
+export const createTempDir = (prefix = 'lambda-fixtures-') => {
+  const base = path.join(os.tmpdir(), prefix);
+  const directory = mkdtempSync(base);
+  ensureDirectory(directory);
+
+  return directory;
+};
+
+export const removeDir = async directory => {
+  await fsPromises.rm(directory, {recursive: true, force: true});
+};