From c789aa16d1322f03fe7d9cc7ea592cb7f135e04d Mon Sep 17 00:00:00 2001 From: Shoubhit Dash Date: Sun, 4 Oct 2026 17:07:07 +0530 Subject: [PATCH] refactor(ai): remove unused test code (#53126) --- packages/ai/test/effort-updates.test.ts | 3 +-- packages/ai/test/lib/tool-runtime.ts | 1 - packages/ai/test/provider-error.test.ts | 4 +--- packages/ai/test/provider/bedrock-converse.test.ts | 1 - packages/ai/test/provider/cloudflare.test.ts | 2 +- packages/ai/test/provider/evaluation.recorded.test.ts | 8 +------- .../test/provider/openai-chat-reasoning.recorded.test.ts | 1 - packages/ai/test/provider/openai-compatible-chat.test.ts | 2 +- packages/ai/test/provider/openai-responses.test.ts | 1 - packages/ai/test/recorded-scenarios.ts | 5 ----- 10 files changed, 5 insertions(+), 23 deletions(-) diff --git a/packages/ai/test/effort-updates.test.ts b/packages/ai/test/effort-updates.test.ts index 70745bff8b9..3d6f5ec7090 100644 --- a/packages/ai/test/effort-updates.test.ts +++ b/packages/ai/test/effort-updates.test.ts @@ -1,13 +1,12 @@ import { describe, expect, test } from "bun:test" import { Effect } from "effect" -import { LLM, LLMRequest, Message, ToolCallPart } from "../src/index.js" +import { LLM, Message, ToolCallPart } from "../src/index.js" import { Auth, LLMClient } from "../src/route.js" import { compileRequest } from "../src/route/client.js" import { AnthropicMessages } from "../src/protocols/anthropic-messages.js" import { OpenAIResponses } from "../src/protocols/openai-responses.js" import { Gemini } from "../src/protocols/gemini.js" import { GoogleVertexMessages, OpenAI } from "../src/providers.js" -import { applyCachePolicy } from "../src/cache-policy.js" import { applyEffortUpdates } from "../src/effort-updates.js" import { it, testEffect } from "./lib/effect.js" import { dynamicResponse } from "./lib/http.js" diff --git a/packages/ai/test/lib/tool-runtime.ts b/packages/ai/test/lib/tool-runtime.ts index 629720cd191..ff82c8dfe9c 100644 --- a/packages/ai/test/lib/tool-runtime.ts +++ b/packages/ai/test/lib/tool-runtime.ts @@ -8,7 +8,6 @@ import { type ProviderMetadata, type ToolCallPart, ToolResultPart, - type ToolResultValue, type Usage, } from "../../src/schema/index.js" import { type Tools, toDefinitions } from "../../src/tool.js" diff --git a/packages/ai/test/provider-error.test.ts b/packages/ai/test/provider-error.test.ts index 26ceaf8fdec..c325698cd59 100644 --- a/packages/ai/test/provider-error.test.ts +++ b/packages/ai/test/provider-error.test.ts @@ -39,9 +39,7 @@ describe("provider error classification", () => { ] expect(failures).toEqual( - failures.map((failure) => - expect.objectContaining({ _tag: "InvalidRequest", classification: "payload-too-large" }), - ), + failures.map(() => expect.objectContaining({ _tag: "InvalidRequest", classification: "payload-too-large" })), ) }) diff --git a/packages/ai/test/provider/bedrock-converse.test.ts b/packages/ai/test/provider/bedrock-converse.test.ts index 9a6471fa378..e250bfa8e1e 100644 --- a/packages/ai/test/provider/bedrock-converse.test.ts +++ b/packages/ai/test/provider/bedrock-converse.test.ts @@ -21,7 +21,6 @@ import { import { LLMClient } from "../../src/route.js" import { compileRequest } from "../../src/route/client.js" import { AmazonBedrock } from "../../src/providers.js" -import * as BedrockConverse from "../../src/protocols/bedrock-converse.js" import { it } from "../lib/effect.js" import { withProcessEnv } from "../lib/env.js" import { dynamicResponse, fixedResponse } from "../lib/http.js" diff --git a/packages/ai/test/provider/cloudflare.test.ts b/packages/ai/test/provider/cloudflare.test.ts index 6f77d902694..de5e574714d 100644 --- a/packages/ai/test/provider/cloudflare.test.ts +++ b/packages/ai/test/provider/cloudflare.test.ts @@ -1,4 +1,4 @@ -import { describe, expect, test } from "bun:test" +import { describe, expect } from "bun:test" import { ConfigProvider, Effect, Schema } from "effect" import { HttpClientRequest } from "effect/unstable/http" import { LLM, LLMEvent } from "../../src/index.js" diff --git a/packages/ai/test/provider/evaluation.recorded.test.ts b/packages/ai/test/provider/evaluation.recorded.test.ts index fbf3e25a9c6..971d328abda 100644 --- a/packages/ai/test/provider/evaluation.recorded.test.ts +++ b/packages/ai/test/provider/evaluation.recorded.test.ts @@ -52,7 +52,6 @@ describe("experimental Evaluation recorded", () => { typesafe.effect("evaluates choice score and boolean questions", () => assertEvaluation( TypeSafeAI.configure({ apiKey: process.env.TYPESAFE_API_KEY ?? "fixture" }).experimental.evaluation("jev-latest"), - "typesafe", ), ) @@ -61,7 +60,6 @@ describe("experimental Evaluation recorded", () => { OpenCodeZen.configure({ apiKey: process.env.OPENCODE_API_KEY ?? "fixture" }).experimental.evaluation( "jev-1.13-free", ), - "opencode", ), ) @@ -70,15 +68,11 @@ describe("experimental Evaluation recorded", () => { OpenRouter.configure({ apiKey: process.env.OPENROUTER_API_KEY ?? "fixture" }).experimental.evaluation( "typesafe/jev-1.13", ), - "openrouter", ), ) }) -const assertEvaluation = ( - model: EvaluationModel, - metadataKey: "typesafe" | "opencode" | "openrouter", -) => +const assertEvaluation = (model: EvaluationModel) => Effect.gen(function* () { const response = yield* Evaluation.run({ model, state, questions }) expect(response.model).toContain("jev-") diff --git a/packages/ai/test/provider/openai-chat-reasoning.recorded.test.ts b/packages/ai/test/provider/openai-chat-reasoning.recorded.test.ts index 89429916ce6..9809fb6aa65 100644 --- a/packages/ai/test/provider/openai-chat-reasoning.recorded.test.ts +++ b/packages/ai/test/provider/openai-chat-reasoning.recorded.test.ts @@ -1,7 +1,6 @@ import { describe, expect } from "bun:test" import { Effect } from "effect" import { LLM, LLMEvent, LLMResponse, LanguageModel } from "../../src/index.js" -import { OpenAIChat } from "../../src/protocols/openai-chat.js" import * as OpenAICompatible from "../../src/providers/openai-compatible.js" import * as OpenRouter from "../../src/providers/openrouter.js" import { LLMClient } from "../../src/route.js" diff --git a/packages/ai/test/provider/openai-compatible-chat.test.ts b/packages/ai/test/provider/openai-compatible-chat.test.ts index 32cabda1f7c..c7c9405f143 100644 --- a/packages/ai/test/provider/openai-compatible-chat.test.ts +++ b/packages/ai/test/provider/openai-compatible-chat.test.ts @@ -1,4 +1,4 @@ -import { describe, expect, test } from "bun:test" +import { describe, expect } from "bun:test" import { Effect, Schema } from "effect" import { HttpClientRequest } from "effect/unstable/http" import { LLM, LLMRequest, Message, ToolCallPart, ToolChoice, ToolDefinition } from "../../src/index.js" diff --git a/packages/ai/test/provider/openai-responses.test.ts b/packages/ai/test/provider/openai-responses.test.ts index 4e3cf3b91cb..1a9baa29d5c 100644 --- a/packages/ai/test/provider/openai-responses.test.ts +++ b/packages/ai/test/provider/openai-responses.test.ts @@ -3895,7 +3895,6 @@ describe("OpenAI Responses route", () => { it.effect("preserves foreign hosted images as portable image content when storage is enabled", () => Effect.gen(function* () { - const item = { type: "image_generation_call", id: "ig_1", status: "completed", result: "AQID" } const prepared = yield* compileRequest( LLM.request({ model: xaiModel, diff --git a/packages/ai/test/recorded-scenarios.ts b/packages/ai/test/recorded-scenarios.ts index 0722c93e102..740d37607bb 100644 --- a/packages/ai/test/recorded-scenarios.ts +++ b/packages/ai/test/recorded-scenarios.ts @@ -127,11 +127,6 @@ const assistantContent = (events: ReadonlyArray) => export const expectFinish = (events: ReadonlyArray, reason: FinishReason) => expect(events.at(-1)).toMatchObject({ type: "finish", reason: { normalized: reason } }) -export const expectWeatherToolCall = (response: LLMResponse) => - expect(response.toolCalls).toMatchObject([ - { type: "tool-call", id: expect.any(String), name: weatherToolName, input: { city: "Paris" } }, - ]) - export const expectWeatherToolLoop = (events: ReadonlyArray) => { const finishes = events.filter(LLMEvent.is.finish) expect(finishes).toHaveLength(1)