mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-08-08 08:12:20 +03:00
fix(qoder): include actionable CLI_QODER_BIN hint in connection test when qodercli is not found (#9277)
When the Qoder CLI (qodercli) is not detected by getCliRuntimeStatus after an OmniRoute restart (e.g. restricted launch context on Windows where APPDATA/PATH are not inherited), the connection test showed only the non-actionable 'Local CLI runtime is not installed'. Now it surfaces the same buildQoderCliNotFoundHint guidance already used in the executor path, telling the user to set CLI_QODER_BIN to the absolute path of qodercli. Closes #9277 Co-authored-by: diegosouzapw <diegosouzapw@users.noreply.github.com>
This commit is contained in:
committed by
GitHub
parent
3be585ef41
commit
85e518b7f4
1
changelog.d/fixes/9277-fix.plan.md
Normal file
1
changelog.d/fixes/9277-fix.plan.md
Normal file
@@ -0,0 +1 @@
|
||||
- fix(qoder): include actionable CLI_QODER_BIN hint in connection test when qodercli is not found (#9277)
|
||||
@@ -11,6 +11,7 @@ import { getConsistentMachineId } from "@/shared/utils/machineId";
|
||||
import { syncToCloud } from "@/lib/cloudSync";
|
||||
import { validateProviderApiKey } from "@/lib/providers/validation";
|
||||
import { getCliRuntimeStatus } from "@/shared/services/cliRuntime";
|
||||
import { buildQoderCliNotFoundHint } from "@omniroute/open-sse/services/qoderCliResolve.ts";
|
||||
// Use the shared open-sse token refresh with built-in dedup/race-condition cache
|
||||
import { getAccessToken } from "@omniroute/open-sse/services/tokenRefresh.ts";
|
||||
import { rotationGroupFor } from "@omniroute/open-sse/services/refreshSerializer.ts";
|
||||
@@ -206,7 +207,9 @@ async function getProviderRuntimeStatus(connection: any) {
|
||||
|
||||
const runtimeMessage = runtime.installed
|
||||
? `Local CLI runtime is installed but not runnable (${runtime.reason || "healthcheck_failed"})`
|
||||
: "Local CLI runtime is not installed";
|
||||
: provider === "qoder"
|
||||
? buildQoderCliNotFoundHint(runtime.reason || "not_found")
|
||||
: "Local CLI runtime is not installed";
|
||||
|
||||
return {
|
||||
...runtime,
|
||||
|
||||
Reference in New Issue
Block a user