[PR #17540] Allow OpenAI to be used with Terminal Chat #31263

Open
opened 2026-01-31 09:46:09 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/17540

State: closed
Merged: Yes


  • Implements OpenAILLMProvider, which is an implementation of ILMProvider that uses OpenAI
  • Implements an AIConfig on the settings model side, to allow the user to specify which AI provider to use as their current active one (in the case that they have configured more than one LMProvider)

The OpenAI implementation is largely the same as the Azure OpenAI one. The more "new" change in this PR is the AIConfig struct on the settings model side that allows the user to specify which provider is the active one, as well as the logic in TerminalPage for how we update the current active provider based on settings changes

Validation Steps Performed

  • Able to set OpenAI as the active provider
  • OpenAI works in Terminal Chat
**Original Pull Request:** https://github.com/microsoft/terminal/pull/17540 **State:** closed **Merged:** Yes --- - Implements `OpenAILLMProvider`, which is an implementation of `ILMProvider` that uses OpenAI - Implements an `AIConfig` on the settings model side, to allow the user to specify which AI provider to use as their current active one (in the case that they have configured more than one LMProvider) The OpenAI implementation is largely the same as the Azure OpenAI one. The more "new" change in this PR is the `AIConfig` struct on the settings model side that allows the user to specify which provider is the active one, as well as the logic in `TerminalPage` for how we update the current active provider based on settings changes ## Validation Steps Performed - Able to set OpenAI as the active provider - OpenAI works in Terminal Chat
claunia added the pull-request label 2026-01-31 09:46:09 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#31263