feat: add ESLint configuration (#7)

- Add eslint.config.js with TypeScript and React plugins
- Add lint script to package.json
- Re-enable lint step in CI workflow
- Fix all lint errors (unused imports, const assignments)
- 41 warnings remain (mostly @typescript-eslint/no-explicit-any)

Author: dayhaysoos

.github/workflows/test.yml

@@ -35,3 +35,4 @@ jobs:
         run: |
           npm run test
           npm run typecheck
+          npm run lint

convex/component/chat.test.ts

@@ -1,5 +1,5 @@
 /// <reference types="vite/client" />
-import { describe, it, expect, vi } from "vitest";
+import { describe, it, expect } from "vitest";
 import { convexTest } from "convex-test";
 import schema from "./schema";
 import { api } from "./_generated/api";

convex/component/chat.ts

@@ -8,7 +8,7 @@ import {
   validateToolArgs,
 } from "./tools";
 import type { DatabaseChatTool } from "./tools";
-import { DeltaStreamer, type StreamPart } from "./deltaStreamer";
+import { DeltaStreamer } from "./deltaStreamer";
 
 /**
  * Send a message and get a streaming response.

convex/component/conversations.test.ts

@@ -1,5 +1,5 @@
 /// <reference types="vite/client" />
-import { describe, it, expect, beforeEach } from "vitest";
+import { describe, it, expect } from "vitest";
 import { convexTest } from "convex-test";
 import schema from "./schema";
 import { api } from "./_generated/api";

convex/component/schemaTools.test.ts

@@ -22,7 +22,7 @@ describe("schemaTools", () => {
     searchIndexes: [],
   };
 
-  const handlers: SchemaToolHandlers = {
+  const _handlers: SchemaToolHandlers = {
     query: "query_handler",
     count: "count_handler",
     aggregate: "aggregate_handler",

convex/component/schemaTools.ts

@@ -21,11 +21,7 @@
  * ```
  */
 
-import type {
-  DatabaseChatTool,
-  ToolParameterSchema,
-  AutoToolsConfig,
-} from "./tools";
+import type { DatabaseChatTool, AutoToolsConfig } from "./tools";
 
 // =============================================================================
 // Types
@@ -89,7 +85,7 @@ export interface GenerateToolsOptions extends AutoToolsConfig {
 /**
  * Map Convex validator types to JSON Schema types.
  */
-function convexTypeToJsonType(
+function _convexTypeToJsonType(
   convexType: string
 ): "string" | "number" | "boolean" | "array" | "object" {
   const typeMap: Record<string, "string" | "number" | "boolean" | "array" | "object"> = {
@@ -337,7 +333,7 @@ export function generateToolsFromSchema(
   const {
     tables,
     allowedTables,
-    excludeFields,
+    excludeFields: _excludeFields,
     tableDescriptions,
     handlers,
   } = options;

convex/component/stream.test.ts

@@ -1,5 +1,5 @@
 /// <reference types="vite/client" />
-import { describe, it, expect, beforeEach, vi } from "vitest";
+import { describe, it, expect } from "vitest";
 import { convexTest } from "convex-test";
 import schema from "./schema";
 import { api } from "./_generated/api";

eslint.config.js

@@ -0,0 +1,54 @@
+import eslint from "@eslint/js";
+import tseslint from "typescript-eslint";
+import reactPlugin from "eslint-plugin-react";
+import reactHooksPlugin from "eslint-plugin-react-hooks";
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  ...tseslint.configs.recommended,
+  {
+    ignores: [
+      "dist/**",
+      "node_modules/**",
+      "**/convex/_generated/**",
+      "**/_generated/**",
+      "example/dist/**",
+    ],
+  },
+  {
+    files: ["**/*.ts", "**/*.tsx"],
+    plugins: {
+      react: reactPlugin,
+      "react-hooks": reactHooksPlugin,
+    },
+    languageOptions: {
+      parserOptions: {
+        ecmaFeatures: {
+          jsx: true,
+        },
+      },
+    },
+    rules: {
+      // TypeScript rules
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        { argsIgnorePattern: "^_", varsIgnorePattern: "^_" },
+      ],
+      "@typescript-eslint/no-explicit-any": "warn",
+      "@typescript-eslint/no-require-imports": "off",
+
+      // React rules
+      "react/react-in-jsx-scope": "off",
+      "react-hooks/rules-of-hooks": "error",
+      "react-hooks/exhaustive-deps": "warn",
+
+      // General rules
+      "no-console": ["warn", { allow: ["warn", "error"] }],
+    },
+    settings: {
+      react: {
+        version: "detect",
+      },
+    },
+  }
+);

example/convex/chat.ts

@@ -426,7 +426,7 @@ async function callLLMWithTools(
     },
   }));
 
-  let currentMessages = [...messages];
+  const currentMessages = [...messages];
   let loopCount = 0;
   const MAX_LOOPS = 5;
 

example/src/components/Chat.tsx

@@ -1,4 +1,4 @@
-import { useState, useEffect, useRef, useMemo, FormEvent } from "react";
+import { useState, useEffect, useRef, FormEvent } from "react";
 import { useQuery, useMutation, useAction } from "convex/react";
 import { api } from "../../convex/_generated/api";
 import { useRateLimit } from "../hooks/useRateLimit";