test(regression): cover recent landed fix paths

This commit is contained in:
Peter Steinberger
2026-03-07 23:07:06 +00:00
parent c76d29208b
commit 7ab49a7fb7
6 changed files with 146 additions and 0 deletions

View File

@@ -518,6 +518,35 @@ describe("runWithModelFallback", () => {
}
});
it("sanitizes model identifiers in model_not_found warnings", async () => {
setLoggerOverride({ level: "silent", consoleLevel: "warn" });
const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
try {
const cfg = makeCfg();
const run = vi
.fn()
.mockRejectedValueOnce(new Error("Model not found: openai/gpt-6"))
.mockResolvedValueOnce("ok");
const result = await runWithModelFallback({
cfg,
provider: "openai",
model: "gpt-6\u001B[31m\nspoof",
run,
});
expect(result.result).toBe("ok");
const warning = warnSpy.mock.calls[0]?.[0] as string;
expect(warning).toContain('Model "openai/gpt-6spoof" not found');
expect(warning).not.toContain("\u001B");
expect(warning).not.toContain("\n");
} finally {
warnSpy.mockRestore();
setLoggerOverride(null);
resetLogger();
}
});
it("skips providers when all profiles are in cooldown", async () => {
await expectSkippedUnavailableProvider({
providerPrefix: "cooldown-test",

View File

@@ -472,6 +472,39 @@ describe("model-selection", () => {
}
});
it("sanitizes control characters in providerless-model warnings", () => {
setLoggerOverride({ level: "silent", consoleLevel: "warn" });
const warnSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
try {
const cfg: Partial<OpenClawConfig> = {
agents: {
defaults: {
model: { primary: "\u001B[31mclaude-3-5-sonnet\nspoof" },
},
},
};
const result = resolveConfiguredModelRef({
cfg: cfg as OpenClawConfig,
defaultProvider: "google",
defaultModel: "gemini-pro",
});
expect(result).toEqual({
provider: "anthropic",
model: "\u001B[31mclaude-3-5-sonnet\nspoof",
});
const warning = warnSpy.mock.calls[0]?.[0] as string;
expect(warning).toContain('Falling back to "anthropic/claude-3-5-sonnet"');
expect(warning).not.toContain("\u001B");
expect(warning).not.toContain("\n");
} finally {
warnSpy.mockRestore();
setLoggerOverride(null);
resetLogger();
}
});
it("should use default provider/model if config is empty", () => {
const cfg: Partial<OpenClawConfig> = {};
const result = resolveConfiguredModelRef({

View File

@@ -1429,6 +1429,32 @@ describe("applyExtraParamsToAgent", () => {
expect(payload).not.toHaveProperty("store");
});
it("keeps existing context_management when stripping store for supportsStore=false models", () => {
const payload = runResponsesPayloadMutationCase({
applyProvider: "custom-openai-responses",
applyModelId: "gemini-2.5-pro",
model: {
api: "openai-responses",
provider: "custom-openai-responses",
id: "gemini-2.5-pro",
name: "gemini-2.5-pro",
baseUrl: "https://gateway.ai.cloudflare.com/v1/account/gateway/openai",
reasoning: false,
input: ["text"],
cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0 },
contextWindow: 1_000_000,
maxTokens: 65_536,
compat: { supportsStore: false },
} as unknown as Model<"openai-responses">,
payload: {
store: false,
context_management: [{ type: "compaction", compact_threshold: 12_345 }],
},
});
expect(payload).not.toHaveProperty("store");
expect(payload.context_management).toEqual([{ type: "compaction", compact_threshold: 12_345 }]);
});
it("auto-injects OpenAI Responses context_management compaction for direct OpenAI models", () => {
const payload = runResponsesPayloadMutationCase({
applyProvider: "openai",