fix extra bullets

Author: jumptuner

al_test.js

@@ -31,20 +31,21 @@ const renderConfigs = [
     outputName: "resultFromDOCX.docx",
   },
   {
-    enabled: false,
-    templatePath: "../freshtracksback_s4/templates/FreshtracksAllCompanies.odt",
-    options: { convertTo: "pdf" },
-    outputName: "resultReportFromODT.pdf",
+    enabled: true,
+    templatePath:
+      "../freshtracksback_s4/templates/FreshtracksAllCompanies.docx",
+    options: { convertTo: "docx" },
+    outputName: "resultReportFromDOCX.docx",
   },
   {
-    enabled: true,
+    enabled: false,
     templatePath:
       "../freshtracksback_s4/templates/FreshtracksAllCompanies.docx",
     options: { convertTo: "docx" },
     outputName: "resultReportFromDOCX.docx",
   },
   {
-    enabled: true,
+    enabled: false,
     templatePath: "../freshtracksback_s4/templates/FreshtracksPresentation.odp",
     options: {
       convertTo: "pdf",

lib/postprocessor.js

@@ -418,6 +418,13 @@ class DocxPostProcessor extends Processor {
               return drawing;
             }
 
+            // Extract original dimensions and other properties to preserve
+            const extentMatch = /<wp:extent cx="(\d+)" cy="(\d+)"/.exec(
+              drawing
+            );
+            const originalCx = extentMatch ? extentMatch[1] : null;
+            const originalCy = extentMatch ? extentMatch[2] : null;
+
             const [, extension] = mime.split("/");
             let newRelationshipId = this.relationshipMap.get(imageUrl);
             if (!newRelationshipId) {
@@ -460,7 +467,7 @@ class DocxPostProcessor extends Processor {
               );
             }
 
-            // Create updated drawing with new relationship ID
+            // Create updated drawing with new relationship ID but preserve dimensions
             const updatedDrawing = drawing
               .replace(/(title|descr)="[^"]*"/g, '$1=""')
               .replace(/embed="[^"]+"/g, `embed="${newRelationshipId}"`);
@@ -598,10 +605,6 @@ class DocxPostProcessor extends Processor {
         <w:next w:val="TextBody"/>
         <w:qFormat/>
         <w:pPr>
-          <w:numPr>
-            <w:ilvl w:val="0"/>
-            <w:numId w:val="1"/>
-          </w:numPr>
           <w:spacing w:before="240" w:after="120"/>
           <w:outlineLvl w:val="0"/>
         </w:pPr>
@@ -703,6 +706,12 @@ class OdpPostProcessor extends Processor {
             return drawFrame;
           }
 
+          // Extract original dimensions to preserve aspect ratio
+          const widthMatch = /svg:width="([^"]+)"/.exec(drawFrame);
+          const heightMatch = /svg:height="([^"]+)"/.exec(drawFrame);
+          const originalWidth = widthMatch ? widthMatch[1] : null;
+          const originalHeight = heightMatch ? heightMatch[1] : null;
+
           const [, extension] = mime.split("/");
           const [basename, newfile] = this.filestore.fileBaseName(content);
           const imgFile = `Pictures/${basename}.${extension}`;
@@ -730,9 +739,11 @@ class OdpPostProcessor extends Processor {
 
           // Ensure the image is actually displayed
           if (!updatedDrawFrame.includes("<draw:image")) {
+            // Preserve original dimensions if available
+            const imageTag = `<draw:image xlink:href="${imgFile}" xlink:type="simple" xlink:show="embed" xlink:actuate="onLoad"/>`;
             updatedDrawFrame = updatedDrawFrame.replace(
               /<draw:frame/,
-              `<draw:frame><draw:image xlink:href="${imgFile}" xlink:type="simple" xlink:show="embed" xlink:actuate="onLoad"/>`
+              `<draw:frame>${imageTag}`
             );
           }