feat(app): Log tracebacks from failed RPC calls

Closes #1841

api/opentrons/server/rpc.py

@@ -276,8 +276,11 @@ async def make_call(self, func, token):
                 line_no = 'unknown'
             finally:
                 response['$']['status'] = 'error'
-                call_result = '{0} [line {1}]: {2}'.format(
-                    e.__class__.__name__, line_no, str(e))
+                call_result = {
+                    'message': '{0} [line {1}]: {2}'.format(
+                        e.__class__.__name__, line_no, str(e)),
+                    'traceback': traceback.format_exc()
+                }
         finally:
             response['data'] = call_result
         return response

app/src/robot/api-client/index.js

@@ -1,16 +1,33 @@
 // robot api client redux middleware
 // wraps the api client worker to handle API side effects in a different thread
 
+import createLogger from '../../logger'
 import Worker from './worker'
 
+const log = createLogger(__filename)
+
 export default function apiClientMiddleware () {
   const worker = new Worker()
 
   return (store) => {
     const {getState, dispatch} = store
 
     worker.onmessage = function handleWorkerMessage (event) {
-      dispatch(event.data)
+      const action = event.data
+
+      // log error actions
+      if (action && action.payload) {
+        const error = action.error === true
+          ? action.payload
+          : action.payload.error
+
+        if (error) {
+          log.warn('Error response from robot', {action})
+          if (error.traceback) log.warn(error.traceback)
+        }
+      }
+
+      dispatch(action)
     }
 
     // initialize worker

app/src/rpc/client.js

@@ -65,12 +65,12 @@ class RpcContext extends EventEmitter {
     return new Promise((resolve, reject) => {
       let timeout
 
-      const handleError = (reason) => {
+      const handleError = (reason, traceback) => {
         cleanup()
-        reject(new RemoteError(reason, name, args))
+        reject(new RemoteError(reason, name, args, traceback))
       }
 
-      const handleFailure = (result) => handleError(result)
+      const handleFailure = (res) => handleError(res.message, res.traceback)
       const handleNack = (reason) => handleError(`Received NACK with ${reason}`)
 
       const handleAck = () => {

app/src/rpc/remote-error.js

@@ -1,9 +1,10 @@
 // remote call error object
 export default class RemoteError extends Error {
-  constructor (message, methodName, args) {
+  constructor (message, methodName, args, traceback) {
     super(message)
     this.name = this.constructor.name
     this.methodName = methodName
     this.args = args
+    this.traceback = traceback
   }
 }