Upgrade TypeScript tooling and fix test editor config

Author: yyyyaaa

.eslintrc.json

@@ -0,0 +1,77 @@
+const js = require('@eslint/js');
+const globals = require('globals');
+const tsParser = require('@typescript-eslint/parser');
+const tsPlugin = require('@typescript-eslint/eslint-plugin');
+const simpleImportSort = require('eslint-plugin-simple-import-sort');
+const unusedImports = require('eslint-plugin-unused-imports');
+
+const sharedRules = {
+  indent: ['error', 2],
+  quotes: ['error', 'single', { avoidEscape: true, allowTemplateLiterals: true }],
+  'quote-props': ['error', 'as-needed'],
+  semi: ['error', 'always'],
+  'simple-import-sort/imports': 1,
+  'simple-import-sort/exports': 1,
+  'unused-imports/no-unused-imports': 1,
+  'prefer-const': 0,
+  'no-case-declarations': 0,
+  'no-console': 0,
+};
+
+module.exports = [
+  {
+    ignores: ['**/dist/**', '**/node_modules/**'],
+  },
+  {
+    ...js.configs.recommended,
+    files: ['**/*.js', '**/*.cjs', '**/*.mjs'],
+    languageOptions: {
+      ecmaVersion: 'latest',
+      sourceType: 'commonjs',
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        ...globals.jest,
+      },
+    },
+    plugins: {
+      'simple-import-sort': simpleImportSort,
+      'unused-imports': unusedImports,
+    },
+    rules: sharedRules,
+  },
+  {
+    files: ['**/*.ts', '**/*.tsx'],
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+      },
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        ...globals.jest,
+      },
+    },
+    plugins: {
+      '@typescript-eslint': tsPlugin,
+      'simple-import-sort': simpleImportSort,
+      'unused-imports': unusedImports,
+    },
+    rules: {
+      ...tsPlugin.configs.recommended.rules,
+      ...sharedRules,
+      '@typescript-eslint/no-unused-vars': [
+        1,
+        { argsIgnorePattern: 'React|res|next|^_', varsIgnorePattern: '^_' },
+      ],
+      '@typescript-eslint/no-explicit-any': 0,
+      '@typescript-eslint/no-var-requires': 0,
+      '@typescript-eslint/ban-ts-comment': 0,
+      '@typescript-eslint/no-unsafe-declaration-merging': 0,
+      '@typescript-eslint/no-empty-object-type': 0,
+      'no-implicit-globals': 0,
+    },
+  },
+];

eslint.config.cjs

@@ -19,27 +19,30 @@
     "build": "pnpm -r run build",
     "build:dev": "pnpm -r run build:dev",
     "test": "pnpm -r run test",
+    "typecheck": "node ./scripts/typecheck.js",
     "test:live:ollama": "pnpm --filter @agentic-kit/ollama run test:live:smoke",
     "test:live:ollama:extended": "pnpm --filter @agentic-kit/ollama run test:live:extended",
     "lint": "pnpm -r run lint",
     "internal:deps": "makage update-workspace",
     "deps": "pnpm up -r -i -L"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.3",
     "@types/jest": "^29.5.11",
     "@types/node": "^20.12.7",
-    "@typescript-eslint/eslint-plugin": "^8.53.1",
-    "@typescript-eslint/parser": "^8.53.1",
+    "@typescript-eslint/eslint-plugin": "^8.58.2",
+    "@typescript-eslint/parser": "^8.58.2",
     "eslint": "^9.39.2",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-simple-import-sort": "^12.1.0",
     "eslint-plugin-unused-imports": "^4.0.0",
+    "globals": "^17.5.0",
     "jest": "^29.6.2",
     "lerna": "^8.2.3",
     "makage": "^0.1.12",
     "prettier": "^3.8.0",
-    "ts-jest": "^29.4.6",
+    "ts-jest": "^29.4.9",
     "ts-node": "^10.9.2",
-    "typescript": "^5.1.6"
+    "typescript": "^6.0.3"
   }
 }

package.json

@@ -1,7 +1,7 @@
 import {
-  createAssistantMessageEventStream,
   type AssistantMessage,
   type Context,
+  createAssistantMessageEventStream,
   type ModelDescriptor,
 } from 'agentic-kit';
 

packages/agent/__tests__/agent.test.ts

