Skip to content

Update visitor function to visit temporal system nexus payloads#297

Open
spkane31 wants to merge 1 commit into
mainfrom
spk/sne-visitor-updates
Open

Update visitor function to visit temporal system nexus payloads#297
spkane31 wants to merge 1 commit into
mainfrom
spk/sne-visitor-updates

Conversation

@spkane31

Copy link
Copy Markdown
Contributor

What changed?

  • Added generic support for marked system payload envelopes in proxy.VisitPayloads.
  • Introduced the reserved metadata key __temporal_system_payload.
  • When the marker is "true", the visitor decodes the binary protobuf envelope using messageType, visits nested user payloads, and reserializes it without passing the outer envelope to the caller’s codec visitor.
  • Added test coverage.

Why?
Handling system nexus messages based on explicit payload metadata makes traversal generic and avoids service, endpoint, operation, or event-correlation registries.

How did you test it?

  • Added unit tests confirming nested Payload and Payloads mutations survive envelope re-serialization.
  • Added tests for missing/unknown messageType and unsupported encoding.
  • Added a WorkflowService gRPC test covering outbound workflow-task completion and inbound workflow-task polling, verifying only nested payloads reach codec visitors.

@spkane31
spkane31 requested review from a team as code owners July 20, 2026 17:26
@spkane31
spkane31 requested a review from tconley1428 July 20, 2026 19:13
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.

3 participants