pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-31 12:22:14 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: improve API base URL handling and filter for chat models in CopilotToolCard

Browse Source
This commit is contained in:
Alptekin Gülcan
2026-02-28 18:46:50 +00:00
committed by GitHub
parent e4d83e91bb
commit 8383da8a50
2 changed files with 5 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
src/app/(dashboard)/dashboard/cli-tools/CLIToolsPageClient.tsx
View File

@@ -33,7 +33,6 @@ export default function CLIToolsPageClient({ machineId }) {
const [apiKeys, setApiKeys] = useState([]);
const [toolStatuses, setToolStatuses] = useState({});
const [statusesLoaded, setStatusesLoaded] = useState(false);
const [apiBaseUrl, setApiBaseUrl] = useState("");
useEffect(() => {
fetchConnections();
@@ -48,12 +47,6 @@ export default function CLIToolsPageClient({ machineId }) {
if (res.ok) {
const data = await res.json();
setCloudEnabled(data.cloudEnabled || false);
if (typeof window !== "undefined") {
const protocol = window.location.protocol;
const hostname = window.location.hostname;
const apiPort = data?.apiPort || 20128;
setApiBaseUrl(`${protocol}//${hostname}:${apiPort}`);
}
}
} catch (error) {
console.log("Error loading cloud settings:", error);
@@ -155,9 +148,8 @@ export default function CLIToolsPageClient({ machineId }) {
if (cloudEnabled && CLOUD_URL) {
return CLOUD_URL;
}
if (apiBaseUrl) {
return apiBaseUrl;
}
// Use window.location.origin directly — works correctly in Docker/reverse-proxy
// Per @alpgul feedback: don't use baseUrl prop (has port duplication issues)
if (typeof window !== "undefined") {
return window.location.origin;
}

9
src/app/(dashboard)/dashboard/cli-tools/components/CopilotToolCard.tsx
View File

@@ -60,7 +60,7 @@ export default function CopilotToolCard({
.then((data) => {
if (cancelled) return;
const modelList = (data.data || [])
.filter((m: any) => m.id) // Only models with valid IDs
.filter((m: any) => m && !m.type && !m.parent && m.id) // Only chat models with valid IDs
.map((m: any) => ({
value: m.id,
label: m.id,
@@ -112,11 +112,8 @@ export default function CopilotToolCard({
};
const getBaseUrlForConfig = () => {
// Use window.location.origin directly — works correctly in Docker/reverse-proxy
// Per @alpgul feedback: don't use baseUrl prop (has port duplication issues)
const origin =
typeof window !== "undefined" ? window.location.origin : "http://localhost:20128";
return `${origin}/v1/chat/completions`;
const url = baseUrl;
return `${url}/v1/chat/completions`;
};
// Generate the Copilot chatLanguageModels.json config