From dd9d117ab8723fe116135a3f3fec8d1d9466580b Mon Sep 17 00:00:00 2001
From: radhakrishnan <gingeekrishna@gmail.com>
Date: Tue, 22 Jul 2025 20:37:40 +0530
Subject: [PATCH] Fix EMF argument handling to preserve existing args

- Change from assignment to append for inputArgs and outputArgs in EMF handling
- Addresses Sourcery AI feedback about potential argument override
- Ensures compatibility with other file type conversions (e.g., SVG)
---
 src/converters/imagemagick.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/converters/imagemagick.ts b/src/converters/imagemagick.ts
index e3959f8..919308b 100644
--- a/src/converters/imagemagick.ts
+++ b/src/converters/imagemagick.ts
@@ -463,8 +463,8 @@ export function convert(
   // Handle EMF files specifically to avoid LibreOffice delegate issues
   if (fileType === "emf") {
     // Use direct conversion without delegates for EMF files
-    inputArgs = ["-define", "emf:delegate=false", "-density", "300"];
-    outputArgs = ["-background", "white", "-alpha", "remove"];
+    inputArgs.push("-define", "emf:delegate=false", "-density", "300");
+    outputArgs.push("-background", "white", "-alpha", "remove");
   }
 
   return new Promise((resolve, reject) => {