Skip to content

Switch Grok and OpenCode Go Responses to the shared packages - #862

Merged
TheGreatAxios merged 6 commits into
mainfrom
cl-7423-corbits-code-switch-the-grok-and-opencode-go-responses
Sep 10, 2026
Merged

Switch Grok and OpenCode Go Responses to the shared packages#862
TheGreatAxios merged 6 commits into
mainfrom
cl-7423-corbits-code-switch-the-grok-and-opencode-go-responses

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

Host Grok and OpenCode Go Responses adapters bake @corbits/xai-provider and @corbits/openai-responses factories while keeping provider ids and option keys. Grok streams terminate on response.completed via the package isStreamTerminal. In-tree Grok and OpenCode Go Responses adapter forks are gone.

The host adapters were forks of the Codex Responses client. The packages now own that protocol, including Grok's x-grok-* headers and the missing stream-terminal check. The host Grok adapter never ended on response.completed, so streams could hang waiting for [DONE].

  • Provider ids stay grok-responses and openai-responses. The package's openai-compatible-responses id is not registered, so stored reasoning signatures stay valid.
  • Host option keys stay; the factory maps them to package sessionIdOption / fromOption.
  • Exactly one @intx/inference copy must resolve (packages declare it as a peer).
  • Host retains billing URL, discovery URL, usage snapshots, and the src/auth/xai TUI (those move under CL-7424, not here).

Verification

  • bun run typecheck, bun run build, and bun run test pass
  • bun run check passes
  • Adapter tests assert the same wire body against the package, plus isStreamTerminal present

Fixes CL-7423

@linear-code

linear-code Bot commented Sep 10, 2026

Copy link
Copy Markdown

CL-7423

@TheGreatAxios
TheGreatAxios force-pushed the cl-7423-corbits-code-switch-the-grok-and-opencode-go-responses branch from 556ef78 to 1ca081f Compare September 10, 2026 13:29
@TheGreatAxios
TheGreatAxios merged commit 52e811b into main Sep 10, 2026
9 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.

1 participant