Promise chains are usually totally fine and readable, it's when callbacks are passed around without promises or bad abstractions occur that async code is problematic.

More often than not I would not touch code written with promises because it doesn't matter anyways since code using it can still use async/await + sometimes it makes some patterns easier to deal with (f.e. using a dependency written with callbacks and calling resolve from within the callback function)