Skip to content

Reasoning integration tests - #133

Merged
mikeperry-tor merged 5 commits into
13rac1:mainfrom
mikeperry-tor:reasoning_integration_tests
Jul 15, 2026
Merged

Reasoning integration tests#133
mikeperry-tor merged 5 commits into
13rac1:mainfrom
mikeperry-tor:reasoning_integration_tests

Conversation

@mikeperry-tor

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings July 15, 2026 20:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds integration coverage for “reasoning” responses (including GLM-specific repair scenarios) across multiple proxy/provider integration suites, ensuring both plaintext and E2EE pathways handle reasoning fields and multimodal content decoding correctly.

Changes:

  • Introduces shared reasoning integration helpers (runReasoningResponseTests, runGLMReasoningRepairTests) and assertions for both non-stream and SSE streaming responses.
  • Adds reasoning/repair subtests for Tinfoil (cloud + direct) and NEAR (nearcloud + neardirect) integration suites.
  • Adjusts some provider-specific integration POST helpers to include reasoning_effort control and provider-specific tool-call request shapes.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

File Description
internal/proxy/integration_tinfoil_test.go Adds reasoning/repair subtests for Tinfoil and introduces provider-specific chat/tool POST helpers used by the Tinfoil API surface tests.
internal/proxy/integration_reasoning_test.go New shared reasoning test helpers: posting reasoning requests, GLM repair scenarios, and response/SSE validation.
internal/proxy/integration_neardirect_test.go Adds GLM reasoning integration test and switches NEAR Direct chat requests to a helper that explicitly sets reasoning_effort.
internal/proxy/integration_nearcloud_test.go Adds GLM reasoning integration test coverage for NEAR Cloud (plaintext + E2EE, plus repairs).

Comment thread internal/proxy/integration_tinfoil_test.go
Comment thread internal/proxy/integration_tinfoil_test.go
Comment thread internal/proxy/integration_reasoning_test.go
Comment thread internal/proxy/integration_reasoning_test.go
Comment thread internal/proxy/integration_reasoning_test.go
Comment thread internal/proxy/integration_reasoning_test.go

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

Comment thread internal/proxy/integration_tinfoil_test.go
Comment thread internal/proxy/integration_reasoning_test.go

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@mikeperry-tor
mikeperry-tor merged commit 8017400 into 13rac1:main Jul 15, 2026
1 check 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.

2 participants