fix: fix some checks

Author: tsck

eslint.config.js

@@ -9,7 +9,7 @@ import enforceZodV4 from "./eslint-rules/enforce-zod-v4.js";
 
 const testFiles = ["tests/**/*.test.ts", "tests/**/*.ts"];
 
-const files = [...testFiles, "src/**/*.ts", "scripts/**/*.ts"];
+const files = [...testFiles, "src/**/*.ts", "src/**/*.tsx", "scripts/**/*.ts"];
 
 export default defineConfig([
     { files, plugins: { js }, extends: ["js/recommended"] },
@@ -85,6 +85,7 @@ export default defineConfig([
         "global.d.ts",
         "eslint.config.js",
         "vitest.config.ts",
+        "vite.ui.config.ts",
         "src/types/*.d.ts",
         "tests/integration/fixtures/",
         "eslint-rules",

knip.json

@@ -3,11 +3,26 @@
     "src/index.ts!",
     "src/lib.ts!",
     "src/tools/index.ts!",
+    "src/ui/index.ts!",
     "tests/**/*.ts",
     "scripts/**/*.ts",
-    "eslint-rules/*.js"
+    "eslint-rules/*.js",
+    "vite.ui.config.ts"
+  ],
+  "ignore": [
+    "tests/integration/fixtures/curl.mjs",
+    "tests/vitest.d.ts",
+    "src/ui/build/mount.tsx",
+    "src/ui/components/**/*.ts",
+    "src/ui/components/**/*.tsx",
+    "src/ui/hooks/**/*.ts"
+  ],
+  "ignoreDependencies": [
+    "@mongodb-js/atlas-local",
+    "@emotion/css",
+    "@leafygreen-ui/table",
+    "react",
+    "react-dom"
   ],
-  "ignore": ["tests/integration/fixtures/curl.mjs", "tests/vitest.d.ts"],
-  "ignoreDependencies": ["@mongodb-js/atlas-local"],
   "ignoreExportsUsedInFile": true
 }

src/ui/build/mount.tsx

@@ -33,7 +33,7 @@ for (const [path, module] of Object.entries(componentModules)) {
     }
 }
 
-function mount() {
+function mount(): void {
     const container = document.getElementById("root");
     if (!container) {
         console.error("[mount] No #root element found");
@@ -62,4 +62,3 @@ function mount() {
 }
 
 mount();
-

src/ui/components/ListDatabases/ListDatabases.tsx

@@ -29,7 +29,7 @@ function formatBytes(bytes: number): string {
     return Math.round((bytes / Math.pow(k, i)) * 100) / 100 + " " + sizes[i];
 }
 
-export const ListDatabases = () => {
+export const ListDatabases = (): React.ReactElement | null => {
     const { data, isLoading, error } = useRenderData<ListDatabasesOutput>();
 
     if (isLoading) {

src/ui/hooks/useRenderData.ts

@@ -1,5 +1,20 @@
 import { useEffect, useState } from "react";
 
+/** Expected structure of the postMessage data from parent window */
+interface RenderDataMessage {
+    type: string;
+    payload?: {
+        renderData?: unknown;
+    };
+}
+
+/** Return type for the useRenderData hook */
+interface UseRenderDataResult<T> {
+    data: T | null;
+    isLoading: boolean;
+    error: string | null;
+}
+
 /**
  * Hook for receiving render data from parent window via postMessage
  * This is used by iframe-based UI components that receive data from an MCP client
@@ -22,13 +37,13 @@ import { useEffect, useState } from "react";
  * }
  * ```
  */
-export function useRenderData<T = unknown>() {
+export function useRenderData<T = unknown>(): UseRenderDataResult<T> {
     const [data, setData] = useState<T | null>(null);
     const [isLoading, setIsLoading] = useState(true);
     const [error, setError] = useState<string | null>(null);
 
     useEffect(() => {
-        const handleMessage = (event: MessageEvent) => {
+        const handleMessage = (event: MessageEvent<RenderDataMessage>): void => {
             if (event.data?.type !== "ui-lifecycle-iframe-render-data") {
                 // Silently ignore messages that aren't for us
                 return;
@@ -64,7 +79,7 @@ export function useRenderData<T = unknown>() {
         window.addEventListener("message", handleMessage);
         window.parent.postMessage({ type: "ui-lifecycle-iframe-ready" }, "*");
 
-        return () => {
+        return (): void => {
             window.removeEventListener("message", handleMessage);
         };
     }, []);

src/ui/registry/registry.ts

@@ -10,7 +10,6 @@ function getCurrentDir(): string {
     if (typeof __dirname !== "undefined") {
         return __dirname;
     }
-    // @ts-ignore - import.meta is only available in ESM, but this code path is only executed in ESM
     return dirname(fileURLToPath(import.meta.url));
 }
 

tsconfig.json

@@ -4,5 +4,6 @@
     "rootDir": ".",
     "skipLibCheck": true
   },
-  "include": ["**/*"]
+  "include": ["**/*"],
+  "exclude": ["dist", "node_modules"]
 }