Skip to content

fix(sse): route Poe API-key traffic through DefaultExecutor (#8969) - #9014

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
Prudhvivuda:fix/8969-poe-api-executor
Aug 6, 2026
Merged

fix(sse): route Poe API-key traffic through DefaultExecutor (#8969)#9014
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.50from
Prudhvivuda:fix/8969-poe-api-executor

Conversation

@Prudhvivuda

@Prudhvivuda Prudhvivuda commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Stop aliasing canonical poe to PoeWebExecutor so API-key connections use DefaultExecutor against api.poe.com instead of the web GraphQL /api/gql_POST path that returned HTTP 405.
  • Register Poe Chat Completions, Responses, and Claude-only Messages URLs; select the upstream surface from the resolved protocol, and never send GPT models to /v1/messages.
  • Add regression coverage for executor selection, URL normalization, Bearer auth (no Cookie), streaming/non-streaming mocks, and preserved upstream 405s.

Closes #8969

Test plan

  • node --import tsx/esm --test tests/unit/poe-api-executor-regression.test.ts
  • node --import tsx/esm --test tests/unit/chatcore-execution-credentials.test.ts
  • node --import tsx/esm --test tests/unit/executor-poe-web.test.ts
  • node --import tsx/esm --test tests/unit/poe-provider-models-baseurl.test.ts
  • node --import tsx/esm --test tests/unit/web-cookie-providers-new.test.ts
  • Confirm poe-web still uses PoeWebExecutor / cookie GraphQL path
  • Optional live smoke: poe/<model> Chat Completions with a real API key returns 200 (not 405)

…zapw#8969)

Stop aliasing canonical `poe` to PoeWebExecutor so API-key requests hit
api.poe.com Chat Completions / Responses / Claude-only Messages instead of
the web GraphQL path that returned HTTP 405.
@Prudhvivuda
Prudhvivuda force-pushed the fix/8969-poe-api-executor branch from b4767d2 to 60299ad Compare August 5, 2026 14:29
@diegosouzapw
diegosouzapw merged commit 701d60d into diegosouzapw:release/v3.8.50 Aug 6, 2026
9 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(providers): Poe API-key requests routed through PoeWebExecutor return 405

2 participants