Open
Conversation
There was a problem hiding this comment.
Pull request overview
Adds retry/failure handling for queue tasks by extending the task schema and updating the consumer to persist execution outcomes and retry metadata.
Changes:
- Extend
Tasktyping to include retry metadata and afailedstatus. - Update the consumer to record
done/failedoutcomes and re-queue tasks while under the retry limit. - Add a migration to introduce retry-related columns and extend the Postgres
t_statusenum.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| src/types.ts | Updates Task shape to include retry fields, timestamps, and failed status. |
| src/consumer.ts | Implements try/catch retry flow and persists state transitions to the DB. |
| migrations/20260128113327_add_retry_logic.ts | Adds retry columns and extends the t_status enum with failed. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Initial plan * Rename max_retries to max_attempts for clarity Co-authored-by: emilioSp <22614105+emilioSp@users.noreply.github.com> * Clarify log messages and fix migration down function Co-authored-by: emilioSp <22614105+emilioSp@users.noreply.github.com> * Improve migration down function comment Co-authored-by: emilioSp <22614105+emilioSp@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: emilioSp <22614105+emilioSp@users.noreply.github.com> Co-authored-by: Emilio Spatola <emiliosp.me@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.