Skip to content

Commit 127b4bf

Browse files
hntrlchristian-bromann
hntrl
and
christian-bromann
authored
chore(standard-tests): fix CI (#9609)
Co-authored-by: Christian Bromann <git@bromann.dev>
1 parent ca0814c commit 127b4bf

9 files changed

Lines changed: 43 additions & 43 deletions

File tree

‎libs/langchain-core/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
"camelcase": "6",
3636
"decamelize": "1.2.0",
3737
"js-tiktoken": "^1.0.12",
38-
"langsmith": "^0.3.64",
38+
"langsmith": "^0.3.82",
3939
"mustache": "^4.2.0",
4040
"p-queue": "^6.6.2",
4141
"uuid": "^10.0.0",

‎libs/langchain-standard-tests/src/base.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,13 @@ import {
22
BaseChatModel,
33
BaseChatModelCallOptions,
44
} from "@langchain/core/language_models/chat_models";
5-
import { BaseMessageChunk } from "@langchain/core/messages";
5+
import { AIMessageChunk } from "@langchain/core/messages";
66

77
export type RecordStringAny = Record<string, unknown>;
88

99
export type BaseChatModelConstructor<
1010
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
11-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
11+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
1212
ConstructorArgs extends RecordStringAny = RecordStringAny
1313
> = new (args: ConstructorArgs) => BaseChatModel<
1414
CallOptions,
@@ -17,7 +17,7 @@ export type BaseChatModelConstructor<
1717

1818
export type BaseChatModelsTestsFields<
1919
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
20-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
20+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
2121
ConstructorArgs extends RecordStringAny = RecordStringAny
2222
> = {
2323
Cls: BaseChatModelConstructor<
@@ -38,7 +38,7 @@ export type BaseChatModelsTestsFields<
3838

3939
export class BaseChatModelsTests<
4040
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
41-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
41+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
4242
ConstructorArgs extends RecordStringAny = RecordStringAny
4343
> implements
4444
BaseChatModelsTestsFields<CallOptions, OutputMessageType, ConstructorArgs>

‎libs/langchain-standard-tests/src/integration_tests/chat_models.ts‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import { BaseChatModelCallOptions } from "@langchain/core/language_models/chat_m
66
import {
77
AIMessage,
88
AIMessageChunk,
9-
BaseMessageChunk,
9+
BaseMessage,
1010
HumanMessage,
1111
SystemMessage,
1212
ToolMessage,
@@ -29,9 +29,9 @@ import {
2929
BaseChatModelsTests,
3030
BaseChatModelsTestsFields,
3131
RecordStringAny,
32-
} from "../base.ts";
33-
import { TestCallbackHandler } from "../utils.ts";
34-
import { isContentBlock } from "../utils/types.ts";
32+
} from "../base.js";
33+
import { TestCallbackHandler } from "../utils.js";
34+
import { isContentBlock } from "../utils/types.js";
3535

3636
// Placeholder data for content block tests
3737
const TEST_IMAGE_URL =
@@ -89,7 +89,7 @@ const MATH_ADDITION_PROMPT = /* #__PURE__ */ ChatPromptTemplate.fromMessages([
8989

9090
export interface ChatModelIntegrationTestsFields<
9191
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
92-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
92+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
9393
ConstructorArgs extends RecordStringAny = RecordStringAny
9494
> extends BaseChatModelsTestsFields<
9595
CallOptions,
@@ -117,7 +117,7 @@ export interface ChatModelIntegrationTestsFields<
117117

118118
export abstract class ChatModelIntegrationTests<
119119
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
120-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
120+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
121121
ConstructorArgs extends RecordStringAny = RecordStringAny
122122
> extends BaseChatModelsTests<CallOptions, OutputMessageType, ConstructorArgs> {
123123
functionId = "abc123";
@@ -1510,7 +1510,7 @@ export abstract class ChatModelIntegrationTests<
15101510
const modelWithTools = model.bindTools([weatherTool]);
15111511

15121512
// Initialize the conversation with a weather query
1513-
const messages = [
1513+
const messages: BaseMessage[] = [
15141514
new HumanMessage(
15151515
"What's the weather like in San Francisco right now? Use the 'get_current_weather' tool to find the answer."
15161516
),
@@ -1602,7 +1602,7 @@ export abstract class ChatModelIntegrationTests<
16021602
const modelWithTools = model.bindTools([weatherTool]);
16031603

16041604
// Initialize the conversation with a weather query
1605-
const messages = [
1605+
const messages: BaseMessage[] = [
16061606
new HumanMessage(
16071607
"What's the weather like in San Francisco right now? Use the 'get_current_weather' tool to find the answer."
16081608
),

‎libs/langchain-standard-tests/src/integration_tests/vitest.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
import { expect, describe, test } from "vitest";
22
import { BaseChatModelCallOptions } from "@langchain/core/language_models/chat_models";
3-
import { BaseMessageChunk } from "@langchain/core/messages";
4-
import { RecordStringAny } from "../base.ts";
3+
import { AIMessageChunk } from "@langchain/core/messages";
4+
import { RecordStringAny } from "../base.js";
55
import {
66
ChatModelIntegrationTestsFields,
77
ChatModelIntegrationTests as BaseChatModelIntegrationTests,
8-
} from "./chat_models.ts";
8+
} from "./chat_models.js";
99

1010
export abstract class ChatModelIntegrationTests<
1111
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
12-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
12+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
1313
ConstructorArgs extends RecordStringAny = RecordStringAny
1414
> extends BaseChatModelIntegrationTests<
1515
CallOptions,

‎libs/langchain-standard-tests/src/unit_tests/chat_models.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,15 +4,15 @@ import {
44
BaseChatModelCallOptions,
55
LangSmithParams,
66
} from "@langchain/core/language_models/chat_models";
7-
import { BaseMessageChunk } from "@langchain/core/messages";
7+
import { AIMessageChunk } from "@langchain/core/messages";
88
import { z } from "zod/v3";
99
import { InferInteropZodOutput } from "@langchain/core/utils/types";
1010
import { StructuredTool } from "@langchain/core/tools";
1111
import {
1212
BaseChatModelsTests,
1313
BaseChatModelsTestsFields,
1414
RecordStringAny,
15-
} from "../base.ts";
15+
} from "../base.js";
1616

1717
const person = /* #__PURE__ */ z
1818
.object({
@@ -35,7 +35,7 @@ class PersonTool extends StructuredTool {
3535

3636
export abstract class ChatModelUnitTests<
3737
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
38-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
38+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
3939
ConstructorArgs extends RecordStringAny = RecordStringAny
4040
> extends BaseChatModelsTests<CallOptions, OutputMessageType, ConstructorArgs> {
4141
expect: typeof JestExpect | typeof VitestExpect;

‎libs/langchain-standard-tests/src/unit_tests/vitest.ts‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
import { expect, describe, test } from "vitest";
22
import { BaseChatModelCallOptions } from "@langchain/core/language_models/chat_models";
3-
import { BaseMessageChunk } from "@langchain/core/messages";
4-
import { BaseChatModelsTestsFields, RecordStringAny } from "../base.ts";
5-
import { ChatModelUnitTests as BaseChatModelUnitTests } from "./chat_models.ts";
3+
import { AIMessageChunk } from "@langchain/core/messages";
4+
import { BaseChatModelsTestsFields, RecordStringAny } from "../base.js";
5+
import { ChatModelUnitTests as BaseChatModelUnitTests } from "./chat_models.js";
66

77
export abstract class ChatModelUnitTests<
88
CallOptions extends BaseChatModelCallOptions = BaseChatModelCallOptions,
9-
OutputMessageType extends BaseMessageChunk = BaseMessageChunk,
9+
OutputMessageType extends AIMessageChunk = AIMessageChunk,
1010
ConstructorArgs extends RecordStringAny = RecordStringAny
1111
> extends BaseChatModelUnitTests<
1212
CallOptions,

‎libs/langchain-standard-tests/src/vitest.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
export * from "./unit_tests/vitest.ts";
2-
export * from "./integration_tests/vitest.ts";
1+
export * from "./unit_tests/vitest.js";
2+
export * from "./integration_tests/vitest.js";

‎libs/providers/langchain-openai/vitest.config.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,10 @@ export default defineConfig((env) => {
6161
...common.test,
6262
environment: "node",
6363
include: configDefaults.include,
64-
typecheck: { enabled: true },
64+
// Disable typecheck in CI because dependency range tests run in isolated
65+
// Docker environments where workspace dependencies like @langchain/standard-tests
66+
// may not have their transitive dependencies properly resolved
67+
typecheck: { enabled: !process.env.CI },
6568
},
6669
};
6770
});

‎pnpm-lock.yaml‎

Lines changed: 13 additions & 16 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)