Skip to content
This repository was archived by the owner on Sep 11, 2025. It is now read-only.

Improve error logging and debugging#394

Merged
mattjohnsonpint merged 3 commits intomainfrom
mjp/hyp-2278-adjust-error-reporting-in-the-case-of-a-runtime-failure
Sep 26, 2024
Merged

Improve error logging and debugging#394
mattjohnsonpint merged 3 commits intomainfrom
mjp/hyp-2278-adjust-error-reporting-in-the-case-of-a-runtime-failure

Conversation

@mattjohnsonpint
Copy link
Contributor

When an internal runtime error occurs, we need the actual error reported without being user-visible, so that we get a Sentry alert. This makes sense, because the error wasn't in the user's code. However, they still need some indication that an error occurred, or the function run logs are incomplete.

I also added a small change to how console.debug errors are written when HYPERMODE_DEBUG=true, so that we can better troubleshoot errors caused by user code.

@mattjohnsonpint mattjohnsonpint requested a review from a team September 26, 2024 22:53
@linear
Copy link

linear bot commented Sep 26, 2024

HYP-2278 Adjust error reporting in the case of a Runtime failure

In HYP-2261 I made some changes to error reporting, but I went slightly too far because if a Runtime internal error occurs, to the user it looks like their function never ended when they read the logs.

@mattjohnsonpint mattjohnsonpint enabled auto-merge (squash) September 26, 2024 22:53
@mattjohnsonpint mattjohnsonpint merged commit 9751390 into main Sep 26, 2024
@mattjohnsonpint mattjohnsonpint deleted the mjp/hyp-2278-adjust-error-reporting-in-the-case-of-a-runtime-failure branch September 26, 2024 23:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants