Skip to content

Commit f1f1bcc

Browse files
committed
fix: use PUT fallback for provider invalidation and clean up debug logs
1 parent 28bb6e5 commit f1f1bcc

2 files changed

Lines changed: 16 additions & 32 deletions

File tree

backend/app/utils/workforce.py

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -265,18 +265,10 @@ async def eigent_start(self, subtasks: list[Task]):
265265
# Push error event to SSE queue so frontend receives notification
266266
try:
267267
task_lock = get_task_lock(self.api_task_id)
268-
logger.info(
269-
f"[WF-LIFECYCLE] Pushing error to SSE queue, "
270-
f"task_lock={'found' if task_lock else 'None'}"
271-
)
272268
if task_lock is not None:
273269
message, error_code, _ = normalize_error_to_openai_format(
274270
e
275271
)
276-
logger.info(
277-
f"[WF-LIFECYCLE] Error normalized: "
278-
f"error_code={error_code}, message={message[:100]}"
279-
)
280272
await task_lock.put_queue(
281273
ActionErrorData(
282274
data={
@@ -285,9 +277,6 @@ async def eigent_start(self, subtasks: list[Task]):
285277
},
286278
)
287279
)
288-
logger.info(
289-
"[WF-LIFECYCLE] Error event pushed to SSE queue"
290-
)
291280
except Exception as queue_err:
292281
logger.error(
293282
"[WF-LIFECYCLE] Failed to push error to SSE queue: "

src/store/chatStore.ts

Lines changed: 16 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ import {
1818
fetchPut,
1919
getBaseURL,
2020
proxyFetchGet,
21-
proxyFetchPatch,
2221
proxyFetchPost,
2322
proxyFetchPut,
2423
uploadFile,
@@ -1904,26 +1903,22 @@ const chatStore = (initial?: Partial<ChatStore>) =>
19041903
showModelErrorToast(errorMessage, errorCode);
19051904

19061905
// If API key is invalid, mark the provider as invalid on the server
1907-
if (errorCode === 'invalid_api_key' && preferredProviderId) {
1908-
proxyFetchPatch(
1909-
`/api/provider/${preferredProviderId}/invalidate`
1910-
).catch(() => {
1911-
// Fallback: PATCH endpoint may not be deployed yet,
1912-
// use PUT with full provider data to set is_vaild=1
1913-
if (preferredProvider) {
1914-
proxyFetchPut(`/api/provider/${preferredProviderId}`, {
1915-
provider_name: preferredProvider.provider_name,
1916-
model_type: preferredProvider.model_type,
1917-
api_key: preferredProvider.api_key,
1918-
endpoint_url: preferredProvider.endpoint_url || '',
1919-
encrypted_config: preferredProvider.encrypted_config,
1920-
is_vaild: 1,
1921-
prefer: preferredProvider.prefer ?? false,
1922-
}).catch((err: unknown) =>
1923-
console.error('Failed to invalidate provider:', err)
1924-
);
1925-
}
1926-
});
1906+
if (
1907+
errorCode === 'invalid_api_key' &&
1908+
preferredProviderId &&
1909+
preferredProvider
1910+
) {
1911+
proxyFetchPut(`/api/provider/${preferredProviderId}`, {
1912+
provider_name: preferredProvider.provider_name,
1913+
model_type: preferredProvider.model_type,
1914+
api_key: preferredProvider.api_key,
1915+
endpoint_url: preferredProvider.endpoint_url || '',
1916+
encrypted_config: preferredProvider.encrypted_config || null,
1917+
is_vaild: 1,
1918+
prefer: preferredProvider.prefer ?? false,
1919+
}).catch((err: unknown) =>
1920+
console.error('Failed to invalidate provider:', err)
1921+
);
19271922
}
19281923

19291924
// Mark all incomplete tasks as failed

0 commit comments

Comments
 (0)