Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Dec 20, 2025

This is the pattern followed by the OpenAI implementation too. Allows exposing ALL the underlying options, albeit quite a bit less discoverable.

This should be sufficient fallback for any new (auto-updated) features in the upstream GrokClient generated from .proto files if we ever lag behind it on this library.

We make the test fully isolated from xAI calls to avoid spending tokens on a test that doesn't really require hitting the backend (unlike most of the others).

@kzu kzu added the enhancement New feature or request label Dec 20, 2025
@kzu
Copy link
Member Author

kzu commented Dec 20, 2025

42 passed 42 passed 3 skipped

🧪 Details on Ubuntu 24.04.3 LTS

from dotnet-retest v1.0.0 on .NET 10.0.1 with 💜 by @devlooped

@kzu kzu force-pushed the dev/rawrepresentation branch 3 times, most recently from 873be19 to 96c8d05 Compare December 20, 2025 18:09
This is the pattern followed by the OpenAI implementation too. Allows exposing ALL the underlying options, albeit quite a bit less discoverable.

This should be sufficient fallback for any new (auto-updated) features in the upstream GrokClient generated from .proto files if we ever lag behind it on this library.

We make the test fully isolated from xAI calls to avoid spending tokens on a test that doesn't really require hitting the backend (unlike most of the others).
@kzu kzu force-pushed the dev/rawrepresentation branch from 96c8d05 to e8aab11 Compare December 20, 2025 18:09
@kzu kzu enabled auto-merge (rebase) December 20, 2025 18:09
@kzu kzu merged commit 8dc74ec into main Dec 20, 2025
7 checks passed
@kzu kzu deleted the dev/rawrepresentation branch December 20, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants