refactor(jsx): Remove unused HONO_COMPONENT feature (#2139)

* refactor(jsx): Remove unused HONO_COMPONENT feature * chore: denoify
honojs · Feb 3, 2024 · 88f2983 · 88f2983
1 parent 6adc806
commit 88f2983
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 66 deletions.
diff --git a/deno_dist/jsx/index.ts b/deno_dist/jsx/index.ts
@@ -25,9 +25,6 @@ export type { RefObject } from './hooks/index.ts'
 export { createContext, useContext } from './context.ts'
 export type { Context } from './context.ts'
 
-export const HONO_COMPONENT = 'hono-component'
-export const HONO_COMPONENT_ID = 'hono-component-id'
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export type Props = Record<string, any>
 
@@ -205,19 +202,6 @@ export class JSXNode implements HtmlEscaped {
 }
 
 class JSXFunctionNode extends JSXNode {
-  constructor(tag: Function, props: Props, children: Child[]) {
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((tag as any)[HONO_COMPONENT_ID]) {
-      props = Object.keys(props).reduce((acc, key) => {
-        if (/^on[A-Z]/.test(key)) {
-          acc[key] = props[key]
-        }
-        return acc
-      }, {} as Props)
-    }
-    super(tag, props, children)
-  }
-
   toStringToBuffer(buffer: StringBuffer): void {
     const { children } = this
 
@@ -226,18 +210,6 @@ class JSXFunctionNode extends JSXNode {
       children: children.length <= 1 ? children[0] : children,
     })
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((this.tag as any)[HONO_COMPONENT_ID]) {
-      const attr = JSON.stringify({
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        id: (this.tag as any)[HONO_COMPONENT_ID],
-        props: this.props,
-      })
-      const tmpBuf = ['']
-      escapeToBuffer(attr, tmpBuf)
-      buffer[0] += `<${HONO_COMPONENT} data-hono="${tmpBuf[0]}">`
-    }
-
     if (res instanceof Promise) {
       if (globalContexts.length === 0) {
         buffer.unshift('', res)
@@ -261,11 +233,6 @@ class JSXFunctionNode extends JSXNode {
     } else {
       escapeToBuffer(res, buffer)
     }
-
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((this.tag as any)[HONO_COMPONENT_ID]) {
-      buffer[0] += `</${HONO_COMPONENT}>`
-    }
   }
 }
 

diff --git a/src/jsx/index.ts b/src/jsx/index.ts
@@ -25,9 +25,6 @@ export type { RefObject } from './hooks'
 export { createContext, useContext } from './context'
 export type { Context } from './context'
 
-export const HONO_COMPONENT = 'hono-component'
-export const HONO_COMPONENT_ID = 'hono-component-id'
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export type Props = Record<string, any>
 
@@ -205,19 +202,6 @@ export class JSXNode implements HtmlEscaped {
 }
 
 class JSXFunctionNode extends JSXNode {
-  constructor(tag: Function, props: Props, children: Child[]) {
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((tag as any)[HONO_COMPONENT_ID]) {
-      props = Object.keys(props).reduce((acc, key) => {
-        if (/^on[A-Z]/.test(key)) {
-          acc[key] = props[key]
-        }
-        return acc
-      }, {} as Props)
-    }
-    super(tag, props, children)
-  }
-
   toStringToBuffer(buffer: StringBuffer): void {
     const { children } = this
 
@@ -226,18 +210,6 @@ class JSXFunctionNode extends JSXNode {
       children: children.length <= 1 ? children[0] : children,
     })
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((this.tag as any)[HONO_COMPONENT_ID]) {
-      const attr = JSON.stringify({
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        id: (this.tag as any)[HONO_COMPONENT_ID],
-        props: this.props,
-      })
-      const tmpBuf = ['']
-      escapeToBuffer(attr, tmpBuf)
-      buffer[0] += `<${HONO_COMPONENT} data-hono="${tmpBuf[0]}">`
-    }
-
     if (res instanceof Promise) {
       if (globalContexts.length === 0) {
         buffer.unshift('', res)
@@ -261,11 +233,6 @@ class JSXFunctionNode extends JSXNode {
     } else {
       escapeToBuffer(res, buffer)
     }
-
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if ((this.tag as any)[HONO_COMPONENT_ID]) {
-      buffer[0] += `</${HONO_COMPONENT}>`
-    }
   }
 }