Conversation
🦋 Changeset detectedLatest commit: 422d6f8 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
35ba654 to
07d5251
Compare
07d5251 to
8e94ee3
Compare
fdcf354 to
80e8741
Compare
10f2287 to
422d6f8
Compare
There was a problem hiding this comment.
LGTM
All prior comments remain addressed. New changes since last review (abort handle, DeferAbort dedup moved to drain(), async validator support) are clean. The riskiest hunk — prepareDeferOp catching rejections and resolving null to drop ops, with drain() filtering nulls — handles the error path correctly and prevents unhandled rejections from propagating. No new issues found.
What this PR does
Adds transformDeferInput middleware hook and async schema validator support for deferred functions. Refactors LazyOps to store promises (async prep settles after sync ID reservation), moves experiment-key injection after validation/middleware, adds DeferAbort dedup at drain time, and returns an abort() handle from defer(). Integration tests cover all new paths.
Tag @mendral-app with feedback or questions. View session
Summary
Add
transformDeferInputand support for async defer validationsChecklist
Related
Old, PR: #1559
Revert: #1563
Follow Up: #1560