Update eslint

.github/workflows/ci.yml

@@ -62,12 +62,17 @@ jobs:
       with:
         fetch-depth: 0
 
+    - uses: pnpm/action-setup@v4
+      name: Install pnpm
+      with:
+        run_install: false
+
     - name: Set up Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v4
+      uses: actions/setup-node@v6
       with:
         node-version: ${{ matrix.node-version }}
-        cache: 'npm'
-        cache-dependency-path: '**/package-lock.json'
+        cache: 'pnpm'
+        cache-dependency-path: '**/pnpm-lock.yaml'
 
     - name: Setup pnpm
       uses: pnpm/action-setup@v3

eslint.config.mjs

@@ -0,0 +1,25 @@
+import eslint from '@eslint/js';
+import { defineConfig } from 'eslint/config';
+import tseslint from 'typescript-eslint';
+
+export default defineConfig(
+  eslint.configs.recommended,
+  tseslint.configs.recommended,
+  {
+    "rules": {
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          "args": "all",
+          "argsIgnorePattern": "^_",
+          "caughtErrors": "all",
+          "caughtErrorsIgnorePattern": "^_",
+          "destructuredArrayIgnorePattern": "^_",
+          "varsIgnorePattern": "^_",
+          "ignoreRestSiblings": true
+        }
+      ]
+    }
+  }
+);
+

package.json

@@ -44,6 +44,16 @@
   },
   "packageManager": "pnpm@10.20.0",
   "devDependencies": {
-    "nerdbank-gitversioning": "^3.8.118"
+    "@eslint/js": "^9.39.1",
+    "@typescript-eslint/eslint-plugin": "^8.47.0",
+    "@typescript-eslint/parser": "^8.47.0",
+    "eslint": "^9.39.1",
+    "nerdbank-gitversioning": "^3.9.50",
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.47.0"
+  },
+  "dependencies": {
+    "globals": "^16.5.0",
+    "jiti": "^2.6.1"
   }
 }

packages/agents-a365-notifications/package.json

@@ -38,15 +38,17 @@
     "@microsoft/agents-hosting": "*"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.1",
     "@types/jest": "^29.5.14",
     "@types/node": "^20.0.0",
-    "@typescript-eslint/eslint-plugin": "^6.0.0",
-    "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.47.0",
+    "@typescript-eslint/parser": "^8.47.0",
+    "eslint": "^9.39.1",
     "jest": "^29.0.0",
     "rimraf": "^5.0.0",
     "ts-jest": "^29.0.0",
