Skip to content

.NET Workflows - Introduce support for Local variable scope - #944

Merged
Chris (crickman) merged 12 commits into
mainfrom
crickman/workflows-declarative-objectmodel-update-scope
Sep 26, 2025
Merged

.NET Workflows - Introduce support for Local variable scope#944
Chris (crickman) merged 12 commits into
mainfrom
crickman/workflows-declarative-objectmodel-update-scope

Conversation

@crickman

@crickman Chris (crickman) commented Sep 26, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Motivation and Context

Integrate latest Object Model update that enables Local scope as an alias for Topic.

Description

  • Support Local as active default scope.
  • Update tests
  • Add mixed usage test

👉 SAMPLES UPDATED 👈

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

@crickman Chris (crickman) self-assigned this Sep 26, 2025
Copilot AI review requested due to automatic review settings September 26, 2025 17:20
@crickman Chris (crickman) added enhancement .NET Usage: [Issues, PRs], Target: .Net labels Sep 26, 2025
@crickman Chris (crickman) added the workflows Usage: [Issues, PRs], Target: Workflows label Sep 26, 2025
Copilot AI reviewed Sep 26, 2025
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 introduces support for the Local variable scope in .NET workflows, replacing the previous default scope of Topic with Local. The change affects variable declarations, references, and internal workflow state management to use the new scoping model.

Key Changes:

  • Updated default scope from Topic to Local in workflow execution context
  • Modified all sample workflows and test files to use Local scope instead of Topic
  • Added backward compatibility by maintaining Topic scope support alongside Local

Reviewed Changes

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

Show a summary per file
File Description
workflow-samples/*.yaml Updated variable declarations to use Local scope
dotnet/tests/.../Workflows/*.yaml Modified test workflows to use Local scope
WorkflowFormulaState.cs Changed default scope to Local and added scope mapping
WorkflowTest.cs Updated test utilities to use new default scope
Various test files Updated test code to use Local scope references

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread dotnet/src/Microsoft.Agents.Workflows.Declarative/PowerFx/WorkflowFormulaState.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.Workflows.Declarative/PowerFx/WorkflowFormulaState.cs Resolved
Comment thread dotnet/src/Microsoft.Agents.Workflows.Declarative/PowerFx/WorkflowFormulaState.cs Resolved
@crickman Chris (crickman) moved this to In Review in Agent Framework Sep 26, 2025
@crickman
Chris (crickman) added this pull request to the merge queue Sep 26, 2025
Merged via the queue into main with commit 4d63899 Sep 26, 2025
@github-project-automation github-project-automation Bot moved this from In Review to Done in Agent Framework Sep 26, 2025
@crickman
Chris (crickman) deleted the crickman/workflows-declarative-objectmodel-update-scope branch September 30, 2025 18:52
@crickman Chris (crickman) mentioned this pull request Oct 1, 2025
Closed
Reuben Bond (ReubenBond) pushed a commit to ReubenBond/agent-framework that referenced this pull request Oct 28, 2025
@crickman
.NET Workflows - Introduce support for Local variable scope (micros…
187fb88 
…oft#944)

* Prepare for update

* Checkpoint

* Comments

* All fixed

* Mixed casre test added

* Rollback nuget

* Remove redundant restorable

* Namespace
@moonbox3 Evan Mattson (moonbox3) added declarative Usage: [Issues, PRs], Target: declarative agents and workflows and removed declarative-workflow labels Jun 17, 2026

This branch was previously deployed

1 inactive deployment
integration — 70d6f719 Deployed Sep 26, 2025 by crickman
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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
@alliscode Ben Thomas (alliscode) alliscode approved these changes
@RaviPidaparthi Ravi Teja Pidaparthi (RaviPidaparthi) Awaiting requested review from RaviPidaparthi
@pradeepkintali Pradeep Kintali (pradeepkintali) Awaiting requested review from pradeepkintali
+1 more reviewer
@TaoChenOSU Tao Chen (TaoChenOSU) TaoChenOSU approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@crickman Chris (crickman)

Labels

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

Projects

Agent Framework
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@crickman @TaoChenOSU @alliscode @moonbox3