Skip to content

Compile native LLM requests in session tests#26946

Draft
kitlangton wants to merge 1 commit into
llm-native-request-adapterfrom
llm-native-prepare-tests
Draft

Compile native LLM requests in session tests#26946
kitlangton wants to merge 1 commit into
llm-native-request-adapterfrom
llm-native-prepare-tests

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 11, 2026

Summary

  • register native LLM provider routes from the session request adapter
  • add a focused LLMClient.prepare test that compiles session-native requests through OpenAI Responses lowering
  • keep this offline: no provider calls or runtime switching

Test Plan

  • bun run test -- test/session/llm-native.test.ts test/session/llm.test.ts --test-name-pattern native
  • bun typecheck
  • bunx oxlint packages/opencode/src/session/llm.ts packages/opencode/src/session/llm-native.ts packages/opencode/test/session/llm.test.ts packages/opencode/test/session/llm-native.test.ts
  • git diff --check

Stack

  1. Consume native LLM events in session processing #26639
  2. Add native LLM request adapter #26941
  3. Compile native LLM requests in session tests #26946 👈 current
  4. Add native OpenAI runtime opt-in #26947

@kitlangton kitlangton marked this pull request as draft May 11, 2026 21:44
@kitlangton kitlangton force-pushed the llm-native-prepare-tests branch from 6e9c01f to b93cb0e Compare May 11, 2026 22:06
@kitlangton kitlangton force-pushed the llm-native-request-adapter branch 2 times, most recently from 7fff570 to ec718a5 Compare May 11, 2026 22:07
@kitlangton kitlangton force-pushed the llm-native-prepare-tests branch from b93cb0e to 3c7ba1d Compare May 11, 2026 22:07
@kitlangton kitlangton force-pushed the llm-native-request-adapter branch from ec718a5 to 93e1043 Compare May 12, 2026 01:32
@kitlangton kitlangton force-pushed the llm-native-prepare-tests branch from 3c7ba1d to 7b82011 Compare May 12, 2026 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant