From c7ec55c69a0de90117655c86f20b298a46b19399 Mon Sep 17 00:00:00 2001
From: icarus <eurfelux@gmail.com>
Date: Fri, 19 Sep 2025 23:48:58 +0800
Subject: [PATCH] refactor(types): rename ApiModelsRequest to ApiModelsFilter
 for clarity

Update type name and related imports to better reflect its purpose as a filter type rather than a request type
---
 src/main/apiServer/services/models.ts | 4 ++--
 src/renderer/src/api/agent.ts         | 5 +++--
 src/renderer/src/types/apiModels.ts   | 2 +-
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/main/apiServer/services/models.ts b/src/main/apiServer/services/models.ts
index 3e281d68ab..93a1bdb0e1 100644
--- a/src/main/apiServer/services/models.ts
+++ b/src/main/apiServer/services/models.ts
@@ -1,4 +1,4 @@
-import { ApiModel, ApiModelsRequest, ApiModelsResponse } from '../../../renderer/src/types/apiModels'
+import { ApiModel, ApiModelsFilter, ApiModelsResponse } from '../../../renderer/src/types/apiModels'
 import { loggerService } from '../../services/LoggerService'
 import { getAvailableProviders, listAllAvailableModels, transformModelToOpenAI } from '../utils'
 
@@ -6,7 +6,7 @@ const logger = loggerService.withContext('ModelsService')
 
 // Re-export for backward compatibility
 
-export type ModelsFilter = ApiModelsRequest
+export type ModelsFilter = ApiModelsFilter
 
 export class ModelsService {
   async getModels(filter: ModelsFilter): Promise<ApiModelsResponse> {
diff --git a/src/renderer/src/api/agent.ts b/src/renderer/src/api/agent.ts
index 1fbbfa4114..0f5ce46379 100644
--- a/src/renderer/src/api/agent.ts
+++ b/src/renderer/src/api/agent.ts
@@ -3,6 +3,7 @@ import { formatAgentServerError } from '@renderer/utils'
 import {
   AddAgentForm,
   AgentServerErrorSchema,
+  ApiModelsFilter,
   ApiModelsResponse,
   ApiModelsResponseSchema,
   CreateAgentRequest,
@@ -81,7 +82,7 @@ export class AgentApiClient {
   public getSessionMessagesPath = (agentId: string, sessionId: string) =>
     `/${this.apiVersion}/agents/${agentId}/sessions/${sessionId}/messages`
 
-  public modelsPath = (props?: { providerType?: 'anthropic'; limit?: number }) => {
+  public modelsPath = (props?: ApiModelsFilter) => {
     const base = `/${this.apiVersion}/models`
     if (!props) return base
     if (objectKeys(props).length > 0) {
@@ -234,7 +235,7 @@ export class AgentApiClient {
     }
   }
 
-  public async getModels(props?: { providerType?: 'anthropic'; limit?: number }): Promise<ApiModelsResponse> {
+  public async getModels(props?: ApiModelsFilter): Promise<ApiModelsResponse> {
     const url = this.modelsPath(props)
     try {
       const response = await this.axios.get(url)
diff --git a/src/renderer/src/types/apiModels.ts b/src/renderer/src/types/apiModels.ts
index 0df2787320..252ebde912 100644
--- a/src/renderer/src/types/apiModels.ts
+++ b/src/renderer/src/types/apiModels.ts
@@ -32,5 +32,5 @@ export const ApiModelsResponseSchema = z.object({
 
 // Inferred TypeScript types
 export type ApiModel = z.infer<typeof ApiModelSchema>
-export type ApiModelsRequest = z.infer<typeof ApiModelsFilterSchema>
+export type ApiModelsFilter = z.infer<typeof ApiModelsFilterSchema>
 export type ApiModelsResponse = z.infer<typeof ApiModelsResponseSchema>