Visitar URL original
.NET: Fix CheckpointInfo.Parent always null in InProcessRunner by lokitoth · Pull Request #3812 · microsoft/agent-framework · GitHub
Skip to content

.NET: Fix CheckpointInfo.Parent always null in InProcessRunner - #3812

Merged
Jacob Alber (lokitoth) merged 1 commit into
mainfrom
dev/dotnet_workflow/Fix-CheckpointParent
Feb 19, 2026
Merged

.NET: Fix CheckpointInfo.Parent always null in InProcessRunner#3812
Jacob Alber (lokitoth) merged 1 commit into
mainfrom
dev/dotnet_workflow/Fix-CheckpointParent

Conversation

@lokitoth

@lokitoth Jacob Alber (lokitoth) commented Feb 10, 2026

Copy link
Copy Markdown
Contributor

Motivation and Context

We added Checkpoint.Parent to have lineage information, and to enable a better visualization of the various restorable points in the history, particularly during TimeTravel Debugging, but we never filled it in.

Description

Track the last CheckpointInfo in InProcessRunner so that newly created checkpoints reference their parent. When resuming from a checkpoint, the resumed-from checkpoint becomes the parent of the next checkpoint.

Fixes #3796

Adds tests verifying:

  • First checkpoint has null parent
  • Subsequent checkpoints chain parents correctly
  • Checkpoint after resume references the resumed-from checkpoint

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.

@lokitoth Jacob Alber (lokitoth) added .NET Usage: [Issues, PRs], Target: .Net workflows Usage: [Issues, PRs], Target: Workflows labels Feb 10, 2026
@lokitoth Jacob Alber (lokitoth) moved this to In Progress in Agent Framework Feb 10, 2026
@lokitoth
Jacob Alber (lokitoth) marked this pull request as ready for review February 10, 2026 15:08
Copilot AI review requested due to automatic review settings February 10, 2026 15:08
@lokitoth Jacob Alber (lokitoth) moved this from In Progress to In Review in Agent Framework Feb 10, 2026
Copilot AI reviewed Feb 10, 2026
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

Fixes missing checkpoint lineage in the .NET in-process workflow runner by populating Checkpoint.Parent so checkpoints form a navigable chain (including after resume), improving restore-point visualization and time-travel debugging.

Changes:

  • Track the previously committed checkpoint in InProcessRunner and pass it as the parent when committing the next checkpoint.
  • When restoring from a checkpoint, set that checkpoint as the parent of the next checkpoint created.
  • Add unit tests intended to validate parent chaining across normal execution and resume scenarios.

Reviewed changes

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

File Description
dotnet/src/Microsoft.Agents.AI.Workflows/InProc/InProcessRunner.cs Track and apply the last checkpoint info as Checkpoint.Parent, including after restore.
dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Adds tests for parent = null on first checkpoint, chaining on subsequent checkpoints, and parent after resume.

Comment thread dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Resolved
Comment thread dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Resolved
Comment thread dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Resolved
Comment thread dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Resolved
Comment thread dotnet/tests/Microsoft.Agents.AI.Workflows.UnitTests/CheckpointParentTests.cs Resolved
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from a49bb6b to d3d9eef Compare February 10, 2026 15:52
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from d3d9eef to 3ef3001 Compare February 10, 2026 16:16
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from 3ef3001 to 9edf1a6 Compare February 11, 2026 15:04
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from 9edf1a6 to 39a4478 Compare February 11, 2026 15:33
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from 39a4478 to 8ac7da8 Compare February 17, 2026 15:22
Jacob Alber
Fix CheckpointInfo.Parent always null in InProcessRunner (#3796)
bbdc91f 
Track the last CheckpointInfo in InProcessRunner so that newly created
checkpoints reference their parent. When resuming from a checkpoint,
the resumed-from checkpoint becomes the parent of the next checkpoint.

Adds tests verifying:
- First checkpoint has null parent
- Subsequent checkpoints chain parents correctly
- Checkpoint after resume references the resumed-from checkpoint
@lokitoth
Jacob Alber (lokitoth) force-pushed the dev/dotnet_workflow/Fix-CheckpointParent branch from 8ac7da8 to bbdc91f Compare February 19, 2026 13:20
This was referenced Mar 9, 2026
Closed
Closed
Open
This was referenced Mar 16, 2026
Closed
Merged
Closed
Open
Closed
Open
Closed
Closed
Closed
This was referenced Mar 23, 2026
Merged
Closed
This was referenced Apr 6, 2026
Closed
Open
Open
@dependabot dependabot Bot mentioned this pull request Apr 24, 2026
Merged
This was referenced May 2, 2026
Closed
Closed
This was referenced May 9, 2026
Closed
Closed
Closed
Closed
Closed
This was referenced May 18, 2026
Closed
Closed
@dependabot dependabot Bot mentioned this pull request Jun 10, 2026
Closed
@dependabot dependabot Bot mentioned this pull request Jun 24, 2026
Closed
@dependabot dependabot Bot mentioned this pull request Jul 15, 2026
Open

This branch was previously deployed

1 inactive deployment
integration — bbdc91f4 Deployed Feb 19, 2026 by lokitoth via dotnet-build-and-test (net10.0, ubuntu-latest, Release, true, integration) #12424
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
@westey-m westey (westey-m) westey-m approved these changes
+1 more reviewer
@markwallace-microsoft Mark Wallace (markwallace-microsoft) markwallace-microsoft approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

.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.

.NET: [Bug]: The parent parameters of the methods RetrieveIndexAsync and CreateCheckpointAsync in the ICheckpointStore interface are always null.

4 participants

@lokitoth @markwallace-microsoft @westey-m