[WhaleTech (ryo-whaletech)]

Motivation & Context

Cancelling a propagated child request with replacement function-invocation or client kwargs must apply those active settings to the nested continuation. Previously, the wrapper forwarded only runtime tools and the child retained stale kwargs. Explicit replacements must also remain distinct from omitted channels when scoping removes every replacement key.

Description & Review Guide

• What are the major changes? Reuse child-aware kwargs extraction for nested start and cancellation, and forward both kwargs channels during cancellation. Preserve supplied-but-empty values specifically for cancellation. Extend provider-free regressions for wrapper-only typed and legacy replacements, explicit empty values, both and single-channel replacements, omitted inputs, nested workflows, checkpoint restoration, and a downstream AgentExecutor.
• What is the impact of these changes? Nested cancellation applies replacement kwargs and clears the stale omitted peer when one channel is supplied. Omitting both retains previous settings. Fresh-start scoping, public signatures, and serialized state formats are unchanged. Existing child validation still applies; no exception handlers are added or changed. Additional diagnostics identified a separate validation-failure case where parent and child pending states diverge. It reproduces before the empty-value correction and is planned for a separate issue; this PR does not introduce exception suppression or rollback behavior.
• What do you want reviewers to focus on? Omitted versus supplied-but-empty cancellation kwargs, unchanged fresh-start behavior, and typed/legacy routing across nested graph boundaries.

Related Issue

Fixes #8601

Related: merged PR #8596 scopes invocation kwargs by executor graph. This change preserves those boundaries while addressing cancellation continuation. A pre-publication search found no other open PR addressing #8601.

Contribution Checklist

• The code builds clean without any errors or warnings
• All unit tests pass, and I have added new tests where possible

• The PR follows the Contribution Guidelines
• This PR is linked to an issue and there is no other open PR for this issue (see Related Issue above).
• This is not a breaking change. If it is a breaking change, add the breaking change label (or add "[BREAKING]" to the title prefix, before or after any language prefix) — a workflow keeps the label and the title prefix in sync automatically.

[Copilot]

Copilot review overview

🟡 Changes recommended

The stale omitted-channel clearing behavior needs targeted regression coverage.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 1

Open (1)

• Test continuation with only one replacement channel supplied · New

What changed in this PR

Forwards request-scoped invocation kwargs into nested workflow cancellation.

Changes:

• Extracts shared child-kwargs resolution logic.
• Adds nested cancellation regression coverage.

File	Description
_workflow_executor.py	Forwards both kwargs channels during cancellation.
test_workflow_kwargs.py	Tests replacement kwargs in child cancellation.

---

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

[Eduard van Valkenburg (eavanvalkenburg)]

WhaleTech (@ryo-whaletech) This branch currently conflicts with main. Please merge or rebase current main and resolve the conflicts before requesting another review.

[WhaleTech (ryo-whaletech)]

Eduard van Valkenburg (@eavanvalkenburg) Thanks for the heads-up — I’ve updated the branch with current main and resolved the conflict. I’ve re-requested review.