I like inline, anonymous functions within the promise chain because it makes the chain function more like a language control construct: you can follow the logic by reading it the same way you would read an if-else statement. If you feel strongly about it then we can keep it; it's just a style nit from me (I'm more concerned with getting the logic correct). I find it harder to read as is and I find the named var definitions to be unnecessary boilerplate.