Skip to content

Stop production container restart loop from unhandled promise rejections#326

Merged
DarthGigi merged 3 commits into
prodfrom
dev
May 16, 2026
Merged

Stop production container restart loop from unhandled promise rejections#326
DarthGigi merged 3 commits into
prodfrom
dev

Conversation

@DarthGigi
Copy link
Copy Markdown
Member

No description provided.

github-actions Bot and others added 3 commits May 16, 2026 15:38
- card/+server.ts: switch Promise.all to Promise.allSettled so a first-failure
  doesn't leave the other parallel remote-function promises orphaned. Their
  later rejection was becoming an unhandled rejection under Node 24's strict
  default and crashing the process.
- instrumentation.server.ts: add a process-level unhandledRejection handler
  that logs + reports to Sentry instead of letting Node exit. Safety net for
  any future orphans we haven't found yet.

This ends the ~2-minute Dokploy restart loop visible in production logs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 16, 2026

🦋 Changeset detected

Latest commit: cfcedb2

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

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@DarthGigi DarthGigi merged commit b5741bc into prod May 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant