From e8e3e01f312398088f9d473395867957577be05c Mon Sep 17 00:00:00 2001
From: Hagernesh <hagernesh@example.com>
Date: Sat, 4 Jul 2026 07:36:53 +0000
Subject: [PATCH] release order document fix

---
 apps/edr-freight-api/Dockerfile                      | 11 ++++++++++-
 .../train-scheduling/train-scheduling.service.ts     |  7 +++++--
 .../warehouses/warehouse-release-document.service.ts | 12 ++++++++++++
 3 files changed, 27 insertions(+), 3 deletions(-)

diff --git a/apps/edr-freight-api/Dockerfile b/apps/edr-freight-api/Dockerfile
index f9107ed23..b781fb4c0 100644
--- a/apps/edr-freight-api/Dockerfile
+++ b/apps/edr-freight-api/Dockerfile
@@ -7,6 +7,9 @@ RUN apk add --no-cache libc6-compat
 # `--mount=type=cache,target=/pnpm/store` cache actually persists deps across builds.
 ENV PNPM_HOME="/pnpm"
 ENV PATH="$PNPM_HOME:$PATH"
+# Puppeteer uses the system Chromium installed in the runner stage — skip the
+# ~150MB bundled-Chromium download during pnpm install.
+ENV PUPPETEER_SKIP_DOWNLOAD=true
 RUN corepack enable
 WORKDIR /app
 
@@ -32,8 +35,14 @@ RUN --mount=type=cache,id=pnpm,target=/pnpm/store \
     pnpm deploy --filter="@edr/freight-api" --prod --legacy /deploy
 
 FROM node:24.15.0-alpine AS runner
-RUN apk add --no-cache libc6-compat
+# Chromium + fonts for headless PDF rendering (puppeteer). Alpine ships the
+# binary at /usr/bin/chromium-browser, which the PDF renderer auto-detects
+# (also pinned via PUPPETEER_EXECUTABLE_PATH). Without this, PDF generation
+# falls back to a degraded hand-built layout.
+RUN apk add --no-cache libc6-compat \
+    chromium nss freetype harfbuzz ca-certificates ttf-freefont
 ENV NODE_ENV=production
+ENV PUPPETEER_EXECUTABLE_PATH=/usr/bin/chromium-browser
 WORKDIR /app
 RUN addgroup --system --gid 1001 nodejs \
     && adduser --system --uid 1001 --ingroup nodejs nestjs
diff --git a/apps/edr-freight-api/src/modules/train-scheduling/train-scheduling.service.ts b/apps/edr-freight-api/src/modules/train-scheduling/train-scheduling.service.ts
index 32a046356..86e37451e 100644
--- a/apps/edr-freight-api/src/modules/train-scheduling/train-scheduling.service.ts
+++ b/apps/edr-freight-api/src/modules/train-scheduling/train-scheduling.service.ts
@@ -1265,7 +1265,9 @@ export class TrainSchedulingService {
       performedBy: 'DOCUMENT_GENERATION',
     });
     const html = this.buildImportLoadListHtml(loadList);
-    const buffer = await this.pdfDocuments.htmlToPdfBuffer(html);
+    // Generic render — NOT the release-order fallback (would mislabel this as a
+    // gate-clearance / release order when Chromium is unavailable).
+    const buffer = await this.pdfDocuments.renderDocumentHtml(html, 'Import marshalling / load list');
     const reference = loadList.trainNumber ?? loadList.trainScheduleId;
     return {
       filename: `import-marshalling-${this.safeDocumentName(reference)}.pdf`,
@@ -1283,7 +1285,8 @@ export class TrainSchedulingService {
     }
 
     const html = this.buildExportLoadListHtml(schedule);
-    const buffer = await this.pdfDocuments.htmlToPdfBuffer(html);
+    // Generic render — NOT the release-order fallback (see importLoadListDocument).
+    const buffer = await this.pdfDocuments.renderDocumentHtml(html, 'Export marshalling / load list');
     const reference = schedule.trainNumber ?? schedule.id;
     return {
       filename: `export-marshalling-${this.safeDocumentName(reference)}.pdf`,
diff --git a/apps/edr-freight-api/src/modules/warehouses/warehouse-release-document.service.ts b/apps/edr-freight-api/src/modules/warehouses/warehouse-release-document.service.ts
index f8c0dd355..68e630e0b 100644
--- a/apps/edr-freight-api/src/modules/warehouses/warehouse-release-document.service.ts
+++ b/apps/edr-freight-api/src/modules/warehouses/warehouse-release-document.service.ts
@@ -20,6 +20,18 @@ export class WarehouseReleaseDocumentService {
     });
   }
 
+  /**
+   * Render arbitrary document HTML to PDF via the shared renderer WITHOUT the
+   * release-order fallback. Non-release documents (e.g. the import/export
+   * marshalling load list) must use this so a Chromium-less fallback degrades to
+   * a plain-text dump of *their own* content — instead of masquerading as a
+   * "Warehouse Gate Clearance / Release Order", which the release-specific
+   * fallback would otherwise draw regardless of the input HTML.
+   */
+  renderDocumentHtml(html: string, label = 'Document'): Promise<Buffer> {
+    return this.pdf.htmlToPdfBuffer(html, { label });
+  }
+
   private htmlToBasicPdfBuffer(html: string): Buffer {
     const doc = this.extractReleaseDocument(html);
     const body: string[] = [