Nit: unless you have to reference a function from inside itself (f.e. to remove an event listener from inside the listener), there's no longer any good reason to name a function, as the JS debugger has gotten really good at identifying anonymous functions. So this callback need not have the name *onSuccess*.

Also, nit: you shouldn't need to declare the *isAdbRunningPromise* const here; just call *then* directly against the function call that returns a promise, i.e.:

```
      require("adb/adb-running-checker").check().then(function(isAdbRunning) {
```