inngest / inngest-js

The developer platform for easily building reliable workflows with zero infrastructure for TypeScript & JavaScript
https://www.inngest.com/
GNU General Public License v3.0
370 stars 37 forks source link

Fix rare checkpoint hang #622

Closed jpwilliams closed 1 day ago

jpwilliams commented 2 weeks ago

Summary

Fixes a theoretical, rare issue whereby multiple checkpoints can be reported before the next one is awaited, resulting in an entire checkpoint being missed.

At best, this can cause the core loop to hang, and at worst make strange decisions.

[!NOTE] The current checkpoints that are supported mean that we never see this issue. However, this PR guards us against altering the checkpoints in the future and not accounting for this quirk.

For example:

Checklist

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: 8a732814dd0df7f138f76a14f5bf099e2bbdeede

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ------- | ----- | | inngest | Patch |

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