Skip to content

Add payload validation error factory - #1766

Merged
bergundy merged 1 commit into
temporalio:mainfrom
bergundy:payload-validation-error-factory
Aug 20, 2026
Merged

Add payload validation error factory#1766
bergundy merged 1 commit into
temporalio:mainfrom
bergundy:payload-validation-error-factory

Conversation

@bergundy

Copy link
Copy Markdown
Member

Summary

  • export temporalio.converter.create_payload_validation_error
  • create a non-retryable PayloadValidationError ApplicationError with structured details
  • replace positive Nexus test constructions with the public factory while retaining retryable compatibility coverage

Testing

  • poe format
  • focused converter factory and Nexus mapper tests
  • targeted Ruff, Pyright, mypy, basedpyright, and pydocstyle checks

Full optional-suite collection requires AWS and Google ADK dependencies that are not installed in this checkout.

@bergundy
bergundy requested a review from a team as a code owner August 19, 2026 20:52

@VegetarianOrc VegetarianOrc left a comment

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.

Looks good to me! Just a small question about a test.

Comment thread tests/test_converter.py
@bergundy
bergundy merged commit cda0e72 into temporalio:main Aug 20, 2026
29 of 33 checks passed
@bergundy
bergundy deleted the payload-validation-error-factory branch August 20, 2026 20:40
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