feat: add Workflow Insight plugin - #661
Merged
Merged
Conversation
wangyb-A
force-pushed
the
workflow-insight-plugin
branch
from
August 28, 2026 18:06
5d83fc9 to
9cb8201
Compare
wangyb-A
marked this pull request as ready for review
August 28, 2026 18:38
wangyb-A
temporarily deployed
to
ai-pr-review-runtime
August 28, 2026 18:38 — with
GitHub Actions
Inactive
wangyb-A
had a problem deploying
to
ai-pr-review-runtime
August 28, 2026 18:40 — with
GitHub Actions
Failure
wangyb-A
temporarily deployed
to
ai-pr-review-runtime
August 28, 2026 18:40 — with
GitHub Actions
Inactive
This comment has been minimized.
This comment has been minimized.
Contributor
|
Use /ai review command to trigger AI review workflows for draft PRs |
wangyb-A
marked this pull request as draft
August 28, 2026 19:22
wangyb-A
force-pushed
the
workflow-insight-plugin
branch
from
August 28, 2026 21:00
d90f91f to
bdbd88f
Compare
Contributor
Author
|
/ai review |
This comment has been minimized.
This comment has been minimized.
wangyb-A
force-pushed
the
workflow-insight-plugin
branch
from
August 31, 2026 22:49
bdbd88f to
24e28b8
Compare
Contributor
Author
|
/ai review |
This comment has been minimized.
This comment has been minimized.
added 7 commits
September 9, 2026 20:48
Keep S3 and CloudWatch Logs clients out of default consumer dependency trees, document opt-in dependencies, remove the internal review ledger, and align lifecycle documentation with implementation.
wangyb-A
force-pushed
the
workflow-insight-plugin
branch
from
September 9, 2026 20:48
661ff21 to
7f3bdc4
Compare
wangyb-A
temporarily deployed
to
ai-pr-review-runtime
September 9, 2026 20:56 — with
GitHub Actions
Inactive
Contributor
Codex AI reviewNine confirmed issues remain, mainly around checkpoint lifecycle blocking, record fidelity, truncation, and exporter safeguards. Current tests miss important custom-SerDes, null, suspension, and oversized-record paths. Reviewed commit |
Contributor
Author
|
The new findings are being tracked in #679 Fix the first 3 blocking issues, the last one is tracked in the 679 and will be implemented in the next PR. @ParidelPooya |
ParidelPooya
approved these changes
Sep 9, 2026
2 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
#679
Description
Adds the preview Workflow Insight plugin as a new
insight-pluginMaven module.The plugin emits schema-versioned execution records from the existing Java plugin hooks and matches the JavaScript Workflow Insight contract. It includes:
operationsByNamerecord shapesThe implementation uses the invocation operation snapshots, execution payloads, errors, attempts, and serialized operation results already exposed by the Java plugin API. No core hook changes are required.
Demo/Screenshots
No UI change.
insight-plugin/README.mdcontains a usage example. The draft conformance PR linked above contains Java handlers for all 18 Workflow Insight requirements.Checklist
Testing
Unit Tests
Yes. The module adds 35 tests covering record construction, sampling, emission modes, replay/suspension state, operation filtering, result transforms, aggregation, truncation, exporter requests, failure isolation, and flush behavior.
mvn -q -pl insight-plugin -am test mvn -q -pl insight-plugin -am clean verifyBoth pass.
The full repository reactor also passes:
Integration Tests
Yes.
WorkflowInsightPluginTestexercises the plugin throughLocalDurableTestRunner.The draft Workflow Insight conformance suite was also run against deployed Java 21 Lambda functions in
us-west-2:operationsByNameleg: 18 passed, 0 failedSink-specific assertions were capability-gated and exercised by the complementary leg.
Examples
The module README includes configuration and exporter usage. Customer-facing conformance handlers are in aws/aws-durable-execution-conformance-tests#73 rather than this repository, following the repository's conformance-test ownership rule.