fix(jsx): Write the raw value if the value is a HtmlEscaped string (#1739)

* fix(jsx): Write the raw value if the value is a HtmlEscaped string

* chore: denoify

Author: usualoma

deno_dist/jsx/index.ts

@@ -130,10 +130,10 @@ export class JSXNode implements HtmlEscaped {
         buffer[0] += ` ${key}="`
         escapeToBuffer(v, buffer)
         buffer[0] += '"'
-      } else if (typeof v === 'number') {
-        buffer[0] += ` ${key}="${v}"`
       } else if (v === null || v === undefined) {
         // Do nothing
+      } else if (typeof v === 'number' || (v as HtmlEscaped).isEscaped) {
+        buffer[0] += ` ${key}="${v}"`
       } else if (typeof v === 'boolean' && booleanAttributes.includes(key)) {
         if (v) {
           buffer[0] += ` ${key}=""`

src/jsx/index.test.tsx

@@ -365,6 +365,14 @@ describe('render to string', () => {
       expect(template.toString()).toBe('<h1 style="color:red;font-size:small">Hello</h1>')
     })
   })
+
+  describe('HtmlEscaped in props', () => {
+    it('should not be double-escaped', () => {
+      const escapedString = html`${'<html-escaped-string>'}`
+      const template = <span data-text={escapedString}>Hello</span>
+      expect(template.toString()).toBe('<span data-text="&lt;html-escaped-string&gt;">Hello</span>')
+    })
+  })
 })
 
 describe('memo', () => {

src/jsx/index.ts

@@ -130,10 +130,10 @@ export class JSXNode implements HtmlEscaped {
         buffer[0] += ` ${key}="`
         escapeToBuffer(v, buffer)
         buffer[0] += '"'
-      } else if (typeof v === 'number') {
-        buffer[0] += ` ${key}="${v}"`
       } else if (v === null || v === undefined) {
         // Do nothing
+      } else if (typeof v === 'number' || (v as HtmlEscaped).isEscaped) {
+        buffer[0] += ` ${key}="${v}"`
       } else if (typeof v === 'boolean' && booleanAttributes.includes(key)) {
         if (v) {
           buffer[0] += ` ${key}=""`