Skip to content

Handle application script errors consistently. - #75

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
handle-script-errors
Aug 16, 2026
Merged

Handle application script errors consistently.#75
samuel-williams-shopify merged 1 commit into
mainfrom
handle-script-errors

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • define a shared application exception boundary for exception handling middleware
  • handle and report ScriptError failures such as SyntaxError consistently
  • allow process-level exceptions such as Interrupt to propagate
  • cover both handler and mailer behavior

Closes #25.

Testing

  • bundle exec sus test/utopia/exceptions/handler.rb
  • bundle exec sus test/utopia/exceptions/mailer.rb
  • bundle exec rubocop lib/utopia/exceptions/application_errors.rb lib/utopia/exceptions/handler.rb lib/utopia/exceptions/mailer.rb test/utopia/exceptions/.handler/controller.rb test/utopia/exceptions/handler.rb test/utopia/exceptions/mailer.rb
  • bundle exec bake decode:index:coverage lib

Assisted-By: devx/cbdeae41-9308-4071-ad53-58cd92db2946
@samuel-williams-shopify
samuel-williams-shopify merged commit 8d0a604 into main Aug 16, 2026
34 of 40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the handle-script-errors branch August 16, 2026 22:24
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.

Exception handlers

1 participant