Skip to content

Commit 74b679e

Browse files
authored
Merge pull request #1347 from narengogi/fix/image-edits-route-cleaner-implementation
cleaner implementation for image edits route
2 parents 0142e83 + 69b228c commit 74b679e

File tree

12 files changed

+19
-181
lines changed

12 files changed

+19
-181
lines changed

src/handlers/services/cacheService.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ export class CacheService {
3535
'createFinetune',
3636
'retrieveFinetune',
3737
'cancelFinetune',
38+
'imageEdit',
3839
];
3940
return !nonCacheEndpoints.includes(endpoint);
4041
}

src/handlers/services/requestContext.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,8 @@ export class RequestContext {
8787
}
8888

8989
get isStreaming(): boolean {
90+
if (this.endpoint === 'imageEdit' && this.requestBody instanceof FormData)
91+
return this.requestBody.get('stream') === 'true';
9092
return this.params.stream === true;
9193
}
9294

src/providers/azure-ai-inference/api.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,12 @@ const AzureAIInferenceAPI: ProviderAPIConfig = {
5050
...(azureDeploymentName && {
5151
'azureml-model-deployment': azureDeploymentName,
5252
}),
53-
...(['createTranscription', 'createTranslation', 'uploadFile'].includes(
54-
fn
55-
)
53+
...([
54+
'createTranscription',
55+
'createTranslation',
56+
'uploadFile',
57+
'imageEdit',
58+
].includes(fn)
5659
? {
5760
'Content-Type': 'multipart/form-data',
5861
}
@@ -119,6 +122,7 @@ const AzureAIInferenceAPI: ProviderAPIConfig = {
119122
embed: '/embeddings',
120123
realtime: '/realtime',
121124
imageGenerate: '/images/generations',
125+
imageEdit: '/images/edits',
122126
createSpeech: '/audio/speech',
123127
createTranscription: '/audio/transcriptions',
124128
createTranslation: '/audio/translations',
@@ -165,6 +169,7 @@ const AzureAIInferenceAPI: ProviderAPIConfig = {
165169
}
166170
case 'realtime':
167171
case 'imageGenerate':
172+
case 'imageEdit':
168173
case 'createSpeech':
169174
case 'createTranscription':
170175
case 'createTranslation':

src/providers/azure-ai-inference/index.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ const AzureAIInferenceAPIConfig: ProviderConfigs = {
3232
api: AzureAIInferenceAPI,
3333
chatComplete: AzureAIInferenceChatCompleteConfig,
3434
imageGenerate: AzureOpenAIImageGenerateConfig,
35+
imageEdit: {},
3536
createSpeech: AzureOpenAICreateSpeechConfig,
3637
createFinetune: OpenAICreateFinetuneConfig,
3738
createTranscription: {},

src/providers/azure-ai-inference/utils.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { AZURE_AI_INFERENCE } from '../../globals';
22
import { OpenAIErrorResponseTransform } from '../openai/utils';
3-
import { ErrorResponse } from '../types';
43

54
export const AzureAIInferenceResponseTransform = (
65
response: any,

src/providers/azure-openai/api.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,8 @@ const AzureOpenAIAPIConfig: ProviderAPIConfig = {
4545
if (
4646
fn === 'createTranscription' ||
4747
fn === 'createTranslation' ||
48-
fn === 'uploadFile'
48+
fn === 'uploadFile' ||
49+
fn === 'imageEdit'
4950
) {
5051
headersObj['Content-Type'] = 'multipart/form-data';
5152
}

src/providers/azure-openai/imageEdits.ts

Lines changed: 0 additions & 81 deletions
This file was deleted.

src/providers/azure-openai/index.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,17 +36,13 @@ import {
3636
OpenAIListInputItemsResponseTransformer,
3737
} from '../open-ai-base';
3838
import { AZURE_OPEN_AI } from '../../globals';
39-
import {
40-
AzureOpenAIImageEditConfig,
41-
AzureOpenAIImageEditResponseTransform,
42-
} from './imageEdits';
4339

4440
const AzureOpenAIConfig: ProviderConfigs = {
4541
complete: AzureOpenAICompleteConfig,
4642
embed: AzureOpenAIEmbedConfig,
4743
api: AzureOpenAIAPIConfig,
4844
imageGenerate: AzureOpenAIImageGenerateConfig,
49-
imageEdit: AzureOpenAIImageEditConfig,
45+
imageEdit: {},
5046
chatComplete: AzureOpenAIChatCompleteConfig,
5147
createSpeech: AzureOpenAICreateSpeechConfig,
5248
createFinetune: OpenAICreateFinetuneConfig,
@@ -68,7 +64,6 @@ const AzureOpenAIConfig: ProviderConfigs = {
6864
chatComplete: AzureOpenAIResponseTransform,
6965
embed: AzureOpenAIEmbedResponseTransform,
7066
imageGenerate: AzureOpenAIImageGenerateResponseTransform,
71-
imageEdit: AzureOpenAIImageEditResponseTransform,
7267
createSpeech: AzureOpenAICreateSpeechResponseTransform,
7368
createTranscription: AzureOpenAICreateTranscriptionResponseTransform,
7469
createTranslation: AzureOpenAICreateTranslationResponseTransform,

src/providers/openai/api.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ const OpenAIAPIConfig: ProviderAPIConfig = {
1717
if (
1818
fn === 'createTranscription' ||
1919
fn === 'createTranslation' ||
20-
fn === 'uploadFile'
20+
fn === 'uploadFile' ||
21+
fn === 'imageEdit'
2122
)
2223
headersObj['Content-Type'] = 'multipart/form-data';
2324

src/providers/openai/imageEdits.ts

Lines changed: 0 additions & 81 deletions
This file was deleted.

0 commit comments

Comments
 (0)