Skip to content

Process special Temporal builtin prefixes#630

Merged
roxblnfk merged 13 commits into
masterfrom
builtin-prefixes
Jul 2, 2025
Merged

Process special Temporal builtin prefixes#630
roxblnfk merged 13 commits into
masterfrom
builtin-prefixes

Conversation

@roxblnfk

@roxblnfk roxblnfk commented Jun 27, 2025

Copy link
Copy Markdown
Collaborator

What was changed

Add EntityNameValidator for names validation;
Validate Workflow, Activity, Signal, Query, Update, and TaskQueue names.

Why?

To prevent registering invalid entities in runtime.

Checklist

  1. Closes [Feature Request] Special behavior for Temporal built-in prefixes #552
  2. How was this tested:
    • added tests
  3. Any docs updates needed?

Validate Workflow, Activity, Signal, Query, Update, and TaskQueue names.
@vercel

vercel Bot commented Jun 27, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
php ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 2, 2025 7:29am

@roxblnfk

Copy link
Copy Markdown
Collaborator Author

@Sushisource should the SDK process and return something useful for these queries: __stack_trace and __enhanced_stack_trace?

@roxblnfk
roxblnfk marked this pull request as ready for review June 30, 2025 09:09
@roxblnfk
roxblnfk requested a review from wolfy-j as a code owner June 30, 2025 09:09
@roxblnfk
roxblnfk requested a review from Sushisource June 30, 2025 09:09
Comment thread src/Internal/Transport/Router/InvokeQuery.php Outdated
@roxblnfk
roxblnfk merged commit fadb187 into master Jul 2, 2025
111 of 115 checks passed
@roxblnfk
roxblnfk deleted the builtin-prefixes branch July 2, 2025 07:49
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.

[Feature Request] Special behavior for Temporal built-in prefixes

3 participants