Visitar URL original
Remove GitHub models provider from project templates by jeffhandley · Pull Request #7667 · dotnet/extensions · GitHub
Skip to content

Remove GitHub models provider from project templates - #7667

Merged
jeffhandley merged 5 commits into
dotnet:mainfrom
jeffhandley:jeffhandley/remove-github-models
Jul 29, 2026
Merged

Remove GitHub models provider from project templates#7667
jeffhandley merged 5 commits into
dotnet:mainfrom
jeffhandley:jeffhandley/remove-github-models

Conversation

@jeffhandley

@jeffhandley jeffhandley commented Jul 29, 2026 •
edited by dotnet-policy-service Bot
Loading

Copy link
Copy Markdown
Member

jeffhandley and others added 2 commits July 28, 2026 16:42
@jeffhandley
Remove GitHub Models from aichatweb template
abaa484 
The provider argument becomes required with no default

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc263661-1b95-4f80-a7f4-3a4d3cd275ff
@jeffhandley
Remove GitHub Models from aiagent-webapi template
89305a1 
The provider argument becomes required with no default

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc263661-1b95-4f80-a7f4-3a4d3cd275ff
@jeffhandley
jeffhandley requested a review from jozkee July 29, 2026 15:07
@jeffhandley jeffhandley self-assigned this Jul 29, 2026
Copilot AI review requested due to automatic review settings July 29, 2026 15:07
@jeffhandley
jeffhandley requested a review from a team as a code owner July 29, 2026 15:07
Copilot AI reviewed Jul 29, 2026
View reviewed changes

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

This PR updates the .NET AI project templates to remove the retired GitHub Models provider and aligns template docs/tests/snapshots accordingly.

Changes:

  • Remove githubmodels as a selectable --provider in aichatweb and aiagent-webapi templates (docs + conditional code).
  • Update integration tests to stop using GitHub Models defaults and to enumerate only the remaining providers.
  • Refresh snapshot baselines by removing the GitHub Models-based verified outputs.

Reviewed changes

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

Show a summary per file
File Description
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/.template.config/template.json Removes GitHub Models choice and marks provider selection as required.
src/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates/templates/AIAgentWebApi-CSharp/.template.config/template.json Removes GitHub Models choice and marks provider selection as required.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.Web/Program.cs Removes GH Models branch from generated app wiring.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.Web/Program.Aspire.cs Removes GH Models branch from Aspire wiring.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.Web/README.md Removes GH Models instructions from template README.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/README.Aspire.md Removes GH Models instructions from Aspire README.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.AppHost/AppHost.cs Removes GH Models connection-string guidance/branching for Aspire AppHost.
src/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates/templates/AIAgentWebApi-CSharp/Program.cs Removes GH Models branch from generated app wiring.
src/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates/templates/AIAgentWebApi-CSharp/README.md Removes GH Models prerequisites/setup/troubleshooting from template README.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.Web/AIChatWeb-CSharp.Web.csproj-in Removes GH Models-related conditional package reference paths.
src/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates/templates/AIAgentWebApi-CSharp/AIAgentWebApi-CSharp.csproj-in Removes GH Models from conditional OpenAI package reference gate.
src/ProjectTemplates/README.md Updates provider option list to exclude GitHub Models.
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/AIChatWebSnapshotTests.cs Updates snapshot test scenarios to use explicit non-GH providers.
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/AIChatWebExecutionTests.cs Removes githubmodels from execution test provider matrix.
test/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates.IntegrationTests/AIAgentWebAPISnapshotTests.cs Updates snapshot test scenarios to use explicit non-GH providers.
test/ProjectTemplates/Microsoft.Agents.AI.ProjectTemplates.IntegrationTests/AIAgentWebAPIExecutionTests.cs Removes githubmodels from execution test provider matrix.
test/ProjectTemplates//Snapshots/ Removes GitHub Models-based verified snapshot outputs and introduces/updates non-GH snapshots (where present).
Comments suppressed due to low confidence (2)

src/ProjectTemplates/README.md:152

  • Same as above for the aichatweb example: it still shows --provider as optional, but the template configuration now requires it. Update the usage example (or restore a default provider) so docs and template behavior stay aligned.
    dotnet new aichatweb `
        [--provider <azureopenai | ollama | openai>] `
        [--vector-store <azureaisearch | local | qdrant>] `
        [--aspire] `
        [--managed-identity]

test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/AIChatWebSnapshotTests.cs:34

  • The snapshot case --aspire + --provider=openai will look for Snapshots/aichatweb/aichatweb.A_oai.verified, but no such snapshot baseline exists in the current snapshot set. Add the missing baseline (or adjust/remove this test case) to avoid a failing snapshot run.
    [InlineData("--aspire", "--provider=openai")]
    [InlineData("--aspire", "--provider=azureopenai", "--vector-store=azureaisearch")]

Comment thread src/ProjectTemplates/README.md Resolved
Comment thread .../Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/.template.config/template.json Resolved
Comment thread ...oft.Agents.AI.ProjectTemplates/templates/AIAgentWebApi-CSharp/.template.config/template.json Resolved
Comment thread ...rojectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/AIChatWebSnapshotTests.cs Resolved
@jeffhandley
Document required AI provider
3ef4494 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7
@jeffhandley
Add OpenAI template snapshots
62d358f 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7
@jeffhandley jeffhandley added the breaking-change Issue or PR that represents a breaking API or functional change over a previous release. label Jul 29, 2026
@jeffhandley
Update agent template snapshots
35a7a34 
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7
jeffhandley
jeffhandley commented Jul 29, 2026
View reviewed changes

@jeffhandley jeffhandley left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Some of the baselines look like renames or renames with changes with how git processes the diff, but they were really deletions of github baselines and additions of new baselines for openai.

@jeffhandley
jeffhandley merged commit e240b28 into dotnet:main Jul 29, 2026
5 checks passed
@jeffhandley
jeffhandley deleted the jeffhandley/remove-github-models branch July 29, 2026 18:50
@jeffhandley jeffhandley mentioned this pull request Jul 29, 2026
Merged
jeffhandley added a commit that referenced this pull request Jul 29, 2026
@jeffhandley
Remove GitHub models provider from project templates (#7667)
bbf67c4 
* Remove GitHub Models from aichatweb template

The provider argument becomes required with no default

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc263661-1b95-4f80-a7f4-3a4d3cd275ff

* Remove GitHub Models from aiagent-webapi template

The provider argument becomes required with no default

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc263661-1b95-4f80-a7f4-3a4d3cd275ff

* Document required AI provider

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7

* Add OpenAI template snapshots

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7

* Update agent template snapshots

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: dc263661-1b95-4f80-a7f4-3a4d3cd275ff
Copilot-Session: f24c9c9d-985a-4bf4-865a-a7ea086a23e7
This was referenced Aug 11, 2026
Closed
Merged
Closed
Merged
Merged
Merged
Merged
Closed
Merged
Closed
Closed
Closed
This was referenced Aug 27, 2026
Closed
Closed
Closed
Merged
Merged
Merged
Closed
Merged
Merged
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Merged
Closed
Closed
Closed
Merged
Closed
Closed
Closed
Closed
Closed
Merged
Merged
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@jozkee jozkee jozkee approved these changes

Assignees

@jeffhandley jeffhandley

Labels

breaking-change Issue or PR that represents a breaking API or functional change over a previous release.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jeffhandley @jozkee