fix: changes after testing

Author: sf-pallavi-das

package.json

@@ -18,18 +18,10 @@
     "@xmldom/xmldom": "^0.9.8",
     "cli-progress": "^3.12.0",
     "diff": "^5.1.0",
-    "form-data": "^4.0.4",
     "jsdom": "^25.0.0",
-    "jsonwebtoken": "^9.0.2",
     "lodash.chunk": "^4.2.0",
-    "marked": "^16.1.2",
-    "minimatch": "^10.0.3",
-    "nanoid": "^5.1.5",
     "open": "^8.4.2",
-    "semver": "^7.7.2",
     "shelljs": "^0.8.5",
-    "tough-cookie": "^5.1.2",
-    "trim-newlines": "^5.0.0",
     "tslib": "^2"
   },
   "devDependencies": {
@@ -94,7 +86,13 @@
     "brace-expansion": "^2.0.2",
     "tmp": "^0.2.4",
     "request": "^2.88.2",
-    "lodash.template": "^4.5.0"
+    "lodash.template": "^4.5.0",
+    "@babel/types": "^7.25.6",
+    "@babel/traverse": "^7.25.6",
+    "@babel/parser": "^7.25.6",
+    "@babel/core": "^7.25.6",
+    "@babel/template": "^7.25.6",
+    "xml2js": "^0.5.0"
   },
   "engines": {
     "node": ">=12.0.0"
@@ -131,18 +129,18 @@
   },
   "repository": "salesforcecli/plugin-omnistudio-migration-tool",
   "scripts": {
-    "build": "sf-build && yarn copy-assets",
+    "build": "sf-build",
     "clean": "sf-clean",
     "clean-all": "sf-clean all",
     "clean:lib": "shx rm -rf lib && shx rm -rf coverage && shx rm -rf .nyc_output && shx rm -f oclif.manifest.json",
-    "compile": "sf-compile && yarn copy-assets",
+    "compile": "sf-compile",
     "copy-assets": "shx cp -r src/templates lib/ && shx cp -r src/styles lib/ && shx cp -r src/javascripts lib/",
     "docs": "sf-docs",
     "format": "sf-format",
     "lint": "sf-lint",
     "postpack": "shx rm -f oclif.manifest.json",
     "posttest": "yarn lint && yarn test:deprecation-policy && yarn test:command-reference",
-    "prepack": "sf-prepack && yarn copy-assets",
+    "prepack": "sf-prepack",
     "prepare": "sf-install",
     "pretest": "sf-compile-test",
     "test": "sf-test",

src/utils/XMLUtil.ts

@@ -50,13 +50,8 @@ export class XMLUtil {
 
   // Parse XML string to JSON
   public parse(xmlString: string): any {
-    try {
-      const doc = this.parser.parseFromString(xmlString, 'text/xml');
-      return this.xmlToJson(doc.documentElement as any);
-    } catch (error) {
-      // Return empty object for invalid XML instead of throwing
-      return {};
-    }
+    const doc = this.parser.parseFromString(xmlString, 'text/xml');
+    return this.xmlToJson(doc.documentElement as any);
   }
 
   // Convert JSON object to XML

test/utils/XMLUtil.test.ts

@@ -93,13 +93,13 @@ describe('XMLUtil', () => {
       });
     });
 
-    it('should handle invalid XML gracefully', () => {
+    it('should throw for invalid xml', () => {
       const invalidXml = '<root><unclosed>';
 
-      // xmldom doesn't throw errors, it returns a document with errors
+      // xmldom to throw error for mismatched tags after package update to @xmldom/xmldom.
       expect(() => {
         xmlUtil.parse(invalidXml);
-      }).to.not.throw();
+      }).to.throw();
     });
   });