[dotnet-port-fixes] Add checkpointed hosted-agent identity parity

[Michelle Clayton (michelle-clayton-work)]

Summary

Ports the .NET checkpoint-identity regression coverage from 6a3d5352042959eb276dc52e1e7f69a6fe5e4e8a by adding Go tests that prove workflow-agent sessions only resume across reconstruction when hosted inner agents keep stable identities. The change also adds matching checkpoint sample guidance explaining that hosted agents must be recreated with the same agent ID and, when set, the same name so hosted executor IDs remain compatible with saved checkpoints.

Ported .NET PRs

• .NET: Add regression tests and sample guidance for stable agent IDs in checkpointed workflows agent-framework#7415 - add regression tests and sample guidance for stable agent IDs in checkpointed workflows

Breaking Changes

No.

Tests and Examples

• Added TestNew_SerializedSessionWithStableHostedAgentIdentity_ResumesAcrossReconstruction
• Added TestNew_SerializedSessionWithUnstableHostedAgentIdentity_FailsAcrossReconstruction
• Added TestNew_SerializedSessionWithRenamedHostedAgent_FailsAcrossReconstruction
• Updated examples/03-workflows/checkpoint/checkpoint_and_rehydrate/main.go with hosted-agent checkpoint identity guidance
• Ran go test ./workflow/agentworkflow -run 'TestNew_SerializedSession(WithStableHostedAgentIdentity_ResumesAcrossReconstruction|WithUnstableHostedAgentIdentity_FailsAcrossReconstruction|WithRenamedHostedAgent_FailsAcrossReconstruction)$'

Notes

• Upstream commit inspected: 6a3d5352042959eb276dc52e1e7f69a6fe5e4e8a
• Recent prefixed Go tracking PRs checked included #459, #677, #583, #582, #539, #538, #445, and #395; none covered this checkpointed hosted-agent identity case.
• The worktree contains an unrelated untracked file at .github/agents/port-candidate-selector.agent.md; it is not part of this PR.

Generated by .NET to Go Fixes and Test Porting Agent · gpt54 · 346.6 AIC · ⌖ 23.9 AIC · ⊞ 24.2K · ◷

Closes #791

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Note

This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.

[George Adams (gdams)]

Copilot resolve the merge conflicts in this pull request

[Copilot]

Copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in 1f40b0f.

main had already landed equivalent checkpoint-identity regression tests (TestNew_SerializedSessionResumesHostedAgentCheckpointAcrossReconstruction, ...FailsAcrossReconstructionWithoutStableHostedAgentID, ...FailsAcrossReconstructionWhenHostedAgentNameChanges), so I dropped this branch's duplicate tests and helpers in workflow/agentworkflow/workflow_test.go in favor of main's versions. The remaining net change is the hosted-agent checkpoint identity guidance comment in examples/03-workflows/checkpoint/checkpoint_and_rehydrate/main.go.

Verified with go build ./... and go test ./... — all passing.

[github-actions]

Parity Review: ✅ Approved

This PR ports regression test coverage and matching sample comment guidance from upstream .NET commit 6a3d5352 (microsoft/agent-framework#7415).

Scope: Only workflow/agentworkflow/workflow_test.go (new tests) and a 4-line comment block in examples/03-workflows/checkpoint/checkpoint_and_rehydrate/main.go are changed. No exported Go APIs are added, removed, or modified.

Cross-repo alignment: The hosted-agent checkpoint identity constraint (stable agent ID + name → stable executor ID) is semantically aligned with the upstream .NET regression tests being ported. The sample comment faithfully reflects the same guidance present in the .NET reference.

Verdict: No parity issues found. No public API surface changed — public-api-change label is not warranted.

Generated by Go API Consistency Review Agent for #803 · sonnet46 · 17.7 AIC · ⌖ 5.56 AIC · ⊞ 5.7K · ◷