-    "typescript": "^5.0.0"
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.47.0"
   },
   "engines": {
     "node": ">=18.0.0"

packages/agents-a365-notifications/src/agent-notification.ts

@@ -46,6 +46,7 @@ function addAgenticRoute(
 function onAgentNotificationInternal(
   app: AgentApplication<TurnState>,
   channelId: string,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   handler: any,
   rank = 32767,
   autoSignInHandlers?: string[]) {
@@ -86,6 +87,7 @@ function onAgentNotificationInternal(
 function onLifecycleNotificationInternal(
   app: AgentApplication<TurnState>,
   lifecycleEvent: string,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   handler: any,
   rank = 32767,
   autoSignInHandlers?: string[]) {

packages/agents-a365-observability-extensions-openai/package.json

@@ -51,15 +51,17 @@
     "@opentelemetry/instrumentation": "*"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.1",
     "@types/jest": "^29.5.12",
     "@types/node": "^20.0.0",
-    "@typescript-eslint/eslint-plugin": "^6.0.0",
-    "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.47.0",
+    "@typescript-eslint/parser": "^8.47.0",
+    "eslint": "^9.39.1",
     "jest": "^29.7.0",
     "rimraf": "^6.0.0",
     "ts-jest": "^29.2.0",
-    "typescript": "^5.0.0"
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.47.0"
   },
   "peerDependencies": {
     "@openai/agents": "^0.1.5"

packages/agents-a365-observability-extensions-openai/src/OpenAIAgentsTraceInstrumentor.ts

@@ -98,9 +98,8 @@ export class OpenAIAgentsTraceInstrumentor extends InstrumentationBase<OpenAIAge
     // Register the processor directly using the imported setTraceProcessors function
     // This bypasses the OpenTelemetry instrumentation patching mechanism
     try {
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       setTraceProcessors([this.processor as TracingProcessor]);
-    } catch (error) {
+    } catch (_error) {
       // Silent failure - processor registration failed
     }
 
@@ -123,7 +122,7 @@ export class OpenAIAgentsTraceInstrumentor extends InstrumentationBase<OpenAIAge
     // Reset trace processors using direct import
     try {
       setTraceProcessors([]);
-    } catch (error) {
+    } catch (_error) {
       // Silent failure - processor cleanup failed
     }
 

packages/agents-a365-observability-extensions-openai/src/OpenAIAgentsTraceProcessor.ts

@@ -338,7 +338,7 @@ export class OpenAIAgentsTraceProcessor implements TracingProcessor {
   /**
    * Process handoff span data
    */
-  private processHandoffSpanData(otelSpan: OtelSpan, data: SpanData, traceId: string): void {
+  private processHandoffSpanData(_otelSpan: OtelSpan, data: SpanData, traceId: string): void {
     const handoffData = data as Record<string, unknown>;
     if (handoffData.to_agent && handoffData.from_agent) {
       const key = `${handoffData.to_agent}:${traceId}`;

packages/agents-a365-observability-tokencache/package.json

@@ -42,20 +42,22 @@
     "pack": "npm pack --pack-destination=../"
   },
   "dependencies": {
-    "@microsoft/agents-hosting": "^1.1.0-alpha.85",
+    "@microsoft/agents-a365-observability": "workspace:*",
     "@microsoft/agents-a365-runtime": "workspace:*",
-    "@microsoft/agents-a365-observability": "workspace:*"
+    "@microsoft/agents-hosting": "^1.1.0-alpha.85"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.1",
     "@types/jest": "^29.5.12",
     "@types/node": "^20.0.0",
-    "@typescript-eslint/eslint-plugin": "^6.0.0",
-    "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.47.0",
+    "@typescript-eslint/parser": "^8.47.0",
+    "eslint": "^9.39.1",
     "jest": "^29.7.0",
     "rimraf": "^6.0.0",
     "ts-jest": "^29.2.0",
-    "typescript": "^5.0.0"
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.47.0"
   },
   "engines": {
     "node": ">=18.0.0"

packages/agents-a365-observability/package.json

@@ -48,15 +48,17 @@
     "@opentelemetry/semantic-conventions": "*"
   },
   "devDependencies": {
+    "@eslint/js": "^9.39.1",
     "@types/jest": "^29.5.14",
     "@types/node": "^20.0.0",
-    "@typescript-eslint/eslint-plugin": "^6.0.0",
-    "@typescript-eslint/parser": "^6.0.0",
-    "eslint": "^8.0.0",
+    "@typescript-eslint/eslint-plugin": "^8.47.0",
+    "@typescript-eslint/parser": "^8.47.0",
+    "eslint": "^9.39.1",
     "jest": "^29.0.0",
     "rimraf": "^5.0.0",
     "ts-jest": "^29.0.0",
-    "typescript": "^5.0.0"
+    "typescript": "^5.9.3",
+    "typescript-eslint": "^8.47.0"
   },
   "engines": {
     "node": ">=18.0.0"

packages/agents-a365-observability/src/ObservabilityBuilder.ts

@@ -136,6 +136,7 @@ export class ObservabilityBuilder {
     // 2. batch processor that actually ships spans out
     const batchProcessor = this.createBatchProcessor();
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     const globalProvider: any = trace.getTracerProvider();
 
     const canAddProcessors =
@@ -194,9 +195,11 @@ export class ObservabilityBuilder {
   /**
    * Helper to avoid double-registering same processor type.
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   private attachProcessorIfMissing(provider: any, processor: any) {
     const active = provider._activeSpanProcessor?._spanProcessors;
     const alreadyAdded = Array.isArray(active) &&
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       active.some((p: any) => p?.constructor?.name === processor.constructor.name);
 
     if (!alreadyAdded) {

packages/agents-a365-observability/src/tracing/exporter/Agent365Exporter.ts

@@ -127,7 +127,7 @@ export class Agent365Exporter implements SpanExporter {
         code: anyFailure ? ExportResultCode.FAILED : ExportResultCode.SUCCESS
       });
 
-    } catch (error) {
+    } catch (_error) {
       // Exporters should not raise; signal failure
       resultCallback({ code: ExportResultCode.FAILED });
     }

packages/agents-a365-observability/src/tracing/middleware/BaggageBuilder.ts

@@ -224,13 +224,16 @@ export class BaggageBuilder {
    * @param pairs Dictionary or iterable of key-value pairs
    * @returns Self for method chaining
    */
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   setPairs(pairs: Record<string, any> | Iterable<[string, any]> | null | undefined): BaggageBuilder {
     if (!pairs) {
       return this;
     }
 
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     let entries: Iterable<[string, any]>;
     if (Symbol.iterator in Object(pairs)) {
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
       entries = pairs as Iterable<[string, any]>;
     } else {
       entries = Object.entries(pairs);