Why do you want to contribute?
Hi Trigger.dev maintainers, I would like to contribute a focused fix for issue #3726.
I prepared a branch that updates ExponentialBackoff.execute() so maxElapsed also accounts for callback execution time, including attemptTimeoutMs paths. My original PR was auto-closed because I am not vouched yet: #3760.
Branch: https://github.com/nikhil-shukl/trigger.dev/tree/codex/fix-backoff-max-elapsed
Closed PR: #3760
Issue: #3726
I also addressed the automated review finding about AttemptTimeout bypassing the elapsed check and added focused regression coverage.
Prior contributions or relevant experience
Relevant work for this request:
GitHub: https://github.com/nikhil-shukl
Why do you want to contribute?
Hi Trigger.dev maintainers, I would like to contribute a focused fix for issue #3726.
I prepared a branch that updates
ExponentialBackoff.execute()somaxElapsedalso accounts for callback execution time, includingattemptTimeoutMspaths. My original PR was auto-closed because I am not vouched yet: #3760.Branch: https://github.com/nikhil-shukl/trigger.dev/tree/codex/fix-backoff-max-elapsed
Closed PR: #3760
Issue: #3726
I also addressed the automated review finding about
AttemptTimeoutbypassing the elapsed check and added focused regression coverage.Prior contributions or relevant experience
Relevant work for this request:
ExponentialBackoff.execute()maxElapsed behavior.vitest run packages/core/test/backoff.test.ts --config vitest.temp.config.mjs(2 tests passed).GitHub: https://github.com/nikhil-shukl