From 8bd3c4057d485d6b1e47f15bd48e333e25b9951e Mon Sep 17 00:00:00 2001
From: Dylan Piercey <dpiercey@ebay.com>
Date: Mon, 27 Jun 2022 15:11:49 -0700
Subject: [PATCH] fix: onCloseTagStart event for text mode tags (#117)

---
 .changeset/late-cougars-attend.md                      |  5 +++++
 .../fixtures/cdata/__snapshots__/cdata.expected.txt    |  5 +++--
 .../__snapshots__/html-comment-tag.expected.txt        | 10 ++++++----
 ...placeholder-in-template-literal-string.expected.txt |  5 +++--
 .../__snapshots__/parsed-text-style-tag.expected.txt   |  5 +++--
 ...placeholder-body-multi-line-js-comment.expected.txt |  5 +++--
 ...laceholder-body-single-line-js-comment.expected.txt |  5 +++--
 .../placeholder-within-script-tag.expected.txt         |  5 +++--
 ...laceholder-within-script-within-string.expected.txt |  5 +++--
 .../script-escaped-placeholder.expected.txt            |  3 ++-
 .../script-single-line-comment.expected.txt            |  5 +++--
 .../__snapshots__/script-tag-entities.expected.txt     |  5 +++--
 .../fixtures/script/__snapshots__/script.expected.txt  |  5 +++--
 src/states/CLOSE_TAG.ts                                |  4 ++++
 14 files changed, 47 insertions(+), 25 deletions(-)
 create mode 100644 .changeset/late-cougars-attend.md

diff --git a/.changeset/late-cougars-attend.md b/.changeset/late-cougars-attend.md
new file mode 100644
index 00000000..438e5441
--- /dev/null
+++ b/.changeset/late-cougars-attend.md
@@ -0,0 +1,5 @@
+---
+"htmljs-parser": patch
+---
+
+Fix issue with onCloseTagStart not called for text mode tags (eg style, script, textarea & html-comment).
diff --git a/src/__tests__/fixtures/cdata/__snapshots__/cdata.expected.txt b/src/__tests__/fixtures/cdata/__snapshots__/cdata.expected.txt
index 502f223e..bde1ee30 100644
--- a/src/__tests__/fixtures/cdata/__snapshots__/cdata.expected.txt
+++ b/src/__tests__/fixtures/cdata/__snapshots__/cdata.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <html-comment><![CDATA[[if lt IE 9]><div><![endif]]]></html-comment>
- │  ││           ││        │                               │           ╰─ closeTagEnd(html-comment)
- │  ││           ││        │                               ╰─ closeTagName "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
diff --git a/src/__tests__/fixtures/html-comment-tag/__snapshots__/html-comment-tag.expected.txt b/src/__tests__/fixtures/html-comment-tag/__snapshots__/html-comment-tag.expected.txt
index 16c1da4d..a657a0b0 100644
--- a/src/__tests__/fixtures/html-comment-tag/__snapshots__/html-comment-tag.expected.txt
+++ b/src/__tests__/fixtures/html-comment-tag/__snapshots__/html-comment-tag.expected.txt
@@ -7,8 +7,9 @@
  │  │ ╰─ comment "<!--123-->"
  ╰─ ╰─ text "\n  "
 3╭─   <html-comment>abc</html-comment>
- │  │ ││           ││    │           ╰─ closeTagEnd(html-comment)
- │  │ ││           ││    ╰─ closeTagName "html-comment"
+ │  │ ││           ││  │ │           ╰─ closeTagEnd(html-comment)
+ │  │ ││           ││  │ ╰─ closeTagName "html-comment"
+ │  │ ││           ││  ╰─ closeTagStart "</"
  │  │ ││           │╰─ text "abc"
  │  │ ││           ╰─ openTagEnd
  │  │ │╰─ tagName "html-comment"
@@ -19,8 +20,9 @@
  │  │ ╰─ comment "<!--[if lt IE 9]><script src=\"...\"></script><![endif]-->"
  ╰─ ╰─ text "\n  "
 5╭─   <html-comment>[if lt IE 9]><script src="..."></script><![endif]</html-comment>
- │  │ ││           ││                                                  │           ╰─ closeTagEnd(html-comment)
- │  │ ││           ││                                                  ╰─ closeTagName "html-comment"
+ │  │ ││           ││                                                │ │           ╰─ closeTagEnd(html-comment)
+ │  │ ││           ││                                                │ ╰─ closeTagName "html-comment"
+ │  │ ││           ││                                                ╰─ closeTagStart "</"
  │  │ ││           │╰─ text "[if lt IE 9]><script src=\"...\"></script><![endif]"
  │  │ ││           ╰─ openTagEnd
  │  │ │╰─ tagName "html-comment"
diff --git a/src/__tests__/fixtures/parsed-text-placeholder-in-template-literal-string/__snapshots__/parsed-text-placeholder-in-template-literal-string.expected.txt b/src/__tests__/fixtures/parsed-text-placeholder-in-template-literal-string/__snapshots__/parsed-text-placeholder-in-template-literal-string.expected.txt
index 5e34fce5..23d1d244 100644
--- a/src/__tests__/fixtures/parsed-text-placeholder-in-template-literal-string/__snapshots__/parsed-text-placeholder-in-template-literal-string.expected.txt
+++ b/src/__tests__/fixtures/parsed-text-placeholder-in-template-literal-string/__snapshots__/parsed-text-placeholder-in-template-literal-string.expected.txt
@@ -9,6 +9,7 @@
 5├─         ${bar}
 6├─     */
 7╭─ </script>
- │    │     ╰─ closeTagEnd(script)
- ╰─   ╰─ closeTagName "script"
+ │  │ │     ╰─ closeTagEnd(script)
+ │  │ ╰─ closeTagName "script"
+ ╰─ ╰─ closeTagStart "</"
 8╰─ 
\ No newline at end of file
diff --git a/src/__tests__/fixtures/parsed-text-style-tag/__snapshots__/parsed-text-style-tag.expected.txt b/src/__tests__/fixtures/parsed-text-style-tag/__snapshots__/parsed-text-style-tag.expected.txt
index c66d2d41..b6def4ab 100644
--- a/src/__tests__/fixtures/parsed-text-style-tag/__snapshots__/parsed-text-style-tag.expected.txt
+++ b/src/__tests__/fixtures/parsed-text-style-tag/__snapshots__/parsed-text-style-tag.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <style>/* block comment within <style> */"string within \"<style>\""'string within \'<style>\''</style>
- │  ││    ││                                                                                         │    ╰─ closeTagEnd(style)
- │  ││    ││                                                                                         ╰─ closeTagName "style"
+ │  ││    ││                                                                                       │ │    ╰─ closeTagEnd(style)
+ │  ││    ││                                                                                       │ ╰─ closeTagName "style"
+ │  ││    ││                                                                                       ╰─ closeTagStart "</"
  │  ││    │╰─ text "/* block comment within <style> */\"string within \\\"<style>\\\"\"'string within \\'<style>\\''"
  │  ││    ╰─ openTagEnd
  │  │╰─ tagName "style"
diff --git a/src/__tests__/fixtures/placeholder-body-multi-line-js-comment/__snapshots__/placeholder-body-multi-line-js-comment.expected.txt b/src/__tests__/fixtures/placeholder-body-multi-line-js-comment/__snapshots__/placeholder-body-multi-line-js-comment.expected.txt
index 7629995e..53bc8d3f 100644
--- a/src/__tests__/fixtures/placeholder-body-multi-line-js-comment/__snapshots__/placeholder-body-multi-line-js-comment.expected.txt
+++ b/src/__tests__/fixtures/placeholder-body-multi-line-js-comment/__snapshots__/placeholder-body-multi-line-js-comment.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <script>/* Copyright $!{date} */</script>
- │  ││     ││                         │     ╰─ closeTagEnd(script)
- │  ││     ││                         ╰─ closeTagName "script"
+ │  ││     ││                       │ │     ╰─ closeTagEnd(script)
+ │  ││     ││                       │ ╰─ closeTagName "script"
+ │  ││     ││                       ╰─ closeTagStart "</"
  │  ││     │╰─ text "/* Copyright $!{date} */"
  │  ││     ╰─ openTagEnd
  │  │╰─ tagName "script"
diff --git a/src/__tests__/fixtures/placeholder-body-single-line-js-comment/__snapshots__/placeholder-body-single-line-js-comment.expected.txt b/src/__tests__/fixtures/placeholder-body-single-line-js-comment/__snapshots__/placeholder-body-single-line-js-comment.expected.txt
index c5bcdc46..54200b6c 100644
--- a/src/__tests__/fixtures/placeholder-body-single-line-js-comment/__snapshots__/placeholder-body-single-line-js-comment.expected.txt
+++ b/src/__tests__/fixtures/placeholder-body-single-line-js-comment/__snapshots__/placeholder-body-single-line-js-comment.expected.txt
@@ -4,5 +4,6 @@
  │  │╰─ tagName "script"
  ╰─ ╰─ openTagStart
 2╭─ </script>
- │    │     ╰─ closeTagEnd(script)
- ╰─   ╰─ closeTagName "script"
\ No newline at end of file
+ │  │ │     ╰─ closeTagEnd(script)
+ │  │ ╰─ closeTagName "script"
+ ╰─ ╰─ closeTagStart "</"
\ No newline at end of file
diff --git a/src/__tests__/fixtures/placeholder-within-script-tag/__snapshots__/placeholder-within-script-tag.expected.txt b/src/__tests__/fixtures/placeholder-within-script-tag/__snapshots__/placeholder-within-script-tag.expected.txt
index c3a54558..93d2840a 100644
--- a/src/__tests__/fixtures/placeholder-within-script-tag/__snapshots__/placeholder-within-script-tag.expected.txt
+++ b/src/__tests__/fixtures/placeholder-within-script-tag/__snapshots__/placeholder-within-script-tag.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <script>Hello ${xyz}!</script>
- │  ││     ││     │ │   │  │     ╰─ closeTagEnd(script)
- │  ││     ││     │ │   │  ╰─ closeTagName "script"
+ │  ││     ││     │ │   ││ │     ╰─ closeTagEnd(script)
+ │  ││     ││     │ │   ││ ╰─ closeTagName "script"
+ │  ││     ││     │ │   │╰─ closeTagStart "</"
  │  ││     ││     │ │   ╰─ text
  │  ││     ││     │ ╰─ placeholder:escape.value "xyz"
  │  ││     ││     ╰─ placeholder:escape "${xyz}"
diff --git a/src/__tests__/fixtures/placeholder-within-script-within-string/__snapshots__/placeholder-within-script-within-string.expected.txt b/src/__tests__/fixtures/placeholder-within-script-within-string/__snapshots__/placeholder-within-script-within-string.expected.txt
index f7cb00ed..dad3a109 100644
--- a/src/__tests__/fixtures/placeholder-within-script-within-string/__snapshots__/placeholder-within-script-within-string.expected.txt
+++ b/src/__tests__/fixtures/placeholder-within-script-within-string/__snapshots__/placeholder-within-script-within-string.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <script>alert("Hello ${xyz}!")</script>
- │  ││     ││            │ │   │    │     ╰─ closeTagEnd(script)
- │  ││     ││            │ │   │    ╰─ closeTagName "script"
+ │  ││     ││            │ │   │  │ │     ╰─ closeTagEnd(script)
+ │  ││     ││            │ │   │  │ ╰─ closeTagName "script"
+ │  ││     ││            │ │   │  ╰─ closeTagStart "</"
  │  ││     ││            │ │   ╰─ text "!\")"
  │  ││     ││            │ ╰─ placeholder:escape.value "xyz"
  │  ││     ││            ╰─ placeholder:escape "${xyz}"
diff --git a/src/__tests__/fixtures/script-escaped-placeholder/__snapshots__/script-escaped-placeholder.expected.txt b/src/__tests__/fixtures/script-escaped-placeholder/__snapshots__/script-escaped-placeholder.expected.txt
index 755c373d..5767d35f 100644
--- a/src/__tests__/fixtures/script-escaped-placeholder/__snapshots__/script-escaped-placeholder.expected.txt
+++ b/src/__tests__/fixtures/script-escaped-placeholder/__snapshots__/script-escaped-placeholder.expected.txt
@@ -11,4 +11,5 @@
 4╭─ </script>
  │  │ │     ╰─ closeTagEnd(script)
  │  │ ╰─ closeTagName "script"
- ╰─ ╰─ text "\n"
\ No newline at end of file
+ │  ├─ text "\n"
+ ╰─ ╰─ closeTagStart "</"
\ No newline at end of file
diff --git a/src/__tests__/fixtures/script-single-line-comment/__snapshots__/script-single-line-comment.expected.txt b/src/__tests__/fixtures/script-single-line-comment/__snapshots__/script-single-line-comment.expected.txt
index 4fd0a5dc..d2f5368d 100644
--- a/src/__tests__/fixtures/script-single-line-comment/__snapshots__/script-single-line-comment.expected.txt
+++ b/src/__tests__/fixtures/script-single-line-comment/__snapshots__/script-single-line-comment.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <script>// this is a comment</script>
- │  ││     ││                     │     ╰─ closeTagEnd(script)
- │  ││     ││                     ╰─ closeTagName "script"
+ │  ││     ││                   │ │     ╰─ closeTagEnd(script)
+ │  ││     ││                   │ ╰─ closeTagName "script"
+ │  ││     ││                   ╰─ closeTagStart "</"
  │  ││     │╰─ text "// this is a comment"
  │  ││     ╰─ openTagEnd
  │  │╰─ tagName "script"
diff --git a/src/__tests__/fixtures/script-tag-entities/__snapshots__/script-tag-entities.expected.txt b/src/__tests__/fixtures/script-tag-entities/__snapshots__/script-tag-entities.expected.txt
index f2063498..88a9cd57 100644
--- a/src/__tests__/fixtures/script-tag-entities/__snapshots__/script-tag-entities.expected.txt
+++ b/src/__tests__/fixtures/script-tag-entities/__snapshots__/script-tag-entities.expected.txt
@@ -1,6 +1,7 @@
 1╭─ <script>console.log('<div>Hello ${input.name}</div>');</script>
- │  ││     ││                       │ │          │          │     ╰─ closeTagEnd(script)
- │  ││     ││                       │ │          │          ╰─ closeTagName "script"
+ │  ││     ││                       │ │          │        │ │     ╰─ closeTagEnd(script)
+ │  ││     ││                       │ │          │        │ ╰─ closeTagName "script"
+ │  ││     ││                       │ │          │        ╰─ closeTagStart "</"
  │  ││     ││                       │ │          ╰─ text "</div>');"
  │  ││     ││                       │ ╰─ placeholder:escape.value "input.name"
  │  ││     ││                       ╰─ placeholder:escape "${input.name}"
diff --git a/src/__tests__/fixtures/script/__snapshots__/script.expected.txt b/src/__tests__/fixtures/script/__snapshots__/script.expected.txt
index 24b8ba31..9028ac80 100644
--- a/src/__tests__/fixtures/script/__snapshots__/script.expected.txt
+++ b/src/__tests__/fixtures/script/__snapshots__/script.expected.txt
@@ -4,5 +4,6 @@
  │  │╰─ tagName "script"
  ╰─ ╰─ openTagStart
 2╭─ /* block comment within <script> */"string within \"<script>\""'string within \'<script>\''</script>
- │                                                                                               │     ╰─ closeTagEnd(script)
- ╰─                                                                                              ╰─ closeTagName "script"
\ No newline at end of file
+ │                                                                                             │ │     ╰─ closeTagEnd(script)
+ │                                                                                             │ ╰─ closeTagName "script"
+ ╰─                                                                                            ╰─ closeTagStart "</"
\ No newline at end of file
diff --git a/src/states/CLOSE_TAG.ts b/src/states/CLOSE_TAG.ts
index bc8fd01f..7919cfeb 100644
--- a/src/states/CLOSE_TAG.ts
+++ b/src/states/CLOSE_TAG.ts
@@ -63,6 +63,10 @@ export function checkForClosingTag(parser: Parser) {
 
   if (match) {
     parser.endText();
+    parser.options.onCloseTagStart?.({
+      start: curPos - 1,
+      end: curPos + 1,
+    });
 
     if (
       ensureExpectedCloseTag(parser, {