From ff91b26d7cbd1628ff41537afe50f5d70eea902c Mon Sep 17 00:00:00 2001
From: Sirius <sirius@clawd.bot>
Date: Sun, 22 Feb 2026 04:04:25 +0100
Subject: [PATCH] kaizen: remove redundant eslint-disable comments in
 defineMiddleware
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The @typescript-eslint/no-confusing-void-expression rule is globally set
to 'off' in eslint.config.js (line 102). Three inline disable comments
in defineMiddleware.ts were therefore redundant dead weight.

No behaviour change — comment-only cleanup.
---
 packages/core/src/middleware/defineMiddleware.ts | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/packages/core/src/middleware/defineMiddleware.ts b/packages/core/src/middleware/defineMiddleware.ts
index c4ab3988..eac7cdc6 100644
--- a/packages/core/src/middleware/defineMiddleware.ts
+++ b/packages/core/src/middleware/defineMiddleware.ts
@@ -192,7 +192,6 @@ export function defineMiddleware(definition: MiddlewareDefinition): Middleware {
     }
 
     // Call next and capture result (guaranteed!)
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression
     const result = await next();
 
     // Run after hook
@@ -220,7 +219,6 @@ export function defineMiddleware(definition: MiddlewareDefinition): Middleware {
 defineMiddleware.before = (hook: BeforeHook): Middleware => {
   return (async (context, next) => {
     await hook(context);
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression
     return await next();
   }) as Middleware;
 };
@@ -239,7 +237,6 @@ defineMiddleware.before = (hook: BeforeHook): Middleware => {
  */
 defineMiddleware.after = (hook: AfterHook): Middleware => {
   return (async (context, next) => {
-    // eslint-disable-next-line @typescript-eslint/no-confusing-void-expression
     const result = await next();
     await hook(context);
     return result;