Skip to content

Enable ASYNC Step input snapshots by default - #509

Merged
longquanzheng merged 1 commit into
mainfrom
enable-async-step-input-snapshots
Sep 19, 2026
Merged

longquanzheng merged 1 commit into
mainfrom
enable-async-step-input-snapshots

Conversation

@longquanzheng

Copy link
Copy Markdown
Contributor

Summary

  • Turn blobStore.asyncStepInputSnapshotsEnabled back on by default when Blob Store is enabled, so semantic history keeps successful ASYNC local Step method inputs.
  • Keep disable as an optional cost optimization: omit the flag or set it to true for the default; set false to skip snapshot writes.
  • Update Server docs, Dex Web unavailable copy, and tests so operators can opt out without changing the default.

Test plan

  • make -C server unitTests
  • Dex Web EventDetails vitest
  • make docs-prose-check and cd docs && npm run check
  • Temporal/Cadence web_api integ coverage for default-on and explicit-off snapshots (CI)

skill-dex-developer stays blocked until this change is in a published Dex release. SuperVerse should disable the flag in its own Dex Server configs.

Made with Cursor

Keeping this history is the default Server behavior. Operators can still turn it off to skip the extra Blob Store cost.
@codecov

codecov Bot commented Sep 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...ent/history/async_step_input_snapshot_populator.go 50.00% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@longquanzheng
longquanzheng merged commit b142f19 into main Sep 19, 2026
47 checks passed
@longquanzheng
longquanzheng deleted the enable-async-step-input-snapshots branch September 19, 2026 05:19
@longquanzheng
longquanzheng restored the enable-async-step-input-snapshots branch September 19, 2026 05:43
@longquanzheng
longquanzheng deleted the enable-async-step-input-snapshots branch September 19, 2026 23:12
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