fix: regression with comments inside parsed text state (#132)

Author: DylanPiercey

.changeset/neat-cobras-battle.md

@@ -0,0 +1,5 @@
+---
+"htmljs-parser": patch
+---
+
+Fix regression which caused script tags with a trailing comment as the same line as the closing tag to not always parse properly.

.changeset/quick-seals-watch.md

@@ -0,0 +1,5 @@
+---
+"htmljs-parser": patch
+---
+
+Remove unecessary check for cdata inside parsed text state.

src/__tests__/fixtures/cdata/__snapshots__/cdata.expected.txt

@@ -1,9 +1,3 @@
-1╭─ <html-comment><![CDATA[[if lt IE 9]><div><![endif]]]></html-comment>
- │  ││           ││        │                             │ │           ╰─ closeTagEnd(html-comment)
- │  ││           ││        │                             │ ╰─ closeTagName "html-comment"
- │  ││           ││        │                             ╰─ closeTagStart "</"
- │  ││           ││        ╰─ cdata.value "[if lt IE 9]><div><![endif]"
- │  ││           │╰─ cdata "<![CDATA[[if lt IE 9]><div><![endif]]]>"
- │  ││           ╰─ openTagEnd
- │  │╰─ tagName "html-comment"
- ╰─ ╰─ openTagStart
+1╭─ <![CDATA[[if lt IE 9]><div><![endif]]]>
+ │  │        ╰─ cdata.value "[if lt IE 9]><div><![endif]"
+ ╰─ ╰─ cdata "<![CDATA[[if lt IE 9]><div><![endif]]]>"

src/__tests__/fixtures/cdata/input.marko

@@ -1 +1 @@
-<html-comment><![CDATA[[if lt IE 9]><div><![endif]]]></html-comment>
+<![CDATA[[if lt IE 9]><div><![endif]]]>

src/__tests__/fixtures/script-single-line-comment/__snapshots__/script-single-line-comment.expected.txt

@@ -5,4 +5,33 @@
  │  ││     │╰─ text "// this is a comment"
  │  ││     ╰─ openTagEnd
  │  │╰─ tagName "script"
- ╰─ ╰─ openTagStart
+ ╰─ ╰─ openTagStart
+2├─ 
+3╭─ <div>
+ │  ││  ╰─ openTagEnd
+ │  │╰─ tagName "div"
+ ╰─ ╰─ openTagStart
+4╭─   <script>// this is a comment</script>
+ │  │ ││     ││                   │ │     ╰─ closeTagEnd(script)
+ │  │ ││     ││                   │ ╰─ closeTagName "script"
+ │  │ ││     ││                   ╰─ closeTagStart "</"
+ │  │ ││     │╰─ text "// this is a comment"
+ │  │ ││     ╰─ openTagEnd
+ │  │ │╰─ tagName "script"
+ │  │ ╰─ openTagStart
+ ╰─ ╰─ text "\n  "
+5╭─   <span>hi</span>
+ │  │ ││   ││ │ │   ╰─ closeTagEnd(span)
+ │  │ ││   ││ │ ╰─ closeTagName "span"
+ │  │ ││   ││ ╰─ closeTagStart "</"
+ │  │ ││   │╰─ text "hi"
+ │  │ ││   ╰─ openTagEnd
+ │  │ │╰─ tagName "span"
+ │  │ ╰─ openTagStart
+ ╰─ ╰─ text "\n  "
+6╭─ </div>
+ │  │ │  ╰─ closeTagEnd(div)
+ │  │ ╰─ closeTagName "div"
+ │  ├─ text "\n"
+ ╰─ ╰─ closeTagStart "</"
+7╰─ 

src/__tests__/fixtures/script-single-line-comment/input.marko

@@ -1 +1,6 @@
-<script>// this is a comment</script>
+<script>// this is a comment</script>
+
+<div>
+  <script>// this is a comment</script>
+  <span>hi</span>
+</div>

src/states/JS_COMMENT_LINE.ts

@@ -21,11 +21,12 @@ export const JS_COMMENT_LINE: StateDefinition = {
     if (
       !this.isConcise &&
       code === CODE.OPEN_ANGLE_BRACKET &&
-      this.activeTag?.type === TagType.text
+      this.activeTag?.type === TagType.text &&
+      STATE.checkForClosingTag(this)
     ) {
-      // First, see if we need to see if we reached the closing tag
+      // We need to see if we reached the closing tag
       // eg: <script>//foo</script>
-      STATE.checkForClosingTag(this);
+      this.exitState();
     }
   },
 

src/states/PARSED_TEXT_CONTENT.ts

@@ -29,10 +29,7 @@ export const PARSED_TEXT_CONTENT: StateDefinition<ParsedTextContentMeta> = {
   char(code) {
     switch (code) {
       case CODE.OPEN_ANGLE_BRACKET:
-        if (
-          this.isConcise ||
-          !(STATE.checkForClosingTag(this) || STATE.checkForCDATA(this))
-        ) {
+        if (this.isConcise || !STATE.checkForClosingTag(this)) {
           this.startText();
         }
         break;