Migrate all Airbrake usage to Appsignal (#10801)

## Summary of the problem
<!-- Why these changes are being made? What problem does it solve? Link
any related issues to provide more details. -->
There's only room for one error reporter in this codebase...
and Appsignal gives us better reports!

## Describe your changes
<!-- Explain your thought process to the solution and provide a quick
summary of the changes. -->
Changed all `Airbrake.notify` and `notify_airbrake` statements to use
the Rails error reporter, which is handled by Appsignal. Also changed
the frontend error reporting to use Appsignal, and updated docs.

Before deploying, make sure to set the `APPSIGNAL_API_KEY` environment
variable to the appropriate key from Appsignal -
https://docs.appsignal.com/front-end/installation.html#creating-a-push-api-key

---------

Co-authored-by: David Cornu <[email protected]>
Co-authored-by: Luke Oldenburg <[email protected]>
Co-authored-by: Gary Tou <[email protected]>

Author: 4 people

Gemfile

@@ -113,7 +113,6 @@ gem "diffy" # rendering diffs (comments)
 gem "webauthn", "~> 3.2"
 
 gem "ahoy_matey" # analytics
-gem "airbrake" # exception tracking
 gem "blazer" # business intelligence tool/dashboard
 
 gem "geo_pattern" # create procedurally generated patterns for Cards

Gemfile.lock

@@ -127,10 +127,6 @@ GEM
       activesupport (>= 7)
       device_detector (>= 1)
       safely_block (>= 0.4)
-    airbrake (13.0.5)
-      airbrake-ruby (~> 6.0)
-    airbrake-ruby (6.2.2)
-      rbtree3 (~> 0.6)
     airrecord (1.0.12)
       faraday (>= 1.0, < 3.0)
       faraday-net_http_persistent
@@ -657,7 +653,6 @@ GEM
       ffi (~> 1.0)
     rbs (3.9.2)
       logger
-    rbtree3 (0.7.1)
     rdoc (6.14.0)
       erb
       psych (>= 4.0.0)
@@ -881,7 +876,6 @@ DEPENDENCIES
   actual_db_schema
   ahoy_email (~> 2.4)
   ahoy_matey
-  airbrake
   airrecord (~> 1.0)
   annotaterb
   api-pagination

app/controllers/api/v4/stripe_cards_controller.rb

@@ -68,7 +68,7 @@ def create
         render :show
 
       rescue => e
-        notify_airbrake(e)
+        Rails.error.report e
         render json: { error: "internal_server_error" }, status: :internal_server_error
       end
 

app/controllers/errors_controller.rb

@@ -22,7 +22,7 @@ def timeout
 
   def error
     @code = params[:code]
-    Airbrake.notify("/#{@code} rendered.")
+    Rails.error.unexpected "/#{@code} rendered."
     render status: params[:code], layout: "application"
   end
 

app/controllers/stripe_controller.rb

@@ -169,7 +169,7 @@ def handle_charge_dispute_funds_withdrawn(event)
 
       donation = Donation.find_by(stripe_payment_intent_id: dispute[:payment_intent])
 
-      return notify_airbrake("Received charge dispute on nonexistent donation") if donation.nil?
+      return Rails.error.unexpected("Received charge dispute on nonexistent donation") if donation.nil?
 
       # Let's un-front the transaction.
       donation.canonical_pending_transactions.update_all(fronted: false)
@@ -178,7 +178,7 @@ def handle_charge_dispute_funds_withdrawn(event)
 
       invoice = Invoice.find_by(stripe_charge_id: dispute[:charge])
 
-      return notify_airbrake("Received charge dispute on nonexistent invoice") if invoice.nil?
+      return Rails.error.unexpected("Received charge dispute on nonexistent invoice") if invoice.nil?
 
       invoice.canonical_pending_transactions.update_all(fronted: false)
     end
@@ -306,14 +306,14 @@ def handle_issuing_dispute_funds_reinstated(event)
         }
       )
     elsif dispute["status"] != "won"
-      Airbrake.notify("Dispute with funds reinstated but without a win: #{dispute["id"]}")
+      Rails.error.unexpected "Dispute with funds reinstated but without a win: #{dispute["id"]}"
     elsif dispute["currency"] != "usd"
-      Airbrake.notify("Dispute with funds reinstated but non-USD currency. Must be manually handled.")
+      Rails.error.unexpected "Dispute with funds reinstated but non-USD currency. Must be manually handled."
     end
   end
 
   def handle_refund_failed(event)
-    Airbrake.notify("Refund failed on Stripe: #{event}.")
+    Rails.error.unexpected "Refund failed on Stripe: #{event}."
   end
 
 end

app/controllers/webauthn_credentials_controller.rb

@@ -43,7 +43,7 @@ def create
 
       redirect_back fallback_location: edit_user_path(user), flash: { success: "Registered security key!" }
     rescue WebAuthn::Error => e
-      Airbrake.notify(e)
+      Rails.error.report e
       redirect_back fallback_location: edit_user_path(user), flash: { error: "Something went wrong registering a security key." }
     end
   end

app/helpers/application_helper.rb

@@ -226,7 +226,7 @@ def help_phone
 
   def format_date(date)
     if date.nil?
-      Airbrake.notify("Hey! date is nil here")
+      Rails.error.unexpected "Hey! date is nil here"
       return nil
     end
 

app/javascript/airbrake.js

@@ -0,0 +1,7 @@
+/* global APPSIGNAL_FRONTEND */
+
+import Appsignal from '@appsignal/javascript'
+
+export const appsignal = new Appsignal({
+  key: APPSIGNAL_FRONTEND,
+})

app/javascript/appsignal.js

@@ -1,6 +1,6 @@
 import { Controller } from '@hotwired/stimulus'
 import submitForm from '../common/submitForm'
-import airbrake from '../airbrake'
+import { appsignal } from '../appsignal'
 
 let dropzone
 
@@ -17,7 +17,6 @@ function extractId(dataTransfer) {
     receiptId = imgTag.getAttribute('data-receipt-id')
   } catch (err) {
     console.error(err)
-    // airbrake?.notify(err)
   }
 
   if (!receiptId) {
@@ -33,7 +32,7 @@ function extractId(dataTransfer) {
       receiptId = imageElement.getAttribute('data-receipt-id')
     } catch (err) {
       console.error(err)
-      airbrake?.notify(err)
+      appsignal.sendError(err)
     }
   }