-
Notifications
You must be signed in to change notification settings - Fork 61k
Qwen #6538
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Qwen #6538
Changes from 3 commits
861d854
4aaa9db
e3fc9ee
cdeb278
b07760f
ee25e4a
8af4d7a
a043f03
4ec717f
972f957
b42c515
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
@@ -1,22 +1,16 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { getServerSideConfig } from "@/app/config/server"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
ALIBABA_BASE_URL, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
ApiPath, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
ModelProvider, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
ServiceProvider, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} from "@/app/constant"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { ALIBABA_BASE_URL, ApiPath, ModelProvider } from "@/app/constant"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { prettyObject } from "@/app/utils/format"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { NextRequest, NextResponse } from "next/server"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { auth } from "@/app/api/auth"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
import { isModelNotavailableInServer } from "@/app/utils/model"; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const serverConfig = getServerSideConfig(); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
export async function handle( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
req: NextRequest, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ params }: { params: { path: string[] } }, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.log("[Alibaba Route] params ", params); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// console.log("[Alibaba Route] params ", params); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (req.method === "OPTIONS") { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return NextResponse.json({ body: "OK" }, { status: 200 }); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -42,7 +36,9 @@ async function request(req: NextRequest) { | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const controller = new AbortController(); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// alibaba use base url or just remove the path | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
let path = `${req.nextUrl.pathname}`.replaceAll(ApiPath.Alibaba, ""); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
let path = `${req.nextUrl.pathname}` | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
.replaceAll(ApiPath.Alibaba, "") | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
.replace("/api", ""); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
let baseUrl = serverConfig.alibabaUrl || ALIBABA_BASE_URL; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -65,6 +61,9 @@ async function request(req: NextRequest) { | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const fetchUrl = `${baseUrl}${path}`; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.log("[Alibaba] fetchUrl", fetchUrl); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const fetchOptions: RequestInit = { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
headers: { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"Content-Type": "application/json", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -83,28 +82,77 @@ async function request(req: NextRequest) { | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (serverConfig.customModels && req.body) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const clonedBody = await req.text(); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
fetchOptions.body = clonedBody; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
let jsonBody: any = {}; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
try { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
jsonBody = JSON.parse(clonedBody); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// Move input.messages to messages at the root level if present | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (jsonBody.input && Array.isArray(jsonBody.input.messages)) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
jsonBody.messages = jsonBody.input.messages; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// Remove input.messages to avoid duplication | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
delete jsonBody.input; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
jsonBody.stream = true; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+87
to
+100
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix performance issue with delete operator. The Replace the delete operation with object destructuring: - // Move input.messages to messages at the root level if present
- if (jsonBody.input && Array.isArray(jsonBody.input.messages)) {
- jsonBody.messages = jsonBody.input.messages;
-
- // Remove input.messages to avoid duplication
- delete jsonBody.input;
-
- jsonBody.stream = true;
- }
+ // Move input.messages to messages at the root level if present
+ if (jsonBody.input && Array.isArray(jsonBody.input.messages)) {
+ const { input, ...bodyWithoutInput } = jsonBody;
+ jsonBody = {
+ ...bodyWithoutInput,
+ messages: input.messages,
+ stream: true
+ };
+ } 📝 Committable suggestion
Suggested change
🧰 Tools🪛 Biome (1.9.4)[error] 95-95: Avoid the delete operator which can impact performance. Unsafe fix: Use an undefined assignment instead. (lint/performance/noDelete) 🤖 Prompt for AI Agents
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const current_model = jsonBody?.model; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.log("[Alibaba] custom models", current_model); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
//kiem tra xem model co phai la qwen-vl hay khong (vision model) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (current_model && current_model.startsWith("qwen-vl")) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.log("[Alibaba] current model is qwen-vl"); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.log("xu ly hinh anh trong message"); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// Reformat image objects in messages | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (Array.isArray(jsonBody.messages)) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
jsonBody.messages = jsonBody.messages.map((msg: any) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (Array.isArray(msg.content)) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
msg.content = msg.content.map((item: any) => { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if (item && typeof item === "object" && "image" in item) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
type: "image_url", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
image_url: { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
url: item.image, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return item; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return msg; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+105
to
+129
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Refactor vision model processing and use English comments. The vision model processing logic has several maintainability issues:
Consider refactoring like this: - //kiem tra xem model co phai la qwen-vl hay khong (vision model)
- if (current_model && current_model.startsWith("qwen-vl")) {
- console.log("[Alibaba] current model is qwen-vl");
- console.log("xu ly hinh anh trong message");
-
- // Reformat image objects in messages
- if (Array.isArray(jsonBody.messages)) {
- jsonBody.messages = jsonBody.messages.map((msg: any) => {
- if (Array.isArray(msg.content)) {
- msg.content = msg.content.map((item: any) => {
- if (item && typeof item === "object" && "image" in item) {
- return {
- type: "image_url",
- image_url: {
- url: item.image,
- },
- };
- }
- return item;
- });
- }
- return msg;
- });
- }
- }
+ // Check if model is a qwen-vl vision model
+ if (current_model?.startsWith("qwen-vl")) {
+ console.log("[Alibaba] Processing vision model:", current_model);
+ jsonBody.messages = transformVisionMessages(jsonBody.messages);
+ } Add this helper function: function transformVisionMessages(messages: any[]): any[] {
if (!Array.isArray(messages)) return messages;
return messages.map((msg) => {
if (!Array.isArray(msg.content)) return msg;
return {
...msg,
content: msg.content.map((item) => {
if (item && typeof item === "object" && "image" in item) {
return {
type: "image_url",
image_url: { url: item.image },
};
}
return item;
}),
};
});
} 🧰 Tools🪛 Biome (1.9.4)[error] 106-106: Change to an optional chain. Unsafe fix: Change to an optional chain. (lint/complexity/useOptionalChain) 🤖 Prompt for AI Agents
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// console.log("[Alibaba] request body json", jsonBody); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
fetchOptions.body = JSON.stringify(jsonBody); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} catch (e) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
fetchOptions.body = clonedBody; // fallback if not JSON | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
const jsonBody = JSON.parse(clonedBody) as { model?: string }; | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// console.log("[Alibaba] request body", fetchOptions.body); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// not undefined and is false | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
if ( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
isModelNotavailableInServer( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
serverConfig.customModels, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
jsonBody?.model as string, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
ServiceProvider.Alibaba as string, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
return NextResponse.json( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
error: true, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
message: `you are not allowed to use ${jsonBody?.model} model`, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
status: 403, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// if ( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// isModelNotavailableInServer( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// serverConfig.customModels, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// jsonBody?.model as string, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// ServiceProvider.Alibaba as string, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// ) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// return NextResponse.json( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// error: true, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// message: `you are not allowed to use ${jsonBody?.model} model`, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// }, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// status: 403, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// }, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// ); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
// } | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+141
to
+157
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Remove commented model availability check code. This large block of commented code should either be removed entirely if no longer needed, or uncommented and fixed if the functionality is still required. If this functionality is no longer needed, remove the commented code: - // not undefined and is false
- // if (
- // isModelNotavailableInServer(
- // serverConfig.customModels,
- // jsonBody?.model as string,
- // ServiceProvider.Alibaba as string,
- // )
- // ) {
- // return NextResponse.json(
- // {
- // error: true,
- // message: `you are not allowed to use ${jsonBody?.model} model`,
- // },
- // {
- // status: 403,
- // },
- // );
- // } If this functionality should be retained, please uncomment and ensure the imports are available.
🤖 Prompt for AI Agents
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} catch (e) { | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
console.error(`[Alibaba] filter`, e); | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
|
@@ -151,7 +151,8 @@ export class DeepSeekApi implements LLMApi { | |||||||||
controller, | ||||||||||
// parseSSE | ||||||||||
(text: string, runTools: ChatMessageTool[]) => { | ||||||||||
// console.log("parseSSE", text, runTools); | ||||||||||
console.log("parseSSE", text, runTools); | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Make debug logging conditional or remove it. The enabled debug log will add noise to production logs and potentially expose sensitive data. Consider making it conditional based on a debug flag or environment variable. - console.log("parseSSE", text, runTools);
+ if (process.env.NODE_ENV === 'development' || process.env.DEBUG_DEEPSEEK) {
+ console.log("parseSSE", text, runTools);
+ } Or if debugging is no longer needed: - console.log("parseSSE", text, runTools); 📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents
|
||||||||||
|
||||||||||
const json = JSON.parse(text); | ||||||||||
const choices = json.choices as Array<{ | ||||||||||
delta: { | ||||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -153,81 +153,35 @@ export class SiliconflowApi implements LLMApi { | |
tools as any, | ||
funcs, | ||
controller, | ||
// parseSSE | ||
// parseSSE mới cho SiliconFlow response | ||
(text: string, runTools: ChatMessageTool[]) => { | ||
// console.log("parseSSE", text, runTools); | ||
// Parse chuỗi JSON trả về thành đối tượng | ||
const json = JSON.parse(text); | ||
const choices = json.choices as Array<{ | ||
delta: { | ||
content: string | null; | ||
tool_calls: ChatMessageTool[]; | ||
reasoning_content: string | null; | ||
}; | ||
}>; | ||
const tool_calls = choices[0]?.delta?.tool_calls; | ||
if (tool_calls?.length > 0) { | ||
const index = tool_calls[0]?.index; | ||
const id = tool_calls[0]?.id; | ||
const args = tool_calls[0]?.function?.arguments; | ||
if (id) { | ||
runTools.push({ | ||
id, | ||
type: tool_calls[0]?.type, | ||
function: { | ||
name: tool_calls[0]?.function?.name as string, | ||
arguments: args, | ||
}, | ||
}); | ||
} else { | ||
// @ts-ignore | ||
runTools[index]["function"]["arguments"] += args; | ||
} | ||
} | ||
const reasoning = choices[0]?.delta?.reasoning_content; | ||
const content = choices[0]?.delta?.content; | ||
|
||
// Skip if both content and reasoning_content are empty or null | ||
if ( | ||
(!reasoning || reasoning.length === 0) && | ||
(!content || content.length === 0) | ||
) { | ||
return { | ||
isThinking: false, | ||
content: "", | ||
}; | ||
} | ||
// Lấy nội dung trả lời từ output.text | ||
const content = json?.output?.text ?? ""; | ||
|
||
if (reasoning && reasoning.length > 0) { | ||
return { | ||
isThinking: true, | ||
content: reasoning, | ||
}; | ||
} else if (content && content.length > 0) { | ||
// Nếu không có nội dung trả lời, trả về trạng thái không suy nghĩ và nội dung rỗng | ||
if (!content || content.length === 0) { | ||
return { | ||
isThinking: false, | ||
content: content, | ||
content: "", | ||
}; | ||
} | ||
|
||
// Trả về trạng thái không suy nghĩ và nội dung trả lời | ||
return { | ||
isThinking: false, | ||
content: "", | ||
content: content, | ||
}; | ||
}, | ||
// processToolMessage, include tool_calls message and tool call results | ||
// processToolMessage: SiliconFlow không có tool_call nên giữ nguyên hoặc để rỗng | ||
( | ||
requestPayload: RequestPayload, | ||
toolCallMessage: any, | ||
toolCallResult: any[], | ||
) => { | ||
// @ts-ignore | ||
requestPayload?.messages?.splice( | ||
// @ts-ignore | ||
requestPayload?.messages?.length, | ||
0, | ||
toolCallMessage, | ||
...toolCallResult, | ||
); | ||
// Không cần xử lý tool_call, có thể để trống hoặc giữ nguyên nếu muốn tương thích | ||
}, | ||
Comment on lines
+156
to
185
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Verification correctAddress code commenting language consistency and verify functional completeness. The code has several issues that need attention:
Apply this diff to fix the language consistency: - // parseSSE mới cho SiliconFlow response
+ // New parseSSE for SiliconFlow response
(text: string, runTools: ChatMessageTool[]) => {
- // Parse chuỗi JSON trả về thành đối tượng
+ // Parse JSON response string to object
const json = JSON.parse(text);
- // Lấy nội dung trả lời từ output.text
+ // Extract response content from output.text
const content = json?.output?.text ?? "";
- // Nếu không có nội dung trả lời, trả về trạng thái không suy nghĩ và nội dung rỗng
+ // If no response content, return non-thinking state with empty content
if (!content || content.length === 0) {
return {
isThinking: false,
content: "",
};
}
- // Trả về trạng thái không suy nghĩ và nội dung trả lời
+ // Return non-thinking state with response content
return {
isThinking: false,
content: content,
};
},
- // processToolMessage: SiliconFlow không có tool_call nên giữ nguyên hoặc để rỗng
+ // processToolMessage: SiliconFlow does not support tool calls
(
requestPayload: RequestPayload,
toolCallMessage: any,
toolCallResult: any[],
) => {
- // Không cần xử lý tool_call, có thể để trống hoặc giữ nguyên nếu muốn tương thích
+ // No tool call processing needed for SiliconFlow
}, Please verify that removing tool call functionality doesn't break existing features: 🏁 Script executed: #!/bin/bash
# Description: Check if SiliconFlow was previously using tool calls
# Expected: Find any references to tool calls in SiliconFlow context
echo "Searching for tool call usage in SiliconFlow context..."
rg -A 3 -B 3 "tool.*call.*siliconflow|siliconflow.*tool.*call" --ignore-case
echo "Searching for SiliconFlow model configurations..."
rg -A 5 "siliconflow.*tool|tool.*siliconflow" --ignore-case
echo "Checking for any SiliconFlow tool call tests..."
fd -e test.ts -e spec.ts --exec rg -l "siliconflow.*tool" {} \; Length of output: 1488 Address code commenting language consistency and verify functional completeness. The code has several issues that need attention:
Apply this diff to fix the language consistency: - // parseSSE mới cho SiliconFlow response
+ // New parseSSE for SiliconFlow response
(text: string, runTools: ChatMessageTool[]) => {
- // Parse chuỗi JSON trả về thành đối tượng
+ // Parse JSON response string to object
const json = JSON.parse(text);
- // Lấy nội dung trả lời từ output.text
+ // Extract response content from output.text
const content = json?.output?.text ?? "";
- // Nếu không có nội dung trả lời, trả về trạng thái không suy nghĩ và nội dung rỗng
+ // If no response content, return non-thinking state with empty content
if (!content || content.length === 0) {
return {
isThinking: false,
content: "",
};
}
- // Trả về trạng thái không suy nghĩ và nội dung trả lời
+ // Return non-thinking state with response content
return {
isThinking: false,
content: content,
};
},
- // processToolMessage: SiliconFlow không có tool_call nên giữ nguyên hoặc để rỗng
+ // processToolMessage: SiliconFlow does not support tool calls
(
requestPayload: RequestPayload,
toolCallMessage: any,
toolCallResult: any[],
) => {
- // Không cần xử lý tool_call, có thể để trống hoặc giữ nguyên nếu muốn tương thích
+ // No tool call processing needed for SiliconFlow
}, Please verify that removing tool call functionality doesn't break existing features: #!/bin/bash
# Description: Check if SiliconFlow was previously using tool calls
# Expected: Find any references to tool calls in SiliconFlow context
echo "Searching for tool call usage in SiliconFlow context..."
rg -A 3 -B 3 "tool.*call.*siliconflow|siliconflow.*tool.*call" --ignore-case
echo "Searching for SiliconFlow model configurations..."
rg -A 5 "siliconflow.*tool|tool.*siliconflow" --ignore-case
echo "Checking for any SiliconFlow tool call tests..."
fd -e test.ts -e spec.ts --exec rg -l "siliconflow.*tool" {} \; 🤖 Prompt for AI Agents
|
||
options, | ||
); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Remove commented console.log statements instead of leaving them commented out.
Commented-out debug logs should be removed entirely to keep the codebase clean, rather than left as commented code.
Also applies to: 30-30, 65-65, 81-81, 103-103, 131-131, 138-138, 159-159
🤖 Prompt for AI Agents