Skip to content

CROSSLINK-264 retry-action#622

Draft
adamdickmeiss wants to merge 8 commits into
mainfrom
CROSSLINK-264-ask-retry-action
Draft

CROSSLINK-264 retry-action#622
adamdickmeiss wants to merge 8 commits into
mainfrom
CROSSLINK-264-ask-retry-action

Conversation

@adamdickmeiss
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings May 29, 2026 11:58
@adamdickmeiss adamdickmeiss marked this pull request as draft May 29, 2026 11:58
Copy link
Copy Markdown
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 retry-related entries to the returnables patron-request state model for CROSSLINK-264, introducing a requester-side retry event and supplier-side retry action.

Changes:

  • Adds accept-retry event on requester SENT, transitioning to NEW.
  • Adds ask-retry action on supplier VALIDATED, transitioning to NEW.
  • Keeps generated runtime JSON in sync with misc/returnables.yaml.

Reviewed changes

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

File Description
misc/returnables.yaml Adds retry event/action declarations to the source state model.
broker/patron_request/service/statemodels/returnables.json Updates the generated embedded runtime state model with the same retry declarations.

Comment thread misc/returnables.yaml Outdated
Comment thread misc/returnables.yaml Outdated
Copilot AI review requested due to automatic review settings June 1, 2026 12:31
Copy link
Copy Markdown
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

Comment thread misc/returnables.yaml
Comment thread misc/returnables.yaml
Comment thread broker/patron_request/service/statemodels/returnables.json
Comment thread broker/patron_request/service/statemodels/returnables.json
Comment thread misc/returnables.yaml
Comment thread broker/patron_request/service/statemodel_capabilities.go
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 1, 2026 12:38
adamdickmeiss and others added 2 commits June 1, 2026 14:38
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 6 comments.

Comment thread broker/patron_request/service/statemodels/returnables.json
Comment thread misc/returnables.yaml
Comment thread misc/returnables.yaml
Comment thread misc/returnables.yaml
Comment thread broker/patron_request/service/statemodel_capabilities.go
Comment thread broker/patron_request/service/statemodels/returnables.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants