Skip to content

fix: remove @antora/pdf-extension to unblock CI#3140

Merged
gtrivedi88 merged 1 commit into
eclipse-che:mainfrom
gtrivedi88:fix-remove-pdf-extension
Jul 2, 2026
Merged

fix: remove @antora/pdf-extension to unblock CI#3140
gtrivedi88 merged 1 commit into
eclipse-che:mainfrom
gtrivedi88:fix-remove-pdf-extension

Conversation

@gtrivedi88

Copy link
Copy Markdown
Contributor

Summary

  • Remove @antora/pdf-extension from antora-playbook-for-development.yml

Why

The @antora/pdf-extension requires asciidoctor-pdf (Ruby gem) to convert assembled AsciiDoc to PDF. This gem was never installed in the container. After the base image (quay.io/podman/stable:latest) updated, the extension started crashing with:

FATAL (antora): ENOENT: no such file or directory, open
  'build/assembler-pdf/docs/next/_exports/discover.pdf'

This breaks all PR checks — both "Build and validate pull request" and "Build and verify container".

PDFs are not served on the live site. The extension comment confirms it was used for "monolithic AsciiDoc files", not actual PDFs. The npm packages (@antora/assembler, @antora/pdf-extension) remain in the Containerfile for potential future use.

Test plan

  • "Build and validate pull request" check passes
  • "Build and verify container" check passes
  • Live site preview renders correctly on Netlify

Made with Cursor

The @antora/pdf-extension requires the asciidoctor-pdf Ruby gem to
convert assembled AsciiDoc to PDF. This gem was never installed in the
che-docs container, so PDF generation silently failed. After the base
image (quay.io/podman/stable:latest) updated, the extension started
crashing with ENOENT instead of failing silently:

  FATAL (antora): ENOENT: no such file or directory, open
  'build/assembler-pdf/docs/next/_exports/discover.pdf'

This breaks all PR checks. Since PDFs are not served on the live site
and the extension comment confirms it was used for "monolithic AsciiDoc
files" (not actual PDFs), removing it is safe.

The @antora/assembler and @antora/pdf-extension npm packages remain in
the Containerfile for potential future use.

Co-authored-by: Cursor <cursoragent@cursor.com>
@gtrivedi88 gtrivedi88 requested review from a team, deerskindoll and mkuznyetsov as code owners July 1, 2026 18:18
@gtrivedi88 gtrivedi88 requested a review from tolusha July 1, 2026 18:18
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

🎊 Navigate the preview: https://6a46170ab3b70f6d34f830b9--eclipse-che-docs-pr.netlify.app 🎊

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Click here to review and test in web IDE: Contribute

@gtrivedi88 gtrivedi88 merged commit c88aee6 into eclipse-che:main Jul 2, 2026
7 checks passed
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.

2 participants