From 98339c85967704ca3a524252f0daba0454e251b9 Mon Sep 17 00:00:00 2001 From: Sirius Date: Mon, 23 Feb 2026 04:05:19 +0100 Subject: [PATCH] chore: remove dead eslint-disable comments from example files The weather-app example project has no lint target in NX, so the inline `eslint-disable-line no-console` comments in handlers and middleware were never evaluated and served no purpose. Console output is intentional in example code anyway. Removes 3 unnecessary suppression comments. --- examples/weather-app/handlers/app-lifecycle.ts | 2 +- examples/weather-app/middleware/logging.ts | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/examples/weather-app/handlers/app-lifecycle.ts b/examples/weather-app/handlers/app-lifecycle.ts index e51b7729..56480166 100644 --- a/examples/weather-app/handlers/app-lifecycle.ts +++ b/examples/weather-app/handlers/app-lifecycle.ts @@ -7,6 +7,6 @@ import { defineHandler, Events } from "@mcp-apps-kit/codegen"; export default defineHandler({ event: Events.APP_START, handler: async (payload) => { - console.log(`[weather-app] Server started on port ${payload.port}`); // eslint-disable-line no-console + console.log(`[weather-app] Server started on port ${payload.port}`); }, }); diff --git a/examples/weather-app/middleware/logging.ts b/examples/weather-app/middleware/logging.ts index 39969dc8..5f30f5d3 100644 --- a/examples/weather-app/middleware/logging.ts +++ b/examples/weather-app/middleware/logging.ts @@ -11,11 +11,11 @@ import { defineMiddleware } from "@mcp-apps-kit/codegen"; export default defineMiddleware({ before: async (context) => { context.state.set("startTime", Date.now()); - console.log(`[weather-app] Tool called: ${context.toolName}`); // eslint-disable-line no-console + console.log(`[weather-app] Tool called: ${context.toolName}`); }, after: async (context) => { const startTime = context.state.get("startTime") as number; const duration = Date.now() - startTime; - console.log(`[weather-app] Tool completed: ${context.toolName} (${duration}ms)`); // eslint-disable-line no-console + console.log(`[weather-app] Tool completed: ${context.toolName} (${duration}ms)`); }, });