@@ -0,0 +1,10 @@
+{
+  "extends": "../tsconfig.json",
+  "compilerOptions": {
+    "noEmit": true,
+    "rootDir": "..",
+    "types": ["jest", "node"]
+  },
+  "include": ["./**/*.ts", "../src/**/*.ts"],
+  "exclude": ["../dist", "../node_modules"]
+}

packages/agent/__tests__/tsconfig.json

@@ -7,7 +7,7 @@ module.exports = {
       'ts-jest',
       {
         babelConfig: false,
-        tsconfig: 'tsconfig.json',
+        tsconfig: '__tests__/tsconfig.json',
       },
     ],
   },

packages/agent/jest.config.js

@@ -1,11 +1,10 @@
 import {
-  complete,
-  createToolResultMessage,
-  createUserMessage,
-  stream,
   type AssistantMessage,
   type Context,
+  createToolResultMessage,
+  createUserMessage,
   type Message,
+  stream,
   type StreamOptions,
 } from 'agentic-kit';
 
@@ -198,30 +197,30 @@ export class Agent {
 
     for await (const event of streamResult) {
       switch (event.type) {
-        case 'start':
-          this._state.streamMessage = event.partial;
-          this.emit({ type: 'message_start', message: event.partial });
-          break;
-        case 'text_start':
-        case 'text_delta':
-        case 'text_end':
-        case 'thinking_start':
-        case 'thinking_delta':
-        case 'thinking_end':
-        case 'toolcall_start':
-        case 'toolcall_delta':
-        case 'toolcall_end':
-          this._state.streamMessage = event.partial;
-          this.emit({
-            type: 'message_update',
-            message: event.partial,
-            assistantMessageEvent: event,
-          });
-          break;
-        case 'done':
-        case 'error':
-          this._state.streamMessage = null;
-          break;
+      case 'start':
+        this._state.streamMessage = event.partial;
+        this.emit({ type: 'message_start', message: event.partial });
+        break;
+      case 'text_start':
+      case 'text_delta':
+      case 'text_end':
+      case 'thinking_start':
+      case 'thinking_delta':
+      case 'thinking_end':
+      case 'toolcall_start':
+      case 'toolcall_delta':
+      case 'toolcall_end':
+        this._state.streamMessage = event.partial;
+        this.emit({
+          type: 'message_update',
+          message: event.partial,
+          assistantMessageEvent: event,
+        });
+        break;
+      case 'done':
+      case 'error':
+        this._state.streamMessage = null;
+        break;
       }
     }
 

packages/agent/src/agent.ts

@@ -95,22 +95,22 @@ function validateSchema(schema: JsonSchema, value: unknown, path: string): strin
 
 function matchesType(type: string, value: unknown): boolean {
   switch (type) {
-    case 'array':
-      return Array.isArray(value);
-    case 'boolean':
-      return typeof value === 'boolean';
-    case 'integer':
-      return typeof value === 'number' && Number.isInteger(value);
-    case 'null':
-      return value === null;
-    case 'number':
-      return typeof value === 'number';
-    case 'object':
-      return isPlainObject(value);
-    case 'string':
-      return typeof value === 'string';
-    default:
-      return true;
+  case 'array':
+    return Array.isArray(value);
+  case 'boolean':
+    return typeof value === 'boolean';
+  case 'integer':
+    return typeof value === 'number' && Number.isInteger(value);
+  case 'null':
+    return value === null;
+  case 'number':
+    return typeof value === 'number';
+  case 'object':
+    return isPlainObject(value);
+  case 'string':
+    return typeof value === 'string';
+  default:
+    return true;
   }
 }
 

packages/agent/src/validation.ts

@@ -2,9 +2,9 @@ import {
   AgentKit,
   createAssistantMessageEventStream,
   getMessageText,
-  transformMessages,
   type ModelDescriptor,
   type ProviderAdapter,
+  transformMessages,
 } from '../src';
 
 function createFakeModel(): ModelDescriptor {

packages/agentic-kit/__tests__/adapter.test.ts

@@ -0,0 +1,10 @@
+{
+  "extends": "../tsconfig.json",
+  "compilerOptions": {
+    "noEmit": true,
+    "rootDir": "..",
+    "types": ["jest", "node"]
+  },
+  "include": ["./**/*.ts", "../src/**/*.ts"],
+  "exclude": ["../dist", "../node_modules"]
+}