Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@

### 🔧 Bug Fixes

- **nous-research:** append /chat/completions to provider baseUrl so DefaultExecutor's default URL builder hits the correct endpoint instead of returning 404 ([#2826])
- **fix(quota):** honor explicit per-connection `quotaPreflightEnabled: false` even when the provider has global window defaults — adds early-return guard before the AND-of-negations gate in auth.ts ([#2831])
- **api:** include noAuth providers (opencode, etc.) in `/v1/models` active aliases so their models surface without a DB connection row (#2798)
- **opencode-go:** route Qwen3.x via Claude messages format and repair `fixMissingToolResponses` helper for Claude-shape upstreams (#2791 — thanks @jeferssonlemes)
Expand Down
2 changes: 1 addition & 1 deletion open-sse/config/providerRegistry.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3955,7 +3955,7 @@ export const REGISTRY: Record<string, RegistryEntry> = {
alias: "nous",
format: "openai",
executor: "default",
baseUrl: "https://inference-api.nousresearch.com/v1",
baseUrl: "https://inference-api.nousresearch.com/v1/chat/completions",
authType: "apikey",
authHeader: "bearer",
models: [
Expand Down
36 changes: 36 additions & 0 deletions tests/unit/executor-nous-research.test.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
/**
* Regression test for #2826 — NOUS-RESEARCH provider always returns 404 Not Found
*
* `nous-research` uses `executor: "default"`. `DefaultExecutor.buildUrl()`'s default
* switch case returns `config.baseUrl` verbatim. Without `/chat/completions` appended
* every outbound request hits `/v1` and gets a 404.
*
* Fix: set `baseUrl` in providerRegistry.ts to include `/chat/completions`.
*/
import test from "node:test";
import assert from "node:assert/strict";

import { DefaultExecutor } from "../../open-sse/executors/default.ts";

test("nous-research DefaultExecutor.buildUrl() returns a URL ending with /chat/completions", () => {
const executor = new DefaultExecutor("nous-research");

const url = executor.buildUrl("Hermes-4-405B", true, 0, null);

assert.match(
url,
/\/chat\/completions$/,
`Expected URL to end with /chat/completions, got: ${url}`
);
});

test("nous-research DefaultExecutor.buildUrl() targets the correct inference endpoint", () => {
const executor = new DefaultExecutor("nous-research");

const url = executor.buildUrl("Hermes-4-70B", false, 0, null);

assert.equal(
url,
"https://inference-api.nousresearch.com/v1/chat/completions"
);
});