Skip to content

Conversation

@mcook-dev
Copy link

@mcook-dev mcook-dev commented Sep 16, 2025

Adding a workflow initiator property that captures the name claim of the authorized user if there is one


This change is Reviewable

@mcook-dev
Copy link
Author

@sfmskywalker

@mcook-dev
Copy link
Author

@mcook-dev
Copy link
Author

@sfmskywalker Any thoughts on this?

@sfmskywalker
Copy link
Member

@mcook-dev thanks for the PR! I'll need to find some time to review. Thank you.

Copy link
Contributor

Copilot AI left a comment

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 adds a workflow initiator property that captures the user who initiated a workflow execution, threading this information through the entire workflow system.

  • Adds Initiator property to core workflow entities and execution contexts
  • Updates API endpoints to capture the authenticated user's name as the initiator
  • Ensures the initiator information flows through all workflow creation and execution paths

Reviewed Changes

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

Show a summary per file
File Description
WorkflowExecutionContext.cs Adds Initiator property and updates constructors to accept initiator parameter
WorkflowState.cs Adds Initiator property to workflow state representation
WorkflowInstance.cs Adds Initiator property to the main workflow instance entity
WorkflowInstanceOptions.cs Adds Initiator property to workflow creation options
StartWorkflowRequest.cs Adds Initiator property to workflow start request
CreateWorkflowInstanceRequest.cs Adds Initiator property to instance creation request
CreateAndRunWorkflowInstanceRequest.cs Adds Initiator property to create-and-run request
LocalWorkflowClient.cs Updates to pass initiator through workflow creation calls
DistributedWorkflowClient.cs Updates to pass initiator through distributed workflow creation
DefaultWorkflowStarter.cs Updates to pass initiator from start request to creation request
WorkflowInstanceFactory.cs Updates to set initiator from options when creating instances
WorkflowStateExtractor.cs Updates to extract initiator from execution context
EndpointBase.cs Updates to capture authenticated user's name as initiator
WorkflowInstanceModel.cs Adds Initiator property to API model
Mappers.cs Updates mapper to include initiator in response
Api.Client models Adds Initiator property to client-side models

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

mcook-dev and others added 4 commits September 23, 2025 12:35
…xecute/EndpointBase.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…Options.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ext.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mcook-dev
Copy link
Author

@sfmskywalker Have the build dependencies been fixed so this can merge?

@sfmskywalker
Copy link
Member

Not yet, main is still out of date. Current focus lies with 3.6.0, but I’ll try to update main when I get a chance.

@mcook-dev
Copy link
Author

@sfmskywalker Anything I can help with regarding 3.6.0?

@sfmskywalker
Copy link
Member

Not at the moment, but I appreciate the offer and will keep this in mind, thank you!

@mcook-dev
Copy link
Author

@sfmskywalker no merge? that's upsetting

@sfmskywalker
Copy link
Member

@mcook-dev Not sure how this was closed, but I don’t think it was intentional. Let me reopen.

@sfmskywalker sfmskywalker reopened this Nov 11, 2025
@sfmskywalker sfmskywalker force-pushed the main branch 2 times, most recently from 4e58970 to ae1bb77 Compare December 10, 2025 17:56
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