Skip to content

Update Model to use chained invoke over invoke#67

Merged
4 commits merged intomainfrom
invoke-to-chained-invoke
Oct 20, 2025
Merged

Update Model to use chained invoke over invoke#67
4 commits merged intomainfrom
invoke-to-chained-invoke

Conversation

@ghost
Copy link

@ghost ghost commented Oct 20, 2025

  • [Chore] Update OperationType to use ChainedInvoke instead of invoke
  • [chore] Set ReplayChildren to type alias
  • [Chore] Update Model with rename of invoke to chained-invoke

Testing CR running integ tests:

TESTING_SDK_BRANCH: update-model

Quinn Sinclair and others added 3 commits October 20, 2025 00:09
Ctx:
- Model has been updated and the invoke member of the ChainedInvoke enum
  has now become ChainedInvoke over invoke.

This commit updates the model to use chainedinvoke instead.
To ease the process of updating models, we are setting up replayChildren
as a type alias.
Ctx: Internal model has been updated, replacing invokeOptions and
InvokeDetails with ChainedInvokeDetails and ChainedInvokeOptions
ghost pushed a commit to aws/aws-durable-execution-sdk-python-testing that referenced this pull request Oct 20, 2025
- Updates InvokeOptions -> ChainedInvokeOptions
- Invoke<Status>Details -> ChainedInvoke<Status>Details

This pr matches the changes done in aws/aws-durable-execution-sdk-python#67
@ghost ghost marked this pull request as ready for review October 20, 2025 16:09
Formatting etc
@ghost ghost force-pushed the invoke-to-chained-invoke branch from 21a5a0d to a6e2a09 Compare October 20, 2025 16:51
@ghost ghost merged commit fbd96b9 into main Oct 20, 2025
6 checks passed
ghost pushed a commit to aws/aws-durable-execution-sdk-python-testing that referenced this pull request Oct 20, 2025
- Updates InvokeOptions -> ChainedInvokeOptions
- Invoke<Status>Details -> ChainedInvoke<Status>Details

This pr matches the changes done in aws/aws-durable-execution-sdk-python#67
This pull request was closed.
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