pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-08-02 21:32:10 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(tests): remove duplicate assertion in schema coercion & fix(cli): ignore system vars in env check

Browse Source
This commit is contained in:
diegosouzapw
2026-05-23 21:50:45 -03:00
parent 39e8c9105e
commit 9c94be6597
2 changed files with 21 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
scripts/check/check-env-doc-sync.mjs
View File

@@ -39,6 +39,8 @@ const IGNORE_FROM_CODE = new Set([
"PATH",
"HOME",
"USER",
"LOGNAME",
"XDG_CURRENT_DESKTOP",
"PWD",
"SHELL",
"TERM",

58
tests/unit/schema-coercion.test.ts
View File

@@ -131,28 +131,6 @@ test("injectEmptyReasoningContentForToolCalls supports DeepSeek V4 models across
}
});
test("injectEmptyReasoningContentForToolCalls skips non-reasoning models", () => {
const messages = [{ role: "assistant", tool_calls: [{ id: "call_1" }] }];
const result = injectEmptyReasoningContentForToolCalls(
messages,
"deepseek",
"deepseek-v3"
) as Array<{ reasoning_content?: string }>;
assert.equal(result[0].reasoning_content, undefined);
});
const messages = [{ role: "assistant", tool_calls: [{ id: "call_1" }] }];
const result = injectEmptyReasoningContentForToolCalls(
messages,
"deepseek",
"deepseek-reasoner"
) as Array<{ reasoning_content?: string }>;
assert.equal(result[0].reasoning_content, undefined);
});
test("injectEmptyReasoningContentForToolCalls supports Kimi K2 models", () => {
const messages = [
{ role: "user", content: "hello" },
@@ -160,11 +138,9 @@ test("injectEmptyReasoningContentForToolCalls supports Kimi K2 models", () => {
];
for (const model of ["kimi-k2", "kimi-k2.5", "kimi-k2.6", "kimi-k2-thinking"]) {
const result = injectEmptyReasoningContentForToolCalls(
messages,
"moonshot",
model
) as Array<{ reasoning_content?: string }>;
const result = injectEmptyReasoningContentForToolCalls(messages, "moonshot", model) as Array<{
reasoning_content?: string;
}>;
assert.equal(result[1].reasoning_content, "", `should inject reasoning_content for ${model}`);
}
@@ -204,13 +180,15 @@ test("injectEmptyReasoningContentForToolCalls supports all reasoning replay prov
];
for (const { provider, model } of reasoningProviders) {
const result = injectEmptyReasoningContentForToolCalls(
messages,
provider,
model
) as Array<{ reasoning_content?: string }>;
const result = injectEmptyReasoningContentForToolCalls(messages, provider, model) as Array<{
reasoning_content?: string;
}>;
assert.equal(result[1].reasoning_content, "", `should inject reasoning_content for ${provider}/${model}`);
assert.equal(
result[1].reasoning_content,
"",
`should inject reasoning_content for ${provider}/${model}`
);
}
});
@@ -225,12 +203,14 @@ test("injectEmptyReasoningContentForToolCalls skips non-reasoning models", () =>
];
for (const { provider, model } of nonReasoningModels) {
const result = injectEmptyReasoningContentForToolCalls(
messages,
provider,
model
) as Array<{ reasoning_content?: string }>;
const result = injectEmptyReasoningContentForToolCalls(messages, provider, model) as Array<{
reasoning_content?: string;
}>;
assert.equal(result[0].reasoning_content, undefined, `should NOT inject reasoning_content for ${provider}/${model}`);
assert.equal(
result[0].reasoning_content,
undefined,
`should NOT inject reasoning_content for ${provider}/${model}`
);
}
});