Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 17, 2026

  • Delete the src/Extensions.Grok/ directory
  • Delete the test file src/Tests/GrokTests.cs
  • Remove Extensions.Grok project reference from solution file Extensions.AI.slnx
  • Remove Extensions.Grok project reference from src/Extensions/Extensions.csproj
  • Add xAI nuget package reference to src/Extensions/Extensions.csproj (moved to correct ItemGroup)
  • Remove Extensions.Grok project reference from src/Tests/Tests.csproj
  • Update namespace references in src/Extensions/ChatClientProviders.cs from Devlooped.Extensions.AI.Grok to xAI
  • Update namespace references in src/Agents/ConfigurableAIAgent.cs from Devlooped.Extensions.AI.Grok to xAI
  • Update readme.md to replace the Grok section with a reference to xAI package
  • Verify the build passes
  • Run code review
  • Run security check (CodeQL)
Original prompt

Replace the Extensions.Grok project with the nuget package xAI which provides the same behavior now. Delete the entire project and any test files related to it.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: kzu <169707+kzu@users.noreply.github.com>
@kzu kzu marked this pull request as ready for review January 17, 2026 19:11
@kzu
Copy link
Member

kzu commented Jan 17, 2026

12 passed 12 passed 13 skipped

🧪 Details on Ubuntu 24.04.3 LTS

from retest v1.1.0 on .NET 10.0.2 with 💜 by @devlooped

Copilot AI changed the title [WIP] Remove Extensions.Grok project and migrate to xAI package Replace Extensions.Grok project with xAI nuget package Jan 17, 2026
Copilot AI requested a review from kzu January 17, 2026 19:13
Copy link
Member

@kzu kzu left a comment

Choose a reason for hiding this comment

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

Check if the xAI namespace is necessary at all in the ConfigurableAIAgent since we now use providers for the creation of the specific clients and there shouldn't be anything xAI/Grok specific there?
Also, package references should go in their own ItemGroup in MSBuild, alongside other PackageReference elements, just like ProjectReference.

Co-authored-by: kzu <169707+kzu@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 17, 2026

Check if the xAI namespace is necessary at all in the ConfigurableAIAgent since we now use providers for the creation of the specific clients and there shouldn't be anything xAI/Grok specific there?
...

The xAI namespace is still needed in ConfigurableAIAgent.cs for GrokChatOptions (lines 119-120) - it provides Grok-specific properties like Search and Include that need to be deserialized from config when the provider is "xai". Without it, users configuring Grok-specific options would lose those settings.

Moved the xAI PackageReference to the correct ItemGroup with other PackageReferences in fe62bb2.

Copilot AI requested a review from kzu January 17, 2026 19:16
@kzu kzu enabled auto-merge (rebase) January 17, 2026 19:26
@kzu kzu merged commit eaf2073 into main Jan 17, 2026
4 checks passed
@kzu kzu deleted the copilot/remove-extensions-grok-project branch January 17, 2026 19:27
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