Skip to content

.NET: fix: use allow list for configuration keys - #8200

Draft
Vincent Biret (baywet) wants to merge 17 commits into
mainfrom
fix/configuration-keys
Draft

.NET: fix: use allow list for configuration keys#8200
Vincent Biret (baywet) wants to merge 17 commits into
mainfrom
fix/configuration-keys

Conversation

@baywet

@baywet Vincent Biret (baywet) commented Sep 9, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Vincent Biret vibiret@microsoft.com

@baywet
fix: use allow list for configuration keys
e803079 
Signed-off-by: Vincent Biret <vibiret@microsoft.com>
Copilot AI balanced review requested due to automatic review settings September 9, 2026 16:46
@baywet Vincent Biret (baywet) self-assigned this Sep 9, 2026
@agent-framework-automation agent-framework-automation Bot added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Sep 9, 2026
@github-actions github-actions Bot changed the title fix: use allow list for configuration keys .NET: fix: use allow list for configuration keys Sep 9, 2026
github-code-quality[bot]
github-code-quality Bot found potential problems Sep 9, 2026
View reviewed changes
Comment thread dotnet/src/Microsoft.Agents.AI.Workflows.Declarative/PowerFx/WorkflowExpressionEngine.cs Fixed
Comment thread dotnet/src/Microsoft.Agents.AI.Workflows.Declarative/PowerFx/WorkflowExpressionEngine.cs Fixed
Comment thread dotnet/src/Microsoft.Agents.AI.Declarative/PromptAgentFactory.cs Fixed

This comment was marked as outdated.

Sign in to view

github-actions[bot]

This comment was marked as outdated.

Sign in to view

@baywet @github-code-quality
Apply batched suggestions from code review
d2842d4 
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@github-code-quality

This comment was marked as outdated.

Sign in to view
@baywet @github-code-quality
Potential fix for pull request finding 'Missed opportunity to use Sel…
2e5089e 
…ect'

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
@baywet
Address declarative workflow review feedback
181ce0d 
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
github-code-quality[bot]
github-code-quality Bot found potential problems Sep 9, 2026
View reviewed changes
Comment thread dotnet/src/Microsoft.Agents.AI.Declarative/ChatClient/ChatClientPromptAgentFactory.cs Fixed
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot September 10, 2026 13:29

This comment was marked as outdated.

Sign in to view

@baywet
Fix declarative sensitivity review comments
9a551d0 
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
@baywet
Vincent Biret (baywet) requested a balanced review from Copilot September 10, 2026 17:52

This comment was marked as outdated.

Sign in to view

@baywet
Fix declarative review feedback
1ea1ed2 
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>

This comment was marked as outdated.

Sign in to view
Copilot AI deployed to github-app-auth September 11, 2026 18:32 Active
@baywet

Vincent Biret (baywet) commented Sep 11, 2026

Copy link
Copy Markdown
Member Author

Copilot resolve the merge conflicts in this pull request

Copilot AI deployed to integration September 11, 2026 18:33 Active
Copilot AI and others added 3 commits September 11, 2026 18:40
@baywet
Merge remote-tracking branch 'origin/main' into fix/configuration-keys
c7694fe 
# Conflicts:
#	dotnet/src/Microsoft.Agents.AI.Workflows.Declarative/DeclarativeWorkflowBuilder.cs
#	dotnet/src/Microsoft.Agents.AI.Workflows.Declarative/PowerFx/WorkflowFormulaState.cs

Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
@baywet
Merge main and resolve conflicts
8a23310 
Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>
@baywet
Revert "Merge main and resolve conflicts"
93cc03e 
This reverts commit 8a23310.

Co-authored-by: baywet <7905502+baywet@users.noreply.github.com>

Copilot AI commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

Resolved in c7694fe by merging origin/main and fixing the declarative workflow conflicts. Final head 93cc03e removes temporary SDK workaround files from the tree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments
Copilot code review Copilot
Copilot review effort, defaults to Balanced
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
Copilot code review Copilot Copilot left review comments
@eavanvalkenburg Eduard van Valkenburg (eavanvalkenburg) Awaiting requested review from eavanvalkenburg eavanvalkenburg will be requested when the pull request is marked ready for review eavanvalkenburg is a code owner
@SergeyMenshykh SergeyMenshykh Awaiting requested review from SergeyMenshykh SergeyMenshykh will be requested when the pull request is marked ready for review SergeyMenshykh is a code owner
@rogerbarreto Roger Barreto (rogerbarreto) Awaiting requested review from rogerbarreto rogerbarreto will be requested when the pull request is marked ready for review rogerbarreto is a code owner
@westey-m westey (westey-m) Awaiting requested review from westey-m westey-m will be requested when the pull request is marked ready for review westey-m is a code owner
@moonbox3 Evan Mattson (moonbox3) Awaiting requested review from moonbox3 moonbox3 will be requested when the pull request is marked ready for review moonbox3 is a code owner
@peibekwe Peter Ibekwe (peibekwe) Awaiting requested review from peibekwe peibekwe will be requested when the pull request is marked ready for review peibekwe is a code owner
+1 more reviewer
@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@baywet Vincent Biret (baywet)

Labels

.NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@baywet