chore: remove no-empty-function suppressions with descriptive noop comments (#158)

Author: gabrypavanello

packages/ui/src/adapters/openai.ts

@@ -898,8 +898,9 @@ export class OpenAIAdapter implements ProtocolAdapter {
     // ChatGPT doesn't support partial/streaming tool input
     // Return a no-op unsubscribe function
     this.log("debug", "onToolInputPartial is not supported on ChatGPT");
-    // eslint-disable-next-line @typescript-eslint/no-empty-function
-    return () => {};
+    return () => {
+      /* noop — ChatGPT doesn't support partial tool input */
+    };
   }
 
   // === Bidirectional Tool Support ===

packages/ui/src/client.ts

@@ -223,8 +223,9 @@ export function createAppsClient<T extends ToolDefs = ToolDefs>(
     setupSizeChangedNotifications(): () => void {
       // Only run in browser environments
       if (typeof window === "undefined" || typeof ResizeObserver === "undefined") {
-        // eslint-disable-next-line @typescript-eslint/no-empty-function
-        return () => {};
+        return () => {
+          /* noop — no ResizeObserver in non-browser env */
+        };
       }
 
       const observer = new ResizeObserver((entries) => {