I'm just realizing now, it looks like we actually initialize `success` to `False` above (line 261), so this assert shouldn't ever actually get tripped. Do we want to initialize it to `None` instead?