Skip to content

feat(plugin): Add result accessor to OperationEndInfo - #596

Open
ayushiahjolia wants to merge 1 commit into
mainfrom
feat/plugin-operation-end-result
Open

feat(plugin): Add result accessor to OperationEndInfo#596
ayushiahjolia wants to merge 1 commit into
mainfrom
feat/plugin-operation-end-result

Conversation

@ayushiahjolia

Copy link
Copy Markdown
Contributor

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Issue Link, if available

#579

Description

OperationEndInfo exposed error() for failed operations but had no way for plugins to access the serialized result of successful operations.

  • Added String result field to the OperationEndInfo record
  • Added extractResult(Operation) in PluginInfoConverter that pulls the serialized result from the operation's type-specific details (stepDetails, callbackDetails, chainedInvokeDetails, contextDetails)

Demo/Screenshots

N/A

Checklist

  • I have filled out every section of the PR template
  • I have thoroughly tested this change

Testing

Unit Tests

Have unit tests been written for these changes? Added

Integration Tests

Have integration tests been written for these changes? Added

Examples

Has a new example been added for the change? (if applicable) N/A

@ayushiahjolia
ayushiahjolia deployed to ai-pr-review August 5, 2026 23:49 — with GitHub Actions Active
@ayushiahjolia
ayushiahjolia requested a deployment to ai-pr-review-runtime August 5, 2026 23:49 — with GitHub Actions Waiting
@ayushiahjolia
ayushiahjolia requested a deployment to ai-pr-review-runtime August 5, 2026 23:49 — with GitHub Actions Waiting
@ayushiahjolia
ayushiahjolia marked this pull request as ready for review August 5, 2026 23:51
@ayushiahjolia
ayushiahjolia requested a review from a team August 5, 2026 23:51
@ayushiahjolia
ayushiahjolia enabled auto-merge (squash) August 5, 2026 23:51
@ayushiahjolia
ayushiahjolia marked this pull request as draft August 5, 2026 23:53
auto-merge was automatically disabled August 5, 2026 23:53

Pull request was converted to draft

@ayushiahjolia
ayushiahjolia force-pushed the feat/plugin-operation-end-result branch from 70c49fd to 85d282c Compare August 5, 2026 23:55
@ayushiahjolia
ayushiahjolia marked this pull request as ready for review August 5, 2026 23:57
@ayushiahjolia
ayushiahjolia enabled auto-merge (squash) August 5, 2026 23:57
boolean isReplay,
Throwable error) {}
Throwable error,
String result) {}

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.

Why serialized result?

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