Skip to content

feat: libreoffice extension for docx/pptx to PDF conversion#3646

Closed
CuboYe wants to merge 8 commits into
triggerdotdev:mainfrom
CuboYe:feat/libreoffice-extension
Closed

feat: libreoffice extension for docx/pptx to PDF conversion#3646
CuboYe wants to merge 8 commits into
triggerdotdev:mainfrom
CuboYe:feat/libreoffice-extension

Conversation

@CuboYe
Copy link
Copy Markdown

@CuboYe CuboYe commented May 17, 2026

Closes #1361

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 17, 2026

⚠️ No Changeset found

Latest commit: 8d70ad2

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

Hi @CuboYe, thanks for your interest in contributing!

This project requires that pull request authors are vouched, and you are not in the list of vouched users.

This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details.

@github-actions github-actions Bot closed this May 17, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 17, 2026

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a2f1e888-875c-4a77-85a2-697d8cd0aadb

📥 Commits

Reviewing files that changed from the base of the PR and between a8280f1 and 8d70ad2.

⛔ Files ignored due to path filters (5)
  • references/libreoffice-convert/.gitignore is excluded by !references/**
  • references/libreoffice-convert/package.json is excluded by !references/**
  • references/libreoffice-convert/src/index.ts is excluded by !references/**
  • references/libreoffice-convert/trigger.config.ts is excluded by !references/**
  • references/libreoffice-convert/tsconfig.json is excluded by !references/**
📒 Files selected for processing (2)
  • packages/build/src/extensions/core.ts
  • packages/build/src/extensions/core/libreoffice.ts

Walkthrough

This PR adds a new LibreOffice build extension that provides Docker system package installation and npm dependency management for PDF conversion. The extension is configurable via a minimal option that switches between a lightweight package set (writer, calc, impress with fonts) and the full LibreOffice package. It resolves the local libreoffice-convert npm package version with a fallback to latest. The extension only executes during deploy target builds and registers two build layers for apt packages and npm dependencies. The new extension is exposed through the core module's re-export surface.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
⚔️ Resolve merge conflicts
  • Resolve merge conflict in branch feat/libreoffice-extension

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create a libreoffice build extension to allow docx/pptx to PDF conversion

1 participant