Merge 13d07079df into abb260632f

.changeset/deep-rivers-attend.md

@@ -1,5 +0,0 @@
----
-'@hono/node-ws': patch
----
-
-Fix missing code and reason on `CloseEvent`

packages/node-ws/src/index.test.ts

@@ -170,21 +170,18 @@ describe('WebSocket helper', () => {
   })
 
   it('CloseEvent should be executed without crash', async () => {
-    const testCode = 3001
-    const testReason = 'Test!'
     app.get(
       '/',
       upgradeWebSocket(() => ({
-        onClose(event) {
+        onClose() {
-          expect(event.code).toBe(testCode)
+          // doing some stuff here
-          expect(event.reason).toBe(testReason)
         },
       }))
     )
 
     const ws = new WebSocket('ws://localhost:3030/')
     await new Promise<void>((resolve) => ws.on('open', resolve))
-    ws.close(testCode, testReason)
+    ws.close()
   })
 
   it('Should be able to send and receive binary content with good length', async () => {

packages/node-ws/src/index.ts

@@ -126,8 +126,8 @@ export const createNodeWebSocket = (init: NodeWebSocketInit): NodeWebSocket => {
               )
             }
           })
-          ws.on('close', (code, reason) => {
+          ws.on('close', () => {
-            events.onClose?.(new CloseEvent('close', { code, reason: reason.toString() }), ctx)
+            events.onClose?.(new CloseEvent('close'), ctx)
           })
           ws.on('error', (error) => {
             events.onError?.(