@adobe/helix-importer 3.1.2 → 3.1.4

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [3.1.4](https://github.com/adobe/helix-importer/compare/v3.1.3...v3.1.4) (2024-01-17)
+
+
+### Bug Fixes
+
+* import comments cleanup logic ([#290](https://github.com/adobe/helix-importer/issues/290)) ([deec675](https://github.com/adobe/helix-importer/commit/deec675c6e88f0035a3d5b3d88b4068f106a8f80))
+
+## [3.1.3](https://github.com/adobe/helix-importer/compare/v3.1.2...v3.1.3) (2024-01-06)
+
+
+### Bug Fixes
+
+* **deps:** update adobe fixes ([#286](https://github.com/adobe/helix-importer/issues/286)) ([27acb37](https://github.com/adobe/helix-importer/commit/27acb37aacf312dfbbaca8338bddec7e5a69e46e))
+
 ## [3.1.2](https://github.com/adobe/helix-importer/compare/v3.1.1...v3.1.2) (2023-12-30)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-importer",
-  "version": "3.1.2",
+  "version": "3.1.4",
   "description": "Helix Importer tool: create md / docx from html",
   "main": "src/index.js",
   "type": "module",
@@ -29,7 +29,7 @@
   "devDependencies": {
     "@adobe/eslint-config-helix": "2.0.5",
     "@adobe/helix-docx2md": "1.5.0",
-    "@adobe/helix-mediahandler": "2.4.3",
+    "@adobe/helix-mediahandler": "2.4.5",
     "@esm-bundle/chai": "4.3.4-fix.0",
     "@semantic-release/changelog": "6.0.3",
     "@semantic-release/exec": "6.0.3",
@@ -37,12 +37,12 @@
     "@web/test-runner": "0.18.0",
     "@web/test-runner-commands": "0.9.0",
     "@web/test-runner-mocha": "0.9.0",
-    "c8": "8.0.1",
-    "chai": "4.3.10",
+    "c8": "9.1.0",
+    "chai": "5.0.0",
     "dirname-filename-esm": "1.1.1",
     "eslint": "8.56.0",
     "husky": "8.0.3",
-    "jsdom": "23.0.1",
+    "jsdom": "23.2.0",
     "lint-staged": "15.2.0",
     "mocha": "10.2.0",
     "mocha-multi-reporters": "1.5.1",
@@ -54,7 +54,7 @@
   "license": "Apache-2.0",
   "dependencies": {
     "@adobe/helix-markdown-support": "7.1.0",
-    "@adobe/helix-md2docx": "2.1.38",
+    "@adobe/helix-md2docx": "2.1.39",
     "@adobe/mdast-util-gridtables": "3.0.2",
     "@adobe/remark-gridtables": "2.0.2",
     "form-data": "4.0.0",

package/src/utils/DOMUtils.js

@@ -139,7 +139,8 @@ export default class DOMUtils {
     // eslint-disable-next-line no-param-reassign
     document.body.innerHTML = document.body.innerHTML
       // remove html comments
-      .replace(/<!--(?!>)[\S\s]*?-->/gm, '');
+      .replace(/><!--(?!>)[\S\s]*?-->/gm, '>')
+      .replace(/\n<!--(?!>)[\S\s]*?-->/gm, '\n');
   }
 
   static removeSpans(document) {

package/test/utils/DOMUtils.spec.js

@@ -182,6 +182,8 @@ describe('DOMUtils#removeCommments tests', () => {
     test('<p><!-- useless comment \n multiline --></p>', '<p></p>');
     test('<p><!-- useless comment \n multiline \n multiline --></p>', '<p></p>');
     test('<!-- useless comment --><p>The content stays</p><!-- another useless comment with \n line break -->', '<p>The content stays</p>');
+    test('<p>This is a paragraph.</p>\n\x3C!--\n<p>Look at this cool image:</p>\n<img border="0" src="pic_trulli.jpg" alt="Trulli">\n-->\n<p>This is a paragraph too.</p>\x3C!-- same line -->\n\x3C!-- single line -->', '<p>This is a paragraph.</p>\n\n<p>This is a paragraph too.</p>\n');
+    test('<div some-crazy-attribute="" <!--=""><!-- useless comment --></div>', '<div some-crazy-attribute="" <!--=""></div>');
   });
 });