Merge pull request #14 from Countly/ar2rsawseen/main

eslint for tests

Author: ar2rsawseen

.github/workflows/test.yml

@@ -29,6 +29,7 @@ jobs:
 
       - name: Run TypeScript compiler check
         run: npm run build
+        
   test:
     runs-on: ubuntu-latest
     steps:
@@ -67,7 +68,6 @@ jobs:
 
   test-build:
     runs-on: ubuntu-latest
-    needs: test
 
     steps:
       - name: Checkout code

eslint.config.js

@@ -10,14 +10,11 @@ export default [
       'coverage/**',
       '**/*.js',
       '**/*.d.ts',
-      'dist/**',
-      'tests/**',
-      '*.config.ts',
-      '*.config.js'
+      'dist/**'
     ]
   },
   {
-    files: ['src/**/*.ts'],
+    files: ['**/*.ts'],
     languageOptions: {
       parser: tsparser,
       parserOptions: {

package.json

@@ -5,7 +5,7 @@
   "main": "build/index.js",
   "type": "module",
   "scripts": {
-    "build": "tsc",
+    "build": "tsc -p tsconfig.build.json",
     "dev": "tsc --watch",
     "start": "node build/index.js --http",
     "start:stdio": "node build/index.js",

tests/tools-config.test.ts

@@ -58,8 +58,8 @@ describe('Tools Configuration', () => {
     it('should have valid CRUD operations for all tools', () => {
       const validOperations = ['C', 'R', 'U', 'D'];
 
-      for (const [category, config] of Object.entries(TOOL_CATEGORIES)) {
-        for (const [tool, operation] of Object.entries(config.operations)) {
+      for (const [_category, config] of Object.entries(TOOL_CATEGORIES)) {
+        for (const [_tool, operation] of Object.entries(config.operations)) {
           expect(validOperations).toContain(operation);
         }
       }

tests/tools.test.ts

@@ -1,5 +1,4 @@
 import { describe, it, expect, beforeEach } from 'vitest';
-import { ErrorCode } from '@modelcontextprotocol/sdk/types.js';
 import {
   normalizeServerUrl,
   parseTimeout,

tsconfig.build.json

@@ -0,0 +1,18 @@
+{
+    "extends": "./tsconfig.json",
+    "compilerOptions": {
+        "rootDir": "./src",
+        "outDir": "./build"
+    },
+    "include": [
+        "src/**/*"
+    ],
+    "exclude": [
+        "node_modules",
+        "build",
+        "tests",
+        "examples",
+        "**/*.test.ts",
+        "**/*.spec.ts"
+    ]
+}

tsconfig.json

@@ -3,9 +3,10 @@
     "target": "ES2022",
     "module": "ESNext",
     "moduleResolution": "node",
-    "lib": ["ES2022"],
+    "lib": [
+      "ES2022"
+    ],
     "outDir": "./build",
-    "rootDir": "./src",
     "strict": true,
     "esModuleInterop": true,
     "skipLibCheck": true,
@@ -17,12 +18,13 @@
     "allowSyntheticDefaultImports": true
   },
   "include": [
-    "src/**/*"
+    "src/**/*",
+    "tests/**/*",
+    "*.config.ts"
   ],
   "exclude": [
     "node_modules",
     "build",
-    "tests",
     "examples"
   ]
-}
+}