Skip to content

Upgrade Temporal SDK flow header handling - #504

Merged
longquanzheng merged 2 commits into
mainfrom
codex/omit-redundant-flow-type-header
Sep 18, 2026
Merged

longquanzheng merged 2 commits into
mainfrom
codex/omit-redundant-flow-type-header

Conversation

@longquanzheng

@longquanzheng longquanzheng commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • upgrade every Dex Go workspace module to Temporal SDK fork v1.47.1-superdurable.3
  • omit inherited Dex flow type headers for Step and RPC activities that register a FlowTypeProvider
  • retain inherited headers for SubFlow and system activities without that provider

Upstream SDK change: superdurable/temporal-sdk-go#2

Published release: https://github.com/superdurable/temporal-sdk-go/releases/tag/v1.47.1-superdurable.3

Why

Step and RPC activity inputs already contain the Flow type. Persisting the same value in __temporal_sdk_dex_flow_type duplicates data in Temporal history. The new SDK treats a registered FlowTypeProvider as authoritative while preserving legacy header consumption and inheritance for activities without that provider.

Validation

  • make -C server unitTests
  • make -C server temporalIntegTests integrationCoverageArgs="-run=^TestTemporalMetrics -temporalHostPort=127.0.0.1:7234" against an isolated Temporal dev server
  • GOWORK=off go mod tidy -diff for server, server/cmd/server, cli, and protos/codec-server
  • make -C sdk-go docsCheck
  • GOWORK=off go test ./... in protos/codec-server
  • upstream SDK check, unit, replay, and focused integration tests with default cache and WORKFLOW_CACHE_SIZE=0
  • full GitHub CI matrix

@codecov

codecov Bot commented Sep 18, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@longquanzheng
longquanzheng marked this pull request as ready for review September 18, 2026 17:41
@longquanzheng
longquanzheng merged commit 33d79fb into main Sep 18, 2026
46 checks passed
@longquanzheng
longquanzheng deleted the codex/omit-redundant-flow-type-header branch September 18, 2026 17:43
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.

1 participant