Skip to content

fix: evaluate rate limit CEL expressions with parent outputs - #4544

Open
jatinder14 wants to merge 1 commit into
hatchet-dev:mainfrom
jatinder14:fix/rate-limit-parents-cel
Open

fix: evaluate rate limit CEL expressions with parent outputs#4544
jatinder14 wants to merge 1 commit into
hatchet-dev:mainfrom
jatinder14:fix/rate-limit-parents-cel

Conversation

@jatinder14

Copy link
Copy Markdown

Summary

  • compile step expressions with the step-run CEL environment
  • bind parent task outputs as the documented parents map during task creation and replay
  • cover parent output access in step-run CEL evaluation

Test plan

  • go test ./internal/cel

Fixes #4380

Made with Cursor

Use the step-run CEL environment and expose the serialized parents map so dynamic rate limits can reference upstream task output.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

@jatinder-mahajan-certifyos is attempting to deploy a commit to the Hatchet Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions Bot added the engine Related to the core Hatchet engine label Jul 27, 2026
@jatinder14
jatinder14 force-pushed the fix/rate-limit-parents-cel branch from 9c6baff to d00a46f Compare July 28, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine Related to the core Hatchet engine

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] rate limit CEL expressions referencing parents pass registration but fail on every run

1 participant