惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

IT之家
IT之家
T
Tailwind CSS Blog
V
V2EX
阮一峰的网络日志
阮一峰的网络日志
H
Help Net Security
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
腾讯CDC
GbyAI
GbyAI
酷 壳 – CoolShell
酷 壳 – CoolShell
H
Hackread – Cybersecurity News, Data Breaches, AI and More
Last Week in AI
Last Week in AI
A
About on SuperTechFans
L
LangChain Blog
Engineering at Meta
Engineering at Meta
F
Fortinet All Blogs
G
Google Developers Blog
The Cloudflare Blog
云风的 BLOG
云风的 BLOG
D
Docker
博客园 - 聂微东
博客园 - 司徒正美
Recent Announcements
Recent Announcements
MyScale Blog
MyScale Blog
U
Unit 42

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
fix(openai): prefer configured Codex OAuth for images · o...
steipete · 2026-04-24 · via Recent Commits to openclaw:main

@@ -2,16 +2,25 @@ import { afterEach, describe, expect, it, vi } from "vitest";

22

import { buildOpenAIImageGenerationProvider } from "./image-generation-provider.js";

3344

const {

5+

ensureAuthProfileStoreMock,

56

isProviderApiKeyConfiguredMock,

7+

listProfilesForProviderMock,

68

resolveApiKeyForProviderMock,

79

postJsonRequestMock,

810

postMultipartRequestMock,

911

assertOkOrThrowHttpErrorMock,

1012

resolveProviderHttpRequestConfigMock,

1113

} = vi.hoisted(() => ({

14+

ensureAuthProfileStoreMock: vi.fn(() => ({ version: 1, profiles: {} })),

1215

isProviderApiKeyConfiguredMock: vi.fn<

1316

(params: { provider: string; agentDir?: string }) => boolean

1417

>(() => false),

18+

listProfilesForProviderMock: vi.fn(

19+

(store: { profiles?: Record<string, { provider?: string }> }, provider: string) =>

20+

Object.entries(store.profiles ?? {})

21+

.filter(([, profile]) => profile.provider === provider)

22+

.map(([profileId]) => profileId),

23+

),

1524

resolveApiKeyForProviderMock: vi.fn(

1625

async (_params?: {

1726

provider?: string;

@@ -31,7 +40,9 @@ const {

3140

}));

32413342

vi.mock("openclaw/plugin-sdk/provider-auth", () => ({

43+

ensureAuthProfileStore: ensureAuthProfileStoreMock,

3444

isProviderApiKeyConfigured: isProviderApiKeyConfiguredMock,

45+

listProfilesForProvider: listProfilesForProviderMock,

3546

}));

36473748

vi.mock("openclaw/plugin-sdk/provider-auth-runtime", () => ({

@@ -96,10 +107,28 @@ function mockCodexAuthOnly() {

96107

});

97108

}

98109110+

function createCodexOAuthAuthStore() {

111+

return {

112+

version: 1 as const,

113+

profiles: {

114+

"openai-codex:default": {

115+

type: "oauth" as const,

116+

provider: "openai-codex",

117+

access: "codex-access",

118+

refresh: "codex-refresh",

119+

expires: Date.now() + 60_000,

120+

},

121+

},

122+

};

123+

}

124+99125

describe("openai image generation provider", () => {

100126

afterEach(() => {

127+

ensureAuthProfileStoreMock.mockReset();

128+

ensureAuthProfileStoreMock.mockReturnValue({ version: 1, profiles: {} });

101129

isProviderApiKeyConfiguredMock.mockReset();

102130

isProviderApiKeyConfiguredMock.mockReturnValue(false);

131+

listProfilesForProviderMock.mockClear();

103132

resolveApiKeyForProviderMock.mockReset();

104133

resolveApiKeyForProviderMock.mockResolvedValue({ apiKey: "openai-key" });

105134

postJsonRequestMock.mockReset();

@@ -412,6 +441,99 @@ describe("openai image generation provider", () => {

412441

});

413442

});

414443444+

it("uses configured Codex OAuth directly instead of probing an available OpenAI API key", async () => {

445+

resolveApiKeyForProviderMock.mockImplementation(async (params?: { provider?: string }) => {

446+

if (params?.provider === "openai") {

447+

return { apiKey: "openai-key", source: "OPENAI_API_KEY", mode: "api-key" };

448+

}

449+

if (params?.provider === "openai-codex") {

450+

return { apiKey: "codex-key", source: "profile:openai-codex:default", mode: "oauth" };

451+

}

452+

return {};

453+

});

454+

mockCodexImageStream({ imageData: "codex-image" });

455+456+

const provider = buildOpenAIImageGenerationProvider();

457+

const authStore = createCodexOAuthAuthStore();

458+

const result = await provider.generateImage({

459+

provider: "openai",

460+

model: "gpt-image-2",

461+

prompt: "Draw using configured Codex auth",

462+

cfg: {},

463+

authStore,

464+

});

465+466+

expect(resolveApiKeyForProviderMock).toHaveBeenCalledTimes(1);

467+

expect(resolveApiKeyForProviderMock).toHaveBeenCalledWith(

468+

expect.objectContaining({

469+

provider: "openai-codex",

470+

store: authStore,

471+

}),

472+

);

473+

expect(resolveApiKeyForProviderMock).not.toHaveBeenCalledWith(

474+

expect.objectContaining({

475+

provider: "openai",

476+

}),

477+

);

478+

expect(postJsonRequestMock).toHaveBeenCalledWith(

479+

expect.objectContaining({

480+

url: "https://chatgpt.com/backend-api/codex/responses",

481+

}),

482+

);

483+

expect(result.images[0]?.buffer).toEqual(Buffer.from("codex-image"));

484+

});

485+486+

it("uses direct OpenAI auth when custom OpenAI image config is explicit", async () => {

487+

mockGeneratedPngResponse();

488+

resolveApiKeyForProviderMock.mockImplementation(async (params?: { provider?: string }) => {

489+

if (params?.provider === "openai") {

490+

return { apiKey: "openai-key", source: "models.json", mode: "api-key" };

491+

}

492+

if (params?.provider === "openai-codex") {

493+

return { apiKey: "codex-key", source: "profile:openai-codex:default", mode: "oauth" };

494+

}

495+

return {};

496+

});

497+498+

const provider = buildOpenAIImageGenerationProvider();

499+

const authStore = createCodexOAuthAuthStore();

500+

await provider.generateImage({

501+

provider: "openai",

502+

model: "gpt-image-2",

503+

prompt: "Draw using explicit direct config",

504+

cfg: {

505+

models: {

506+

providers: {

507+

openai: {

508+

baseUrl: "https://api.openai.com/v1",

509+

apiKey: "OPENAI_API_KEY",

510+

models: [],

511+

},

512+

},

513+

},

514+

},

515+

authStore,

516+

});

517+518+

expect(resolveApiKeyForProviderMock).toHaveBeenCalledTimes(1);

519+

expect(resolveApiKeyForProviderMock).toHaveBeenCalledWith(

520+

expect.objectContaining({

521+

provider: "openai",

522+

store: authStore,

523+

}),

524+

);

525+

expect(resolveApiKeyForProviderMock).not.toHaveBeenCalledWith(

526+

expect.objectContaining({

527+

provider: "openai-codex",

528+

}),

529+

);

530+

expect(postJsonRequestMock).toHaveBeenCalledWith(

531+

expect.objectContaining({

532+

url: "https://api.openai.com/v1/images/generations",

533+

}),

534+

);

535+

});

536+415537

it("sends Codex reference images as Responses input images", async () => {

416538

mockCodexAuthOnly();

417539

mockCodexImageStream();