fix: strip HTML tags properly (#110)

kkoomen · Oct 17, 2020 · 8e55d37 · 8e55d37
1 parent bbbcbd4
commit 8e55d37
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 4 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -20,7 +20,7 @@ if (languageParser) {
   // parser instead of the TSX parser.
   const sourceCode = fs
     .readFileSync(filepath, { encoding: 'utf8', flag: 'r' })
-    .replace(/<\/?[^>]+(>|$)/g, '');
+    .replace(/(?:<?[^>]+>(.*)<\/[^>]+>|<\/[^>]+\/>)/g, '$1');
 
   const tree = parser.parse(sourceCode);
   const parserService = getParserService(language, [

diff --git a/test/filetypes/javascript/jsx.vader b/test/filetypes/javascript/jsx.vader
@@ -3,13 +3,19 @@
 # ==============================================================================
 Given javascript (function that contains JSX):
   const Test = ({ someVal }) => {
-    return <div>{true && someVal ? <small className="helper-text">{someVal}</small> : null}</div>;
+    return (
+      <div>
+        {true && someVal ? <small className="helper-text">{someVal}</small> : null}
+        <span />
+        <span />
+      </div>
+    );
   };
 
 Do (trigger doge):
   \<C-d>
 
-Expect javascript (generated comment with a description and a @return tag):
+Expect javascript (generated comment with a description, @param and @return tags):
   /**
    * [TODO:description]
    *
@@ -18,5 +24,11 @@ Expect javascript (generated comment with a description and a @return tag):
    * @return {[TODO:type]} [TODO:description]
    */
   const Test = ({ someVal }) => {
-    return <div>{true && someVal ? <small className="helper-text">{someVal}</small> : null}</div>;
+    return (
+      <div>
+        {true && someVal ? <small className="helper-text">{someVal}</small> : null}
+        <span />
+        <span />
+      </div>
+    );
   };