Skip to content

add AiCredential objects.#121

Merged
AlexandruGal merged 5 commits into
mainfrom
task/PRC-5053/Ai-Decisional
Apr 22, 2026
Merged

add AiCredential objects.#121
AlexandruGal merged 5 commits into
mainfrom
task/PRC-5053/Ai-Decisional

Conversation

@AlexandruGal

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 22, 2026 13:13

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 initial model types to represent AI API credential configuration and structured-output behavior within Ringhel.Procesio.Action.Core.Models.Credentials.

Changes:

  • Introduces StructuredOutputMode enum for structured output/tooling behavior selection.
  • Adds AiCredentialConfiguration to hold AI API endpoint/auth/request-related settings.
  • Adds AiCredentialManager to pair AI credential config with the shared IHttpClient.

Reviewed changes

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

File Description
Ringhel.Procesio.Action.Core/Models/Credentials/AI-API/StructuredOutputMode.cs New enum describing structured output/tool usage modes.
Ringhel.Procesio.Action.Core/Models/Credentials/AI-API/AiCredentialManager.cs New manager object holding AI credential config + IHttpClient.
Ringhel.Procesio.Action.Core/Models/Credentials/AI-API/AiCredentialConfiguration.cs New configuration object for AI API credentials and defaults.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Ringhel.Procesio.Action.Core/Models/Credentials/AI-API/AiCredentialManager.cs Outdated
Comment thread Ringhel.Procesio.Action.Core/Models/Credentials/AI-API/StructuredOutputMode.cs Outdated
@AlexandruGal AlexandruGal merged commit c28f489 into main Apr 22, 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