Skip to content

Commit 550a9b4

Browse files
committed
test(ci): bound image tool iMessage fixtures
1 parent 169effa commit 550a9b4

2 files changed

Lines changed: 12 additions & 1 deletion

File tree

src/agents/tools/image-tool.test.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -544,6 +544,9 @@ function installImageUnderstandingProviderDeps(
544544
loadImageWebMediaRuntime?: NonNullable<
545545
Parameters<typeof testing.setProviderDepsForTest>[0]
546546
>["loadImageWebMediaRuntime"];
547+
resolveImageCompressionPolicy?: NonNullable<
548+
Parameters<typeof testing.setProviderDepsForTest>[0]
549+
>["resolveImageCompressionPolicy"];
547550
resolveModelAsync?: NonNullable<
548551
Parameters<typeof testing.setProviderDepsForTest>[0]
549552
>["resolveModelAsync"];
@@ -574,6 +577,9 @@ function installImageUnderstandingProviderDeps(
574577
capability === "image" ? ["openai", "anthropic"] : [],
575578
resolveDefaultMediaModel: ({ providerId, capability }) =>
576579
capability === "image" ? defaultImageModels.get(providerId.toLowerCase()) : undefined,
580+
...(options?.resolveImageCompressionPolicy
581+
? { resolveImageCompressionPolicy: options.resolveImageCompressionPolicy }
582+
: {}),
577583
...(options?.resolveModelAsync ? { resolveModelAsync: options.resolveModelAsync } : {}),
578584
...(options?.loadImageWebMediaRuntime
579585
? { loadImageWebMediaRuntime: options.loadImageWebMediaRuntime }
@@ -587,6 +593,7 @@ function installImageUnderstandingProviderStubs(...providers: MediaUnderstanding
587593

588594
function installFastLocalImageProviderStubs(...providers: MediaUnderstandingProvider[]) {
589595
installImageUnderstandingProviderDeps(providers, {
596+
resolveImageCompressionPolicy: async ({ imageCount }) => ({ imageCount }),
590597
resolveModelAsync: async (provider, model) => ({
591598
model: {
592599
id: model,

src/agents/tools/image-tool.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ const imageToolProviderDeps = {
111111
resolveDefaultMediaModel,
112112
resolveBundledStaticCatalogModel,
113113
resolveModelAsync,
114+
resolveImageCompressionPolicy,
114115
loadImageWebMediaRuntime,
115116
};
116117

@@ -173,6 +174,7 @@ export const testing = {
173174
resolveDefaultMediaModel?: typeof resolveDefaultMediaModel;
174175
resolveBundledStaticCatalogModel?: typeof resolveBundledStaticCatalogModel;
175176
resolveModelAsync?: typeof resolveModelAsync;
177+
resolveImageCompressionPolicy?: typeof resolveImageCompressionPolicy;
176178
loadImageWebMediaRuntime?: typeof loadImageWebMediaRuntime;
177179
}) {
178180
imageToolProviderDeps.buildProviderRegistry =
@@ -190,6 +192,8 @@ export const testing = {
190192
imageToolProviderDeps.resolveBundledStaticCatalogModel =
191193
overrides?.resolveBundledStaticCatalogModel ?? resolveBundledStaticCatalogModel;
192194
imageToolProviderDeps.resolveModelAsync = overrides?.resolveModelAsync ?? resolveModelAsync;
195+
imageToolProviderDeps.resolveImageCompressionPolicy =
196+
overrides?.resolveImageCompressionPolicy ?? resolveImageCompressionPolicy;
193197
imageToolProviderDeps.loadImageWebMediaRuntime =
194198
overrides?.loadImageWebMediaRuntime ?? loadImageWebMediaRuntime;
195199
},
@@ -830,7 +834,7 @@ export function createImageTool(options?: {
830834
"No image model is configured. Set agents.defaults.imageModel or configure an image-capable provider.",
831835
);
832836
}
833-
const imageCompression = await resolveImageCompressionPolicy({
837+
const imageCompression = await imageToolProviderDeps.resolveImageCompressionPolicy({
834838
cfg: options?.config,
835839
imageModelConfig,
836840
modelOverride,

0 commit comments

Comments
 (0)